| name | workflow-review |
| description | ローカルの git 差分を 3 観点 (TDD / コード品質+ルール準拠+構造 / プロダクト readiness) で並列レビュー。本体ロジックは review-tdd / review-quality / review-product-readiness subagent に委譲し、本 skill はメインセッション向けの薄い orchestrator として「差分検出 → PHASE_CONTEXT 組み立て → subagent 並列起動 → 結果集約・整形表示 → 修正アクション選択」を担当する。セキュリティレビューは security-guidance プラグイン (Stop hook の LLM diff review + Edit 時の pattern 検知) に委譲しており本 skill の対象外。fatal 判定は本 skill では行わず、呼び出し側に任せる。 |
| argument-hint | [--staged | --all] |
| allowed-tools | Bash, Read, Glob, Grep, Agent, AskUserQuestion |
/workflow-review - 3 観点並列コードレビュー (subagent wrapper)
3 つの review subagent を並列起動してコードレビューを行う薄い orchestrator。
設計方針
- 本体ロジック:
claude/agents/review-{tdd,quality,product-readiness}.md (subagent × 3。quality は rules 準拠とアーキテクチャ heuristic を統合済み)
- 本 skill: ユーザー向けエントリポイント。差分検出と PHASE_CONTEXT 組み立て、subagent 並列起動、結果集約・整形表示
- dev-impl (実装ループ) は本 skill を呼ばない (dev-impl 本体が Step 4.2d で review subagent を観点 gating 付きで直接起動する)。本 skill は手動レビュー用
- 観点拡張: 観点を増やしたい場合は
claude/agents/review-<観点>.md を追加して本 skill の起動リストに加える
skill / agent 責務分担の詳細は skills/README.md 参照。
パフォーマンス観点 (perf) はどこに?
旧 workflow-review にあった「パフォーマンス」観点は、現在 review-quality subagent の中の「アルゴリズム効率 / アンチパターン (重複・過剰抽象)」に統合済。専用 perf agent を作るかは future improvement (実プロジェクトで perf 指摘が薄いと感じたら別 agent 化を検討)。
セキュリティ観点はどこに?
セキュリティレビューは自作 subagent 化せず、Anthropic 公式プラグイン security-guidance@claude-plugins-official に委譲している。Edit/Write 時の pattern 検知と Stop hook での LLM diff review が自動で走るため、本 skill の並列レビューには含めない (git commit 毎のエージェント型レビュー層は ENABLE_COMMIT_REVIEW=0 で無効化済、粒度が過剰なため)。
使い方
/workflow-review # working tree 全変更 (unstaged + staged)
/workflow-review --staged # ステージ済みのみ
実行手順
Step 1: 差分検出
git rev-parse --git-dir
git status --porcelain
git diff [--staged]
git rev-parse HEAD
git リポジトリでない場合: 「gitリポジトリではありません」表示して終了。
変更 0 件の場合: 「レビュー対象がありません」表示して終了。
Step 2: PHASE_CONTEXT 組み立て (簡素版)
メインセッション利用なので「フェーズ」概念は無い。差分範囲を working_tree として扱い、PHASE_CONTEXT を最小限で組み立てる:
phase_name: "working-tree-review"
phase_start_sha: <HEAD>
diff_range: working_tree
related_source_files:
- <変更ファイル一覧>
related_rules_paths:
- rules/core/tdd.md
- rules/core/design.md
- rules/core/testing.md
- rules/core/commit.md
- <変更ファイル拡張子に応じて path 別 rules を追加>
design_overview: |
<docs/DESIGN.md があれば全文 or 抜粋>
design_detail: |
<docs/DESIGN_DETAIL_APP.md / docs/DESIGN_DETAIL_INFRA.md があれば全文 or 抜粋 (旧形式 docs/DESIGN_DETAIL.md しか無ければそれを使う)>
dev_server:
url: <検出できた URL>
start_command: <package.json の dev/start script>
output_path_prefix: /tmp/review-<観点>-working-tree.json
呼び出し側は Skill({ skill: "workflow-review", args: { phase_start_sha, phase_name, dimensions, dev_server, ... } }) で args を渡せる。本 skill は受け取った args をそのまま ctx として使う (args 未指定のフィールドは上記 yaml サンプルの手動収集ロジックで埋める):
const ctx = {
phase_name: args.phase_name ?? "working-tree-review",
phase_start_sha: args.phase_start_sha ?? execSync("git rev-parse HEAD").trim(),
related_source_files: args.related_source_files ?? gitStatusPorcelainFiles(),
related_rules_paths: args.related_rules_paths ?? defaultRulesPaths(),
design_overview: args.design_overview ?? readIfExists("docs/DESIGN.md"),
design_detail: args.design_detail ??
[readIfExists("docs/DESIGN_DETAIL_APP.md"), readIfExists("docs/DESIGN_DETAIL_INFRA.md")]
.filter(Boolean).join("\n\n") || readIfExists("docs/DESIGN_DETAIL.md"),
dev_server: args.dev_server,
}
dev_server が無い場合は省略してよい (review-product-readiness が URL 不在で no-op になる)。
args.dimensions (optional array, 例: ["tdd"] のような観点名の配列) が指定されていれば、その観点の subagent だけを起動する (呼び出し側での絞り込み用)。未指定時 (手動 /workflow-review 実行時など) は 3 観点フル起動。
Step 3: subagent 並列起動
Agent ツールを同一メッセージ内に複数 tool_use として並べて並列起動:
const DIMENSIONS = ["tdd", "quality", "product_readiness"]
const targetDimensions = args.dimensions?.length ? args.dimensions : DIMENSIONS
const AGENT_TYPE = {
tdd: "review-tdd",
quality: "review-quality",
product_readiness: "review-product-readiness",
}
const reviews = await Promise.all(
targetDimensions.map(d => Agent({ subagent_type: AGENT_TYPE[d], prompt: reviewPromptFor(d, ctx) }))
)
各 subagent は output_path に JSON を Write、stdout には絶対パス 1 行のみを返す。
reviewPromptFor(dimension, ctx)
各 subagent には dimension に応じて rules path を絞った prompt を渡す:
- tdd:
rules/core/tdd.md, rules/core/testing.md
- quality:
rules/core/design.md + 拡張子別 (rules/frontend/react/*.md 等) + design_overview / design_detail (DESIGN 整合チェック用)
- product_readiness: design_overview の「ゴール」(G_E2E 含む) と design_detail の「UX 設計」を必ず含める +
dev_server (start_command と URL) を渡す。CLI / API のみなら本 dimension は no-op で素通り
Step 4: 集約
起動した subagent (targetDimensions) それぞれの JSON を Read してパース、findings を 1 つのフラット配列にまとめる:
const findings = []
for (const r of reviews) {
const json = JSON.parse(await Read(r.trim()))
for (const f of json.findings) findings.push({ ...f, dimension: json.dimension })
}
return { findings, dimensions_reviewed: targetDimensions }
本 skill は fatal 判定を行わない。集約結果を「全 findings」としてそのまま返す / 表示する。
dimensions_reviewed は呼び出し側が findings を観点単位で洗い替えする際に使う (再実行しなかった観点の古い findings を誤って消さないため)。
Step 5: 結果整形表示 (メインセッション利用時)
サマリーテーブル → 観点別 findings の順で表示:
# Code Review Report
## Summary
| 観点 | high | medium | low |
|---|---|---|---|
| tdd | X | X | X |
| quality | X | X | X |
| product_readiness | X | X | X |
| **合計** | **X** | **X** | **X** |
## findings (severity 順)
### [high][quality] レイヤ境界違反 (domain → infra import)
**ファイル**: `src/domain/user.ts:12`
**内容**: <message>
**推奨**: <fix_proposal>
### [high][tdd] テストファイルが存在しない
...
Step 6: 修正アクション選択
AskUserQuestion で次のアクションを確認:
AskUserQuestion({
questions: [{
question: "レビューで X 件の high / Y 件の medium が検出されました。どう進めますか？",
header: "次のアクション",
options: [
{ label: "high をすべて修正", description: "メインループで TDD 修正 (tdd-guard が順序を強制)" },
{ label: "high + medium を修正", description: "同上、対象拡張" },
{ label: "個別に選択", description: "修正対象を選んで TDD 修正" },
{ label: "終了", description: "修正せず完了" }
],
multiSelect: false
}]
})
「修正」選択時は findings を修正タスクとしてメインループで直接 TDD 修正サイクルを実行する (RED: findings を再現する失敗テスト → GREEN → REFACTOR。順序は tdd-guard hook が強制)。修正後は同じ観点を再レビューして解消を確認する。
範囲外 (やらないこと)
- 観点別の検査ロジック → 各
review-* subagent
- fatal 判定 (severity 基準で「修正必須」を決める) → 呼び出し側 (本 skill 内の AskUserQuestion)
- 修正実行の委譲 → しない (メインループで直接 TDD 修正する)
- コミット →
workflow-commit
関連
- subagent:
review-tdd / review-quality / review-product-readiness (本体ロジック)
- セキュリティレビュー:
security-guidance@claude-plugins-official プラグイン (本 skill の外、Edit/Write pattern 検知 + Stop hook LLM diff review)
- 連携 skill:
workflow-commit (修正後のコミット)
- 上位: なし (dev-impl は本 skill を経由せず Step 4.2d で review subagent を直接起動する)
