@pascal-app/mcp 1.0.4 → 1.0.6

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

@@ -1,7 +1,7 @@
 import { z } from 'zod';
 import { SceneNotFoundError, SceneVersionConflictError } from '../../storage/types.js';
 import { DESTRUCTIVE_TOOL_ANNOTATIONS } from '../annotations.js';
-import { ErrorCode, throwMcpError } from '../errors.js';
+import { ProtocolErrorCode, throwMcpError } from '../errors.js';
 export const deleteSceneInput = {
     id: z.string().min(1).max(64),
     expectedVersion: z.number().int().positive().optional(),
@@ -15,8 +15,8 @@ export function registerDeleteScene(server, operations) {
     server.registerTool('delete_scene', {
         title: 'Delete scene',
         description: "Remove a stored scene by id. In Pascal this never deletes the project: it hides it from Pascal's public listings until it is next saved with content, and answers `deleted: false, hidden: true` with a message; its owner can delete it from the projects page. Only someone who can manage the project may hide it. A local scene store deletes the scene. Optionally pass `expectedVersion` for optimistic concurrency.",
-        inputSchema: deleteSceneInput,
-        outputSchema: deleteSceneOutput,
+        inputSchema: z.object(deleteSceneInput),
+        outputSchema: z.object(deleteSceneOutput),
         annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
     }, async ({ id, expectedVersion }) => {
         try {
@@ -30,16 +30,16 @@ export function registerDeleteScene(server, operations) {
         }
         catch (err) {
             if (err instanceof SceneNotFoundError) {
-                throwMcpError(ErrorCode.InvalidParams, 'scene_not_found', { id });
+                throwMcpError(ProtocolErrorCode.InvalidParams, 'scene_not_found', { id });
             }
             if (err instanceof SceneVersionConflictError) {
-                throwMcpError(ErrorCode.InvalidRequest, 'version_conflict', {
+                throwMcpError(ProtocolErrorCode.InvalidRequest, 'version_conflict', {
                     id,
                     expectedVersion,
                 });
             }
             const msg = err instanceof Error ? err.message : String(err);
-            throwMcpError(ErrorCode.InternalError, msg);
+            throwMcpError(ProtocolErrorCode.InternalError, msg);
         }
     });
 }

package/dist/tools/scene-lifecycle/get-project-status.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 getProjectStatusInput: {

package/dist/tools/scene-lifecycle/get-project-status.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get-project-status.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/get-project-status.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAA;AACxE,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAKvD,eAAO,MAAM,qBAAqB;;CAEjC,CAAA;AAED,eAAO,MAAM,sBAAsB;;;;;;;;;;;;;;;;;;;;;;CAsBlC,CAAA;AAED,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,eAAe,GAAG,IAAI,CA4C7F"}
+{"version":3,"file":"get-project-status.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/get-project-status.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAC7D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAKvD,eAAO,MAAM,qBAAqB;IAChC,EAAE;CACH,CAAA;AAED,eAAO,MAAM,sBAAsB;IACjC,EAAE;IACF,SAAS;IACT,IAAI;IACJ,SAAS;IACT,GAAG;IACH,OAAO;IACP,YAAY;IACZ,gBAAgB;IAChB,aAAa;IACb,YAAY;IACZ,qBAAqB;IACrB,OAAO;IACP,OAAO;IACP,SAAS;IACT,SAAS;IACT,SAAS;IACT,SAAS;IACT,SAAS;IACT,QAAQ;IACR,cAAc;IACd,QAAQ;CACT,CAAA;AAED,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,eAAe,GAAG,IAAI,CA4C7F"}

package/dist/tools/scene-lifecycle/get-project-status.js

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 import { DESTRUCTIVE_TOOL_ANNOTATIONS } from '../annotations.js';
-import { ErrorCode, McpError, throwMcpError } from '../errors.js';
+import { ProtocolError, ProtocolErrorCode, throwMcpError } from '../errors.js';
 import { currentLevelContext, projectStatusPayload } from './metadata.js';
 export const getProjectStatusInput = {
     id: z.string().min(1).max(64),
@@ -32,14 +32,14 @@ export function registerGetProjectStatus(server, operations) {
     server.registerTool('get_project_status', {
         title: 'Get project status',
         description: 'Authoritative status/debug call for a Pascal project: editor URL, browser-visible version, latest saved version, published version, node count, and graph hash.',
-        inputSchema: getProjectStatusInput,
-        outputSchema: getProjectStatusOutput,
+        inputSchema: z.object(getProjectStatusInput),
+        outputSchema: z.object(getProjectStatusOutput),
         annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
     }, async ({ id }) => {
         try {
             const status = await operations.getProjectStatus(id);
             if (!status) {
-                throwMcpError(ErrorCode.InvalidParams, 'project_not_found', { id });
+                throwMcpError(ProtocolErrorCode.InvalidParams, 'project_not_found', { id });
             }
             const activeScene = operations.getActiveScene();
             if (activeScene?.id !== status.id) {
@@ -62,10 +62,10 @@ export function registerGetProjectStatus(server, operations) {
             };
         }
         catch (err) {
-            if (err instanceof McpError)
+            if (err instanceof ProtocolError)
                 throw err;
             const msg = err instanceof Error ? err.message : String(err);
-            throwMcpError(ErrorCode.InternalError, msg);
+            throwMcpError(ProtocolErrorCode.InternalError, msg);
         }
     });
 }

package/dist/tools/scene-lifecycle/index.d.ts

@@ -1,4 +1,4 @@
-import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+import type { McpServer } from '@modelcontextprotocol/server';
 import type { SceneOperations } from '../../operations/index.js';
 /**
  * Register the scene-lifecycle MCP tools (`save_scene`, `load_scene`,

package/dist/tools/scene-lifecycle/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAA;AACxE,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AASvD;;;;;GAKG;AACH,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,eAAe,GAAG,IAAI,CAQhG;AAED,OAAO,EAAE,kBAAkB,EAAE,mBAAmB,EAAE,qBAAqB,EAAE,MAAM,kBAAkB,CAAA;AACjG,OAAO,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,MAAM,gBAAgB,CAAA;AACzF,OAAO,EACL,qBAAqB,EACrB,sBAAsB,EACtB,wBAAwB,GACzB,MAAM,sBAAsB,CAAA;AAC7B,OAAO,EAAE,eAAe,EAAE,gBAAgB,EAAE,kBAAkB,EAAE,MAAM,eAAe,CAAA;AACrF,OAAO,EAAE,cAAc,EAAE,eAAe,EAAE,iBAAiB,EAAE,MAAM,cAAc,CAAA;AACjF,OAAO,EAAE,mBAAmB,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAA;AACzF,OAAO,EAAE,iBAAiB,EAAE,cAAc,EAAE,eAAe,EAAE,MAAM,cAAc,CAAA"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAC7D,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AASvD;;;;;GAKG;AACH,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,eAAe,GAAG,IAAI,CAQhG;AAED,OAAO,EAAE,kBAAkB,EAAE,mBAAmB,EAAE,qBAAqB,EAAE,MAAM,kBAAkB,CAAA;AACjG,OAAO,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,MAAM,gBAAgB,CAAA;AACzF,OAAO,EACL,qBAAqB,EACrB,sBAAsB,EACtB,wBAAwB,GACzB,MAAM,sBAAsB,CAAA;AAC7B,OAAO,EAAE,eAAe,EAAE,gBAAgB,EAAE,kBAAkB,EAAE,MAAM,eAAe,CAAA;AACrF,OAAO,EAAE,cAAc,EAAE,eAAe,EAAE,iBAAiB,EAAE,MAAM,cAAc,CAAA;AACjF,OAAO,EAAE,mBAAmB,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAA;AACzF,OAAO,EAAE,iBAAiB,EAAE,cAAc,EAAE,eAAe,EAAE,MAAM,cAAc,CAAA"}

package/dist/tools/scene-lifecycle/list-scenes.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 listScenesInput: {

package/dist/tools/scene-lifecycle/list-scenes.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"list-scenes.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/list-scenes.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAA;AACxE,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAMvD,eAAO,MAAM,eAAe;;;CAG3B,CAAA;AAED,eAAO,MAAM,gBAAgB;;;;;;;;;;;;;;;;;CAmB5B,CAAA;AAED,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,eAAe,GAAG,IAAI,CA4BvF"}
+{"version":3,"file":"list-scenes.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/list-scenes.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAC7D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAMvD,eAAO,MAAM,eAAe;IAC1B,SAAS;IACT,KAAK;CACN,CAAA;AAED,eAAO,MAAM,gBAAgB;IAC3B,MAAM;;;;;;;;;;;;;;;;CAkBP,CAAA;AAED,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,eAAe,GAAG,IAAI,CA4BvF"}

package/dist/tools/scene-lifecycle/list-scenes.js

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 import { READ_ONLY_TOOL_ANNOTATIONS } from '../annotations.js';
-import { ErrorCode, throwMcpError } from '../errors.js';
+import { ProtocolErrorCode, throwMcpError } from '../errors.js';
 const DEFAULT_LIMIT = 100;
 export const listScenesInput = {
     projectId: z.string().optional(),
@@ -28,8 +28,8 @@ export function registerListScenes(server, operations) {
     server.registerTool('list_scenes', {
         title: 'List scenes',
         description: 'List scenes in the SceneStore. Optionally filter by `projectId` and cap results with `limit` (default 100).',
-        inputSchema: listScenesInput,
-        outputSchema: listScenesOutput,
+        inputSchema: z.object(listScenesInput),
+        outputSchema: z.object(listScenesOutput),
         annotations: READ_ONLY_TOOL_ANNOTATIONS,
     }, async ({ projectId, limit }) => {
         try {
@@ -45,7 +45,7 @@ export function registerListScenes(server, operations) {
         }
         catch (err) {
             const msg = err instanceof Error ? err.message : String(err);
-            throwMcpError(ErrorCode.InternalError, msg);
+            throwMcpError(ProtocolErrorCode.InternalError, msg);
         }
     });
 }

package/dist/tools/scene-lifecycle/load-scene.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 loadSceneInput: {
@@ -20,8 +20,8 @@ export declare const loadSceneOutput: {
     published: z.ZodBoolean;
     isDraft: z.ZodBoolean;
     saveMode: z.ZodEnum<{
-        draft: "draft";
         checkpoint: "checkpoint";
+        draft: "draft";
     }>;
     graphHash: z.ZodOptional<z.ZodString>;
     levelIds: z.ZodArray<z.ZodString>;

package/dist/tools/scene-lifecycle/load-scene.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"load-scene.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/load-scene.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAA;AACxE,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAKvD,eAAO,MAAM,cAAc;;CAE1B,CAAA;AAED,eAAO,MAAM,eAAe;;;;;;;;;;;;;;;;;;;;;;CAmB3B,CAAA;AAED,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,eAAe,GAAG,IAAI,CAiClF"}
+{"version":3,"file":"load-scene.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/load-scene.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAC7D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAKvD,eAAO,MAAM,cAAc;IACzB,EAAE;CACH,CAAA;AAED,eAAO,MAAM,eAAe;IAC1B,EAAE;IACF,IAAI;IACJ,SAAS;IACT,YAAY;IACZ,OAAO;IACP,SAAS;IACT,SAAS;IACT,OAAO;IACP,SAAS;IACT,SAAS;IACT,GAAG;IACH,SAAS;IACT,SAAS;IACT,OAAO;IACP,QAAQ;;;;IACR,SAAS;IACT,QAAQ;IACR,cAAc;CACf,CAAA;AAED,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,eAAe,GAAG,IAAI,CAiClF"}

package/dist/tools/scene-lifecycle/load-scene.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 { currentLevelContext, sceneMetaPayload } from './metadata.js';
 export const loadSceneInput = {
     id: z.string().min(1).max(64),
@@ -29,13 +29,13 @@ export function registerLoadScene(server, bridge) {
     server.registerTool('load_scene', {
         title: 'Load scene',
         description: 'Load a scene from the SceneStore into the bridge. Returns the scene metadata. Throws `scene_not_found` if the id does not exist.',
-        inputSchema: loadSceneInput,
-        outputSchema: loadSceneOutput,
+        inputSchema: z.object(loadSceneInput),
+        outputSchema: z.object(loadSceneOutput),
         annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
     }, async ({ id }) => {
         const result = await bridge.loadStoredScene(id);
         if (!result) {
-            throwMcpError(ErrorCode.InvalidParams, 'scene_not_found', { id });
+            throwMcpError(ProtocolErrorCode.InvalidParams, 'scene_not_found', { id });
         }
         try {
             bridge.loadJSON(result.graph);
@@ -43,7 +43,7 @@ export function registerLoadScene(server, bridge) {
         }
         catch (err) {
             const msg = err instanceof Error ? err.message : String(err);
-            throwMcpError(ErrorCode.InvalidRequest, `load_failed: ${msg}`, { id });
+            throwMcpError(ProtocolErrorCode.InvalidRequest, `load_failed: ${msg}`, { id });
         }
         const payload = {
             ...sceneMetaPayload(result, result.graph),

package/dist/tools/scene-lifecycle/metadata.d.ts

@@ -22,7 +22,6 @@ export declare function sceneMetaPayload(meta: SceneMeta, graph?: SceneGraph): {
     graphHash: string | undefined;
 };
 export declare function projectStatusPayload(status: ProjectStatus, nextStep?: string): {
-    nextStep?: string | undefined;
     id: string;
     projectId: string;
     name: string;
@@ -41,6 +40,7 @@ export declare function projectStatusPayload(status: ProjectStatus, nextStep?: s
     graphHash: string | null;
     createdAt: string;
     updatedAt: string;
+    nextStep?: string | undefined;
 };
 export declare function currentLevelContext(operations: SceneOperations): {
     levelIds: string[];

package/dist/tools/scene-lifecycle/rename-scene.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 renameSceneInput: {

package/dist/tools/scene-lifecycle/rename-scene.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"rename-scene.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/rename-scene.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAA;AACxE,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAKvD,eAAO,MAAM,gBAAgB;;;;CAI5B,CAAA;AAED,eAAO,MAAM,iBAAiB;;;;;;;;;;;CAW7B,CAAA;AAED,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,eAAe,GAAG,IAAI,CA+CxF"}
+{"version":3,"file":"rename-scene.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/rename-scene.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAC7D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAKvD,eAAO,MAAM,gBAAgB;IAC3B,EAAE;IACF,OAAO;IACP,eAAe;CAChB,CAAA;AAED,eAAO,MAAM,iBAAiB;IAC5B,EAAE;IACF,IAAI;IACJ,SAAS;IACT,YAAY;IACZ,OAAO;IACP,SAAS;IACT,SAAS;IACT,OAAO;IACP,SAAS;IACT,SAAS;CACV,CAAA;AAED,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,SAAS,EAAE,UAAU,EAAE,eAAe,GAAG,IAAI,CA+CxF"}

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

@@ -1,7 +1,7 @@
 import { z } from 'zod';
 import { SceneNotFoundError, SceneVersionConflictError } from '../../storage/types.js';
 import { DESTRUCTIVE_TOOL_ANNOTATIONS } from '../annotations.js';
-import { ErrorCode, throwMcpError } from '../errors.js';
+import { ProtocolErrorCode, throwMcpError } from '../errors.js';
 export const renameSceneInput = {
     id: z.string().min(1).max(64),
     newName: z.string().min(1).max(200),
@@ -23,8 +23,8 @@ export function registerRenameScene(server, operations) {
     server.registerTool('rename_scene', {
         title: 'Rename scene',
         description: 'Rename a scene in the SceneStore. Returns the updated SceneMeta. Optionally pass `expectedVersion` for optimistic concurrency.',
-        inputSchema: renameSceneInput,
-        outputSchema: renameSceneOutput,
+        inputSchema: z.object(renameSceneInput),
+        outputSchema: z.object(renameSceneOutput),
         annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
     }, async ({ id, newName, expectedVersion }) => {
         try {
@@ -50,16 +50,16 @@ export function registerRenameScene(server, operations) {
         }
         catch (err) {
             if (err instanceof SceneNotFoundError) {
-                throwMcpError(ErrorCode.InvalidParams, 'scene_not_found', { id });
+                throwMcpError(ProtocolErrorCode.InvalidParams, 'scene_not_found', { id });
             }
             if (err instanceof SceneVersionConflictError) {
-                throwMcpError(ErrorCode.InvalidRequest, 'version_conflict', {
+                throwMcpError(ProtocolErrorCode.InvalidRequest, 'version_conflict', {
                     id,
                     expectedVersion,
                 });
             }
             const msg = err instanceof Error ? err.message : String(err);
-            throwMcpError(ErrorCode.InternalError, msg);
+            throwMcpError(ProtocolErrorCode.InternalError, msg);
         }
     });
 }

package/dist/tools/scene-lifecycle/save-scene.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 saveSceneInput: {
@@ -8,8 +8,8 @@ export declare const saveSceneInput: {
     expectedVersion: z.ZodOptional<z.ZodNumber>;
     replace: z.ZodOptional<z.ZodBoolean>;
     saveMode: z.ZodDefault<z.ZodEnum<{
-        draft: "draft";
         checkpoint: "checkpoint";
+        draft: "draft";
     }>>;
     publish: z.ZodOptional<z.ZodBoolean>;
     thumbnail: z.ZodOptional<z.ZodString>;
@@ -32,12 +32,14 @@ export declare const saveSceneOutput: {
     published: z.ZodBoolean;
     isDraft: z.ZodBoolean;
     saveMode: z.ZodEnum<{
-        draft: "draft";
         checkpoint: "checkpoint";
+        draft: "draft";
     }>;
     graphHash: z.ZodOptional<z.ZodString>;
     levelIds: z.ZodArray<z.ZodString>;
     defaultLevelId: z.ZodNullable<z.ZodString>;
+    /** What the server changed in a provided graph before saving it, one line each. */
+    notes: z.ZodOptional<z.ZodArray<z.ZodString>>;
 };
 export declare function registerSaveScene(server: McpServer, bridge: SceneOperations): void;
 //# sourceMappingURL=save-scene.d.ts.map

package/dist/tools/scene-lifecycle/save-scene.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"save-scene.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/save-scene.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAA;AAIxE,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAOvD,eAAO,MAAM,cAAc;;;;;;;;;;;;;;CAkC1B,CAAA;AAED,eAAO,MAAM,eAAe;;;;;;;;;;;;;;;;;;;;;;CAmB3B,CAAA;AAgCD,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,eAAe,GAAG,IAAI,CAgHlF"}
+{"version":3,"file":"save-scene.d.ts","sourceRoot":"","sources":["../../../src/tools/scene-lifecycle/save-scene.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAA;AAU7D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAOvD,eAAO,MAAM,cAAc;IACzB,EAAE;IACF,IAAI;IACJ,SAAS;IACT,eAAe;IACf,OAAO;IAMP,QAAQ;;;;IAMR,OAAO;IAMP,SAAS;IACT,mBAAmB;IAInB,KAAK;CAMN,CAAA;AAED,eAAO,MAAM,eAAe;IAC1B,EAAE;IACF,IAAI;IACJ,SAAS;IACT,YAAY;IACZ,OAAO;IACP,SAAS;IACT,SAAS;IACT,OAAO;IACP,SAAS;IACT,SAAS;IACT,GAAG;IACH,SAAS;IACT,SAAS;IACT,OAAO;IACP,QAAQ;;;;IACR,SAAS;IACT,QAAQ;IACR,cAAc;IACd,mFAAmF;IACnF,KAAK;CACN,CAAA;AAgFD,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,eAAe,GAAG,IAAI,CAqIlF"}

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

@@ -1,9 +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 = {
@@ -31,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(),
@@ -52,7 +54,50 @@ 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;
 /**
@@ -73,14 +118,14 @@ async function requireSceneLoadedFrom(bridge, target) {
             : 0;
     if (nodeCount <= HOLDS_CONTENT)
         return;
-    throwMcpError(ErrorCode.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 });
+    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. 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: saveSceneInput,
-        outputSchema: saveSceneOutput,
+        inputSchema: z.object(saveSceneInput),
+        outputSchema: z.object(saveSceneOutput),
         annotations: DESTRUCTIVE_TOOL_ANNOTATIONS,
     }, 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,
@@ -94,25 +139,28 @@ export function registerSaveScene(server, bridge) {
             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 = [];
@@ -129,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, nodes: migration.nodes };
+            sceneGraph = { ...graph, nodes };
         }
         try {
             const meta = await bridge.saveScene({
@@ -152,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) }],
@@ -160,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

@@ -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;

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,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;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,iBAAiB,CAAC;IAYzE,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/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: {
@@ -10,8 +10,8 @@ export declare const setUnitMembersOutput: {
     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: {
@@ -12,8 +12,8 @@ export declare const setZoneOutput: {
     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"}