Review Design

review-design

実装前の配置・パターン判定を Q1-Q3 で選択された reviewer subset (anti-pattern 必須 + DDD / Hexagonal / Clean Arch 選択) で並列レビュー → 必須 Devil's Advocate critique → 致命指摘があれば plan ファイルを直接書き換える。

Task complexity tier (skip / scope decision)

| Row | 状況 | アクション | |---|---|---| | 1 | 1 ファイル .design-review.md (プランファイルパスの拡張子直前に .design-review` を挿入したパス) へ同内容を保存する。保存内容・パス規則・プラン不在時の扱いは [references/final-report-format.md](references/final-report-format.md) を参照。

Three execution modes / DA escalation conditions / Fatal vs single-trigger 全表は [references/escalation-rules.md](references/escalation-rules.md) を参照。

委譲実行 (subagent として起動された場合)

Task ツールで委譲された場合、単独起動 (メイン会話でユーザーが直接起動) の動作に次を追加する。判定基準は「AskUserQuestion が利用可能ツールに無いか」で機械的に行う。

• 入力解決: Plan File Info: と会話文脈は単独起動時のみ有効な経路であり、委譲時は試みない。起動プロンプト本文の明示指定のみを $ARGUMENTS として扱う。$ARGUMENTS からプランファイルパスも feature description も得られない場合のみ、「不足入力: レビュー対象のプランファイルまたは feature description」を最終メッセージで返し即座に終了する (返答を待たない)。パス文字列が渡されているが指す先のファイルが存在しない場合も同様に即時終了する (内容を捏造せず、その旨を返して完結する)。
• Step 3 の Task 不可時: reviewer dispatch (Task) が利用可能ツール一覧に無い場合、main agent が [references/reviewer-modes.md](references/reviewer-modes.md) の Parallel Review fallback に従い agents/*.md を自ら Read して適用する。
• Design It Twice (deep-module-reviewer escalation): [references/deep-modules.md](references/deep-modules.md) の問題空間提示 (Step 1) は、対話承認者がいない (= AskUserQuestion が利用可能ツールに無い) 実行では提示のみで確認を待たず、即座に Step 2 (sub-agent 並列起動) へ進む。
• ${CLAUDE_PLUGIN_ROOT} の解決: agents/*.md を Read で直接実行しており本文中に ${CLAUDE_PLUGIN_ROOT} が生文字列で残る場合、いま読んでいる agent ファイルの 1 階層上 (skills/review-design/) を skill root とみなして解決する。nested Task へ埋め込むパスは解決後の絶対パスにする。
• 完了報告: Step 6 の最終メッセージ・保存ファイルの規定は [references/final-report-format.md](references/final-report-format.md) のとおり (委譲・単独起動で同一)。

Quality standards

• Criticism first — default verdict is "there is a problem"。一貫性 (existing patterns) と最小構成 (minimum viable structure) は Q1 matrix と anti-pattern-checker が判定するため、ここでは再掲しない。

Gotchas（観測済みの罠 — 実測で判明したものを 1 件 1 行で追記）

Advanced

• [references/escalation-rules.md](references/escalation-rules.md) — Three execution modes / DA escalation conditions / Fatal vs single-trigger 全表
• [references/reviewer-modes.md](references/reviewer-modes.md) — DA prompts (inline & subagent), feedback loop details, in-context fallback rules.
• [references/final-report-format.md](references/final-report-format.md) — Problem-free / problem-found templates, "1-issue-1-line" granularity, fallback tag examples.
• [references/detailed-workflow.md](references/detailed-workflow.md) — When Quick Start does not resolve the design (directory placement, dependency direction, pattern selection).
• Reviewer specs: agents/anti-pattern-checker.md, agents/ddd-reviewer.md, agents/hexagonal-reviewer.md, agents/clean-architecture-reviewer.md, agents/deep-module-reviewer.md.
• [references/deep-modules-quickref.md](references/deep-modules-quickref.md) / [references/deep-modules.md](references/deep-modules.md) — deep-module-reviewer の早見表 / 詳細 (用語・依存 4 分類・Design It Twice escalation)。
• [references/rails-patterns.md](references/rails-patterns.md) — Rails プロジェクトでの配置判断 (Model / Service / Form Object 等の第一候補表)。対象が Rails の場合、anti-pattern-checker の判定や [references/detailed-workflow.md](references/detailed-workflow.md) の配置場所決定で参照する。

Companion skills

• /define-acceptance-criteria — define AC after design review, before implementation.

Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

• Author: YasuakiOmokawa
• Source: YasuakiOmokawa/skills
• License: MIT

Install and usage instructions live in the source repository linked above.