takt 0.49.0 → 0.50.0

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

@@ -1,5 +1,5 @@
 name: review-default
-description: Multi-perspective Code Review - reviews changes in parallel from 6 perspectives (including coding review) and outputs consolidated results
+description: Multi-perspective Code Review - reviews changes with specialist parallel reviewers, then runs merge-readiness review and synthesis in a parallel final gate.
 workflow_config:
   provider_options:
     codex:
@@ -148,12 +148,14 @@ steps:
           - condition: approved
           - condition: needs_fix
 
-      - name: pure-review
+      - name: coding-review
         tags:
           - review
         edit: false
-        persona: pure-reviewer
-        policy: review
+        persona: coding-reviewer
+        policy:
+          - review
+          - coding
         provider_options:
           claude:
             allowed_tools:
@@ -163,23 +165,34 @@ steps:
               - Bash
               - WebSearch
               - WebFetch
-        instruction: review-pure
+        instruction: review-coding
         output_contracts:
           report:
-            - name: pure-review.md
-              format: pure-review
+            - name: coding-review.md
+              format: coding-review
         rules:
           - condition: approved
           - condition: needs_fix
 
-      - name: coding-review
+    rules:
+      - condition: all("approved")
+        next: final-gate
+      - condition: any("needs_fix")
+        next: supervise
+
+  - name: final-gate
+    tags:
+      - review
+      - final-gate
+    parallel:
+      - name: merge-readiness-review
         tags:
           - review
+          - final-gate
+          - merge-readiness
         edit: false
-        persona: coding-reviewer
-        policy:
-          - review
-          - coding
+        persona: merge-readiness-reviewer
+        policy: review
         provider_options:
           claude:
             allowed_tools:
@@ -189,24 +202,130 @@ steps:
               - Bash
               - WebSearch
               - WebFetch
-        instruction: review-coding
+        instruction: review-merge-readiness
         output_contracts:
           report:
-            - name: coding-review.md
-              format: coding-review
+            - name: merge-readiness-review.md
+              format: merge-readiness-review
         rules:
           - condition: approved
           - condition: needs_fix
+      - name: review-synthesis
+        tags:
+          - review
+          - final-gate
+          - supervise
+        edit: false
+        persona: supervisor
+        policy: review
+        provider_options:
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - WebSearch
+              - WebFetch
+        pass_previous_response: false
+        rules:
+          - condition: approved
+        instruction: &review_synthesis_instruction |
+          ## Review Results
+          {previous_response}
+
+          **This is a review-only workflow.** Do NOT run tests or builds.
+          Your role is to synthesize the review results and produce a final summary.
+
+          **Tasks:**
+          1. Read all review reports in the Report Directory
+             - `00-review-target.md` (Review target information)
+             - `01-architecture-review.md` (Architecture review)
+             - `02-security-review.md` (Security review)
+             - `03-qa-review.md` (QA review)
+             - `04-testing-review.md` (Testing review)
+             - `05-coding-review.md` (Coding review)
+          2. Synthesize the 5 review results
+          3. Produce a consolidated review summary with overall verdict
+
+          **Review Summary output contract:**
+          ```markdown
+          # Review Summary
+
+          ## Overall Verdict: APPROVE / REJECT
+
+          ## Summary
+          {2-3 sentences consolidating all review results}
+
+          ## Review Results
+          | Review | Result | Key Findings |
+          |--------|--------|--------------|
+          | Architecture | APPROVE/REJECT | {Brief finding} |
+          | Security | APPROVE/REJECT | {Brief finding} |
+          | QA | APPROVE/REJECT | {Brief finding} |
+          | Testing | APPROVE/REJECT | {Brief finding} |
+          | Coding | APPROVE/REJECT | {Brief finding} |
+
+          ## Issues Requiring Attention
+          | # | Severity | Source | Location | Issue |
+          |---|----------|--------|----------|-------|
+          | 1 | High | Security | `file:line` | Description |
+
+          ## Improvement Suggestions
+          - {Consolidated suggestions from all reviews}
+          ```
+        output_contracts: &review_synthesis_output_contracts
+          report:
+            - name: review-summary.md
+              format: |
+                ```markdown
+                # Review Summary
 
+                ## Overall Verdict: APPROVE / REJECT
+
+                ## Summary
+                {2-3 sentences consolidating all review results}
+
+                ## Review Results
+                | Review | Result | Key Findings |
+                |--------|--------|--------------|
+                | Architecture | APPROVE/REJECT | {Overview} |
+                | Security | APPROVE/REJECT | {Overview} |
+                | QA | APPROVE/REJECT | {Overview} |
+                | Testing | APPROVE/REJECT | {Overview} |
+                | Coding | APPROVE/REJECT | {Overview} |
+
+                ## Current Iteration Findings (new)
+                | # | finding_id | Severity | Source | Location | Issue | Fix Suggestion |
+                |---|------------|----------|--------|----------|-------|----------------|
+                | 1 | SUM-NEW-src-file-L42 | High | Security | `file:line` | Description | Suggestion |
+
+                ## Carry-over Findings (persists)
+                | # | finding_id | Source | Previous Evidence | Current Evidence | Issue |
+                |---|------------|--------|-------------------|------------------|-------|
+                | 1 | SUM-PERSIST-src-file-L77 | Architecture | `file:line` | `file:line` | Description |
+
+                ## Resolved Findings (resolved)
+                | finding_id | Source | Resolution Evidence |
+                |------------|--------|---------------------|
+                | SUM-RESOLVED-src-file-L10 | QA | `file:line` |
+
+                ## Improvement Suggestions
+                - {Consolidated suggestions from all reviews}
+
+                ## Rejection Gate
+                - REJECT is valid only when at least one finding exists in `new` or `persists`
+                - Findings without `finding_id` are invalid
+                ```
     rules:
       - condition: all("approved")
