@pascal-app/mcp 1.0.3 → 1.0.5

package/dist/tools/scene-lifecycle/save-scene.js

@@ -1,8 +1,11 @@
+import { applySceneWrites, seatWrittenRoofs, } from '@pascal-app/core/agent-operations';
+import { isAgentRefusal } from '@pascal-app/core/agent-tools';
+import { migrateLegacyWallAssemblies } from '@pascal-app/core/scene-migrations';
 import { AnyNode } from '@pascal-app/core/schema';
 import { z } from 'zod';
 import { SceneVersionConflictError } from '../../storage/types.js';
 import { DESTRUCTIVE_TOOL_ANNOTATIONS } from '../annotations.js';
-import { ErrorCode, throwMcpError } from '../errors.js';
+import { ProtocolErrorCode, throwMcpError, toolError } from '../errors.js';
 import { appendLiveSceneEvent } from '../live-sync.js';
 import { currentLevelContext, sceneMetaPayload } from './metadata.js';
 export const saveSceneInput = {
@@ -10,14 +13,18 @@ export const saveSceneInput = {
     name: z.string().min(1).max(200),
     projectId: z.string().optional(),
     expectedVersion: z.number().int().positive().optional(),
+    replace: z
+        .boolean()
+        .optional()
+        .describe("Write this session's scene over a project or scene it was not loaded from, replacing what that one holds."),
     saveMode: z
         .enum(['draft', 'checkpoint'])
         .default('draft')
-        .describe('`draft` updates the browser-visible working model without polluting version history. `checkpoint` creates a meaningful saved version.'),
+        .describe('`draft` updates the browser-visible working model without polluting version history. `checkpoint` creates a meaningful saved version; it does not publish.'),
     publish: z
         .boolean()
         .optional()
-        .describe('For checkpoint saves, publish the checkpoint as the browser-visible version.'),
+        .describe("Hosted Pascal: also publish this checkpoint, making it the version the project's viewers see. Only when the user asks to publish."),
     thumbnail: z.string().url().optional(),
     includeCurrentScene: z
         .boolean()
@@ -26,7 +33,7 @@ export const saveSceneInput = {
     graph: z
         .record(z.string(), z.unknown())
         .optional()
-        .describe('Full SceneGraph { nodes, rootNodeIds, collections? } to save instead of the bridge state.'),
+        .describe("Full SceneGraph { nodes, rootNodeIds, collections? } to save instead of the bridge state. Judged against the scene it is saved over (the session's scene when the save goes to it, else the stored scene `id` names) as apply_patch is: a roof it puts on the walls of the storey holding it goes to the level above them, said in notes, and storey content on a roof level is refused. A graph with no scene under it is saved as it is."),
 };
 export const saveSceneOutput = {
     id: z.string(),
@@ -47,36 +54,117 @@ export const saveSceneOutput = {
     graphHash: z.string().optional(),
     levelIds: z.array(z.string()),
     defaultLevelId: z.string().nullable(),
+    /** What the server changed in a provided graph before saving it, one line each. */
+    notes: z.array(z.string()).optional(),
 };
+const same = (a, b) => JSON.stringify(a) === JSON.stringify(b);
+/** A graph as writes over the scene in hand, so a rule on writes judges only what it changes. */
+function writesOver(scene, nodes) {
+    const writes = [];
+    for (const node of Object.values(nodes)) {
+        const prior = scene[node.id];
+        if (!prior) {
+            writes.push({ op: 'create', node });
+            continue;
+        }
+        const next = node;
+        const data = Object.fromEntries([...new Set([...Object.keys(prior), ...Object.keys(next)])]
+            .filter((key) => !same(prior[key], next[key]))
+            .map((key) => [key, next[key]]));
+        if (Object.keys(data).length)
+            writes.push({ op: 'update', id: node.id, data });
+    }
+    for (const id of Object.keys(scene))
+        if (!nodes[id])
+            writes.push({ op: 'delete', id });
+    return writes;
+}
+/**
+ * The scene a provided graph is saved over, as it stands: the session's when the save goes to the
+ * scene in hand, else the stored one. Null for a new scene: a graph with nothing under it is saved
+ * as it is, its roofs never judged against an unrelated scene (a legacy project saved from a fresh
+ * session would otherwise read as all new writes, and be migrated or refused).
+ */
+async function savedOver(bridge, target) {
+    const active = bridge.getActiveScene();
+    const inHand = active &&
+        (target.id !== undefined
+            ? active.id === target.id
+            : target.projectId !== undefined && active.projectId === target.projectId);
+    if (inHand)
+        return bridge.getNodes();
+    if (target.id === undefined)
+        return null;
+    const stored = await bridge.loadStoredScene(target.id).catch(() => null);
+    return stored?.graph.nodes ?? null;
+}
+/** A saved scene that is more than the default site, building and level. */
+const HOLDS_CONTENT = 3;
+/**
+ * The session's scene goes over a project or scene only when it was loaded from it (or created
+ * for it). After a server reload a session starts over on a blank scene, and saving it by id wrote
+ * it over the project's draft (2026-10-03: 8 levels and 10 imported plans lost).
+ */
+async function requireSceneLoadedFrom(bridge, target) {
+    if (target.id === undefined && target.projectId === undefined)
+        return;
+    const active = bridge.getActiveScene();
+    if (target.id !== undefined ? active?.id === target.id : active?.projectId === target.projectId)
+        return;
+    const nodeCount = target.projectId !== undefined && bridge.canGetProjectStatus
+        ? ((await bridge.getProjectStatus(target.projectId))?.nodeCount ?? 0)
+        : target.id !== undefined
+            ? Object.keys((await bridge.loadStoredScene(target.id))?.graph.nodes ?? {}).length
+            : 0;
+    if (nodeCount <= HOLDS_CONTENT)
+        return;
+    throwMcpError(ProtocolErrorCode.InvalidRequest, `scene_not_loaded: this session's scene was not loaded from ${target.projectId ?? target.id}, which holds ${nodeCount} nodes — the server may have reloaded and this session started over. Call load_scene with it first, or pass replace: true to write this scene over it.`, { ...target, nodeCount });
+}
 export function registerSaveScene(server, bridge) {
     server.registerTool('save_scene', {
         title: 'Save scene',
-        description: 'Save the current scene (or a provided graph) to the SceneStore. Defaults to a browser-visible draft save so agents can iterate without creating many project versions. Use saveMode: "checkpoint" for meaningful version history.',
-        inputSchema: saveSceneInput,
-        outputSchema: saveSceneOutput,
+        description: 'Save the current scene (or a provided graph) to the SceneStore. Without id or projectId it saves to the project this session is bound to (create_project or load_scene). Defaults to a browser-visible draft save so agents can iterate without creating many project versions. Use saveMode: "checkpoint" for meaningful version history.',
+        inputSchema: z.object(saveSceneInput),
+        outputSchema: z.object(saveSceneOutput),
         annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
-    }, async ({ id, name, projectId, expectedVersion, saveMode, publish, thumbnail, includeCurrentScene, graph, }) => {
+    }, async ({ id, name, projectId, expectedVersion, saveMode, publish, thumbnail, includeCurrentScene, graph, replace, }) => {
+        // Without a target, the session's scene goes where it came from (create_project, load_scene,
+        // an earlier save). An agent was refused "Call create_project first" on its first save, right
+        // after create_project bound the session to its project.
+        const bound = includeCurrentScene && id === undefined && projectId === undefined
+            ? bridge.getActiveScene()
+            : null;
+        if (bound) {
+            id = bound.id;
+            projectId = bound.projectId ?? undefined;
+        }
         let sceneGraph;
+        let notes = [];
+        if (includeCurrentScene && !replace)
+            await requireSceneLoadedFrom(bridge, { id, projectId });
         if (includeCurrentScene) {
             const validation = bridge.validateScene();
             if (!validation.valid) {
-                throwMcpError(ErrorCode.InvalidRequest, 'scene_invalid', { errors: validation.errors });
+                throwMcpError(ProtocolErrorCode.InvalidRequest, 'scene_invalid', {
+                    errors: validation.errors,
+                });
             }
             sceneGraph = bridge.exportSceneGraph();
         }
         else {
             if (!graph) {
-                throwMcpError(ErrorCode.InvalidParams, 'graph_required: pass `graph` when includeCurrentScene is false');
+                throwMcpError(ProtocolErrorCode.InvalidParams, 'graph_required: pass `graph` when includeCurrentScene is false');
             }
             // Security: revalidate every node with AnyNode schema (including the
             // AssetUrl allowlist) BEFORE persisting. Without this, the save_scene
             // graph arg is a bypass for the URL hardening in A7. See P4 report.
             const rawNodes = graph.nodes;
             if (!rawNodes || typeof rawNodes !== 'object') {
-                throwMcpError(ErrorCode.InvalidParams, 'graph.nodes must be an object');
+                throwMcpError(ProtocolErrorCode.InvalidParams, 'graph.nodes must be an object');
             }
+            const migration = migrateLegacyWallAssemblies(rawNodes);
             const errors = [];
-            for (const [nodeId, node] of Object.entries(rawNodes)) {
+            for (const [nodeId, node] of Object.entries(migration.nodes)) {
                 const res = AnyNode.safeParse(node);
                 if (!res.success) {
                     for (const issue of res.error.issues) {
@@ -89,9 +177,28 @@ export function registerSaveScene(server, bridge) {
                 }
             }
             if (errors.length > 0) {
-                throwMcpError(ErrorCode.InvalidParams, 'graph_invalid', { errors });
+                throwMcpError(ProtocolErrorCode.InvalidParams, 'graph_invalid', { errors });
+            }
+            // The roof-level rule, as apply_patch holds it: what the graph changes in the scene it
+            // replaces.
+            const scene = await savedOver(bridge, { id, projectId });
+            let nodes = migration.nodes;
+            if (scene) {
+                const writes = writesOver(scene, nodes);
+                const written = writes.length;
+                try {
+                    notes = seatWrittenRoofs(writes, scene);
+                }
+                catch (error) {
+                    if (!isAgentRefusal(error))
+                        throw error;
+                    // An index into the graph's writes means nothing to the caller; the ids do.
+                    const { index: _index, ...details } = error.details;
+                    return toolError(error.message, { code: error.code, ...details });
+                }
+                nodes = applySceneWrites(nodes, writes.slice(written));
             }
-            sceneGraph = graph;
+            sceneGraph = { ...graph, nodes };
         }
         try {
             const meta = await bridge.saveScene({
@@ -112,6 +219,7 @@ export function registerSaveScene(server, bridge) {
             const payload = {
                 ...sceneMetaPayload(meta, sceneGraph),
                 ...currentLevelContext(bridge),
+                ...(notes.length ? { notes } : {}),
             };
             return {
                 content: [{ type: 'text', text: JSON.stringify(payload) }],
@@ -120,13 +228,13 @@ export function registerSaveScene(server, bridge) {
         }
         catch (err) {
             if (err instanceof SceneVersionConflictError) {
-                throwMcpError(ErrorCode.InvalidRequest, 'version_conflict', {
+                throwMcpError(ProtocolErrorCode.InvalidRequest, 'version_conflict', {
                     expectedVersion,
                     id,
                 });
             }
             const msg = err instanceof Error ? err.message : String(err);
-            throwMcpError(ErrorCode.InvalidRequest, msg);
+            throwMcpError(ProtocolErrorCode.InvalidRequest, msg);
         }
     });
 }

package/dist/tools/scene-lifecycle/test-utils.d.ts

@@ -1,6 +1,6 @@
 import { SceneBridge } from '../../bridge/scene-bridge.js';
 import { type SceneOperations } from '../../operations/index.js';
-import { type ProjectCreateOptions, type ProjectStatus, type SceneEvent, type SceneEventAppendOptions, type SceneEventListOptions, type SceneListOptions, type SceneMeta, type SceneMutateOptions, type SceneSaveOptions, type SceneStore, type SceneWithGraph } from '../../storage/types.js';
+import { type ProjectCreateOptions, type ProjectStatus, type SceneDeleteResult, type SceneEvent, type SceneEventAppendOptions, type SceneEventListOptions, type SceneListOptions, type SceneMeta, type SceneMutateOptions, type SceneSaveOptions, type SceneStore, type SceneWithGraph } from '../../storage/types.js';
 export type StoredTextContent = {
     type: string;
     text: string;
@@ -20,7 +20,7 @@ export declare function createTestSceneOperations(options?: {
  * `expectedVersion`.
  */
 export declare class InMemorySceneStore implements SceneStore {
-    readonly backend: "sqlite";
+    readonly backend: 'sqlite';
     private readonly data;
     private readonly projects;
     private readonly events;
@@ -32,7 +32,7 @@ export declare class InMemorySceneStore implements SceneStore {
     save(opts: SceneSaveOptions): Promise<SceneMeta>;
     load(id: string): Promise<SceneWithGraph | null>;
     list(opts?: SceneListOptions): Promise<SceneMeta[]>;
-    delete(id: string, opts?: SceneMutateOptions): Promise<boolean>;
+    delete(id: string, opts?: SceneMutateOptions): Promise<SceneDeleteResult>;
     rename(id: string, newName: string, opts?: SceneMutateOptions): Promise<SceneMeta>;
     appendSceneEvent(opts: SceneEventAppendOptions): Promise<SceneEvent>;
     listSceneEvents(sceneId: string, opts?: SceneEventListOptions): Promise<SceneEvent[]>;

package/dist/tools/scene-lifecycle/test-utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"test-utils.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/test-utils.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,2BAA2B,CAAA;AACvD,OAAO,EAAyB,KAAK,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAC9E,OAAO,EACL,KAAK,oBAAoB,EACzB,KAAK,aAAa,EAClB,KAAK,UAAU,EACf,KAAK,uBAAuB,EAC5B,KAAK,qBAAqB,EAC1B,KAAK,gBAAgB,EACrB,KAAK,SAAS,EACd,KAAK,kBAAkB,EAEvB,KAAK,gBAAgB,EACrB,KAAK,UAAU,EAEf,KAAK,cAAc,EACpB,MAAM,qBAAqB,CAAA;AAG5B,MAAM,MAAM,iBAAiB,GAAG;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAA;CAAE,CAAA;AAE9D,wBAAgB,aAAa,CAAC,OAAO,EAAE,iBAAiB,EAAE,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAEnF;AAED,wBAAgB,yBAAyB,CAAC,OAAO,CAAC,EAAE;IAClD,MAAM,CAAC,EAAE,WAAW,CAAA;IACpB,KAAK,CAAC,EAAE,kBAAkB,CAAA;CAC3B,GAAG;IACF,MAAM,EAAE,WAAW,CAAA;IACnB,KAAK,EAAE,kBAAkB,CAAA;IACzB,UAAU,EAAE,eAAe,CAAA;CAC5B,CAKA;AAED;;;;GAIG;AACH,qBAAa,kBAAmB,YAAW,UAAU;IACnD,QAAQ,CAAC,OAAO,EAAG,QAAQ,CAAS;IACpC,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAoC;IACzD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAWtB;IACH,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAmB;IAC1C,OAAO,CAAC,SAAS,CAAI;IACrB,OAAO,CAAC,cAAc,CAAI;IAC1B,OAAO,CAAC,YAAY,CAAI;IAElB,aAAa,CAAC,IAAI,EAAE,oBAAoB,GAAG,OAAO,CAAC,aAAa,CAAC;IAejE,gBAAgB,CAAC,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,aAAa,GAAG,IAAI,CAAC;IAK3D,IAAI,CAAC,IAAI,EAAE,gBAAgB,GAAG,OAAO,CAAC,SAAS,CAAC;IA+DhD,IAAI,CAAC,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,GAAG,IAAI,CAAC;IAShD,IAAI,CAAC,IAAI,CAAC,EAAE,gBAAgB,GAAG,OAAO,CAAC,SAAS,EAAE,CAAC;IAanD,MAAM,CAAC,EAAE,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,OAAO,CAAC;IAW/D,MAAM,CAAC,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,SAAS,CAAC;IAmBlF,gBAAgB,CAAC,IAAI,EAAE,uBAAuB,GAAG,OAAO,CAAC,UAAU,CAAC;IAapE,eAAe,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,UAAU,EAAE,CAAC;IAU3F,OAAO,CAAC,YAAY;IAiBpB,OAAO,CAAC,MAAM;IAoBd,OAAO,CAAC,eAAe;CA0BxB"}
+{"version":3,"file":"test-utils.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/test-utils.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,2BAA2B,CAAA;AACvD,OAAO,EAAyB,KAAK,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAC9E,OAAO,EACL,KAAK,oBAAoB,EACzB,KAAK,aAAa,EAClB,KAAK,iBAAiB,EACtB,KAAK,UAAU,EACf,KAAK,uBAAuB,EAC5B,KAAK,qBAAqB,EAC1B,KAAK,gBAAgB,EACrB,KAAK,SAAS,EACd,KAAK,kBAAkB,EAEvB,KAAK,gBAAgB,EACrB,KAAK,UAAU,EAEf,KAAK,cAAc,EACpB,MAAM,qBAAqB,CAAA;AAG5B,MAAM,MAAM,iBAAiB,GAAG;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAA;CAAE,CAAA;AAE9D,wBAAgB,aAAa,CAAC,OAAO,EAAE,iBAAiB,EAAE,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAEnF;AAED,wBAAgB,yBAAyB,CAAC,OAAO,CAAC,EAAE;IAClD,MAAM,CAAC,EAAE,WAAW,CAAA;IACpB,KAAK,CAAC,EAAE,kBAAkB,CAAA;CAC3B,GAAG;IACF,MAAM,EAAE,WAAW,CAAA;IACnB,KAAK,EAAE,kBAAkB,CAAA;IACzB,UAAU,EAAE,eAAe,CAAA;CAC5B,CAKA;AAED;;;;GAIG;AACH,qBAAa,kBAAmB,YAAW,UAAU;IACnD,QAAQ,CAAC,OAAO,EAAG,QAAQ,CAAS;IACpC,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAoC;IACzD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAWtB;IACH,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAmB;IAC1C,OAAO,CAAC,SAAS,CAAI;IACrB,OAAO,CAAC,cAAc,CAAI;IAC1B,OAAO,CAAC,YAAY,CAAI;IAElB,aAAa,CAAC,IAAI,EAAE,oBAAoB,GAAG,OAAO,CAAC,aAAa,CAAC,CAatE;IAEK,gBAAgB,CAAC,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,aAAa,GAAG,IAAI,CAAC,CAGhE;IAEK,IAAI,CAAC,IAAI,EAAE,gBAAgB,GAAG,OAAO,CAAC,SAAS,CAAC,CA6DrD;IAEK,IAAI,CAAC,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,GAAG,IAAI,CAAC,CAOrD;IAEK,IAAI,CAAC,IAAI,CAAC,EAAE,gBAAgB,GAAG,OAAO,CAAC,SAAS,EAAE,CAAC,CAWxD;IAEK,MAAM,CAAC,EAAE,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,iBAAiB,CAAC,CAU9E;IAEK,MAAM,CAAC,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,SAAS,CAAC,CAiBvF;IAEK,gBAAgB,CAAC,IAAI,EAAE,uBAAuB,GAAG,OAAO,CAAC,UAAU,CAAC,CAWzE;IAEK,eAAe,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,UAAU,EAAE,CAAC,CAQ1F;IAED,OAAO,CAAC,YAAY;IAiBpB,OAAO,CAAC,MAAM;IAoBd,OAAO,CAAC,eAAe;CA0BxB"}

package/dist/tools/scene-lifecycle/test-utils.js

@@ -130,7 +130,8 @@ export class InMemorySceneStore {
         if (opts?.expectedVersion !== undefined && rec.version !== opts.expectedVersion) {
             throw new SceneVersionConflictError(`Expected version ${opts.expectedVersion}, have ${rec.version}`);
         }
-        return this.data.delete(id);
+        this.data.delete(id);
+        return { deleted: true, hidden: false };
     }
     async rename(id, newName, opts) {
         const rec = this.data.get(id);

package/dist/tools/set-unit-members.d.ts

@@ -1,4 +1,4 @@
-import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+import type { McpServer } from '@modelcontextprotocol/server';
 import { z } from 'zod';
 import type { SceneOperations } from '../operations/index.js';
 export declare const setUnitMembersInput: {
@@ -6,10 +6,12 @@ export declare const setUnitMembersInput: {
     memberZoneIds: z.ZodArray<z.ZodString>;
 };
 export declare const setUnitMembersOutput: {
+    project: z.ZodOptional<z.ZodNullable<z.ZodString>>;
+    unsaved: z.ZodOptional<z.ZodString>;
     persistence: z.ZodOptional<z.ZodObject<{
         status: z.ZodEnum<{
-            unbound: "unbound";
             events_unsupported: "events_unsupported";
+            unbound: "unbound";
         }>;
         warning: z.ZodString;
     }, z.core.$strip>>;

package/dist/tools/set-unit-members.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"set-unit-members.d.ts","sourceRoot":"","sources":["../../src/tools/set-unit-members.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAA;AAExE,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAOpD,eAAO,MAAM,mBAAmB;;;CAG/B,CAAA;AAED,eAAO,MAAM,oBAAoB;;;;;;;;;;CAIhC,CAAA;AAED,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,eAAe,GAAG,IAAI,CAgCvF"}
+{"version":3,"file":"set-unit-members.d.ts","sourceRoot":"","sources":["../../src/tools/set-unit-members.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAE7D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAOpD,eAAO,MAAM,mBAAmB;IAC9B,MAAM;IACN,aAAa;CACd,CAAA;AAED,eAAO,MAAM,oBAAoB;;;;;;;;;;;;CAIhC,CAAA;AAED,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,eAAe,GAAG,IAAI,CAmCvF"}

package/dist/tools/set-unit-members.js

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 import { DESTRUCTIVE_TOOL_ANNOTATIONS } from './annotations.js';
-import { ErrorCode, throwMcpError } from './errors.js';
+import { ProtocolErrorCode, throwMcpError } from './errors.js';
 import { liveSyncOutput, persistencePayload, publishLiveSceneSnapshot } from './live-sync.js';
 import { NodeIdSchema } from './schemas.js';
 import { validateUnitMembers } from './unit-members.js';
@@ -17,19 +17,19 @@ export function registerSetUnitMembers(server, bridge) {
     server.registerTool('set_unit_members', {
         title: 'Set unit members',
         description: 'Replace the zones referenced by a unit. Every zone must belong to a level in the same building.',
-        inputSchema: setUnitMembersInput,
-        outputSchema: setUnitMembersOutput,
+        inputSchema: z.object(setUnitMembersInput),
+        outputSchema: z.object(setUnitMembersOutput),
         annotations: { ...DESTRUCTIVE_TOOL_ANNOTATIONS, idempotentHint: true },
     }, async ({ unitId, memberZoneIds }) => {
         const unit = bridge.getNode(unitId);
         if (!unit) {
-            throwMcpError(ErrorCode.InvalidParams, `Unit not found: ${unitId}`);
+            throwMcpError(ProtocolErrorCode.InvalidParams, `Unit not found: ${unitId}`);
         }
         if (unit.type !== 'unit') {
-            throwMcpError(ErrorCode.InvalidParams, `Node ${unitId} is a ${unit.type}, expected unit`);
+            throwMcpError(ProtocolErrorCode.InvalidParams, `Node ${unitId} is a ${unit.type}, expected unit`);
         }
         if (!unit.parentId) {
-            throwMcpError(ErrorCode.InvalidParams, `Unit ${unitId} must belong to a building`);
+            throwMcpError(ProtocolErrorCode.InvalidParams, `Unit ${unitId} must belong to a building`);
         }
         const members = validateUnitMembers(bridge, unit.parentId, memberZoneIds);
         bridge.updateNode(unit.id, { members });

package/dist/tools/set-zone.d.ts

@@ -1,4 +1,4 @@
-import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+import type { McpServer } from '@modelcontextprotocol/server';
 import { z } from 'zod';
 import type { SceneOperations } from '../operations/index.js';
 export declare const setZoneInput: {
@@ -8,10 +8,12 @@ export declare const setZoneInput: {
     properties: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
 };
 export declare const setZoneOutput: {
+    project: z.ZodOptional<z.ZodNullable<z.ZodString>>;
+    unsaved: z.ZodOptional<z.ZodString>;
     persistence: z.ZodOptional<z.ZodObject<{
         status: z.ZodEnum<{
-            unbound: "unbound";
             events_unsupported: "events_unsupported";
+            unbound: "unbound";
         }>;
         warning: z.ZodString;
     }, z.core.$strip>>;

package/dist/tools/set-zone.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"set-zone.d.ts","sourceRoot":"","sources":["../../src/tools/set-zone.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAA;AAGxE,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAMpD,eAAO,MAAM,YAAY;;;;;CAKxB,CAAA;AAED,eAAO,MAAM,aAAa;;;;;;;;;CAGzB,CAAA;AAED,wBAAgB,eAAe,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,eAAe,GAAG,IAAI,CAiDhF"}
+{"version":3,"file":"set-zone.d.ts","sourceRoot":"","sources":["../../src/tools/set-zone.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAI7D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAMpD,eAAO,MAAM,YAAY;IACvB,OAAO;IACP,OAAO;IACP,KAAK;IACL,UAAU;CACX,CAAA;AAED,eAAO,MAAM,aAAa;;;;;;;;;;;CAGzB,CAAA;AAED,wBAAgB,eAAe,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,eAAe,GAAG,IAAI,CAwDhF"}

package/dist/tools/set-zone.js

@@ -1,7 +1,8 @@
-import { ZoneNode } from '@pascal-app/core/schema';
+import { isAllocatedRoomName, newZone, walledRoomAt } from '@pascal-app/core';
+import { roofLevelRefusal } from '@pascal-app/core/agent-operations';
 import { z } from 'zod';
 import { ADDITIVE_TOOL_ANNOTATIONS } from './annotations.js';
-import { ErrorCode, throwMcpError } from './errors.js';
+import { ProtocolErrorCode, refusalResult, throwMcpError } from './errors.js';
 import { liveSyncOutput, persistencePayload, publishLiveSceneSnapshot } from './live-sync.js';
 import { NodeIdSchema, Vec2Schema } from './schemas.js';
 export const setZoneInput = {
@@ -17,30 +18,41 @@ export const setZoneOutput = {
 export function registerSetZone(server, bridge) {
     server.registerTool('set_zone', {
         title: 'Set zone',
-        description: 'Create a polygonal zone on the given level. label is stored as the zone name and properties are merged into metadata.',
-        inputSchema: setZoneInput,
-        outputSchema: setZoneOutput,
+        description: 'Create a polygonal zone on the given level. label is stored as the zone name and properties are merged into metadata. A polygon along walls that already enclose a room is that room, never a second zone: a room still named "Room N", or already named label, takes the label and properties; a room a person named otherwise is refused (rename it with update_node only when the person asked).',
+        inputSchema: z.object(setZoneInput),
+        outputSchema: z.object(setZoneOutput),
         annotations: ADDITIVE_TOOL_ANNOTATIONS,
     }, async ({ levelId, polygon, label, properties }) => {
         const parent = bridge.getNode(levelId);
         if (!parent) {
-            throwMcpError(ErrorCode.InvalidParams, `Level not found: ${levelId}`);
+            throwMcpError(ProtocolErrorCode.InvalidParams, `Level not found: ${levelId}`);
         }
         if (parent.type !== 'level') {
-            throwMcpError(ErrorCode.InvalidParams, `Node ${levelId} is a ${parent.type}, expected level`);
+            throwMcpError(ProtocolErrorCode.InvalidParams, `Node ${levelId} is a ${parent.type}, expected level`);
         }
-        if (typeof parent.metadata === 'object' &&
-            parent.metadata !== null &&
-            'role' in parent.metadata &&
-            parent.metadata.role === 'roof') {
-            throwMcpError(ErrorCode.InvalidParams, `Roof support level ${levelId} is not an occupied story; create zones on an occupied level instead`);
+        const onRoofLevel = roofLevelRefusal(bridge.getNodes(), levelId, 'a zone');
+        if (onRoofLevel)
+            return refusalResult(onRoofLevel);
+        const walled = walledRoomAt(bridge.getNodes(), levelId, polygon);
+        if (walled && !isAllocatedRoomName(walled.name) && walled.name !== label) {
+            throwMcpError(ProtocolErrorCode.InvalidParams, `The walls here already enclose the room "${walled.name}" (${walled.id}), which a person named; a zone "${label}" would stack a second room on it. Rename it with update_node only if the person asked; otherwise leave it.`);
+        }
+        let id;
+        if (walled) {
+            id = walled.id;
+            bridge.updateNode(id, {
+                name: label,
+                metadata: { ...walled.metadata, ...properties },
+            });
+        }
+        else {
+            const zone = newZone({
+                name: label,
+                polygon: polygon,
+                metadata: properties ?? {},
+            });
+            id = bridge.createNode(zone, levelId);
         }
-        const zone = ZoneNode.parse({
-            name: label,
-            polygon: polygon,
-            metadata: properties ?? {},
-        });
-        const id = bridge.createNode(zone, levelId);
         const persistence = await publishLiveSceneSnapshot(bridge, 'set_zone');
         const payload = { zoneId: id, ...persistencePayload(persistence) };
         return {

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

@@ -0,0 +1,40 @@
+import type { McpServer } from '@modelcontextprotocol/server';
+import { type AddObjectInput, type AgentHostRuntime, type AgentOperation, type AssetViewLoader, type SceneChanges } from '@pascal-app/core/agent-operations';
+import { type AgentToolAnnotations, type AgentToolContract, type DesignBrief } from '@pascal-app/core/agent-tools';
+import { type CompiledGeometryScript } from '@pascal-app/core/schema';
+import { z } from 'zod';
+import type { Patch } from '../bridge/scene-bridge.js';
+import type { SceneOperations } from '../operations/index.js';
+import { type AssetCatalog } from './asset-catalog.js';
+/**
+ * A tool the server runs as a core-style operation: parse with the contract, plan with the
+ * operation, apply its changes through the bridge in one history step, answer with what the scene
+ * then holds. `annotations` are the host's to choose from its own review of the operation; a
+ * plugin contract's `annotations` are only its hints.
+ */
+export type OperationTool = {
+    contract: AgentToolContract;
+    operation: AgentOperation;
+    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. */
+    assetViews?: AssetViewLoader;
+    /** Compiles one add_object module for a batch before the synchronous operations run. */
+    buildObject?: (input: Omit<AddObjectInput, 'compiled'>) => Promise<CompiledGeometryScript>;
+};
+export declare function toPatches(changes: SceneChanges): Patch[];
+/** How an outcome reaches this bridge: its patches, its derived construction, its session record. */
+export declare function agentRuntime(bridge: SceneOperations): AgentHostRuntime;
+export declare function briefInForce(bridge: SceneOperations): DesignBrief | null | undefined;
+export declare function registerSharedTools(server: McpServer, bridge: SceneOperations, catalog?: AssetCatalog, hosts?: PlanReadingHosts): void;
+/**
+ * Registers one operation tool, as the shared tools are: a host adds a plugin's `'operation'`
+ * tools with it, from the plugin's `./agent-tools` contract and `./agent-operations` operation.
+ */
+export declare function registerOperationTool(server: McpServer, bridge: SceneOperations, tool: OperationTool, catalog?: AssetCatalog, hosts?: PlanReadingHosts): void;
+//# sourceMappingURL=shared-tools.d.ts.map

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

@@ -0,0 +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"}