takt 0.2.0 → 0.2.1

package/resources/global/en/workflows/default.yaml

@@ -39,6 +39,7 @@ steps:
       | Situation | Judgment |
       |-----------|----------|
       | Requirements clear and implementable | DONE |
+      | User is asking a question (not an implementation task) | ANSWER |
       | Requirements unclear, insufficient info | BLOCKED |
 
       ## Output Format
@@ -46,6 +47,7 @@ steps:
       | Situation | Tag |
       |-----------|-----|
       | Analysis complete | `[PLANNER:DONE]` |
+      | Question answered | `[PLANNER:ANSWER]` |
       | Insufficient info | `[PLANNER:BLOCKED]` |
 
       ### Output Examples
@@ -55,6 +57,13 @@ steps:
       [PLANNER:DONE]
       ```
 
+      **ANSWER case:**
+      ```
+      {Answer to the question}
+
+      [PLANNER:ANSWER]
+      ```
+
       **BLOCKED case:**
       ```
       [PLANNER:BLOCKED]
@@ -80,10 +89,15 @@ steps:
       ## Instructions
       Analyze the task and create an implementation plan.
 
+      **Judgment criteria:**
+      - If the user input is an implementation task → create a plan and output `[PLANNER:DONE]`
+      - If the user input is a question → research, answer, and output `[PLANNER:ANSWER]`
+      - If there is insufficient information → output `[PLANNER:BLOCKED]`
+
       **Note:** If returned from implement step (Previous Response exists),
       review and revise the plan based on that feedback (replan).
 
-      **Tasks:**
+      **Tasks (for implementation tasks):**
       1. Understand the requirements
       2. Identify impact scope
       3. Decide implementation approach
@@ -117,6 +131,8 @@ steps:
     transitions:
       - condition: done
         next_step: implement
+      - condition: answer
+        next_step: COMPLETE
       - condition: blocked
         next_step: ABORT
 
@@ -131,6 +147,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ REQUIRED: Status Output Rules ⚠️
 
@@ -360,6 +377,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ REQUIRED: Status Output Rules ⚠️
 
@@ -545,6 +563,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ REQUIRED: Status Output Rules ⚠️
 
@@ -727,6 +746,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ REQUIRED: Status Output Rules ⚠️
 
@@ -796,6 +816,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ REQUIRED: Status Output Rules ⚠️
 

package/resources/global/en/workflows/simple.yaml

@@ -40,6 +40,7 @@ steps:
       | Situation | Judgment |
       |-----------|----------|
       | Requirements clear and implementable | DONE |
+      | User is asking a question (not an implementation task) | ANSWER |
       | Requirements unclear, insufficient info | BLOCKED |
 
       ## Output Format
@@ -47,6 +48,7 @@ steps:
       | Situation | Tag |
       |-----------|-----|
       | Analysis complete | `[PLANNER:DONE]` |
+      | Question answered | `[PLANNER:ANSWER]` |
       | Insufficient info | `[PLANNER:BLOCKED]` |
 
       ### Output Examples
@@ -56,6 +58,13 @@ steps:
       [PLANNER:DONE]
       ```
 
+      **ANSWER case:**
+      ```
+      {Answer to the question}
+
+      [PLANNER:ANSWER]
+      ```
+
       **BLOCKED case:**
       ```
       [PLANNER:BLOCKED]
@@ -81,10 +90,15 @@ steps:
       ## Instructions
       Analyze the task and create an implementation plan.
 
+      **Judgment criteria:**
+      - If the user input is an implementation task → create a plan and output `[PLANNER:DONE]`
+      - If the user input is a question → research, answer, and output `[PLANNER:ANSWER]`
+      - If there is insufficient information → output `[PLANNER:BLOCKED]`
+
       **Note:** If returned from implement step (Previous Response exists),
       review and revise the plan based on that feedback (replan).
 
-      **Tasks:**
+      **Tasks (for implementation tasks):**
       1. Understand the requirements
       2. Identify impact scope
       3. Decide implementation approach
