takt 0.47.0 → 0.48.0

package/builtins/en/workflows/backend-maintenance.yaml

@@ -0,0 +1,540 @@
+name: backend-maintenance
+description: Strict backend maintenance workflow for existing production systems
+workflow_config:
+  provider_options:
+    codex:
+      network_access: true
+    opencode:
+      network_access: true
+max_steps: 60
+initial_step: plan
+loop_monitors:
+  - cycle:
+      - ai-antipattern-review-1st
+      - ai-antipattern-fix
+    threshold: 3
+    judge:
+      persona: supervisor
+      instruction: loop-monitor-ai-antipattern-fix
+      rules:
+        - condition: Healthy (making progress)
+          next: ai-antipattern-review-1st
+        - condition: Unproductive (no improvement)
+          next: reviewers_1
+  - cycle:
+      - reviewers_1
+      - fix
+    threshold: 3
+    judge:
+      persona: supervisor
+      instruction: loop-monitor-reviewers-fix
+      rules:
+        - condition: converging (findings decreasing, fixes applied)
+          next: reviewers_1
+        - condition: unproductive (same findings repeating)
+          next: supervise
+steps:
+  - name: plan
+    tags:
+      - plan
+    edit: false
+    persona: planner
+    policy:
+      - existing-system-respect
+    knowledge:
+      - backend
+      - architecture
+      - existing-system
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+          - Bash
+          - WebSearch
+          - WebFetch
+    instruction: plan-maintenance
+    rules:
+      - condition: Task analysis and planning is complete
+        next: write_tests
+      - condition: Requirements are unclear and planning cannot proceed
+        next: ABORT
+    output_contracts:
+      report:
+        - name: plan.md
+          format: plan
+  - name: write_tests
+    tags:
+      - coding
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+      - existing-system-respect
+    knowledge:
+      - backend
+      - architecture
+      - unit-testing
+      - e2e-testing
+      - existing-system
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+          - Edit
+          - Write
+          - Bash
+          - WebSearch
+          - WebFetch
+    required_permission_mode: edit
+    instruction: write-tests-maintenance
+    rules:
+      - condition: Test creation is complete
+        next: implement
+      - condition: Skipping test creation as target is not yet implemented
+        next: implement
+      - condition: Cannot proceed with test creation
+        next: plan
+      - condition: User input required for clarification
+        next: write_tests
+        requires_user_input: true
+        interactive_only: true
+    output_contracts:
+      report:
+        - name: test-report.md
+          format: test-report
+  - name: implement
+    tags:
+      - coding
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+      - existing-system-respect
+    session: refresh
+    knowledge:
+      - backend
+      - security
+      - architecture
+      - existing-system
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+          - Edit
+          - Write
+          - Bash
+          - WebSearch
+          - WebFetch
+    instruction: implement-maintenance
+    rules:
+      - condition: Implementation is complete
+        next: ai-antipattern-review-1st
+      - condition: No implementation (report only)
+        next: ai-antipattern-review-1st
+      - condition: Cannot proceed with implementation
+        next: ai-antipattern-review-1st
+      - condition: User input required
+        next: implement
+        requires_user_input: true
+        interactive_only: true
+    output_contracts:
+      report:
+        - name: maintenance-scope.md
+          format: maintenance-scope
+        - name: coder-decisions.md
+          format: coder-decisions
+  - name: ai-antipattern-review-1st
+    tags:
+      - review
+    edit: false
+    persona: ai-antipattern-reviewer
+    policy:
+      - review
+      - ai-antipattern
+      - existing-system-respect
+    knowledge:
+      - existing-system
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+          - WebSearch
+          - WebFetch
+    instruction: ai-antipattern-review
+    rules:
+      - condition: No AI-specific issues found
+        next: reviewers_1
+      - condition: AI-specific issues detected
+        next: ai-antipattern-fix
+    output_contracts:
+      report:
+        - name: ai-antipattern-review.md
+          format: ai-antipattern-review
+  - name: ai-antipattern-fix
+    tags:
+      - coding
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+      - existing-system-respect
+    session: refresh
+    knowledge:
+      - backend
+      - security
+      - architecture
+      - existing-system
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+          - Edit
+          - Write
+          - Bash
+          - WebSearch
+          - WebFetch
+    instruction: ai-antipattern-fix
+    pass_previous_response: false
+    rules:
+      - condition: AI Reviewer's issues have been fixed
+        next: ai-antipattern-review-1st
+      - condition: No fix needed (verified target files/spec)
+        next: ai-antipattern-no-fix
+      - condition: Unable to proceed with fixes
+        next: ai-antipattern-no-fix
+  - name: ai-antipattern-no-fix
+    tags:
+      - review
+    edit: false
+    persona: architecture-reviewer
+    policy:
+      - review
+      - existing-system-respect
+    knowledge:
+      - existing-system
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+    rules:
+      - condition: ai-antipattern-review-1st's findings are valid (fix required)
+        next: ai-antipattern-fix
+      - condition: ai-antipattern-fix's judgment is valid (no fix needed)
+        next: reviewers_1
+    instruction: arbitrate
+  - name: reviewers_1
+    tags:
+      - review
+    parallel:
+      - name: arch-review
+        tags:
+          - review
+        edit: false
+        persona: architecture-reviewer
+        policy:
+          - review
+          - existing-system-respect
+        knowledge:
+          - architecture
+          - backend
+          - existing-system
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - WebSearch
+              - WebFetch
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-arch
+        output_contracts:
+          report:
+            - name: architect-review.md
+              format: architecture-review
+      - name: testing-review
+        tags:
+          - review
+        edit: false
+        persona: testing-reviewer
+        policy:
+          - review
+          - testing
+          - existing-system-respect
+        knowledge:
+          - unit-testing
+          - e2e-testing
+          - existing-system
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - WebSearch
+              - WebFetch
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-test
+        output_contracts:
+          report:
+            - name: testing-review.md
+              format: testing-review
+      - name: ai-antipattern-review-2nd
+        tags:
+          - review
+        edit: false
+        persona: ai-antipattern-reviewer
+        policy:
+          - review
+          - ai-antipattern
+          - existing-system-respect
+        knowledge:
+          - existing-system
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - WebSearch
+              - WebFetch
+        instruction: ai-antipattern-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        output_contracts:
+          report:
+            - name: ai-antipattern-review.md
+              format: ai-antipattern-review
+    rules:
+      - condition: all("approved")
+        next: reviewers_2
+      - condition: any("needs_fix")
+        next: fix
+  - name: reviewers_2
+    tags:
+      - review
+    parallel:
+      - name: security-review
+        tags:
+          - review
+        edit: false
+        persona: security-reviewer
+        policy:
+          - review
+          - existing-system-respect
+        knowledge:
+          - security
+          - existing-system
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - WebSearch
+              - WebFetch
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-security
+        output_contracts:
+          report:
+            - name: security-review.md
+              format: security-review
+      - name: qa-review
+        tags:
+          - review
+        edit: false
+        persona: qa-reviewer
+        policy:
+          - review
+          - qa
+          - existing-system-respect
+        knowledge:
+          - existing-system
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - WebSearch
+              - WebFetch
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-qa
+        output_contracts:
+          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
+          - coding
+        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
+      - condition: any("needs_fix")
+        next: fix
+  - name: fix
+    tags:
+      - coding
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+      - existing-system-respect
+    knowledge:
+      - backend
+      - security
+      - architecture
+      - existing-system
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+          - Edit
+          - Write
+          - Bash
+          - WebSearch
+          - WebFetch
+    required_permission_mode: edit
+    pass_previous_response: false
+    rules:
+      - condition: Fix complete
+        next: reviewers_1
+      - condition: Cannot proceed, insufficient info
+        next: plan
+    instruction: fix-maintenance
+  - name: supervise
+    tags:
+      - review
+    edit: false
+    persona: dual-supervisor
+    policy:
+      - review
+      - existing-system-respect
+    knowledge:
+      - existing-system
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+          - WebSearch
+          - WebFetch
+    instruction: supervise-maintenance
+    pass_previous_response: false
+    rules:
+      - condition: All validations pass and ready to merge
+        next: COMPLETE
+      - condition: Issues detected during final review
+        next: fix_supervisor
+    output_contracts:
+      report:
+        - name: supervisor-validation.md
+          format: supervisor-validation
+        - name: summary.md
+          format: summary
+          use_judge: false
+  - name: fix_supervisor
+    tags:
+      - coding
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+      - existing-system-respect
+    knowledge:
+      - backend
+      - security
+      - architecture
+      - existing-system
+    provider_options:
+      claude:
+        allowed_tools:
+          - Read
+          - Glob
+          - Grep
+          - Edit
+          - Write
+          - Bash
+          - WebSearch
+          - WebFetch
+    instruction: fix-maintenance
+    pass_previous_response: false
+    rules:
+      - condition: Supervisor's issues have been fixed
+        next: supervise
+      - condition: Unable to proceed with fixes
+        next: plan

