@codeyam/codeyam-cli 0.1.0-staging.73a4bf4 → 0.1.0-staging.78c1cad

package/codeyam-cli/src/webserver/__tests__/api.interactive-switch-scenario.test.js

@@ -0,0 +1,98 @@
+import { action } from "../app/routes/api.interactive-switch-scenario.js";
+import fs from 'fs';
+import path from 'path';
+import os from 'os';
+// Mock dependencies
+jest.mock('../../state', () => ({
+    getProjectRoot: jest.fn(),
+}));
+jest.mock('../../utils/editorSeedAdapter', () => ({
+    runSeedAdapter: jest.fn(),
+    detectSeedAdapter: jest.fn(),
+}));
+jest.mock('../editorProxy', () => ({
+    invalidateScenarioCache: jest.fn(),
+}));
+import { getProjectRoot } from "../../state.js";
+import { runSeedAdapter, detectSeedAdapter, } from "../../utils/editorSeedAdapter.js";
+import { invalidateScenarioCache } from "../editorProxy.js";
+const mockGetProjectRoot = getProjectRoot;
+const mockRunSeedAdapter = runSeedAdapter;
+const mockDetectSeedAdapter = detectSeedAdapter;
+const mockInvalidateScenarioCache = invalidateScenarioCache;
+const createActionArgs = (request) => ({
+    request,
+    params: {},
+    context: {},
+    unstable_pattern: '',
+});
+describe('api.interactive-switch-scenario', () => {
+    let tempDir;
+    beforeEach(() => {
+        jest.clearAllMocks();
+        tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'interactive-switch-test-'));
+        mockGetProjectRoot.mockReturnValue(tempDir);
+        // Create .codeyam directory
+        fs.mkdirSync(path.join(tempDir, '.codeyam', 'editor-scenarios'), {
+            recursive: true,
+        });
+    });
+    afterEach(() => {
+        fs.rmSync(tempDir, { recursive: true, force: true });
+    });
+    it('should always seed even when same scenario is already in active-scenario.json', async () => {
+        const scenarioId = 'test-scenario-123';
+        // Write active-scenario.json with the SAME scenarioId (simulating stale state)
+        fs.writeFileSync(path.join(tempDir, '.codeyam', 'active-scenario.json'), JSON.stringify({ scenarioId, scenarioSlug: 'Test', type: 'application' }));
+        // Write scenario data file with application type
+        fs.writeFileSync(path.join(tempDir, '.codeyam', 'editor-scenarios', `${scenarioId}.json`), JSON.stringify({ type: 'application', seed: { flight: [] } }));
+        // Write seed data file
+        fs.writeFileSync(path.join(tempDir, '.codeyam', 'editor-scenarios', `${scenarioId}.seed.json`), JSON.stringify({ flight: [{ id: 1, name: 'test' }] }));
+        // Set up seed adapter
+        const adapterPath = path.join(tempDir, '.codeyam', 'seed-adapter.ts');
+        fs.writeFileSync(adapterPath, '');
+        mockDetectSeedAdapter.mockReturnValue(adapterPath);
+        mockRunSeedAdapter.mockResolvedValue({
+            success: true,
+            output: 'Seed complete',
+            durationMs: 100,
+        });
+        const request = new Request('http://localhost/api/interactive-switch-scenario', {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify({ scenarioId, scenarioName: 'Test Scenario' }),
+        });
+        const response = await action(createActionArgs(request));
+        const body = await response.json();
+        expect(body.success).toBe(true);
+        expect(body.seeded).toBe(true);
+        // The key assertion: seed adapter was called even though same scenarioId was in active-scenario.json
+        expect(mockRunSeedAdapter).toHaveBeenCalledTimes(1);
+    });
+    it('should invalidate proxy cache after switching', async () => {
+        const scenarioId = 'test-scenario-456';
+        fs.writeFileSync(path.join(tempDir, '.codeyam', 'editor-scenarios', `${scenarioId}.json`), JSON.stringify({ type: 'component' }));
+        const request = new Request('http://localhost/api/interactive-switch-scenario', {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify({ scenarioId }),
+        });
+        await action(createActionArgs(request));
+        expect(mockInvalidateScenarioCache).toHaveBeenCalledTimes(1);
+    });
+    it('should return 400 when scenarioId is missing', async () => {
+        const request = new Request('http://localhost/api/interactive-switch-scenario', {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify({}),
+        });
+        const response = await action(createActionArgs(request));
+        expect(response.status).toBe(400);
+    });
+    it('should return 405 for non-POST methods', async () => {
+        const request = new Request('http://localhost/api/interactive-switch-scenario', { method: 'GET' });
+        const response = await action(createActionArgs(request));
+        expect(response.status).toBe(405);
+    });
+});
+//# sourceMappingURL=api.interactive-switch-scenario.test.js.map

package/codeyam-cli/src/webserver/__tests__/api.interactive-switch-scenario.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"api.interactive-switch-scenario.test.js","sourceRoot":"","sources":["../../../../../src/webserver/__tests__/api.interactive-switch-scenario.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,EAAE,MAAM,+CAA+C,CAAC;AACvE,OAAO,EAAE,MAAM,IAAI,CAAC;AACpB,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,EAAE,MAAM,IAAI,CAAC;AAEpB,oBAAoB;AACpB,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,GAAG,EAAE,CAAC,CAAC;IAC9B,cAAc,EAAE,IAAI,CAAC,EAAE,EAAE;CAC1B,CAAC,CAAC,CAAC;AAEJ,IAAI,CAAC,IAAI,CAAC,+BAA+B,EAAE,GAAG,EAAE,CAAC,CAAC;IAChD,cAAc,EAAE,IAAI,CAAC,EAAE,EAAE;IACzB,iBAAiB,EAAE,IAAI,CAAC,EAAE,EAAE;CAC7B,CAAC,CAAC,CAAC;AAEJ,IAAI,CAAC,IAAI,CAAC,gBAAgB,EAAE,GAAG,EAAE,CAAC,CAAC;IACjC,uBAAuB,EAAE,IAAI,CAAC,EAAE,EAAE;CACnC,CAAC,CAAC,CAAC;AAEJ,OAAO,EAAE,cAAc,EAAE,MAAM,aAAa,CAAC;AAC7C,OAAO,EACL,cAAc,EACd,iBAAiB,GAClB,MAAM,+BAA+B,CAAC;AACvC,OAAO,EAAE,uBAAuB,EAAE,MAAM,gBAAgB,CAAC;AAEzD,MAAM,kBAAkB,GAAG,cAE1B,CAAC;AACF,MAAM,kBAAkB,GAAG,cAE1B,CAAC;AACF,MAAM,qBAAqB,GAAG,iBAE7B,CAAC;AACF,MAAM,2BAA2B,GAC/B,uBAEC,CAAC;AAEJ,MAAM,gBAAgB,GAAG,CAAC,OAAgB,EAAE,EAAE,CAAC,CAAC;IAC9C,OAAO;IACP,MAAM,EAAE,EAAE;IACV,OAAO,EAAE,EAAE;IACX,gBAAgB,EAAE,EAAE;CACrB,CAAC,CAAC;AAEH,QAAQ,CAAC,iCAAiC,EAAE,GAAG,EAAE;IAC/C,IAAI,OAAe,CAAC;IAEpB,UAAU,CAAC,GAAG,EAAE;QACd,IAAI,CAAC,aAAa,EAAE,CAAC;QACrB,OAAO,GAAG,EAAE,CAAC,WAAW,CACtB,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,0BAA0B,CAAC,CACnD,CAAC;QACF,kBAAkB,CAAC,eAAe,CAAC,OAAO,CAAC,CAAC;QAE5C,4BAA4B;QAC5B,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,kBAAkB,CAAC,EAAE;YAC/D,SAAS,EAAE,IAAI;SAChB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,EAAE,CAAC,MAAM,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+EAA+E,EAAE,KAAK,IAAI,EAAE;QAC7F,MAAM,UAAU,GAAG,mBAAmB,CAAC;QAEvC,+EAA+E;QAC/E,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,sBAAsB,CAAC,EACtD,IAAI,CAAC,SAAS,CAAC,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,EAAE,IAAI,EAAE,aAAa,EAAE,CAAC,CAC1E,CAAC;QAEF,iDAAiD;QACjD,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,kBAAkB,EAAE,GAAG,UAAU,OAAO,CAAC,EACxE,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,aAAa,EAAE,IAAI,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,CAAC,CAC9D,CAAC;QAEF,uBAAuB;QACvB,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CACP,OAAO,EACP,UAAU,EACV,kBAAkB,EAClB,GAAG,UAAU,YAAY,CAC1B,EACD,IAAI,CAAC,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,EAAE,CAAC,CACtD,CAAC;QAEF,sBAAsB;QACtB,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,iBAAiB,CAAC,CAAC;QACtE,EAAE,CAAC,aAAa,CAAC,WAAW,EAAE,EAAE,CAAC,CAAC;QAClC,qBAAqB,CAAC,eAAe,CAAC,WAAW,CAAC,CAAC;QACnD,kBAAkB,CAAC,iBAAiB,CAAC;YACnC,OAAO,EAAE,IAAI;YACb,MAAM,EAAE,eAAe;YACvB,UAAU,EAAE,GAAG;SAChB,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,IAAI,OAAO,CACzB,kDAAkD,EAClD;YACE,MAAM,EAAE,MAAM;YACd,OAAO,EAAE,EAAE,cAAc,EAAE,kBAAkB,EAAE;YAC/C,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,UAAU,EAAE,YAAY,EAAE,eAAe,EAAE,CAAC;SACpE,CACF,CAAC;QAEF,MAAM,QAAQ,GAAG,MAAM,MAAM,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC,CAAC;QACzD,MAAM,IAAI,GAAG,MAAM,QAAQ,CAAC,IAAI,EAAE,CAAC;QAEnC,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAChC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC/B,qGAAqG;QACrG,MAAM,CAAC,kBAAkB,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,MAAM,UAAU,GAAG,mBAAmB,CAAC;QAEvC,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,kBAAkB,EAAE,GAAG,UAAU,OAAO,CAAC,EACxE,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,CAAC,CACtC,CAAC;QAEF,MAAM,OAAO,GAAG,IAAI,OAAO,CACzB,kDAAkD,EAClD;YACE,MAAM,EAAE,MAAM;YACd,OAAO,EAAE,EAAE,cAAc,EAAE,kBAAkB,EAAE;YAC/C,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,UAAU,EAAE,CAAC;SACrC,CACF,CAAC;QAEF,MAAM,MAAM,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC,CAAC;QAExC,MAAM,CAAC,2BAA2B,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAC/D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,MAAM,OAAO,GAAG,IAAI,OAAO,CACzB,kDAAkD,EAClD;YACE,MAAM,EAAE,MAAM;YACd,OAAO,EAAE,EAAE,cAAc,EAAE,kBAAkB,EAAE;YAC/C,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC;SACzB,CACF,CAAC;QAEF,MAAM,QAAQ,GAAG,MAAM,MAAM,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC,CAAC;QACzD,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;QACtD,MAAM,OAAO,GAAG,IAAI,OAAO,CACzB,kDAAkD,EAClD,EAAE,MAAM,EAAE,KAAK,EAAE,CAClB,CAAC;QAEF,MAAM,QAAQ,GAAG,MAAM,MAAM,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC,CAAC;QACzD,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/codeyam-cli/src/webserver/__tests__/buildPtyEnv.test.js

@@ -0,0 +1,35 @@
+import { buildPtyEnv } from "../terminalServer.js";
+describe('buildPtyEnv', () => {
+    const originalEnv = process.env;
+    beforeEach(() => {
+        process.env = { ...originalEnv };
+    });
+    afterEach(() => {
+        process.env = originalEnv;
+    });
+    test('sets CODEYAM_EDITOR_ACTIVE=1 when editorMode is true', () => {
+        const env = buildPtyEnv({ editorMode: true });
+        expect(env.CODEYAM_EDITOR_ACTIVE).toBe('1');
+    });
+    test('does not set CODEYAM_EDITOR_ACTIVE when editorMode is false', () => {
+        const env = buildPtyEnv({ editorMode: false });
+        expect(env.CODEYAM_EDITOR_ACTIVE).toBeUndefined();
+    });
+    test('does not set CODEYAM_EDITOR_ACTIVE when no options provided', () => {
+        const env = buildPtyEnv();
+        expect(env.CODEYAM_EDITOR_ACTIVE).toBeUndefined();
+    });
+    test('passes through PATH and HOME from process.env', () => {
+        process.env.PATH = '/usr/bin:/usr/local/bin';
+        process.env.HOME = '/Users/testuser';
+        const env = buildPtyEnv();
+        expect(env.PATH).toBe('/usr/bin:/usr/local/bin');
+        expect(env.HOME).toBe('/Users/testuser');
+    });
+    test('sets TERM to xterm-256color by default', () => {
+        delete process.env.TERM;
+        const env = buildPtyEnv();
+        expect(env.TERM).toBe('xterm-256color');
+    });
+});
+//# sourceMappingURL=buildPtyEnv.test.js.map

package/codeyam-cli/src/webserver/__tests__/buildPtyEnv.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"buildPtyEnv.test.js","sourceRoot":"","sources":["../../../../../src/webserver/__tests__/buildPtyEnv.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAEhD,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;IAC3B,MAAM,WAAW,GAAG,OAAO,CAAC,GAAG,CAAC;IAEhC,UAAU,CAAC,GAAG,EAAE;QACd,OAAO,CAAC,GAAG,GAAG,EAAE,GAAG,WAAW,EAAE,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,OAAO,CAAC,GAAG,GAAG,WAAW,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAChE,MAAM,GAAG,GAAG,WAAW,CAAC,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC,CAAC;QAC9C,MAAM,CAAC,GAAG,CAAC,qBAAqB,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACvE,MAAM,GAAG,GAAG,WAAW,CAAC,EAAE,UAAU,EAAE,KAAK,EAAE,CAAC,CAAC;QAC/C,MAAM,CAAC,GAAG,CAAC,qBAAqB,CAAC,CAAC,aAAa,EAAE,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACvE,MAAM,GAAG,GAAG,WAAW,EAAE,CAAC;QAC1B,MAAM,CAAC,GAAG,CAAC,qBAAqB,CAAC,CAAC,aAAa,EAAE,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACzD,OAAO,CAAC,GAAG,CAAC,IAAI,GAAG,yBAAyB,CAAC;QAC7C,OAAO,CAAC,GAAG,CAAC,IAAI,GAAG,iBAAiB,CAAC;QACrC,MAAM,GAAG,GAAG,WAAW,EAAE,CAAC;QAC1B,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,yBAAyB,CAAC,CAAC;QACjD,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;IAC3C,CAAC,CAAC,CAAC;IAEH,IAAI,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAClD,OAAO,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC;QACxB,MAAM,GAAG,GAAG,WAAW,EAAE,CAAC;QAC1B,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/codeyam-cli/src/webserver/__tests__/clientErrors.test.js

@@ -0,0 +1,107 @@
+import { parseClientErrors, parseVisibleText } from "../app/lib/clientErrors.js";
+describe('parseClientErrors', () => {
+    it('extracts console.error lines', () => {
+        const output = `[JournalCapture] Page console.error: Uncaught TypeError: x is not a function`;
+        expect(parseClientErrors(output)).toEqual([
+            'Uncaught TypeError: x is not a function',
+        ]);
+    });
+    it('extracts network failure lines', () => {
+        const output = `[JournalCapture] Network failed: GET http://localhost:3113/api/products — net::ERR_CONNECTION_REFUSED`;
+        expect(parseClientErrors(output)).toEqual([
+            'Network failed: GET http://localhost:3113/api/products — net::ERR_CONNECTION_REFUSED',
+        ]);
+    });
+    it('filters out /api/health network failures', () => {
+        const output = [
+            `[JournalCapture] Network failed: GET http://127.0.0.1:3111/api/health — net::ERR_CONNECTION_REFUSED`,
+            `[JournalCapture] Network failed: GET http://localhost:3113/api/products — net::ERR_CONNECTION_REFUSED`,
+            `[JournalCapture] Page console.error: Something broke`,
+        ].join('\n');
+        expect(parseClientErrors(output)).toEqual([
+            'Network failed: GET http://localhost:3113/api/products — net::ERR_CONNECTION_REFUSED',
+            'Something broke',
+        ]);
+    });
+    it('filters out /__codeyam__/preview-health network failures', () => {
+        const output = [
+            `[JournalCapture] Network failed: POST http://localhost:3112/__codeyam__/preview-health — net::ERR_ABORTED`,
+            `[JournalCapture] Network failed: GET http://localhost:3113/api/products — net::ERR_CONNECTION_REFUSED`,
+        ].join('\n');
+        expect(parseClientErrors(output)).toEqual([
+            'Network failed: GET http://localhost:3113/api/products — net::ERR_CONNECTION_REFUSED',
+        ]);
+    });
+    it('returns empty array for clean output', () => {
+        const output = `[JournalCapture] Screenshot saved\n[JournalCapture] Done`;
+        expect(parseClientErrors(output)).toEqual([]);
+    });
+    it('extracts HTTP error status lines (e.g. 404)', () => {
+        const output = `[JournalCapture] HTTP error: status=404, url=http://localhost:3112/library/collections/col-1`;
+        expect(parseClientErrors(output)).toEqual([
+            'HTTP error: status=404, url=http://localhost:3112/library/collections/col-1',
+        ]);
+    });
+    it('extracts HTTP 500 error status lines', () => {
+        const output = `[JournalCapture] HTTP error: status=500, url=http://localhost:3112/api/crash`;
+        expect(parseClientErrors(output)).toEqual([
+            'HTTP error: status=500, url=http://localhost:3112/api/crash',
+        ]);
+    });
+    it('collects HTTP errors alongside console errors', () => {
+        const output = [
+            `[JournalCapture] HTTP error: status=404, url=http://localhost:3112/missing`,
+            `[JournalCapture] Page console.error: Cannot read properties of null`,
+        ].join('\n');
+        expect(parseClientErrors(output)).toEqual([
+            'HTTP error: status=404, url=http://localhost:3112/missing',
+            'Cannot read properties of null',
+        ]);
+    });
+    it('extracts API response error lines with response body', () => {
+        const output = `[JournalCapture] API response error: GET http://localhost:3112/api/items/abc-123 → 404 — {"error":"not found"}`;
+        expect(parseClientErrors(output)).toEqual([
+            'API response error: GET http://localhost:3112/api/items/abc-123 → 404 — {"error":"not found"}',
+        ]);
+    });
+    it('collects API response errors alongside HTTP errors and console errors', () => {
+        const output = [
+            `[JournalCapture] HTTP error: status=404, url=http://localhost:3112/page/abc`,
+            `[JournalCapture] API response error: GET http://localhost:3112/api/items/abc-123 → 404 — {"error":"Item not found"}`,
+            `[JournalCapture] Page console.error: Unhandled error in component`,
+        ].join('\n');
+        expect(parseClientErrors(output)).toEqual([
+            'HTTP error: status=404, url=http://localhost:3112/page/abc',
+            'API response error: GET http://localhost:3112/api/items/abc-123 → 404 — {"error":"Item not found"}',
+            'Unhandled error in component',
+        ]);
+    });
+});
+describe('parseVisibleText', () => {
+    it('extracts visible text from capture output', () => {
+        const output = '[JournalCapture] Visible text: Save to Library | Tech Chronicle';
+        expect(parseVisibleText(output)).toBe('Save to Library | Tech Chronicle');
+    });
+    it('returns null when no visible text line exists', () => {
+        const output = '[JournalCapture] Page loaded: status=200';
+        expect(parseVisibleText(output)).toBeNull();
+    });
+    it('extracts empty page marker', () => {
+        const output = '[JournalCapture] Visible text: (empty page)';
+        expect(parseVisibleText(output)).toBe('(empty page)');
+    });
+    it('extracts from multi-line output with other log lines', () => {
+        const output = [
+            '[JournalCapture] Page loaded: status=200, url=http://localhost:3113/',
+            '[JournalCapture] Page title: Margo',
+            '[JournalCapture] Visible text: Save to Library | Tech Chronicle | The Future of Web Dev',
+            '[JournalCapture] RESULT:{"success":true,"path":"/tmp/screenshot.png"}',
+        ].join('\n');
+        expect(parseVisibleText(output)).toBe('Save to Library | Tech Chronicle | The Future of Web Dev');
+    });
+    it('extracts extraction failure marker', () => {
+        const output = '[JournalCapture] Visible text: (extraction failed)';
+        expect(parseVisibleText(output)).toBe('(extraction failed)');
+    });
+});
+//# sourceMappingURL=clientErrors.test.js.map

package/codeyam-cli/src/webserver/__tests__/clientErrors.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"clientErrors.test.js","sourceRoot":"","sources":["../../../../../src/webserver/__tests__/clientErrors.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAE9E,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,8BAA8B,EAAE,GAAG,EAAE;QACtC,MAAM,MAAM,GAAG,8EAA8E,CAAC;QAC9F,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;YACxC,yCAAyC;SAC1C,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;QACxC,MAAM,MAAM,GAAG,uGAAuG,CAAC;QACvH,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;YACxC,sFAAsF;SACvF,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,MAAM,GAAG;YACb,qGAAqG;YACrG,uGAAuG;YACvG,sDAAsD;SACvD,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAEb,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;YACxC,sFAAsF;YACtF,iBAAiB;SAClB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,MAAM,MAAM,GAAG;YACb,2GAA2G;YAC3G,uGAAuG;SACxG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAEb,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;YACxC,sFAAsF;SACvF,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC9C,MAAM,MAAM,GAAG,0DAA0D,CAAC;QAC1E,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,MAAM,GAAG,8FAA8F,CAAC;QAC9G,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;YACxC,6EAA6E;SAC9E,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC9C,MAAM,MAAM,GAAG,8EAA8E,CAAC;QAC9F,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;YACxC,6DAA6D;SAC9D,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,MAAM,GAAG;YACb,4EAA4E;YAC5E,qEAAqE;SACtE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAEb,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;YACxC,2DAA2D;YAC3D,gCAAgC;SACjC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,MAAM,MAAM,GAAG,gHAAgH,CAAC;QAChI,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;YACxC,+FAA+F;SAChG,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,GAAG,EAAE;QAC/E,MAAM,MAAM,GAAG;YACb,6EAA6E;YAC7E,qHAAqH;YACrH,mEAAmE;SACpE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAEb,MAAM,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;YACxC,4DAA4D;YAC5D,oGAAoG;YACpG,8BAA8B;SAC/B,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,MAAM,MAAM,GACV,iEAAiE,CAAC;QACpE,MAAM,CAAC,gBAAgB,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,kCAAkC,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,MAAM,GAAG,0CAA0C,CAAC;QAC1D,MAAM,CAAC,gBAAgB,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;QACpC,MAAM,MAAM,GAAG,6CAA6C,CAAC;QAC7D,MAAM,CAAC,gBAAgB,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,MAAM,MAAM,GAAG;YACb,sEAAsE;YACtE,oCAAoC;YACpC,yFAAyF;YACzF,uEAAuE;SACxE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACb,MAAM,CAAC,gBAAgB,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CACnC,0DAA0D,CAC3D,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,MAAM,GAAG,oDAAoD,CAAC;QACpE,MAAM,CAAC,gBAAgB,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;IAC/D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/codeyam-cli/src/webserver/__tests__/dependency-smoke.test.js

@@ -0,0 +1,66 @@
+/**
+ * General smoke test to catch dependency incompatibilities that would crash
+ * the server at startup. Auto-discovers all runtime dependencies from the
+ * webserver's package.json and verifies each can be loaded without errors.
+ *
+ * ESM-only packages (like react-markdown) are expected to fail require() —
+ * the server loads them via import(). But if a CJS package internally tries
+ * to require() an ESM-only transitive dep, that's a real conflict this test
+ * catches (e.g. react-syntax-highlighter v15→v16 broke because refractor v5
+ * became ESM-only).
+ */
+import * as fs from 'fs';
+import * as path from 'path';
+const webserverPkg = JSON.parse(fs.readFileSync(path.join(__dirname, '..', 'package.json'), 'utf8'));
+const deps = Object.keys(webserverPkg.dependencies || {}).filter((dep) => !dep.startsWith('@types/'));
+/**
+ * Check if a package declares "type": "module" in its package.json.
+ * These packages are ESM-only and expected to fail require().
+ */
+function isPackageEsmOnly(dep) {
+    let resolved;
+    try {
+        resolved = require.resolve(dep);
+    }
+    catch {
+        // Can't even resolve → likely ESM-only without a CJS entry
+        return true;
+    }
+    try {
+        // Walk up from the resolved entry point to find the package's package.json
+        let dir = path.dirname(resolved);
+        for (let i = 0; i < 5; i++) {
+            const pkgPath = path.join(dir, 'package.json');
+            if (fs.existsSync(pkgPath)) {
+                const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
+                return pkg.type === 'module';
+            }
+            dir = path.dirname(dir);
+        }
+    }
+    catch { }
+    return false;
+}
+describe('server dependency smoke tests', () => {
+    it.each(deps)('%s loads without CJS/ESM conflicts', (dep) => {
+        try {
+            require(dep);
+        }
+        catch (e) {
+            // If the package itself is ESM-only ("type": "module"), require() is
+            // expected to fail. The server loads these via import() and they work fine.
+            if (isPackageEsmOnly(dep)) {
+                return;
+            }
+            // For CJS packages, ANY error is a real problem — including internal
+            // CJS→ESM conflicts where a CJS dep tries to require() an ESM-only
+            // transitive dependency (exactly the react-syntax-highlighter v16 bug).
+            throw e;
+        }
+    });
+    // Sub-path imports that the server actually uses at runtime
+    it('react-syntax-highlighter CJS prism styles load', () => {
+        expect(() => require('react-syntax-highlighter/dist/cjs/styles/prism')).not.toThrow();
+    });
+});
+//# sourceMappingURL=dependency-smoke.test.js.map

package/codeyam-cli/src/webserver/__tests__/dependency-smoke.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"dependency-smoke.test.js","sourceRoot":"","sources":["../../../../../src/webserver/__tests__/dependency-smoke.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AACH,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAE7B,MAAM,YAAY,GAAG,IAAI,CAAC,KAAK,CAC7B,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,cAAc,CAAC,EAAE,MAAM,CAAC,CACpE,CAAC;AAEF,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,YAAY,CAAC,YAAY,IAAI,EAAE,CAAC,CAAC,MAAM,CAC9D,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,GAAG,CAAC,UAAU,CAAC,SAAS,CAAC,CACpC,CAAC;AAEF;;;GAGG;AACH,SAAS,gBAAgB,CAAC,GAAW;IACnC,IAAI,QAAgB,CAAC;IACrB,IAAI,CAAC;QACH,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IAClC,CAAC;IAAC,MAAM,CAAC;QACP,2DAA2D;QAC3D,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,CAAC;QACH,2EAA2E;QAC3E,IAAI,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;QACjC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YAC3B,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;YAC/C,IAAI,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,CAAC;gBAC3B,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,CAAC;gBACzD,OAAO,GAAG,CAAC,IAAI,KAAK,QAAQ,CAAC;YAC/B,CAAC;YACD,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;QAC1B,CAAC;IACH,CAAC;IAAC,MAAM,CAAC,CAAA,CAAC;IAEV,OAAO,KAAK,CAAC;AACf,CAAC;AAED,QAAQ,CAAC,+BAA+B,EAAE,GAAG,EAAE;IAC7C,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,oCAAoC,EAAE,CAAC,GAAG,EAAE,EAAE;QAC1D,IAAI,CAAC;YACH,OAAO,CAAC,GAAG,CAAC,CAAC;QACf,CAAC;QAAC,OAAO,CAAM,EAAE,CAAC;YAChB,qEAAqE;YACrE,4EAA4E;YAC5E,IAAI,gBAAgB,CAAC,GAAG,CAAC,EAAE,CAAC;gBAC1B,OAAO;YACT,CAAC;YAED,qEAAqE;YACrE,mEAAmE;YACnE,wEAAwE;YACxE,MAAM,CAAC,CAAC;QACV,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,4DAA4D;IAC5D,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,CAAC,GAAG,EAAE,CACV,OAAO,CAAC,gDAAgD,CAAC,CAC1D,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;IAClB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}