review-designlisted

Q1-Q3 で reviewer subset を選び、並列レビューと Devil's Advocate を行う。fatal は設計へ反映する。 ## Task complexity tier (skip / scope 判定) | Row | 状況 | アクション | |---|---|---| | 1 | 既存 class 内の局所的な method 追加・修正のみで、配置決定済みかつ新しい責務・seam・外部境界を追加しない | **skip** (本 skill 不要) | | 2 | production file 追加 or 配置に迷う or class/module 境界をまたぐ既存パターン拡張 | Q1-Q3 → matrix で reviewer subset 決定 | | 3 | 新しい責務境界 (module / interface / seam) / 複数 production file に新しい責務を追加 / domain 跨り | Q1-Q3 でも **all 5 reviewers を default** | | 4 | auth / billing / payment / migration / security (= **territory**, semantic 判定であり keyword match ではない) | 配置が自明でも実行 + DA を subagent dispatch | Row 4 → 3 → 2 → 1 の順で最初に該当する行を使う。ただし Row 3+4 compound は Row 3 の all 5 reviewers と Row 4 の DA dispatch を併用する。Row 1 の局所 method 変更は Row 2 の「既存パターン拡張」に含めない。既存属性への read-only predicate / getter 追加 (例: `def admin?; role == 'admin'; end`) は Row 4 に該当せず Row 1 で **skip 可**。新たな write path / 新規ガード (`before_action :require_admin!` 等) / 新規 callback (after_save で session/token 操作) を含めば Row 4 で **強制実行**。判定不能なら Quick Start へ。territory の境界例は [references/task-tier-boundaries.md](references/task-tier-boundaries.md) を Read。 ## Quick Start: Q1-Q3 → reviewer 選定 | Q | 判定 | |---|---| | Q1: 類似機能あり? | Yes → Q1.1 を評価してから Q2 / No → Q2 | | Q2: 責務を一句で言える? | Yes → 1 file, Q3 へ / "X and Y" → 分割してから Q3 | | Q3: テスト可能 (deps 差替可)? | 外部依存なしは Yes。No → DI / 引数で注入 (matrix への影響なし、Leaky Abstraction / Feature Envy 検出は anti-pattern-checker に委ねる) | **Q1.1 健全性チェック (Q1=Yes のみ)**: 1. 対象の振る舞いを支える tests が通る