takt 0.44.0 → 0.46.0

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

@@ -129,6 +129,27 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: pure-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
         edit: false
         persona: coding-reviewer

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

@@ -141,6 +141,27 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: pure-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
         edit: false
         persona: coding-reviewer

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

@@ -117,6 +117,27 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: pure-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
         edit: false
         persona: coding-reviewer

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, coding — 6 parallel reviewers with iterative fixes)
+description: Multi-perspective review + fix loop (architecture, security, QA, testing, pure review, coding — 6 parallel reviewers with iterative fixes)
 workflow_config:
   provider_options:
     codex:
@@ -149,9 +149,9 @@ steps:
           - condition: approved
           - condition: needs_fix
 
-      - name: requirements-review
+      - name: pure-review
         edit: false
-        persona: requirements-reviewer
+        persona: pure-reviewer
         policy: review
         provider_options:
           claude:
@@ -162,11 +162,11 @@ steps:
               - Bash
               - WebSearch
               - WebFetch
-        instruction: review-requirements
+        instruction: review-pure
         output_contracts:
           report:
-            - name: requirements-review.md
-              format: requirements-review
+            - name: pure-review.md
+              format: pure-review
         rules:
           - condition: approved
           - condition: needs_fix

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

@@ -166,6 +166,27 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: pure-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
         edit: false
         persona: coding-reviewer

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

@@ -142,6 +142,27 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: pure-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
         edit: false
         persona: coding-reviewer

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

@@ -141,6 +141,27 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: pure-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
         edit: false
         persona: coding-reviewer

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

@@ -187,9 +187,9 @@ steps:
           - condition: approved
           - condition: needs_fix
 
-      - name: requirements-review
+      - name: pure-review
         edit: false
-        persona: requirements-reviewer
+        persona: pure-reviewer
         policy: review
         provider_options:
           claude:
@@ -200,11 +200,11 @@ steps:
               - Bash
               - WebSearch
               - WebFetch
-        instruction: review-requirements
+        instruction: review-pure
         output_contracts:
           report:
-            - name: requirements-review.md
-              format: requirements-review
+            - name: pure-review.md
+              format: pure-review
         rules:
           - condition: approved
           - condition: needs_fix

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

@@ -128,6 +128,27 @@ steps:
           - condition: approved
           - condition: needs_fix
 
+      - name: pure-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
         edit: false
         persona: coding-reviewer

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

@@ -162,9 +162,9 @@ steps:
           - condition: approved
           - condition: needs_fix
 
-      - name: requirements-review
+      - name: pure-review
         edit: false
-        persona: requirements-reviewer
+        persona: pure-reviewer
         policy: review
         provider_options:
           claude:
@@ -175,11 +175,11 @@ steps:
               - Bash
               - WebSearch
               - WebFetch
-        instruction: review-requirements
+        instruction: review-pure
         output_contracts:
           report:
-            - name: requirements-review.md
-              format: requirements-review
+            - name: pure-review.md
+              format: pure-review
         rules:
           - condition: approved
           - condition: needs_fix
@@ -243,7 +243,7 @@ steps:
          - `03-qa-review.md` (QA review)
          - `04-testing-review.md` (Testing review)
          - `05-ai-antipattern-review.md` (AI antipattern review)
-         - `06-requirements-review.md` (Requirements review)
+         - `06-pure-review.md` (Requirements review)
          - `07-coding-review.md` (Coding review)
       2. Synthesize the 7 review results
       3. Produce a consolidated review summary with overall verdict

package/builtins/en/workflows/takt-default-refresh-all.yaml

@@ -1,5 +1,5 @@
 name: takt-default-refresh-all
-description: "TAKT development workflow (all-step session refresh comparison: plan → write tests → team-leader implement → AI antipattern review → 5-parallel review → fix → supervise → complete)"
+description: "TAKT development workflow (all-step session refresh comparison: plan → write tests → team-leader implement → AI antipattern review → parallel review → fix → supervise → complete)"
 workflow_config:
   provider_options:
     codex:
@@ -333,9 +333,9 @@ steps:
           report:
             - name: testing-review.md
               format: testing-review
-      - name: requirements-review
+      - name: pure-review
         edit: false
-        persona: requirements-reviewer
+        persona: pure-reviewer
         session: refresh
         policy: review
         provider_options:
@@ -350,11 +350,11 @@ steps:
         rules:
           - condition: approved
           - condition: needs_fix
-        instruction: review-requirements
+        instruction: review-pure
         output_contracts:
           report:
