@imgly/codesign-mcp 0.1.6 → 0.1.7

package/dist/test/test/tools/preview-harness.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"preview-harness.js","sourceRoot":"","sources":["../../../../test/tools/preview-harness.ts"],"names":[],"mappings":"AAAA,2HAA2H;AAC3H,OAAO,EAAE,KAAK,EAAE,MAAM,2BAA2B,CAAC;AAClD,OAAO,EAAsB,aAAa,EAAE,MAAM,6BAA6B,CAAC;AAChF,OAAO,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAC7D,OAAO,EAAE,mBAAmB,EAAE,MAAM,kCAAkC,CAAC;AAEvE,MAAM,UAAU,kBAAkB;IAChC,MAAM,EAAE,GAAG,IAAI,mBAAmB,EAAE,CAAC;IACrC,MAAM,KAAK,GAAG,IAAI,aAAa,CAAC,EAAE,CAAC,CAAC;IACpC,MAAM,UAAU,GAAG;QACjB,KAAK,EAAE;YACL,MAAM,EAAE,KAAK,EAAE,GAAW,EAAE,KAAa,EAAE,EAAE,CAC3C,IAAI,IAAI,CAAC,CAAC,IAAI,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;SAC3C;KACF,CAAC;IACF,MAAM,OAAO,GAAkB;QAC7B,MAAM,EAAE,GAAG,EAAE,CAAC,UAAU;QACxB,KAAK,EAAE,KAAK,IAAI,EAAE,GAAE,CAAC;QACrB,cAAc,EAAE,KAAK,IAAI,EAAE,GAAE,CAAC;QAC9B,YAAY,EAAE,KAAK,IAAI,EAAE,CAAC,EAAE;KAC7B,CAAC;IACF,MAAM,OAAO,GAAG,IAAI,aAAa,CAAC,EAAE,IAAI,EAAE,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;IACxD,OAAO,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,IAAI,KAAK,EAAE,EAAE,CAAC;AACrD,CAAC","sourcesContent":["// Shared preview-tool test harness — a plain module (no test registrations) so importing it never re-runs preview.test.ts.\nimport { Mutex } from '../../src/engine/mutex.js';\nimport { type EngineAdapter, EngineSession } from '../../src/engine/session.js';\nimport { RevisionStore } from '../../src/revisions/store.js';\nimport { InMemoryWorkspaceFS } from '../../src/workspace/in-memory.js';\n\nexport function makePreviewHarness() {\n  const fs = new InMemoryWorkspaceFS();\n  const store = new RevisionStore(fs);\n  const fakeEngine = {\n    block: {\n      export: async (_id: number, _mime: string) =>\n        new Blob([new Uint8Array([1, 2, 3, 4])])\n    }\n  };\n  const adapter: EngineAdapter = {\n    create: () => fakeEngine,\n    reset: async () => {},\n    loadFromString: async () => {},\n    saveToString: async () => ''\n  };\n  const session = new EngineSession({ size: 1, adapter });\n  return { store, session, engineLock: new Mutex() };\n}\n"]}

package/dist/test/test/tools/preview.telemetry.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=preview.telemetry.test.d.ts.map

package/dist/test/test/tools/preview.telemetry.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"preview.telemetry.test.d.ts","sourceRoot":"","sources":["../../../../test/tools/preview.telemetry.test.ts"],"names":[],"mappings":""}

package/dist/test/test/tools/preview.telemetry.test.js

@@ -1,36 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { createPreviewTool } from '../../src/tools/preview.js';
-import { makePreviewHarness } from './preview-harness.js';
-describe('preview telemetry', () => {
-    it('records engine_execute and no write phases when revision is supplied', async () => {
-        const reports = [];
-        const harness = makePreviewHarness();
-        const root = await harness.store.write({
-            scene: 'SCENE_FOR_PREVIEW',
-            parent: null,
-            title: 'T'
-        });
-        const tool = createPreviewTool({
-            ...harness,
-            telemetry: { record: (r) => reports.push(r) }
-        });
-        const res = await tool.call({ blockId: 1, revision: root.id });
-        expect(res.isError).toBeUndefined();
-        expect(reports).toHaveLength(1);
-        const phases = reports[0].spans.map((s) => s.phase);
-        expect(phases).toContain('engine_execute');
-        expect(phases).not.toContain('engine_scene_save');
-        expect(reports[0].revision).toBe(root.id);
-    });
-    it('records succeeded=false when blockId is missing', async () => {
-        const reports = [];
-        const tool = createPreviewTool({
-            ...makePreviewHarness(),
-            telemetry: { record: (r) => reports.push(r) }
-        });
-        const res = await tool.call({});
-        expect(res.isError).toBe(true);
-        expect(reports[0].succeeded).toBe(false);
-    });
-});
-//# sourceMappingURL=preview.telemetry.test.js.map

package/dist/test/test/tools/preview.telemetry.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"preview.telemetry.test.js","sourceRoot":"","sources":["../../../../test/tools/preview.telemetry.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,4BAA4B,CAAC;AAE/D,OAAO,EAAE,kBAAkB,EAAE,MAAM,sBAAsB,CAAC;AAE1D,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;QACpF,MAAM,OAAO,GAAqB,EAAE,CAAC;QACrC,MAAM,OAAO,GAAG,kBAAkB,EAAE,CAAC;QACrC,MAAM,IAAI,GAAG,MAAM,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC;YACrC,KAAK,EAAE,mBAAmB;YAC1B,MAAM,EAAE,IAAI;YACZ,KAAK,EAAE,GAAG;SACX,CAAC,CAAC;QACH,MAAM,IAAI,GAAG,iBAAiB,CAAC;YAC7B,GAAG,OAAO;YACV,SAAS,EAAE,EAAE,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE;SAC9C,CAAC,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,EAAE,EAAE,CAAC,CAAC;QAC/D,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,aAAa,EAAE,CAAC;QACpC,MAAM,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAChC,MAAM,MAAM,GAAG,OAAO,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;QACpD,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;QAC3C,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,MAAM,OAAO,GAAqB,EAAE,CAAC;QACrC,MAAM,IAAI,GAAG,iBAAiB,CAAC;YAC7B,GAAG,kBAAkB,EAAE;YACvB,SAAS,EAAE,EAAE,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE;SAC9C,CAAC,CAAC;QACH,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAChC,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC/B,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC3C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["import { describe, it, expect } from 'vitest';\nimport { createPreviewTool } from '../../src/tools/preview.js';\nimport type { ToolCallReport } from '../../src/tools/spans.js';\nimport { makePreviewHarness } from './preview-harness.js';\n\ndescribe('preview telemetry', () => {\n  it('records engine_execute and no write phases when revision is supplied', async () => {\n    const reports: ToolCallReport[] = [];\n    const harness = makePreviewHarness();\n    const root = await harness.store.write({\n      scene: 'SCENE_FOR_PREVIEW',\n      parent: null,\n      title: 'T'\n    });\n    const tool = createPreviewTool({\n      ...harness,\n      telemetry: { record: (r) => reports.push(r) }\n    });\n    const res = await tool.call({ blockId: 1, revision: root.id });\n    expect(res.isError).toBeUndefined();\n    expect(reports).toHaveLength(1);\n    const phases = reports[0].spans.map((s) => s.phase);\n    expect(phases).toContain('engine_execute');\n    expect(phases).not.toContain('engine_scene_save');\n    expect(reports[0].revision).toBe(root.id);\n  });\n\n  it('records succeeded=false when blockId is missing', async () => {\n    const reports: ToolCallReport[] = [];\n    const tool = createPreviewTool({\n      ...makePreviewHarness(),\n      telemetry: { record: (r) => reports.push(r) }\n    });\n    const res = await tool.call({});\n    expect(res.isError).toBe(true);\n    expect(reports[0].succeeded).toBe(false);\n  });\n});\n"]}

package/dist/test/test/tools/preview.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=preview.test.d.ts.map

package/dist/test/test/tools/preview.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"preview.test.d.ts","sourceRoot":"","sources":["../../../../test/tools/preview.test.ts"],"names":[],"mappings":""}

package/dist/test/test/tools/preview.test.js

