Delegate review

delegate-review

差分のコード/ドキュメントレビューを委譲する。task_type=review、既定モデル opus（指摘品質が成果物に直結し判断比重が高いため）。実行系分岐（Codex / Devin / Cursor / Claude）は dispatch.sh が行う。

スクリプトパス

• Claude Code: skill_dir=.claude/skills/delegate-review
• Codex: skill_dir=.agents/skills/delegate-review

以降のコマンド例は Claude Code の .claude/skills/delegate-review を使う。Codex で使う場合は、同じ相対構造の .agents/skills/delegate-review に読み替える。

モデル価格参照

コスト分析・単価比較が必要な場合のみ、<skill_dir>/model-token-prices.json を読む。このデータは参照用であり、delegate の起動可否判定には使わない。

委譲する前に（コストゲート）

review は、main が差分全体を読むと context を膨らませる一次レビューに使う。大きめの diff、複数ファイルにまたがる変更、広い影響範囲の確認が必要な差分を発火条件にする。一方、数行の diff、main が既に読んだ差分、style / typo 程度の軽微レビューは委譲せず main が直接処理する。 レビュー対象はコード差分に限らず、README / spec / design docs / changelog などのドキュメント差分も含める。 review の作業を委譲する場合は、この skill を使う。generic な subagent へ流す運用はしない。

実行フロー（one-shot）

1. リクエスト作成: Objective / Scope / Context / Constraints の Markdown を stdin で渡す。レビュー対象の差分範囲（base/head・対象パス等）を Scope に明記する。request は terse に書く: diff 本体やファイル内容は貼らず、範囲指定とパス参照で渡す（main の出力＝課金トークンを増やさないため）。
   • ユーザーが会話でモデルや effort を指定した場合は、run 呼び出しにインライン env を前置する（例: DELEGATE_REVIEW_MODEL=gpt-5.5@high bash .../run.sh ...）。exit 6 の場合は、許容値列挙を含む stderr の 1 行をそのままユーザーへの説明に使う。
2. 実行: out="$(printf '%s' "$req_md" | bash .claude/skills/delegate-review/scripts/run.sh review DELEGATE_REVIEW_MODEL opus "$PARENT_TASK_TYPE_CHAIN" "$REQUESTER_SESSION_ID")"（top-level 起動なら $PARENT_TASK_TYPE_CHAIN は空でよい）。
   • run は内部で prepare → dispatch → read-response を順に実行し、stdout は成功・失敗とも単一 JSON（exit_code / status / content / content_truncated / response_file / observe_file / run_dir）を返す。
   • selector 省略時の既定は decision。decision は大規模 response でも Summary と Findings / Blockers の要点を 1 回で返す。第 6 位置引数は read-response の selector であり、prepare.sh の第 6 位置引数 session_mode とは意味が異なる。
   • exit code は内部スクリプトを透過する。exit 3=前提不足 / exit 4=委譲サイクルなら中止する。
   • run は dispatch 前に observe_file: <path> を stderr へ先出しする。強制終了時はその path を復旧経路にする。
   • 非対話モードの親（claude -p 等）では run を必ずフォアグラウンドで実行し、委譲所要時間より長い Bash timeout（Claude Code なら BASH_DEFAULT_TIMEOUT_MS / BASH_MAX_TIMEOUT_MS または Bash tool の timeout 引数）を設定する。
3. レスポンス消費と裏取り: status="$(printf '%s' "$out" | bash .claude/skills/delegate-review/scripts/read-json.sh .status)" / content="$(printf '%s' "$out" | bash .claude/skills/delegate-review/scripts/read-json.sh .content)" を読む。content_truncated が true なら response_file="$(printf '%s' "$out" | bash .claude/skills/delegate-review/scripts/read-json.sh .response_file)" を取り出し、bash .claude/skills/delegate-review/scripts/read-response.sh "$response_file" <N> で必要 section だけ段階読みする。読了後、worker の本文を 要約し直さない（echo しない）。重要 findings は該当する file:line 周辺または該当 diff だけを main が確認する。問題なしの報告や軽微 findings は差分全体を再読せず、Scope と Summary の整合を確認する。

高度なフロー（個別スクリプト）

dispatch 中の observe 監視、background 実行など、途中で親の判断を挟むフローでは従来の個別スクリプトを使う。