-        next: supervise
+        next: COMPLETE
       - condition: any("needs_fix")
-        next: supervise
+        next: COMPLETE
 
   - name: supervise
     tags:
       - review
+      - supervise
     edit: false
     persona: supervisor
     policy: review
@@ -222,93 +341,5 @@ steps:
     rules:
       - condition: Review synthesis complete
         next: COMPLETE
-    instruction: |
-      ## Review Results
-      {previous_response}
-
-      **This is a review-only workflow.** Do NOT run tests or builds.
-      Your role is to synthesize the review results and produce a final summary.
-
-      **Tasks:**
-      1. Read all review reports in the Report Directory
-         - `00-review-target.md` (Review target information)
-         - `01-architecture-review.md` (Architecture review)
-         - `02-security-review.md` (Security review)
-         - `03-qa-review.md` (QA review)
-         - `04-testing-review.md` (Testing review)
-         - `05-pure-review.md` (Requirements review)
-         - `06-coding-review.md` (Coding review)
-      2. Synthesize the 6 review results
-      3. Produce a consolidated review summary with overall verdict
-
-      **Review Summary output contract:**
-      ```markdown
-      # Review Summary
-
-      ## Overall Verdict: APPROVE / REJECT
-
-      ## Summary
-      {2-3 sentences consolidating all review results}
-
-      ## Review Results
-      | Review | Result | Key Findings |
-      |--------|--------|--------------|
-      | Architecture | APPROVE/REJECT | {Brief finding} |
-      | Security | APPROVE/REJECT | {Brief finding} |
-      | QA | APPROVE/REJECT | {Brief finding} |
-      | Testing | APPROVE/REJECT | {Brief finding} |
-      | Requirements | APPROVE/REJECT | {Brief finding} |
-      | Coding | APPROVE/REJECT | {Brief finding} |
-
-      ## Issues Requiring Attention
-      | # | Severity | Source | Location | Issue |
-      |---|----------|--------|----------|-------|
-      | 1 | High | Security | `file:line` | Description |
-
-      ## Improvement Suggestions
-      - {Consolidated suggestions from all reviews}
-      ```
-    output_contracts:
-      report:
-        - name: review-summary.md
-          format: |
-            ```markdown
-            # Review Summary
-
-            ## Overall Verdict: APPROVE / REJECT
-
-            ## Summary
-            {2-3 sentences consolidating all review results}
-
-            ## Review Results
-            | Review | Result | Key Findings |
-            |--------|--------|--------------|
-            | Architecture | APPROVE/REJECT | {Overview} |
-            | Security | APPROVE/REJECT | {Overview} |
-            | QA | APPROVE/REJECT | {Overview} |
-            | Testing | APPROVE/REJECT | {Overview} |
-            | Requirements | APPROVE/REJECT | {Overview} |
-            | Coding | APPROVE/REJECT | {Overview} |
-
-            ## Current Iteration Findings (new)
-            | # | finding_id | Severity | Source | Location | Issue | Fix Suggestion |
-            |---|------------|----------|--------|----------|-------|----------------|
-            | 1 | SUM-NEW-src-file-L42 | High | Security | `file:line` | Description | Suggestion |
-
-            ## Carry-over Findings (persists)
-            | # | finding_id | Source | Previous Evidence | Current Evidence | Issue |
-            |---|------------|--------|-------------------|------------------|-------|
-            | 1 | SUM-PERSIST-src-file-L77 | Architecture | `file:line` | `file:line` | Description |
-
-            ## Resolved Findings (resolved)
-            | finding_id | Source | Resolution Evidence |
-            |------------|--------|---------------------|
-            | SUM-RESOLVED-src-file-L10 | QA | `file:line` |
-
-            ## Improvement Suggestions
-            - {Consolidated suggestions from all reviews}
-
-            ## Rejection Gate
-            - REJECT is valid only when at least one finding exists in `new` or `persists`
-            - Findings without `finding_id` are invalid
-            ```
+    instruction: *review_synthesis_instruction
+    output_contracts: *review_synthesis_output_contracts

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