-            - name: requirements-review.md
-              format: requirements-review
+            - name: pure-review.md
+              format: pure-review
       - name: ai-antipattern-review-2nd
         edit: false
         persona: ai-antipattern-reviewer

package/builtins/en/workflows/takt-default-refresh-fast.yaml

@@ -331,9 +331,9 @@ steps:
           report:
             - name: testing-review.md
               format: testing-review
-      - name: requirements-review
+      - name: pure-review
         edit: false
-        persona: requirements-reviewer
+        persona: pure-reviewer
         session: refresh
         policy: review
         provider_options:
@@ -348,11 +348,11 @@ steps:
         rules:
           - condition: approved
           - condition: needs_fix
-        instruction: review-requirements
+        instruction: review-pure
         output_contracts:
           report:
-            - name: requirements-review.md
-              format: requirements-review
+            - name: pure-review.md
+              format: pure-review
       - name: ai-antipattern-review-2nd
         edit: false
         persona: ai-antipattern-reviewer

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

@@ -1,5 +1,5 @@
 name: takt-default
-description: TAKT development workflow (plan -> write_tests -> draft (implement + AI self-review) -> peer-review (5 parallel reviewers + fix) -> supervise -> COMPLETE).
+description: TAKT development workflow (plan -> write_tests -> draft (implement + AI self-review) -> peer-review (parallel reviewers + fix) -> supervise -> COMPLETE).
 workflow_config:
   provider_options:
     codex:
@@ -14,14 +14,7 @@ steps:
     persona: planner
     knowledge: architecture
     provider_options:
-      claude:
-        allowed_tools:
-          - Read
-          - Glob
-          - Grep
-          - Bash
-          - WebSearch
-          - WebFetch
+      $ref: provider-options/review-readonly.yaml
     rules:
       - condition: Requirements are clear and implementable
         next: write_tests
@@ -50,16 +43,7 @@ steps:
       - unit-testing
       - e2e-testing
     provider_options:
-      claude:
-        allowed_tools:
-          - Read
-          - Glob
-          - Grep
-          - Edit
-          - Write
-          - Bash
-          - WebSearch
-          - WebFetch
+      $ref: provider-options/edit.yaml
     required_permission_mode: edit
     instruction: write-tests-first
     rules:
@@ -103,14 +87,7 @@ steps:
     policy: review
     knowledge: architecture
     provider_options:
-      claude:
-        allowed_tools:
-          - Read
-          - Glob
-          - Grep
-          - Bash
-          - WebSearch
-          - WebFetch
+      $ref: provider-options/review-readonly.yaml
     pass_previous_response: false
     rules:
       - condition: All clear

package/builtins/en/workflows/terraform.yaml

@@ -119,6 +119,27 @@ steps:
           report:
             - name: ai-antipattern-review.md
               format: ai-antipattern-review
+      - name: pure-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
         edit: false
         persona: coding-reviewer
@@ -161,11 +182,11 @@ steps:
           - WebFetch
     instruction: supervise
     rules:
-      - condition: All reviewers approved and task requirements met
+      - condition: All reviewers approved and mergeable quality
         next: COMPLETE
       - condition: Only AI-specific issues need fixing
         next: ai-antipattern-fix
-      - condition: Only convention violations or requirements unmet
+      - condition: Only convention violations or mergeability issues need fixing
         next: supervise_fix
       - condition: Multiple types of issues found
         next: fix_both

package/builtins/ja/config.yaml

@@ -46,6 +46,15 @@ language: ja         # 表示言語: ja | en
 #   provider_events: false         # providerイベントログを記録
 #   usage_events: false            # 使用量イベントログを記録
 
+# OpenTelemetry observability
+# observability:
+#   enabled: true                  # OpenTelemetry SDK を初期化
+#   monitor: true                  # .takt/runs/<run>/monitor.json を出力
+#   session_log_exporter: true     # span 由来の shadow session log を出力
+#   usage_events_phase: true       # phase 粒度の usage event log を出力
+# OTEL_EXPORTER_OTLP_ENDPOINT を設定すると span と metric も OTLP HTTP で送信します。
+# 任意の OTLP 上書き: OTEL_EXPORTER_OTLP_TRACES_ENDPOINT, OTEL_EXPORTER_OTLP_METRICS_ENDPOINT.
+
 # アナリティクス
 # analytics:
 #   enabled: true                  # ローカルアナリティクス収集を有効化

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

@@ -15,6 +15,7 @@ Report Directory内のレポートを一次情報として参照してくださ
 - 必須変更と関連変更だけを実装する
 - 変更ファイル内という理由だけで、作法改善、改名、ファイル移動、hook戻り値変更、コメント削除、テスト期待値変更を行わない
 - 既存構造で要求を満たせる場合は、より一般的な作法へ寄せるためだけの構造変更をしない
