review

実行時の注意事項

• ワークフローで入力を求めて停止するよう指示されている場合は、ユーザーに直接尋ねてください。
• リポジトリ固有の指示は、AGENTS.md、TODO.md、TODOS.md に記載されている可能性が高いものとして扱ってください。
• ワークフローの意図はそのまま維持しつつ、環境固有の表現は現在のツールセットに合わせて翻訳してください。

PR マージ前レビュー

review ワークフローを実行しています。テストでは検出できない構造上の問題がないか、現在のブランチと main の差分を分析してください。

---

ステップ 1：ブランチの確認

1. git branch --show-current を実行して、現在のブランチを確認してください。
2. main にいる場合は、「レビュー対象はありません — main にいるか、main との差分がありません。」 と出力して終了してください。
3. git fetch origin main --quiet && git diff origin/main --stat を実行して、差分があるか確認してください。差分がない場合は、同じメッセージを出力して終了してください。

---

ステップ 2：チェックリストを読む

references/checklist.md を読んでください。

ファイルを読めない場合は、停止してエラーを報告してください。 チェックリストなしで先に進んではいけません。

---

ステップ 3：差分を取得する

ローカルの main が古いことによる誤検知を避けるため、最新の main を取得してください。

git fetch origin main --quiet

git diff origin/main を実行して、差分全体を取得してください。これには、最新の main に対するコミット済みの変更と未コミットの変更の両方が含まれます。

---

ステップ 4：2 段階レビュー

差分に対してチェックリストを 2 段階で適用してください。

1. 第 1 段階（重大）： SQL とデータの安全性、LLM 出力の信頼境界
2. 第 2 段階（参考情報）： 条件付き副作用、マジックナンバーと文字列の結合、デッドコードと一貫性、LLM プロンプトの問題、テストの不足、ビュー／フロントエンド

チェックリストに記載された出力形式に従ってください。抑制ルールを尊重し、「フラグを立てない」セクションに記載された項目はフラグを立てないでください。

---

ステップ 5：指摘事項を出力する

重大なものと参考情報の両方を含め、すべての指摘事項を必ず出力してください。 ユーザーにはすべての問題を提示する必要があります。

• 重大な問題が見つかった場合：すべての指摘事項を出力した後、重大な問題ごとに、問題、推奨する修正、選択肢（A：今すぐ修正、B：了承、C：誤検知 — 対応しない）を含むメッセージを別々にして、ユーザーに直接尋ねてください。 すべての重大な問題について回答を得た後、各問題に対してユーザーが選んだ内容をまとめて出力してください。いずれかの問題でユーザーが A（修正）を選んだ場合は、推奨する修正を適用してください。B／C のみが選択された場合は、対応不要です。
• 重大ではない問題だけが見つかった場合：指摘事項を出力してください。それ以上の対応は不要です。
• 問題が見つからなかった場合：Pre-Landing Review: No issues found. を出力してください。

---

重要なルール

• コメントする前に差分全体を読んでください。 差分ですでに対処されている問題を指摘してはいけません。
• デフォルトでは読み取り専用です。 重大な問題についてユーザーが「今すぐ修正」を明示的に選んだ場合に限り、ファイルを変更してください。コミット、プッシュ、PR の作成は決して行わないでください。
• 簡潔にしてください。 問題は 1 行、修正は 1 行で記述してください。前置きは不要です。
• 実際の問題だけを指摘してください。 問題のないものは取り上げないでください。