takt 0.42.0 → 0.44.0

package/builtins/en/workflows/review-dual-cqrs.yaml

@@ -153,6 +153,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise

package/builtins/en/workflows/review-dual.yaml

@@ -129,6 +129,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise

package/builtins/en/workflows/review-fix-backend-cqrs.yaml

@@ -141,6 +141,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise

package/builtins/en/workflows/review-fix-backend.yaml

@@ -117,6 +117,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise

package/builtins/en/workflows/review-fix-default.yaml

@@ -1,5 +1,5 @@
 name: review-fix-default
-description: Multi-perspective review + fix loop (architecture, security, QA, testing, requirements — 5 parallel reviewers with iterative fixes)
+description: Multi-perspective review + fix loop (architecture, security, QA, testing, requirements, coding — 6 parallel reviewers with iterative fixes)
 workflow_config:
   provider_options:
     codex:
@@ -171,6 +171,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise

package/builtins/en/workflows/review-fix-dual-cqrs.yaml

@@ -166,6 +166,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise

package/builtins/en/workflows/review-fix-dual.yaml

@@ -142,6 +142,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise

package/builtins/en/workflows/review-fix-frontend.yaml

@@ -141,6 +141,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise

package/builtins/en/workflows/review-fix-takt-default.yaml

@@ -1,5 +1,5 @@
 name: review-fix-takt-default
-description: "TAKT-focused multi-perspective review + fix loop (6-parallel initial review, AI antipattern pre-review on fix iterations)"
+description: "TAKT-focused multi-perspective review + fix loop (7-parallel initial review including coding review, AI antipattern pre-review on fix iterations, supervisor-finding fixes)"
 workflow_config:
   provider_options:
     codex:
@@ -209,6 +209,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise
@@ -334,7 +356,7 @@ steps:
       - condition: All checks passed
         next: COMPLETE
       - condition: Requirements unmet, tests failing, build errors
-        next: ABORT
+        next: fix_supervisor
     output_contracts:
       report:
         - name: supervisor-validation.md
@@ -342,3 +364,31 @@ steps:
         - name: summary.md
           format: summary
           use_judge: false
+  - name: fix_supervisor
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+    knowledge:
+      - takt
+      - architecture
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+          - Edit
+          - Write
+          - Bash
+          - WebSearch
+          - WebFetch
+    required_permission_mode: edit
+    instruction: fix-supervisor
+    pass_previous_response: false
+    rules:
+      - condition: Fixes for supervisor findings complete
+        next: supervise
+      - condition: Unable to proceed with fixes
+        next: supervise

package/builtins/en/workflows/review-frontend.yaml

@@ -128,6 +128,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise

package/builtins/en/workflows/review-takt-default.yaml

@@ -1,5 +1,5 @@
 name: review-takt-default
-description: "TAKT-focused multi-perspective review (6 reviewers including AI antipattern review)"
+description: "TAKT-focused multi-perspective review (7 reviewers including AI antipattern and coding review)"
 workflow_config:
   provider_options:
     codex:
@@ -184,6 +184,28 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+
     rules:
       - condition: all("approved")
         next: supervise
@@ -222,7 +244,8 @@ steps:
          - `04-testing-review.md` (Testing review)
          - `05-ai-antipattern-review.md` (AI antipattern review)
          - `06-requirements-review.md` (Requirements review)
-      2. Synthesize the 6 review results
+         - `07-coding-review.md` (Coding review)
+      2. Synthesize the 7 review results
       3. Produce a consolidated review summary with overall verdict
 
       **Review Summary output contract:**
@@ -243,6 +266,7 @@ steps:
       | Testing | APPROVE/REJECT | {Brief finding} |
       | AI Antipattern | APPROVE/REJECT | {Brief finding} |
       | Requirements | APPROVE/REJECT | {Brief finding} |
+      | Coding | APPROVE/REJECT | {Brief finding} |
 
       ## Issues Requiring Attention
       | # | Severity | Source | Location | Issue |