@@ -167,12 +167,14 @@ steps:
           - condition: approved
           - condition: needs_fix
 
-      - name: pure-review
+      - name: coding-review
         tags:
           - review
         edit: false
-        persona: pure-reviewer
-        policy: review
+        persona: coding-reviewer
+        policy:
+          - review
+          - coding
         provider_options:
           claude:
             allowed_tools:
@@ -182,22 +184,34 @@ steps:
               - Bash
               - WebSearch
               - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
         rules:
           - condition: approved
           - condition: needs_fix
-        instruction: review-pure
-        output_contracts:
-          report:
-            - name: pure-review.md
-              format: pure-review
-      - name: coding-review
+
+    rules:
+      - condition: all("approved")
+        next: final-gate
+      - condition: any("needs_fix")
+        next: supervise
+
+  - name: final-gate
+    tags:
+      - review
+      - final-gate
+    parallel:
+      - name: merge-readiness-review
         tags:
           - review
+          - final-gate
+          - merge-readiness
         edit: false
-        persona: coding-reviewer
-        policy:
-          - review
-          - coding
+        persona: merge-readiness-reviewer
+        policy: review
         provider_options:
           claude:
             allowed_tools:
@@ -207,44 +221,58 @@ steps:
               - Bash
               - WebSearch
               - WebFetch
-        instruction: review-coding
+        instruction: review-merge-readiness
         output_contracts:
           report:
-            - name: coding-review.md
-              format: coding-review
+            - name: merge-readiness-review.md
+              format: merge-readiness-review
         rules:
           - condition: approved
           - condition: needs_fix
-
+      - name: review-synthesis
+        tags:
+          - review
+          - final-gate
+          - supervise
+        edit: false
+        persona: supervisor
+        policy: review
+        provider_options: &review_synthesis_provider_options
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - WebSearch
+              - WebFetch
+        pass_previous_response: false
+        instruction: &review_synthesis_instruction supervise
+        output_contracts: &review_synthesis_output_contracts
+          report:
+            - name: supervisor-validation.md
+              format: supervisor-validation
+            - name: summary.md
+              format: summary
+              use_judge: false
+        rules:
+          - condition: approved
     rules:
       - condition: all("approved")
-        next: supervise
+        next: COMPLETE
       - condition: any("needs_fix")