+- 仕様変更で廃止された旧設計について、旧仕様の不在確認だけのコード・テストを残さない
 - 実装後に全差分を確認し、不要変更を戻す
 
 **Maintenance Scope出力契約（実装開始時に作成）:**
@@ -58,6 +59,7 @@ Report Directory内のレポートを一次情報として参照してくださ
 2. 各 `##` セクションをすべて列挙する（取捨選択しない）
 3. 列挙した各セクションの REJECT 基準と自分の実装を照合する
 4. 全差分を確認し、要求外の改名、移動、コメント削除、UI文言変更、アクセシビリティ名変更、テスト期待値変更が残っていないか確認する
+5. 仕様変更で旧設計を置換した場合、旧設計の不在確認だけのコード・テストが残っていないか確認する
 
 **必須出力（見出しを含める）**
 ## 作業結果

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

@@ -9,6 +9,7 @@ Report Directory内のレポートを一次情報として参照してくださ
 - ビルド確認は必須。実装完了後、ビルド（型チェック）を実行し、型エラーがないことを確認
 - テスト実行は必須。ビルド成功後、必ずテストを実行して結果を確認
 - ファイル名・設定キー名などの契約文字列を新規導入する場合は、定数として1箇所で定義すること
+- 境界変更（権限、拒否、外部実行、共有状態、状態遷移）を含む場合は、変更した境界・主要な状態軸・期待する振る舞い・対応する検証を簡潔な表で示すこと
 
 **Scope出力契約（実装開始時に作成）:**
 ```markdown
@@ -41,9 +42,7 @@ Small / Medium / Large
 ```
 
 **実装完了前の自己チェック（必須）:**
-
 ビルドとテストを実行する前に、次の手順で Policy の REJECT 基準を自己点検してください。
-
 1. Policy の Source Path を Read ツールで開き、全文を取得する
 2. 各 `##` セクションをすべて列挙する（取捨選択しない）
 3. 列挙した各セクションの REJECT 基準と自分の実装を照合する
@@ -57,3 +56,5 @@ Small / Medium / Large
 - {ビルド実行結果}
 ## テスト結果
 - {テスト実行コマンドと結果}
+## 境界変更チェック
+- {境界変更がある場合: 変更した境界、状態軸、期待する振る舞い、対応する検証。該当なしの場合は「該当なし」}

package/builtins/ja/facets/instructions/review-arch.md

@@ -1,5 +1,5 @@
 **アーキテクチャと設計**のレビューに集中してください。
-AI 特有の問題はレビューしないでください（ai-antipattern-review-1st ステップで実施済み）。
+他のレビュアーや別ステップの有無を前提にせず、このレビューの観点で確認すべき問題を検出してください。
 
 手順:
 1. Knowledge と Policy の Source Path を Read ツールで開き、全文を取得する

package/builtins/ja/facets/instructions/review-coding.md

@@ -3,6 +3,7 @@
 手順:
 1. タスクの意図、計画、変更差分、実行証跡を確認する
 2. 実装上のバグ、既存動作のリグレッション、セキュリティリスク、テスト不足を探す
-3. 指摘は現在の差分に起因し、ユーザーが修正すべきものだけにする
-4. 各指摘には場所、影響、修正方針を含める
-5. 根拠のない推測、好みだけの変更、今回の差分と無関係な既存問題は指摘しない
+3. 副作用や状態変更を伴う差分は、入口・正常終了・早期終了・例外・後片付けの経路を追う
+4. 指摘は現在の差分に起因し、ユーザーが修正すべきものだけにする
+5. 各指摘には場所、影響、修正方針を含める
+6. 根拠のない推測、好みだけの変更、今回の差分と無関係な既存問題は指摘しない

package/builtins/ja/facets/instructions/review-cqrs-es.md

@@ -1,5 +1,5 @@
 **CQRS（コマンドクエリ責務分離）と Event Sourcing（イベントソーシング）**のレビューに集中してください。
-AI 特有の問題はレビューしないでください（ai-antipattern-review-1st ステップで実施済み）。
+他のレビュアーや別ステップの有無を前提にせず、このレビューの観点で確認すべき問題を検出してください。
 
 手順:
 1. Knowledge と Policy の Source Path を Read ツールで開き、全文を取得する

package/builtins/ja/facets/instructions/review-pure.md

@@ -0,0 +1 @@
+今回の修正がマージ可能な品質かレビューしてください。

package/builtins/ja/facets/instructions/review-qa.md

