takt 0.1.3 → 0.1.5

package/resources/global/en/workflows/default.yaml

@@ -1,5 +1,15 @@
 # Default TAKT Workflow
 # Plan -> Coder -> Architect Review -> AI Review -> Security Review -> Supervisor Approval
+#
+# Template Variables:
+#   {iteration}      - Workflow-wide turn count (total steps executed across all agents)
+#   {max_iterations} - Maximum iterations allowed for the workflow
+#   {step_iteration} - Per-step iteration count (how many times THIS step has been executed)
+#   {task}           - Original user request
+#   {previous_response} - Output from the previous step
+#   {git_diff}       - Current uncommitted changes (git diff)
+#   {user_inputs}    - Accumulated user inputs during workflow
+#   {report_dir}     - Report directory name (e.g., "20250126-143052-task-summary")
 
 name: default
 description: Standard development workflow with planning and specialized reviews
@@ -18,9 +28,45 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Judgment Criteria
+
+      | Situation | Judgment |
+      |-----------|----------|
+      | Requirements clear and implementable | DONE |
+      | Requirements unclear, insufficient info | BLOCKED |
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | Analysis complete | `[PLANNER:DONE]` |
+      | Insufficient info | `[PLANNER:BLOCKED]` |
+
+      ### Output Examples
+
+      **DONE case:**
+      ```
+      [PLANNER:DONE]
+      ```
+
+      **BLOCKED case:**
+      ```
+      [PLANNER:BLOCKED]
+
+      Clarifications needed:
+      - {Question 1}
+      - {Question 2}
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: plan (Task Analysis)
       - Report Directory: .takt/reports/{report_dir}/
       - Report File: .takt/reports/{report_dir}/00-plan.md
@@ -44,7 +90,7 @@ steps:
 
       **Report output:** Output to the `Report File` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Report format:**
       ```markdown
@@ -67,9 +113,6 @@ steps:
       ## Clarifications Needed (if any)
       - {Unclear points or items requiring confirmation}
       ```
-
-      Output [PLANNER:DONE] when complete.
-      Output [PLANNER:BLOCKED] if requirements are unclear.
     pass_previous_response: true
     transitions:
       - condition: done
@@ -88,9 +131,43 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | Implementation complete | `[CODER:DONE]` |
+      | Cannot decide/insufficient info | `[CODER:BLOCKED]` |
+
+      **Important**: When in doubt, `[BLOCKED]`. Don't decide on your own.
+
+      ### Output Examples
+
+      **DONE case:**
+      ```
+      Implementation complete.
+      - Created: `src/auth/service.ts`, `tests/auth.test.ts`
+      - Modified: `src/routes.ts`
+
+      [CODER:DONE]
+      ```
+
+      **BLOCKED case:**
+      ```
+      [CODER:BLOCKED]
+
+      Reason: DB schema is undefined, cannot implement
+      Required info: users table structure
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: implement
       - Report Directory: .takt/reports/{report_dir}/
       - Report Files:
@@ -109,7 +186,7 @@ steps:
 
       **Report output:** Output to the `Report Files` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Scope report format (create at implementation start):**
       ```markdown
@@ -140,9 +217,6 @@ steps:
       - **Options Considered**: {List of options}
       - **Reason**: {Why this option was chosen}
       ```
-
-      Include [CODER:DONE] when complete.
-      Include [CODER:BLOCKED] if you cannot proceed (returns to plan).
     transitions:
       - condition: done
         next_step: review
@@ -157,9 +231,68 @@ steps:
       - Grep
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Judgment Criteria
+
+      | Situation | Judgment |
+      |-----------|----------|
+      | Structural issues | REJECT |
+      | Design principle violations | REJECT |
+      | Security issues | REJECT |
+      | Insufficient tests | REJECT |
+      | Minor improvements needed | IMPROVE |
+      | No issues | APPROVE |
+
+      **How to use IMPROVE:**
+      - Design is acceptable but there are points that could be better
+      - Examples: naming improvements, small refactoring, adding comments
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | No issues | `[ARCHITECT:APPROVE]` |
+      | Minor improvements needed | `[ARCHITECT:IMPROVE]` |
+      | Structural changes required | `[ARCHITECT:REJECT]` |
+
+      ### Output Examples
+
+      **APPROVE case:**
+      ```
+      [ARCHITECT:APPROVE]
+
+      Positive points:
+      - Appropriate module organization
+      - Single responsibility maintained
+      ```
+
+      **IMPROVE case:**
+      ```
+      [ARCHITECT:IMPROVE]
+
+      Improvements:
+      - Improve naming: `data` → `userData`
+      - Add comments
+      ```
+
+      **REJECT case:**
+      ```
+      [ARCHITECT:REJECT]
+
+      Issues:
+      1. File size exceeded
+         - Location: `src/services/user.ts` (523 lines)
+         - Fix: Split into 3 files
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: review (Architecture Review)
       - Report Directory: .takt/reports/{report_dir}/
       - Report File: .takt/reports/{report_dir}/03-architect-review.md
@@ -175,14 +308,11 @@ steps:
       ## Instructions
       Focus on **architecture and design** review. Do NOT review AI-specific issues (that's the next step).
 
-      Review the changes and provide feedback:
-      - [ARCHITECT:APPROVE] if no issues
-      - [ARCHITECT:IMPROVE] if minor improvements needed
-      - [ARCHITECT:REJECT] if structural changes are needed (list specific issues)
+      Review the changes and provide feedback.
 
       **Report output:** Output to the `Report File` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Report format:**
       ```markdown
