fix(breezing): resolve 5 contradictions in Phase 0 design

1. Replace project-analyzer with dedicated plan-analyst agent
   (project-analyzer is for tech stack detection, not task planning)
2. Move breezing-active.json write before Phase 0 for compaction safety
3. Change V1-V4 "skip" to "inform" after Phase 0
   (Phase 0 = strategic review, V1-V4 = technical validation - different roles)
4. Remove circular Phase 0 auto-trigger from V1-V4 results
   (V1-V4 runs in Phase A which is after Phase 0)
5. Add Planner ↔ Critic direct dialogue in Round 2
   (article emphasizes intra-round teammate discussion)

https://claude.ai/code/session_011wQtWaurYoaTaofxCsistr

Author: claude

agents/plan-analyst.md

@@ -0,0 +1,107 @@
+---
+name: plan-analyst
+description: タスク計画を分析し、粒度・依存関係・owns推定・リスク評価を行う
+tools: [Read, Glob, Grep]
+disallowedTools: [Write, Edit, Bash, Task]
+model: sonnet
+color: cyan
+memory: project
+---
+
+# Plan Analyst Agent
+
+Plans.md のタスク分解を分析し、実装前に粒度・依存関係・ファイル所有権・リスクを評価する専門エージェント。
+
+---
+
+## 永続メモリの活用
+
+### 分析開始前
+
+1. **メモリを確認**: 過去のタスク分析結果、プロジェクト固有の依存パターンを参照
+2. 前回の分析で学んだファイル構造や命名規約を活用
+
+### 分析完了後
+
+以下を学んだ場合、メモリに追記:
+
+- **ファイル所有権パターン**: 「認証系は src/auth/ + src/middleware.ts」等
+- **依存関係パターン**: 「DB マイグレーションは必ず先行」等
+- **粒度の知見**: 「UI タスクは 5 ファイル以内に収まる傾向」等
+
+---
+
+## 分析観点
+
+### 1. タスク粒度評価
+
+各タスクについて以下を判定:
+
+| 判定 | 条件 |
+|---|---|
+| `appropriate` | 推定ファイル数 ≤ 10、記述が具体的、受入条件あり |
+| `too_broad` | 推定ファイル数 > 10、サブタスク 5+ |
+| `too_vague` | ファイルパス/コンポーネント名/API 名がゼロ |
+| `too_small` | 単独では意味をなさない（他タスクとの統合を推奨） |
+
+### 2. owns 推定
+
+コードベースを Glob/Grep で調査し、各タスクの影響ファイルを推定:
+
+```
+1. タスク説明のキーワードからファイル検索
+   例: "ログインフォーム" → Glob("**/Login*.tsx")
+2. 関連ディレクトリの推定
+   例: "認証" → src/auth/, src/lib/auth/
+3. import/export 依存の追跡
+   例: middleware.ts が auth/ 内のモジュールを import
+```
+
+### 3. 依存関係提案
+
+- 同一ファイルを触るタスク間の依存を検出
+- 暗黙の依存を推定（API ← フロント、DB スキーマ ← アプリ層）
+- 不要な依存チェーンの指摘（並列度の改善提案）
+
+### 4. リスク評価
+
+| リスクレベル | 条件 |
+|---|---|
+| `high` | セキュリティ関連、外部 API 連携、DB スキーマ変更 |
+| `medium` | 複数タスクの統合点、共有ユーティリティの変更 |
+| `low` | 独立した UI コンポーネント、テスト追加 |
+
+---
+
+## 報告フォーマット
+
+```json
+{
+  "tasks": [
+    {
+      "id": "4.1",
+      "title": "タスク名",
+      "estimated_owns": ["src/path/file.ts"],
+      "granularity": "appropriate",
+      "risk": "low",
+      "notes": "分析メモ"
+    }
+  ],
+  "proposed_dependencies": [
+    {"from": "4.1", "to": "4.2", "reason": "依存理由"}
+  ],
+  "parallelism_assessment": {
+    "independent_tasks": 3,
+    "max_parallel": 2,
+    "bottleneck": "タスク 4.2 が長い依存チェーンの起点"
+  }
+}
+```
+
+---
+
+## 制約
+
+- **Read-only**: Write, Edit, Bash は使用禁止
+- コードベースの調査は Glob/Grep/Read のみ使用
+- 実装の提案はしない、分析と評価のみ

skills/breezing/references/execution-flow.md

