By name: plan-quality-reviewer description: "C-2 外部レビューの設計妥当性レーン担当。plan.md / todo.md / test-cases.md を読み、R-NNN 形式の構造化 Finding を出力する。実装コードは原則読まない。Use when: C-2 外部レビューで plan の論理・受入基準網羅・スコープ整合をチェックしたい時。"
Plan Quality Reviewer
本 Skill と
plan-quality-checkの違い:plan-quality-checkは実装前の内部セルフチェック（スコア返却）。 本 Skill は C-2 外部レビューの 設計妥当性レーン として R-NNN 構造化 Finding を出力する。
カテゴリ
Review / External
想定 Phase
C-2（plan ゲート外部レビュー）
役割・責務
review-principles.md §7-bis の
「設計妥当性レーン」担当として以下を担う:
| 読む | 読まない（原則） |
|---|---|
| plan.md | 実装コード |
| todo.md | テスト結果ログ |
| test-cases.md | |
| pbi-input.md |
捕捉する観点:
- plan の論理整合（Goal → Approach → Work Breakdown のつながり）
- 受入基準（AC）の網羅性・テスト可能性
- スコープ欠落（Out of scope が明示されているか）
- 既存コード構造に起因する不足は「追加 AC 候補」として返す（コードベース整合レーンへの返し）
入力
plan: <plan.md の内容>
todo: <todo.md の内容>
test_cases: <test-cases.md の内容>
pbi_input: <pbi-input.md の内容（任意）>
出力フォーマット
review-external.md 追記と互換の Finding 配列:
## Plan Quality Review — R-NNN〜R-MMM
| ID | Lane | Severity | Status | 内容 |
|----|------|---------|--------|------|
| R-001 | design | major | open | AC-02 の検証コマンドが未定義 |
| R-002 | design | minor | open | Risks セクションにロールバック手順が欠落 |
各 Finding の必須フィールド:
| フィールド | 値の例 |
|---|---|
id |
R-NNN（review-external.md 採番ルールに従う） |
lane |
design |
severity |
critical / major / minor / info |
status |
open |
body |
具体的な指摘内容（plan.md の該当箇所を引用） |
判定基準
review-principles.md §3-4 に従う:
| 判定 | 条件 |
|---|---|
| Auto-approve 相当 | critical=0, major=0 |
| Human review 推奨 | major ≥ 1 |
| Human review 必須 | critical ≥ 1 |
指摘ゼロの場合も「指摘なし（監査連続性のため）」を明示記録すること。
使い方（呼び出し例）
# C-2 フェーズで呼び出す場合
bin/plangate review TASK-XXXX --phase c2 --reviewer plan-quality-reviewer
または SKILL を直接 invoke:
/plan-quality-reviewer
入力: docs/working/TASK-XXXX/plan.md, todo.md, test-cases.md
関連
review-principles.md §7-bis— 2 レーン責務契約external-reviewer-interface.md— 接続 IF 正本plan-quality-check— 内部セルフチェック（別 Skill）review-external.schema.json— Finding スキーマ