metyatech/skill-pr-review-workflow

Workflow de rétroaction de revue de PR

Traitement des retours de revue

• Après avoir traité les retours de revue d'une PR, l'agent DOIT résoudre les fils de discussion de revue correspondants avant de conclure. Si l'agent ne dispose pas des autorisations nécessaires, il DOIT le mentionner explicitement.
• Avant de demander une re-revue, l'agent DOIT exécuter la suite de vérification pertinente et DOIT résumer les résultats (commandes et résultats) dans le commentaire ou la description de la PR.
• Pour les retours causés par une erreur d'origine agent ou une lacune de raisonnement réutilisable, l'agent DOIT évaluer si la prévention de la récurrence doit être placée dans des tests, des vérifications, des compétences ou des règles. Lorsqu'une mise à jour de règle ou de compétence est justifiée, l'agent DOIT effectuer la mise à jour dans le même ensemble de modifications. Lorsqu'elle n'est pas justifiée, l'agent DOIT indiquer brièvement pourquoi la passerelle réutilisable la plus précoce se trouve ailleurs.

Demande de re-revue

• Après avoir poussé des correctifs, l'agent DOIT demander une re-revue uniquement aux relecteurs qui ont posté des retours traités dans le cycle en cours.
• L'agent NE DOIT PAS demander de re-revue aux relecteurs (y compris les relecteurs IA) qui n'ont pas posté de retours traités ou qui ont déjà indiqué qu'il n'y avait aucun problème actionable.
• S'il ne reste aucun relecteur applicable, l'agent DOIT demander qui doit relire ensuite.

Re-revue par un bot d'IA

Lorsque des bots de revue Codex ou Copilot sont configurés pour le dépôt, l'agent DOIT déclencher une re-revue uniquement pour le(s) bot(s) qui a (ont) posté des retours traités.

• Pour une re-revue Codex : commenter @codex review sur la PR.
• Pour une re-revue Copilot : utiliser l'API GitHub pour supprimer et redemander le relecteur bot copilot-pull-request-reviewer[bot]. L'agent NE DOIT PAS s'appuyer sur une affectation de relecteur basée sur l'interface utilisateur.
  • Supprimer : DELETE /repos/{owner}/{repo}/pulls/{pr}/requested_reviewers avec le corps {"reviewers":["copilot-pull-request-reviewer[bot]"]}.
  • Ajouter : POST /repos/{owner}/{repo}/pulls/{pr}/requested_reviewers avec le corps {"reviewers":["copilot-pull-request-reviewer[bot]"]}.

Gestion des notifications GitHub

• L'agent DOIT utiliser DELETE /notifications/threads/{id} (HTTP 204) pour marquer les notifications comme faites (supprime la notification de la boîte de réception et la déplace vers l'onglet Terminé).
• L'agent NE DOIT PAS utiliser PATCH /notifications/threads/{id}, qui marque comme lu mais laisse la notification dans la boîte de réception.
• Après avoir traité les notifications, l'agent DOIT supprimer en masse toutes les notifications restantes lues mais non faites en utilisant la même API DELETE.

Nettoyage post-merge

• Après avoir terminé une PR, l'agent DOIT la fusionner, synchroniser la branche cible et supprimer la branche de la PR localement et à distance.