@@ -273,6 +297,7 @@ steps:
             | Testing | APPROVE/REJECT | {Overview} |
             | AI Antipattern | APPROVE/REJECT | {Overview} |
             | Requirements | APPROVE/REJECT | {Overview} |
+            | Coding | APPROVE/REJECT | {Overview} |
 
             ## Current Iteration Findings (new)
             | # | finding_id | Severity | Source | Location | Issue | Fix Suggestion |

package/builtins/en/workflows/terraform.yaml

@@ -119,10 +119,31 @@ steps:
           report:
             - name: ai-antipattern-review.md
               format: ai-antipattern-review
+      - name: coding-review
+        edit: false
+        persona: coding-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
     rules:
-      - condition: all("Complies with Terraform conventions", "No AI-specific issues")
+      - condition: all("Complies with Terraform conventions", "No AI-specific issues", "approved")
         next: supervise
-      - condition: any("Convention violations found", "AI-specific issues found")
+      - condition: any("Convention violations found", "AI-specific issues found", "needs_fix")
         next: supervise
   - name: supervise
     edit: false

package/builtins/ja/config.yaml

@@ -7,7 +7,7 @@
 language: ja         # 表示言語: ja | en
 
 # デフォルトプロバイダー・モデル
-# provider: claude                    # デフォルトプロバイダー: claude | codex | opencode | cursor | copilot | mock
+# provider: claude                    # デフォルトプロバイダー: claude | claude-sdk | claude-terminal | codex | opencode | cursor | copilot | kiro | mock
 # model: sonnet                       # デフォルトモデル（プロバイダーにそのまま渡される）
 
 # 実行制御
@@ -91,10 +91,18 @@ language: ja         # 表示言語: ja | en
 # runtime:
 #   prepare: [node, gradle, ./custom-script.sh]
 
+# workflow YAML command gate ポリシー
+# workflow_command_gates:
+#   custom_scripts: false
+
 # ワークフローレベルのオーバーライド
 # workflow_overrides:
 #   quality_gates:
-#     - "All tests pass"
+#     - "All tests pass" # AI への完了条件
+#     - type: command    # step 完了時に機械実行される gate
+#       name: quality-check
+#       command: "./.takt/quality-gates/check.sh"
+#       timeout_ms: 300000
 #   quality_gates_edit_only: true
 #   steps:
 #     review:
@@ -114,12 +122,14 @@ language: ja         # 表示言語: ja | en
 # openrouter_api_key: "..."        # OpenRouter APIキー
 # opencode_api_key: "..."          # OpenCode APIキー
 # cursor_api_key: "..."            # Cursor APIキー
+# kiro_api_key: "..."              # Kiro APIキー
 
 # CLIパス
 # codex_cli_path: "/absolute/path/to/codex"    # Codex CLI絶対パス
 # claude_cli_path: "/absolute/path/to/claude"  # Claude Code CLI絶対パス
 # cursor_cli_path: "/absolute/path/to/cursor"  # cursor-agent CLI絶対パス
 # copilot_cli_path: "/absolute/path/to/copilot" # Copilot CLI絶対パス
+# kiro_cli_path: "/absolute/path/to/kiro-cli"   # Kiro CLI絶対パス
 # copilot_github_token: "ghp_..."              # Copilot GitHubトークン
 
 # その他

package/builtins/ja/facets/instructions/dual-team-leader-implement.md

@@ -6,10 +6,13 @@
 
 1. 分解の可否を判断する
    - 変更対象ファイルを特定し、ファイル間の依存関係を確認する
-   - 横断的関心事（共有型・ID・イベント）がある場合は分解せず1パートで実装する
+   - まず並行可能な責務境界を探す
+   - 横断的関心事（共有型・ID・イベント）がある場合も、基盤 part → 消費 part → 検証 part に段階分けできるか検討する
    - 変更ファイル数が少ない場合、リファクタ・リネーム系の場合も1パートで実装する
+   - `parts.length === 1` になる場合も、検証分離や段階分けができないか先に検討する
+   - 「実装と検証」のような巨大な単一 part は避ける
 
-2. 分解する場合: フロントエンドとバックエンドでの分割を優先的に検討する
+2. 分解する場合: フロントエンドとバックエンドでの分割を優先的に検討し、難しい場合はファイルをレイヤー/モジュール単位でグループ化する
    - **デザイン参照があり、バックエンド変更が明示要件でない場合は分解しない。** UI の見た目・構造・文言・間隔は相互依存が強く、分割すると忠実再現を壊しやすいため、1パートでまとめて実装する
    - **デザイン参照がある場合、同一画面を構成する UI コンポーネント群は同じパートにまとめる。** ヘッダー・フィルタ・カード・バナー・モーダルなど、見た目の整合性に関わる要素を別パートへ分散しない
    - フロントエンド（UI・コンポーネント・スタイル）とバックエンド（API・ロジック・データ層）で分けるのが最も自然な分割軸
@@ -19,6 +22,7 @@
    - 型・インターフェースの依存がある場合は、依存元と依存先を同じグループにまとめる
    - 1つのファイルを複数のパートに割り当てない
    - テストファイルと実装ファイルは同じパートにまとめる
+   - 実装 part と検証 part を分ける
 
 3. 各パートに排他的なファイル担当を割り当てる
    - 各パートの instruction に以下を必ず明記する：
@@ -32,6 +36,7 @@
    - テスト済みの場合は「既存テストがパスするよう実装する」と明記する
    - 品質ゲート（Quality Gates）を参照し、必要な検証は専用の単独パートとして計画する
    - 並列の実装パートには、全体ビルド・全体テストを重複して実行させない
+   - npm test / npm run test:e2e:mock は各実装 part に重複して持たせない
 
 **制約:**
 - テストやビルドが必要な場合は、他パートの変更が揃った後に、単独の検証パートで実行する

package/builtins/ja/facets/instructions/fix-maintenance.md

@@ -0,0 +1,43 @@
+Report Directory内のレビューレポートを確認し、指摘事項を既存契約を保つ最小差分で修正してください。
+
+**修正の原則:**
+- 指摘の「修正案」が示されている場合はそれに沿った対応を優先し、独自の迂回策を取らない
+- 指摘の対象コードを直接修正すること。テストやドキュメントの追加で指摘を回避しない
+- 指摘を「修正必須」「確認のみ」「要求外」に分ける
+- 修正必須の指摘だけを対象にする
+- 指摘対象外のリファクタ、改名、コメント削除、テスト期待値変更を混ぜない
+
+**レポート参照方針:**
+- Report Directory内の最新レビューレポートを一次情報として参照してください。
+- 過去イテレーションのレポートは `{ファイル名}.{タイムスタンプ}` 形式で同ディレクトリに保存されています（例: `architect-review.md.20260304T123456Z`）。各レポートについて `{レポート名}.*` パターンで Glob を実行し、タイムスタンプ降順で最大2件まで読み、persists / reopened の傾向を把握してから修正に入ること。
+
+**完了条件（以下をすべて満たすこと）:**
+- 今回受け取った全指摘（new / reopened）のうち、修正必須のものを修正したこと
+- 同一 `family_tag` の潜在箇所を同時に修正したこと（部分修正で再発させない）
+- `family_tag` ごとに最低1件の再発防止テストを追加したこと（config契約/境界チェック系は必須）
+- 複数レビュアーから同一 `family_tag` の指摘が届いている場合は1件に統合して対応したこと
+- 修正後に全差分を確認し、指摘や要求と無関係な変更を戻したこと
+
+**重要**: 修正後、ビルド（型チェック）とテストを必ず実行してください。
+
+**必須出力（見出しを含める）**
+## 作業結果
+- {実施内容の要約}
+## 指摘への対応
+- {修正必須、確認のみ、要求外の分類と対応}
+## 変更内容
+- {必須変更と関連変更の要約}
+## 戻した不要変更
+- {戻した変更。なければ「なし」}
+## ビルド結果
+- {ビルド実行結果}
+## テスト結果
+- {テスト実行コマンドと結果}
+## 収束ゲート
+| 指標 | 件数 |
+|------|------|
+| new（今回修正） | {N} |
+| reopened（再発修正） | {N} |
+| persists（前回から継続、今回未着手） | {N} |
+## 証拠
+- {確認したファイル/検索/差分/ログの要点を列挙}