@@ -118,6 +132,8 @@ steps:
     transitions:
       - condition: done
         next_step: implement
+      - condition: answer
+        next_step: COMPLETE
       - condition: blocked
         next_step: ABORT
 
@@ -132,6 +148,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ REQUIRED: Status Output Rules ⚠️
 

package/resources/global/ja/agents/default/ai-reviewer.md

@@ -51,11 +51,15 @@ AI生成コードには特有の特徴があります:
 | 古いパターン | 学習データからの非推奨アプローチの使用 |
 | 過剰エンジニアリング | タスクに不要な抽象化レイヤーの追加 |
 | 過小エンジニアリング | 現実的なシナリオのエラーハンドリングの欠如 |
+| 配線忘れ | 機構は実装されているが、エントリポイントから渡されていない |
 
 **検証アプローチ:**
 1. このコードは実際にコンパイル/実行できるか？
 2. インポートされたモジュール/関数は存在するか？
 3. このライブラリバージョンでAPIは正しく使用されているか？
+4. 新しいパラメータ/フィールドが追加された場合、呼び出し元から実際に渡されているか？
+   - AIは個々のファイル内では正しく実装するが、ファイル横断の結合を忘れがち
+   - `options.xxx ?? fallback` で常にフォールバックが使われていないか grep で確認
 
 ### 3. コピペパターン検出
 
@@ -96,7 +100,26 @@ AI生成コードには特有の特徴があります:
 
 **原則:** 最良のコードは、問題を解決する最小限のコード。
 
-### 6. 決定トレーサビリティレビュー
+### 6. デッドコード検出
+
+**AIは新しいコードを追加するが、不要になったコードの削除を忘れることが多い。**
+
+| パターン | 例 |
+|---------|-----|
+| 未使用の関数・メソッド | リファクタリング後に残った旧実装 |
+| 未使用の変数・定数 | 条件変更で不要になった定義 |
+| 到達不能コード | 早期returnの後に残った処理、常に真/偽になる条件分岐 |
+| 未使用のインポート・依存 | 削除された機能のimport文やパッケージ依存 |
+| 孤立したエクスポート・公開API | 実体が消えたのにre-exportやindex登録が残っている |
+| 未使用のインターフェース・型定義 | 実装側が変更されたのに残った古い型 |
+| 無効化されたコード | コメントアウトされたまま放置されたコード |
+
+**検証アプローチ:**
+1. 変更・削除されたコードを参照している箇所がないか grep で確認
+2. 公開モジュール（index ファイル等）のエクスポート一覧と実体が一致しているか確認
+3. 新規追加されたコードに対応する古いコードが残っていないか確認
+
+### 7. 決定トレーサビリティレビュー
 
 **Coderの決定ログが妥当か検証する。**
 

package/resources/global/ja/agents/default/architect.md

