takt 0.14.0 → 0.16.0

package/README.md

@@ -366,7 +366,7 @@ movements:
     policy: coding
     knowledge: architecture
     edit: true
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Implementation complete
         next: review
@@ -578,12 +578,28 @@ concurrency: 1           # Parallel task count for takt run (1-10, default: 1 =
 task_poll_interval_ms: 500  # Polling interval for new tasks during takt run (100-5000, default: 500)
 interactive_preview_movements: 3  # Movement previews in interactive mode (0-10, default: 3)
 
+# Runtime environment defaults (applies to all pieces unless piece_config.runtime overrides)
+# runtime:
+#   prepare:
+#     - gradle    # Prepare Gradle cache/config in .runtime/
+#     - node      # Prepare npm cache in .runtime/
+
 # Per-persona provider overrides (optional)
 # Route specific personas to different providers without duplicating pieces
 # persona_providers:
 #   coder: codex             # Run coder on Codex
 #   ai-antipattern-reviewer: claude  # Keep reviewers on Claude
 
+# Provider-specific permission profiles (optional)
+# Priority: project override → global override → project default → global default → required_permission_mode (floor)
+# provider_profiles:
+#   codex:
+#     default_permission_mode: full
+#     movement_permission_overrides:
+#       ai_review: readonly
+#   claude:
+#     default_permission_mode: edit
+
 # API Key configuration (optional)
 # Can be overridden by environment variables TAKT_ANTHROPIC_API_KEY / TAKT_OPENAI_API_KEY / TAKT_OPENCODE_API_KEY
 anthropic_api_key: sk-ant-...  # For Claude (Anthropic)
@@ -733,7 +749,7 @@ movements:
   - name: implement
     persona: coder
     edit: true
-    permission_mode: edit
+    required_permission_mode: edit
     pass_previous_response: true
     rules:
       - condition: Complete
@@ -802,7 +818,7 @@ Special `next` values: `COMPLETE` (success), `ABORT` (failure)
 | `allowed_tools` | - | List of tools agent can use (Read, Glob, Grep, Edit, Write, Bash, etc.) |
 | `provider` | - | Override provider for this movement (`claude`, `codex`, or `opencode`) |
 | `model` | - | Override model for this movement |
-| `permission_mode` | - | Permission mode: `readonly`, `edit`, `full` (provider-independent) |
+| `required_permission_mode` | - | Required minimum permission mode: `readonly`, `edit`, `full` (acts as floor; actual mode resolved via `provider_profiles`) |
 | `provider_options` | - | Provider-specific options (e.g. `codex.network_access`, `opencode.network_access`) |
 | `output_contracts` | - | Output contract definitions for report files |
 | `quality_gates` | - | AI directives for movement completion requirements |
@@ -817,8 +833,14 @@ piece_config:
       network_access: true
     opencode:
       network_access: true
+  runtime:
+    prepare:
+      - gradle
+      - node
 ```
 
+Runtime `prepare` entries can be builtin presets (`gradle`, `node`) or paths to custom shell scripts. Scripts receive `TAKT_RUNTIME_ROOT` and related env vars, and can export additional variables via stdout.
+
 ## API Usage Example
 
 ```typescript

package/builtins/en/config.yaml

@@ -1,85 +1,92 @@
-# TAKT Global Configuration
+# TAKT global configuration sample
 # Location: ~/.takt/config.yaml
 
-# ── Basic ──
-
-# Language (en | ja)
+# ---- Core ----
 language: en
-
-# Default piece when no piece is specified
 default_piece: default
-
-# Log level (debug | info | warn | error)
 log_level: info
 
-# ── Provider & Model ──
-
-# Provider runtime (claude | codex)
+# ---- Provider ----
+# provider: claude | codex | opencode | mock
 provider: claude
 
-# Default model (optional)
-# Claude: opus, sonnet, haiku
-# Codex: gpt-5.2-codex, gpt-5.1-codex, etc.
+# Model (optional)
+# Claude examples: opus, sonnet, haiku
+# Codex examples: gpt-5.2-codex, gpt-5.1-codex
+# OpenCode format: provider/model
 # model: sonnet
 
