@pascal-app/mcp 1.0.6 → 1.1.0

package/dist/tools/patch-guards.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"patch-guards.d.ts","sourceRoot":"","sources":["../../src/tools/patch-guards.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,KAAK,gBAAgB,EACrB,KAAK,iBAAiB,EAKvB,MAAM,kBAAkB,CAAA;AAKzB,OAAO,EAAE,OAAO,EAAE,KAAK,SAAS,EAAyB,MAAM,yBAAyB,CAAA;AACxF,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,wBAAwB,CAAA;AAEnD,MAAM,MAAM,gBAAgB,GACxB,aAAa,GACb,iBAAiB,GACjB,iBAAiB,GACjB,gBAAgB,GAChB,gBAAgB,GAChB,qBAAqB,GACrB,gBAAgB,GAEhB,eAAe,GACf,kBAAkB,GAClB,gBAAgB,GAChB,gBAAgB,GAEhB,YAAY,GACZ,yBAAyB,GACzB,mBAAmB,CAAA;AAEvB;;;;;;;GAOG;AACH,MAAM,MAAM,UAAU,GAAG;IACvB,IAAI,EAAE,MAAM,CAAA;IACZ,KAAK,EAAE,MAAM,CAAA;IACb,GAAG,CAAC,OAAO,EAAE,KAAK,EAAE,EAAE,KAAK,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,GAAG,MAAM,EAAE,GAAG,IAAI,CAAA;CACjF,CAAA;AAID,4FAA4F;AAC5F,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,UAAU,QAKnD;AAED,qFAAqF;AACrF,wBAAgB,cAAc,CAC5B,OAAO,EAAE,KAAK,EAAE,EAChB,KAAK,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,GACvC,MAAM,EAAE,CAKV;AAED;;;;GAIG;AACH,qBAAa,iBAAkB,SAAQ,KAAK;IAExC,QAAQ,CAAC,IAAI,EAAE,gBAAgB;IAC/B,QAAQ,CAAC,UAAU,EAAE,MAAM;IAC3B,QAAQ,CAAC,MAAM,EAAE,MAAM;IAHzB,YACW,IAAI,EAAE,gBAAgB,EACtB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,MAAM,EACvB,MAAM,EAAE,MAAM,EAIf;CACF;AAkDD;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AACH,wBAAgB,wBAAwB,CACtC,OAAO,EAAE,SAAS,KAAK,EAAE,EACzB,KAAK,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,EACxC,WAAW,EAAE,SAAS,SAAS,EAAE,EACjC,YAAY,CAAC,EAAE,CAAC,KAAK,EAAE,iBAAiB,EAAE,GAAG,EAAE,SAAS,EAAE,KAAK,gBAAgB,GAC9E,IAAI,CA+KN"}
+{"version":3,"file":"patch-guards.d.ts","sourceRoot":"","sources":["../../src/tools/patch-guards.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,KAAK,gBAAgB,EACrB,KAAK,iBAAiB,EAMvB,MAAM,kBAAkB,CAAA;AAKzB,OAAO,EAAE,OAAO,EAAE,KAAK,SAAS,EAAyB,MAAM,yBAAyB,CAAA;AACxF,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,wBAAwB,CAAA;AAEnD,MAAM,MAAM,gBAAgB,GACxB,aAAa,GACb,iBAAiB,GACjB,iBAAiB,GACjB,gBAAgB,GAChB,gBAAgB,GAChB,qBAAqB,GACrB,gBAAgB,GAEhB,eAAe,GACf,kBAAkB,GAClB,gBAAgB,GAChB,gBAAgB,GAEhB,YAAY,GACZ,yBAAyB,GACzB,mBAAmB,CAAA;AAEvB;;;;;;;GAOG;AACH,MAAM,MAAM,UAAU,GAAG;IACvB,IAAI,EAAE,MAAM,CAAA;IACZ,KAAK,EAAE,MAAM,CAAA;IACb,GAAG,CAAC,OAAO,EAAE,KAAK,EAAE,EAAE,KAAK,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,GAAG,MAAM,EAAE,GAAG,IAAI,CAAA;CACjF,CAAA;AAID,4FAA4F;AAC5F,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,UAAU,QAKnD;AAED,qFAAqF;AACrF,wBAAgB,cAAc,CAC5B,OAAO,EAAE,KAAK,EAAE,EAChB,KAAK,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,GACvC,MAAM,EAAE,CAKV;AAED;;;;GAIG;AACH,qBAAa,iBAAkB,SAAQ,KAAK;IAExC,QAAQ,CAAC,IAAI,EAAE,gBAAgB;IAC/B,QAAQ,CAAC,UAAU,EAAE,MAAM;IAC3B,QAAQ,CAAC,MAAM,EAAE,MAAM;IAHzB,YACW,IAAI,EAAE,gBAAgB,EACtB,UAAU,EAAE,MAAM,EAClB,MAAM,EAAE,MAAM,EACvB,MAAM,EAAE,MAAM,EAIf;CACF;AAkDD;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AACH,wBAAgB,wBAAwB,CACtC,OAAO,EAAE,SAAS,KAAK,EAAE,EACzB,KAAK,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,EACxC,WAAW,EAAE,SAAS,SAAS,EAAE,EACjC,YAAY,CAAC,EAAE,CAAC,KAAK,EAAE,iBAAiB,EAAE,GAAG,EAAE,SAAS,EAAE,KAAK,gBAAgB,GAC9E,IAAI,CAwMN"}

package/dist/tools/patch-guards.js

@@ -1,4 +1,4 @@
-import { nodeRegistry, planNodeDeletion, previewDefaultGutterRefresh, validateNodeRelations, } from '@pascal-app/core';
+import { nodeRegistry, planNodeDeletion, previewDefaultGutterRefresh, validateNodeRelations, wouldCreateHierarchyCycle, } from '@pascal-app/core';
 import { scriptedFieldRefusal, unknownMaterialPresetRefusal, } from '@pascal-app/core/agent-operations';
 import { AnyNode, nodeKindOf, parseNode } from '@pascal-app/core/schema';
 const guards = [];
@@ -104,15 +104,25 @@ function withoutChild(parent, childId) {
  * Lives in the tool layer so every bridge behind `apply_patch` inherits it.
  */
 export function assertPatchKeepsIdentity(patches, nodes, rootNodeIds, planDeletion) {
-    const initialIds = new Set(Object.keys(nodes));
-    let scene = {
-        nodes: { ...nodes },
-        rootNodeIds: [...rootNodeIds],
-        collections: {},
+    // What the patch writes, over the scene; the whole scene is copied only for a preview that reads
+    // all of it (deletes, gutter refreshes, a reparent), so an edit reads only the ids it names.
+    const written = new Map();
+    let whole = null;
+    const scene = () => {
+        if (!whole) {
+            const all = { ...nodes };
+            for (const [id, node] of written)
+                all[id] = node;
+            whole = { nodes: all, rootNodeIds: [...rootNodeIds], collections: {} };
+        }
+        return whole;
     };
-    const at = (id) => scene.nodes[id];
+    const at = (id) => whole ? whole.nodes[id] : (written.get(id) ?? nodes[id]);
     const put = (node) => {
-        scene.nodes[node.id] = node;
+        if (whole)
+            whole.nodes[node.id] = node;
+        else
+            written.set(node.id, node);
     };
     // Default gutters and downspouts a delete run refreshes: the store may keep,
     // move or replace them with freshly minted ids, so later ops in the same
@@ -133,7 +143,7 @@ export function assertPatchKeepsIdentity(patches, nodes, rootNodeIds, planDeleti
         const roofId = node.type === 'roof' ? node.id : node.type === 'roof-segment' ? node.parentId : null;
         if (!roofId)
             return;
-        const preview = previewDefaultGutterRefresh(scene.nodes, [roofId]);
+        const preview = previewDefaultGutterRefresh(scene().nodes, [roofId]);
         for (const id of preview.unsettledIds)
             regenerated.set(id, index);
         for (const id of preview.regeneratedHostIds)
@@ -142,13 +152,13 @@ export function assertPatchKeepsIdentity(patches, nodes, rootNodeIds, planDeleti
     let pendingDeletes = [];
     const flushDeletes = (index) => {
         const plan = planDeletion
-            ? planDeletion(scene, pendingDeletes)
-            : planNodeDeletion(scene, pendingDeletes, { mintDefaults: false });
+            ? planDeletion(scene(), pendingDeletes)
+            : planNodeDeletion(scene(), pendingDeletes, { mintDefaults: false });
         for (const id of plan.unsettledIds)
             regenerated.set(id, index);
         for (const id of plan.regeneratedHostIds)
             regeneratedHosts.set(id, index);
-        scene = { nodes: plan.nodes, rootNodeIds: plan.rootNodeIds, collections: plan.collections };
+        whole = { nodes: plan.nodes, rootNodeIds: plan.rootNodeIds, collections: plan.collections };
         pendingDeletes = [];
     };
     patches.forEach((patch, index) => {
@@ -166,7 +176,7 @@ export function assertPatchKeepsIdentity(patches, nodes, rootNodeIds, planDeleti
             const effectiveParentId = patch.parentId ?? node.parentId;
             if (effectiveParentId)
                 refuseRegenerated(index, effectiveParentId);
-            if (at(node.id) || (!planDeletion && initialIds.has(node.id))) {
+            if (at(node.id) || (!planDeletion && nodes[node.id] !== undefined)) {
                 throw new PatchRefusedError('node_exists', index, node.id, planDeletion || at(node.id)
                     ? `create id "${node.id}" already exists. Use op "update" to change it, or delete it earlier in the same patch to replace it.`
                     : `create id "${node.id}" existed when the patch started, and this bridge cannot preview its deletes. Delete it in one apply_patch call and create it in the next.`);
@@ -175,6 +185,9 @@ export function assertPatchKeepsIdentity(patches, nodes, rootNodeIds, planDeleti
             if (patch.parentId !== undefined && !at(patch.parentId)) {
                 throw new Error(`invalid patch: patches[${index}] create parentId "${patch.parentId}" not found`);
             }
+            if (wouldCreateHierarchyCycle(node.id, parentId, at)) {
+                throw new PatchRefusedError('invalid_parent', index, node.id, `create cannot place "${node.id}" under ${JSON.stringify(parentId)}: it would create a hierarchy cycle.`);
+            }
             put({ ...node, parentId });
             const parent = parentId ? at(parentId) : undefined;
             if (parent)
@@ -217,11 +230,15 @@ export function assertPatchKeepsIdentity(patches, nodes, rootNodeIds, planDeleti
                 }
             }
             if ('parentId' in data && data.parentId !== current.parentId) {
+                const newParentId = typeof data.parentId === 'string' ? data.parentId : null;
+                if (wouldCreateHierarchyCycle(patch.id, newParentId, at)) {
+                    throw new PatchRefusedError('invalid_parent', index, patch.id, `update cannot move "${patch.id}" under ${JSON.stringify(data.parentId)}: it would create a hierarchy cycle.`);
+                }
                 const newParent = typeof data.parentId === 'string' ? at(data.parentId) : undefined;
                 if (!(newParent && Array.isArray(newParent.children))) {
                     throw new PatchRefusedError('invalid_parent', index, patch.id, `update cannot move "${patch.id}" under ${JSON.stringify(data.parentId)}: ${newParent ? `a ${newParent.type} holds no children` : 'no such node'}.`);
                 }
-                const before = { ...scene.nodes };
+                const before = { ...scene().nodes };
                 const oldParent = current.parentId ? at(current.parentId) : undefined;
                 if (oldParent)
                     put(withoutChild(oldParent, patch.id));
@@ -229,7 +246,7 @@ export function assertPatchKeepsIdentity(patches, nodes, rootNodeIds, planDeleti
                 put(merged);
                 unsettleRoofRefresh(merged, index);
                 try {
-                    validateNodeRelations(before, scene.nodes, [patch.id]);
+                    validateNodeRelations(before, scene().nodes, [patch.id]);
                 }
                 catch (err) {
                     throw new PatchRefusedError('invalid_parent', index, patch.id, `${newParent.type} "${newParent.id}" does not accept "${patch.id}": ${err instanceof Error ? err.message : String(err)}`);

package/dist/tools/project-files.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"project-files.d.ts","sourceRoot":"","sources":["../../src/tools/project-files.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAC7D,OAAO,EAEL,KAAK,uBAAuB,EAC7B,MAAM,mCAAmC,CAAA;AAQ1C,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,uBAAuB,QAmC3F"}
+{"version":3,"file":"project-files.d.ts","sourceRoot":"","sources":["../../src/tools/project-files.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAC7D,OAAO,EAEL,KAAK,uBAAuB,EAC7B,MAAM,mCAAmC,CAAA;AAS1C,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,uBAAuB,QAmC3F"}

package/dist/tools/project-files.js

@@ -1,11 +1,11 @@
 import { executeProjectFileTool, } from '@pascal-app/core/agent-operations';
-import { isAgentRefusal, PROJECT_FILE_TOOL_CONTRACTS, refusalPayload, } from '@pascal-app/core/agent-tools';
+import { isAgentRefusal, PROJECT_FILE_TOOL_CONTRACTS, refusalPayload, surfaceDescription, } from '@pascal-app/core/agent-tools';
 import { z } from 'zod';
 export function registerProjectFileTools(server, execute) {
     for (const contract of PROJECT_FILE_TOOL_CONTRACTS) {
         server.registerTool(contract.name, {
             title: contract.title,
-            description: contract.description,
+            description: surfaceDescription(contract, 'mcp'),
             inputSchema: z.object(contract.input).strict(),
             annotations: contract.annotations,
         }, async (input, ctx) => {

package/dist/tools/room-tools.d.ts

@@ -19,6 +19,7 @@ export declare const addDoorOutput: {
         unchanged: z.ZodOptional<z.ZodLiteral<true>>;
     }, z.core.$strip>;
     doorId: z.ZodString;
+    sillHeight: z.ZodOptional<z.ZodNumber>;
     project: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     unsaved: z.ZodOptional<z.ZodString>;
     persistence: z.ZodOptional<z.ZodObject<{

package/dist/tools/room-tools.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"room-tools.d.ts","sourceRoot":"","sources":["../../src/tools/room-tools.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAyB7D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AACpD,OAAO,EAAmB,KAAK,kBAAkB,EAAc,MAAM,cAAc,CAAA;AAMnF,mFAAmF;AACnF,eAAO,MAAM,aAAa;;;;;;;;;;;;;;;;;;;;;;;;;CAA0C,CAAA;AACpE,eAAO,MAAM,eAAe;;;;;;;;;;;;;;;;;;;;;;;;;;CAA4C,CAAA;AAwHxE,wBAAgB,eAAe,CAC7B,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,eAAe,CAAC,EAAE,kBAAkB,GACnC,IAAI,CA8BN;AAED,wBAAgB,iBAAiB,CAC/B,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,eAAe,CAAC,EAAE,kBAAkB,GACnC,IAAI,CA8BN;AAED,gGAAgG;AAChG,wBAAgB,iBAAiB,CAC/B,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,eAAe,CAAC,EAAE,kBAAkB,GACnC,IAAI,CAGN"}
+{"version":3,"file":"room-tools.d.ts","sourceRoot":"","sources":["../../src/tools/room-tools.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AA0B7D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AACpD,OAAO,EAAmB,KAAK,kBAAkB,EAAc,MAAM,cAAc,CAAA;AAOnF,mFAAmF;AACnF,eAAO,MAAM,aAAa;;;;;;;;;;;;;;;;;;;;;;;;;;CAA0C,CAAA;AACpE,eAAO,MAAM,eAAe;;;;;;;;;;;;;;;;;;;;;;;;;;CAA4C,CAAA;AA0HxE,wBAAgB,eAAe,CAC7B,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,eAAe,CAAC,EAAE,kBAAkB,GACnC,IAAI,CAiCN;AAED,wBAAgB,iBAAiB,CAC/B,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,eAAe,CAAC,EAAE,kBAAkB,GACnC,IAAI,CAiCN;AAED,gGAAgG;AAChG,wBAAgB,iBAAiB,CAC/B,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,eAAe,CAAC,EAAE,kBAAkB,GACnC,IAAI,CAGN"}

package/dist/tools/room-tools.js

@@ -1,12 +1,13 @@
 import { isScriptedNode, scriptedObjectMeta } from '@pascal-app/core';
 import { achievedChanges, addWallOpening, editedScriptLanguage, editedScriptParams, rebuiltOpeningResult, rescriptOpening, } from '@pascal-app/core/agent-operations';
-import { addDoorOutput as addDoorResult, addDoorTool, addWindowOutput as addWindowResult, addWindowTool, isAgentRefusal, } from '@pascal-app/core/agent-tools';
+import { addDoorOutput as addDoorResult, addDoorTool, addWindowOutput as addWindowResult, addWindowTool, isAgentRefusal, surfaceDescription, } from '@pascal-app/core/agent-tools';
 import { GeometryReuseFields } from '@pascal-app/core/schema';
 import { z } from 'zod';
 import { compileAndStore, readScript } from './add-object.js';
 import { ADDITIVE_TOOL_ANNOTATIONS } from './annotations.js';
 import { refusalResult, toolError } from './errors.js';
 import { liveSyncOutput, persistencePayload, publishLiveSceneSnapshot } from './live-sync.js';
+import { inScene } from './scene-scope.js';
 import { toPatches } from './shared-tools.js';
 /** The contract's one result (core `addWallOpening`), and the live sync's note. */
 export const addDoorOutput = { ...addDoorResult, ...liveSyncOutput };
@@ -43,7 +44,7 @@ async function rebuildOpening(kind, bridge, host, input) {
                 ...GeometryReuseFields.parse(input),
             },
         });
-        outcome = rescriptOpening(nodes, { ...input, compiled }, { activeLevelId: bridge.activeLevelId?.() ?? null });
+        outcome = inScene(bridge, () => rescriptOpening(nodes, { ...input, compiled }, { activeLevelId: bridge.activeLevelId?.() ?? null }));
     }
     catch (error) {
         if (isAgentRefusal(error))
@@ -94,7 +95,7 @@ async function compileOpeningScript(kind, bridge, host, input) {
 export function registerAddDoor(server, bridge, geometryScripts) {
     server.registerTool(addDoorTool.name, {
         title: addDoorTool.title,
-        description: addDoorTool.description,
+        description: surfaceDescription(addDoorTool, 'mcp'),
         inputSchema: z.object(addDoorTool.input),
         outputSchema: z.object(addDoorOutput),
         annotations: ADDITIVE_TOOL_ANNOTATIONS,
@@ -106,11 +107,12 @@ export function registerAddDoor(server, bridge, geometryScripts) {
             return compiled.error;
         let opening;
         try {
-            opening = addWallOpening(bridge.getNodes(), {
+            // Planned after the compile was awaited: back in the scene's scope for its ids.
+            opening = inScene(bridge, () => addWallOpening(bridge.getNodes(), {
                 kind: 'door',
                 ...input,
                 compiled: compiled.script,
-            });
+            }));
         }
         catch (error) {
             return refusalResult(error);
@@ -123,7 +125,7 @@ export function registerAddDoor(server, bridge, geometryScripts) {
 export function registerAddWindow(server, bridge, geometryScripts) {
     server.registerTool(addWindowTool.name, {
         title: addWindowTool.title,
-        description: addWindowTool.description,
+        description: surfaceDescription(addWindowTool, 'mcp'),
         inputSchema: z.object(addWindowTool.input),
         outputSchema: z.object(addWindowOutput),
         annotations: ADDITIVE_TOOL_ANNOTATIONS,
@@ -135,11 +137,12 @@ export function registerAddWindow(server, bridge, geometryScripts) {
             return compiled.error;
         let opening;
         try {
-            opening = addWallOpening(bridge.getNodes(), {
+            // Planned after the compile was awaited: back in the scene's scope for its ids.
+            opening = inScene(bridge, () => addWallOpening(bridge.getNodes(), {
                 kind: 'window',
                 ...input,
                 compiled: compiled.script,
-            });
+            }));
         }
         catch (error) {
             return refusalResult(error);

package/dist/tools/scene-scope.d.ts

@@ -0,0 +1,4 @@
+import type { SceneOperations } from '../operations/index.js';
+/** Runs `run` in the scene's scope when the operations have one (`SceneOperations.runInScene`). */
+export declare function inScene<T>(operations: SceneOperations, run: () => T): T;
+//# sourceMappingURL=scene-scope.d.ts.map

package/dist/tools/scene-scope.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"scene-scope.d.ts","sourceRoot":"","sources":["../../src/tools/scene-scope.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAEpD,mGAAmG;AACnG,wBAAgB,OAAO,CAAC,CAAC,EAAE,UAAU,EAAE,eAAe,EAAE,GAAG,EAAE,MAAM,CAAC,GAAG,CAAC,CAEvE"}

package/dist/tools/scene-scope.js

@@ -0,0 +1,4 @@
+/** Runs `run` in the scene's scope when the operations have one (`SceneOperations.runInScene`). */
+export function inScene(operations, run) {
+    return operations.runInScene ? operations.runInScene(run) : run();
+}

package/dist/tools/shared-tools.d.ts

@@ -18,8 +18,6 @@ export type OperationTool = {
     annotations: AgentToolAnnotations;
     outputSchema?: Record<string, z.ZodType>;
     envelope?: (bridge: SceneOperations) => Record<string, unknown>;
-    /** Reads the host's item library: only these calls wait for it (the hosted one is a query). */
-    catalog?: true;
 };
 export type PlanReadingHosts = {
     /** Reads catalog pictures and surfaces before view_assets and paint. */

package/dist/tools/shared-tools.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"shared-tools.d.ts","sourceRoot":"","sources":["../../src/tools/shared-tools.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAC7D,OAAO,EACL,KAAK,cAAc,EAGnB,KAAK,gBAAgB,EACrB,KAAK,cAAc,EACnB,KAAK,eAAe,EAUpB,KAAK,YAAY,EAGlB,MAAM,mCAAmC,CAAA;AAC1C,OAAO,EACL,KAAK,oBAAoB,EACzB,KAAK,iBAAiB,EAStB,KAAK,WAAW,EA+BjB,MAAM,8BAA8B,CAAA;AAErC,OAAO,EAGL,KAAK,sBAAsB,EAG5B,MAAM,yBAAyB,CAAA;AAChC,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,wBAAwB,CAAA;AACnD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAMpD,OAAO,EAAE,KAAK,YAAY,EAAkB,MAAM,iBAAiB,CAAA;AASnE;;;;;GAKG;AACH,MAAM,MAAM,aAAa,GAAG;IAC1B,QAAQ,EAAE,iBAAiB,CAAA;IAC3B,SAAS,EAAE,cAAc,CAAA;IACzB,WAAW,EAAE,oBAAoB,CAAA;IACjC,YAAY,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,OAAO,CAAC,CAAA;IACxC,QAAQ,CAAC,EAAE,CAAC,MAAM,EAAE,eAAe,KAAK,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;IAC/D,+FAA+F;IAC/F,OAAO,CAAC,EAAE,IAAI,CAAA;CACf,CAAA;AAED,MAAM,MAAM,gBAAgB,GAAG;IAC7B,wEAAwE;IACxE,UAAU,CAAC,EAAE,eAAe,CAAA;IAC5B,wFAAwF;IACxF,WAAW,CAAC,EAAE,CAAC,KAAK,EAAE,IAAI,CAAC,cAAc,EAAE,UAAU,CAAC,KAAK,OAAO,CAAC,sBAAsB,CAAC,CAAA;CAC3F,CAAA;AAmaD,wBAAgB,SAAS,CAAC,OAAO,EAAE,YAAY,GAAG,KAAK,EAAE,CAmBxD;AAED,qGAAqG;AACrG,wBAAgB,YAAY,CAAC,MAAM,EAAE,eAAe,GAAG,gBAAgB,CAsBtE;AAED,wBAAgB,YAAY,CAAC,MAAM,EAAE,eAAe,GAAG,WAAW,GAAG,IAAI,GAAG,SAAS,CAIpF;AAUD,wBAAgB,mBAAmB,CACjC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,OAAO,GAAE,YAA6B,EACtC,KAAK,GAAE,gBAAqB,GAC3B,IAAI,CAGN;AAED;;;GAGG;AACH,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,IAAI,EAAE,aAAa,EACnB,OAAO,GAAE,YAA6B,EACtC,KAAK,GAAE,gBAAqB,GAC3B,IAAI,CAuHN"}
+{"version":3,"file":"shared-tools.d.ts","sourceRoot":"","sources":["../../src/tools/shared-tools.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAC7D,OAAO,EACL,KAAK,cAAc,EAGnB,KAAK,gBAAgB,EACrB,KAAK,cAAc,EACnB,KAAK,eAAe,EASpB,KAAK,YAAY,EAGlB,MAAM,mCAAmC,CAAA;AAC1C,OAAO,EACL,KAAK,oBAAoB,EACzB,KAAK,iBAAiB,EAUtB,KAAK,WAAW,EAyCjB,MAAM,8BAA8B,CAAA;AAGrC,OAAO,EAGL,KAAK,sBAAsB,EAI5B,MAAM,yBAAyB,CAAA;AAChC,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,wBAAwB,CAAA;AACnD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAMpD,OAAO,EAAE,KAAK,YAAY,EAAkB,MAAM,iBAAiB,CAAA;AAUnE;;;;;GAKG;AACH,MAAM,MAAM,aAAa,GAAG;IAC1B,QAAQ,EAAE,iBAAiB,CAAA;IAC3B,SAAS,EAAE,cAAc,CAAA;IACzB,WAAW,EAAE,oBAAoB,CAAA;IACjC,YAAY,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,OAAO,CAAC,CAAA;IACxC,QAAQ,CAAC,EAAE,CAAC,MAAM,EAAE,eAAe,KAAK,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;CAChE,CAAA;AAED,MAAM,MAAM,gBAAgB,GAAG;IAC7B,wEAAwE;IACxE,UAAU,CAAC,EAAE,eAAe,CAAA;IAC5B,wFAAwF;IACxF,WAAW,CAAC,EAAE,CAAC,KAAK,EAAE,IAAI,CAAC,cAAc,EAAE,UAAU,CAAC,KAAK,OAAO,CAAC,sBAAsB,CAAC,CAAA;CAC3F,CAAA;AAoeD,wBAAgB,SAAS,CAAC,OAAO,EAAE,YAAY,GAAG,KAAK,EAAE,CAmBxD;AAED,qGAAqG;AACrG,wBAAgB,YAAY,CAAC,MAAM,EAAE,eAAe,GAAG,gBAAgB,CAyBtE;AAED,wBAAgB,YAAY,CAAC,MAAM,EAAE,eAAe,GAAG,WAAW,GAAG,IAAI,GAAG,SAAS,CAIpF;AA+CD,wBAAgB,mBAAmB,CACjC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,OAAO,GAAE,YAA6B,EACtC,KAAK,GAAE,gBAAqB,GAC3B,IAAI,CAGN;AAED;;;GAGG;AACH,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,eAAe,EACvB,IAAI,EAAE,aAAa,EACnB,OAAO,GAAE,YAA6B,EACtC,KAAK,GAAE,gBAAqB,GAC3B,IAAI,CAwIN"}

package/dist/tools/shared-tools.js

@@ -1,13 +1,15 @@
-import { AGENT_OPERATIONS, achievedChanges, applyAgentOutcome, assetViewNeeds, assetViewsReadCatalog, briefReadsCatalog, buildBatchObjects, createAssetViewLoader, createFetchAssetViewIO, furnishingRefusal, startOfSession, writeCollections, } from '@pascal-app/core/agent-operations';
-import { addCornerWindowTool, addFenceTool, addLevelTool, addObjectTool, addWallTool, createRoofTool, createRoomTool, createStairTool, deleteNodeTool, designBriefTool, duplicateLevelTool, findByTypeTool, fitStairTool, furnishRoomTool, getLevelSummaryTool, getNodeTool, getSiteTool, getWallsTool, getZonesTool, listLevelsTool, measureStairTool, mergeWindowsTool, moveRoofsToRoofLevelTool, paintTool, placeBuildingTool, placeDesignTool, placeInRoomTool, placeItemsTool, ROOM_TOOL_CONTRACTS, refuse, runBatchTool, searchAssetsTool, searchMaterialsTool, setCheckpointTool, setGroundLevelTool, validateDesignTool, verifySceneTool, viewAssetsTool, } from '@pascal-app/core/agent-tools';
+import { AGENT_OPERATIONS, achievedChanges, applyAgentOutcome, assetViewNeeds, buildBatchObjects, createAssetViewLoader, createFetchAssetViewIO, furnishingRefusal, readCatalog, startOfSession, writeCollections, } from '@pascal-app/core/agent-operations';
+import { addCornerWindowTool, addFenceTool, addLevelTool, addObjectTool, addWallTool, createRoofTool, createRoomTool, createStairTool, createUnitTool, deleteNodeTool, designBriefTool, duplicateLevelTool, duplicateNodeOutput, duplicateNodeTool, findByTypeTool, fitStairTool, furnishRoomTool, getLevelSummaryTool, getNodeTool, getSiteTool, getWallsTool, getZonesTool, isAgentRefusal, listLevelsTool, listUnitsTool, measureStairTool, mergeWindowsTool, moveNodeOutput, moveNodeTool, moveRoofsToRoofLevelTool, paintTool, placeBuildingTool, placeDesignTool, placeInRoomTool, placeItemsTool, ROOM_TOOL_CONTRACTS, refuse, runBatchTool, searchAssetsTool, searchMaterialsTool, setCheckpointTool, setGroundLevelTool, setUnitMembersTool, setWallTypeTool, surfaceDescription, validateDesignTool, verifySceneModelResult, verifySceneTool, viewAssetsTool, } from '@pascal-app/core/agent-tools';
 import { DesignValidationSchema } from '@pascal-app/core/procedural-items';
-import { MaterialSchema, SceneMaterial, } from '@pascal-app/core/schema';
+import { untracked } from '@pascal-app/core/runtime';
+import { MaterialSchema, SceneMaterial, UnitNode, } from '@pascal-app/core/schema';
 import { z } from 'zod';
 import { ADDITIVE_TOOL_ANNOTATIONS, DESTRUCTIVE_TOOL_ANNOTATIONS, READ_ONLY_TOOL_ANNOTATIONS, } from './annotations.js';
 import { builtInCatalog } from './asset-catalog.js';
 import { registerCollectionTools } from './collections.js';
 import { refusalResult } from './errors.js';
 import { liveSyncOutput, persistencePayload, publishLiveSceneSnapshot } from './live-sync.js';
+import { inScene } from './scene-scope.js';
 import { ROOM_TOOL_ANNOTATIONS, structureOutput } from './structure-tools.js';
 const defaultAssetViews = createAssetViewLoader(createFetchAssetViewIO());
 const jsonObject = z.record(z.string(), z.unknown());
@@ -118,7 +120,11 @@ const SHARED_TOOLS = [
         contract: getWallsTool,
         operation: AGENT_OPERATIONS.get_walls,
         annotations: READ_ONLY_TOOL_ANNOTATIONS,
-        outputSchema: { levelId: z.string(), walls: z.array(jsonObject) },
+        outputSchema: {
+            levelId: z.string(),
+            walls: z.array(jsonObject),
+            wallTypes: z.array(jsonObject),
+        },
     },
     {
         contract: getZonesTool,
@@ -181,7 +187,7 @@ const SHARED_TOOLS = [
             issues: z.array(z.looseObject({
                 type: z.string(),
                 message: z.string(),
-                severity: z.literal('info').optional(),
+                severity: z.enum(['info', 'warning']).optional(),
                 nodeId: z.string().optional(),
                 path: z.string().optional(),
                 zoneId: z.string().optional(),
@@ -230,6 +236,11 @@ const SHARED_TOOLS = [
         operation: AGENT_OPERATIONS.add_wall,
         annotations: ADDITIVE_TOOL_ANNOTATIONS,
     },
+    {
+        contract: setWallTypeTool,
+        operation: AGENT_OPERATIONS.set_wall_type,
+        annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
+    },
     {
         contract: addFenceTool,
         operation: AGENT_OPERATIONS.add_fence,
@@ -269,7 +280,6 @@ const SHARED_TOOLS = [
         contract: placeItemsTool,
         operation: AGENT_OPERATIONS.place_items,
         annotations: ADDITIVE_TOOL_ANNOTATIONS,
-        catalog: true,
     },
     {
         contract: deleteNodeTool,
@@ -277,6 +287,18 @@ const SHARED_TOOLS = [
         annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
         outputSchema: { deletedIds: z.array(z.string()), ...achievedOutput, ...liveSyncOutput },
     },
+    {
+        contract: moveNodeTool,
+        operation: AGENT_OPERATIONS.move_node,
+        annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
+        outputSchema: { ...moveNodeOutput, ...liveSyncOutput },
+    },
+    {
+        contract: duplicateNodeTool,
+        operation: AGENT_OPERATIONS.duplicate_node,
+        annotations: ADDITIVE_TOOL_ANNOTATIONS,
+        outputSchema: { ...duplicateNodeOutput, ...liveSyncOutput },
+    },
     {
         contract: createRoomTool,
         operation: AGENT_OPERATIONS.create_room,
@@ -310,7 +332,6 @@ const SHARED_TOOLS = [
         contract: furnishRoomTool,
         operation: AGENT_OPERATIONS.furnish_room,
         annotations: ADDITIVE_TOOL_ANNOTATIONS,
-        catalog: true,
         outputSchema: {
             ok: z.literal(true),
             placed: z.number(),
@@ -343,7 +364,6 @@ const SHARED_TOOLS = [
         contract: placeInRoomTool,
         operation: AGENT_OPERATIONS.place_in_room,
         annotations: ADDITIVE_TOOL_ANNOTATIONS,
-        catalog: true,
         outputSchema: {
             ok: z.literal(true),
             itemId: z.string(),
@@ -379,13 +399,11 @@ const SHARED_TOOLS = [
         contract: searchAssetsTool,
         operation: AGENT_OPERATIONS.search_assets,
         annotations: READ_ONLY_TOOL_ANNOTATIONS,
-        catalog: true,
     },
     {
         contract: viewAssetsTool,
         operation: AGENT_OPERATIONS.view_assets,
         annotations: READ_ONLY_TOOL_ANNOTATIONS,
-        catalog: true,
     },
     {
         contract: searchMaterialsTool,
@@ -401,6 +419,51 @@ const SHARED_TOOLS = [
             outputSchema: { ...structureOutput, ...achievedOutput },
         };
     }),
+    {
+        contract: createUnitTool,
+        operation: AGENT_OPERATIONS.create_unit,
+        annotations: ADDITIVE_TOOL_ANNOTATIONS,
+        outputSchema: { unitId: z.string(), ...achievedOutput, ...liveSyncOutput },
+    },
+    {
+        contract: setUnitMembersTool,
+        operation: AGENT_OPERATIONS.set_unit_members,
+        annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
+        outputSchema: {
+            unitId: z.string(),
+            memberCount: z.number(),
+            ...achievedOutput,
+            ...liveSyncOutput,
+        },
+    },
+    {
+        contract: listUnitsTool,
+        operation: AGENT_OPERATIONS.list_units,
+        annotations: READ_ONLY_TOOL_ANNOTATIONS,
+        outputSchema: {
+            units: z.array(z.object({
+                unitId: z.string(),
+                name: z.string(),
+                kind: UnitNode.shape.kind,
+                color: z.string(),
+                members: z.array(z.string()),
+                report: z.object({
+                    memberCount: z.number(),
+                    levelSpan: z
+                        .object({ minOrdinal: z.number(), maxOrdinal: z.number(), count: z.number() })
+                        .nullable(),
+                    grossAreaM2: z.number(),
+                    members: z.array(z.object({
+                        zoneId: z.string(),
+                        name: z.string(),
+                        levelId: z.string().nullable(),
+                        levelOrdinal: z.number().nullable(),
+                        areaM2: z.number(),
+                    })),
+                }),
+            })),
+        },
+    },
     {
         contract: runBatchTool,
         operation: AGENT_OPERATIONS.run_batch,
@@ -439,7 +502,8 @@ export function agentRuntime(bridge) {
             if (next.length)
                 bridge.applyPatch(next);
             if (changes.collections && Object.keys(changes.collections).length)
-                bridge.setCollections(writeCollections(bridge.getCollections(), changes.collections));
+                // Copying the others is not a read: the operation recorded what it read of them.
+                bridge.setCollections(writeCollections(untracked(bridge.getCollections()), changes.collections));
         },
         reconcile: () => {
             bridge.deriveStructure();
@@ -458,6 +522,37 @@ function writesScene(changes) {
         (toPatches(changes).length ||
             (changes.collections && Object.keys(changes.collections).length)));
 }
+/**
+ * search_assets as the model reads it: each item's id, name and placement facts; the category
+ * and the floor's null attachTo stay in the structured result.
+ */
+function searchAssetsModelResult(output) {
+    const groups = output.groups;
+    return {
+        ...output,
+        groups: groups.map((group) => ({
+            ...group,
+            results: group.results.map(({ id, name, dimensions, attachTo }) => ({
+                id,
+                name,
+                dimensions,
+                ...(attachTo && { attachTo }),
+            })),
+        })),
+    };
+}
+/**
+ * What the model reads of a tool's result: for verify_scene, the findings grouped and trimmed
+ * (`verifySceneModelResult`); for search_assets, the items without what placing them does not
+ * need; every other result as it is. The structured result keeps the whole.
+ */
+function modelResult(name, output, input) {
+    if (name === verifySceneTool.name)
+        return verifySceneModelResult(output, input);
+    if (name === searchAssetsTool.name)
+        return searchAssetsModelResult(output);
+    return output;
+}
 export function registerSharedTools(server, bridge, catalog = builtInCatalog, hosts = {}) {
     for (const tool of SHARED_TOOLS)
         registerOperationTool(server, bridge, tool, catalog, hosts);
@@ -470,7 +565,7 @@ export function registerSharedTools(server, bridge, catalog = builtInCatalog, ho
 export function registerOperationTool(server, bridge, tool, catalog = builtInCatalog, hosts = {}) {
     server.registerTool(tool.contract.name, {
         title: tool.contract.title,
-        description: tool.contract.description,
+        description: surfaceDescription(tool.contract, 'mcp'),
         inputSchema: z.object(tool.contract.input),
         // Loose: a client that listed the tools rejects any field the schema leaves out, and the
         // operations in core grow fields (verify_scene's guesses) that this list would miss.
@@ -478,26 +573,34 @@ export function registerOperationTool(server, bridge, tool, catalog = builtInCat
         annotations: tool.annotations,
     }, async (input) => {
         let outcome;
+        const nodes = bridge.getNodes();
         // A copy of the map: a host may write its own in place (the hosted bridge does), and a
-        // "before" that grows with the call reads every creation as unchanged.
-        const before = { ...bridge.getNodes() };
+        // "before" that grows with the call reads every creation as unchanged. Copying is not a
+        // read: on a detached scene, the operation records what it selects.
+        const before = { ...untracked(nodes) };
         // The agent's session begins with its first call, before anything is written.
         const first = bridge.agentSession && startOfSession(bridge.agentSession.read(), before);
         if (first)
             bridge.agentSession?.keep(first);
-        // Gate before catalog I/O or module compilation.
+        // Gate before module compilation. Under an open source the gate reads the catalog's tags to
+        // let technical equipment and plants through; a batch's calls are checked one by one. Any
+        // other refused call reads nothing of the library.
         const designBrief = briefInForce(bridge);
+        const gateRead = designBrief?.source === 'open'
+            ? await readCatalog(before, tool.contract.name, input, catalog)
+            : undefined;
         if (designBrief !== undefined) {
-            const refusal = furnishingRefusal(tool.contract.name, input, designBrief);
-            if (refusal)
+            const refusal = furnishingRefusal(tool.contract.name, input, designBrief, {
+                session: bridge.agentSession?.read(),
+                catalog: gateRead?.catalog,
+            });
+            if (refusal) {
+                // A refusal can ask the session to remember what it refused (a catalog piece under an open source).
+                if (refusal.keep)
+                    bridge.agentSession?.keep(refusal.keep);
                 return refusalResult(refusal);
+            }
         }
-        const batchPlacesInRoom = tool.contract.name === runBatchTool.name &&
-            Array.isArray(input.calls) &&
-            input.calls.some((call) => typeof call === 'object' &&
-                call !== null &&
-                'tool' in call &&
-                call.tool === placeInRoomTool.name);
         const batchesObjects = tool.contract.name === runBatchTool.name &&
             Array.isArray(input.calls) &&
             input.calls.some((call) => typeof call === 'object' &&
@@ -512,12 +615,10 @@ export function registerOperationTool(server, bridge, tool, catalog = builtInCat
             ...(bridge.nodeSchema && { nodeSchema: (kind) => bridge.nodeSchema(kind) }),
             session: bridge.agentSession?.read(),
             ...(designBrief !== undefined && { designBrief }),
-            ...((tool.catalog ||
-                batchPlacesInRoom ||
-                assetViewsReadCatalog(tool.contract.name, input) ||
-                briefReadsCatalog(tool.contract.name, input)) && { catalog: await catalog() }),
+            // What the call reads of the library: all of a list, or what it names of a source.
+            ...(gateRead ?? (await readCatalog(before, tool.contract.name, input, catalog))),
             ...(batchesObjects && {
-                batchObjects: await buildBatchObjects(input.calls, hosts.buildObject ?? refuseScripts),
+                batchObjects: await buildBatchObjects(input.calls, hosts.buildObject ?? refuseScripts, designBrief),
             }),
         };
         const needs = assetViewNeeds(before, tool.contract.name, input, context);
@@ -526,9 +627,12 @@ export function registerOperationTool(server, bridge, tool, catalog = builtInCat
                 pictures: needs.pictures,
             });
         try {
-            outcome = tool.operation(before, input, context);
+            outcome = inScene(bridge, () => tool.operation(nodes, input, context));
         }
         catch (error) {
+            // A refusal can ask the session to remember what it refused (a catalog piece under an open source).
+            if (isAgentRefusal(error) && error.keep)
+                bridge.agentSession?.keep(error.keep);
             return refusalResult(error);
         }
         let result = outcome.result;
@@ -563,11 +667,22 @@ export function registerOperationTool(server, bridge, tool, catalog = builtInCat
         });
         if (pictures.length)
             return {
-                content: [{ type: 'text', text: JSON.stringify(shown) }, ...pictures],
+                content: [
+                    {
+                        type: 'text',
+                        text: JSON.stringify(modelResult(tool.contract.name, shown, input)),
+                    },
+                    ...pictures,
+                ],
                 structuredContent: shown,
             };
         return {
-            content: [{ type: 'text', text: JSON.stringify(payload) }],
+            content: [
+                {
+                    type: 'text',
+                    text: JSON.stringify(modelResult(tool.contract.name, payload, input)),
+                },
+            ],
             structuredContent: payload,
         };
     });

package/dist/tools/templates/create-house-from-brief.js

@@ -64,7 +64,7 @@ function countNodeTypes(nodes) {
 export function registerCreateHouseFromBrief(server, bridge) {
     server.registerTool('create_house_from_brief', {
         title: 'Create house from brief',
-        description: 'High-level hosted workflow for external agents: choose a starter house from a brief, create/save/publish it, and return the editor URL. Use semantic tools afterward for exact customization.',
+        description: "Replace the session's scene with a starter template and save it as an unpublished draft, returning its editor URL. The template follows bedroomCount, rooms and landscaping only: garden-house for landscaping or a garden, patio or yard room, else empty-studio up to one bedroom, two-bedroom for two and garden-house for more. brief, style and constraints neither choose nor shape it; the summary repeats brief. It saves into projectId's project, or a new one named projectName or after the template; with no scene store it saves nothing. Refine it with the semantic tools afterward.",
         inputSchema: z.object(createHouseFromBriefInput),
         outputSchema: z.object(createHouseFromBriefOutput),
         annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,

package/dist/tools/view-scene.js

@@ -1,12 +1,12 @@
 import { confirmSceneViewPresentation, sceneViewNote, sceneViewPlan, } from '@pascal-app/core/agent-operations';
-import { refuse, viewSceneTool } from '@pascal-app/core/agent-tools';
+import { refuse, surfaceDescription, viewSceneTool } from '@pascal-app/core/agent-tools';
 import { z } from 'zod';
 import { READ_ONLY_TOOL_ANNOTATIONS } from './annotations.js';
 import { refusalResult } from './errors.js';
 export function registerViewScene(server, operations, views) {
     server.registerTool(viewSceneTool.name, {
         title: viewSceneTool.title,
-        description: viewSceneTool.description,
+        description: surfaceDescription(viewSceneTool, 'mcp'),
         inputSchema: z.object(viewSceneTool.input),
         annotations: READ_ONLY_TOOL_ANNOTATIONS,
     }, async (input) => {

package/examples/coordinate-conventions-demo.md

@@ -32,6 +32,7 @@ you're in.
 
 Open the scene in Pascal (or render it from the JSON) and you can read
 each claim directly off the geometry:
+
 1. **`[x, z] → world (x, 0, z)` is exact, no sign flip.** Demo A and
    Demo B both sit flat on the floor at Y = 0; their polygon vertices
    round-trip through `save_scene` / `get_scene` byte-identical to what

package/examples/photo-to-scene.md

@@ -33,6 +33,7 @@ The agent reads the attachment as a data URI and issues a single tool call:
 ```
 
 Optional knobs:
+
 - `save` (default `true`) — if `false`, the response includes `graph`
   inline instead of persisting to the `SceneStore`.
 - `defaultWallThickness` (default `0.2` m) — used when the vision model

package/examples/renovate-from-photos.md

@@ -12,6 +12,7 @@ tools to propose a renovation plan grounded in real photos.
 ## The brief
 
 The user drops four photos into the chat:
+
 1. A floorplan PDF page (exported as PNG).
 2. A photo of the current living room.
 3. A photo of the current kitchen.
@@ -147,6 +148,7 @@ against the corresponding perimeter wall:
 
 Guided by the reference photo's analysis (bright neutrals, open plan,
 minimal furnishing), the agent proposes a single logical patch:
+
 - Remove the partition wall between Living and Kitchen.
 - Relocate the kitchen island further west.
 - Delete the bulky TV unit item; leave the sofa and coffee table.