review-loop

レビューループ

修正する価値のある項目が見つからないラウンドに至るまで、レビュー、修正、再レビューを繰り返します。修正も新しいコードであり、それ自体にレビューが必要です。

ルール

1. 毎ラウンド、新しいサブエージェントでレビューします。 <target> --sub を指定して review-code を実行します（デフォルトの対象は diff です）。review-code が見つからない場合は、サブエージェントを起動し、正確性、セキュリティ、リソース、パフォーマンスに関する不具合をレビューさせ、それぞれの指摘について修正する価値があるかを Yes／Judgment call／No で評価させます。自分で行った修正を、その場で自分がレビューしてはいけません。

2. --second-opinion を渡すのは第1ラウンドのみです。 第1ラウンドでは変更全体をレビューします。以降のラウンドでは修正差分をレビューするため、別のエージェントを追加してもコストが増える一方で、確認範囲はほとんど広がりません。Final Call は修正前のコードを読んでいますが、ルール1には抵触しません。

3. 第2ラウンド以降は、対象がパスを含む場合も含め、すべて diff を使用します。 GitHub にはプッシュされていない修正が表示されず、コードベース全体の再レビューはコンテキストを無駄にし、パスを指定したレビューでは関連ファイルの修正を見落とす可能性があります。pr <number> の場合は、先にその PR をチェックアウトします。

4. Yes はすべて修正し、Judgment call は Trivial または Small の場合のみ修正します。 No はスキップします。ラウンドに Final Call がある場合は、代わりにその判断に従います。Fix は修正、Optional は Trivial または Small の場合のみ修正、Skip はスキップします。スキップした指摘が後のラウンドで再び挙がっても、再度修正したり、反論したりしてはいけません。

5. 各ラウンドの修正後に、リポジトリのチェックを実行します（make build／make test／make lint、それらがない場合はネイティブのコマンド）。不具合があれば、次のレビューに進む前に修正します。

6. 新たに修正する項目がないラウンドで終了します。 最大5ラウンドまでとし、第5ラウンドは実行可能な指摘が見つかってもレビューのみです。そのレビュー後は編集せず、未解決の指摘をユーザーに引き継ぎます。修正済みの指摘がいずれかのラウンドで再び挙がった場合は、修正を中止して未解決の指摘を報告します。

7. 特に指示がない限り、修正ごとにコミットします。 --commit each（デフォルト）は、修正した指摘ごとに1つのコミットを作成します。--commit end は、ループ終了後に1つのコミットを作成します。プッシュは決して行いません。デフォルトブランチ上にいる場合は、最初のコミットの前にブランチを作成します。そうしないと、diff の基準が HEAD となり、コミット済みの修正が次のレビューから除外されます。

8. 最後に次の内容を報告します。 実施したラウンド数と終了理由、修正内容（path:line — 変更内容）、スキップした指摘と各理由、未解決の指摘、チェック結果。