package/builtins/en/workflows/draft.yaml

@@ -43,7 +43,7 @@ loop_monitors:
 steps:
   - name: implement
     tags:
-      - coding
+      - leader
     edit: true
     persona: coder
     policy:
@@ -57,6 +57,8 @@ steps:
     required_permission_mode: edit
     team_leader:
       max_parts: 2
+      part_tags:
+        - coding
       part_persona: coder
       part_edit: true
       part_permission_mode: edit

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

@@ -6,7 +6,7 @@ workflow_config:
       network_access: true
     opencode:
       network_access: true
-max_steps: 30
+max_steps: 60
 initial_step: plan
 loop_monitors:
   - cycle:

package/builtins/en/workflows/frontend.yaml

@@ -6,7 +6,7 @@ workflow_config:
       network_access: true
     opencode:
       network_access: true
-max_steps: 30
+max_steps: 60
 initial_step: plan
 loop_monitors:
   - cycle:

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

@@ -1,4 +1,4 @@
-Report Directory内のレビューレポートを確認し、指摘事項を既存契約を保つ最小差分で修正してください。
+Report Directory内のレビューレポートを確認し、指摘事項を既存契約を保ちつつ、要求と因果関係のある範囲で修正してください。
 
 **修正の原則:**
 - 指摘の「修正案」が示されている場合はそれに沿った対応を優先し、独自の迂回策を取らない

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

@@ -1,4 +1,4 @@
-計画に従い、既存契約を保った最小差分で実装してください。
+計画に従い、既存契約を保ちつつ、要求と因果関係のある範囲で実装してください。
 Workflow Contextに示されたReport Directory内のファイルのみ参照してください。他のレポートディレクトリは検索/参照しないでください。
 Report Directory内のレポートを一次情報として参照してください。不足情報の補完が必要な場合に限り、Previous Responseや会話履歴を補助的に参照して構いません（Previous Responseは提供されない場合があります）。情報が競合する場合は、Report Directory内のレポートと実際のファイル内容を優先してください。
 
@@ -14,7 +14,7 @@ Report Directory内のレポートを一次情報として参照してくださ
 - 実装前に差分予定を「必須変更」「関連変更」「不要変更」に分類する
 - 必須変更と関連変更だけを実装する
 - 変更ファイル内という理由だけで、作法改善、改名、ファイル移動、hook戻り値変更、コメント削除、テスト期待値変更を行わない
-- 既存構造で要求を満たせる場合は、より一般的な作法へ寄せるためだけの構造変更をしない
+- 要求と因果関係のない構造変更をしない
 - 仕様変更で廃止された旧設計について、旧仕様の不在確認だけのコード・テストを残さない
 - 実装後に全差分を確認し、不要変更を戻す
 

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

@@ -1,4 +1,4 @@
-既存フロントエンド機能の保守変更としてタスクを分析し、設計を含めた最小差分の実装方針を立ててください。
+既存機能の保守変更としてタスクを分析し、設計を含めた因果差分の実装方針を立ててください。
 
 **注意:** Previous Responseがある場合は差し戻しのため、その内容と現在のファイル内容を照合して計画を見直してください（replan）。
 
@@ -29,7 +29,7 @@
    - 変更ファイル内、近い責務、一般作法という理由だけでは関連変更にしない
    - 不要変更は Coder に実装させない
 6. ファイル構成・設計パターンを決定する（必要な場合）
-   - 既存構造で要求を満たせる場合は、理想的でなくても既存構造を優先する
+   - 要求と因果関係のない範囲では、理想的でなくても既存構造を優先する
 7. 実装アプローチを決める
    - 実装アプローチがナレッジ・ポリシーの制約に違反しないか照合する
    - 利用者向け機能の追加や変更がある場合、利用者がその機能へ到達する条件・入口・起動経路を固定する
@@ -48,4 +48,4 @@
 ## 既存契約
 - {守るべき既存契約}
 ## 実装方針
-- {最小差分の方針}
+- {因果差分の方針}

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

@@ -8,7 +8,9 @@
 5. 共通ヘルパー、正規化関数、builder、adapter の追加・変更がある場合、既存の同種分岐にも同じ契約が適用されているか確認する
 6. 型、スキーマ、validator、resolver のいずれかが変わった場合、対応する契約も同じ変更で整合しているか確認する
 7. 複数レイヤーで解決・合成される値は、単体の正規化処理だけでなく、実際の入口から検証処理までの経路を追う
-8. 副作用や状態変更を伴う差分は、入口・正常終了・早期終了・例外・後片付けの経路を追う
-9. 指摘は現在の差分に起因し、ユーザーが修正すべきものだけにする
-10. 各指摘には場所、影響、修正方針を含める
-11. 根拠のない推測、好みだけの変更、今回の差分と無関係な既存問題は指摘しない
+8. 実行とは別に同じ値を表示・検証・説明する入口がある場合、主実行経路と同じ正規化済み入力・override 順・resolver で解決されているか比較する
+9. テストがある場合、値の存在だけでなく、未設定・設定・不正値・override・継承・非継承・非対応対象など元要件の分岐条件を踏んでいるか確認する
+10. 副作用や状態変更を伴う差分は、入口・正常終了・早期終了・例外・後片付けの経路を追う
+11. 指摘は現在の差分に起因し、ユーザーが修正すべきものだけにする
+12. 各指摘には場所、影響、修正方針を含める
+13. 根拠のない推測、好みだけの変更、今回の差分と無関係な既存問題は指摘しない

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

@@ -11,4 +11,9 @@
 2. 外部契約があるのに入力位置（root body / query / path）の検証が欠けている場合、カバレッジ不足として扱う
 3. 副作用や状態遷移を伴う変更では、正常系だけでなく代表的な失敗経路のテスト有無を確認する
 4. 共通化・抽象化で契約を統一した変更では、新しい共通経路だけでなく既存の同種分岐にも契約テストがあるか確認する
-5. モックで代替した検証は、実連携まで確認したものとして扱わない
+5. 設定値、実行時に選択される能力・バックエンド・オプション、権限の変更では、値の存在ではなく、未設定・設定・override・継承・非継承など元要件の分岐条件を踏むテストか確認する
+6. 実行とは別に同じ値を表示・検証・説明する入口がある変更では、表示値だけでなく実行時の解決入力と一致することを検証しているか確認する
+7. モックで代替した検証は、実連携まで確認したものとして扱わない
+8. 禁止・拒否・非継承・非対応・隔離の検証が、完全一致文字列の不在だけでなく、観測単位を抽出して禁止値ごとに確認しているか確認する
+9. 新しい設定境界では、空文字列、空白だけの文字列、空配列、大小文字違いなどの正規化対象を必要に応じて確認する
+10. E2E の timeout、cleanup、強制終了の扱いが既存の同種テスト規約と一致しているか確認する

package/builtins/ja/facets/knowledge/backend.md

@@ -218,6 +218,8 @@ fun confirm(confirmedBy: String): OrderConfirmedEvent {
 | 読み取り用のクラス名やメソッド名なのに副作用を持つ | REJECT |
 | 単純な参照APIが問い合わせ層を呼び、レスポンスDTOに変換するだけ | OK |
 | 単純な状態変更APIが構造検証と認可境界の解決後にコマンドを1つ送るだけ | OK |
+| Controller向けの読み取り調整役が認可境界、複数Read Model、ページング等を扱う | ApplicationService または ReadService として表現 |
+| QueryHandler と同じ領域に QueryService という名前の送信側・調整側コンポーネントを置く | 警告。クエリ受信側と混同しやすい |
 | 複数のRead Model参照、外部連携、複数コマンド、結果待機をControllerに置く | REJECT。UseCase層に分離 |
 | UseCaseが別サービスへの薄い委譲だけでドメイン上の判断や調整を持たない | 削除を検討 |
 

package/builtins/ja/facets/knowledge/cqrs-es.md

@@ -471,6 +471,18 @@ Query側はイベント駆動のPubSubモデルで動作する。Projection が
 | Command と Query を同じサービスに混在させる | REJECT。責務と命名を分離 |
 | Query側で存在確認やスコープ確認を行い、呼び出し元がコマンドを送る | OK |
 
+### QueryHandler と ApplicationService の命名
+
+CQRSではクエリを受けるコンポーネントを QueryHandler と呼び、クエリを送る入口は QueryGateway / QueryBus として扱う。Controller から読み取りユースケースを呼ぶ facade は、QueryHandler と混同しないよう ApplicationService または ReadService と名付ける。
+
+| 基準 | 判定 |
+|------|------|
+| Query を受けて Read Model を参照し、Query結果の型を返す | QueryHandler |
+| Controller から複数Query、認可境界、ページング、DTO組み立てを調整する | ApplicationService または ReadService |
+| Query送信や読み取り調整だけのクラスを QueryService と呼ぶ | 警告。QueryHandler と混同しやすい |
+| QueryHandler がHTTPリクエスト/レスポンスやController都合のエラー変換を知る | REJECT |
+| 追加判断のない単純な読み取り wrapper を作る | 削除を検討。Controller から QueryGateway 直でもよい |
+
 レイヤー間の型:
 - `application/query/` - Query結果の型（例: `OrderDetail`）
 - `adapter/protocol/` - RESTレスポンスの型（例: `OrderDetailResponse`）