package/builtins/ja/facets/instructions/implement-maintenance.md

@@ -0,0 +1,72 @@
+計画に従い、既存契約を保った最小差分で実装してください。
+Workflow Contextに示されたReport Directory内のファイルのみ参照してください。他のレポートディレクトリは検索/参照しないでください。
+Report Directory内のレポートを一次情報として参照してください。不足情報の補完が必要な場合に限り、Previous Responseや会話履歴を補助的に参照して構いません（Previous Responseは提供されない場合があります）。情報が競合する場合は、Report Directory内のレポートと実際のファイル内容を優先してください。
+
+**重要**: 実装と同時に単体テストを追加してください。
+- 新規作成したクラス・関数には単体テストを追加
+- 既存コードを変更した場合は該当するテストを更新。ただし既存期待値を実装都合で緩和しない
+- テストファイルの配置: プロジェクトの規約に従う
+- ビルド確認は必須。実装完了後、ビルド（型チェック）を実行し、型エラーがないことを確認
+- テスト実行は必須。ビルド成功後、必ずテストを実行して結果を確認
+- ファイル名・設定キー名などの契約文字列を新規導入する場合は、定数として1箇所で定義すること
+
+**保守変更の追加制約:**
+- 実装前に差分予定を「必須変更」「関連変更」「不要変更」に分類する
+- 必須変更と関連変更だけを実装する
+- 変更ファイル内という理由だけで、作法改善、改名、ファイル移動、hook戻り値変更、コメント削除、テスト期待値変更を行わない
+- 既存構造で要求を満たせる場合は、より一般的な作法へ寄せるためだけの構造変更をしない
+- 実装後に全差分を確認し、不要変更を戻す
+
+**Maintenance Scope出力契約（実装開始時に作成）:**
+```markdown
+# 保守変更スコープ
+
+## タスク
+{タスクの1行要約}
+
+## 必須変更
+| ファイル | 理由 | 要求との対応 |
+|----------|------|--------------|
+| {ファイル} | {理由} | {対応する要求} |
+
+## 関連変更
+| ファイル | 理由 | 必須変更との関係 |
+|----------|------|------------------|
+| {ファイル} | {理由} | {関係} |
+
+## 変更しない既存契約
+| 契約 | 対象 | 守る内容 |
+|------|------|----------|
+| {契約種別} | {対象} | {守る内容} |
+```
+
+**Decisions出力契約（実装完了時、決定がある場合のみ）:**
+```markdown
+# 決定ログ
+
+## 1. {決定内容}
+- **背景**: {なぜ決定が必要だったか}
+- **検討した選択肢**: {選択肢リスト}
+- **理由**: {選んだ理由}
+```
+
+**実装完了前の自己チェック（必須）:**
+
+ビルドとテストを実行する前に、次の手順で Policy の REJECT 基準を自己点検してください。
+
+1. Policy の Source Path を Read ツールで開き、全文を取得する
+2. 各 `##` セクションをすべて列挙する（取捨選択しない）
+3. 列挙した各セクションの REJECT 基準と自分の実装を照合する
+4. 全差分を確認し、要求外の改名、移動、コメント削除、UI文言変更、アクセシビリティ名変更、テスト期待値変更が残っていないか確認する
+
+**必須出力（見出しを含める）**
+## 作業結果
+- {実施内容の要約}
+## 変更内容
+- {必須変更と関連変更の要約}
+## 戻した不要変更
+- {戻した変更。なければ「なし」}
+## ビルド結果
+- {ビルド実行結果}
+## テスト結果
+- {テスト実行コマンドと結果}