@@ -4,3 +4,4 @@
 1. Knowledge と Policy の Source Path を Read ツールで開き、全文を取得する
 2. それぞれの `##` セクションをすべて列挙する（取捨選択しない）
 3. 列挙した各セクションの判定基準を変更差分と照合し、該当する問題を検出する
+4. 検証済みの範囲と未確認の範囲を分け、未確認の主要経路を動作確認済みとして扱わない

package/builtins/ja/facets/instructions/review-terraform.md

@@ -1,5 +1,5 @@
 **Terraform 規約準拠**のレビューに集中してください。
-AI 特有の問題はレビューしないでください（ai-antipattern-review-1st ステップで実施済み）。
+他のレビュアーや別ステップの有無を前提にせず、このレビューの観点で確認すべき問題を検出してください。
 
 手順:
 1. Knowledge と Policy の Source Path を Read ツールで開き、全文を取得する

package/builtins/ja/facets/instructions/review-test.md

@@ -9,3 +9,5 @@
 
 1. Report Directory 内のテスト計画・テストスコープに関するレポートと実装されたテストを突合する
 2. 外部契約があるのに入力位置（root body / query / path）の検証が欠けている場合、カバレッジ不足として扱う
+3. 副作用や状態遷移を伴う変更では、正常系だけでなく代表的な失敗経路のテスト有無を確認する
+4. モックで代替した検証は、実連携まで確認したものとして扱わない

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

@@ -13,11 +13,13 @@
 2. 各要件について、実装されたコード（ファイル:行）を特定する
 3. コードが要件を満たしていることを実際に確認する（ファイルを読む、実行済みテストやビルドの証跡を確認する）
    - 片側のケースだけを確認して複合要件全体を ✅ にしてはいけない
-   - 計画レポートや requirements-review の判断を鵜呑みにせず、要件ごとに独立照合する
+   - 計画レポートや pure-review の判断を鵜呑みにせず、マージ可否を独立検証する
    - 充足していない要件が 1 つでもあれば REJECT する
 4. 保守変更スコープを検証する
    - 必須変更、関連変更、不要変更の分類が妥当か確認する
    - コメント、型名、ファイル配置、UI文言、アクセシビリティ名、テスト期待値が要求外で変わっていないか確認する
+   - 仕様変更で旧設計を置換した場合、旧仕様を否定するだけのコード・テストが残っていないか確認する
+   - 本番コードの最終差分がない対象で、テストだけが旧仕様の不在確認として変わっていないか確認する
    - 一般的な品質改善や作法改善だけが理由の差分が残っていれば REJECT する
 5. 前段レビューの finding を再評価する
    - finding がコード上成立しない場合は false positive として記録する
@@ -28,7 +30,7 @@
 
 - 要約レポートは一次証跡としては扱わない。一次証跡は、実行結果を記録したレポート、具体的な確認内容を記したレビューレポート、実コードの順で確認する
 - 実行結果を記したレポート内の `ビルド結果` / `テスト結果` は一次証跡として扱ってよい
-- `architecture-review` / `qa-review` / `testing-review` / `security-review` / `requirements-review` では、各レポートの `検証証跡` セクションを優先して確認する
+- `architecture-review` / `qa-review` / `testing-review` / `security-review` / `pure-review` では、各レポートの `検証証跡` セクションを優先して確認する
 - `検証証跡` の各項目は、確認対象・確認内容・結果が揃っている場合のみ補助証跡として扱う。いずれかが欠ける場合は、その項目を `未確認` として扱う
 - 証跡同士が矛盾する場合は、`実行結果を記したレポート > 具体的な確認内容を持つレビューレポート > 要約レポート` の優先順位で判断する
 
@@ -50,7 +52,7 @@
 - ❌ が1件でもある場合は REJECT 必須
 - 根拠なしの ✅ は無効（実コードで確認すること）
 - 一部ケースしか確認できていない場合は ✅ にしない
-- 計画レポートの判断を鵜呑みにせず、要件ごとに独立照合する
+- 計画レポートの判断を鵜呑みにせず、マージ可否を独立検証する
 
 ## 保守スコープチェック
 
@@ -71,7 +73,7 @@
 
 - 前段レビューの結論と最終判定が異なる場合は、必ず根拠を書く
 - false_positive / overreach を判定した場合は、task と plan のどちらに照らして不適切か明記する
-- requirements-review の結論を覆す場合は、その理由を必ず根拠付きで明記する
+- pure-review の結論を覆す場合は、その理由を必ず根拠付きで明記する
 
 ## 検証サマリー
 | 項目 | 状態 | 確認方法 |