1. 準備（集約）: 前提チェック→モデル解決→チェーン確認→リクエスト生成を prepare.sh 1 本に畳む。Objective / Scope / Context / Constraints の Markdown を stdin で渡す。レビュー対象の差分範囲（base/head・対象パス等）を Scope に明記する。request は terse に書く: diff 本体やファイル内容は貼らず、範囲指定とパス参照で渡す（main の出力＝課金トークンを増やさないため）。exit 3=前提不足 / exit 4=委譲サイクルなら中止。
   • ユーザーが会話でモデルや effort を指定した場合は、prepare 呼び出しにインライン env を前置する（例: DELEGATE_REVIEW_MODEL=gpt-5.5@high bash .../prepare.sh ...）。prepare が exit 6 の場合は、許容値列挙を含む stderr の 1 行をそのままユーザーへの説明に使う。
   • out="$(printf '%s' "$req_md" | bash .claude/skills/delegate-review/scripts/prepare.sh review DELEGATE_REVIEW_MODEL opus "$PARENT_TASK_TYPE_CHAIN" "$REQUESTER_SESSION_ID")"（top-level 起動なら $PARENT_TASK_TYPE_CHAIN は空でよい）
   • model="$(printf '%s' "$out" | bash .claude/skills/delegate-review/scripts/read-json.sh .model)" / request_file="$(printf '%s' "$out" | bash .claude/skills/delegate-review/scripts/read-json.sh .request_file)" / response_file="$(printf '%s' "$out" | bash .claude/skills/delegate-review/scripts/read-json.sh .response_file)" / run_dir="$(printf '%s' "$out" | bash .claude/skills/delegate-review/scripts/read-json.sh .run_dir)" / observe_file="$(printf '%s' "$out" | bash .claude/skills/delegate-review/scripts/read-json.sh .observe_file)"
2. 実行: bash .claude/skills/delegate-review/scripts/dispatch.sh "$model" review "$request_file" "$response_file" "$run_dir" "$observe_file"。モデル名プレフィックスによる実行系分岐（Codex / Devin / Cursor / Claude）は dispatch.sh が行う。stdout は response_file のパスのみ。非対話モードの親（claude -p 等）では dispatch を必ずフォアグラウンドで実行し、委譲所要時間より長い Bash timeout（Claude Code なら BASH_DEFAULT_TIMEOUT_MS / BASH_MAX_TIMEOUT_MS または Bash tool の timeout 引数）を設定する。実行中の通常監視は observe_file から state.phase / state.started_at / heartbeat.ts / heartbeat.stdout_bytes / heartbeat.stderr_bytes / heartbeat.last_stream_change_at だけを read-json.sh で読む。state.phase は prepared | running | superseded | stalled | ended。prepared / superseded は dispatch されなかった observe（state.started_at == null、usage は未設定で read-json.sh では null 相当）なので、usage を集計する場合は分母から除外する。
3. レスポンス読み取り: bash .claude/skills/delegate-review/scripts/read-response.sh "$response_file" decision。decision は大規模 response でも Summary と Findings / Blockers の要点を 1 回で返す。さらに必要な詳細があれば ... "$response_file" <N> で追加取得する。読了後、worker の本文を 要約し直さない（echo しない）。main のユーザー向け応答は Summary を指す 1 行に留める（main の出力＝課金トークンを増やさないため。spec.md §6）。
4. 裏取りフェーズ（必須）: status を先に確認し、必要時のみ Findings section を引く。重要 findings は該当する file:line 周辺または該当 diff だけを main が確認する。問題なしの報告や軽微 findings は差分全体を再読せず、Scope と Summary の整合を確認する。

待ち時間の隠蔽（対話親向け）

対話親では dispatch.sh（または run.sh）を background で実行し、observe_file の state.phase / heartbeat を確認して ended 後に read-response.sh する運用で体感待ち時間を隠蔽できる。総所要時間（wall time）は変わらない体感改善であり、非対話モードの親では従来どおりフォアグラウンド実行必須。

制約

• read-only。ファイル編集・git の書き込み操作・push はしない（差分を読んで指摘を報告するだけ）
• read-only 種別のため session reuse（resumable / follow-up）は使わない
• 指摘は報告 Markdown の Findings section に収め、各 finding に severity / file:line / 根拠 / 影響 / 推奨対応を含める
• task_type_chain 内種別への再委譲はしない（別種別 delegate は可）
• main は worker 出力を echo / 再要約しない。ユーザー向けは Summary を指す 1 行に留める（出力＝課金トークンを増やさないため。spec.md §6）