@vcad/mcp 0.10.0-main.115 → 0.10.0-main.116

package/README.md

@@ -6,4 +6,4 @@ vcad's MCP server as a self-contained bundle (server + BRep kernel WASM).
 { "mcpServers": { "vcad": { "command": "npx", "args": ["-y", "@vcad/mcp"] } } }
 ```
 
-Built from 8ad12f403ad86c964867f288c8ef53b7790be8c8 at 2026-10-20T23:56:43.258Z. Source: https://github.com/ecto/vcad
+Built from 72d19cb9100f1a027b8c00459e6f0994d641f28e at 2026-10-21T00:31:46.946Z. Source: https://github.com/ecto/vcad

package/index.mjs

@@ -40941,24 +40941,6 @@ var init_registry2 = __esm({
       setWasm(wasm3) {
         this.wasm = wasm3;
       }
-      /**
-       * Plan a CRUD tool call via the Rust executor. Returns `null` if the
-       * wasm binding isn't present (old kernel build, test harness, etc.)
-       * so callers can fall back to the TS `executeCrud` path.
-       *
-       * `docJson` is a `JSON.stringify(document)` of the current store
-       * snapshot — the planner needs it for id/existence validation.
-       */
-      planCrud(tool, args, docJson) {
-        if (!this.wasm?.plan_chat_tool)
-          return null;
-        try {
-          const raw = this.wasm.plan_chat_tool(tool, JSON.stringify(args), docJson);
-          return JSON.parse(raw);
-        } catch {
-          return null;
-        }
-      }
       /** Get all loaded schema entries. */
       getSchemas() {
         return this.schemas;
@@ -41539,20 +41521,6 @@ function partIdMap(doc) {
   const copy = { roots: doc.roots, partIds: { ...doc.partIds ?? {} } };
   return ensurePartIds(copy).partIds;
 }
-function movePartId(doc, oldRoot, newRoot) {
-  const map = idMap(doc);
-  const from = String(oldRoot);
-  const to = String(newRoot);
-  if (from === to)
-    return;
-  const id = map[from] ?? mint(map, newRoot);
-  delete map[from];
-  map[to] = id;
-}
-function forgetPartId(doc, root) {
-  if (doc.partIds)
-    delete doc.partIds[String(root)];
-}
 function rootForPartId(doc, id) {
   const key3 = String(id).trim();
   if (!key3)
@@ -41577,9 +41545,6 @@ function rootForPartId(doc, id) {
     return done(byName, "name");
   return void 0;
 }
-function rootIndexForPartId(doc, id) {
-  return rootForPartId(doc, id)?.index ?? -1;
-}
 function partIdDeprecation(input, resolved) {
   if (!resolved || resolved.matchedBy === "stable")
     return void 0;
@@ -41595,150 +41560,6 @@ var init_part_ids = __esm({
 });
 
 // ../core/dist/commands/document-mutations.js
-function nextNodeId(doc) {
-  let max = 0;
-  for (const k of Object.keys(doc.nodes)) {
-    const n = Number(k);
-    if (Number.isFinite(n) && n > max)
-      max = n;
-  }
-  return max + 1;
-}
-function findRootIndex(doc, partId) {
-  return rootIndexForPartId(doc, partId);
-}
-function collectChildNodeIds(doc, rootId) {
-  const seen = /* @__PURE__ */ new Set();
-  const stack = [rootId];
-  while (stack.length > 0) {
-    const id = stack.pop();
-    if (seen.has(id))
-      continue;
-    seen.add(id);
-    const node = doc.nodes[String(id)];
-    if (!node)
-      continue;
-    for (const child of opChildren(node.op))
-      stack.push(child);
-  }
-  return seen;
-}
-function opChildren(op) {
-  switch (op.type) {
-    case "Union":
-    case "Difference":
-    case "Intersection":
-      return [op.left, op.right];
-    case "Translate":
-    case "Rotate":
-    case "Scale":
-    case "LinearPattern":
-    case "CircularPattern":
-    case "Shell":
-    case "Fillet":
-    case "Chamfer":
-    case "EdgeBlend":
-      return [op.child];
-    case "Extrude":
-    case "Revolve":
-      return [op.sketch];
-    case "Sweep":
-      return [op.sketch];
-    case "Loft":
-      return op.sketches ?? [];
-    default:
-      return [];
-  }
-}
-function applyToolOutcome(doc, outcome) {
-  switch (outcome.kind) {
-    case "add_feature": {
-      const id = nextNodeId(doc);
-      const node = {
-        id,
-        name: outcome.name ?? null,
-        op: outcome.op
-      };
-      doc.nodes[String(id)] = node;
-      const children = new Set(opChildren(node.op));
-      const consumedIdxs = [];
-      doc.roots.forEach((r, i) => {
-        if (children.has(r.root))
-          consumedIdxs.push(i);
-      });
-      if (consumedIdxs.length > 0) {
-        const first = doc.roots[consumedIdxs[0]];
-        const firstPartId = String(first.root);
-        doc.roots[consumedIdxs[0]] = { ...first, root: id };
-        movePartId(doc, first.root, id);
-        if (doc.part_materials[firstPartId] !== void 0) {
-          doc.part_materials[String(id)] = doc.part_materials[firstPartId];
-          delete doc.part_materials[firstPartId];
-        }
-        for (let k = consumedIdxs.length - 1; k >= 1; k--) {
-          const idx = consumedIdxs[k];
-          delete doc.part_materials[String(doc.roots[idx].root)];
-          forgetPartId(doc, doc.roots[idx].root);
-          doc.roots.splice(idx, 1);
-        }
-      } else {
-        const entry = { root: id, material: "default" };
-        doc.roots.push(entry);
-      }
-      return { partId: partIdForNode(doc, id), nodeId: String(id) };
-    }
-    case "remove_part": {
-      const idx = findRootIndex(doc, outcome.part_id);
-      if (idx < 0) {
-        throw new Error(`remove_part: no root entry with id "${outcome.part_id}" in document`);
-      }
-      const entry = doc.roots[idx];
-      const removedId = partIdForRoot(doc, entry);
-      doc.roots.splice(idx, 1);
-      forgetPartId(doc, entry.root);
-      const reachableFromOthers = /* @__PURE__ */ new Set();
-      for (const r of doc.roots) {
-        for (const id of collectChildNodeIds(doc, r.root)) {
-          reachableFromOthers.add(id);
-        }
-      }
-      const exclusive = collectChildNodeIds(doc, entry.root);
-      for (const id of exclusive) {
-        if (!reachableFromOthers.has(id))
-          delete doc.nodes[String(id)];
-      }
-      delete doc.part_materials[String(entry.root)];
-      return { partId: removedId };
-    }
-    case "set_part_material": {
-      const idx = findRootIndex(doc, outcome.part_id);
-      if (idx < 0) {
-        throw new Error(`set_part_material: no root entry with id "${outcome.part_id}"`);
-      }
-      const prev = doc.roots[idx];
-      doc.roots[idx] = { root: prev.root, material: outcome.material, visible: prev.visible };
-      doc.part_materials[String(prev.root)] = outcome.material;
-      return { partId: partIdForRoot(doc, prev) };
-    }
-    case "update_params": {
-      const node = doc.nodes[outcome.node_id];
-      if (!node) {
-        throw new Error(`update_params: no node with id "${outcome.node_id}" in document`);
-      }
-      const op = node.op;
-      for (const [k, v] of Object.entries(outcome.params)) {
-        if (k === "type")
-          continue;
-        op[k] = v;
-      }
-      const owner = doc.roots.find((r) => collectChildNodeIds(doc, r.root).has(Number(outcome.node_id)));
-      return {
-        partId: owner ? partIdForRoot(doc, owner) : outcome.node_id,
-        nodeId: outcome.node_id
-      };
-    }
-  }
-}
 function listPartsFromDocument(doc) {
   return doc.roots.map((entry) => {
     const node = doc.nodes[String(entry.root)];
@@ -42132,6 +41953,26 @@ var init_CHANGELOG = __esm({
             "apply_edits"
           ]
         },
+        {
+          id: "2026-10-20-mcp-apply-edits-engine",
+          version: "0.10.0",
+          date: "2026-10-20",
+          category: "fix",
+          title: "apply_edits runs on the document engine",
+          summary: `MCP apply_edits now applies every op through the Rust document engine, like the single-op tools; "@N" resolves to op N's stable part id. The legacy planner fallback is gone.`,
+          features: [
+            "mcp",
+            "engine",
+            "batch-edit"
+          ],
+          mcpTools: [
+            "apply_edits",
+            "create",
+            "update",
+            "delete",
+            "set_material"
+          ]
+        },
         {
           id: "2026-10-20-lossless-import",
           version: "undefined",
@@ -55894,23 +55735,31 @@ function engineErrorMessage(toolName, args, err9) {
   }
 }
 function runMutationViaEngine(toolName, toolArgs, doc, documentId) {
-  if (!ENGINE_LOWERED_TOOLS.has(toolName)) return null;
-  if (process.env.VCAD_MCP_ENGINE_APPLY === "0") return null;
+  if (!ENGINE_LOWERED_TOOLS.has(toolName)) {
+    throw new Error(`Tool "${toolName}" is not lowered by the document engine.`);
+  }
   const wasm3 = getKernelWasmSync();
   const factory = wasm3?.WasmDocumentEngine;
-  if (!factory || typeof factory.withReplica !== "function") return null;
+  if (!factory || typeof factory.withReplica !== "function") {
+    throw new Error(
+      `${toolName}: kernel WASM has no WasmDocumentEngine \u2014 ensure the kernel is initialized and up to date.`
+    );
+  }
   const cachedEngine = takeCachedEngine(documentId, doc);
   const engine = cachedEngine ?? factory.withReplica(0);
   let keep = false;
   try {
-    if (typeof engine.apply_tool !== "function") return null;
+    if (typeof engine.apply_tool !== "function") {
+      throw new Error(
+        `${toolName}: kernel WASM predates WasmDocumentEngine.apply_tool \u2014 rebuild the kernel WASM.`
+      );
+    }
     const deprecation = typeof toolArgs.part_id === "string" ? partIdDeprecation(toolArgs.part_id, rootForPartId(doc, toolArgs.part_id)) : void 0;
     const args = engineArgs(toolName, toolArgs, doc);
     if (!cachedEngine) engine.import_ir(JSON.stringify(doc));
     const envelope = JSON.parse(engine.apply_tool(toolName, JSON.stringify(args)));
     if (!envelope.ok || !envelope.result) {
       const err9 = envelope.error ?? { kind: "invalid", message: "engine returned no result" };
-      if (err9.kind === "unsupported") return null;
       throw new Error(engineErrorMessage(toolName, args, err9));
     }
     const next = envelope.result.document;
@@ -55933,7 +55782,9 @@ function runMutationViaEngine(toolName, toolArgs, doc, documentId) {
       payload.part_id = args.part_id ?? args.part_ids;
       payload.material = toolArgs.material;
     } else {
-      const partId = envelope.result.createdFeatureId ?? (typeof args.part_id === "string" ? args.part_id : void 0);
+      const params = args.params;
+      const edited = args.part_id ?? args.child ?? params?.child;
+      const partId = envelope.result.createdFeatureId ?? (typeof edited === "string" ? edited : void 0);
       if (partId) {
         payload.part_id = partId;
         const part = parts.find((p) => p.id === partId);
@@ -55951,102 +55802,9 @@ function runMutationViaEngine(toolName, toolArgs, doc, documentId) {
     if (!keep) engine.free?.();
   }
 }
-function runMutation(toolName, rawArgs, doc, documentId, opts = {}) {
+function runMutation(toolName, rawArgs, doc, documentId) {
   const toolArgs = normalizeNodeRefs(toolName, rawArgs, doc);
-  if (opts.engine !== false) {
-    const viaEngine = runMutationViaEngine(toolName, toolArgs, doc, documentId);
-    if (viaEngine) return viaEngine;
-  }
-  if (documentId) markDocumentWritten(documentId);
-  if (toolName === "delete") {
-    const partId = String(toolArgs.part_id ?? "");
-    if (!partId) throw new Error("delete: missing `part_id`");
-    const deprecation = partIdDeprecation(partId, rootForPartId(doc, partId));
-    const applied2 = applyToolOutcome(doc, {
-      kind: "remove_part",
-      part_id: partId
-    });
-    return {
-      content: [
-        {
-          type: "text",
-          text: JSON.stringify({
-            document_id: documentId,
-            removed: applied2.partId,
-            ...deprecation ? { warnings: [deprecation] } : {}
-          })
-        }
-      ]
-    };
-  }
-  if (toolName === "set_material") {
-    const partId = String(toolArgs.part_id ?? "");
-    const material = String(
-      toolArgs.material ?? ""
-    );
-    if (!partId) throw new Error("set_material: missing `part_id` (bulk forms not yet supported via MCP)");
-    if (!material) throw new Error("set_material: missing `material`");
-    const deprecation = partIdDeprecation(partId, rootForPartId(doc, partId));
-    const applied2 = applyToolOutcome(doc, {
-      kind: "set_part_material",
-      part_id: partId,
-      material
-    });
-    return {
-      content: [
-        {
-          type: "text",
-          text: JSON.stringify({
-            document_id: documentId,
-            part_id: applied2.partId,
-            material,
-            ...deprecation ? { warnings: [deprecation] } : {}
-          })
-        }
-      ]
-    };
-  }
-  const plannerArgs = toolName === "create" ? applyParamDefaults(toolArgs) : toolArgs;
-  const planned = commandRegistry.planCrud(
-    toolName,
-    plannerArgs,
-    JSON.stringify(doc)
-  );
-  if (!planned) {
-    throw new Error(
-      `Tool "${toolName}" \u2014 Rust planner unavailable. Ensure commandRegistry.setWasm() has been called during MCP server boot.`
-    );
-  }
-  if (planned.status === "error") {
-    throw new Error(enrichPlannerError(toolName, plannerArgs, planned.result));
-  }
-  if (!planned.outcome) {
-    return {
-      content: [
-        {
-          type: "text",
-          text: JSON.stringify({
-            document_id: documentId,
-            result: planned.result
-          })
-        }
-      ]
-    };
-  }
-  const applied = applyToolOutcome(doc, planned.outcome);
-  return {
-    content: [
-      {
-        type: "text",
-        text: JSON.stringify({
-          document_id: documentId,
-          result: planned.result,
-          part_id: applied.partId,
-          node_id: applied.nodeId
-        })
-      }
-    ]
-  };
+  return runMutationViaEngine(toolName, toolArgs, doc, documentId);
 }
 function handleMeasureRead(toolName, doc, args, engine, pose) {
   if (!engine) {
@@ -58248,7 +58006,7 @@ function normalizeOp(op, index) {
   }
   return { name, args };
 }
-function resolveSymbolicRefs(args, createdIds, index, doc) {
+function resolveSymbolicRefs(args, createdIds, index) {
   const resolve7 = (v) => {
     if (typeof v !== "string") return v;
     const m = SYMBOLIC_REF.exec(v);
@@ -58262,7 +58020,7 @@ function resolveSymbolicRefs(args, createdIds, index, doc) {
     const id = createdIds[refIdx];
     if (id === void 0) {
       throw new Error(
-        `"@${refIdx}" refers to op ${refIdx}, which did not create a node`
+        `"@${refIdx}" refers to op ${refIdx}, which did not create a part`
       );
     }
     return id;
@@ -58271,12 +58029,6 @@ function resolveSymbolicRefs(args, createdIds, index, doc) {
   for (const key3 of ["part_id", "node_id"]) {
     if (key3 in out) out[key3] = resolve7(out[key3]);
   }
-  if (typeof args.part_id === "string" && SYMBOLIC_REF.test(args.part_id)) {
-    const nodeId = String(out.part_id);
-    if (doc.roots.some((r) => String(r.root) === nodeId)) {
-      out.part_id = partIdForNode(doc, nodeId);
-    }
-  }
   if (out.params && typeof out.params === "object" && !Array.isArray(out.params)) {
     const params = { ...out.params };
     for (const key3 of ["child", "left", "right", "sketch"]) {
@@ -58298,12 +58050,9 @@ function applyOps(doc, ops, documentId) {
     try {
       result = runMutation(
         name,
-        resolveSymbolicRefs(args, createdIds, i, doc),
+        resolveSymbolicRefs(args, createdIds, i),
         doc,
-        documentId,
-        // Legacy planner: "@N" and raw numeric child refs address the
-        // planner's node numbering (see runMutation).
-        { engine: false }
+        documentId
       );
     } catch (err9) {
       throw new BatchOpError(i, name, err9);
@@ -58317,7 +58066,7 @@ function applyOps(doc, ops, documentId) {
     const { document_id: _docId, ...rest } = parsed;
     void _docId;
     createdIds.push(
-      name === "create" && typeof rest.node_id === "string" ? rest.node_id : void 0
+      name === "create" && typeof rest.part_id === "string" ? rest.part_id : void 0
     );
     results.push({ index: i, op: name, ...rest });
   }
@@ -58405,7 +58154,6 @@ var init_batch_edit = __esm({
     init_registry_dispatch();
     init_integrity();
     init_tool_def();
-    init_part_ids2();
     ALLOWED_OPS = ["create", "update", "delete", "set_material"];
     MAX_BATCH_OPS = 50;
     applyEditsSchema = {
@@ -58417,7 +58165,7 @@ var init_batch_edit = __esm({
         },
         ops: {
           type: "array",
-          description: `Ordered list of edit ops applied atomically (all-or-nothing). Each op is \`{op, ...args}\` where op is one of create/update/delete/set_material and the remaining fields are that tool's arguments (minus document_id): create {op, type, params}, update {op, node_id|part_id, ...fields}, delete {op, part_id}, set_material {op, part_id, material}. Later ops may reference nodes created by earlier ops in the same batch symbolically: "@N" (in params.child/left/right/sketch or part_id/node_id) resolves to the node created by op index N \u2014 no need to predict node ids. Max ${MAX_BATCH_OPS} per call.`,
+          description: `Ordered list of edit ops applied atomically (all-or-nothing). Each op is \`{op, ...args}\` where op is one of create/update/delete/set_material and the remaining fields are that tool's arguments (minus document_id): create {op, type, params}, update {op, node_id|part_id, ...fields}, delete {op, part_id}, set_material {op, part_id, material}. Later ops may reference parts from earlier ops in the same batch symbolically: "@N" (in params.child/left/right/sketch or part_id/node_id) resolves to the part created by op index N (for an in-place edit such as translate, the part it edited) \u2014 no need to predict ids. Max ${MAX_BATCH_OPS} per call.`,
           items: { type: "object" }
         },
         dry_run: {
@@ -58445,7 +58193,7 @@ var init_batch_edit = __esm({
       {
         name: "apply_edits",
         pack: null,
-        description: "Apply an ordered list of edit ops (create/update/delete/set_material) to a session document atomically \u2014 all-or-nothing. One call replaces N single-node round-trips: it returns one aggregated `changed` diff over every op and one integrity certificate, and a subsequent `undo` rewinds the whole batch. Any op failing restores the pre-call document byte-for-byte and reports the failing op index. Pass `dry_run: true` to validate and get the per-op plan without mutating. Each op is `{op, ...args}` (e.g. {op:'create', type:'cube', params:{size:{x,y,z}}}); later ops reference nodes created earlier in the batch as \"@N\" (op index N), so a boolean chain never predicts node ids.",
+        description: "Apply an ordered list of edit ops (create/update/delete/set_material) to a session document atomically \u2014 all-or-nothing. One call replaces N single-node round-trips: it returns one aggregated `changed` diff over every op and one integrity certificate, and a subsequent `undo` rewinds the whole batch. Any op failing restores the pre-call document byte-for-byte and reports the failing op index. Pass `dry_run: true` to validate and get the per-op plan without mutating. Each op is `{op, ...args}` (e.g. {op:'create', type:'cube', params:{size:{x,y,z}}}); later ops reference parts created earlier in the batch as \"@N\" (op index N), so a boolean chain never predicts node ids.",
         inputSchema: applyEditsSchema,
         handler: (a, c) => applyEdits(a, c.engine),
         // mount: a batch edit is a milestone — refresh the viewer at the bottom
@@ -87491,8 +87239,8 @@ var init_server3 = __esm({
     init_order_feed();
     init_animate();
     PKG_VERSION = (() => {
-      if ("0.10.0-main.115") {
-        return "0.10.0-main.115";
+      if ("0.10.0-main.116") {
+        return "0.10.0-main.116";
       }
       try {
         const req = createRequire2(import.meta.url);
@@ -87501,8 +87249,8 @@ var init_server3 = __esm({
         return "0.0.0";
       }
     })();
-    BUILD_SHA = "8ad12f403ad86c964867f288c8ef53b7790be8c8";
-    BUILD_TIME = "2026-10-20T23:56:43.258Z";
+    BUILD_SHA = "72d19cb9100f1a027b8c00459e6f0994d641f28e";
+    BUILD_TIME = "2026-10-21T00:31:46.946Z";
     SHORT_SHA = BUILD_SHA === "unknown" ? "unknown" : BUILD_SHA.slice(0, 7);
     VERSION_WITH_BUILD = SHORT_SHA === "unknown" ? PKG_VERSION : `${PKG_VERSION}+${SHORT_SHA}`;
     INSTANCE_ID = randomUUID7().slice(0, 8);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vcad/mcp",
-  "version": "0.10.0-main.115",
+  "version": "0.10.0-main.116",
   "description": "vcad MCP server — parametric CAD + PCB design tools for AI agents (self-contained: bundled server + kernel WASM)",
   "license": "MIT",
   "type": "module",
@@ -19,7 +19,7 @@
   },
   "homepage": "https://vcad.io",
   "vcadBuild": {
-    "sha": "8ad12f403ad86c964867f288c8ef53b7790be8c8",
-    "builtAt": "2026-10-20T23:56:43.258Z"
+    "sha": "72d19cb9100f1a027b8c00459e6f0994d641f28e",
+    "builtAt": "2026-10-21T00:31:46.946Z"
   }
 }