| name | pr-review-automation |
| description | Use when Codex writes PR review comments with Automation blocks, triages Codex review findings into follow-up Issues, or decides whether a review finding can be auto-fixed by opencode or claude-code, or needs human review. |
pr-review-automation
Codex の PR レビュー指摘を、実害のある指摘、修正担当 agent、後続 Issue に整理するための skill。
いつ使うか
- Codex が PR review で指摘を書くとき
- PR review コメントを Issue 化するとき
severity / auto-fix / fix-agent を判定するとき
- 自動修正対象にしてよいか迷うとき
レビュー指摘の基準
指摘するのは、PR の意図、Issue 本文、差分、既存コードの契約を照合して、実害のある不具合・回帰・テスト不足・運用リスクがある場合だけ。
Codex は line-level の不具合、実行不能手順、このリポジトリの local rule と配布物の衝突、GitHub 上の Codex review から読める AGENTS.md 指示、Automation / 修正プロンプト による自動修正導線を重点的に見る。
root AGENTS.md / CLAUDE.md の managed block 外、README、scripts、CI workflow、既存 worktree / branch helper を確認し、配布された managed block / skill の手順が repo-local rule と矛盾しないか確認する。特に <trunk> / default branch / worktree 作成 / PR base / test command / deploy path は、共通テンプレートの推測ではなく repo-local の正本を優先する。
nit、好み、将来改善、再現性が不明な懸念、具体 task 化できない設計相談は指摘しない。既存仕様を変える必要がある場合は、実装提案ではなく人間判断が必要な論点として明示する。
レビューコメント形式
指摘ごとに以下を含める。
### 問題
<対象ファイル、関数、条件分岐、データ構造を含めて具体化>
### 影響
<ユーザー影響、データ不整合、セキュリティリスク、運用リスク>
### 修正方針
- <変更対象>
- <維持すべき既存仕様>
- <変更してはいけない範囲>
### テスト方針
<追加 / 更新すべきテスト。不要なら理由>
### 修正プロンプト
<修正担当 agent に渡す prompt。対象 PR / review comment URL、作業 branch、修正範囲、変更してはいけない範囲、確認コマンド、完了時の返信・resolve 方針を含める>
### Automation
- `severity: P0 | P1 | P2 | nit`
- `auto-fix: safe | needs-human-review`
- `fix-agent: opencode | claude-code | none`
Automation 判定
auto-fix: safe: 追加の仕様判断なしに修正でき、既存 PR の意図を変えない場合だけ
auto-fix: needs-human-review: 仕様判断、外部制約、正本変更、または高リスク領域を含む場合
fix-agent: opencode: 単一ファイル / 単一関数、10 行未満、テスト追加・修正不要、期待 diff が文章 1 行で書ける
fix-agent: claude-code: opencode で扱えない修正、複数ファイル、テスト追加・修正、近傍理解、PR 全体の意図を踏まえた修正が必要
auto-fix: needs-human-review の場合は fix-agent: none
DB スキーマ変更、データ移行、認証・認可、課金、外部 API 仕様変更、インフラ変更、secret / token / permission の扱い変更を含む場合は、必ず needs-human-review / none。
auto-fix: safe を付ける場合、### 修正プロンプト はそれだけで修正担当 agent が着手できる具体度にする。修正プロンプトが書けない指摘は自動修正対象にしない。
Issue 化分類
PR レビュー指摘や自動発見された後続課題は、次の 3 種類だけ Issue 化してよい。
これらは分類名であり、GitHub label 名ではない。
opencode-ready: opencode が 1 試行で完了できる超小粒 task
claude-code-ready: claude-code が追加の仕様判断なしに着手できる実装 task
needs-refinement: 実害はあるが、要件・期待結果・対象範囲・検証方法に曖昧さが残る整理 task
分類は opencode-ready → claude-code-ready → needs-refinement の順に判定する。1 つの指摘を複数 Issue に分解してよいが、「ついでに棚卸し」「ついでにテスト追加」を 1 Issue に詰め込まない。
起票時の label は次に揃える。
opencode-ready: type:task, agent:opencode, status:implementation-ready
claude-code-ready: type:task, agent:claude-code, status:implementation-ready
needs-refinement: status:needs-refinement を付け、PO 整理が必要なら agent:codex を付ける
Issue 化しないもの
- 元 PR / 元 Story をマージしてはいけない blocker
- nit、感想、将来改善
- 再現性が不明な懸念
- 具体 task 化できない設計相談
- 既存 Issue と重複するもの
Issue 本文に含めるもの
- 元レビューコメント URL または発見元
- 背景
- Goal
- Scope
- Out of Scope
- Acceptance Criteria
- Verification
- Open Questions
- 起票経路と性質
起票後は作成した Issue URL と付与したラベルをユーザーに報告する。