From e41c1702a3688fb54a437e7934f471cb62ac138d Mon Sep 17 00:00:00 2001 From: Mathis Date: Sat, 11 Jul 2026 23:25:25 +0200 Subject: [PATCH] =?UTF-8?q?Mise=20=C3=A0=20jour=20:=20Migration=20du=20sui?= =?UTF-8?q?vi=20des=20bugs=20et=20fonctionnalit=C3=A9=20via=20MP=20vers=20?= =?UTF-8?q?ForgeJo?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- commandes/bug_report.py | 110 +++++++++++++++++++++++++++++----------- 1 file changed, 81 insertions(+), 29 deletions(-) diff --git a/commandes/bug_report.py b/commandes/bug_report.py index cd359a2..bb389bb 100644 --- a/commandes/bug_report.py +++ b/commandes/bug_report.py @@ -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: