pr-review-cycle

name	pr_review_cycle
description	PR 完成時にコードレビューを行い、指摘修正→再レビューのサイクルを回すプロトコル (Claude Code)。

PR レビューサイクル

発動条件

• 実装が完了し、lint / test 全パス後の PR プッシュ時。

レビュー手段 (Claude Code)

手段	用途
/code-review スラッシュコマンド	現在の差分 (作業ブランチ) を Claude が観点別にレビュー。--comment で PR にインラインコメント投稿、--fix で修正適用も可
レビュー用サブエージェント (Task/Agent)	独立した観点 (correctness / security / 可読性) で並列にレビューさせ、結論を統合
GitHub の PR レビュー	リポジトリに設定された自動レビューア (例: GitHub Actions / 外部ボット) があればそれも併用。gh pr view --comments で取得

[!WARNING] 多重レビューの防止: 同じ差分に重複してレビューを走らせない。再レビュー前に必ず gh pr view <PR番号> --comments で未対応の指摘が残っていないか確認する。

プロトコル

1. 初回レビュー

• 作業ブランチで /code-review を実行する (またはレビュー用サブエージェントを起動)。
• GitHub 側に自動レビューアがいれば、PR 作成後にその指摘も gh pr view <PR番号> --comments で回収する。

2. 指摘への対応

1. 各指摘を分析し、正当なものは修正する。
2. 仕様を変えずにテストだけを変えてはならない (テスト失敗→実装で対応する)。
3. 修正後は cargo fmt → cargo clippy --workspace --all-targets -- -D warnings → cargo test --workspace を全実行する。
4. 棄却する指摘は、PR コメントで理由を明記する (gh pr comment <PR番号> --body "...")。

3. 再レビュー (コード修正した場合)

• 修正をプッシュし、再度 /code-review を実行する (または GitHub 側に再レビューを依頼)。
• 返答・棄却理由の説明と、新たなレビュー依頼は別アクションとして扱い、混同しない。

4. サイクルの終了とマージ基準

指摘レベル別の対応ルール:

レベル	対応
Critical	必ず修正。修正後に再レビュー必須
High	必ず修正。修正後に再レビュー必須
Medium	内容が妥当であれば修正する
Low	判断して対応。棄却する場合は PR コメントで理由を残す

マージ判定フロー:

1. Critical / High 指摘が 0 件になるまでサイクルを繰り返す。
2. Medium 指摘を対応し、CI (Test ワークフロー) がパスしたらマージ可能。
3. main への直接 push / 直接マージは禁止。必ず PR 経由でマージする。