metyatech/skill-pr-review-workflow

PR-Review-Feedback-Workflow

Review-Feedback adressieren

• Nachdem das PR-Review-Feedback adressiert wurde, MUSS der Agent die entsprechenden Review-Threads auflösen, bevor er abschließt. Fehlt dem Agent die Berechtigung, MUSS er dies explizit angeben.
• Bevor der Agent ein Review erneut anfordert, MUSS er die relevanten Testsuiten ausführen und die Ergebnisse (Befehle und Ergebnisse) im PR-Kommentar oder in der PR-Beschreibung zusammenfassen.
• Bei Feedback, das durch einen agentenverursachten Fehler oder eine wiederverwendbare Denklücke entstanden ist, MUSS der Agent bewerten, ob die Vermeidung von Wiederholungen in Tests, Checks, Skills oder Rules erfolgen sollte. Falls eine Aktualisierung einer Regel oder eines Skills gerechtfertigt ist, MUSS der Agent diese Aktualisierung als Teil desselben Änderungssatzes durchführen. Ist dies nicht gerechtfertigt, MUSS der Agent kurz begründen, warum die früheste wiederverwendbare Hürde woanders liegt.

Reviews erneut anfordern

• Nach dem Pushen von Korrekturen MUSS der Agent das Review nur von den Prüfern erneut anfordern, die in der aktuellen Runde Feedback gegeben haben.
• Der Agent DARF KEIN Review von Prüfern (einschließlich KI-Prüfern) erneut anfordern, die kein adressiertes Feedback gegeben haben oder bereits angegeben haben, dass keine umsetzbaren Probleme vorliegen.
• Wenn kein zutreffender Prüfer mehr übrig ist, MUSS der Agent fragen, wer als nächstes prüfen soll.

KI-Review-Bot erneute Prüfung

Wenn Codex- oder Copilot-Review-Bots für das Repository konfiguriert sind, MUSS der Agent eine erneute Prüfung nur für den/die Bot(s) auslösen, der/die adressiertes Feedback gepostet hat/haben.

• Für Codex erneute Prüfung: Kommentieren Sie @codex review auf dem PR.
• Für Copilot erneute Prüfung: Verwenden Sie die GitHub-API, um den Bot-Prüfer copilot-pull-request-reviewer[bot] zu entfernen und erneut anzufordern. Der Agent DARF SICH NICHT auf eine UI-basierte Prüferzuweisung verlassen.
  • Entfernen: DELETE /repos/{owner}/{repo}/pulls/{pr}/requested_reviewers mit Body {"reviewers":["copilot-pull-request-reviewer[bot]"]}.
  • Hinzufügen: POST /repos/{owner}/{repo}/pulls/{pr}/requested_reviewers mit Body {"reviewers":["copilot-pull-request-reviewer[bot]"]}.

Verwaltung von GitHub-Benachrichtigungen

• Der Agent MUSS DELETE /notifications/threads/{id} (HTTP 204) verwenden, um Benachrichtigungen als erledigt zu markieren (entfernt die Benachrichtigung aus dem Posteingang und verschiebt sie auf den Tab „Erledigt“).
• Der Agent DARF NICHT PATCH /notifications/threads/{id} verwenden, da dies als gelesen markiert, aber die Benachrichtigung im Posteingang belässt.
• Nach der Verarbeitung von Benachrichtigungen MUSS der Agent alle verbleibenden, gelesenen, aber nicht erledigten Benachrichtigungen mit derselben DELETE-API in großen Mengen löschen.

Bereinigung nach dem Merge

• Nach Abschluss eines PRs MUSS der Agent ihn mergen, den Zielbranch synchronisieren und den PR-Branch sowohl lokal als auch remote löschen.