@@ -1,75 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { Mutex } from '../../src/engine/mutex.js';
-import { EngineSession } from '../../src/engine/session.js';
-import { RevisionStore } from '../../src/revisions/store.js';
-import { createPreviewTool } from '../../src/tools/preview.js';
-import { InMemoryWorkspaceFS } from '../../src/workspace/in-memory.js';
-const engineLock = new Mutex();
-function makeDeps() {
-    const fs = new InMemoryWorkspaceFS();
-    const store = new RevisionStore(fs);
-    const fakeEngine = {
-        _scene: '',
-        block: {
-            export: vi.fn(async (_id, _mime) => new Blob([new Uint8Array([1, 2, 3, 4])]))
-        }
-    };
-    const adapter = {
-        create: () => fakeEngine,
-        reset: async (e) => {
-            e._scene = 'EMPTY';
-        },
-        loadFromString: async (e, s) => {
-            e._scene = s;
-        },
-        saveToString: async (e) => e._scene
-    };
-    const session = new EngineSession({ size: 1, adapter });
-    return { fs, store, session, fakeEngine, adapter };
-}
-describe('preview tool', () => {
-    it('renders the active engine state when no revision is supplied', async () => {
-        const { store, session, fakeEngine } = makeDeps();
-        const slot = await session.acquire(null);
-        session.commit(slot, 'rev_warm');
-        fakeEngine._scene = 'WARM';
-        const tool = createPreviewTool({ store, session, engineLock });
-        const result = await tool.call({ blockId: 42 });
-        expect(result.isError).toBeFalsy();
-        expect(result.content[0].type).toBe('image');
-        expect(fakeEngine.block.export).toHaveBeenCalledWith(42, 'image/webp', expect.any(Object));
-        // Engine state is preserved (NOT reset) when no revision is supplied.
-        expect(fakeEngine._scene).toBe('WARM');
-    });
-    it('returns isError when no revision supplied AND no warm slot exists', async () => {
-        const { store, session } = makeDeps();
-        const tool = createPreviewTool({ store, session, engineLock });
-        const result = await tool.call({ blockId: 42 });
-        expect(result.isError).toBe(true);
-        expect(result.content[0].text).toMatch(/no.*engine.*state/i);
-    });
-    it('loads a revision into the engine when `revision` is supplied', async () => {
-        const { store, session, adapter } = makeDeps();
-        const root = await store.write({
-            scene: 'SCENE_FOR_PREVIEW',
-            parent: null,
-            title: 'T'
-        });
-        const tool = createPreviewTool({ store, session, engineLock });
-        // adapter.loadFromString is set as a plain async fn (not a vi.fn) above, so wrap it
-        const loadSpy = vi.spyOn(adapter, 'loadFromString');
-        await tool.call({ blockId: 42, revision: root.id });
-        expect(loadSpy).toHaveBeenCalledWith(expect.anything(), 'SCENE_FOR_PREVIEW');
-    });
-    it('clamps width/height to 1800 px', async () => {
-        const { store, session, fakeEngine } = makeDeps();
-        const slot = await session.acquire(null);
-        session.commit(slot, 'rev_warm');
-        const tool = createPreviewTool({ store, session, engineLock });
-        await tool.call({ blockId: 42, width: 9999, height: 9999 });
-        const call = fakeEngine.block.export.mock
-            .calls[0];
-        expect(call[2]).toMatchObject({ targetWidth: 1800, targetHeight: 1800 });
-    });
-});
-//# sourceMappingURL=preview.test.js.map