@@ -230,9 +360,43 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | Improvements complete | `[CODER:DONE]` |
+      | Cannot decide/insufficient info | `[CODER:BLOCKED]` |
+
+      **Important**: When in doubt, `[BLOCKED]`. Don't decide on your own.
+
+      ### Output Examples
+
+      **DONE case:**
+      ```
+      Improvements complete.
+      - Improved naming: `data` → `userData`
+      - Added comments
+
+      [CODER:DONE]
+      ```
+
+      **BLOCKED case:**
+      ```
+      [CODER:BLOCKED]
+
+      Reason: Improvement intent unclear
+      Required info: Specific improvement details
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: improve
 
       ## Architect Feedback (This is the latest instruction - prioritize this)
@@ -254,8 +418,6 @@ steps:
       - Adding/fixing comments
       - Code organization
 
-      Include [CODER:DONE] when complete.
-      Include [CODER:BLOCKED] if you cannot proceed.
     pass_previous_response: true
     transitions:
       - condition: done
@@ -271,9 +433,52 @@ steps:
       - Grep
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Judgment Criteria
+
+      | Situation | Judgment |
+      |-----------|----------|
+      | Incorrect assumptions (affecting behavior) | REJECT |
+      | Plausible-but-wrong code | REJECT |
+      | Significant context mismatch with codebase | REJECT |
+      | Scope creep | APPROVE (with warning noted) |
+      | Minor style deviations only | APPROVE |
+      | Code fits context and works | APPROVE |
+
+      **Note:** Scope creep is noted as a warning but doesn't warrant REJECT alone.
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | No AI-specific issues | `[AI_REVIEW:APPROVE]` |
+      | Issues found | `[AI_REVIEW:REJECT]` |
+
+      ### Output Examples
+
+      **APPROVE case:**
+      ```
+      [AI_REVIEW:APPROVE]
+
+      Verification result: No issues
+      ```
+
+      **REJECT case:**
+      ```
+      [AI_REVIEW:REJECT]
+
+      Issues:
+      1. Non-existent API used: `fetch.json()` → `response.json()`
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: ai_review (AI-Generated Code Review)
       - Report Directory: .takt/reports/{report_dir}/
       - Report File: .takt/reports/{report_dir}/04-ai-review.md
@@ -293,13 +498,9 @@ steps:
       - Context fit with existing codebase
       - Scope creep detection
 
-      Include:
-      - [AI_REVIEW:APPROVE] if no AI-specific issues found
-      - [AI_REVIEW:REJECT] if issues detected (list specific problems)
-
       **Report output:** Output to the `Report File` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Report format:**
       ```markdown
@@ -344,9 +545,42 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | AI issue fixes complete | `[CODER:DONE]` |
+      | Cannot decide/insufficient info | `[CODER:BLOCKED]` |
+
+      **Important**: When in doubt, `[BLOCKED]`. Don't decide on your own.
+
+      ### Output Examples
+
+      **DONE case:**
+      ```
+      Fixed AI Reviewer's issues.
+      - Fixed non-existent API: `fetch.json()` → `response.json()`
+
+      [CODER:DONE]
+      ```
+
+      **BLOCKED case:**
+      ```
+      [CODER:BLOCKED]
+
+      Reason: Fix method unclear
+      Required info: Alternative API specification
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: ai_fix
 
       ## AI Review Feedback (This is the latest instruction - prioritize this)
@@ -366,8 +600,6 @@ steps:
       - Aligning with existing codebase patterns
       - Removing scope creep
 
-      Include [CODER:DONE] when complete.
-      Include [CODER:BLOCKED] if you cannot proceed.
     pass_previous_response: true
     transitions:
       - condition: done
@@ -383,9 +615,49 @@ steps:
       - Grep
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Judgment Criteria
+
+      | Situation | Judgment |
+      |-----------|----------|
+      | Critical vulnerability | REJECT |
+      | Medium severity vulnerability | REJECT |
+      | Minor issues/warnings only | APPROVE (note warnings) |
+      | No security issues | APPROVE |
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | No security issues | `[SECURITY:APPROVE]` |
+      | Vulnerabilities require fixes | `[SECURITY:REJECT]` |
+
+      ### Output Examples
+
+      **APPROVE case:**
+      ```
+      [SECURITY:APPROVE]
+
+      No security issues detected.
+      ```
+
+      **REJECT case:**
+      ```
+      [SECURITY:REJECT]
+
+      Vulnerabilities:
+      1. SQL Injection: `src/db.ts:42`
+         Fix: Use parameterized query
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: security_review
       - Report Directory: .takt/reports/{report_dir}/
       - Report File: .takt/reports/{report_dir}/05-security-review.md
@@ -405,13 +677,9 @@ steps:
       - Data exposure risks
       - Cryptographic weaknesses
 
-      Include:
-      - [SECURITY:APPROVE] if no security issues found
-      - [SECURITY:REJECT] if vulnerabilities detected (list specific issues)
-
       **Report output:** Output to the `Report File` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Report format:**
       ```markdown
@@ -459,9 +727,42 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | Security fixes complete | `[CODER:DONE]` |
+      | Cannot decide/insufficient info | `[CODER:BLOCKED]` |
+
+      **Important**: Security issues require highest priority.
+
+      ### Output Examples
+
+      **DONE case:**
+      ```
+      Fixed security issues.
+      - SQL injection fix: Changed to parameterized query
+
+      [CODER:DONE]
+      ```
+
+      **BLOCKED case:**
+      ```
+      [CODER:BLOCKED]
+
+      Reason: Fix method unclear
+      Required info: Auth library specification
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: security_fix
 
       ## Security Review Feedback (This is the latest instruction - prioritize this)
@@ -477,8 +778,6 @@ steps:
       **Important**: Fix the vulnerabilities identified in the security review.
       Security issues require highest priority.
 
-      Include [CODER:DONE] when complete.
-      Include [CODER:BLOCKED] if you cannot proceed.
     pass_previous_response: true
     transitions:
       - condition: done
@@ -497,9 +796,43 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | Architect's issues fixed | `[CODER:DONE]` |
+      | Cannot decide/insufficient info | `[CODER:BLOCKED]` |
+
+      **Important**: When in doubt, `[BLOCKED]`. Don't decide on your own.
+
+      ### Output Examples
+
+      **DONE case:**
+      ```
+      Fixed Architect's issues.
+      - File split: Split into 3 files
+      - Added type definitions
+
+      [CODER:DONE]
+      ```
+
+      **BLOCKED case:**
+      ```
+      [CODER:BLOCKED]
+
+      Reason: Fix approach unclear
+      Required info: Specific split method
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: fix
 
       ## Architect Feedback (This is the latest instruction - prioritize this)
@@ -516,8 +849,6 @@ steps:
       The "Original User Request" is reference information, not the latest instruction.
       Review the session conversation history and fix the issues raised by the Architect.
 
-      Include [CODER:DONE] when complete.
-      Include [CODER:BLOCKED] if you cannot proceed.
     pass_previous_response: true
     transitions:
       - condition: done
@@ -534,9 +865,55 @@ steps:
       - Bash
       - WebSearch
       - WebFetch
+    status_rules_prompt: |
+      # ⚠️ REQUIRED: Status Output Rules ⚠️
+
+      **Without this tag, the workflow will stop.**
+      Your final output MUST include a status tag following the rules below.
+
+      ## Judgment Criteria
+
+      | Situation | Judgment |
+      |-----------|----------|
+      | Requirements not met | REJECT |
+      | Tests failing | REJECT |
+      | Build fails | REJECT |
+      | Workarounds remaining | REJECT |
+      | All OK | APPROVE |
+
+      **Principle**: When in doubt, REJECT. Don't give ambiguous approval.
+
+      ## Output Format
+
+      | Situation | Tag |
+      |-----------|-----|
+      | Final approval | `[SUPERVISOR:APPROVE]` |
+      | Return for fixes | `[SUPERVISOR:REJECT]` |
+
+      ### Output Examples
+
+      **APPROVE case:**
+      ```
+      [SUPERVISOR:APPROVE]
+
+      Verification results:
+      - Tests: ✅ All passed
+      - Build: ✅ Succeeded
+      - Requirements met: ✅
+      ```
+
+      **REJECT case:**
+      ```
+      [SUPERVISOR:REJECT]
+
+      Issues:
+      1. Tests failing: `npm test` - 2 failures
+      2. Requirements not met: Login feature not implemented
+      ```
     instruction_template: |
       ## Workflow Context
-      - Iteration: {iteration}/{max_iterations}
+      - Iteration: {iteration}/{max_iterations} (workflow-wide)
+      - Step Iteration: {step_iteration} (times this step has run)
       - Step: supervise (final verification)
       - Report Directory: .takt/reports/{report_dir}/
       - Report Files:
@@ -564,7 +941,7 @@ steps:
 
       **Report output:** Output to the `Report Files` specified above.
       - If file does not exist: Create new file
-      - If file exists: Append with `## Iteration {iteration}` section
+      - If file exists: Append with `## Iteration {step_iteration}` section
 
       **Validation report format:**
       ```markdown
@@ -619,10 +996,6 @@ steps:
       npm run build
       ```
       ```
-
-      Output:
-      - [SUPERVISOR:APPROVE] if ready to merge
-      - [SUPERVISOR:REJECT] if issues found (specify the issues)
     transitions:
       - condition: approved
         next_step: COMPLETE