@@ -7,20 +7,22 @@ Agent Teams を活用した `/breezing` の実行フロー。Lead は状況に
 ```text
 /breezing --discuss 認証機能からユーザー管理まで完了して
     ↓
+Step 0: breezing-active.json 即時書き込み（全 Phase の前に実行）
+    ↓
 ┌─────────────────────────────────────────────────────────────┐
 │ Phase 0: Planning Discussion（--discuss 時のみ）             │
 │                                                              │
 │  Planner + Critic を spawn → 計画議論 2-3 ラウンド          │
+│  Planner ↔ Critic 直接対話 + Lead 調整                      │
 │  → 精査済み計画をユーザーに提示 → Planner/Critic shutdown   │
 │                                                              │
-│  ※ --discuss なし + バリデーション問題なし → Phase 0 スキップ │
+│  ※ --discuss なし → Phase 0 スキップ                        │
 │  ※ 詳細: planning-discussion.md 参照                        │
 └─────────────────────────────────────────────────────────────┘
     ↓
 ┌─────────────────────────────────────────────────────────────┐
 │ Phase A: Pre-delegate（ユーザーのパーミッションモード維持）   │
 │                                                              │
-│  Step 0: breezing-active.json 即時書き込み                   │
 │  Step 1: 環境チェック                                        │
 │  Step 2: 範囲確認（ユーザー承認）                            │
 │  Step 3: Team 初期化 → TaskCreate → Teammates spawn          │
@@ -63,29 +65,9 @@ Agent Teams を活用した `/breezing` の実行フロー。Lead は状況に
 > delegate に切り替えると、bypass が失われて承認プロンプトが発生する。
 > Phase A/C では delegate に入らないことで、ユーザーのパーミッションモードを維持する。
 
-## Phase 0: Planning Discussion（--discuss 時のみ）
-
-`--discuss` フラグ指定時、またはタスク粒度バリデーションで warning 3+ 検出時に起動。
-
-```text
-1. Planner (project-analyzer) + Critic (plan-critic) を spawn
-2. Round 1: Planner がタスク分析（owns 推定・依存提案・粒度評価・リスク）
-3. Round 2: Critic が Red Teaming 検証（ゴール達成性・粒度・依存・並列化・リスク・代替案）
-4. Round 3: Lead が統合判断 → ユーザーに提示
-5. (必要なら) ユーザーが Plans.md 修正 → 追加ラウンド（最大 3 ラウンド）
-6. Planner/Critic shutdown → Phase A へ
-```
-
-Phase 0 で得られた情報（owns 推定、依存提案等）は Phase A に引き継がれ、
-バリデーション済みタスクは V1〜V4 チェックをスキップ可能。
-
-詳細: [planning-discussion.md](references/planning-discussion.md) 参照
-
-## 準備ステージ
-
-### 0. breezing-active.json 即時書き込み（最優先）
+## Step 0: breezing-active.json 即時書き込み（最優先）
 
-**環境チェックよりも前に実行する。** Compaction 対策として、モード情報を永続化する。
+**Phase 0・Phase A のいずれよりも前に実行する。** Compaction 対策として、モード情報を永続化する。
 
 ```jsonc
 // .claude/state/breezing-active.json に即時書き込み
@@ -119,6 +101,27 @@ Phase 0 で得られた情報（owns 推定、依存提案等）は Phase A に
   ・Team 初期化の失敗
 ```
 
+## Phase 0: Planning Discussion（--discuss 時のみ）
+
+`--discuss` フラグ指定時に起動。Step 0 の後、Phase A の前に実行する。
+
+```text
+1. Planner (plan-analyst) + Critic (plan-critic) を spawn
+2. Round 1: Planner がタスク分析 → Planner ↔ Critic 直接対話で疑問点を解消
+3. Round 2: Critic が Red Teaming 検証 → Planner に確認が必要な点を直接質問
+4. Round 3: Lead が両者の分析を統合 → ユーザーに提示
+5. (必要なら) ユーザーが Plans.md 修正 → 追加ラウンド（最大 3 ラウンド）
+6. Planner/Critic shutdown → Phase A へ
+```
+
+Phase 0 で得られた情報（owns 推定、依存提案等）は Phase A に引き継がれ、
+Phase A の V1〜V4 バリデーションの**参考情報**として活用する（ただしスキップはしない。
+Phase 0 は戦略/アーキテクチャ評価、V1〜V4 は技術的詳細チェックで役割が異なるため）。
+
+詳細: [planning-discussion.md](references/planning-discussion.md) 参照
+
+## 準備ステージ（Phase A）
+
 ### 1. 環境チェック
 
 ```bash

skills/breezing/references/planning-discussion.md