@@ -384,7 +384,40 @@ function createOrder(data: OrderData) {
 - 3回の重複 → 即抽出
 - ドメインが異なる重複 → 抽象化しない（例: 顧客用バリデーションと管理者用バリデーションは別物）
 
-### 8. 品質特性
+### 8. 呼び出しチェーン検証
+
+**新しいパラメータ・フィールドが追加された場合、変更ファイル内だけでなく呼び出し元も検証する。**
+
+**検証手順:**
+1. 新しいオプショナルパラメータや interface フィールドを見つけたら、`Grep` で全呼び出し元を検索
+2. 全呼び出し元が新しいパラメータを渡しているか確認
+3. フォールバック値（`?? default`）がある場合、フォールバックが使われるケースが意図通りか確認
+
+**危険パターン:**
+
+| パターン | 問題 | 検出方法 |
+|---------|------|---------|
+| `options.xxx ?? fallback` で全呼び出し元が `xxx` を省略 | 機能が実装されているのに常にフォールバック | grep で呼び出し元を確認 |
+| テストがモックで直接値をセット | 実際の呼び出しチェーンを経由しない | テストの構築方法を確認 |
+| `executeXxx()` が内部で使う `options` を引数で受け取らない | 上位から値を渡す口がない | 関数シグネチャを確認 |
+
+**具体例:**
+
+```typescript
+// ❌ 配線漏れ: projectCwd を受け取る口がない
+export async function executeWorkflow(config, cwd, task) {
+  const engine = new WorkflowEngine(config, cwd, task);  // options なし
+}
+
+// ✅ 配線済み: projectCwd を渡せる
+export async function executeWorkflow(config, cwd, task, options?) {
+  const engine = new WorkflowEngine(config, cwd, task, options);
+}
+```
+
+**このパターンを見つけたら REJECT。** 個々のファイルが正しくても、結合されていなければ機能しない。
+
+### 9. 品質特性
 
 | 特性 | 確認観点 |
 |------|---------|
@@ -392,7 +425,7 @@ function createOrder(data: OrderData) {
 | Maintainability | 変更・修正が容易か |
 | Observability | ログ・監視が可能な設計か |
 
-### 9. 大局観
+### 10. 大局観
 
 **注意**: 細かい「クリーンコード」の指摘に終始しない。
 
@@ -403,7 +436,7 @@ function createOrder(data: OrderData) {
 - ビジネス要件と整合しているか
 - 命名がドメインと一貫しているか
 
-### 10. 変更スコープの評価
+### 11. 変更スコープの評価
 
 **変更スコープを確認し、レポートに記載する（ブロッキングではない）。**
 
@@ -422,7 +455,7 @@ function createOrder(data: OrderData) {
 **提案として記載すること（ブロッキングではない）:**
 - 分割可能な場合は分割案を提示
 
-### 11. 堂々巡りの検出
+### 12. 堂々巡りの検出
 
 レビュー回数が渡される場合（例: 「レビュー回数: 3回目」）、回数に応じて判断を変える。
 

package/resources/global/ja/agents/default/coder.md

@@ -90,6 +90,7 @@
 | 構文エラー | ビルド・コンパイル |
 | テスト | テスト実行 |
 | 要求充足 | 元のタスク要求と照合 |
+| デッドコード | 変更・削除した機能を参照する未使用コードが残っていないか確認（未使用の関数、変数、インポート、エクスポート、型定義、到達不能コード） |
 
 **すべて確認してから `[DONE]` を出力。**
 

package/resources/global/ja/workflows/default.yaml

@@ -39,6 +39,7 @@ steps:
       | 状況 | 判定 |
       |------|------|
       | 要件が明確で実装可能 | DONE |
+      | ユーザーが質問をしている（実装タスクではない） | ANSWER |
       | 要件が不明確、情報不足 | BLOCKED |
 
       ## 出力フォーマット
@@ -46,6 +47,7 @@ steps:
       | 状況 | タグ |
       |------|------|
       | 分析完了 | `[PLANNER:DONE]` |
+      | 質問への回答 | `[PLANNER:ANSWER]` |
       | 情報不足 | `[PLANNER:BLOCKED]` |
 
       ### 出力例
@@ -55,6 +57,13 @@ steps:
       [PLANNER:DONE]
       ```
 
+      **ANSWER の場合:**
+      ```
+      {質問への回答}
+
+      [PLANNER:ANSWER]
+      ```
+
       **BLOCKED の場合:**
       ```
       [PLANNER:BLOCKED]
@@ -80,10 +89,15 @@ steps:
       ## Instructions
       タスクを分析し、実装方針を立ててください。
 
+      **判断基準:**
+      - ユーザーの入力が実装タスクの場合 → 計画を立てて `[PLANNER:DONE]`
+      - ユーザーの入力が質問の場合 → 調査・回答して `[PLANNER:ANSWER]`
+      - 情報不足の場合 → `[PLANNER:BLOCKED]`
+
       **注意:** Previous Responseがある場合は差し戻しのため、
       その内容を踏まえて計画を見直してください（replan）。
 
-      **やること:**
+      **やること（実装タスクの場合）:**
       1. タスクの要件を理解する
       2. 影響範囲を特定する
       3. 実装アプローチを決める
@@ -117,6 +131,8 @@ steps:
     transitions:
       - condition: done
         next_step: implement
+      - condition: answer
+        next_step: COMPLETE
       - condition: blocked
         next_step: ABORT
 
@@ -131,6 +147,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ 必須: ステータス出力ルール ⚠️
 
@@ -319,7 +336,9 @@ steps:
       - 構造・設計の妥当性
       - コード品質
       - 変更スコープの適切性
-      - **テストカバレッジ**: 実装に対応する単体テストが追加されているか
+      - テストカバレッジ
+      - デッドコード
+      - 呼び出しチェーン検証
 
       **レポート出力:** 上記の `Report File` に出力してください。
       - ファイルが存在しない場合: 新規作成
@@ -339,6 +358,8 @@ steps:
       - [x] コード品質
       - [x] 変更スコープ
       - [x] テストカバレッジ
+      - [x] デッドコード
+      - [x] 呼び出しチェーン検証
 
       ## 問題点（REJECTの場合）
       | # | 場所 | 問題 | 修正案 |
@@ -372,6 +393,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ 必須: ステータス出力ルール ⚠️
 
@@ -556,6 +578,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ 必須: ステータス出力ルール ⚠️
 
@@ -737,6 +760,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ 必須: ステータス出力ルール ⚠️
 
@@ -805,6 +829,7 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
       # ⚠️ 必須: ステータス出力ルール ⚠️
 

package/resources/global/ja/workflows/expert-review.yaml

@@ -2,9 +2,9 @@
 # CQRS+ES、フロントエンド、セキュリティ、QAの専門家によるレビューワークフロー
 #
 # フロー:
-# plan -> implement -> cqrs_es_review -> frontend_review -> ai_review -> security_review -> qa_review -> supervise -> COMPLETE
-#                          ↓                  ↓                 ↓              ↓                ↓            ↓
-#                     fix_cqrs_es       fix_frontend        ai_fix       fix_security       fix_qa    fix_supervisor
+# plan -> implement -> architect_review -> cqrs_es_review -> frontend_review -> ai_review -> security_review -> qa_review -> supervise -> COMPLETE
+#                          ↓                   ↓                  ↓                 ↓              ↓                ↓            ↓
+#                     fix_architect        fix_cqrs_es       fix_frontend        ai_fix       fix_security       fix_qa    fix_supervisor
 #
 # 修正時の戻り先はCoderが判断:
 # - fix_security: MINOR→security_review, MAJOR→cqrs_es_review
@@ -196,12 +196,169 @@ steps:
       進行できない場合は [CODER:BLOCKED] を出力し、planに戻ります。
     transitions:
       - condition: done
+        next_step: architect_review
+      - condition: blocked
+        next_step: plan
+
+  # ===========================================
+  # Phase 2: Architecture Review
+  # ===========================================
+  - name: architect_review
+    agent: ~/.takt/agents/default/architect.md
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+      - WebSearch
+      - WebFetch
+    status_rules_prompt: |
+      # ⚠️ 必須: ステータス出力ルール ⚠️
+
+      **このタグがないとワークフローが停止します。**
+      最終出力には必ず以下のルールに従ったステータスタグを含めてください。
+
+      ## 判定基準
+
+      | 状況 | 判定 |
+      |------|------|
+      | 構造に問題がある | REJECT |
+      | 設計原則違反がある | REJECT |
+      | 呼び出しチェーンの配線漏れ | REJECT |
+      | テストが不十分 | REJECT |
+      | 改善すべき点がある（軽微） | IMPROVE |
+      | 問題なし | APPROVE |
+
+      ## 出力フォーマット
+
+      | 状況 | タグ |
+      |------|------|
+      | 問題なし | `[ARCHITECT:APPROVE]` |
+      | 軽微な改善必要 | `[ARCHITECT:IMPROVE]` |
+      | 構造的な修正必要 | `[ARCHITECT:REJECT]` |
+    instruction_template: |
+      ## Workflow Context
+      - Iteration: {iteration}/{max_iterations}（ワークフロー全体）
+      - Step Iteration: {step_iteration}（このステップの実行回数）
+      - Step: architect_review (アーキテクチャレビュー)
+      - Report Directory: .takt/reports/{report_dir}/
+      - Report File: .takt/reports/{report_dir}/03-architect-review.md
+
+      ## Original User Request (ワークフロー開始時の元の要求)
+      {task}
+
+      ## Git Diff
+      ```diff
+      {git_diff}
+      ```
+
+      ## Instructions
+      **アーキテクチャと設計**のレビューに集中してください。
+
+      **レビュー観点:**
+      - 構造・設計の妥当性
+      - コード品質
+      - 変更スコープの適切性
+      - テストカバレッジ
+      - デッドコード
+      - 呼び出しチェーン検証
+
+      **レポート出力:** 上記の `Report File` に出力してください。
+      - ファイルが存在しない場合: 新規作成
+      - ファイルが存在する場合: `## Iteration {step_iteration}` セクションを追記
+
+      **レポートフォーマット:**
+      ```markdown
+      # アーキテクチャレビュー
+
+      ## 結果: APPROVE / IMPROVE / REJECT
+
+      ## サマリー
+      {1-2文で結果を要約}
+
+      ## 確認した観点
+      - [x] 構造・設計
+      - [x] コード品質
+      - [x] 変更スコープ
+      - [x] テストカバレッジ
+      - [x] デッドコード
+      - [x] 呼び出しチェーン検証
+
+      ## 問題点（REJECTの場合）
+      | # | 場所 | 問題 | 修正案 |
+      |---|------|------|--------|
+      | 1 | `src/file.ts:42` | 問題の説明 | 修正方法 |
+
+      ## 改善提案（任意・ブロッキングではない）
+      - {将来的な改善提案}
+      ```
+
+      **認知負荷軽減ルール:**
+      - APPROVE + 問題なし → サマリーのみ（5行以内）
+      - APPROVE + 軽微な提案 → サマリー + 改善提案（15行以内）
+      - REJECT → 問題点を表形式で（30行以内）
+    transitions:
+      - condition: approved
         next_step: cqrs_es_review
+      - condition: improve
+        next_step: fix_architect
+      - condition: rejected
+        next_step: fix_architect
+
+  - name: fix_architect
+    agent: ~/.takt/agents/default/coder.md
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+      - Edit
+      - Write
+      - Bash
+      - WebSearch
+      - WebFetch
+    permission_mode: acceptEdits
+    status_rules_prompt: |
+      # ⚠️ 必須: ステータス出力ルール ⚠️
+
+      **このタグがないとワークフローが停止します。**
+      最終出力には必ず以下のルールに従ったステータスタグを含めてください。
+
+      ## 出力フォーマット
+
+      | 状況 | タグ |
+      |------|------|
+      | 修正完了 | `[CODER:DONE]` |
+      | 進行不可 | `[CODER:BLOCKED]` |
+    instruction_template: |
+      ## Workflow Context
+      - Iteration: {iteration}/{max_iterations}（ワークフロー全体）
+      - Step Iteration: {step_iteration}（このステップの実行回数）
+      - Step: fix_architect
+
+      ## Architect Feedback (これが最新の指示です - 優先して対応してください)
+      {previous_response}
+
+      ## Original User Request (ワークフロー開始時の元の要求 - 参考情報)
+      {task}
+
+      ## Additional User Inputs
+      {user_inputs}
+
+      ## Instructions
+      **重要**: Architectのフィードバックに対応してください。
+      「Original User Request」は参考情報であり、最新の指示ではありません。
+      セッションの会話履歴を確認し、Architectの指摘事項を修正してください。
+
+      完了時は [CODER:DONE] を含めてください。
+      進行できない場合は [CODER:BLOCKED] を含めてください。
+    pass_previous_response: true
+    transitions:
+      - condition: done
+        next_step: architect_review
       - condition: blocked
         next_step: plan
 
   # ===========================================
-  # Phase 2: CQRS+ES Review
+  # Phase 3: CQRS+ES Review
   # ===========================================
   - name: cqrs_es_review
     agent: ~/.takt/agents/expert-review/cqrs-es-reviewer.md
@@ -229,7 +386,7 @@ steps:
       - Step Iteration: {step_iteration}（このステップの実行回数）
       - Step: cqrs_es_review (CQRS+ES専門レビュー)
       - Report Directory: .takt/reports/{report_dir}/
-      - Report File: .takt/reports/{report_dir}/03-cqrs-es-review.md
+      - Report File: .takt/reports/{report_dir}/04-cqrs-es-review.md
 
       ## Original User Request
       {task}
@@ -376,7 +533,7 @@ steps:
       - Step Iteration: {step_iteration}（このステップの実行回数）
       - Step: frontend_review (フロントエンド専門レビュー)
       - Report Directory: .takt/reports/{report_dir}/
-      - Report File: .takt/reports/{report_dir}/04-frontend-review.md
+      - Report File: .takt/reports/{report_dir}/05-frontend-review.md
 
       ## Original User Request
       {task}
@@ -523,7 +680,7 @@ steps:
       - Step Iteration: {step_iteration}（このステップの実行回数）
       - Step: ai_review (AI生成コードレビュー)
       - Report Directory: .takt/reports/{report_dir}/
-      - Report File: .takt/reports/{report_dir}/05-ai-review.md
+      - Report File: .takt/reports/{report_dir}/06-ai-review.md
 
       ## Original User Request (ワークフロー開始時の元の要求)
       {task}
@@ -664,7 +821,7 @@ steps:
       - Step Iteration: {step_iteration}（このステップの実行回数）
       - Step: security_review (セキュリティ専門レビュー)
       - Report Directory: .takt/reports/{report_dir}/
-      - Report File: .takt/reports/{report_dir}/06-security-review.md
+      - Report File: .takt/reports/{report_dir}/07-security-review.md
 
       ## Original User Request
       {task}
@@ -818,7 +975,7 @@ steps:
       - Step Iteration: {step_iteration}（このステップの実行回数）
       - Step: qa_review (QA専門レビュー)
       - Report Directory: .takt/reports/{report_dir}/
-      - Report File: .takt/reports/{report_dir}/07-qa-review.md
+      - Report File: .takt/reports/{report_dir}/08-qa-review.md
 
       ## Original User Request
       {task}
@@ -978,7 +1135,7 @@ steps:
       - Step: supervise (最終確認)
       - Report Directory: .takt/reports/{report_dir}/
       - Report Files:
-        - Validation: .takt/reports/{report_dir}/08-supervisor-validation.md
+        - Validation: .takt/reports/{report_dir}/09-supervisor-validation.md
         - Summary: .takt/reports/{report_dir}/summary.md
 
       ## Original User Request
@@ -991,6 +1148,7 @@ steps:
 
       ## Previous Reviews Summary
       このステップに到達したということは、以下のレビューがすべてAPPROVEされています：
+      - Architecture Review: APPROVED
       - CQRS+ES Review: APPROVED
       - Frontend Review: APPROVED
       - AI Review: APPROVED
@@ -1054,6 +1212,7 @@ steps:
       ## レビュー結果
       | レビュー | 結果 |
       |---------|------|
+      | Architecture | ✅ APPROVE |
       | CQRS+ES | ✅ APPROVE |
       | Frontend | ✅ APPROVE |
       | AI Review | ✅ APPROVE |