package/dist/test/test/tools/preview.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"preview.test.js","sourceRoot":"","sources":["../../../../test/tools/preview.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAClD,OAAO,EAAE,KAAK,EAAE,MAAM,2BAA2B,CAAC;AAClD,OAAO,EAAsB,aAAa,EAAE,MAAM,6BAA6B,CAAC;AAChF,OAAO,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAC7D,OAAO,EAAE,iBAAiB,EAAE,MAAM,4BAA4B,CAAC;AAC/D,OAAO,EAAE,mBAAmB,EAAE,MAAM,kCAAkC,CAAC;AAEvE,MAAM,UAAU,GAAG,IAAI,KAAK,EAAE,CAAC;AAO/B,SAAS,QAAQ;IACf,MAAM,EAAE,GAAG,IAAI,mBAAmB,EAAE,CAAC;IACrC,MAAM,KAAK,GAAG,IAAI,aAAa,CAAC,EAAE,CAAC,CAAC;IACpC,MAAM,UAAU,GAAe;QAC7B,MAAM,EAAE,EAAE;QACV,KAAK,EAAE;YACL,MAAM,EAAE,EAAE,CAAC,EAAE,CACX,KAAK,EAAE,GAAW,EAAE,KAAa,EAAE,EAAE,CACnC,IAAI,IAAI,CAAC,CAAC,IAAI,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAC3C;SACF;KACF,CAAC;IACF,MAAM,OAAO,GAA8B;QACzC,MAAM,EAAE,GAAG,EAAE,CAAC,UAAU;QACxB,KAAK,EAAE,KAAK,EAAE,CAAC,EAAE,EAAE;YACjB,CAAC,CAAC,MAAM,GAAG,OAAO,CAAC;QACrB,CAAC;QACD,cAAc,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE;YAC7B,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC;QACf,CAAC;QACD,YAAY,EAAE,KAAK,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM;KACpC,CAAC;IACF,MAAM,OAAO,GAAG,IAAI,aAAa,CAAa,EAAE,IAAI,EAAE,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;IACpE,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,OAAO,EAAE,CAAC;AACrD,CAAC;AAED,QAAQ,CAAC,cAAc,EAAE,GAAG,EAAE;IAC5B,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,GAAG,QAAQ,EAAE,CAAC;QAClD,MAAM,IAAI,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;QACzC,OAAO,CAAC,MAAM,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;QACjC,UAAU,CAAC,MAAM,GAAG,MAAM,CAAC;QAC3B,MAAM,IAAI,GAAG,iBAAiB,CAAC,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,CAAC;QAC/D,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC,CAAC;QAChD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,EAAE,CAAC;QACnC,MAAM,CAAE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QACnE,MAAM,CAAC,UAAU,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,oBAAoB,CAClD,EAAE,EACF,YAAY,EACZ,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CACnB,CAAC;QACF,sEAAsE;QACtE,MAAM,CAAC,UAAU,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IACzC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mEAAmE,EAAE,KAAK,IAAI,EAAE;QACjF,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,GAAG,QAAQ,EAAE,CAAC;QACtC,MAAM,IAAI,GAAG,iBAAiB,CAAC,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,CAAC;QAC/D,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC,CAAC;QAChD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClC,MAAM,CAAE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAAC,CAAC,OAAO,CAC1D,oBAAoB,CACrB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,OAAO,EAAE,GAAG,QAAQ,EAAE,CAAC;QAC/C,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,KAAK,CAAC;YAC7B,KAAK,EAAE,mBAAmB;YAC1B,MAAM,EAAE,IAAI;YACZ,KAAK,EAAE,GAAG;SACX,CAAC,CAAC;QACH,MAAM,IAAI,GAAG,iBAAiB,CAAC,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,CAAC;QAC/D,oFAAoF;QACpF,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,gBAAgB,CAAC,CAAC;QACpD,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,QAAQ,EAAE,IAAI,CAAC,EAAE,EAAE,CAAC,CAAC;QACpD,MAAM,CAAC,OAAO,CAAC,CAAC,oBAAoB,CAClC,MAAM,CAAC,QAAQ,EAAE,EACjB,mBAAmB,CACpB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,KAAK,IAAI,EAAE;QAC9C,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,GAAG,QAAQ,EAAE,CAAC;QAClD,MAAM,IAAI,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;QACzC,OAAO,CAAC,MAAM,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;QACjC,MAAM,IAAI,GAAG,iBAAiB,CAAC,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,CAAC;QAC/D,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5D,MAAM,IAAI,GAAI,UAAU,CAAC,KAAK,CAAC,MAAmC,CAAC,IAAI;aACpE,KAAK,CAAC,CAAC,CAAC,CAAC;QACZ,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,YAAY,EAAE,IAAI,EAAE,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["import { describe, expect, it, vi } from 'vitest';\nimport { Mutex } from '../../src/engine/mutex.js';\nimport { type EngineAdapter, EngineSession } from '../../src/engine/session.js';\nimport { RevisionStore } from '../../src/revisions/store.js';\nimport { createPreviewTool } from '../../src/tools/preview.js';\nimport { InMemoryWorkspaceFS } from '../../src/workspace/in-memory.js';\n\nconst engineLock = new Mutex();\n\ninterface FakeEngine {\n  _scene: string;\n  block: { export: ReturnType<typeof vi.fn> };\n}\n\nfunction makeDeps() {\n  const fs = new InMemoryWorkspaceFS();\n  const store = new RevisionStore(fs);\n  const fakeEngine: FakeEngine = {\n    _scene: '',\n    block: {\n      export: vi.fn(\n        async (_id: number, _mime: string) =>\n          new Blob([new Uint8Array([1, 2, 3, 4])])\n      )\n    }\n  };\n  const adapter: EngineAdapter<FakeEngine> = {\n    create: () => fakeEngine,\n    reset: async (e) => {\n      e._scene = 'EMPTY';\n    },\n    loadFromString: async (e, s) => {\n      e._scene = s;\n    },\n    saveToString: async (e) => e._scene\n  };\n  const session = new EngineSession<FakeEngine>({ size: 1, adapter });\n  return { fs, store, session, fakeEngine, adapter };\n}\n\ndescribe('preview tool', () => {\n  it('renders the active engine state when no revision is supplied', async () => {\n    const { store, session, fakeEngine } = makeDeps();\n    const slot = await session.acquire(null);\n    session.commit(slot, 'rev_warm');\n    fakeEngine._scene = 'WARM';\n    const tool = createPreviewTool({ store, session, engineLock });\n    const result = await tool.call({ blockId: 42 });\n    expect(result.isError).toBeFalsy();\n    expect((result.content[0] as { type: string }).type).toBe('image');\n    expect(fakeEngine.block.export).toHaveBeenCalledWith(\n      42,\n      'image/webp',\n      expect.any(Object)\n    );\n    // Engine state is preserved (NOT reset) when no revision is supplied.\n    expect(fakeEngine._scene).toBe('WARM');\n  });\n\n  it('returns isError when no revision supplied AND no warm slot exists', async () => {\n    const { store, session } = makeDeps();\n    const tool = createPreviewTool({ store, session, engineLock });\n    const result = await tool.call({ blockId: 42 });\n    expect(result.isError).toBe(true);\n    expect((result.content[0] as { text: string }).text).toMatch(\n      /no.*engine.*state/i\n    );\n  });\n\n  it('loads a revision into the engine when `revision` is supplied', async () => {\n    const { store, session, adapter } = makeDeps();\n    const root = await store.write({\n      scene: 'SCENE_FOR_PREVIEW',\n      parent: null,\n      title: 'T'\n    });\n    const tool = createPreviewTool({ store, session, engineLock });\n    // adapter.loadFromString is set as a plain async fn (not a vi.fn) above, so wrap it\n    const loadSpy = vi.spyOn(adapter, 'loadFromString');\n    await tool.call({ blockId: 42, revision: root.id });\n    expect(loadSpy).toHaveBeenCalledWith(\n      expect.anything(),\n      'SCENE_FOR_PREVIEW'\n    );\n  });\n\n  it('clamps width/height to 1800 px', async () => {\n    const { store, session, fakeEngine } = makeDeps();\n    const slot = await session.acquire(null);\n    session.commit(slot, 'rev_warm');\n    const tool = createPreviewTool({ store, session, engineLock });\n    await tool.call({ blockId: 42, width: 9999, height: 9999 });\n    const call = (fakeEngine.block.export as ReturnType<typeof vi.fn>).mock\n      .calls[0];\n    expect(call[2]).toMatchObject({ targetWidth: 1800, targetHeight: 1800 });\n  });\n});\n"]}

package/dist/test/test/tools/rename.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=rename.test.d.ts.map

package/dist/test/test/tools/rename.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"rename.test.d.ts","sourceRoot":"","sources":["../../../../test/tools/rename.test.ts"],"names":[],"mappings":""}

package/dist/test/test/tools/rename.test.js

@@ -1,16 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { RevisionStore } from '../../src/revisions/store.js';
-import { createRenameTool } from '../../src/tools/rename.js';
-import { InMemoryWorkspaceFS } from '../../src/workspace/in-memory.js';
-describe('rename tool', () => {
-    it('renames the root reachable from the given revision', async () => {
-        const store = new RevisionStore(new InMemoryWorkspaceFS());
-        const r1 = await store.write({ scene: 'a', parent: null, title: 'Old' });
-        const r2 = await store.write({ scene: 'b', parent: r1.id });
-        const tool = createRenameTool({ store });
-        const result = await tool.call({ revision: r2.id, title: 'New' });
-        expect(result.content[0].text).toContain('ok');
-        expect(await store.currentTitle(r1.id)).toBe('New');
-    });
-});
-//# sourceMappingURL=rename.test.js.map

package/dist/test/test/tools/rename.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"rename.test.js","sourceRoot":"","sources":["../../../../test/tools/rename.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAC7D,OAAO,EAAE,gBAAgB,EAAE,MAAM,2BAA2B,CAAC;AAC7D,OAAO,EAAE,mBAAmB,EAAE,MAAM,kCAAkC,CAAC;AAEvE,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;IAC3B,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAClE,MAAM,KAAK,GAAG,IAAI,aAAa,CAAC,IAAI,mBAAmB,EAAE,CAAC,CAAC;QAC3D,MAAM,EAAE,GAAG,MAAM,KAAK,CAAC,KAAK,CAAC,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC,CAAC;QACzE,MAAM,EAAE,GAAG,MAAM,KAAK,CAAC,KAAK,CAAC,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;QAC5D,MAAM,IAAI,GAAG,gBAAgB,CAAC,EAAE,KAAK,EAAE,CAAC,CAAC;QACzC,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC,CAAC;QAClE,MAAM,CAAE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACrE,MAAM,CAAC,MAAM,KAAK,CAAC,YAAY,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["import { describe, expect, it } from 'vitest';\nimport { RevisionStore } from '../../src/revisions/store.js';\nimport { createRenameTool } from '../../src/tools/rename.js';\nimport { InMemoryWorkspaceFS } from '../../src/workspace/in-memory.js';\n\ndescribe('rename tool', () => {\n  it('renames the root reachable from the given revision', async () => {\n    const store = new RevisionStore(new InMemoryWorkspaceFS());\n    const r1 = await store.write({ scene: 'a', parent: null, title: 'Old' });\n    const r2 = await store.write({ scene: 'b', parent: r1.id });\n    const tool = createRenameTool({ store });\n    const result = await tool.call({ revision: r2.id, title: 'New' });\n    expect((result.content[0] as { text: string }).text).toContain('ok');\n    expect(await store.currentTitle(r1.id)).toBe('New');\n  });\n});\n"]}

package/dist/test/test/tools/save.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=save.test.d.ts.map

package/dist/test/test/tools/save.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"save.test.d.ts","sourceRoot":"","sources":["../../../../test/tools/save.test.ts"],"names":[],"mappings":""}

package/dist/test/test/tools/save.test.js

@@ -1,137 +0,0 @@
-import { describe, expect, it, vi } from 'vitest';
-import { Mutex } from '../../src/engine/mutex.js';
-import { EngineSession } from '../../src/engine/session.js';
-import { RevisionStore } from '../../src/revisions/store.js';
-import { createSaveTool } from '../../src/tools/save.js';
-import { LocalViewerUrlBuilder } from '../../src/viewer/url-builder.js';
-import { InMemoryWorkspaceFS } from '../../src/workspace/in-memory.js';
-const engineLock = new Mutex();
-const urlBuilder = new LocalViewerUrlBuilder({ base: 'http://127.0.0.1:9000' });
-function makeDeps() {
-    const fs = new InMemoryWorkspaceFS();
-    const store = new RevisionStore(fs);
-    const fakeEngine = {
-        _scene: '',
-        _resourcesLoaded: false,
-        scene: {
-            getPages: vi.fn(() => [10]),
-            // The archive bundles the asset marker ONLY if resources were loaded
-            // first — mirrors saveToArchive's "assets accessible when called" contract.
-            saveToArchive: vi.fn(async () => {
-                const body = fakeEngine._resourcesLoaded ? 'ZIP+ASSET' : 'ZIP';
-                return new Blob([new TextEncoder().encode(body)]);
-            })
-        },
-        block: {
-            forceLoadResources: vi.fn(async () => {
-                fakeEngine._resourcesLoaded = true;
-            })
-        }
-    };
-    const adapter = {
-        create: () => fakeEngine,
-        reset: async (e) => {
-            e._scene = 'EMPTY';
-        },
-        loadFromString: async (e, s) => {
-            e._scene = s;
-        },
-        saveToString: async (e) => e._scene
-    };
-    const session = new EngineSession({ size: 1, adapter });
-    return { fs, store, session, fakeEngine };
-}
-describe('save tool', () => {
-    it('writes archive.zip and returns uri + httpUrl, no path', async () => {
-        const { fs, store, session } = makeDeps();
-        const root = await store.write({
-            scene: 'SCENE_ABC',
-            parent: null,
-            title: 'T'
-        });
-        const tool = createSaveTool({ fs, store, session, engineLock, urlBuilder });
-        const result = await tool.call({ revision: root.id });
-        const body = JSON.parse(result.content[0].text);
-        const shard = root.id.slice(0, 2);
-        const rest = root.id.slice(2);
-        expect(body.uri).toBe(`workspace://exports/${shard}/${rest}/archive.zip`);
-        expect(body.httpUrl).toBe(`http://127.0.0.1:9000/r/exports/${shard}/${rest}/archive.zip`);
-        expect(body.path).toBeUndefined();
-        expect(body.format).toBe('archive');
-        expect(body.revision).toBe(root.id);
-        expect(await fs.exists(`exports/${shard}/${rest}/archive.zip`)).toBe(true);
-    });
-    it('respects local `outPath` escape hatch — returns path, not httpUrl', async () => {
-        const { fs, store, session } = makeDeps();
-        const root = await store.write({ scene: 'S', parent: null, title: 'T' });
-        const { mkdtemp, rm } = await import('node:fs/promises');
-        const { tmpdir } = await import('node:os');
-        const { join } = await import('node:path');
-        const dir = await mkdtemp(join(tmpdir(), 'save-out-'));
-        const outPath = join(dir, 'design.zip');
-        const tool = createSaveTool({
-            fs,
-            store,
-            session,
-            engineLock,
-            urlBuilder,
-            allowOutPath: true
-        });
-        const result = await tool.call({ revision: root.id, outPath });
-        const body = JSON.parse(result.content[0].text);
-        expect(body.path).toBe(outPath);
-        expect(body.httpUrl).toBeUndefined();
-        await rm(dir, { recursive: true });
-    });
-    it('rejects `outPath` when not allowed (Hosted/sandbox)', async () => {
-        const { fs, store, session } = makeDeps();
-        const root = await store.write({ scene: 'S', parent: null, title: 'T' });
-        const tool = createSaveTool({
-            fs,
-            store,
-            session,
-            engineLock,
-            urlBuilder,
-            allowOutPath: false
-        });
-        const result = await tool.call({
-            revision: root.id,
-            outPath: '/tmp/x.zip'
-        });
-        expect(result.isError).toBe(true);
-        expect(result.content[0].text).toMatch(/outPath.*not.*supported/i);
-    });
-    it('omits `outPath` from the schema when not allowed (Hosted), advertises it when allowed (Local)', () => {
-        const { fs, store, session } = makeDeps();
-        const base = { fs, store, session, engineLock, urlBuilder };
-        const hosted = createSaveTool({ ...base, allowOutPath: false });
-        const local = createSaveTool({ ...base, allowOutPath: true });
-        const props = (s) => s.properties;
-        expect(props(hosted.inputSchema)).not.toHaveProperty('outPath');
-        expect(props(local.inputSchema)).toHaveProperty('outPath');
-    });
-    it('errors when revision is missing', async () => {
-        const { fs, store, session } = makeDeps();
-        const tool = createSaveTool({ fs, store, session, engineLock, urlBuilder });
-        const result = await tool.call({});
-        expect(result.isError).toBe(true);
-        expect(result.content[0].text).toMatch(/revision/i);
-    });
-    it('force-loads resources before archiving so assets are bundled', async () => {
-        const { fs, store, session, fakeEngine } = makeDeps();
-        const root = await store.write({ scene: 'S', parent: null, title: 'T' });
-        const tool = createSaveTool({ fs, store, session, engineLock, urlBuilder });
-        await tool.call({ revision: root.id });
-        // forceLoadResources must run with the scene's pages...
-        expect(fakeEngine.block.forceLoadResources).toHaveBeenCalledWith([10]);
-        // ...and BEFORE saveToArchive.
-        const callOrder = (m) => m.mock.invocationCallOrder[0];
-        expect(callOrder(fakeEngine.block.forceLoadResources)).toBeLessThan(callOrder(fakeEngine.scene.saveToArchive));
-        // ...so the produced archive actually contains the asset bytes.
-        const shard = root.id.slice(0, 2);
-        const rest = root.id.slice(2);
-        const zip = await fs.readFile(`exports/${shard}/${rest}/archive.zip`, 'utf8');
-        expect(zip).toContain('ASSET');
-    });
-});
-//# sourceMappingURL=save.test.js.map

package/dist/test/test/tools/save.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"save.test.js","sourceRoot":"","sources":["../../../../test/tools/save.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAClD,OAAO,EAAE,KAAK,EAAE,MAAM,2BAA2B,CAAC;AAClD,OAAO,EAAsB,aAAa,EAAE,MAAM,6BAA6B,CAAC;AAChF,OAAO,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAC7D,OAAO,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,iCAAiC,CAAC;AACxE,OAAO,EAAE,mBAAmB,EAAE,MAAM,kCAAkC,CAAC;AAEvE,MAAM,UAAU,GAAG,IAAI,KAAK,EAAE,CAAC;AAC/B,MAAM,UAAU,GAAG,IAAI,qBAAqB,CAAC,EAAE,IAAI,EAAE,uBAAuB,EAAE,CAAC,CAAC;AAYhF,SAAS,QAAQ;IACf,MAAM,EAAE,GAAG,IAAI,mBAAmB,EAAE,CAAC;IACrC,MAAM,KAAK,GAAG,IAAI,aAAa,CAAC,EAAE,CAAC,CAAC;IACpC,MAAM,UAAU,GAAe;QAC7B,MAAM,EAAE,EAAE;QACV,gBAAgB,EAAE,KAAK;QACvB,KAAK,EAAE;YACL,QAAQ,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC;YAC3B,qEAAqE;YACrE,4EAA4E;YAC5E,aAAa,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE;gBAC9B,MAAM,IAAI,GAAG,UAAU,CAAC,gBAAgB,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,CAAC;gBAC/D,OAAO,IAAI,IAAI,CAAC,CAAC,IAAI,WAAW,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YACpD,CAAC,CAAC;SACH;QACD,KAAK,EAAE;YACL,kBAAkB,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE;gBACnC,UAAU,CAAC,gBAAgB,GAAG,IAAI,CAAC;YACrC,CAAC,CAAC;SACH;KACF,CAAC;IACF,MAAM,OAAO,GAA8B;QACzC,MAAM,EAAE,GAAG,EAAE,CAAC,UAAU;QACxB,KAAK,EAAE,KAAK,EAAE,CAAC,EAAE,EAAE;YACjB,CAAC,CAAC,MAAM,GAAG,OAAO,CAAC;QACrB,CAAC;QACD,cAAc,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE;YAC7B,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC;QACf,CAAC;QACD,YAAY,EAAE,KAAK,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM;KACpC,CAAC;IACF,MAAM,OAAO,GAAG,IAAI,aAAa,CAAa,EAAE,IAAI,EAAE,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;IACpE,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC;AAC5C,CAAC;AAED,QAAQ,CAAC,WAAW,EAAE,GAAG,EAAE;IACzB,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;QACrE,MAAM,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,GAAG,QAAQ,EAAE,CAAC;QAC1C,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,KAAK,CAAC;YAC7B,KAAK,EAAE,WAAW;YAClB,MAAM,EAAE,IAAI;YACZ,KAAK,EAAE,GAAG;SACX,CAAC,CAAC;QACH,MAAM,IAAI,GAAG,cAAc,CAAC,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,UAAU,EAAE,CAAC,CAAC;QAC5E,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,EAAE,EAAE,CAAC,CAAC;QACtD,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAOnE,CAAC;QACF,MAAM,KAAK,GAAG,IAAI,CAAC,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;QAClC,MAAM,IAAI,GAAG,IAAI,CAAC,EAAE,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;QAC9B,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,uBAAuB,KAAK,IAAI,IAAI,cAAc,CAAC,CAAC;QAC1E,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CACvB,mCAAmC,KAAK,IAAI,IAAI,cAAc,CAC/D,CAAC;QACF,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,aAAa,EAAE,CAAC;QAClC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACpC,MAAM,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACpC,MAAM,CAAC,MAAM,EAAE,CAAC,MAAM,CAAC,WAAW,KAAK,IAAI,IAAI,cAAc,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC7E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mEAAmE,EAAE,KAAK,IAAI,EAAE;QACjF,MAAM,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,GAAG,QAAQ,EAAE,CAAC;QAC1C,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,KAAK,CAAC,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,EAAE,CAAC,CAAC;QACzE,MAAM,EAAE,OAAO,EAAE,EAAE,EAAE,GAAG,MAAM,MAAM,CAAC,kBAAkB,CAAC,CAAC;QACzD,MAAM,EAAE,MAAM,EAAE,GAAG,MAAM,MAAM,CAAC,SAAS,CAAC,CAAC;QAC3C,MAAM,EAAE,IAAI,EAAE,GAAG,MAAM,MAAM,CAAC,WAAW,CAAC,CAAC;QAC3C,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,WAAW,CAAC,CAAC,CAAC;QACvD,MAAM,OAAO,GAAG,IAAI,CAAC,GAAG,EAAE,YAAY,CAAC,CAAC;QACxC,MAAM,IAAI,GAAG,cAAc,CAAC;YAC1B,EAAE;YACF,KAAK;YACL,OAAO;YACP,UAAU;YACV,UAAU;YACV,YAAY,EAAE,IAAI;SACnB,CAAC,CAAC;QACH,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,EAAE,EAAE,OAAO,EAAE,CAAC,CAAC;QAC/D,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAGnE,CAAC;QACF,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAChC,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,aAAa,EAAE,CAAC;QACrC,MAAM,EAAE,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IACrC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,MAAM,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,GAAG,QAAQ,EAAE,CAAC;QAC1C,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,KAAK,CAAC,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,EAAE,CAAC,CAAC;QACzE,MAAM,IAAI,GAAG,cAAc,CAAC;YAC1B,EAAE;YACF,KAAK;YACL,OAAO;YACP,UAAU;YACV,UAAU;YACV,YAAY,EAAE,KAAK;SACpB,CAAC,CAAC;QACH,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC;YAC7B,QAAQ,EAAE,IAAI,CAAC,EAAE;YACjB,OAAO,EAAE,YAAY;SACtB,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClC,MAAM,CAAE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAAC,CAAC,OAAO,CAC1D,0BAA0B,CAC3B,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+FAA+F,EAAE,GAAG,EAAE;QACvG,MAAM,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,GAAG,QAAQ,EAAE,CAAC;QAC1C,MAAM,IAAI,GAAG,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,UAAU,EAAE,CAAC;QAC5D,MAAM,MAAM,GAAG,cAAc,CAAC,EAAE,GAAG,IAAI,EAAE,YAAY,EAAE,KAAK,EAAE,CAAC,CAAC;QAChE,MAAM,KAAK,GAAG,cAAc,CAAC,EAAE,GAAG,IAAI,EAAE,YAAY,EAAE,IAAI,EAAE,CAAC,CAAC;QAC9D,MAAM,KAAK,GAAG,CAAC,CAAU,EAAE,EAAE,CAC1B,CAA6C,CAAC,UAAU,CAAC;QAC5D,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,GAAG,CAAC,cAAc,CAAC,SAAS,CAAC,CAAC;QAChE,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC,CAAC,cAAc,CAAC,SAAS,CAAC,CAAC;IAC7D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iCAAiC,EAAE,KAAK,IAAI,EAAE;QAC/C,MAAM,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,GAAG,QAAQ,EAAE,CAAC;QAC1C,MAAM,IAAI,GAAG,cAAc,CAAC,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,UAAU,EAAE,CAAC,CAAC;QAC5E,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACnC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClC,MAAM,CAAE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,GAAG,QAAQ,EAAE,CAAC;QACtD,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,KAAK,CAAC,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,EAAE,CAAC,CAAC;QACzE,MAAM,IAAI,GAAG,cAAc,CAAC,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,UAAU,EAAE,CAAC,CAAC;QAC5E,MAAM,IAAI,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,CAAC,EAAE,EAAE,CAAC,CAAC;QAEvC,wDAAwD;QACxD,MAAM,CAAC,UAAU,CAAC,KAAK,CAAC,kBAAkB,CAAC,CAAC,oBAAoB,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;QACvE,+BAA+B;QAC/B,MAAM,SAAS,GAAG,CAAC,CAA2B,EAAU,EAAE,CACxD,CAAC,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC;QAChC,MAAM,CAAC,SAAS,CAAC,UAAU,CAAC,KAAK,CAAC,kBAAkB,CAAC,CAAC,CAAC,YAAY,CACjE,SAAS,CAAC,UAAU,CAAC,KAAK,CAAC,aAAa,CAAC,CAC1C,CAAC;QAEF,gEAAgE;QAChE,MAAM,KAAK,GAAG,IAAI,CAAC,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;QAClC,MAAM,IAAI,GAAG,IAAI,CAAC,EAAE,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;QAC9B,MAAM,GAAG,GAAG,MAAM,EAAE,CAAC,QAAQ,CAC3B,WAAW,KAAK,IAAI,IAAI,cAAc,EACtC,MAAM,CACP,CAAC;QACF,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;IACjC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["import { describe, expect, it, vi } from 'vitest';\nimport { Mutex } from '../../src/engine/mutex.js';\nimport { type EngineAdapter, EngineSession } from '../../src/engine/session.js';\nimport { RevisionStore } from '../../src/revisions/store.js';\nimport { createSaveTool } from '../../src/tools/save.js';\nimport { LocalViewerUrlBuilder } from '../../src/viewer/url-builder.js';\nimport { InMemoryWorkspaceFS } from '../../src/workspace/in-memory.js';\n\nconst engineLock = new Mutex();\nconst urlBuilder = new LocalViewerUrlBuilder({ base: 'http://127.0.0.1:9000' });\n\ninterface FakeEngine {\n  _scene: string;\n  _resourcesLoaded: boolean;\n  scene: {\n    getPages: ReturnType<typeof vi.fn>;\n    saveToArchive: ReturnType<typeof vi.fn>;\n  };\n  block: { forceLoadResources: ReturnType<typeof vi.fn> };\n}\n\nfunction makeDeps() {\n  const fs = new InMemoryWorkspaceFS();\n  const store = new RevisionStore(fs);\n  const fakeEngine: FakeEngine = {\n    _scene: '',\n    _resourcesLoaded: false,\n    scene: {\n      getPages: vi.fn(() => [10]),\n      // The archive bundles the asset marker ONLY if resources were loaded\n      // first — mirrors saveToArchive's \"assets accessible when called\" contract.\n      saveToArchive: vi.fn(async () => {\n        const body = fakeEngine._resourcesLoaded ? 'ZIP+ASSET' : 'ZIP';\n        return new Blob([new TextEncoder().encode(body)]);\n      })\n    },\n    block: {\n      forceLoadResources: vi.fn(async () => {\n        fakeEngine._resourcesLoaded = true;\n      })\n    }\n  };\n  const adapter: EngineAdapter<FakeEngine> = {\n    create: () => fakeEngine,\n    reset: async (e) => {\n      e._scene = 'EMPTY';\n    },\n    loadFromString: async (e, s) => {\n      e._scene = s;\n    },\n    saveToString: async (e) => e._scene\n  };\n  const session = new EngineSession<FakeEngine>({ size: 1, adapter });\n  return { fs, store, session, fakeEngine };\n}\n\ndescribe('save tool', () => {\n  it('writes archive.zip and returns uri + httpUrl, no path', async () => {\n    const { fs, store, session } = makeDeps();\n    const root = await store.write({\n      scene: 'SCENE_ABC',\n      parent: null,\n      title: 'T'\n    });\n    const tool = createSaveTool({ fs, store, session, engineLock, urlBuilder });\n    const result = await tool.call({ revision: root.id });\n    const body = JSON.parse((result.content[0] as { text: string }).text) as {\n      uri: string;\n      httpUrl?: string;\n      path?: string;\n      bytes: number;\n      format: string;\n      revision: string;\n    };\n    const shard = root.id.slice(0, 2);\n    const rest = root.id.slice(2);\n    expect(body.uri).toBe(`workspace://exports/${shard}/${rest}/archive.zip`);\n    expect(body.httpUrl).toBe(\n      `http://127.0.0.1:9000/r/exports/${shard}/${rest}/archive.zip`\n    );\n    expect(body.path).toBeUndefined();\n    expect(body.format).toBe('archive');\n    expect(body.revision).toBe(root.id);\n    expect(await fs.exists(`exports/${shard}/${rest}/archive.zip`)).toBe(true);\n  });\n\n  it('respects local `outPath` escape hatch — returns path, not httpUrl', async () => {\n    const { fs, store, session } = makeDeps();\n    const root = await store.write({ scene: 'S', parent: null, title: 'T' });\n    const { mkdtemp, rm } = await import('node:fs/promises');\n    const { tmpdir } = await import('node:os');\n    const { join } = await import('node:path');\n    const dir = await mkdtemp(join(tmpdir(), 'save-out-'));\n    const outPath = join(dir, 'design.zip');\n    const tool = createSaveTool({\n      fs,\n      store,\n      session,\n      engineLock,\n      urlBuilder,\n      allowOutPath: true\n    });\n    const result = await tool.call({ revision: root.id, outPath });\n    const body = JSON.parse((result.content[0] as { text: string }).text) as {\n      path?: string;\n      httpUrl?: string;\n    };\n    expect(body.path).toBe(outPath);\n    expect(body.httpUrl).toBeUndefined();\n    await rm(dir, { recursive: true });\n  });\n\n  it('rejects `outPath` when not allowed (Hosted/sandbox)', async () => {\n    const { fs, store, session } = makeDeps();\n    const root = await store.write({ scene: 'S', parent: null, title: 'T' });\n    const tool = createSaveTool({\n      fs,\n      store,\n      session,\n      engineLock,\n      urlBuilder,\n      allowOutPath: false\n    });\n    const result = await tool.call({\n      revision: root.id,\n      outPath: '/tmp/x.zip'\n    });\n    expect(result.isError).toBe(true);\n    expect((result.content[0] as { text: string }).text).toMatch(\n      /outPath.*not.*supported/i\n    );\n  });\n\n  it('omits `outPath` from the schema when not allowed (Hosted), advertises it when allowed (Local)', () => {\n    const { fs, store, session } = makeDeps();\n    const base = { fs, store, session, engineLock, urlBuilder };\n    const hosted = createSaveTool({ ...base, allowOutPath: false });\n    const local = createSaveTool({ ...base, allowOutPath: true });\n    const props = (s: unknown) =>\n      (s as { properties: Record<string, unknown> }).properties;\n    expect(props(hosted.inputSchema)).not.toHaveProperty('outPath');\n    expect(props(local.inputSchema)).toHaveProperty('outPath');\n  });\n\n  it('errors when revision is missing', async () => {\n    const { fs, store, session } = makeDeps();\n    const tool = createSaveTool({ fs, store, session, engineLock, urlBuilder });\n    const result = await tool.call({});\n    expect(result.isError).toBe(true);\n    expect((result.content[0] as { text: string }).text).toMatch(/revision/i);\n  });\n\n  it('force-loads resources before archiving so assets are bundled', async () => {\n    const { fs, store, session, fakeEngine } = makeDeps();\n    const root = await store.write({ scene: 'S', parent: null, title: 'T' });\n    const tool = createSaveTool({ fs, store, session, engineLock, urlBuilder });\n    await tool.call({ revision: root.id });\n\n    // forceLoadResources must run with the scene's pages...\n    expect(fakeEngine.block.forceLoadResources).toHaveBeenCalledWith([10]);\n    // ...and BEFORE saveToArchive.\n    const callOrder = (m: ReturnType<typeof vi.fn>): number =>\n      m.mock.invocationCallOrder[0];\n    expect(callOrder(fakeEngine.block.forceLoadResources)).toBeLessThan(\n      callOrder(fakeEngine.scene.saveToArchive)\n    );\n\n    // ...so the produced archive actually contains the asset bytes.\n    const shard = root.id.slice(0, 2);\n    const rest = root.id.slice(2);\n    const zip = await fs.readFile(\n      `exports/${shard}/${rest}/archive.zip`,\n      'utf8'\n    );\n    expect(zip).toContain('ASSET');\n  });\n});\n"]}

package/dist/test/test/tools/schema-best-practices.d.ts

@@ -1,13 +0,0 @@
-export type Severity = 'error' | 'warn';
-export interface SchemaFinding {
-    /** Human-readable location inside the schema, e.g. `properties.source`. */
-    path: string;
-    /** Stable rule id (see RULES below). */
-    rule: string;
-    severity: Severity;
-    message: string;
-}
-/** Lint one tool's input schema. Returns all findings (errors + warnings). */
-export declare function lintToolSchema(schema: unknown): SchemaFinding[];
-export declare const errorsOnly: (findings: SchemaFinding[]) => SchemaFinding[];
-//# sourceMappingURL=schema-best-practices.d.ts.map

package/dist/test/test/tools/schema-best-practices.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"schema-best-practices.d.ts","sourceRoot":"","sources":["../../../../test/tools/schema-best-practices.ts"],"names":[],"mappings":"AA2BA,MAAM,MAAM,QAAQ,GAAG,OAAO,GAAG,MAAM,CAAC;AAExC,MAAM,WAAW,aAAa;IAC5B,2EAA2E;IAC3E,IAAI,EAAE,MAAM,CAAC;IACb,wCAAwC;IACxC,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,QAAQ,CAAC;IACnB,OAAO,EAAE,MAAM,CAAC;CACjB;AA8JD,8EAA8E;AAC9E,wBAAgB,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,aAAa,EAAE,CAI/D;AAED,eAAO,MAAM,UAAU,GAAI,UAAU,aAAa,EAAE,KAAG,aAAa,EACpB,CAAC"}

package/dist/test/test/tools/schema-best-practices.js

@@ -1,169 +0,0 @@
-// packages/codesign-mcp/test/tools/schema-best-practices.ts
-//
-// A tiny JSON-Schema linter for MCP *tool input schemas*, encoding the
-// cross-vendor best practices for reliable LLM tool calling. It exists because
-// a real eval regression traced back to a schema shape, not a model bug:
-// `load`'s `source` used a `oneOf` union, and a weaker model
-// (openrouter:xiaomi/mimo-v2.5-pro) serialized the whole object as an escaped
-// JSON string on every attempt — the exact `oneOf` failure documented against
-// Claude too (samchon/openapi#88). OpenAI sidesteps it by banning `oneOf` in
-// strict mode outright.
-//
-// Sources for the rules below:
-//   - samchon/openapi#88 — Claude fills `oneOf` unions as a JSON string.
-//   - OpenAI structured outputs: `oneOf`/`allOf`/`$ref`(recursive) unsupported
-//     in strict mode; root must be `object`; `additionalProperties:false`
-//     required; only a subset of JSON Schema is honoured.
-//   - Cohere structured outputs: `anyOf`/`allOf`/`oneOf`/`dependencies` not
-//     enforced.
-//   - Vendor + practitioner guidance (Anyscale, agenta, Cohere, LangChain):
-//     "keep tool schemas as flat as possible; deep nesting lowers generation
-//     quality"; "invest in descriptions/enums — +10-20% accuracy".
-//
-// The linter is deliberately conservative: HARD ERRORS are the constructs
-// documented to break or degrade tool calls across providers; WARNINGS are
-// quality nudges that shouldn't fail CI. `schema-best-practices.test.ts`
-// asserts zero errors over every registered tool and prints warnings.
-// Object-nesting depth at which we start nudging toward a flatter schema.
-// Root object = depth 1; one nested object (e.g. `source`) = depth 2 (fine).
-// Depth 3+ is where the "keep it flat" guidance starts to bite, so we warn.
-const MAX_OBJECT_DEPTH_BEFORE_WARN = 2;
-// Composition / reference keywords that are unsupported or unenforced by at
-// least one major provider's strict tool-calling path — each is a hard error.
-const BANNED_COMPOSITION_KEYS = [
-    'oneOf', // OpenAI: banned in strict mode; Claude/mimo: stringify the value
-    'allOf', // OpenAI/Cohere: unsupported/unenforced
-    'not', // unsupported in strict schemas
-    '$ref', // no external/recursive refs in strict mode — inline instead
-    '$defs',
-    'definitions',
-    'prefixItems', // tuple validation unsupported
-    'dependencies',
-    'dependentSchemas',
-    'dependentRequired'
-];
-const SCALAR_TYPES = new Set([
-    'string',
-    'number',
-    'integer',
-    'boolean',
-    'null'
-]);
-function isRecord(value) {
-    return typeof value === 'object' && value !== null && !Array.isArray(value);
-}
-/** True when a schema node only ever produces a scalar (safe `anyOf` arm). */
-function isScalarSchema(node) {
-    if ('properties' in node || 'items' in node)
-        return false;
-    const t = node.type;
-    if (typeof t === 'string')
-        return SCALAR_TYPES.has(t);
-    if (Array.isArray(t))
-        return t.every((x) => SCALAR_TYPES.has(String(x)));
-    // No `type` at all → can't prove it's scalar; treat as complex.
-    return false;
-}
-/**
- * Lint a single JSON-Schema node (recursively). `path` is the location label,
- * `depth` counts object nesting, `isRoot` gates the "root must be object" rule.
- */
-function lintNode(node, path, depth, isRoot, out) {
-    if (!isRecord(node))
-        return;
-    // Root of a tool input schema MUST be an object — every tool-call API
-    // requires it, and a non-object root cannot carry named arguments.
-    if (isRoot && node.type !== 'object') {
-        out.push({
-            path,
-            rule: 'root-must-be-object',
-            severity: 'error',
-            message: `tool input schema root must be { type: "object" }, got ${JSON.stringify(node.type)}`
-        });
-    }
-    // Banned composition/reference keywords anywhere in the tree.
-    for (const key of BANNED_COMPOSITION_KEYS) {
-        if (key in node) {
-            out.push({
-                path: `${path}.${key}`,
-                rule: `no-${key.replace('$', '')}`,
-                severity: 'error',
-                message: `\`${key}\` is unsupported/unenforced by strict tool-calling on at least one major provider — flatten it (prefer sibling optional fields + runtime validation, or \`type: [...]\` for nullability).`
-            });
-        }
-    }
-    // `anyOf` is the one union OpenAI keeps in strict mode, but only scalar arms
-    // are safe. An object/array arm reintroduces the discriminated-union shape
-    // that models stringify — so allow scalar unions (incl. the `["string",
-    // "null"]` nullable idiom) and flag complex arms.
-    if (Array.isArray(node.anyOf)) {
-        node.anyOf.forEach((arm, i) => {
-            if (isRecord(arm) && !isScalarSchema(arm)) {
-                out.push({
-                    path: `${path}.anyOf[${i}]`,
-                    rule: 'anyof-scalar-arms-only',
-                    severity: 'error',
-                    message: 'anyOf arms must be scalar types; an object/array arm is a discriminated union that weaker models serialize as a JSON string. Flatten to sibling fields.'
-                });
-            }
-            lintNode(arm, `${path}.anyOf[${i}]`, depth, false, out);
-        });
-    }
-    if (node.type === 'object') {
-        // Strict tool-calling (OpenAI et al.) requires closed objects.
-        if (node.additionalProperties !== false) {
-            out.push({
-                path,
-                rule: 'object-additional-properties-false',
-                severity: 'error',
-                message: 'object schemas must set `additionalProperties: false` for strict-mode adherence.'
-            });
-        }
-        if (depth > MAX_OBJECT_DEPTH_BEFORE_WARN) {
-            out.push({
-                path,
-                rule: 'keep-schema-flat',
-                severity: 'warn',
-                message: `object nesting depth ${depth} — deep nesting lowers generation quality; keep tool schemas as flat as the domain allows.`
-            });
-        }
-        const props = node.properties;
-        if (isRecord(props)) {
-            for (const [name, sub] of Object.entries(props)) {
-                const subPath = `${path}.properties.${name}`;
-                // Descriptions measurably improve tool-call accuracy (+10-20%).
-                if (isRecord(sub) &&
-                    (typeof sub.description !== 'string' || sub.description.trim() === '')) {
-                    out.push({
-                        path: subPath,
-                        rule: 'property-needs-description',
-                        severity: 'warn',
-                        message: `property \`${name}\` has no description — add one (+accuracy).`
-                    });
-                }
-                lintNode(sub, subPath, depth + 1, false, out);
-            }
-        }
-    }
-    if (node.type === 'array') {
-        if (!isRecord(node.items)) {
-            out.push({
-                path,
-                rule: 'array-needs-typed-items',
-                severity: 'warn',
-                message: 'array schema has no typed `items` — models generate array elements more reliably with a concrete item schema.'
-            });
-        }
-        else {
-            lintNode(node.items, `${path}.items`, depth, false, out);
-        }
-    }
-}
-/** Lint one tool's input schema. Returns all findings (errors + warnings). */
-export function lintToolSchema(schema) {
-    const findings = [];
-    lintNode(schema, '$', 1, true, findings);
-    return findings;
-}
-export const errorsOnly = (findings) => findings.filter((f) => f.severity === 'error');
-//# sourceMappingURL=schema-best-practices.js.map

package/dist/test/test/tools/schema-best-practices.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"schema-best-practices.js","sourceRoot":"","sources":["../../../../test/tools/schema-best-practices.ts"],"names":[],"mappings":"AAAA,4DAA4D;AAC5D,EAAE;AACF,uEAAuE;AACvE,+EAA+E;AAC/E,yEAAyE;AACzE,6DAA6D;AAC7D,8EAA8E;AAC9E,8EAA8E;AAC9E,6EAA6E;AAC7E,wBAAwB;AACxB,EAAE;AACF,+BAA+B;AAC/B,yEAAyE;AACzE,+EAA+E;AAC/E,0EAA0E;AAC1E,0DAA0D;AAC1D,4EAA4E;AAC5E,gBAAgB;AAChB,4EAA4E;AAC5E,6EAA6E;AAC7E,mEAAmE;AACnE,EAAE;AACF,0EAA0E;AAC1E,2EAA2E;AAC3E,yEAAyE;AACzE,sEAAsE;AAatE,0EAA0E;AAC1E,6EAA6E;AAC7E,4EAA4E;AAC5E,MAAM,4BAA4B,GAAG,CAAC,CAAC;AAEvC,4EAA4E;AAC5E,8EAA8E;AAC9E,MAAM,uBAAuB,GAAG;IAC9B,OAAO,EAAE,kEAAkE;IAC3E,OAAO,EAAE,wCAAwC;IACjD,KAAK,EAAE,gCAAgC;IACvC,MAAM,EAAE,6DAA6D;IACrE,OAAO;IACP,aAAa;IACb,aAAa,EAAE,+BAA+B;IAC9C,cAAc;IACd,kBAAkB;IAClB,mBAAmB;CACX,CAAC;AAEX,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC;IAC3B,QAAQ;IACR,QAAQ;IACR,SAAS;IACT,SAAS;IACT,MAAM;CACP,CAAC,CAAC;AAEH,SAAS,QAAQ,CAAC,KAAc;IAC9B,OAAO,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,KAAK,IAAI,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC;AAC9E,CAAC;AAED,8EAA8E;AAC9E,SAAS,cAAc,CAAC,IAA6B;IACnD,IAAI,YAAY,IAAI,IAAI,IAAI,OAAO,IAAI,IAAI;QAAE,OAAO,KAAK,CAAC;IAC1D,MAAM,CAAC,GAAG,IAAI,CAAC,IAAI,CAAC;IACpB,IAAI,OAAO,CAAC,KAAK,QAAQ;QAAE,OAAO,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;IACtD,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC;QAAE,OAAO,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IACzE,gEAAgE;IAChE,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;GAGG;AACH,SAAS,QAAQ,CACf,IAAa,EACb,IAAY,EACZ,KAAa,EACb,MAAe,EACf,GAAoB;IAEpB,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC;QAAE,OAAO;IAE5B,sEAAsE;IACtE,mEAAmE;IACnE,IAAI,MAAM,IAAI,IAAI,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;QACrC,GAAG,CAAC,IAAI,CAAC;YACP,IAAI;YACJ,IAAI,EAAE,qBAAqB;YAC3B,QAAQ,EAAE,OAAO;YACjB,OAAO,EAAE,0DAA0D,IAAI,CAAC,SAAS,CAC/E,IAAI,CAAC,IAAI,CACV,EAAE;SACJ,CAAC,CAAC;IACL,CAAC;IAED,8DAA8D;IAC9D,KAAK,MAAM,GAAG,IAAI,uBAAuB,EAAE,CAAC;QAC1C,IAAI,GAAG,IAAI,IAAI,EAAE,CAAC;YAChB,GAAG,CAAC,IAAI,CAAC;gBACP,IAAI,EAAE,GAAG,IAAI,IAAI,GAAG,EAAE;gBACtB,IAAI,EAAE,MAAM,GAAG,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,CAAC,EAAE;gBAClC,QAAQ,EAAE,OAAO;gBACjB,OAAO,EAAE,KAAK,GAAG,4LAA4L;aAC9M,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,6EAA6E;IAC7E,2EAA2E;IAC3E,wEAAwE;IACxE,kDAAkD;IAClD,IAAI,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC;QAC9B,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE;YAC5B,IAAI,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,EAAE,CAAC;gBAC1C,GAAG,CAAC,IAAI,CAAC;oBACP,IAAI,EAAE,GAAG,IAAI,UAAU,CAAC,GAAG;oBAC3B,IAAI,EAAE,wBAAwB;oBAC9B,QAAQ,EAAE,OAAO;oBACjB,OAAO,EACL,yJAAyJ;iBAC5J,CAAC,CAAC;YACL,CAAC;YACD,QAAQ,CAAC,GAAG,EAAE,GAAG,IAAI,UAAU,CAAC,GAAG,EAAE,KAAK,EAAE,KAAK,EAAE,GAAG,CAAC,CAAC;QAC1D,CAAC,CAAC,CAAC;IACL,CAAC;IAED,IAAI,IAAI,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;QAC3B,+DAA+D;QAC/D,IAAI,IAAI,CAAC,oBAAoB,KAAK,KAAK,EAAE,CAAC;YACxC,GAAG,CAAC,IAAI,CAAC;gBACP,IAAI;gBACJ,IAAI,EAAE,oCAAoC;gBAC1C,QAAQ,EAAE,OAAO;gBACjB,OAAO,EACL,kFAAkF;aACrF,CAAC,CAAC;QACL,CAAC;QAED,IAAI,KAAK,GAAG,4BAA4B,EAAE,CAAC;YACzC,GAAG,CAAC,IAAI,CAAC;gBACP,IAAI;gBACJ,IAAI,EAAE,kBAAkB;gBACxB,QAAQ,EAAE,MAAM;gBAChB,OAAO,EAAE,wBAAwB,KAAK,4FAA4F;aACnI,CAAC,CAAC;QACL,CAAC;QAED,MAAM,KAAK,GAAG,IAAI,CAAC,UAAU,CAAC;QAC9B,IAAI,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;YACpB,KAAK,MAAM,CAAC,IAAI,EAAE,GAAG,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;gBAChD,MAAM,OAAO,GAAG,GAAG,IAAI,eAAe,IAAI,EAAE,CAAC;gBAC7C,gEAAgE;gBAChE,IACE,QAAQ,CAAC,GAAG,CAAC;oBACb,CAAC,OAAO,GAAG,CAAC,WAAW,KAAK,QAAQ,IAAI,GAAG,CAAC,WAAW,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,EACtE,CAAC;oBACD,GAAG,CAAC,IAAI,CAAC;wBACP,IAAI,EAAE,OAAO;wBACb,IAAI,EAAE,4BAA4B;wBAClC,QAAQ,EAAE,MAAM;wBAChB,OAAO,EAAE,cAAc,IAAI,8CAA8C;qBAC1E,CAAC,CAAC;gBACL,CAAC;gBACD,QAAQ,CAAC,GAAG,EAAE,OAAO,EAAE,KAAK,GAAG,CAAC,EAAE,KAAK,EAAE,GAAG,CAAC,CAAC;YAChD,CAAC;QACH,CAAC;IACH,CAAC;IAED,IAAI,IAAI,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;QAC1B,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC;YAC1B,GAAG,CAAC,IAAI,CAAC;gBACP,IAAI;gBACJ,IAAI,EAAE,yBAAyB;gBAC/B,QAAQ,EAAE,MAAM;gBAChB,OAAO,EACL,+GAA+G;aAClH,CAAC,CAAC;QACL,CAAC;aAAM,CAAC;YACN,QAAQ,CAAC,IAAI,CAAC,KAAK,EAAE,GAAG,IAAI,QAAQ,EAAE,KAAK,EAAE,KAAK,EAAE,GAAG,CAAC,CAAC;QAC3D,CAAC;IACH,CAAC;AACH,CAAC;AAED,8EAA8E;AAC9E,MAAM,UAAU,cAAc,CAAC,MAAe;IAC5C,MAAM,QAAQ,GAAoB,EAAE,CAAC;IACrC,QAAQ,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,QAAQ,CAAC,CAAC;IACzC,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,MAAM,CAAC,MAAM,UAAU,GAAG,CAAC,QAAyB,EAAmB,EAAE,CACvE,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAC","sourcesContent":["// packages/codesign-mcp/test/tools/schema-best-practices.ts\n//\n// A tiny JSON-Schema linter for MCP *tool input schemas*, encoding the\n// cross-vendor best practices for reliable LLM tool calling. It exists because\n// a real eval regression traced back to a schema shape, not a model bug:\n// `load`'s `source` used a `oneOf` union, and a weaker model\n// (openrouter:xiaomi/mimo-v2.5-pro) serialized the whole object as an escaped\n// JSON string on every attempt — the exact `oneOf` failure documented against\n// Claude too (samchon/openapi#88). OpenAI sidesteps it by banning `oneOf` in\n// strict mode outright.\n//\n// Sources for the rules below:\n//   - samchon/openapi#88 — Claude fills `oneOf` unions as a JSON string.\n//   - OpenAI structured outputs: `oneOf`/`allOf`/`$ref`(recursive) unsupported\n//     in strict mode; root must be `object`; `additionalProperties:false`\n//     required; only a subset of JSON Schema is honoured.\n//   - Cohere structured outputs: `anyOf`/`allOf`/`oneOf`/`dependencies` not\n//     enforced.\n//   - Vendor + practitioner guidance (Anyscale, agenta, Cohere, LangChain):\n//     \"keep tool schemas as flat as possible; deep nesting lowers generation\n//     quality\"; \"invest in descriptions/enums — +10-20% accuracy\".\n//\n// The linter is deliberately conservative: HARD ERRORS are the constructs\n// documented to break or degrade tool calls across providers; WARNINGS are\n// quality nudges that shouldn't fail CI. `schema-best-practices.test.ts`\n// asserts zero errors over every registered tool and prints warnings.\n\nexport type Severity = 'error' | 'warn';\n\nexport interface SchemaFinding {\n  /** Human-readable location inside the schema, e.g. `properties.source`. */\n  path: string;\n  /** Stable rule id (see RULES below). */\n  rule: string;\n  severity: Severity;\n  message: string;\n}\n\n// Object-nesting depth at which we start nudging toward a flatter schema.\n// Root object = depth 1; one nested object (e.g. `source`) = depth 2 (fine).\n// Depth 3+ is where the \"keep it flat\" guidance starts to bite, so we warn.\nconst MAX_OBJECT_DEPTH_BEFORE_WARN = 2;\n\n// Composition / reference keywords that are unsupported or unenforced by at\n// least one major provider's strict tool-calling path — each is a hard error.\nconst BANNED_COMPOSITION_KEYS = [\n  'oneOf', // OpenAI: banned in strict mode; Claude/mimo: stringify the value\n  'allOf', // OpenAI/Cohere: unsupported/unenforced\n  'not', // unsupported in strict schemas\n  '$ref', // no external/recursive refs in strict mode — inline instead\n  '$defs',\n  'definitions',\n  'prefixItems', // tuple validation unsupported\n  'dependencies',\n  'dependentSchemas',\n  'dependentRequired'\n] as const;\n\nconst SCALAR_TYPES = new Set([\n  'string',\n  'number',\n  'integer',\n  'boolean',\n  'null'\n]);\n\nfunction isRecord(value: unknown): value is Record<string, unknown> {\n  return typeof value === 'object' && value !== null && !Array.isArray(value);\n}\n\n/** True when a schema node only ever produces a scalar (safe `anyOf` arm). */\nfunction isScalarSchema(node: Record<string, unknown>): boolean {\n  if ('properties' in node || 'items' in node) return false;\n  const t = node.type;\n  if (typeof t === 'string') return SCALAR_TYPES.has(t);\n  if (Array.isArray(t)) return t.every((x) => SCALAR_TYPES.has(String(x)));\n  // No `type` at all → can't prove it's scalar; treat as complex.\n  return false;\n}\n\n/**\n * Lint a single JSON-Schema node (recursively). `path` is the location label,\n * `depth` counts object nesting, `isRoot` gates the \"root must be object\" rule.\n */\nfunction lintNode(\n  node: unknown,\n  path: string,\n  depth: number,\n  isRoot: boolean,\n  out: SchemaFinding[]\n): void {\n  if (!isRecord(node)) return;\n\n  // Root of a tool input schema MUST be an object — every tool-call API\n  // requires it, and a non-object root cannot carry named arguments.\n  if (isRoot && node.type !== 'object') {\n    out.push({\n      path,\n      rule: 'root-must-be-object',\n      severity: 'error',\n      message: `tool input schema root must be { type: \"object\" }, got ${JSON.stringify(\n        node.type\n      )}`\n    });\n  }\n\n  // Banned composition/reference keywords anywhere in the tree.\n  for (const key of BANNED_COMPOSITION_KEYS) {\n    if (key in node) {\n      out.push({\n        path: `${path}.${key}`,\n        rule: `no-${key.replace('$', '')}`,\n        severity: 'error',\n        message: `\\`${key}\\` is unsupported/unenforced by strict tool-calling on at least one major provider — flatten it (prefer sibling optional fields + runtime validation, or \\`type: [...]\\` for nullability).`\n      });\n    }\n  }\n\n  // `anyOf` is the one union OpenAI keeps in strict mode, but only scalar arms\n  // are safe. An object/array arm reintroduces the discriminated-union shape\n  // that models stringify — so allow scalar unions (incl. the `[\"string\",\n  // \"null\"]` nullable idiom) and flag complex arms.\n  if (Array.isArray(node.anyOf)) {\n    node.anyOf.forEach((arm, i) => {\n      if (isRecord(arm) && !isScalarSchema(arm)) {\n        out.push({\n          path: `${path}.anyOf[${i}]`,\n          rule: 'anyof-scalar-arms-only',\n          severity: 'error',\n          message:\n            'anyOf arms must be scalar types; an object/array arm is a discriminated union that weaker models serialize as a JSON string. Flatten to sibling fields.'\n        });\n      }\n      lintNode(arm, `${path}.anyOf[${i}]`, depth, false, out);\n    });\n  }\n\n  if (node.type === 'object') {\n    // Strict tool-calling (OpenAI et al.) requires closed objects.\n    if (node.additionalProperties !== false) {\n      out.push({\n        path,\n        rule: 'object-additional-properties-false',\n        severity: 'error',\n        message:\n          'object schemas must set `additionalProperties: false` for strict-mode adherence.'\n      });\n    }\n\n    if (depth > MAX_OBJECT_DEPTH_BEFORE_WARN) {\n      out.push({\n        path,\n        rule: 'keep-schema-flat',\n        severity: 'warn',\n        message: `object nesting depth ${depth} — deep nesting lowers generation quality; keep tool schemas as flat as the domain allows.`\n      });\n    }\n\n    const props = node.properties;\n    if (isRecord(props)) {\n      for (const [name, sub] of Object.entries(props)) {\n        const subPath = `${path}.properties.${name}`;\n        // Descriptions measurably improve tool-call accuracy (+10-20%).\n        if (\n          isRecord(sub) &&\n          (typeof sub.description !== 'string' || sub.description.trim() === '')\n        ) {\n          out.push({\n            path: subPath,\n            rule: 'property-needs-description',\n            severity: 'warn',\n            message: `property \\`${name}\\` has no description — add one (+accuracy).`\n          });\n        }\n        lintNode(sub, subPath, depth + 1, false, out);\n      }\n    }\n  }\n\n  if (node.type === 'array') {\n    if (!isRecord(node.items)) {\n      out.push({\n        path,\n        rule: 'array-needs-typed-items',\n        severity: 'warn',\n        message:\n          'array schema has no typed `items` — models generate array elements more reliably with a concrete item schema.'\n      });\n    } else {\n      lintNode(node.items, `${path}.items`, depth, false, out);\n    }\n  }\n}\n\n/** Lint one tool's input schema. Returns all findings (errors + warnings). */\nexport function lintToolSchema(schema: unknown): SchemaFinding[] {\n  const findings: SchemaFinding[] = [];\n  lintNode(schema, '$', 1, true, findings);\n  return findings;\n}\n\nexport const errorsOnly = (findings: SchemaFinding[]): SchemaFinding[] =>\n  findings.filter((f) => f.severity === 'error');\n"]}

package/dist/test/test/tools/schema-best-practices.test.d.ts

@@ -1,3 +0,0 @@
-import { type SchemaFinding } from './schema-best-practices.js';
-export declare function collectSchemaErrors(): SchemaFinding[];
-//# sourceMappingURL=schema-best-practices.test.d.ts.map

package/dist/test/test/tools/schema-best-practices.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"schema-best-practices.test.d.ts","sourceRoot":"","sources":["../../../../test/tools/schema-best-practices.test.ts"],"names":[],"mappings":"AAeA,OAAO,EAGL,KAAK,aAAa,EACnB,MAAM,4BAA4B,CAAC;AAgNpC,wBAAgB,mBAAmB,IAAI,aAAa,EAAE,CAIrD"}