@@ -21,17 +21,20 @@ Phase A: Pre-delegate（通常フローに合流）
 | 条件 | Phase 0 起動 |
 |---|---|
 | `--discuss` フラグ指定 | 常に起動 |
-| タスク粒度バリデーション(V1〜V4)で warning 3+ | 自動起動を推奨（Lead 判断） |
-| タスク数 10+ | 自動起動を推奨（Lead 判断） |
-| フラグなし + 問題なし | スキップ（直接 Phase A へ） |
+| フラグなし | スキップ（直接 Phase A へ） |
+
+> **注**: Phase A の V1〜V4 バリデーションで多数の warning が見つかった場合、
+> Lead は「次回は `--discuss` の使用を推奨」とユーザーに提案できる。
+> ただし Phase 0 は Phase A の**前**に実行されるため、
+> V1〜V4 の結果を見てから Phase 0 に戻ることはできない。
 
 ## Team 構成（Phase 0 限定）
 
 ```text
 Lead ─── 議論の調整、最終判断
   │
   ├── Planner (sonnet) ─── タスク分析・依存推定・粒度精査
-  │     subagent_type: claude-code-harness:project-analyzer
+  │     subagent_type: claude-code-harness:plan-analyst
   │     mode: "bypassPermissions"
   │
   └── Critic (sonnet) ─── Red Teaming・批判的検証
@@ -81,13 +84,23 @@ Planner → SendMessage → Lead:
   }
 ```
 
-### Round 2: Critic の批判的レビュー
+### Round 2: Critic の批判的レビュー + Planner ↔ Critic 直接対話
+
+**記事の知見**: Agent Teams の最大の強みは「Teammate 間の直接対話（intra-round discussion）」。
+Lead を仲介せず、Planner と Critic が直接やりとりして疑問点を解消する。
 
 ```text
 Lead → SendMessage → Critic:
   「Planner の分析結果を踏まえて、計画を批判的に検証してください。
-   Red Teaming チェックリスト（ゴール達成性・粒度・依存・並列化・リスク・代替案）
-   の各観点で問題を指摘してください。」
+   不明点は Planner に直接質問してください。」
+
+Critic → SendMessage → Planner:  ← Teammate 間直接対話
+  「タスク 4.2 が 4.1 の認証 API に依存するとのことですが、
+   4.2 は JWT 検証だけなので独立実装可能では？」
+
+Planner → SendMessage → Critic:  ← Teammate 間直接対話
+  「src/middleware.ts を確認したところ、4.1 で作成する loginHandler の
+   レスポンス型を 4.2 の JWT 検証が参照しています。依存は正当です。」
 
 Critic → SendMessage → Lead:
   {
@@ -113,8 +126,9 @@ Critic → SendMessage → Lead:
         "suggestion": "タスク 4.4 を独立化できないか検討"
       }
     ],
+    "planner_consultations": 1,
     "parallelism_score": "medium",
-    "summary": "概ね妥当だが、タスク 4.3 の具体化とテストタスクの追加が推奨"
+    "summary": "概ね妥当だが、タスク 4.3 の具体化とテストタスクの追加が推奨。依存関係は Planner と確認済み。"
   }
 ```
 
@@ -166,9 +180,11 @@ Phase 0 で得られた情報は Phase A に引き継ぐ:
 
 ```text
 Phase 0 → Phase A への引き継ぎ:
-  1. Planner の estimated_owns → Phase A Step 3 の owns 推定に活用
+  1. Planner の estimated_owns → Phase A Step 3 の owns 推定に活用（Glob 再検索を省略可能）
   2. Planner の proposed_dependencies → Phase A Step 3 の addBlockedBy に反映
-  3. Critic の findings → バリデーション済みとして V1〜V4 チェックをスキップ可能
+  3. Critic の findings → V1〜V4 バリデーションの参考情報（スキップはしない）
+     ※ Phase 0 は戦略/アーキテクチャ評価、V1〜V4 は技術的詳細チェック。
+       役割が異なるため、Phase 0 を経てもバリデーションは必ず実行する。
   4. 修正された Plans.md → Phase A の入力として使用
 ```
 

skills/breezing/references/team-composition.md

@@ -214,7 +214,7 @@ codex-review スキルの 4 エキスパートを MCP 経由で並列呼び出
 
 | 項目 | 設定 |
 |------|------|
-| **subagent_type** | `claude-code-harness:project-analyzer` |
+| **subagent_type** | `claude-code-harness:plan-analyst` |
 | **モデル** | sonnet |
 | **数** | 1 (常に) |
 | **責務** | タスク分析、owns 推定、依存関係提案、粒度評価、リスク評価 |