-        next: supervise
+        next: COMPLETE
 
   - name: supervise
     tags:
       - review
+      - supervise
     edit: false
     persona: supervisor
     policy: review
-    provider_options:
-      claude:
-        allowed_tools:
-          - Read
-          - Glob
-          - Grep
-          - WebSearch
-          - WebFetch
+    provider_options: *review_synthesis_provider_options
     pass_previous_response: false
-    instruction: supervise
-    output_contracts:
-      report:
-        - name: supervisor-validation.md
-          format: supervisor-validation
-        - name: summary.md
-          format: summary
-          use_judge: false
+    instruction: *review_synthesis_instruction
+    output_contracts: *review_synthesis_output_contracts
     rules:
       - condition: Review integration complete
         next: COMPLETE

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

@@ -141,12 +141,14 @@ steps:
           - condition: approved
           - condition: needs_fix
 
-      - name: pure-review
+      - name: coding-review
         tags:
           - review
         edit: false
-        persona: pure-reviewer
-        policy: review
+        persona: coding-reviewer
+        policy:
+          - review
+          - coding
         provider_options:
           claude:
             allowed_tools:
@@ -156,22 +158,34 @@ steps:
               - Bash
               - WebSearch
               - WebFetch
+        instruction: review-coding
+        output_contracts:
+          report:
+            - name: coding-review.md
+              format: coding-review
         rules:
           - condition: approved
           - condition: needs_fix
-        instruction: review-pure
-        output_contracts:
-          report:
-            - name: pure-review.md
-              format: pure-review
-      - name: coding-review
+
+    rules:
+      - condition: all("approved")
+        next: final-gate
+      - condition: any("needs_fix")
+        next: supervise
+
+  - name: final-gate
+    tags:
+      - review
+      - final-gate
+    parallel:
+      - name: merge-readiness-review
         tags:
           - review
+          - final-gate
+          - merge-readiness
         edit: false
-        persona: coding-reviewer
-        policy:
-          - review
-          - coding
+        persona: merge-readiness-reviewer
+        policy: review
         provider_options:
           claude:
             allowed_tools:
@@ -181,44 +195,58 @@ steps:
               - Bash
               - WebSearch
               - WebFetch
-        instruction: review-coding
+        instruction: review-merge-readiness
         output_contracts:
           report:
-            - name: coding-review.md
-              format: coding-review
+            - name: merge-readiness-review.md
+              format: merge-readiness-review
         rules:
           - condition: approved
           - condition: needs_fix
-
+      - name: review-synthesis
+        tags:
+          - review
+          - final-gate
+          - supervise
+        edit: false
+        persona: supervisor
+        policy: review
+        provider_options: &review_synthesis_provider_options
+          claude:
+            allowed_tools:
+              - Read
+              - Glob
+              - Grep
+              - WebSearch
+              - WebFetch
+        pass_previous_response: false
+        instruction: &review_synthesis_instruction supervise
+        output_contracts: &review_synthesis_output_contracts
+          report:
+            - name: supervisor-validation.md
+              format: supervisor-validation
+            - name: summary.md
+              format: summary
+              use_judge: false
+        rules:
+          - condition: approved
     rules:
       - condition: all("approved")
-        next: supervise
+        next: COMPLETE
       - condition: any("needs_fix")
-        next: supervise
+        next: COMPLETE
 
   - name: supervise
     tags:
       - review
+      - supervise
     edit: false
     persona: supervisor
     policy: review
-    provider_options:
-      claude:
-        allowed_tools:
-          - Read
-          - Glob
-          - Grep
-          - WebSearch
-          - WebFetch
+    provider_options: *review_synthesis_provider_options
     pass_previous_response: false
-    instruction: supervise
-    output_contracts:
-      report:
-        - name: supervisor-validation.md
-          format: supervisor-validation
-        - name: summary.md
-          format: summary
-          use_judge: false
+    instruction: *review_synthesis_instruction
+    output_contracts: *review_synthesis_output_contracts
     rules:
       - condition: Review integration complete
         next: COMPLETE