takt 0.45.0 → 0.47.0

package/builtins/ja/facets/policies/testing.md

@@ -20,6 +20,9 @@
 | 新しい振る舞い | テスト必須。テストがなければ REJECT |
 | バグ修正 | リグレッションテスト必須。テストがなければ REJECT |
 | 振る舞いの変更 | テストの更新必須。更新がなければ REJECT |
+| 副作用・状態遷移の変更 | 正常系と代表的な失敗経路の検証必須。失敗経路が未検証なら REJECT |
+| 共通化・抽象化による契約変更 | 新しい共通経路だけでなく、既存の同種分岐で契約が保たれることの検証必須 |
+| パーサー・設定境界の変更 | 構文上は有効だが期待shapeと異なる入力、欠落値、個人設定からの汚染を検証必須 |
 | ビルド（型チェック） | ビルド成功必須。失敗は REJECT |
 | エッジケース・境界値 | テスト推奨（Warning） |
 
@@ -50,6 +53,18 @@
 
 非実行資産の変更確認は、必要に応じてレビュー、Markdown lint、リンクチェック、サンプルコマンドの実行で行う。
 
+## 置換された旧仕様のテスト
+
+仕様変更で旧設計の構成要素（UI・API・イベント・状態・ラベルなど）が新設計に置き換えられた場合、テストは新仕様の振る舞いを肯定的に検証する。旧仕様が存在しないことだけを固定しない。
+
+| 基準 | 判定 |
+|------|------|
+| 旧仕様の構成要素が存在・呼び出しされないことだけを検証するテストを追加 | REJECT |
+| 責務を失った実装単位に、旧仕様由来の不在確認テストを残す | REJECT |
+| 本番コードが最終的に変更されていないファイルで、旧仕様否定のためだけにテストを変更 | REJECT |
+| 新仕様を、新しい責務を持つ層（上位モジュール・サービス・統合フロー等）で肯定的に検証 | OK |
+| 旧仕様削除で未使用になったテストを削除し、新仕様の回帰テストへ置き換える | OK |
+
 ## テスト構造: Given-When-Then
 
 ```typescript
@@ -75,6 +90,40 @@ test('ユーザーが存在しない場合、NotFoundエラーを返す', async
 | 明確性 | 失敗時に原因が分かる | 失敗しても原因不明 |
 | 焦点 | 1テスト1概念 | 複数の関心事が混在 |
 
+## 副作用・状態遷移のテスト
+
+副作用や状態遷移を伴う変更は、正常系の成功だけでは十分に検証されたとは扱わない。
+
+| 基準 | 判定 |
+|------|------|
+| 正常系のみで、失敗・中断・早期終了時の状態を検証していない | REJECT |
+| 取得・開始・登録・反映した状態の後片付けや重複実行を検証していない | REJECT |
+| 共有状態や後続処理に影響する変更で、途中失敗後の再実行を検証していない | 警告。影響が主要経路なら REJECT |
+| モックで確認した範囲と未確認の実連携範囲を区別していない | 警告。主要要件なら REJECT |
+| 正常系、代表的な失敗経路、境界的な状態遷移をそれぞれ検証している | OK |
+
+## 契約変更と既存分岐のテスト
+
+共通ヘルパー、正規化関数、builder、adapter などで契約を統一する変更では、変更で追加した経路だけを検証しても十分ではない。既存の同種分岐が同じ契約を満たすことを検証する。
+
+| 基準 | 判定 |
+|------|------|
+| 新しい共通経路のテストだけで、既存の同種分岐に同じ戻り値・副作用・エラー契約が適用されることを検証していない | REJECT |
+| 既存分岐を「現行維持」としたが、差分で追加した契約と矛盾しないことを確認していない | REJECT |
+| 変更対象関数の return / throw / catch / early return を列挙せず、代表的な失敗経路を取りこぼしている | REJECT |
+| 同じ責務の既存分岐にも、戻り値・副作用・イベント・エラー分類の契約を検証するテストがある | OK |
+
+## パーサー・設定境界のテスト
+
+外部ファイル、設定、YAML/JSON、CLI入力などを読む境界では、型定義どおりの入力だけを検証しても十分ではない。
+
+| 基準 | 判定 |
+|------|------|
+| 配列前提のフィールドに object/null/欠落が入った場合の挙動を検証していない | REJECT |
+| ファイル/ディレクトリ判定で、存在する通常ファイル・壊れたリンク・権限エラーを検証していない | REJECT |
+| ユーザーやマシンの既存設定を引き継ぐ可能性があるテストで、空の設定ディレクトリや一時HOMEに隔離していない | REJECT |
+| 構文上有効だが契約外のshapeを、無視・正規化・明示エラーのいずれかに固定している | OK |
+
 ## テストデータとフィクスチャ
 
 テストデータは、テストごとに必要最小限の事実を明示して生成する。共有フィクスチャの破壊的変更や、実契約と異なるモックはテストの信頼性を下げる。

package/builtins/ja/provider-options/edit.yaml

@@ -0,0 +1,21 @@
+claude:
+  allowed_tools:
+    - Read
+    - Glob
+    - Grep
+    - Edit
+    - Write
+    - Bash
+    - WebSearch
+    - WebFetch
+opencode:
+  allowed_tools:
+    - read
+    - glob
+    - grep
+    - edit
+    - write
+    - bash
+    - todowrite
+    - websearch
+    - webfetch

package/builtins/ja/provider-options/review-files.yaml

@@ -0,0 +1,10 @@
+claude:
+  allowed_tools:
+    - Read
+    - Glob
+    - Grep
+opencode:
+  allowed_tools:
+    - read
+    - glob
+    - grep

package/builtins/ja/provider-options/review-readonly.yaml

@@ -0,0 +1,16 @@
+claude:
+  allowed_tools:
+    - Read
+    - Glob
+    - Grep
+    - Bash
+    - WebSearch
+    - WebFetch
+opencode:
+  allowed_tools:
+    - read
+    - glob
+    - grep
+    - bash
+    - websearch
+    - webfetch

package/builtins/ja/provider-options/review-web.yaml

@@ -0,0 +1,14 @@
+claude:
+  allowed_tools:
+    - Read
+    - Glob
+    - Grep
+    - WebSearch
+    - WebFetch
+opencode:
+  allowed_tools:
+    - read
+    - glob
+    - grep
+    - websearch
+    - webfetch

package/builtins/ja/workflow-categories.yaml

@@ -61,6 +61,7 @@ workflow_categories:
   🎵 TAKT開発:
     workflows:
       - takt-default
+      - takt-default-with-fc
       - takt-default-refresh-all
       - takt-default-refresh-fast
       - auto-improvement-loop

package/builtins/ja/workflows/audit-architecture-backend.yaml

@@ -23,6 +23,8 @@ loop_monitors:
 
 steps:
   - name: plan
+    tags:
+      - plan
     persona: planner
     knowledge:
       - backend
@@ -40,6 +42,8 @@ steps:
         next: ABORT
 
   - name: audit
+    tags:
+      - review
     persona: architecture-reviewer
     policy: review
     knowledge:
@@ -60,6 +64,8 @@ steps:
         next: supervise
 
   - name: supervise
+    tags:
+      - review
     persona: supervisor
     policy: review
     knowledge:
@@ -75,6 +81,8 @@ steps:
         next: review
 
   - name: review
+    tags:
+      - review
     persona: architecture-reviewer
     policy: review
     knowledge:

package/builtins/ja/workflows/audit-architecture-dual.yaml

@@ -23,6 +23,8 @@ loop_monitors:
 
 steps:
   - name: plan
+    tags:
+      - plan
     persona: planner
     knowledge:
       - frontend
@@ -41,6 +43,8 @@ steps:
         next: ABORT
 
   - name: audit
+    tags:
+      - review
     persona: architecture-reviewer
     policy: review
     knowledge:
@@ -62,6 +66,8 @@ steps:
         next: supervise
 
   - name: supervise
+    tags:
+      - review
     persona: supervisor
     policy: review
     knowledge:
@@ -78,6 +84,8 @@ steps:
         next: review
 
   - name: review
+    tags:
+      - review
     persona: architecture-reviewer
     policy: review
     knowledge:

package/builtins/ja/workflows/audit-architecture-frontend.yaml

@@ -23,6 +23,8 @@ loop_monitors:
 
 steps:
   - name: plan
+    tags:
+      - plan
     persona: planner
     knowledge:
       - frontend
@@ -41,6 +43,8 @@ steps:
         next: ABORT
 
   - name: audit
+    tags:
+      - review
     persona: frontend-reviewer
     policy: review
     knowledge:
@@ -62,6 +66,8 @@ steps:
         next: supervise
 
   - name: supervise
+    tags:
+      - review
     persona: supervisor
     policy: review
     knowledge:
@@ -78,6 +84,8 @@ steps:
         next: review
 
   - name: review
+    tags:
+      - review
     persona: frontend-reviewer
     policy: review
     knowledge:

package/builtins/ja/workflows/audit-architecture.yaml

@@ -23,6 +23,8 @@ loop_monitors:
 
 steps:
   - name: plan
+    tags:
+      - plan
     persona: planner
     knowledge: architecture
     instruction: architecture-audit-plan
@@ -38,6 +40,8 @@ steps:
         next: ABORT
 
   - name: audit
+    tags:
+      - review
     persona: architecture-reviewer
     policy: review
     knowledge: architecture
@@ -56,6 +60,8 @@ steps:
         next: supervise
 
   - name: supervise
+    tags:
+      - review
     persona: supervisor
     policy: review
     knowledge: architecture
@@ -69,6 +75,8 @@ steps:
         next: review
 
   - name: review
+    tags:
+      - review
     persona: architecture-reviewer
     policy: review
     knowledge: architecture

package/builtins/ja/workflows/audit-e2e.yaml

@@ -29,6 +29,8 @@ loop_monitors:
 
 steps:
   - name: plan
+    tags:
+      - plan
     persona: test-planner
     policy: testing
     knowledge:
@@ -47,6 +49,8 @@ steps:
         next: ABORT
 
   - name: audit
+    tags:
+      - review
     edit: false
     persona: testing-reviewer
     policy:
@@ -69,6 +73,8 @@ steps:
         next: supervise
 
   - name: supervise
+    tags:
+      - review
     edit: false
     persona: supervisor
     policy: review
@@ -83,6 +89,8 @@ steps:
         next: review
 
   - name: review
+    tags:
+      - review
     edit: false
     persona: testing-reviewer
     policy:

package/builtins/ja/workflows/audit-security.yaml

@@ -23,6 +23,8 @@ loop_monitors:
 
 steps:
   - name: plan
+    tags:
+      - plan
     persona: planner
     instruction: audit-security-plan
     edit: false
@@ -35,6 +37,8 @@ steps:
         next: audit
 
   - name: audit
+    tags:
+      - review
     persona: security-reviewer
     knowledge: security
     instruction: audit-security-team-leader
@@ -52,6 +56,8 @@ steps:
         next: supervise
 
   - name: supervise
+    tags:
+      - review
     persona: supervisor
     knowledge: security
     instruction: audit-security-supervise
@@ -63,6 +69,8 @@ steps:
         next: review
 
   - name: review
+    tags:
+      - review
     persona: security-reviewer
     knowledge: security
     instruction: audit-security-review

package/builtins/ja/workflows/audit-unit.yaml

@@ -29,6 +29,8 @@ loop_monitors:
 
 steps:
   - name: plan
+    tags:
+      - plan
     edit: false
     persona: test-planner
     policy: testing
@@ -49,6 +51,8 @@ steps:
         next: ABORT
 
   - name: audit
+    tags:
+      - review
     edit: false
     persona: testing-reviewer
     policy:
@@ -71,6 +75,8 @@ steps:
         next: supervise
 
   - name: supervise
+    tags:
+      - review
     edit: false
     persona: supervisor
     policy: review
@@ -85,6 +91,8 @@ steps:
         next: review
 
   - name: review
+    tags:
+      - review
     edit: false
     persona: testing-reviewer
     policy:

package/builtins/ja/workflows/auto-improvement-loop.yaml

@@ -51,6 +51,8 @@ steps:
         next: plan_fresh_improvement
 
   - name: plan_from_issue
+    tags:
+      - plan
     persona: supervisor
     knowledge:
       - architecture
@@ -136,6 +138,8 @@ steps:
         next: ABORT
 
   - name: plan_fresh_improvement
+    tags:
+      - plan
     persona: supervisor
     knowledge:
       - architecture
@@ -210,6 +214,8 @@ steps:
         next: ABORT
 
   - name: plan_from_existing_pr
+    tags:
+      - plan
     persona: supervisor
     knowledge:
       - architecture

package/builtins/ja/workflows/backend-cqrs-mini.yaml

@@ -23,6 +23,8 @@ loop_monitors:
           next: supervise_fix
 steps:
   - name: plan
+    tags:
+      - plan
     edit: false
     persona: planner
     knowledge:
@@ -52,6 +54,8 @@ steps:
         - name: plan.md
           format: plan
   - name: implement
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -91,8 +95,12 @@ steps:
         - name: coder-decisions.md
           format: coder-decisions
   - name: reviewers
+    tags:
+      - review
     parallel:
       - name: ai-antipattern-review-2nd
+        tags:
+          - review
         edit: false
         persona: ai-antipattern-reviewer
         policy:
@@ -115,6 +123,8 @@ steps:
             - name: ai-antipattern-review.md
               format: ai-antipattern-review
       - name: supervise
+        tags:
+          - review
         edit: false
         persona: supervisor
         policy: review
@@ -153,8 +163,12 @@ steps:
       - condition: any("要求未達成、テスト失敗、ビルドエラー")
         next: supervise_fix
   - name: fix_both
+    tags:
+      - coding
     parallel:
       - name: ai-antipattern-fix-parallel
+        tags:
+          - coding
         edit: true
         persona: coder
         policy:
@@ -182,6 +196,8 @@ steps:
           - condition: 判断できない、情報不足
         instruction: ai-antipattern-fix
       - name: supervise_fix_parallel
+        tags:
+          - coding
         edit: true
         persona: coder
         policy:
@@ -213,6 +229,8 @@ steps:
       - condition: any("修正不要（指摘対象ファイル/仕様の確認済み）", "判断できない、情報不足", "修正を進行できない")
         next: implement
   - name: ai-antipattern-fix
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -245,6 +263,8 @@ steps:
         next: implement
     instruction: ai-antipattern-fix
   - name: supervise_fix
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:

package/builtins/ja/workflows/backend-cqrs.yaml

@@ -35,6 +35,8 @@ loop_monitors:
           next: supervise
 steps:
   - name: plan
+    tags:
+      - plan
     edit: false
     persona: planner
     knowledge:
@@ -61,6 +63,8 @@ steps:
         - name: plan.md
           format: plan
   - name: write_tests
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -101,6 +105,8 @@ steps:
         - name: test-report.md
           format: test-report
   - name: implement
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -142,6 +148,8 @@ steps:
         - name: coder-decisions.md
           format: coder-decisions
   - name: ai-antipattern-review-1st
+    tags:
+      - review
     edit: false
     persona: ai-antipattern-reviewer
     policy:
@@ -166,6 +174,8 @@ steps:
         - name: ai-antipattern-review.md
           format: ai-antipattern-review
   - name: ai-antipattern-fix
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -198,6 +208,8 @@ steps:
       - condition: 修正を進行できない
         next: ai-antipattern-no-fix
   - name: ai-antipattern-no-fix
+    tags:
+      - review
     edit: false
     persona: architecture-reviewer
     policy: review
@@ -214,8 +226,12 @@ steps:
         next: reviewers_1
     instruction: arbitrate
   - name: reviewers_1
+    tags:
+      - review
     parallel:
       - name: cqrs-es-review
+        tags:
+          - review
         edit: false
         persona: cqrs-es-reviewer
         policy: review
@@ -239,6 +255,8 @@ steps:
             - name: cqrs-es-review.md
               format: cqrs-es-review
       - name: testing-review
+        tags:
+          - review
         edit: false
         persona: testing-reviewer
         policy:
@@ -264,6 +282,8 @@ steps:
             - name: testing-review.md
               format: testing-review
       - name: ai-antipattern-review-2nd
+        tags:
+          - review
         edit: false
         persona: ai-antipattern-reviewer
         policy:
@@ -291,8 +311,12 @@ steps:
       - condition: any("needs_fix")
         next: fix
   - name: reviewers_2
+    tags:
+      - review
     parallel:
       - name: security-review
+        tags:
+          - review
         edit: false
         persona: security-reviewer
         policy: review
@@ -314,6 +338,8 @@ steps:
             - name: security-review.md
               format: security-review
       - name: qa-review
+        tags:
+          - review
         edit: false
         persona: qa-reviewer
         policy:
@@ -335,10 +361,37 @@ steps:
           report:
             - name: qa-review.md
               format: qa-review
+      - name: pure-review
+        tags:
+          - review
+        edit: false
+        persona: pure-reviewer
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - Bash
+              - WebSearch
+              - WebFetch
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-pure
+        output_contracts:
+          report:
+            - name: pure-review.md
+              format: pure-review
       - name: coding-review
+        tags:
+          - review
         edit: false
         persona: coding-reviewer
-        policy: review
+        policy:
+          - review
+          - coding
         provider_options:
           claude:
             allowed_tools:
@@ -362,6 +415,8 @@ steps:
       - condition: any("needs_fix")
         next: fix
   - name: fix
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -392,6 +447,8 @@ steps:
         next: plan
     instruction: fix
   - name: supervise
+    tags:
+      - review
     edit: false
     persona: dual-supervisor
     policy: review
@@ -418,6 +475,8 @@ steps:
           format: summary
           use_judge: false
   - name: fix_supervisor
+    tags:
+      - coding
     edit: true
     persona: coder
     policy: