pmx-canvas 0.3.0 → 0.3.2

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

@@ -19,10 +19,7 @@
  */
 import { z } from 'zod';
 import { canvasState } from '../../canvas-state.js';
-import {
-  setCanvasContextPins,
-  syncCanvasRuntimeBackends,
-} from '../../canvas-operations.js';
+import { setCanvasContextPins, syncCanvasRuntimeBackends } from '../../canvas-operations.js';
 import { buildCanvasSummary } from '../../canvas-serialization.js';
 import { mutationHistory } from '../../mutation-history.js';
 import { buildSpatialContext, searchNodes } from '../../spatial-analysis.js';
@@ -36,7 +33,10 @@ const MAX_PINS = 20;
 
 const pinShape = {
   nodeIds: z.unknown().optional().describe('Array of node IDs to pin'),
-  mode: z.unknown().optional().describe('set: replace all pins, add: add to existing pins, remove: unpin these nodes (default: set)'),
+  mode: z
+    .unknown()
+    .optional()
+    .describe('set: replace all pins, add: add to existing pins, remove: unpin these nodes (default: set)'),
 };
 
 const pinSchema = z.looseObject(pinShape);
@@ -52,20 +52,25 @@ const pinOperation = defineOperation<z.infer<typeof pinSchema>, Record<string, u
   },
   mcp: {
     toolName: 'canvas_pin_nodes',
-    description: 'Pin nodes to include them in the agent context. Pinned nodes appear in the canvas://pinned-context resource. The human can also pin nodes by clicking in the browser.',
+    description:
+      'Pin nodes to include them in the agent context. Pinned nodes appear in the canvas://pinned-context resource. The human can also pin nodes by clicking in the browser.',
     extraShape: {
       nodeIds: z.array(z.string()).describe('Array of node IDs to pin'),
-      mode: z.enum(['set', 'add', 'remove']).optional()
+      mode: z
+        .enum(['set', 'add', 'remove'])
+        .optional()
         .describe('set: replace all pins, add: add to existing pins, remove: unpin these nodes (default: set)'),
     },
     // The wire body is { ok, count } (legacy HTTP shape); the tool reports the
     // resulting pin list, so re-read it from the host. (Legacy RemoteCanvasAccess
     // computed the list client-side; the server state is authoritative now.)
     formatResult: async (_result, _input, host) => ({
-      content: [{
-        type: 'text' as const,
-        text: JSON.stringify({ ok: true, pinnedNodeIds: await host.getPinnedNodeIds() }),
-      }],
+      content: [
+        {
+          type: 'text' as const,
+          text: JSON.stringify({ ok: true, pinnedNodeIds: await host.getPinnedNodeIds() }),
+        },
+      ],
     }),
   },
   handler: (input, ctx) => {
@@ -103,7 +108,8 @@ const searchOperation = defineOperation<z.infer<typeof searchSchema>, Record<str
   },
   mcp: {
     toolName: 'canvas_search',
-    description: 'Search for nodes by title or content keywords. Returns matching nodes ranked by relevance with snippets. Much faster than reading the full layout when you need to find specific nodes.',
+    description:
+      'Search for nodes by title or content keywords. Returns matching nodes ranked by relevance with snippets. Much faster than reading the full layout when you need to find specific nodes.',
     extraShape: {
       query: z.string().describe('Search query — matches against node titles, content, and file paths'),
       limit: z.number().optional().describe('Max results to return (default: 10)'),
@@ -119,14 +125,20 @@ const searchOperation = defineOperation<z.infer<typeof searchSchema>, Record<str
       const results = Array.isArray(body.results) ? body.results : [];
       const limit = typeof input.limit === 'number' ? input.limit : 10;
       return {
-        content: [{
-          type: 'text' as const,
-          text: JSON.stringify({
-            query: input.query,
-            resultCount: results.length,
-            results: results.slice(0, limit),
-          }, null, 2),
-        }],
+        content: [
+          {
+            type: 'text' as const,
+            text: JSON.stringify(
+              {
+                query: input.query,
+                resultCount: results.length,
+                results: results.slice(0, limit),
+              },
+              null,
+              2,
+            ),
+          },
+        ],
       };
     },
   },
@@ -136,9 +148,12 @@ const searchOperation = defineOperation<z.infer<typeof searchSchema>, Record<str
       return { results: [], query: q };
     }
     const rawLimit = input.limit;
-    const limit = typeof rawLimit === 'number'
-      ? rawLimit
-      : typeof rawLimit === 'string' && rawLimit.trim() !== '' ? Number(rawLimit) : Number.NaN;
+    const limit =
+      typeof rawLimit === 'number'
+        ? rawLimit
+        : typeof rawLimit === 'string' && rawLimit.trim() !== ''
+          ? Number(rawLimit)
+          : Number.NaN;
     const results = searchNodes(canvasState.getLayout().nodes, q);
     return {
       results: Number.isFinite(limit) && limit > 0 ? results.slice(0, Math.floor(limit)) : results,
@@ -178,10 +193,12 @@ async function formatUndoRedoResult(result: unknown, host: OperationMcpToolHost)
   const historyBody = isRecord(history) ? history : {};
   const body = isRecord(result) ? result : {};
   return {
-    content: [{
-      type: 'text' as const,
-      text: JSON.stringify({ ...body, canUndo: historyBody.canUndo, canRedo: historyBody.canRedo }),
-    }],
+    content: [
+      {
+        type: 'text' as const,
+        text: JSON.stringify({ ...body, canUndo: historyBody.canUndo, canRedo: historyBody.canRedo }),
+      },
+    ],
   };
 }
 
@@ -200,7 +217,8 @@ const undoOperation = defineOperation<z.infer<typeof undoRedoSchema>, Record<str
   },
   mcp: {
     toolName: 'canvas_undo',
-    description: 'Undo the last canvas mutation. Returns a description of what was undone. Use this to backtrack when an approach is wrong — explore without fear.',
+    description:
+      'Undo the last canvas mutation. Returns a description of what was undone. Use this to backtrack when an approach is wrong — explore without fear.',
     formatResult: (result, _input, host) => formatUndoRedoResult(result, host),
   },
   handler: async (_input, ctx) => {

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

@@ -69,7 +69,8 @@ const snapshotListOperation = defineOperation<z.infer<typeof snapshotListSchema>
   },
   mcp: {
     toolName: 'canvas_list_snapshots',
-    description: 'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). List saved canvas snapshots with IDs, names, timestamps, and node/edge counts. Defaults to the 20 newest snapshots; pass all=true to return every snapshot.',
+    description:
+      'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). List saved canvas snapshots with IDs, names, timestamps, and node/edge counts. Defaults to the 20 newest snapshots; pass all=true to return every snapshot.',
     extraShape: {
       limit: z.number().optional().describe('Maximum snapshots to return (default: 20)'),
       query: z.string().optional().describe('Optional case-insensitive ID/name filter'),
@@ -84,9 +85,7 @@ const snapshotListOperation = defineOperation<z.infer<typeof snapshotListSchema>
   handler: (input) => {
     const body: Record<string, unknown> = input;
     // Legacy precedence: ?q= wins over ?query= even when empty.
-    const query = typeof body.q === 'string'
-      ? body.q
-      : typeof body.query === 'string' ? body.query : undefined;
+    const query = typeof body.q === 'string' ? body.q : typeof body.query === 'string' ? body.query : undefined;
     const limit = parsePositiveIntegerParam(body.limit);
     return listCanvasSnapshots({
       ...(limit !== undefined ? { limit } : {}),
@@ -117,7 +116,8 @@ const snapshotSaveOperation = defineOperation<z.infer<typeof snapshotSaveSchema>
   },
   mcp: {
     toolName: 'canvas_snapshot',
-    description: 'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Save the current canvas state as a named snapshot. Snapshots persist to disk and can be restored later.',
+    description:
+      'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Save the current canvas state as a named snapshot. Snapshots persist to disk and can be restored later.',
     extraShape: {
       name: z.string().describe('Name for this snapshot (e.g., "before refactor", "investigation v2")'),
     },
@@ -154,7 +154,8 @@ const snapshotGcOperation = defineOperation<z.infer<typeof snapshotGcSchema>, Re
   },
   mcp: {
     toolName: 'canvas_gc_snapshots',
-    description: 'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Delete old saved canvas snapshots, keeping the newest N snapshots. Use dryRun=true to preview deletions.',
+    description:
+      'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Delete old saved canvas snapshots, keeping the newest N snapshots. Use dryRun=true to preview deletions.',
     extraShape: {
       keep: z.number().optional().describe('Number of newest snapshots to keep (default: 20)'),
       dryRun: z.boolean().optional().describe('Preview deletions without removing snapshot files'),
@@ -168,11 +169,8 @@ const snapshotGcOperation = defineOperation<z.infer<typeof snapshotGcSchema>, Re
     const keepValue = body.keep;
     // Legacy coercion replicated as-is, including the Number('junk') → NaN
     // pass-through (gcCanvasSnapshots falls back to its own default).
-    const keep = typeof keepValue === 'number'
-      ? keepValue
-      : typeof keepValue === 'string'
-        ? Number(keepValue)
-        : undefined;
+    const keep =
+      typeof keepValue === 'number' ? keepValue : typeof keepValue === 'string' ? Number(keepValue) : undefined;
     const dryRun = body.dryRun === true || body['dry-run'] === true;
     return gcCanvasSnapshots({
       ...(keep !== undefined ? { keep } : {}),
@@ -207,7 +205,8 @@ const snapshotDiffOperation = defineOperation<z.infer<typeof snapshotDiffSchema>
   },
   mcp: {
     toolName: 'canvas_diff',
-    description: 'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Compare the current canvas state against a saved snapshot. Shows added/removed/modified nodes and edges. Pass either a snapshot name or ID.',
+    description:
+      'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Compare the current canvas state against a saved snapshot. Shows added/removed/modified nodes and edges. Pass either a snapshot name or ID.',
     extraShape: {
       snapshot: z.string().describe('Snapshot name or ID to compare against'),
     },
@@ -242,9 +241,7 @@ const snapshotDiffQueryOperation = defineOperation<z.infer<typeof snapshotDiffQu
   },
   handler: (input) => {
     // Legacy precedence: ?name= wins over ?id= even when empty.
-    const name = typeof input.name === 'string'
-      ? input.name
-      : typeof input.id === 'string' ? input.id : '';
+    const name = typeof input.name === 'string' ? input.name : typeof input.id === 'string' ? input.id : '';
     if (!name.trim()) throw new OperationError('Missing snapshot name or id.');
     return diffSnapshotCore(name);
   },
@@ -273,7 +270,11 @@ function buildSnapshotRestoreSummary(layout: unknown): Record<string, unknown> {
 }
 
 const snapshotRestoreShape = {
-  id: z.string().optional().catch(undefined).describe('Snapshot ID or name to restore (from canvas_snapshot or snapshot list)'),
+  id: z
+    .string()
+    .optional()
+    .catch(undefined)
+    .describe('Snapshot ID or name to restore (from canvas_snapshot or snapshot list)'),
 };
 
 const snapshotRestoreSchema = z.looseObject(snapshotRestoreShape);
@@ -289,21 +290,28 @@ const snapshotRestoreOperation = defineOperation<z.infer<typeof snapshotRestoreS
   },
   mcp: {
     toolName: 'canvas_restore',
-    description: 'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Restore the canvas to a previously saved snapshot. Use canvas_snapshot to save first. Pass either the snapshot ID or name to restore.',
+    description:
+      'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Restore the canvas to a previously saved snapshot. Use canvas_snapshot to save first. Pass either the snapshot ID or name to restore.',
     extraShape: {
       id: z.string().describe('Snapshot ID or name to restore (from canvas_snapshot or snapshot list)'),
     },
     formatResult: async (_result, input, host) => {
       const layout = await host.invoker().invoke('layout.get', { includeBlobs: true });
       return {
-        content: [{
-          type: 'text' as const,
-          text: JSON.stringify({
-            ok: true,
-            restored: input.id,
-            summary: buildSnapshotRestoreSummary(layout),
-          }, null, 2),
-        }],
+        content: [
+          {
+            type: 'text' as const,
+            text: JSON.stringify(
+              {
+                ok: true,
+                restored: input.id,
+                summary: buildSnapshotRestoreSummary(layout),
+              },
+              null,
+              2,
+            ),
+          },
+        ],
       };
     },
   },
@@ -338,7 +346,8 @@ const snapshotDeleteOperation = defineOperation<z.infer<typeof snapshotDeleteSch
   },
   mcp: {
     toolName: 'canvas_delete_snapshot',
-    description: 'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Delete a saved snapshot by ID.',
+    description:
+      'Deprecated: folds into the canvas_snapshot composite in v0.4 (actions save|list|restore|delete|gc|diff). Delete a saved snapshot by ID.',
     extraShape: {
       id: z.string().describe('Snapshot ID to delete'),
     },

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

@@ -26,7 +26,8 @@ const validateGetOperation = defineOperation<z.infer<typeof validateGetSchema>,
   },
   mcp: {
     toolName: 'canvas_validate',
-    description: 'Validate the current canvas layout. Distinguishes true node collisions from expected group-child containment and reports missing edge endpoints.',
+    description:
+      'Validate the current canvas layout. Distinguishes true node collisions from expected group-child containment and reports missing edge endpoints.',
     formatResult: (result) => ({
       content: [{ type: 'text' as const, text: JSON.stringify(result, null, 2) }],
     }),

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

@@ -14,11 +14,7 @@
  */
 import { z } from 'zod';
 import { canvasState } from '../../canvas-state.js';
-import {
-  arrangeCanvasNodes,
-  clearCanvas,
-  fitCanvasView,
-} from '../../canvas-operations.js';
+import { arrangeCanvasNodes, clearCanvas, fitCanvasView } from '../../canvas-operations.js';
 import { validateCanvasLayout } from '../../canvas-validation.js';
 import { defineOperation, OperationError, type Operation } from '../types.js';
 import { closeNodeAppSession, isRecord } from './nodes.js';
@@ -42,17 +38,20 @@ const arrangeOperation = defineOperation<z.infer<typeof arrangeSchema>, Record<s
   },
   mcp: {
     toolName: 'canvas_arrange',
-    description: 'Auto-arrange all nodes on the canvas. Layouts: grid (default), column (vertical stack), flow (horizontal row).',
+    description:
+      'Auto-arrange all nodes on the canvas. Layouts: grid (default), column (vertical stack), flow (horizontal row).',
     extraShape: {
       layout: z.enum(['grid', 'column', 'flow']).optional().describe('Arrangement layout (default: grid)'),
     },
     // Legacy tool reported { ok: true, layout } regardless of the arrange
     // result (it ignored the arranged count and validation outcome).
     formatResult: (result, input) => ({
-      content: [{
-        type: 'text' as const,
-        text: JSON.stringify({ ok: true, layout: typeof input.layout === 'string' ? input.layout : 'grid' }),
-      }],
+      content: [
+        {
+          type: 'text' as const,
+          text: JSON.stringify({ ok: true, layout: typeof input.layout === 'string' ? input.layout : 'grid' }),
+        },
+      ],
     }),
   },
   handler: (input) => {
@@ -93,7 +92,8 @@ const focusOperation = defineOperation<z.infer<typeof focusSchema>, Record<strin
   },
   mcp: {
     toolName: 'canvas_focus_node',
-    description: 'Bring a node into focus. By default the viewport pans so the node is centered. Pass noPan=true to raise/select the node without moving the human\'s camera (useful when reacting to background events without disrupting the human\'s current view).',
+    description:
+      "Bring a node into focus. By default the viewport pans so the node is centered. Pass noPan=true to raise/select the node without moving the human's camera (useful when reacting to background events without disrupting the human's current view).",
     extraShape: {
       id: z.string().describe('Node ID to focus on'),
       noPan: z
@@ -104,10 +104,12 @@ const focusOperation = defineOperation<z.infer<typeof focusSchema>, Record<strin
     formatResult: (result) => {
       const body = isRecord(result) ? result : {};
       return {
-        content: [{
-          type: 'text' as const,
-          text: JSON.stringify({ ok: true, focused: body.focused, panned: body.panned }),
-        }],
+        content: [
+          {
+            type: 'text' as const,
+            text: JSON.stringify({ ok: true, focused: body.focused, panned: body.panned }),
+          },
+        ],
       };
     },
   },
@@ -155,7 +157,8 @@ const fitOperation = defineOperation<z.infer<typeof fitSchema>, Record<string, u
   },
   mcp: {
     toolName: 'canvas_fit_view',
-    description: 'Fit the canvas viewport to all nodes or a selected subset. Useful before screenshots and whole-board review.',
+    description:
+      'Fit the canvas viewport to all nodes or a selected subset. Useful before screenshots and whole-board review.',
     extraShape: {
       nodeIds: z.array(z.string()).optional().describe('Optional node IDs to fit instead of the whole canvas'),
     },
@@ -211,9 +214,50 @@ const clearOperation = defineOperation<z.infer<typeof clearSchema>, Record<strin
   },
 });
 
+// ── viewport.set (plan-009 C1 slice 3) ────────────────────────
+
+const viewportSetShape = {
+  x: z.unknown().optional().describe('Viewport x offset'),
+  y: z.unknown().optional().describe('Viewport y offset'),
+  scale: z.unknown().optional().describe('Viewport zoom scale'),
+  recordHistory: z.unknown().optional().describe('Pass false to skip the undo-history entry'),
+};
+
+const viewportSetSchema = z.looseObject(viewportSetShape);
+
+const viewportSetOperation = defineOperation<z.infer<typeof viewportSetSchema>, Record<string, unknown>>({
+  name: 'viewport.set',
+  mutates: false,
+  input: viewportSetSchema,
+  inputShape: viewportSetShape,
+  http: {
+    method: 'POST',
+    path: '/api/canvas/viewport',
+  },
+  // Legacy wire: emits canvas-viewport-update (never a layout update).
+  handler: (input, ctx) => {
+    const body: Record<string, unknown> = input;
+    const next = {
+      x: typeof body.x === 'number' ? body.x : canvasState.viewport.x,
+      y: typeof body.y === 'number' ? body.y : canvasState.viewport.y,
+      scale: typeof body.scale === 'number' ? body.scale : canvasState.viewport.scale,
+    };
+    if (body.recordHistory === false) {
+      canvasState.withSuppressedRecording(() => {
+        canvasState.setViewport(next);
+      });
+    } else {
+      canvasState.setViewport(next);
+    }
+    ctx.emit('canvas-viewport-update', { viewport: canvasState.viewport });
+    return { ok: true };
+  },
+});
+
 export const viewportOperations: Operation[] = [
   arrangeOperation,
   focusOperation,
   fitOperation,
   clearOperation,
+  viewportSetOperation,
 ];

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

@@ -72,7 +72,11 @@ async function runWebviewTask<T>(task: () => Promise<T> | T): Promise<T> {
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     let webview: WebviewStatus | undefined;
-    try { webview = getWebviewRunner().status(); } catch { /* runner not wired */ }
+    try {
+      webview = getWebviewRunner().status();
+    } catch {
+      /* runner not wired */
+    }
     throw new OperationError(message, 400, webview ? { webview } : undefined);
   }
 }
@@ -164,7 +168,9 @@ const startOperation = defineOperation<z.infer<typeof startSchema>, WebviewStart
     description:
       'Start or replace the headless Bun.WebView automation session for the current PMX Canvas workbench. Use this before screenshot, evaluate, or resize when no automation session is active.',
     extraShape: {
-      backend: z.enum(['chrome', 'webkit']).optional()
+      backend: z
+        .enum(['chrome', 'webkit'])
+        .optional()
         .describe('Automation backend. Default: webkit on macOS, chrome elsewhere.'),
       width: z.number().optional().describe('Viewport width in pixels (default: 1280)'),
       height: z.number().optional().describe('Viewport height in pixels (default: 800)'),
@@ -184,14 +190,20 @@ const startOperation = defineOperation<z.infer<typeof startSchema>, WebviewStart
       const body = result as { ok?: boolean; webview?: WebviewStatus; error?: string };
       if (body.ok && body.webview) return statusText(body.webview);
       return {
-        content: [{
-          type: 'text' as const,
-          text: JSON.stringify({
-            ok: false,
-            error: body.error ?? 'WebView start failed.',
-            ...(body.webview ? { webview: body.webview } : {}),
-          }, null, 2),
-        }],
+        content: [
+          {
+            type: 'text' as const,
+            text: JSON.stringify(
+              {
+                ok: false,
+                error: body.error ?? 'WebView start failed.',
+                ...(body.webview ? { webview: body.webview } : {}),
+              },
+              null,
+              2,
+            ),
+          },
+        ],
         isError: true,
       };
     },
@@ -228,10 +240,12 @@ const stopOperation = defineOperation<z.infer<typeof stopSchema>, { stopped: boo
     formatResult: (result) => {
       const body = result as { ok: boolean; stopped: boolean; webview: WebviewStatus };
       return {
-        content: [{
-          type: 'text' as const,
-          text: JSON.stringify({ ok: true, stopped: body.stopped, webview: body.webview }, null, 2),
-        }],
+        content: [
+          {
+            type: 'text' as const,
+            text: JSON.stringify({ ok: true, stopped: body.stopped, webview: body.webview }, null, 2),
+          },
+        ],
       };
     },
   },
@@ -262,7 +276,8 @@ const resizeOperation = defineOperation<z.infer<typeof resizeSchema>, WebviewSta
   },
   mcp: {
     toolName: 'canvas_resize',
-    description: 'Resize the active Bun.WebView automation viewport. Requires an active automation session started via canvas_webview_start.',
+    description:
+      'Resize the active Bun.WebView automation viewport. Requires an active automation session started via canvas_webview_start.',
     extraShape: {
       width: z.number().describe('Viewport width in pixels'),
       height: z.number().describe('Viewport height in pixels'),
@@ -287,7 +302,12 @@ const resizeOperation = defineOperation<z.infer<typeof resizeSchema>, WebviewSta
 
 const evaluateShape = {
   expression: z.unknown().optional().describe('JavaScript expression to evaluate in the page context'),
-  script: z.unknown().optional().describe('Multi-statement JavaScript body. The MCP server wraps it in an async IIFE and evaluates the resolved return value.'),
+  script: z
+    .unknown()
+    .optional()
+    .describe(
+      'Multi-statement JavaScript body. The MCP server wraps it in an async IIFE and evaluates the resolved return value.',
+    ),
 };
 const evaluateSchema = z.looseObject(evaluateShape);
 
@@ -302,10 +322,16 @@ const evaluateOperation = defineOperation<z.infer<typeof evaluateSchema>, { valu
   },
   mcp: {
     toolName: 'canvas_evaluate',
-    description: 'Evaluate JavaScript in the active Bun.WebView automation session for the workbench page. Use this to inspect rendered browser state. Requires an active automation session started via canvas_webview_start.',
+    description:
+      'Evaluate JavaScript in the active Bun.WebView automation session for the workbench page. Use this to inspect rendered browser state. Requires an active automation session started via canvas_webview_start.',
     extraShape: {
       expression: z.string().optional().describe('JavaScript expression to evaluate in the page context'),
-      script: z.string().optional().describe('Multi-statement JavaScript body. The MCP server wraps it in an async IIFE and evaluates the resolved return value.'),
+      script: z
+        .string()
+        .optional()
+        .describe(
+          'Multi-statement JavaScript body. The MCP server wraps it in an async IIFE and evaluates the resolved return value.',
+        ),
     },
     // Legacy canvas_evaluate validation: exactly one of expression/script (its
     // own message). Validate here so the MCP tool throws the legacy message

package/src/server/operations/registry.ts

@@ -95,9 +95,8 @@ function linkedIntentId(rawInput: unknown): string | undefined {
 
 function allowedIntentKinds(name: string, rawInput: unknown): readonly PmxAxIntentKind[] | undefined {
   if (name === 'jsonrender.stream') {
-    const input = rawInput && typeof rawInput === 'object' && !Array.isArray(rawInput)
-      ? rawInput as Record<string, unknown>
-      : {};
+    const input =
+      rawInput && typeof rawInput === 'object' && !Array.isArray(rawInput) ? (rawInput as Record<string, unknown>) : {};
     return typeof input.nodeId === 'string' && input.nodeId.length > 0 ? ['edit'] : ['create'];
   }
   return INTENT_KINDS_BY_OPERATION[name];

package/src/server/operations/types.ts

@@ -107,7 +107,9 @@ export interface Operation {
  * Any `z.looseObject(...)` satisfies this.
  */
 export interface OperationInputSchema<I> {
-  safeParse(value: unknown):
+  safeParse(
+    value: unknown,
+  ):
     | { success: true; data: I }
     | { success: false; error: { issues: Array<{ path: PropertyKey[]; message: string }> } };
 }
@@ -126,9 +128,7 @@ export interface OperationDefinition<I extends Record<string, unknown>, O> {
   serialize?: (output: O) => unknown;
 }
 
-export function defineOperation<I extends Record<string, unknown>, O>(
-  def: OperationDefinition<I, O>,
-): Operation {
+export function defineOperation<I extends Record<string, unknown>, O>(def: OperationDefinition<I, O>): Operation {
   return {
     name: def.name,
     mutates: def.mutates,
@@ -139,7 +139,9 @@ export function defineOperation<I extends Record<string, unknown>, O>(
       const parsed = def.input.safeParse(rawInput ?? {});
       if (!parsed.success) {
         const detail = parsed.error.issues
-          .map((issue) => (issue.path.length > 0 ? `${issue.path.map(String).join('.')}: ${issue.message}` : issue.message))
+          .map((issue) =>
+            issue.path.length > 0 ? `${issue.path.map(String).join('.')}: ${issue.message}` : issue.message,
+          )
           .join('; ');
         throw new OperationError(`Invalid input for ${def.name}: ${detail}`, 400);
       }

package/src/server/operations/webview-runner.ts

@@ -69,9 +69,7 @@ export function setWebviewRunner(runner: WebviewRunner | null): void {
 
 export function getWebviewRunner(): WebviewRunner {
   if (!injectedRunner) {
-    throw new Error(
-      'Webview runner is not injected. server.ts must call setWebviewRunner(...) at module load.',
-    );
+    throw new Error('Webview runner is not injected. server.ts must call setWebviewRunner(...) at module load.');
   }
   return injectedRunner;
 }

package/src/server/placement.ts

@@ -1,9 +1,4 @@
-import {
-  findBlocker,
-  findOpenCanvasPosition,
-  overlapsAny,
-  type CanvasPlacementRect,
-} from '../shared/placement.js';
+import { findBlocker, findOpenCanvasPosition, overlapsAny, type CanvasPlacementRect } from '../shared/placement.js';
 
 export { findOpenCanvasPosition, type CanvasPlacementRect } from '../shared/placement.js';
 
@@ -30,7 +25,10 @@ export function computeGroupBounds(
 ): { x: number; y: number; width: number; height: number } | null {
   if (children.length === 0) return null;
 
-  let minX = Infinity, minY = Infinity, maxX = -Infinity, maxY = -Infinity;
+  let minX = Infinity,
+    minY = Infinity,
+    maxX = -Infinity,
+    maxY = -Infinity;
   for (const child of children) {
     minX = Math.min(minX, child.position.x);
     minY = Math.min(minY, child.position.y);
@@ -56,17 +54,12 @@ export function computePackedGroupLayout<T extends CanvasPlacementRect & { id: s
   const positions = new Map<string, { x: number; y: number }>();
   if (children.length === 0) return { positions, bounds: null };
 
-  const sorted = [...children].sort(
-    (a, b) => a.position.y - b.position.y || a.position.x - b.position.x,
-  );
+  const sorted = [...children].sort((a, b) => a.position.y - b.position.y || a.position.x - b.position.x);
   const totalArea = sorted.reduce((sum, child) => sum + child.size.width * child.size.height, 0);
   const widestChild = sorted.reduce((max, child) => Math.max(max, child.size.width), 0);
   const targetRowWidth = Math.max(
     widestChild,
-    Math.min(
-      GROUP_LAYOUT_MAX_ROW_WIDTH,
-      Math.max(GROUP_LAYOUT_MIN_ROW_WIDTH, Math.ceil(Math.sqrt(totalArea))),
-    ),
+    Math.min(GROUP_LAYOUT_MAX_ROW_WIDTH, Math.max(GROUP_LAYOUT_MIN_ROW_WIDTH, Math.ceil(Math.sqrt(totalArea)))),
   );
 
   const startX = Math.min(...sorted.map((child) => child.position.x));
@@ -76,10 +69,7 @@ export function computePackedGroupLayout<T extends CanvasPlacementRect & { id: s
   let rowHeight = 0;
 
   for (const child of sorted) {
-    if (
-      cursorX > startX &&
-      cursorX + child.size.width > startX + targetRowWidth
-    ) {
+    if (cursorX > startX && cursorX + child.size.width > startX + targetRowWidth) {
       cursorX = startX;
       cursorY += rowHeight + GROUP_LAYOUT_GAP_Y;
       rowHeight = 0;