Review Loop

---
description: コード変更後の自動レビュー・修正サイクル。起動するのは、人が /review-loop と打ったときか「工程に回して」と言ったときだけ——AI は自分から起動せず、実装完了時・リファクタリング後・バグ修正後に当たったら人に提案する。
allowed-tools: Bash, Agent, Skill, Read, Write, Edit, Grep, Glob
---

# review-loop

コード変更を、レビューと修正を繰り返して阻害要因が出なくなるまで回す。採点は別 context の役に置き、収束はラウンドごとの記録（`.claude/review-rounds/round-<N>.json`）を機械（`${CLAUDE_PLUGIN_ROOT}/scripts/review-record.py`）で突き合わせて決める。

`${CLAUDE_PLUGIN_ROOT}` はこのプラグインのフォルダ（このファイル SKILL.md のフォルダの 2 つ上。`.claude-plugin/plugin.json` を持つフォルダ）を指し、この本文の中ではスキルを読み込んだ時にその絶対パスへ置き換わる。下の `references/` はこのファイルと同じフォルダの中にある。別ファイルのコマンドやパスに書いた `CLAUDE_PLUGIN_ROOT`（`$` と波括弧で囲んだ形）は置き換わらず Bash の環境変数にも無いので、そのフォルダの絶対パスに読み替えて使え。

## 役割分担（最重要）

このループを回す session が **writer（実装者）**。**writer は局所レビューを機械的に実行して findings を集めることはできるが、ラベル確定・根本診断・収束判定はしない。** それらは全て別 context の **grader subagent** が行う（writer ≠ grader）。writer は grader が確定した `[block]` を直すだけで、ラベルを自分で上げ下げできない。grader の判定を覆したい時も自分で決めず、次のラウンドで別 context の grader に再判定させろ。

**grader を都合よく使うな**: 同一ラウンドで grader を回し直して有利な結果を採るな（やり直しは次ラウンドのみ）。grader subagent の生出力（根本ユニットのキー集合・各 verdict）は最終報告にそのまま転記しろ。

レビュー観点とラベルの正本は `REVIEW.md`（プラグイン同梱の `${CLAUDE_PLUGIN_ROOT}/REVIEW.md` と、対象リポジトリのルートに在ればその `REVIEW.md` の 2 つ。併用の決まりは [references/tools.md](references/tools.md) の「ツールと正本の場所」。以降 `REVIEW.md` と書いたものはその併用を指す）。grader はレビュー開始前に `REVIEW.md` を読むこと。ただしツールを持たない隔離 grader（文脈遮断）は自分では読めないため、writer が該当セクションの本文をプロンプトに貼って渡せ。

武器の使い分け: **局所レビューは公式 skill に委譲**し再実装するな。ただし**欠陥と品質は別のツール**で、片方はもう片方を代替しない——欠陥（correctness）は `pr-review-toolkit:review-pr`、品質（reuse / 簡素化 / 効率 / altitude〔処理を置く層・抽象の高さが合っているか〕）は `/simplify` が担当する（`/simplify` は「バグは探さない」と自ら明記している）。**両方走らせろ**——「簡素化」は `review-pr` の観点にも含まれるが、観点が重なることは片方を省く理由にならない。根本診断・整合性・俯瞰は grader subagent で行う（公式に相当機能がない）。

## 段ごとに読むファイル

この本文は段（P0〜P4 と P-R の工程）の順と読むファイルだけを持ち、手順の中身は持たない。**段に入るたびに、その段のファイルを読んでから手を動かせ。** 会話が圧縮されて前に読んだ手順が見えなくなったら、今いる段のファイルを読み直せ。

1. 始める前に 1 度: [references/tools.md](references/tools.md)（スクリプトと記録の置き場、前のレビューの記録の退避、観点の正本、役の起動方法と渡し方、依存の扱い）
2. P0 基準点と元の目的: [references/p0-baseline.md](references/p0-baseline.md)（BASE〔差分の起点にする commit〕の決め方、手元の CI、前提の実測、目的テキストの凍結、並行 PR、先行議論）
3. P1 素材集め: [references/p1-materials.md](references/p1-materials.md)（局所レビューと各 grader の起動方法、渡す物と渡さない物、作業ツリーの突合）
4. P2 診断: [references/p2-diagnosis.md](references/p2-diagnosis.md)（judge〔P2 の grader。writer が覆せない判定を下す上位の役〕への入力、木探索〔指摘を根本へ遡って束ねる〕、ラベルと disposition〔今直す do-now か先送りの defer か〕、処方、安定キー〔指摘を根本ごとに束ねた単位に付ける「ファイルパス＋シンボル名＋一文要約」の名札。ラウンド間の突合に使う〕、履歴と問いの台帳〔人に聞く候補の一覧〕の再審）
5. P3 修正: [references/p3-fix.md](references/p3-fix.md)（根本ユニット〔P2 が指摘を根本ごとに束ねた単位〕で直す、閉鎖の実証〔直した修正が欠陥を閉じたことの確かめ〕、問いに挙がったユニットを実装しない）
6. P4 再採点: [references/p4-rescore.md](references/p4-rescore.md)（P1〜P2 のやり直し、`review-record.py` の突合と終了コード、台帳と scalar〔差分の規模の数値〕の記録、次への分岐、R1・R2 を P-R を待たずに回し直す条件）
7. P-R 俯瞰: [references/p-r-overview.md](references/p-r-overview.md)（R1〜R4〔冗長と最小性・ゼロベース再導出・全体整合・差分の外の横断リスク〕の起動方法と verdict の写し方）
8. P4 の判定のたびに: [references/stop.md](references/stop.md)（収束の条件、収束以外で止まる 3 つ——答え無しに進める仕事が尽きた・前提不成立・総ラウンドが 5 を超えた〔暴走ガード〕——、問いの種類ごとの再審）
9. 最終報告: [references/report.md](references/report.md)（冒頭 3 行に置く「人が決めること」〔台帳で人に回した問い〕、初見で決められる問いの形、出す前の確かめ、必ず含める項目）

1 ラウンド＝P1→P4 の 1 周で、記録 `round-<N>.json` を 1 つ書く。P4 の判定で `[block]` か直していない do-now が残れば P3 で直し、もう一度 P4（P1・P2 を新しい context でやり直す。ここから次のラウンド）。残りが 0 で CI 緑なら P-R へ進み、R1〜R4 に 1 つでも `redesign-needed` が出たら P1 から次のラウンドに入る。`premise-invalid` など他の verdict の扱いと、止めるかどうかは、P4 の判定のたびに stop.md で決める。

## どの段でも踏むと壊れる所

- 記録のフィールドと値の正本は `review-record.py`。`not_applicable`（条件に当たらない）と `not_run`（飛ばした）を混ぜると、無言の省略が「なし」に化ける。
- 前のラウンドの記録を消すな。突合が `exit 2`（記録が不正）で止まる。`exit 1`（阻害要因あり）と取り違えるな。
- 前のラウンドの grader の出力や修正の経緯を、次のラウンドの P1・P2 の役に渡すな（追認になる）。
- 人に聞く候補は、立った周に止めて聞かず問いの台帳に載せる。止めて聞くのは機械がそう出したときだけ（例外は stop.md）。