pmx-canvas 0.6.1 → 0.6.2

package/src/server/operations/ops/viewport.ts

@@ -135,8 +135,10 @@ const focusOperation = defineOperation<z.infer<typeof focusSchema>, Record<strin
     }
     const focus = canvasState.setAxFocus([nodeId], { source: 'api', recordHistory: false });
     ctx.emit('ax-state-changed', { focus });
-    ctx.emit('canvas-focus-node', { nodeId, noPan });
+    // Apply the server fallback first; the browser then refines it using its
+    // actual canvas region and floating chrome instead of cancelling that fit.
     if (!noPan) ctx.emit('canvas-viewport-update', { viewport: canvasState.viewport });
+    ctx.emit('canvas-focus-node', { nodeId, noPan });
     return { ok: true, focused: nodeId, panned: !noPan, axFocus: focus };
   },
 });
@@ -250,6 +252,9 @@ const viewportSetOperation = defineOperation<z.infer<typeof viewportSetSchema>,
     // sizes to the human's actual window instead of a hardcoded 1440x900
     // (0.4.6 orb feedback #2).
     setClientViewportSize(body.clientWidth, body.clientHeight);
+    // A size-only report is not a camera command. Echoing the stored camera
+    // here would cancel an in-flight browser focus with its stale fallback.
+    if (![body.x, body.y, body.scale].some((value) => typeof value === 'number')) return { ok: true };
     const next = {
       x: typeof body.x === 'number' ? body.x : canvasState.viewport.x,
       y: typeof body.y === 'number' ? body.y : canvasState.viewport.y,

package/src/server/server.ts

@@ -556,22 +556,22 @@ export async function startCanvasAutomationWebView(
       await closeCanvasAutomationWebViewInternal();
     }
 
-    const view = new WebView({
-      width: normalized.width,
-      height: normalized.height,
-      headless: true,
-      backend,
-      dataStore: normalized.dataStoreDir ? { directory: normalized.dataStoreDir } : 'ephemeral',
-    });
-
+    let view: CanvasWebViewLike | null = null;
     try {
+      view = new WebView({
+        width: normalized.width,
+        height: normalized.height,
+        headless: true,
+        backend,
+        dataStore: normalized.dataStoreDir ? { directory: normalized.dataStoreDir } : 'ephemeral',
+      });
       await withCanvasAutomationWebViewTimeout(view.navigate(url), 'starting the workbench automation WebView');
     } catch (error) {
       canvasAutomationWebViewStatus = {
         ...canvasAutomationWebViewStatus,
         lastError: error instanceof Error ? error.message : String(error),
       };
-      await Promise.resolve(view.close()).catch(() => undefined);
+      if (view) await Promise.resolve(view.close()).catch(() => undefined);
       throw error;
     }
 

package/src/server/web-artifacts/scripts/bundle-artifact.sh

@@ -56,14 +56,12 @@ function replay_filtered_stderr() {
 
 function run_with_filtered_stderr() {
   local stderr_file
+  local status
   stderr_file="$(mktemp)"
-  if "$@" 2>"$stderr_file"; then
-    replay_filtered_stderr "$stderr_file"
-    rm -f "$stderr_file"
-    return 0
-  fi
-
-  local status=$?
+  set +e
+  "$@" 2>"$stderr_file"
+  status=$?
+  set -e
   replay_filtered_stderr "$stderr_file"
   rm -f "$stderr_file"
   return "$status"
@@ -109,6 +107,17 @@ function ensure_bundle_dependencies() {
     fi
   done
 
+  # package.json lists the deps, but node_modules may not be in sync (e.g. fresh
+  # CI checkout where node_modules is gitignored). Run a regular install so the
+  # local binaries actually exist before we hand off to Parcel. The build
+  # allowlist is already persisted in package.json's pnpm.onlyBuiltDependencies
+  # by the original `pnpm add` step, so plain `install` is enough here.
+  if [ ! -x "./node_modules/.bin/parcel" ]; then
+    echo "📦 Syncing bundling dependencies into node_modules..."
+    run_pnpm_quiet install
+    return 0
+  fi
+
   echo "✅ Reusing existing bundling dependencies"
 }
 
@@ -150,7 +159,19 @@ rm -rf dist bundle.html
 
 # Build with Parcel
 echo "🔨 Building with Parcel..."
-run_with_filtered_stderr run_local_binary parcel build index.html --dist-dir dist --no-source-maps --log-level error
+parcel_status=0
+run_with_filtered_stderr run_local_binary parcel build index.html --dist-dir dist --no-source-maps --log-level error || parcel_status=$?
+
+# Self-heal a wedged cache: these build projects are REUSED across artifact
+# builds, and a parcel killed mid-write (e.g. the canvas server torn down
+# during a build) leaves a corrupted .parcel-cache that makes every later
+# build die before emitting anything — permanently, until the cache goes.
+# Retry exactly once from a cold cache, then preserve a genuine build failure.
+if [ "$parcel_status" -ne 0 ] || [ ! -s "dist/index.html" ]; then
+  echo "🧹 Parcel build failed or produced no output — clearing .parcel-cache and retrying cold..."
+  rm -rf .parcel-cache dist
+  run_with_filtered_stderr run_local_binary parcel build index.html --dist-dir dist --no-source-maps --log-level error
+fi
 
 if [ ! -s "dist/index.html" ]; then
   echo "❌ Error: Parcel did not produce dist/index.html" >&2