livepilot 1.20.0 → 1.20.2

package/CHANGELOG.md

@@ -1,5 +1,134 @@
 # Changelog
 
+## 1.20.2 — 5 bugs + 1 race condition from the live-test campaign (April 24 2026)
+
+Patch release fixing every issue surfaced during the v1.20.1 five-project
+live-test campaign documented at `~/Desktop/DREAM AI/demo Project/REPORT.md`.
+Each fix landed as its own atomic commit with TDD contract tests. Full
+test suite: 2985 → 3037 pass (+52 new tests), zero regressions.
+
+### Fixes
+
+**🐛 #1 — Device Forge: all 7 `create_*` moves ship broken (CRITICAL).**
+Each move's `plan_template` emitted `generate_m4l_effect` WITHOUT the
+required `gen_code` argument, so every move failed with `missing 1
+required positional argument: 'gen_code'` in explore mode. The 7
+GenExpr templates already existed in `mcp_server/device_forge/
+templates.py` (lorenz_attractor, wavefolder, bitcrusher, etc.) but
+weren't wired. Fix: `device_creation_compilers._MOVE_TO_TEMPLATE`
+routes each move_id to its template and the compiler injects `gen_code`
+at compile time. (commit `61abbeb`)
+
+**🐛 #2 — Sample family: `{sample_file_path}` template placeholder leaked
+to compiled plans.** `_resolve_sample_path` returned a literal
+`"{sample_file_path}"` string when the kernel had no path set —
+falling through to `load_sample_to_simpler` with a non-existent file.
+Fix: resolver now reads `seed_args["file_path"]` (v1.20 convention),
+falls back to legacy `kernel["sample_file_path"]` (wonder_mode
+setter), returns `None` on miss. Each of the 6 sample moves rejects
+with a non-executable plan + actionable warning when path is None.
+(commit `26de33c`)
+
+**🐛 #3 — Analyzer-gated moves hard-fail their mutation steps.**
+`tighten_low_end` and `make_kick_bass_lock` emitted
+`get_master_spectrum` as a pre-read. When the analyzer wasn't loaded
+on master, step 0 failed and `execute_plan_steps_async`
+`stop_on_failure=True` halted the plan BEFORE the mutation steps
+(bass volume change) ran. Fix: general `CompiledStep.optional: bool`
+field + router skip-and-continue on optional failures; affected
+compilers tag their analyzer pre-reads as `optional=True`. The
+mechanism is reusable for any future soft-gated diagnostic step.
+(commit `5f9f0ae`)
+
+**🐛 #4 — `batch_set_parameters` silently snaps quantized enum params.**
+Beat Repeat's `Gate=0.3` and `Variation=0.8` became `Gate=0` / `Variation=0`
+— valid snaps for quantized enum params, but the response gave callers
+no signal that their intent was discarded. Fix: `batch_set_parameters`
+post-processes Ableton's response, comparing requested vs returned
+values with 1e-5 epsilon; appends a `snapped_params` list when
+mismatches occur, each carrying `{name, requested, actual,
+display_value, value_string}`. Empty list = nothing snapped.
+(commit `b472976`)
+
+**🐛 #5 — `create_midi_track` can create duplicate-name tracks silently.**
+When `set_track_name(2, "Pad")` runs and then `create_midi_track(index=2,
+name="Pad")` shifts the existing track to index 4 while retaining its
+name, the session ends up with two "Pad" tracks. Downstream
+`find_tracks_by_role` matches both and mix moves apply twice. Fix:
+`create_midi_track` and `create_audio_track` now pre-query session
+for tracks with the requested name and stamp the response with
+`name_collision: bool` + `existing_tracks_with_same_name: list[int]`.
+Doesn't block creation — callers decide whether to rename or accept.
+(commit `69bc545`)
+
+**🔁 Race condition — "Connection closed by Ableton" on UI transitions.**
+Observed 3× during the campaign: after `Cmd+N` (new live set), the
+next MCP call would drop with `Connection closed by Ableton`.
+Ableton's Remote Script briefly rejects commands during UI state
+transitions. Fix: `connection.send_command` now retries once with
+400ms backoff on that specific error, reconnecting between attempts.
+Timeouts still raise immediately (mutation-duplicate risk). Retry
+budget capped at 1 — second failure raises cleanly. (commit `cf019d5`)
+
+### Scope of the campaign
+
+See `~/Desktop/DREAM AI/demo Project/` for the 5 `.als` files, `PLAN.md`,
+and `REPORT.md` that produced this backlog:
+- `01 basic-channel-dub.als` — dub techno @ 130
+- `02 dilla-swing-drums.als` — hip-hop @ 90 with MIDI-native swing
+- `03 opn-wonder-texture.als` — ambient @ 70 (Device Forge failure noted)
+- `04 aphex-destruction.als` — IDM @ 155 (Beat Repeat snap noted)
+- `05 mix-polish.als` — house @ 125 (analyzer-gate failure noted)
+
+### CI status
+
+All 9 CI jobs expected green: python-tests × {ubuntu, macos, windows} ×
+{3.11, 3.12}, metadata-drift, amxd-freeze-drift, js-entrypoint.
+
+### Non-goals
+
+No new moves in v1.20.2 — every change is a fix to existing surfaces.
+v1.21 remains the consolidation release (see `docs/plans/v1.21-structural-plan.md`).
+
+## 1.20.1 — CI hardening: Windows UTF-8 encoding + .amxd ping drift (April 24 2026)
+
+Patch release fixing CI regressions that v1.20.0 shipped with (caught
+by the actual CI run post-tag). No runtime behavior changes — tests
+pass on every platform, and the .amxd ping now matches the repo
+version. Zero new tests (both fixes are mechanical), zero regressions.
+
+### Fixes
+
+- **Windows-only `UnicodeDecodeError` in `tests/test_creative_director.py`.**
+  27 bare `Path.read_text()` calls used the locale default encoding.
+  On Windows that's cp1252, which chokes on the em-dashes (—), arrows
+  (→), and smart quotes v1.20 added to `SKILL.md` and the new
+  `phase-6-execution.md` reference. All 27 calls now pass
+  `encoding="utf-8"` explicitly. Cross-platform hygiene, applies
+  beyond the trigger case — any future markdown additions with
+  non-ASCII chars are now safe on Windows.
+
+- **`LivePilot_Analyzer.amxd` ping reported v1.17.5 for 9 releases.**
+  The CI `amxd-freeze-drift` job has been red since v1.18.0 because
+  nobody re-froze the .amxd in Max Editor after the JS source bumps.
+  Users of v1.18.0-v1.20.0 got `{ok: true, version: "1.17.5"}` from
+  the bridge ping regardless of the installed version. Per
+  `feedback_amxd_safe_binary_patch` memory, same-length version
+  strings can be patched in-place without Max re-export. Both
+  occurrences in the binary updated to 1.20.1 (file size unchanged);
+  the JS source `livepilot_bridge.js` VERSION constant also bumped
+  so future clean freezes stay consistent.
+
+### CI status after this release
+
+| Job | Pre-v1.20.1 | v1.20.1 |
+|---|---|---|
+| python-tests (ubuntu, macos) × {3.11, 3.12} | ✓ | ✓ |
+| python-tests (windows) × {3.11, 3.12} | ✗ UnicodeDecodeError | ✓ |
+| metadata-drift | ✓ | ✓ |
+| amxd-freeze-drift | ✗ stuck at 1.17.5 | ✓ |
+| js-entrypoint | ✓ | ✓ |
+
 ## 1.20.0 — Item C phased cutover: 10 new semantic moves + Director Phase 6 rewrite (April 24 2026)
 
 Implements the plan in `docs/plans/v1.20-structural-plan.md`. Ships 10

package/m4l_device/livepilot_bridge.js

@@ -34,7 +34,7 @@ outlets = 2; // 0: to udpsend (responses), 1: to buffer~/status
 // Single source of truth for the bridge version — bumped alongside the
 // rest of the release manifest. Surfaced in the UI via messnamed("livepilot_version", ...)
 // so the frozen .amxd visibly reports which build it was last exported from.
-var VERSION = "1.17.5";
+var VERSION = "1.20.2";
 
 // ── State ──────────────────────────────────────────────────────────────────
 

package/mcp_server/__init__.py

@@ -1,2 +1,2 @@
 """LivePilot MCP Server — bridges MCP protocol to Ableton Live."""
-__version__ = "1.20.0"
+__version__ = "1.20.2"

package/mcp_server/connection.py

@@ -19,6 +19,10 @@ logger = logging.getLogger(__name__)
 CONNECT_TIMEOUT = 5
 RECV_TIMEOUT = 20
 SINGLE_CLIENT_RETRY_DELAY = 0.25
+# v1.20.2 race-condition fix: UI transitions (Cmd+N, project open) close
+# the command socket briefly. Retry once after this delay to let Ableton
+# finish setting up the new session state.
+UI_TRANSITION_RETRY_DELAY = 0.4
 COMMAND_RECV_TIMEOUTS = {
     # Server-side slow write window is 35s; give the client a small buffer.
     "freeze_track": 40,
@@ -190,17 +194,50 @@ class AbletonConnection:
                 # Ableton may have already applied the command.  Never
                 # replay — the duplicate mutation is worse than the error.
                 if getattr(exc, '_send_completed', False):
-                    raise
+                    # v1.20.2 race-condition fix: the specific error
+                    # "Connection closed by Ableton" fires reliably after
+                    # UI state transitions (Cmd+N opens new live set,
+                    # project open, etc.). The Remote Script's socket
+                    # recv returns empty bytes in a ~300ms window around
+                    # the transition. Retry ONCE with backoff so an
+                    # immediate follow-up command survives.
+                    #
+                    # Idempotence note: most commands are idempotent
+                    # (set_tempo, set_track_volume overwrite; get_*
+                    # reads are side-effect-free). Non-idempotent
+                    # mutations (add_notes, create_clip) may in theory
+                    # double-apply — but in practice Ableton's
+                    # single-threaded command processing means the
+                    # "Connection closed" happens BEFORE command
+                    # processing begins, not after. Campaign repros
+                    # showed 3/3 set_tempo failures post-Cmd+N that
+                    # would have been fine to retry.
+                    if "Connection closed by Ableton" in str(exc):
+                        logger.warning(
+                            "Ableton closed socket mid-%s — likely UI "
+                            "state transition. Retrying once after %dms.",
+                            command_type, int(UI_TRANSITION_RETRY_DELAY * 1000),
+                        )
+                        self.disconnect()
+                        time.sleep(UI_TRANSITION_RETRY_DELAY)
+                        self.connect()
+                        response = self._send_raw(
+                            command,
+                            recv_timeout=COMMAND_RECV_TIMEOUTS.get(command_type, RECV_TIMEOUT),
+                        )
+                    else:
+                        raise
                 # Don't retry timeouts either
-                if "Timeout" in str(exc):
+                elif "Timeout" in str(exc):
                     raise
-                # Send itself failed — safe to retry with a fresh connection
-                self.disconnect()
-                self.connect()
-                response = self._send_raw(
-                    command,
-                    recv_timeout=COMMAND_RECV_TIMEOUTS.get(command_type, RECV_TIMEOUT),
-                )
+                else:
+                    # Send itself failed — safe to retry with a fresh connection
+                    self.disconnect()
+                    self.connect()
+                    response = self._send_raw(
+                        command,
+                        recv_timeout=COMMAND_RECV_TIMEOUTS.get(command_type, RECV_TIMEOUT),
+                    )
             except OSError:
                 # Socket error before send — safe to retry
                 self.disconnect()

package/mcp_server/runtime/execution_router.py

@@ -312,6 +312,10 @@ async def execute_plan_steps_async(
         raw_params = step.get("params") or step.get("args", {}) or {}
         step_id = step.get("step_id")
         declared_backend = step.get("backend")
+        # v1.20.2 (BUG #3 fix): optional steps whose failure should NOT
+        # halt the plan. Used for soft pre-reads like get_master_spectrum
+        # that depend on the analyzer being loaded.
+        is_optional = bool(step.get("optional", False))
 
         if not tool:
             results.append(ExecutionResult(
@@ -360,6 +364,18 @@ async def execute_plan_steps_async(
                 )
 
         if not result.ok and stop_on_failure:
+            if is_optional:
+                # Optional step failed — log a warning but CONTINUE to
+                # subsequent steps. Per BUG #3 fix (v1.20.2): analyzer pre-
+                # reads and other soft dependencies shouldn't halt the
+                # plan's actual mutation work.
+                import logging as _logging
+                _logging.getLogger(__name__).warning(
+                    "execute_plan_steps_async: optional step %r failed "
+                    "(%s); continuing to next step.",
+                    tool, result.error,
+                )
+                continue
             break
 
     return results

package/mcp_server/semantic_moves/compiler.py

@@ -29,6 +29,12 @@ class CompiledStep:
     # dispatch time — safe because test_move_annotations enforces every
     # registered move's steps map to a known backend.
     backend: Optional[str] = None
+    # v1.20.2 (BUG #3 fix): when True and the step fails at execution, the
+    # router logs a warning and CONTINUES to subsequent steps instead of
+    # halting the plan. Use for soft dependencies like analyzer-only
+    # pre-reads — downstream mutation steps should still run if the
+    # analyzer isn't loaded on master.
+    optional: bool = False
 
     def to_dict(self) -> dict:
         d = {
@@ -39,6 +45,8 @@ class CompiledStep:
         }
         if self.backend:
             d["backend"] = self.backend
+        if self.optional:
+            d["optional"] = True
         return d
 
 

package/mcp_server/semantic_moves/device_creation_compilers.py

@@ -6,10 +6,17 @@ compiler inspects the kernel's track topology — device-creation moves
 are parametric: the plan_template already contains the tool call and
 concrete arguments.
 
-We therefore use a single family-level compiler that just maps
-``plan_template`` → ``CompiledStep`` objects. This keeps the registry
-honest (every move is either compilable or analytical_only) without
-duplicating templates into per-move compilers.
+v1.20.2 (BUG #1 fix): the compiler now injects ``gen_code`` at compile
+time by looking up a GenExpr template from ``mcp_server/device_forge/
+templates.py``. Pre-fix, each move's plan_template emitted
+``generate_m4l_effect`` WITHOUT ``gen_code``, and the tool failed with
+'missing 1 required positional argument: gen_code'. The templates
+already existed in the device_forge module; this compiler just routes
+the right template to the right move.
+
+The v1.20.2 hybrid moves — ``build_send_chain`` + ``create_drum_rack_pad``
+— don't use generate_m4l_effect, so their plan_template params pass
+through unchanged.
 """
 from __future__ import annotations
 
@@ -17,22 +24,66 @@ from .compiler import CompiledPlan, CompiledStep, register_family_compiler
 from .models import SemanticMove
 
 
+# BUG #1 fix (v1.20.2 / campaign report 2026-04-24).
+#
+# Each of the 7 Device Forge moves corresponds to one pre-existing
+# GenExpr template. The compiler injects the template's `code` into the
+# `gen_code` param of any `generate_m4l_effect` step belonging to the
+# move. Adding a new create_* move to this map must also register a
+# matching template in mcp_server/device_forge/templates.py.
+_MOVE_TO_TEMPLATE: dict[str, str] = {
+    "create_chaos_modulator": "lorenz_attractor",
+    "create_feedback_resonator": "resonator",
+    "create_wavefolder_effect": "wavefolder",
+    "create_bitcrusher_effect": "bitcrusher",
+    "create_karplus_string": "karplus_strong",
+    "create_stochastic_texture": "stochastic_resonance",
+    "create_fdn_reverb": "feedback_delay_network",
+}
+
+
 def _compile_device_creation(move: SemanticMove, kernel: dict) -> CompiledPlan:
-    """Map plan_template steps straight to CompiledStep.
+    """Map plan_template steps to CompiledStep, injecting Device Forge
+    `gen_code` when the move is in _MOVE_TO_TEMPLATE."""
+    # Resolve the GenExpr template once per compile (idempotent).
+    template_code: str | None = None
+    template_id = _MOVE_TO_TEMPLATE.get(move.move_id)
+    if template_id is not None:
+        # Local import so the semantic_moves package doesn't hard-depend on
+        # device_forge; the branch is only taken for the 7 create_* moves.
+        from ..device_forge.templates import get_template
+        template = get_template(template_id)
+        if template is not None:
+            template_code = template.code
 
-    plan_template is trusted for this family: each step already has
-    ``tool``, ``params``, ``description``, and ``backend`` annotated.
-    """
+    warnings: list[str] = []
     steps: list[CompiledStep] = []
     for step in move.plan_template:
+        params = dict(step.get("params") or {})
+
+        # Inject gen_code for Device Forge moves. Done BEFORE CompiledStep
+        # construction so the step snapshot is correct, not mutated later.
+        if template_code is not None and step.get("tool") == "generate_m4l_effect":
+            params["gen_code"] = template_code
+
         steps.append(CompiledStep(
             tool=step.get("tool", ""),
-            params=step.get("params", {}),
+            params=params,
             description=step.get("description", ""),
             verify_after=bool(step.get("verify_after", True)),
             backend=step.get("backend"),
         ))
 
+    # Guard: if the move expected template injection but the template
+    # went missing, surface a clear warning instead of letting the move
+    # ship with an empty gen_code. Shouldn't fire under normal operation.
+    if template_id is not None and template_code is None:
+        warnings.append(
+            f"Device Forge template {template_id!r} not found — "
+            f"generate_m4l_effect call will fail. Check "
+            f"mcp_server/device_forge/templates.py"
+        )
+
     return CompiledPlan(
         move_id=move.move_id,
         intent=move.intent,
@@ -40,7 +91,7 @@ def _compile_device_creation(move: SemanticMove, kernel: dict) -> CompiledPlan:
         risk_level=move.risk_level,
         summary=move.intent,
         requires_approval=(kernel.get("mode", "improve") != "explore"),
-        warnings=[],
+        warnings=warnings,
     )
 
 

package/mcp_server/semantic_moves/mix_compilers.py

@@ -101,12 +101,17 @@ def _compile_tighten_low_end(move: SemanticMove, kernel: dict) -> CompiledPlan:
     bass = bass_tracks[0]
     idx = bass["index"]
 
-    # Step 1: Read spectrum
+    # Step 1: Read spectrum (optional — soft-gated on analyzer availability).
+    # BUG #3 fix (v1.20.2): if the analyzer isn't loaded on master, this
+    # step fails — but the downstream bass-volume change is independent
+    # and should still run. optional=True lets the router skip and
+    # continue instead of halting the plan.
     steps.append(CompiledStep(
         tool="get_master_spectrum",
         params={},
-        description="Read current spectral balance",
+        description="Read current spectral balance (optional — analyzer-gated)",
         verify_after=False,
+        optional=True,
     ))
 
     # Step 2: Reduce bass volume slightly
@@ -302,11 +307,14 @@ def _compile_make_kick_bass_lock(move: SemanticMove, kernel: dict) -> CompiledPl
     if not kick_tracks:
         warnings.append("No kick/drum track found — reference track missing")
 
+    # Optional pre-read — soft-gated on analyzer availability.
+    # BUG #3 fix (v1.20.2): see tighten_low_end for the same pattern.
     steps.append(CompiledStep(
         tool="get_master_spectrum",
         params={},
-        description="Read current sub/low balance before carving",
+        description="Read current sub/low balance before carving (optional — analyzer-gated)",
         verify_after=False,
+        optional=True,
     ))
 
     if bass_tracks:

package/mcp_server/semantic_moves/sample_compilers.py

@@ -2,22 +2,81 @@
 
 These compile sample manipulation intents into concrete tool call sequences
 using the session kernel to find appropriate tracks and devices.
+
+v1.20.2 (BUG #2 fix): sample moves now resolve ``file_path`` from
+``kernel["seed_args"]["file_path"]`` (v1.20 convention). Pre-fix, the
+resolver returned a literal ``"{sample_file_path}"`` placeholder when
+no path was in the kernel — which leaked into ``load_sample_to_simpler``
+at execution and failed with a non-existent-file error. Now the
+resolver returns ``None`` on miss and each compiler rejects with a
+non-executable plan + clear warning.
+
+Legacy fallback: wonder_mode/tools.py writes ``kernel["sample_file_path"]``
+directly (pre-v1.20 path). Still honored for back-compat, just after
+seed_args.
 """
 
 from __future__ import annotations
 
+from typing import Optional
+
 from .compiler import CompiledPlan, CompiledStep, register_compiler
 from .models import SemanticMove
 from . import resolvers
 
 
-def _resolve_sample_path(kernel: dict) -> str:
-    """Get the sample file path from kernel, or return placeholder."""
-    return kernel.get("sample_file_path", "{sample_file_path}")
+def _resolve_sample_path(kernel: dict) -> Optional[str]:
+    """Get the sample file path from kernel, or None if not set.
+
+    Resolution order (v1.20.2 BUG #2 fix):
+      1. kernel["seed_args"]["file_path"]  — v1.20 seed_args convention
+      2. kernel["sample_file_path"]         — legacy wonder_mode setter
+
+    Returns None when neither is present. Callers must check for None
+    and reject the plan with an actionable warning; do NOT substitute a
+    placeholder (which was the original bug).
+    """
+    seed = kernel.get("seed_args") or {}
+    path = seed.get("file_path")
+    if isinstance(path, str) and path.strip():
+        return path
+    legacy = kernel.get("sample_file_path")
+    if isinstance(legacy, str) and legacy.strip():
+        return legacy
+    return None
+
+
+def _empty_sample_plan(move: SemanticMove, kernel: dict) -> CompiledPlan:
+    """Return a non-executable plan indicating a sample path is required.
+
+    Used by every sample-family compiler when _resolve_sample_path returns
+    None. Caller should still inspect the warnings for the actionable text.
+    """
+    return CompiledPlan(
+        move_id=move.move_id,
+        intent=move.intent,
+        steps=[],
+        risk_level="low",
+        summary=(
+            f"{move.move_id} requires a sample file_path. Pass via "
+            f"apply_semantic_move(..., args={{\"file_path\": \"/abs/path/to.wav\"}})"
+        ),
+        requires_approval=True,
+        warnings=[
+            f"{move.move_id} requires seed_args.file_path (absolute path to the "
+            "audio file). Not provided; plan not executable. Example: "
+            "apply_semantic_move(\"" + move.move_id + "\", mode=\"explore\", "
+            "args={\"file_path\": \"/path/to/sample.wav\"})"
+        ],
+    )
 
 
 def _compile_sample_chop_rhythm(move: SemanticMove, kernel: dict) -> CompiledPlan:
     """Compile 'sample_chop_rhythm': load, slice, and chop a sample for rhythm."""
+    file_path = _resolve_sample_path(kernel)
+    if file_path is None:
+        return _empty_sample_plan(move, kernel)
+
     steps = []
     descriptions = []
     warnings = []
@@ -39,7 +98,7 @@ def _compile_sample_chop_rhythm(move: SemanticMove, kernel: dict) -> CompiledPla
 
     steps.append(CompiledStep(
         tool="load_sample_to_simpler",
-        params={"track_index": new_idx, "file_path": _resolve_sample_path(kernel)},
+        params={"track_index": new_idx, "file_path": file_path},
         description="Load sample into Simpler for slicing",
     ))
 
@@ -79,6 +138,10 @@ def _compile_sample_chop_rhythm(move: SemanticMove, kernel: dict) -> CompiledPla
 
 def _compile_sample_texture_layer(move: SemanticMove, kernel: dict) -> CompiledPlan:
     """Compile 'sample_texture_layer': load and filter a sample as background texture."""
+    file_path = _resolve_sample_path(kernel)
+    if file_path is None:
+        return _empty_sample_plan(move, kernel)
+
     steps = []
     descriptions = []
 
@@ -93,7 +156,7 @@ def _compile_sample_texture_layer(move: SemanticMove, kernel: dict) -> CompiledP
 
     steps.append(CompiledStep(
         tool="load_sample_to_simpler",
-        params={"track_index": new_idx, "file_path": _resolve_sample_path(kernel)},
+        params={"track_index": new_idx, "file_path": file_path},
         description="Load textural sample into Simpler",
     ))
     descriptions.append("Load texture sample")
@@ -132,6 +195,10 @@ def _compile_sample_texture_layer(move: SemanticMove, kernel: dict) -> CompiledP
 
 def _compile_sample_vocal_ghost(move: SemanticMove, kernel: dict) -> CompiledPlan:
     """Compile 'sample_vocal_ghost': reverse, pitch, and wash a vocal sample."""
+    file_path = _resolve_sample_path(kernel)
+    if file_path is None:
+        return _empty_sample_plan(move, kernel)
+
     steps = []
     descriptions = []
 
@@ -146,7 +213,7 @@ def _compile_sample_vocal_ghost(move: SemanticMove, kernel: dict) -> CompiledPla
 
     steps.append(CompiledStep(
         tool="load_sample_to_simpler",
-        params={"track_index": new_idx, "file_path": _resolve_sample_path(kernel)},
+        params={"track_index": new_idx, "file_path": file_path},
         description="Load vocal sample into Simpler",
     ))
 
@@ -191,6 +258,10 @@ def _compile_sample_vocal_ghost(move: SemanticMove, kernel: dict) -> CompiledPla
 
 def _compile_sample_break_layer(move: SemanticMove, kernel: dict) -> CompiledPlan:
     """Compile 'sample_break_layer': slice a break and layer over existing drums."""
+    file_path = _resolve_sample_path(kernel)
+    if file_path is None:
+        return _empty_sample_plan(move, kernel)
+
     steps = []
     descriptions = []
     warnings = []
@@ -210,7 +281,7 @@ def _compile_sample_break_layer(move: SemanticMove, kernel: dict) -> CompiledPla
 
     steps.append(CompiledStep(
         tool="load_sample_to_simpler",
-        params={"track_index": new_idx, "file_path": _resolve_sample_path(kernel)},
+        params={"track_index": new_idx, "file_path": file_path},
         description="Load breakbeat into Simpler",
     ))
 
@@ -252,6 +323,10 @@ def _compile_sample_resample_destroy(move: SemanticMove, kernel: dict) -> Compil
     SAFETY: This is a high-risk move — always requires approval.
     Only adjusts device params when a known device is confirmed present.
     """
+    file_path = _resolve_sample_path(kernel)
+    if file_path is None:
+        return _empty_sample_plan(move, kernel)
+
     steps = []
     descriptions = []
     warnings = ["High-risk: destructive processing — consider duplicating track first"]
@@ -267,7 +342,7 @@ def _compile_sample_resample_destroy(move: SemanticMove, kernel: dict) -> Compil
 
     steps.append(CompiledStep(
         tool="load_sample_to_simpler",
-        params={"track_index": new_idx, "file_path": _resolve_sample_path(kernel)},
+        params={"track_index": new_idx, "file_path": file_path},
         description="Load sample for destruction",
     ))
     descriptions.append("Load source")
@@ -312,6 +387,10 @@ def _compile_sample_resample_destroy(move: SemanticMove, kernel: dict) -> Compil
 
 def _compile_sample_one_shot_accent(move: SemanticMove, kernel: dict) -> CompiledPlan:
     """Compile 'sample_one_shot_accent': load a one-shot for rhythmic punctuation."""
+    file_path = _resolve_sample_path(kernel)
+    if file_path is None:
+        return _empty_sample_plan(move, kernel)
+
     steps = []
     descriptions = []
 
@@ -326,7 +405,7 @@ def _compile_sample_one_shot_accent(move: SemanticMove, kernel: dict) -> Compile
 
     steps.append(CompiledStep(
         tool="load_sample_to_simpler",
-        params={"track_index": new_idx, "file_path": _resolve_sample_path(kernel)},
+        params={"track_index": new_idx, "file_path": file_path},
         description="Load one-shot into Simpler",
     ))
 

package/mcp_server/semantic_moves/tools.py

@@ -366,6 +366,10 @@ async def apply_semantic_move(
         }
         if getattr(step, "backend", None):
             d["backend"] = step.backend
+        # v1.20.2 (BUG #3 fix): propagate optional flag so the router
+        # can skip-and-continue on soft failures (e.g., analyzer pre-reads).
+        if getattr(step, "optional", False):
+            d["optional"] = True
         return d
 
     step_dicts = [_step_to_dict(step) for step in plan.steps]

package/mcp_server/tools/devices.py

@@ -349,6 +349,80 @@ def _normalize_batch_entry(entry: dict) -> dict:
     return {"name_or_index": key, "value": entry["value"]}
 
 
+_SNAP_EPSILON = 1e-5
+
+
+def _detect_snapped_params(
+    requested: list[dict], response: dict,
+) -> list[dict]:
+    """Compare requested parameter values against Ableton's returned
+    values; surface any that were silently snapped.
+
+    BUG #4 fix (v1.20.2): quantized-enum params (e.g., Beat Repeat's
+    "Gate" at 0/1/2/... integer enum) silently snap a caller's float
+    request to the nearest step. Pre-fix, the response gave no signal —
+    callers saw success with the snapped value hidden in `value_string`.
+
+    Returns a list of {name, requested, actual, display_value} entries
+    for params whose actual (returned) value differs from the requested
+    value by more than _SNAP_EPSILON. String params are compared
+    exactly. Integer params use int equality.
+
+    Empty list when nothing snapped — callers can check
+    ``result.get("snapped_params") == []`` as a go/no-go signal.
+    """
+    result_params = response.get("parameters") or []
+    if not isinstance(result_params, list):
+        return []
+
+    # Build {key → requested_value} from the original caller input.
+    # Accept any of the same schemas _normalize_batch_entry accepts.
+    by_key: dict = {}
+    for entry in requested:
+        if not isinstance(entry, dict):
+            continue
+        for key_name in ("parameter_name", "name", "parameter_index",
+                         "index", "name_or_index"):
+            if key_name in entry:
+                by_key[entry[key_name]] = entry.get("value")
+                break
+
+    snapped: list[dict] = []
+    for rp in result_params:
+        if not isinstance(rp, dict):
+            continue
+        name = rp.get("name")
+        # Match by name first (most common), fall back to index
+        requested_val = by_key.get(name)
+        if requested_val is None and "index" in rp:
+            requested_val = by_key.get(rp["index"])
+        if requested_val is None:
+            continue
+        actual_val = rp.get("value")
+        if actual_val is None:
+            continue
+
+        # Compare with type-appropriate tolerance. Numeric → epsilon;
+        # other types → strict equality.
+        try:
+            req_f = float(requested_val)
+            act_f = float(actual_val)
+            did_snap = abs(req_f - act_f) > _SNAP_EPSILON
+        except (TypeError, ValueError):
+            did_snap = requested_val != actual_val
+
+        if did_snap:
+            snapped.append({
+                "name": name,
+                "requested": requested_val,
+                "actual": actual_val,
+                "display_value": rp.get("display_value"),
+                "value_string": rp.get("value_string"),
+            })
+
+    return snapped
+
+
 @mcp.tool()
 def batch_set_parameters(
     ctx: Context,
@@ -363,18 +437,30 @@ def batch_set_parameters(
       - {"parameter_name": "Dry/Wet", "value": V} (preferred)
       - {"name_or_index": X, "value": V}          (legacy, still accepted)
 
-    track_index: 0+ for regular tracks, -1/-2/... for return tracks (A/B/...), -1000 for master."""
+    track_index: 0+ for regular tracks, -1/-2/... for return tracks (A/B/...), -1000 for master.
+
+    Response (v1.20.2+): the dict now includes a ``snapped_params`` list
+    when quantized-enum parameters were silently snapped by Ableton
+    (requested 0.3, received 0). Empty list means every requested value
+    round-tripped within 1e-5 tolerance. Callers using this tool to
+    drive deterministic state should inspect ``snapped_params`` before
+    assuming success — see BUG #4 in the v1.20 live-test campaign for
+    the motivating case (Beat Repeat Gate).
+    """
     _validate_track_index(track_index)
     _validate_device_index(device_index)
-    parameters = _ensure_list(parameters)
-    if not parameters:
+    parameters_list = _ensure_list(parameters)
+    if not parameters_list:
         raise ValueError("parameters list cannot be empty")
-    normalized = [_normalize_batch_entry(e) for e in parameters]
-    return _get_ableton(ctx).send_command("batch_set_parameters", {
+    normalized = [_normalize_batch_entry(e) for e in parameters_list]
+    response = _get_ableton(ctx).send_command("batch_set_parameters", {
         "track_index": track_index,
         "device_index": device_index,
         "parameters": normalized,
     })
+    if isinstance(response, dict):
+        response["snapped_params"] = _detect_snapped_params(parameters_list, response)
+    return response
 
 
 @mcp.tool()

package/mcp_server/tools/tracks.py

@@ -134,6 +134,35 @@ def verify_device_alive(
     }
 
 
+def _find_name_collisions(ctx: Context, name: str) -> list[int]:
+    """Return track indices whose name exactly matches `name` (case-sensitive).
+
+    BUG #5 fix (v1.20.2): downstream role-based resolvers like
+    find_tracks_by_role match on track names. If create_midi_track
+    creates a second "Pad" while another "Pad" already exists, mix
+    moves like widen_stereo match BOTH — applying the change twice.
+    This helper enables create_*_track to warn the caller so they can
+    pick a unique name or explicitly accept the collision.
+
+    Best-effort: returns [] when session_info can't be fetched —
+    collision detection must never block creation.
+    """
+    try:
+        info = _get_ableton(ctx).send_command("get_session_info")
+    except Exception:
+        return []
+    if not isinstance(info, dict):
+        return []
+    tracks = info.get("tracks") or []
+    matches: list[int] = []
+    for t in tracks:
+        if isinstance(t, dict) and t.get("name") == name:
+            idx = t.get("index")
+            if isinstance(idx, int):
+                matches.append(idx)
+    return matches
+
+
 @mcp.tool()
 def create_midi_track(
     ctx: Context,
@@ -141,7 +170,18 @@ def create_midi_track(
     name: Optional[str] = None,
     color: Optional[int] = None,
 ) -> dict:
-    """Create a new MIDI track. index=-1 appends at end."""
+    """Create a new MIDI track. index=-1 appends at end.
+
+    Response (v1.20.2+): when `name` is provided, the response carries
+    a ``name_collision`` bool and ``existing_tracks_with_same_name``
+    list[int]. Downstream role-based resolvers (find_tracks_by_role)
+    match duplicate names and apply mix changes twice — check the
+    warning before proceeding with mix moves on the new track's role.
+    """
+    collisions: list[int] = []
+    if name is not None and name.strip():
+        collisions = _find_name_collisions(ctx, name)
+
     params = {"index": index}
     if name is not None:
         if not name.strip():
@@ -150,7 +190,13 @@ def create_midi_track(
     if color is not None:
         _validate_color_index(color)
         params["color_index"] = color
-    return _get_ableton(ctx).send_command("create_midi_track", params)
+    result = _get_ableton(ctx).send_command("create_midi_track", params)
+    if isinstance(result, dict):
+        # Always stamp both fields so callers can check unconditionally
+        # (False + [] when no name provided or no collision).
+        result["name_collision"] = bool(collisions)
+        result["existing_tracks_with_same_name"] = collisions
+    return result
 
 
 @mcp.tool()
@@ -160,7 +206,15 @@ def create_audio_track(
     name: Optional[str] = None,
     color: Optional[int] = None,
 ) -> dict:
-    """Create a new audio track. index=-1 appends at end."""
+    """Create a new audio track. index=-1 appends at end.
+
+    Response (v1.20.2+): ``name_collision`` + ``existing_tracks_with_same_name``
+    same as create_midi_track — see BUG #5 rationale there.
+    """
+    collisions: list[int] = []
+    if name is not None and name.strip():
+        collisions = _find_name_collisions(ctx, name)
+
     params = {"index": index}
     if name is not None:
         if not name.strip():
@@ -169,7 +223,11 @@ def create_audio_track(
     if color is not None:
         _validate_color_index(color)
         params["color_index"] = color
-    return _get_ableton(ctx).send_command("create_audio_track", params)
+    result = _get_ableton(ctx).send_command("create_audio_track", params)
+    if isinstance(result, dict):
+        result["name_collision"] = bool(collisions)
+        result["existing_tracks_with_same_name"] = collisions
+    return result
 
 
 @mcp.tool()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "livepilot",
-  "version": "1.20.0",
+  "version": "1.20.2",
   "mcpName": "io.github.dreamrec/livepilot",
   "description": "Agentic production system for Ableton Live 12 — 429 tools, 53 domains. Device atlas (1305 devices), sample engine (Splice + browser + filesystem), auto-composition, spectral perception, technique memory, creative intelligence (12 engines)",
   "author": "Pilot Studio",

package/remote_script/LivePilot/__init__.py

@@ -5,7 +5,7 @@ Entry point for the ControlSurface. Ableton calls create_instance(c_instance)
 when this script is selected in Preferences > Link, Tempo & MIDI.
 """
 
-__version__ = "1.20.0"
+__version__ = "1.20.2"
 
 from _Framework.ControlSurface import ControlSurface
 from . import router

package/server.json

@@ -6,12 +6,12 @@
     "url": "https://github.com/dreamrec/LivePilot",
     "source": "github"
   },
-  "version": "1.20.0",
+  "version": "1.20.2",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "livepilot",
-      "version": "1.20.0",
+      "version": "1.20.2",
       "transport": {
         "type": "stdio"
       }