independent-reviewlisted
Install: claude install-skill inakaegg/agent-kit
# Independent Review
## 使用条件
- PRへ出す非自明な変更
- user-visible behavior、public API、persistence、concurrency、async state
- auth、security、privacy、billing、deployment、migration
- architecture boundaryまたは広いrefactor
- 実装者が設計判断へ強くanchoringしている可能性がある
小さなdocs typo、formatterだけの変更等は省略できる。
## 0. レビュー段階の適用範囲とreviewer設定
共通AGENTS §7の区分に従う。
- **重リスク作業**(公開API、永続化、並行・非同期、認証・security、課金、migration、deploy、広いarchitecture):仕様レビュー → 実装計画レビュー → 実装レビュー。仕様と実装計画が1つの短い文書に収まる場合は、仕様と計画のレビューを1回へ統合してよい。
- **通常対象作業**(それ以外のユーザー可視動作、PR化する非自明な変更):実装レビューのみ。仕様・計画はreviewなしで現在taskの `TASK.md` とactive planへ記録する。
reviewerのモデルとreasoning effort:
- 担当にできるモデルは agent-settings のキーが決める(重リスク作業 `REVIEW_MODEL_HEAVY`・通常対象作業 `REVIEW_MODEL_DEFAULT`・読みやすさ `REVIEW_MODEL_READABILITY`。書式は `docs/policies/review.md`)。その範囲内で実装担当と別のモデルを優先し、モデル独立性とコストの両立を図る。
- 設定値が「系統(思考量)」の形で思考量を指定している場合は、その思考量を正本として使う。
- 設定値に思考量の指定がない場合の既定:仕様・実装計画レビューは中位、実装レビューは中位を既定とし重リスク作業だけ高位へ上げる。reviewは検査であり生成ではないため、指定がない限り実装と同じ最高設定にしない。
## 1. 独立性
- 履歴を共有しない別セッションを使う。
- 可能なら実装とは別のtool/modelを使う。
- 実装者の会話履歴、途中仮説、自己評価、期待する結論を渡さない。
- Reviewer processでは、実装者向けの作業手順注入(SessionStart hook、skill案内、plugin指示)を無効化する。プロジェクト規約 `AGENTS.md` は §2 の通り渡すが、実装者の作業手順は渡さない。
- claude: `--setting-sources ""` `--disable-slash-commands` `--no-session-persistence`
- codex: `--ignore-user-config` `--ephemeral` `-s read-only`
- Reviewerはcodeを変更しないread-only roleとする。
- サブエージェントを無制限に使わない。別CLI processを1つ、最大1reviewerとする。
## 2. Reviewerへ渡す資料
Reviewerへ渡すもの:
- 適用される `AGENTS.md`
- `_ai/TASK.md` ま