takt 0.10.0 → 0.11.1

package/README.md

@@ -465,6 +465,7 @@ TAKT includes multiple builtin pieces:
 | `review-only` | Read-only code review piece that makes no changes. |
 | `structural-reform` | Full project review and structural reform: iterative codebase restructuring with staged file splits. |
 | `unit-test` | Unit test focused piece: test analysis → test implementation → review → fix. |
+| `e2e-test` | E2E test focused piece: E2E analysis → E2E implementation → review → fix (Vitest-based E2E flow). |
 
 **Per-persona provider overrides:** Use `persona_providers` in config to route specific personas to different providers (e.g., coder on Codex, reviewers on Claude) without duplicating pieces.
 

package/builtins/en/instructions/ai-fix.md

@@ -1,4 +1,5 @@
 **This is AI Review iteration #{movement_iteration}.**
+Use reports in the Report Directory as the primary source of truth. If additional context is needed, you may consult Previous Response and conversation history as secondary sources (Previous Response may be unavailable). If information conflicts, prioritize reports in the Report Directory and actual file contents.
 
 From the 2nd iteration onward, it means the previous fixes were not actually applied.
 **Your belief that they were "already fixed" is incorrect.**

package/builtins/en/instructions/fix-supervisor.md

@@ -1,4 +1,5 @@
 Fix the issues raised by the supervisor.
+Use reports in the Report Directory as the primary source of truth. If additional context is needed, you may consult Previous Response and conversation history as secondary sources (Previous Response may be unavailable). If information conflicts, prioritize reports in the Report Directory and actual file contents.
 
 The supervisor has flagged problems from an overall perspective.
 Address items in order of priority, starting with the highest.

package/builtins/en/instructions/fix.md

@@ -1,5 +1,6 @@
 Address the reviewer's feedback.
-Review the session conversation history and fix the issues raised by the reviewer.
+Use reports in the Report Directory shown in the Piece Context and fix the issues raised by the reviewer.
+Use files in the Report Directory as primary evidence. If additional context is needed, you may consult Previous Response and conversation history as secondary sources (Previous Response may be unavailable). If information conflicts, prioritize reports in the Report Directory and actual file contents.
 
 **Required output (include headings)**
 ## Work results

package/builtins/en/instructions/implement-e2e-test.md

@@ -0,0 +1,51 @@
+Implement E2E tests according to the test plan.
+Refer only to files within the Report Directory shown in the Piece Context. Do not search or reference other report directories.
+
+**Actions:**
+1. Review the test plan report
+2. Implement or update tests following existing E2E layout (e.g., `e2e/specs/`)
+3. Run E2E tests (minimum: `npm run test:e2e:mock`, and targeted spec runs when needed)
+4. If tests fail, analyze root cause, fix test or code, and rerun
+5. Confirm related existing tests are not broken
+
+**Constraints:**
+- Keep the current E2E framework (Vitest) unchanged
+- Keep one scenario per test and make assertions explicit
+- Reuse existing fixtures/helpers/mock strategy for external dependencies
+
+**Scope output contract (create at the start of implementation):**
+```markdown
+# Change Scope Declaration
+
+## Task
+{One-line task summary}
+
+## Planned changes
+| Type | File |
+|------|------|
+| Create | `e2e/specs/example.e2e.ts` |
+
+## Estimated size
+Small / Medium / Large
+
+## Impact area
+- {Affected modules or features}
+```
+
+**Decisions output contract (at implementation completion, only if decisions were made):**
+```markdown
+# Decision Log
+
+## 1. {Decision}
+- **Context**: {Why the decision was needed}
+- **Options considered**: {List of options}
+- **Rationale**: {Reason for the choice}
+```
+
+**Required output (include headings)**
+## Work results
+- {Summary of actions taken}
+## Changes made
+- {Summary of changes}
+## Test results
+- {Command executed and results}

package/builtins/en/instructions/implement.md

@@ -1,5 +1,6 @@
 Implement according to the plan.
 Refer only to files within the Report Directory shown in the Piece Context. Do not search or reference other report directories.
+Use reports in the Report Directory as the primary source of truth. If additional context is needed, you may consult Previous Response and conversation history as secondary sources (Previous Response may be unavailable). If information conflicts, prioritize reports in the Report Directory and actual file contents.
 
 **Important**: Add unit tests alongside the implementation.
 - Add unit tests for newly created classes and functions

package/builtins/en/instructions/plan-e2e-test.md

@@ -0,0 +1,11 @@
+Analyze the target code and identify missing E2E tests.
+
+**Note:** If a Previous Response exists, this is a replan due to rejection.
+Revise the test plan taking that feedback into account.
+
+**Actions:**
+1. Read target features, implementation, and existing E2E specs (`e2e/specs/**/*.e2e.ts`) to understand behavior
+2. Summarize current E2E coverage (happy path, failure path, regression points)
+3. Identify missing E2E scenarios with expected outcomes and observability points
+4. Specify execution commands (`npm run test:e2e:mock` and, when needed, `npx vitest run e2e/specs/<target>.e2e.ts`)
+5. Provide concrete guidance for failure analysis → fix → rerun workflow

package/builtins/en/piece-categories.yaml

@@ -10,7 +10,10 @@ piece_categories:
     pieces:
       - review-fix-minimal
       - review-only
+  🧪 Testing:
+    pieces:
       - unit-test
+      - e2e-test
   🎨 Frontend: {}
   ⚙️ Backend: {}
   🔧 Expert:

package/builtins/en/pieces/coding.yaml

@@ -1,24 +1,6 @@
 name: coding
 description: Lightweight development piece with planning and parallel reviews (plan -> implement -> parallel review -> complete)
 max_iterations: 20
-policies:
-  coding: ../policies/coding.md
-  review: ../policies/review.md
-  testing: ../policies/testing.md
-  ai-antipattern: ../policies/ai-antipattern.md
-knowledge:
-  architecture: ../knowledge/architecture.md
-personas:
-  planner: ../personas/planner.md
-  coder: ../personas/coder.md
-  ai-antipattern-reviewer: ../personas/ai-antipattern-reviewer.md
-  architecture-reviewer: ../personas/architecture-reviewer.md
-instructions:
-  plan: ../instructions/plan.md
-  implement: ../instructions/implement.md
-  ai-review: ../instructions/ai-review.md
-  review-arch: ../instructions/review-arch.md
-  fix: ../instructions/fix.md
 initial_movement: plan
 movements:
   - name: plan
@@ -150,7 +132,3 @@ movements:
       - condition: Cannot determine, insufficient information
         next: ABORT
     instruction: fix
-report_formats:
-  plan: ../output-contracts/plan.md
-  ai-review: ../output-contracts/ai-review.md
-  architecture-review: ../output-contracts/architecture-review.md

package/builtins/en/pieces/compound-eye.yaml

@@ -1,11 +1,6 @@
 name: compound-eye
 description: Multi-model review - send the same instruction to Claude and Codex simultaneously, synthesize both responses
 max_iterations: 10
-knowledge:
-  architecture: ../knowledge/architecture.md
-personas:
-  coder: ../personas/coder.md
-  supervisor: ../personas/supervisor.md
 initial_movement: evaluate
 movements:
   - name: evaluate

package/builtins/en/pieces/default.yaml

@@ -1,32 +1,6 @@
 name: default
 description: Standard development piece with planning and specialized reviews
 max_iterations: 30
-policies:
-  coding: ../policies/coding.md
-  review: ../policies/review.md
-  testing: ../policies/testing.md
-  ai-antipattern: ../policies/ai-antipattern.md
-  qa: ../policies/qa.md
-knowledge:
-  backend: ../knowledge/backend.md
-  architecture: ../knowledge/architecture.md
-personas:
-  planner: ../personas/planner.md
-  coder: ../personas/coder.md
-  ai-antipattern-reviewer: ../personas/ai-antipattern-reviewer.md
-  architecture-reviewer: ../personas/architecture-reviewer.md
-  qa-reviewer: ../personas/qa-reviewer.md
-  supervisor: ../personas/supervisor.md
-instructions:
-  plan: ../instructions/plan.md
-  implement: ../instructions/implement.md
-  ai-review: ../instructions/ai-review.md
-  ai-fix: ../instructions/ai-fix.md
-  arbitrate: ../instructions/arbitrate.md
-  review-arch: ../instructions/review-arch.md
-  review-qa: ../instructions/review-qa.md
-  fix: ../instructions/fix.md
-  supervise: ../instructions/supervise.md
 initial_movement: plan
 loop_monitors:
   - cycle:
@@ -282,10 +256,3 @@ movements:
       report:
         - Validation: 07-supervisor-validation.md
         - Summary: summary.md
-report_formats:
-  plan: ../output-contracts/plan.md
-  ai-review: ../output-contracts/ai-review.md
-  architecture-review: ../output-contracts/architecture-review.md
-  qa-review: ../output-contracts/qa-review.md
-  validation: ../output-contracts/validation.md
-  summary: ../output-contracts/summary.md

package/builtins/en/pieces/e2e-test.yaml

@@ -0,0 +1,236 @@
+name: e2e-test
+description: E2E test focused piece (E2E analysis → E2E implementation → review → fix)
+max_iterations: 20
+initial_movement: plan_test
+loop_monitors:
+  - cycle:
+      - ai_review
+      - ai_fix
+    threshold: 3
+    judge:
+      persona: supervisor
+      instruction_template: |
+        The ai_review ↔ ai_fix loop has repeated {cycle_count} times.
+
+        Review the reports from each cycle and determine whether this loop
+        is healthy (making progress) or unproductive (repeating the same issues).
+
+        **Reports to reference:**
+        - AI Review results: {report:04-ai-review.md}
+
+        **Judgment criteria:**
+        - Are new issues being found/fixed in each cycle?
+        - Are the same findings being repeated?
+        - Are fixes actually being applied?
+      rules:
+        - condition: Healthy (making progress)
+          next: ai_review
+        - condition: Unproductive (no improvement)
+          next: review_test
+movements:
+  - name: plan_test
+    edit: false
+    persona: test-planner
+    policy: testing
+    knowledge:
+      - architecture
+      - backend
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+      - Bash
+      - WebSearch
+      - WebFetch
+    rules:
+      - condition: Test plan complete
+        next: implement_test
+      - condition: User is asking a question (not an E2E test task)
+        next: COMPLETE
+      - condition: Requirements unclear, insufficient info
+        next: ABORT
+        appendix: |
+          Clarifications needed:
+          - {Question 1}
+          - {Question 2}
+    instruction: plan-e2e-test
+    output_contracts:
+      report:
+        - name: 00-test-plan.md
+          format: test-plan
+
+  - name: implement_test
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+    session: refresh
+    knowledge:
+      - backend
+      - architecture
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+      - Edit
+      - Write
+      - Bash
+      - WebSearch
+      - WebFetch
+    permission_mode: edit
+    rules:
+      - condition: Test implementation complete
+        next: ai_review
+      - condition: No implementation (report only)
+        next: ai_review
+      - condition: Cannot proceed, insufficient info
+        next: ai_review
+      - condition: User input required
+        next: implement_test
+        requires_user_input: true
+        interactive_only: true
+    instruction: implement-e2e-test
+    output_contracts:
+      report:
+        - Scope: 02-coder-scope.md
+        - Decisions: 03-coder-decisions.md
+
+  - name: ai_review
+    edit: false
+    persona: ai-antipattern-reviewer
+    policy:
+      - review
+      - ai-antipattern
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+      - WebSearch
+      - WebFetch
+    rules:
+      - condition: No AI-specific issues
+        next: review_test
+      - condition: AI-specific issues found
+        next: ai_fix
+    instruction: ai-review
+    output_contracts:
+      report:
+        - name: 04-ai-review.md
+          format: ai-review
+
+  - name: ai_fix
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+    session: refresh
+    knowledge:
+      - backend
+      - architecture
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+      - Edit
+      - Write
+      - Bash
+      - WebSearch
+      - WebFetch
+    permission_mode: edit
+    rules:
+      - condition: AI issues fixed
+        next: ai_review
+      - condition: No fix needed (verified target files/spec)
+        next: ai_no_fix
+      - condition: Cannot proceed, insufficient info
+        next: ai_no_fix
+    instruction: ai-fix
+
+  - name: ai_no_fix
+    edit: false
+    persona: architecture-reviewer
+    policy: review
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+    rules:
+      - condition: ai_review's findings are valid (fix required)
+        next: ai_fix
+      - condition: ai_fix's judgment is valid (no fix needed)
+        next: review_test
+    instruction: arbitrate
+
+  - name: review_test
+    edit: false
+    persona: qa-reviewer
+    policy:
+      - review
+      - qa
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+      - WebSearch
+      - WebFetch
+    rules:
+      - condition: approved
+        next: supervise
+      - condition: needs_fix
+        next: fix
+    instruction: review-test
+    output_contracts:
+      report:
+        - name: 05-qa-review.md
+          format: qa-review
+
+  - name: fix
+    edit: true
+    persona: coder
+    policy:
+      - coding
+      - testing
+    session: refresh
+    knowledge:
+      - backend
+      - architecture
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+      - Edit
+      - Write
+      - Bash
+      - WebSearch
+      - WebFetch
+    permission_mode: edit
+    rules:
+      - condition: Fix complete
+        next: review_test
+      - condition: Cannot proceed, insufficient info
+        next: plan_test
+    instruction: fix
+
+  - name: supervise
+    edit: false
+    persona: supervisor
+    policy: review
+    allowed_tools:
+      - Read
+      - Glob
+      - Grep
+      - Bash
+      - WebSearch
+      - WebFetch
+    rules:
+      - condition: All checks passed
+        next: COMPLETE
+      - condition: Requirements unmet, tests failing, build errors
+        next: plan_test
+    instruction: supervise
+    output_contracts:
+      report:
+        - Validation: 06-supervisor-validation.md
+        - Summary: summary.md

package/builtins/en/pieces/expert-cqrs.yaml

@@ -1,41 +1,6 @@
 name: expert-cqrs
 description: CQRS+ES, Frontend, Security, QA Expert Review
 max_iterations: 30
-policies:
-  coding: ../policies/coding.md
-  review: ../policies/review.md
-  testing: ../policies/testing.md
-  ai-antipattern: ../policies/ai-antipattern.md
-  qa: ../policies/qa.md
-knowledge:
-  frontend: ../knowledge/frontend.md
-  backend: ../knowledge/backend.md
-  cqrs-es: ../knowledge/cqrs-es.md
-  security: ../knowledge/security.md
-  architecture: ../knowledge/architecture.md
-personas:
-  planner: ../personas/planner.md
-  coder: ../personas/coder.md
-  ai-antipattern-reviewer: ../personas/ai-antipattern-reviewer.md
-  architecture-reviewer: ../personas/architecture-reviewer.md
-  cqrs-es-reviewer: ../personas/cqrs-es-reviewer.md
-  frontend-reviewer: ../personas/frontend-reviewer.md
-  security-reviewer: ../personas/security-reviewer.md
-  qa-reviewer: ../personas/qa-reviewer.md
-  expert-supervisor: ../personas/expert-supervisor.md
-instructions:
-  plan: ../instructions/plan.md
-  implement: ../instructions/implement.md
-  ai-review: ../instructions/ai-review.md
-  ai-fix: ../instructions/ai-fix.md
-  arbitrate: ../instructions/arbitrate.md
-  review-cqrs-es: ../instructions/review-cqrs-es.md
-  review-frontend: ../instructions/review-frontend.md
-  review-security: ../instructions/review-security.md
-  review-qa: ../instructions/review-qa.md
-  fix: ../instructions/fix.md
-  supervise: ../instructions/supervise.md
-  fix-supervisor: ../instructions/fix-supervisor.md
 initial_movement: plan
 movements:
   - name: plan
@@ -61,6 +26,7 @@ movements:
   - name: implement
     edit: true
     persona: coder
+    pass_previous_response: false
     policy:
       - coding
       - testing
@@ -121,6 +87,7 @@ movements:
   - name: ai_fix
     edit: true
     persona: coder
+    pass_previous_response: false
     policy:
       - coding
       - testing
@@ -251,6 +218,7 @@ movements:
   - name: fix
     edit: true
     persona: coder
+    pass_previous_response: false
     policy:
       - coding
       - testing
@@ -299,6 +267,7 @@ movements:
   - name: fix_supervisor
     edit: true
     persona: coder
+    pass_previous_response: false
     policy:
       - coding
       - testing
@@ -323,12 +292,3 @@ movements:
         next: supervise
       - condition: Unable to proceed with fixes
         next: plan
-report_formats:
-  plan: ../output-contracts/plan.md
-  ai-review: ../output-contracts/ai-review.md
-  cqrs-es-review: ../output-contracts/cqrs-es-review.md
-  frontend-review: ../output-contracts/frontend-review.md
-  security-review: ../output-contracts/security-review.md
-  qa-review: ../output-contracts/qa-review.md
-  validation: ../output-contracts/validation.md
-  summary: ../output-contracts/summary.md

package/builtins/en/pieces/expert.yaml

@@ -1,39 +1,6 @@
 name: expert
 description: Architecture, Frontend, Security, QA Expert Review
 max_iterations: 30
-policies:
-  coding: ../policies/coding.md
-  review: ../policies/review.md
-  testing: ../policies/testing.md
-  ai-antipattern: ../policies/ai-antipattern.md
-  qa: ../policies/qa.md
-knowledge:
-  frontend: ../knowledge/frontend.md
-  backend: ../knowledge/backend.md
-  security: ../knowledge/security.md
-  architecture: ../knowledge/architecture.md
-personas:
-  planner: ../personas/planner.md
-  coder: ../personas/coder.md
-  ai-antipattern-reviewer: ../personas/ai-antipattern-reviewer.md
-  architecture-reviewer: ../personas/architecture-reviewer.md
-  frontend-reviewer: ../personas/frontend-reviewer.md
-  security-reviewer: ../personas/security-reviewer.md
-  qa-reviewer: ../personas/qa-reviewer.md
-  expert-supervisor: ../personas/expert-supervisor.md
-instructions:
-  plan: ../instructions/plan.md
-  implement: ../instructions/implement.md
-  ai-review: ../instructions/ai-review.md
-  ai-fix: ../instructions/ai-fix.md
-  arbitrate: ../instructions/arbitrate.md
-  review-arch: ../instructions/review-arch.md
-  review-frontend: ../instructions/review-frontend.md
-  review-security: ../instructions/review-security.md
-  review-qa: ../instructions/review-qa.md
-  fix: ../instructions/fix.md
-  supervise: ../instructions/supervise.md
-  fix-supervisor: ../instructions/fix-supervisor.md
 initial_movement: plan
 movements:
   - name: plan
@@ -59,6 +26,7 @@ movements:
   - name: implement
     edit: true
     persona: coder
+    pass_previous_response: false
     policy:
       - coding
       - testing
@@ -118,6 +86,7 @@ movements:
   - name: ai_fix
     edit: true
     persona: coder
+    pass_previous_response: false
     policy:
       - coding
       - testing
@@ -247,6 +216,7 @@ movements:
   - name: fix
     edit: true
     persona: coder
+    pass_previous_response: false
     policy:
       - coding
       - testing
@@ -294,6 +264,7 @@ movements:
   - name: fix_supervisor
     edit: true
     persona: coder
+    pass_previous_response: false
     policy:
       - coding
       - testing
@@ -317,12 +288,3 @@ movements:
         next: supervise
       - condition: Unable to proceed with fixes
         next: plan
-report_formats:
-  plan: ../output-contracts/plan.md
-  ai-review: ../output-contracts/ai-review.md
-  architecture-review: ../output-contracts/architecture-review.md
-  frontend-review: ../output-contracts/frontend-review.md
-  security-review: ../output-contracts/security-review.md
-  qa-review: ../output-contracts/qa-review.md
-  validation: ../output-contracts/validation.md
-  summary: ../output-contracts/summary.md

package/builtins/en/pieces/magi.yaml

@@ -1,10 +1,6 @@
 name: magi
 description: MAGI Deliberation System - Analyze from 3 perspectives and decide by majority
 max_iterations: 5
-personas:
-  melchior: ../personas/melchior.md
-  balthasar: ../personas/balthasar.md
-  casper: ../personas/casper.md
 initial_movement: melchior
 movements:
   - name: melchior

package/builtins/en/pieces/minimal.yaml

@@ -1,21 +1,6 @@
 name: minimal
 description: Minimal development piece (implement -> parallel review -> fix if needed -> complete)
 max_iterations: 20
-policies:
-  coding: ../policies/coding.md
-  review: ../policies/review.md
-  testing: ../policies/testing.md
-  ai-antipattern: ../policies/ai-antipattern.md
-personas:
-  coder: ../personas/coder.md
-  ai-antipattern-reviewer: ../personas/ai-antipattern-reviewer.md
-  supervisor: ../personas/supervisor.md
-instructions:
-  implement: ../instructions/implement.md
-  review-ai: ../instructions/review-ai.md
-  ai-fix: ../instructions/ai-fix.md
-  supervise: ../instructions/supervise.md
-  fix-supervisor: ../instructions/fix-supervisor.md
 initial_movement: implement
 movements:
   - name: implement
@@ -190,5 +175,3 @@ movements:
       - condition: Cannot proceed, insufficient info
         next: implement
     instruction: fix-supervisor
-report_formats:
-  ai-review: ../output-contracts/ai-review.md

package/builtins/en/pieces/passthrough.yaml

@@ -1,11 +1,6 @@
 name: passthrough
 description: Single-agent thin wrapper. Pass task directly to coder as-is.
 max_iterations: 10
-policies:
-  coding: ../policies/coding.md
-  testing: ../policies/testing.md
-personas:
-  coder: ../personas/coder.md
 initial_movement: execute
 movements:
   - name: execute

package/builtins/en/pieces/research.yaml

@@ -1,10 +1,6 @@
 name: research
 description: Research piece - autonomously executes research without asking questions
 max_iterations: 10
-personas:
-  research-planner: ../personas/research-planner.md
-  research-digger: ../personas/research-digger.md
-  research-supervisor: ../personas/research-supervisor.md
 initial_movement: plan
 movements:
   - name: plan