PRレビューフィードバックワークフロー
レビューフィードバックへの対応
- PRレビューのフィードバックに対応した後、エージェントは対応するレビュースレッドを必ず解決してから作業を終了しなければなりません。権限がない場合は、その旨を明示しなければなりません。
- 再レビューを依頼する前に、エージェントは該当する検証スイートを実行し、その結果(コマンドと結果)をPRコメントまたは説明にまとめなければなりません。
- エージェント起因のミスや再利用可能な推論のギャップが原因のフィードバックについては、エージェントは再発防止をテスト、チェック、スキル、ルールのいずれに含めるべきかを評価しなければなりません。ルールやスキルの更新が妥当な場合、エージェントは同じ変更セットの一部として更新を実施しなければなりません。妥当でない場合は、最も早期の再利用可能なゲートが他にある理由を簡潔に述べなければなりません。
再レビューの依頼
- 修正をプッシュした後、エージェントは現在のラウンドで対応したフィードバックを投稿したレビュー担当者にのみ再レビューを依頼しなければなりません。
- エージェントは、対応したフィードバックを投稿していないレビュー担当者や、すでにアクション可能な問題がないと示したレビュー担当者(AIレビュー担当者を含む)に再レビューを依頼してはなりません。
- 該当するレビュー担当者が残っていない場合、エージェントは次に誰がレビューすべきかを質問しなければなりません。
AIレビューボットの再レビュー
リポジトリにCodexまたはCopilotのレビューボットが設定されている場合、エージェントは対応したフィードバックを投稿したボットに対してのみ再レビューをトリガーしなければなりません。
- Codexの再レビュー: PRに
@codex reviewとコメントします。 - Copilotの再レビュー: GitHub APIを使用してボットレビュー担当者
copilot-pull-request-reviewer[bot]を削除し、再依頼します。エージェントはUIベースのレビュー担当者割り当てに依存してはなりません。- 削除:
DELETE /repos/{owner}/{repo}/pulls/{pr}/requested_reviewersボディは{"reviewers":["copilot-pull-request-reviewer[bot]"]} - 追加:
POST /repos/{owner}/{repo}/pulls/{pr}/requested_reviewersボディは{"reviewers":["copilot-pull-request-reviewer[bot]"]}
- 削除:
GitHub通知管理
- エージェントは
DELETE /notifications/threads/{id}(HTTP 204) を使用して、通知を完了としてマークしなければなりません(受信トレイから通知を削除し、Doneタブに移動します)。 - エージェントは
PATCH /notifications/threads/{id}を使用してはなりません。これは既読にするものの、通知を受信トレイに残します。 - 通知を処理した後、エージェントは同じDELETE APIを使用して、既読だが完了していない残りの通知を一括削除しなければなりません。
マージ後のクリーンアップ
- PRを完了した後、エージェントはそれをマージし、ターゲットブランチを同期し、PRブランチをローカルおよびリモートで削除しなければなりません。