-# Per-persona provider override (optional)
-# Override provider for specific personas. Others use the global provider.
+# Per-persona provider override
 # persona_providers:
 #   coder: codex
-
-# ── API Keys ──
-# Optional. Environment variables take priority:
-#   TAKT_ANTHROPIC_API_KEY, TAKT_OPENAI_API_KEY
-
+#   reviewer: claude
+
+# Provider-specific movement permission policy
+# Priority:
+# 1) project provider_profiles override
+# 2) global provider_profiles override
+# 3) project provider_profiles default
+# 4) global provider_profiles default
+# 5) movement.required_permission_mode (minimum floor)
+# provider_profiles:
+#   codex:
+#     default_permission_mode: full
+#     movement_permission_overrides:
+#       ai_review: readonly
+#   claude:
+#     default_permission_mode: edit
+
+# Provider-specific runtime options
+# provider_options:
+#   codex:
+#     network_access: true
+#   claude:
+#     sandbox:
+#       allow_unsandboxed_commands: true
+
+# ---- API Keys ----
+# Environment variables take priority:
+# TAKT_ANTHROPIC_API_KEY / TAKT_OPENAI_API_KEY / TAKT_OPENCODE_API_KEY
 # anthropic_api_key: ""
 # openai_api_key: ""
+# opencode_api_key: ""
 
-# ── Execution ──
+# ---- Runtime ----
+# Global runtime preparation (piece_config.runtime overrides this)
+# runtime:
+#   prepare:
+#     - gradle
+#     - node
 
-# Worktree (shared clone) directory (default: ../{clone-name} relative to project)
+# ---- Execution ----
 # worktree_dir: ~/takt-worktrees
-
-# Auto-create PR after worktree execution (default: prompt in interactive mode)
 # auto_pr: false
-
-# Prevent macOS idle sleep during execution using caffeinate (default: false)
 # prevent_sleep: false
 
-# ── Parallel Execution (takt run) ──
-
-# Number of tasks to run concurrently (1 = sequential, max: 10)
+# ---- Run Loop ----
 # concurrency: 1
-
-# Polling interval in ms for picking up new tasks (100-5000, default: 500)
 # task_poll_interval_ms: 500
-
-# ── Interactive Mode ──
-
-# Number of movement previews shown in interactive mode (0 to disable, max: 10)
 # interactive_preview_movements: 3
-
-# Branch name generation strategy (romaji: fast default | ai: slow)
 # branch_name_strategy: romaji
 
-# ── Output ──
-
-# Notification sounds (default: true)
-# notification_sound: true
-
-# Minimal output for CI - suppress AI output (default: false)
+# ---- Output ----
 # minimal_output: false
-
-# ── Builtin Pieces ──
-
-# Enable builtin pieces (default: true)
+# notification_sound: true
+# notification_sound_events:
+#   iteration_limit: true
+#   piece_complete: true
+#   piece_abort: true
+#   run_complete: true
+#   run_abort: true
+# observability:
+#   provider_events: true
+
+# ---- Builtins ----
 # enable_builtin_pieces: true
-
-# Exclude specific builtins from loading
 # disabled_builtins:
 #   - magi
 
-# ── Pipeline Mode (--pipeline) ──
-
+# ---- Pipeline ----
 # pipeline:
 #   default_branch_prefix: "takt/"
 #   commit_message_template: "feat: {title} (#{issue})"
@@ -88,14 +95,11 @@ provider: claude
 #     {issue_body}
 #     Closes #{issue}
 
-# ── Preferences ──
-
-# Custom paths for preference files
+# ---- Preferences ----
 # bookmarks_file: ~/.takt/preferences/bookmarks.yaml
 # piece_categories_file: ~/.takt/preferences/piece-categories.yaml
 
-# ── Debug ──
-
+# ---- Debug ----
 # debug:
 #   enabled: false
 #   log_file: ~/.takt/logs/debug.log

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

@@ -0,0 +1,12 @@
+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:03-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?

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

@@ -219,7 +219,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: reviewers

package/builtins/en/pieces/backend.yaml

@@ -216,7 +216,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: reviewers

package/builtins/en/pieces/coding.yaml

@@ -51,7 +51,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Implementation complete
         next: reviewers
@@ -132,7 +132,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: reviewers

package/builtins/en/pieces/default.yaml

@@ -82,7 +82,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Implementation complete
         next: ai_review
@@ -142,7 +142,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: AI issues fixed
         next: ai_review
@@ -234,7 +234,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: reviewers

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

@@ -85,7 +85,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Test implementation complete
         next: ai_review
@@ -145,7 +145,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: AI issues fixed
         next: ai_review
@@ -212,7 +212,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: review_test

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

@@ -9,6 +9,19 @@ piece_config:
 
 max_movements: 30
 initial_movement: plan
+loop_monitors:
+  - cycle:
+      - ai_review
+      - ai_fix
+    threshold: 3
+    judge:
+      persona: supervisor
+      instruction_template: loop-monitor-ai-fix
+      rules:
+        - condition: Healthy (making progress)
+          next: ai_review
+        - condition: Unproductive (same findings repeated or fixes not reflected)
+          next: ai_no_fix
 movements:
   - name: plan
     edit: false
@@ -241,7 +254,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: reviewers

package/builtins/en/pieces/expert.yaml

@@ -9,6 +9,19 @@ piece_config:
 
 max_movements: 30
 initial_movement: plan
+loop_monitors:
+  - cycle:
+      - ai_review
+      - ai_fix
+    threshold: 3
+    judge:
+      persona: supervisor
+      instruction_template: loop-monitor-ai-fix
+      rules:
+        - condition: Healthy (making progress)
+          next: ai_review
+        - condition: Unproductive (same findings repeated or fixes not reflected)
+          next: ai_no_fix
 movements:
   - name: plan
     edit: false
@@ -238,7 +251,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: reviewers

package/builtins/en/pieces/frontend.yaml

@@ -235,7 +235,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: reviewers

package/builtins/en/pieces/minimal.yaml

@@ -25,7 +25,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     instruction: implement
     rules:
       - condition: Implementation complete
@@ -106,7 +106,7 @@ movements:
           - Bash
           - WebSearch
           - WebFetch
-        permission_mode: edit
+        required_permission_mode: edit
         rules:
           - condition: AI Reviewer's issues fixed
           - condition: No fix needed (verified target files/spec)
@@ -126,7 +126,7 @@ movements:
           - Bash
           - WebSearch
           - WebFetch
-        permission_mode: edit
+        required_permission_mode: edit
         rules:
           - condition: Supervisor's issues fixed
           - condition: Cannot proceed, insufficient info
@@ -151,7 +151,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: AI Reviewer's issues fixed
         next: reviewers
@@ -175,7 +175,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Supervisor's issues fixed
         next: reviewers

package/builtins/en/pieces/passthrough.yaml

@@ -25,7 +25,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Task complete
         next: COMPLETE

package/builtins/en/pieces/review-fix-minimal.yaml

@@ -25,7 +25,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     instruction: implement
     rules:
       - condition: Implementation complete
@@ -106,7 +106,7 @@ movements:
           - Bash
           - WebSearch
           - WebFetch
-        permission_mode: edit
+        required_permission_mode: edit
         rules:
           - condition: AI Reviewer's issues fixed
           - condition: No fix needed (verified target files/spec)
@@ -126,7 +126,7 @@ movements:
           - Bash
           - WebSearch
           - WebFetch
-        permission_mode: edit
+        required_permission_mode: edit
         rules:
           - condition: Supervisor's issues fixed
           - condition: Cannot proceed, insufficient info
@@ -151,7 +151,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: AI Reviewer's issues fixed
         next: reviewers
@@ -175,7 +175,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Supervisor's issues fixed
         next: reviewers

package/builtins/en/pieces/structural-reform.yaml

@@ -226,7 +226,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     instruction: implement
     rules:
       - condition: Implementation complete
@@ -309,7 +309,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: reviewers

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

@@ -85,7 +85,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Test implementation complete
         next: ai_review
@@ -145,7 +145,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: AI issues fixed
         next: ai_review
@@ -212,7 +212,7 @@ movements:
       - Bash
       - WebSearch
       - WebFetch
-    permission_mode: edit
+    required_permission_mode: edit
     rules:
       - condition: Fix complete
         next: review_test