Mise à jour : Migration du suivi des bugs et fonctionnalité via MP vers ForgeJo
This commit is contained in:
parent
9b4fbabbc5
commit
e41c1702a3
1 changed files with 81 additions and 29 deletions
|
|
@ -17,6 +17,45 @@ TRACKING_FILE = os.path.join(BASE_DIR, "commandes", "gitlab_issues.json")
|
|||
LEGACY_REPORTS_FILE = os.path.join(BASE_DIR, "data", "gitlab_reports.json")
|
||||
|
||||
|
||||
def classify_webhook_event(payload: Dict[str, Any]) -> Optional[str]:
|
||||
"""Détermine si un webhook correspond à une issue ou à un commentaire."""
|
||||
object_kind = payload.get("object_kind")
|
||||
if object_kind in {"issue", "note"}:
|
||||
return object_kind
|
||||
|
||||
action = str(payload.get("action", "")).lower()
|
||||
if action in {"comment_created", "comment_updated", "commented"} or "comment" in payload:
|
||||
return "note"
|
||||
|
||||
if payload.get("issue") or payload.get("object_attributes") or action in {"open", "reopen", "close", "closed", "updated"}:
|
||||
return "issue"
|
||||
|
||||
return None
|
||||
|
||||
|
||||
def extract_issue_iid(payload: Dict[str, Any]) -> Optional[str]:
|
||||
"""Extrait l’identifiant d’issue depuis plusieurs formats Forgejo/GitLab."""
|
||||
for container in (payload.get("object_attributes", {}), payload.get("issue", {}), payload.get("comment", {})):
|
||||
if not isinstance(container, dict):
|
||||
continue
|
||||
for key in ("iid", "number", "id"):
|
||||
value = container.get(key)
|
||||
if value not in (None, ""):
|
||||
return str(value)
|
||||
return None
|
||||
|
||||
|
||||
def extract_issue_identifier(issue_payload: Optional[Dict[str, Any]]) -> Optional[str]:
|
||||
"""Récupère un identifiant stable à stocker dans le tracking."""
|
||||
if not isinstance(issue_payload, dict):
|
||||
return None
|
||||
for key in ("iid", "number", "id"):
|
||||
value = issue_payload.get(key)
|
||||
if value not in (None, ""):
|
||||
return str(value)
|
||||
return None
|
||||
|
||||
|
||||
def load_tracking():
|
||||
"""Charge le tracking depuis gitlab_issues.json (avec migration automatique)"""
|
||||
tracking = {}
|
||||
|
|
@ -105,8 +144,12 @@ class BugReportModal(disnake.ui.Modal):
|
|||
)
|
||||
|
||||
if result:
|
||||
issue_iid = str(result.get("iid"))
|
||||
# ⚡ NOUVEAU: Utilise le tracking unifié
|
||||
issue_iid = extract_issue_identifier(result)
|
||||
if not issue_iid:
|
||||
kuby_logger.warning("Impossible de déterminer l’ID de l’issue créée depuis le retour Forgejo/GitLab")
|
||||
await interaction.edit_original_response(content="✅ Votre rapport a été envoyé, mais l’identifiant de l’issue n’a pas pu être récupéré automatiquement.")
|
||||
return
|
||||
|
||||
self.cog.issue_data[issue_iid] = {
|
||||
"requester_id": interaction.user.id,
|
||||
"last_note_id": 0,
|
||||
|
|
@ -117,7 +160,7 @@ class BugReportModal(disnake.ui.Modal):
|
|||
content=(
|
||||
"✅ Votre bug a été signalé avec succès ! "
|
||||
"Nous avons bien été averti et le bug sera réglé dans la prochaine mise à jour. "
|
||||
f"[Voir l'issue]({result.get('web_url')})"
|
||||
f"[Voir l'issue]({result.get('web_url') or result.get('html_url')})"
|
||||
)
|
||||
)
|
||||
else:
|
||||
|
|
@ -170,8 +213,12 @@ class FeatureSuggestionModal(disnake.ui.Modal):
|
|||
)
|
||||
|
||||
if result:
|
||||
issue_iid = str(result.get("iid"))
|
||||
# ⚡ NOUVEAU: Utilise le tracking unifié
|
||||
issue_iid = extract_issue_identifier(result)
|
||||
if not issue_iid:
|
||||
kuby_logger.warning("Impossible de déterminer l’ID de l’issue créée depuis le retour Forgejo/GitLab")
|
||||
await interaction.edit_original_response(content="✅ Votre suggestion a été envoyée, mais l’identifiant de l’issue n’a pas pu être récupéré automatiquement.")
|
||||
return
|
||||
|
||||
self.cog.issue_data[issue_iid] = {
|
||||
"requester_id": interaction.user.id,
|
||||
"last_note_id": 0,
|
||||
|
|
@ -182,7 +229,7 @@ class FeatureSuggestionModal(disnake.ui.Modal):
|
|||
content=(
|
||||
"✅ Votre suggestion a été envoyée avec succès ! "
|
||||
"Merci de contribuer à l'amélioration du bot. "
|
||||
f"[Voir l'issue]({result.get('web_url')})"
|
||||
f"[Voir l'issue]({result.get('web_url') or result.get('html_url')})"
|
||||
)
|
||||
)
|
||||
else:
|
||||
|
|
@ -266,18 +313,15 @@ class BugReport(commands.Cog):
|
|||
kuby_logger.warning("Bot still not ready after 10s. Proceeding anyway (best effort).")
|
||||
|
||||
payload = await request.json()
|
||||
event_type = payload.get("object_kind")
|
||||
|
||||
event_type = classify_webhook_event(payload)
|
||||
if event_type not in ["issue", "note"]:
|
||||
kuby_logger.info("Webhook ignoré : type d’événement non reconnu")
|
||||
return web.json_response({"status": "ignored"}, status=200)
|
||||
|
||||
# Récupère l'IID de l'issue
|
||||
issue_iid = str(
|
||||
payload.get("object_attributes", {}).get("iid") or
|
||||
payload.get("issue", {}).get("iid")
|
||||
)
|
||||
|
||||
# Récupère l'ID de l'issue
|
||||
issue_iid = extract_issue_iid(payload)
|
||||
if not issue_iid:
|
||||
kuby_logger.warning("Webhook ignoré : aucun identifiant d’issue trouvé dans le payload")
|
||||
return web.json_response({"status": "no issue id"}, status=200)
|
||||
|
||||
# Vérifie si l'issue est trackée
|
||||
|
|
@ -302,10 +346,18 @@ class BugReport(commands.Cog):
|
|||
|
||||
if event_type == "note":
|
||||
# ⚡ NOUVEAU: Notifie les commentaires via le webhook (instantané)
|
||||
note_attr = payload.get("object_attributes", {})
|
||||
is_system = note_attr.get("system", False)
|
||||
|
||||
if not is_system:
|
||||
if "comment" in payload and isinstance(payload.get("comment"), dict):
|
||||
note_body = payload["comment"].get("body") or payload["comment"].get("content") or ""
|
||||
author_name = (
|
||||
payload.get("sender", {}).get("username")
|
||||
or payload.get("sender", {}).get("login")
|
||||
or payload.get("user", {}).get("name")
|
||||
or "Développeur"
|
||||
)
|
||||
is_system = False
|
||||
else:
|
||||
note_attr = payload.get("object_attributes", {})
|
||||
is_system = note_attr.get("system", False)
|
||||
note_body = (
|
||||
note_attr.get("note") or
|
||||
note_attr.get("body") or
|
||||
|
|
@ -314,17 +366,17 @@ class BugReport(commands.Cog):
|
|||
)
|
||||
author_name = payload.get("user", {}).get("name", "Développeur")
|
||||
|
||||
if note_body:
|
||||
try:
|
||||
embed = disnake.Embed(
|
||||
title="💬 Nouveau commentaire sur votre signalement",
|
||||
description=f"Le développeur a répondu à votre rapport **{issue_title}** :\n\n>>> {note_body}\n\n✍️ **Par :** {author_name}",
|
||||
color=disnake.Color.orange()
|
||||
)
|
||||
embed.set_thumbnail(url=self.bot.user.display_avatar.url)
|
||||
await user.send(embed=embed)
|
||||
kuby_logger.info(f"Commentaire GitLab envoyé à {user} pour l'issue #{issue_iid}")
|
||||
except (disnake.Forbidden, disnake.HTTPException) as e:
|
||||
if not is_system and note_body:
|
||||
try:
|
||||
embed = disnake.Embed(
|
||||
title="💬 Nouveau commentaire sur votre signalement",
|
||||
description=f"Le développeur a répondu à votre rapport **{issue_title}** :\n\n>>> {note_body}\n\n✍️ **Par :** {author_name}",
|
||||
color=disnake.Color.orange()
|
||||
)
|
||||
embed.set_thumbnail(url=self.bot.user.display_avatar.url)
|
||||
await user.send(embed=embed)
|
||||
kuby_logger.info(f"Commentaire envoyé à {user} pour l'issue #{issue_iid}")
|
||||
except (disnake.Forbidden, disnake.HTTPException) as e:
|
||||
kuby_logger.warning(f"Impossible d'envoyer le DM à {user}: {e}")
|
||||
if dev and dev.id != user.id:
|
||||
try:
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue