takt 0.45.0 → 0.47.0

package/builtins/en/workflows/frontend.yaml

@@ -35,6 +35,8 @@ loop_monitors:
           next: supervise
 steps:
   - name: plan
+    tags:
+      - plan
     edit: false
     persona: planner
     policy:
@@ -63,6 +65,8 @@ steps:
         - name: plan.md
           format: plan-frontend
   - name: write_tests
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -103,6 +107,8 @@ steps:
         - name: test-report.md
           format: test-report
   - name: implement
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -145,6 +151,8 @@ steps:
         - name: coder-decisions.md
           format: coder-decisions
   - name: ai-antipattern-review-1st
+    tags:
+      - review
     edit: false
     persona: ai-antipattern-reviewer
     policy:
@@ -169,6 +177,8 @@ steps:
         - name: ai-antipattern-review.md
           format: ai-antipattern-review
   - name: ai-antipattern-fix
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -202,6 +212,8 @@ steps:
       - 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
@@ -218,8 +230,12 @@ steps:
         next: reviewers_1
     instruction: arbitrate
   - name: reviewers_1
+    tags:
+      - review
     parallel:
       - name: arch-review
+        tags:
+          - review
         edit: false
         persona: architecture-reviewer
         policy: review
@@ -244,6 +260,8 @@ steps:
             - name: architect-review.md
               format: architecture-review
       - name: frontend-review
+        tags:
+          - review
         edit: false
         persona: frontend-reviewer
         policy:
@@ -269,6 +287,8 @@ steps:
             - name: frontend-review.md
               format: frontend-review
       - name: testing-review
+        tags:
+          - review
         edit: false
         persona: testing-reviewer
         policy:
@@ -294,6 +314,8 @@ steps:
             - name: testing-review.md
               format: testing-review
       - name: ai-antipattern-review-2nd
+        tags:
+          - review
         edit: false
         persona: ai-antipattern-reviewer
         policy:
@@ -321,8 +343,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
@@ -344,6 +370,8 @@ steps:
             - name: security-review.md
               format: security-review
       - name: qa-review
+        tags:
+          - review
         edit: false
         persona: qa-reviewer
         policy:
@@ -365,10 +393,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:
@@ -392,6 +447,8 @@ steps:
       - condition: any("needs_fix")
         next: fix
   - name: fix
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -423,6 +480,8 @@ steps:
         next: plan
     instruction: fix
   - name: supervise
+    tags:
+      - review
     edit: false
     persona: dual-supervisor
     policy: review
@@ -449,6 +508,8 @@ steps:
           format: summary
           use_judge: false
   - name: fix_supervisor
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:

package/builtins/en/workflows/magi.yaml

@@ -17,6 +17,8 @@ max_steps: 5
 initial_step: melchior
 steps:
   - name: melchior
+    tags:
+      - review
     persona: melchior
     instruction: |
       # MAGI System Initiated
@@ -47,6 +49,8 @@ steps:
       - condition: Judgment completed
         next: balthasar
   - name: balthasar
+    tags:
+      - review
     persona: balthasar
     instruction: |
       # MAGI System Continuing
@@ -81,6 +85,8 @@ steps:
       - condition: Judgment completed
         next: casper
   - name: casper
+    tags:
+      - review
     persona: casper
     instruction: |
       # MAGI System Final Deliberation

package/builtins/en/workflows/peer-review-with-fc.yaml

@@ -0,0 +1,206 @@
+name: peer-review-with-fc
+description: Experimental Finding Contract peer review. Six parallel peer reviewers (+ ai-antipattern-review-2nd) ⇄ fix loop.
+finding_contract:
+  ledger_path: .takt/findings/peer-review-with-fc.json
+  raw_findings_path: .takt/findings/peer-review-with-fc/raw
+  manager:
+    persona: findings-manager
+    instruction: findings-manager
+    output_contract: findings-manager
+subworkflow:
+  callable: true
+  visibility: internal
+  returns:
+    - need_replan
+  params:
+    arch_knowledge:
+      type: facet_ref[]
+      facet_kind: knowledge
+      default:
+        - architecture
+        - takt
+workflow_config:
+  provider_options:
+    codex:
+      network_access: true
+    opencode:
+      network_access: true
+max_steps: 30
+initial_step: reviewers
+loop_monitors:
+  - cycle:
+      - reviewers
+      - fix
+    threshold: 5
+    judge:
+      persona: supervisor
+      instruction: loop-monitor-reviewers-fix
+      rules:
+        - condition: Healthy (findings decreasing, fixes applied)
+          next: reviewers
+        - condition: Unproductive (same findings repeating)
+          next: COMPLETE
+steps:
+  - name: reviewers
+    tags:
+      - review
+    parallel:
+      - name: arch-review
+        tags:
+          - review
+        edit: false
+        persona: architecture-reviewer
+        policy: review
+        knowledge:
+          $param: arch_knowledge
+        provider_options:
+          extends: review-readonly
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-arch
+        output_contracts:
+          report:
+            - name: architect-review.md
+              format: architecture-review-finding-contract
+      - name: security-review
+        tags:
+          - review
+        edit: false
+        persona: security-reviewer
+        policy: review
+        knowledge: security
+        provider_options:
+          extends: review-readonly
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-security
+        output_contracts:
+          report:
+            - name: security-review.md
+              format: security-review-finding-contract
+      - name: qa-review
+        tags:
+          - review
+        edit: false
+        persona: qa-reviewer
+        policy:
+          - review
+          - qa
+        provider_options:
+          extends: review-readonly
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-qa
+        output_contracts:
+          report:
+            - name: qa-review.md
+              format: qa-review-finding-contract
+      - name: testing-review
+        tags:
+          - review
+        edit: false
+        persona: testing-reviewer
+        policy:
+          - review
+          - testing
+        knowledge:
+          - unit-testing
+          - e2e-testing
+        provider_options:
+          extends: review-readonly
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-test
+        output_contracts:
+          report:
+            - name: testing-review.md
+              format: testing-review-finding-contract
+      - name: pure-review
+        tags:
+          - review
+        edit: false
+        persona: pure-reviewer
+        policy: review
+        provider_options:
+          extends: review-readonly
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        instruction: review-pure
+        output_contracts:
+          report:
+            - name: pure-review.md
+              format: pure-review-finding-contract
+      - name: coding-review
+        tags:
+          - review
+        edit: false
+        persona: coding-reviewer
+        policy:
+          - review
+          - coding
+        pass_previous_response: false
+        provider_options:
+          extends: review-readonly
+        instruction: review-coding
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review-finding-contract
+      - name: ai-antipattern-review-2nd
+        tags:
+          - review
+        edit: false
+        persona: ai-antipattern-reviewer
+        policy:
+          - review
+          - ai-antipattern
+        provider_options:
+          extends: review-web
+        instruction: ai-antipattern-review
+        rules:
+          - condition: approved
+          - condition: needs_fix
+        output_contracts:
+          report:
+            - name: ai-antipattern-review.md
+              format: ai-antipattern-review-finding-contract
+    rules:
+      - condition: all("approved") && findings.open.count == 0 && findings.conflicts.count == 0
+        next: COMPLETE
+      - condition: any("needs_fix") && findings.conflicts.count == 0
+        next: fix
+      - condition: findings.conflicts.count == 0 && findings.open.count > 0
+        next: fix
+      - condition: ai("There are unadjudicated findings.conflicts. Choose fix only when a concrete code change by the coder can resolve the conflict. Do not choose it for reviewer disagreement or cases that need human adjudication.")
+        next: fix
+      - condition: findings.conflicts.count > 0
+        return: need_replan
+  - name: fix
+    tags:
+      - coding
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+    knowledge:
+      - takt
+      - architecture
+    provider_options:
+      extends: edit
+    required_permission_mode: edit
+    pass_previous_response: false
+    rules:
+      - condition: Fix complete
+        next: reviewers
+      - condition: Cannot proceed, insufficient info
+        return: need_replan
+    instruction: fix

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

@@ -1,5 +1,5 @@
 name: peer-review
-description: takt-default Phase 2. Six parallel peer reviewers (+ ai-antipattern-review-2nd) ⇄ fix loop.
+description: Six parallel peer reviewers (+ ai-antipattern-review-2nd) ⇄ fix loop.
 subworkflow:
   callable: true
   visibility: internal
@@ -35,22 +35,19 @@ loop_monitors:
           next: COMPLETE
 steps:
   - name: reviewers
+    tags:
+      - review
     parallel:
       - name: arch-review
+        tags:
+          - review
         edit: false
         persona: architecture-reviewer
         policy: review
         knowledge:
           $param: arch_knowledge
         provider_options:
-          claude:
-            allowed_tools:
-              - Read
-              - Glob
-              - Grep
-              - Bash
-              - WebSearch
-              - WebFetch
+          extends: review-readonly
         rules:
           - condition: approved
           - condition: needs_fix
@@ -60,19 +57,14 @@ steps:
             - name: architect-review.md
               format: architecture-review
       - name: security-review
+        tags:
+          - review
         edit: false
         persona: security-reviewer
         policy: review
         knowledge: security
         provider_options:
-          claude:
-            allowed_tools:
-              - Read
-              - Glob
-              - Grep
-              - Bash
-              - WebSearch
-              - WebFetch
+          extends: review-readonly
         rules:
           - condition: approved
           - condition: needs_fix
@@ -82,20 +74,15 @@ steps:
             - name: security-review.md
               format: security-review
       - name: qa-review
+        tags:
+          - review
         edit: false
         persona: qa-reviewer
         policy:
           - review
           - qa
         provider_options:
-          claude:
-            allowed_tools:
-              - Read
-              - Glob
-              - Grep
-              - Bash
-              - WebSearch
-              - WebFetch
+          extends: review-readonly
         rules:
           - condition: approved
           - condition: needs_fix
@@ -105,6 +92,8 @@ steps:
             - name: qa-review.md
               format: qa-review
       - name: testing-review
+        tags:
+          - review
         edit: false
         persona: testing-reviewer
         policy:
@@ -114,14 +103,7 @@ steps:
           - unit-testing
           - e2e-testing
         provider_options:
-          claude:
-            allowed_tools:
-              - Read
-              - Glob
-              - Grep
-              - Bash
-              - WebSearch
-              - WebFetch
+          extends: review-readonly
         rules:
           - condition: approved
           - condition: needs_fix
@@ -130,41 +112,33 @@ steps:
           report:
             - name: testing-review.md
               format: testing-review
-      - name: requirements-review
+      - name: pure-review
+        tags:
+          - review
         edit: false
-        persona: requirements-reviewer
+        persona: pure-reviewer
         policy: review
         provider_options:
-          claude:
-            allowed_tools:
-              - Read
-              - Glob
-              - Grep
-              - Bash
-              - WebSearch
-              - WebFetch
+          extends: review-readonly
         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: coding-review
+        tags:
+          - review
         edit: false
         persona: coding-reviewer
-        policy: review
+        policy:
+          - review
+          - coding
         pass_previous_response: false
         provider_options:
-          claude:
-            allowed_tools:
-              - Read
-              - Glob
-              - Grep
-              - Bash
-              - WebSearch
-              - WebFetch
+          extends: review-readonly
         instruction: review-coding
         rules:
           - condition: approved
@@ -174,19 +148,15 @@ steps:
             - name: coding-review.md
               format: coding-review
       - name: ai-antipattern-review-2nd
+        tags:
+          - review
         edit: false
         persona: ai-antipattern-reviewer
         policy:
           - review
           - ai-antipattern
         provider_options:
-          claude:
-            allowed_tools:
-              - Read
-              - Glob
-              - Grep
-              - WebSearch
-              - WebFetch
+          extends: review-web
         instruction: ai-antipattern-review
         rules:
           - condition: approved
@@ -201,6 +171,8 @@ steps:
       - condition: any("needs_fix")
         next: fix
   - name: fix
+    tags:
+      - coding
     edit: true
     persona: coder
     policy:
@@ -210,16 +182,7 @@ steps:
       - takt
       - architecture
     provider_options:
-      claude:
-        allowed_tools:
-          - Read
-          - Glob
-          - Grep
-          - Edit
-          - Write
-          - Bash
-          - WebSearch
-          - WebFetch
+      extends: edit
     required_permission_mode: edit
     pass_previous_response: false
     rules:

package/builtins/en/workflows/research.yaml

@@ -17,6 +17,8 @@ max_steps: 10
 initial_step: plan
 steps:
   - name: plan
+    tags:
+      - plan
     persona: research-planner
     policy: research
     knowledge: research
@@ -27,6 +29,8 @@ steps:
       - condition: Insufficient information to create a plan
         next: ABORT
   - name: dig
+    tags:
+      - plan
     persona: research-digger
     policy: research
     knowledge: research
@@ -37,6 +41,8 @@ steps:
       - condition: Unable to conduct research
         next: ABORT
   - name: supervise
+    tags:
+      - review
     persona: research-supervisor
     policy: research
     knowledge: research