takt 0.2.0 → 0.2.2

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

@@ -27,10 +27,7 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
 
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
 
       ## Output Format
 
@@ -69,10 +66,7 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
 
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
 
       ## Output Format
 
@@ -116,10 +110,7 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
 
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
 
       ## Output Format
 

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

@@ -31,10 +31,7 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
 
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
 
       ## Output Format
 
@@ -95,10 +92,7 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
 
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
 
       ## Output Format
 
@@ -163,10 +157,7 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
 
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
 
       ## Judgment Criteria
 

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

@@ -30,16 +30,12 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
-
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
-
       ## Judgment Criteria
 
       | 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 +43,7 @@ steps:
       | Situation | Tag |
       |-----------|-----|
       | Analysis complete | `[PLANNER:DONE]` |
+      | Question answered | `[PLANNER:ANSWER]` |
       | Insufficient info | `[PLANNER:BLOCKED]` |
 
       ### Output Examples
@@ -56,6 +53,13 @@ steps:
       [PLANNER:DONE]
       ```
 
+      **ANSWER case:**
+      ```
+      {Answer to the question}
+
+      [PLANNER:ANSWER]
+      ```
+
       **BLOCKED case:**
       ```
       [PLANNER:BLOCKED]
@@ -81,10 +85,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 +127,8 @@ steps:
     transitions:
       - condition: done
         next_step: implement
+      - condition: answer
+        next_step: COMPLETE
       - condition: blocked
         next_step: ABORT
 
@@ -132,12 +143,8 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    permission_mode: acceptEdits
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
-
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
-
       ## Output Format
 
       | Situation | Tag |
@@ -233,11 +240,6 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
-
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
-
       ## Judgment Criteria
 
       | Situation | Judgment |
@@ -348,11 +350,6 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
-
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
-
       ## Judgment Criteria
 
       | Situation | Judgment |
@@ -458,11 +455,6 @@ steps:
       - WebSearch
       - WebFetch
     status_rules_prompt: |
-      # ⚠️ REQUIRED: Status Output Rules ⚠️
-
-      **Without this tag, the workflow will stop.**
-      Your final output MUST include a status tag following the rules below.
-
       ## Judgment Criteria
 
       | Situation | Judgment |

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

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