@codeyam/codeyam-cli 0.1.0-staging.483fdc2 → 0.1.0-staging.57cd50f

package/codeyam-cli/src/utils/__tests__/scenariosManifest.test.js

@@ -0,0 +1,246 @@
+import * as fs from 'fs';
+import * as path from 'path';
+import * as os from 'os';
+import { readManifest, writeManifest, addScenarioToManifest, removeScenarioFromManifest, buildManifestFromRows, syncManifestToDatabase, } from "../scenariosManifest.js";
+function makeTmpDir() {
+    return fs.mkdtempSync(path.join(os.tmpdir(), 'manifest-test-'));
+}
+function makeEntry(overrides = {}) {
+    return {
+        id: 'test-id-1',
+        name: 'Test Scenario',
+        description: null,
+        componentName: null,
+        componentPath: null,
+        url: '/',
+        mockDataFile: 'editor-scenarios/test-id-1.json',
+        screenshotFile: null,
+        createdAt: '2024-01-01T00:00:00.000Z',
+        updatedAt: '2024-01-01T00:00:00.000Z',
+        ...overrides,
+    };
+}
+describe('scenariosManifest', () => {
+    describe('readManifest / writeManifest round-trip', () => {
+        it('should write and read back a manifest', () => {
+            const tmp = makeTmpDir();
+            const manifest = {
+                version: 1,
+                updatedAt: '2024-01-01T00:00:00.000Z',
+                scenarios: [makeEntry()],
+            };
+            writeManifest(tmp, manifest);
+            const result = readManifest(tmp);
+            expect(result).toEqual(manifest);
+        });
+        it('should return null for missing file', () => {
+            const tmp = makeTmpDir();
+            expect(readManifest(tmp)).toBeNull();
+        });
+    });
+    describe('addScenarioToManifest', () => {
+        it('should create manifest if none exists', () => {
+            const tmp = makeTmpDir();
+            const entry = makeEntry();
+            addScenarioToManifest(tmp, entry);
+            const manifest = readManifest(tmp);
+            expect(manifest).not.toBeNull();
+            expect(manifest.scenarios).toHaveLength(1);
+            expect(manifest.scenarios[0].id).toBe('test-id-1');
+        });
+        it('should append to existing manifest', () => {
+            const tmp = makeTmpDir();
+            addScenarioToManifest(tmp, makeEntry({ id: 'a', name: 'A' }));
+            addScenarioToManifest(tmp, makeEntry({ id: 'b', name: 'B' }));
+            const manifest = readManifest(tmp);
+            expect(manifest.scenarios).toHaveLength(2);
+        });
+        it('should deduplicate by id (update existing)', () => {
+            const tmp = makeTmpDir();
+            addScenarioToManifest(tmp, makeEntry({ id: 'a', name: 'Original' }));
+            addScenarioToManifest(tmp, makeEntry({ id: 'a', name: 'Updated' }));
+            const manifest = readManifest(tmp);
+            expect(manifest.scenarios).toHaveLength(1);
+            expect(manifest.scenarios[0].name).toBe('Updated');
+        });
+    });
+    describe('removeScenarioFromManifest', () => {
+        it('should remove a scenario by id', () => {
+            const tmp = makeTmpDir();
+            addScenarioToManifest(tmp, makeEntry({ id: 'a', name: 'A' }));
+            addScenarioToManifest(tmp, makeEntry({ id: 'b', name: 'B' }));
+            removeScenarioFromManifest(tmp, 'a');
+            const manifest = readManifest(tmp);
+            expect(manifest.scenarios).toHaveLength(1);
+            expect(manifest.scenarios[0].id).toBe('b');
+        });
+        it('should handle removing from nonexistent manifest', () => {
+            const tmp = makeTmpDir();
+            // Should not throw
+            removeScenarioFromManifest(tmp, 'nonexistent');
+        });
+        it('should handle removing nonexistent id', () => {
+            const tmp = makeTmpDir();
+            addScenarioToManifest(tmp, makeEntry({ id: 'a', name: 'A' }));
+            removeScenarioFromManifest(tmp, 'nonexistent');
+            const manifest = readManifest(tmp);
+            expect(manifest.scenarios).toHaveLength(1);
+        });
+    });
+    describe('buildManifestFromRows', () => {
+        it('should build manifest from database-like rows', () => {
+            const rows = [
+                {
+                    id: 'row-1',
+                    name: 'Scenario 1',
+                    description: 'A scenario',
+                    component_name: 'Button',
+                    component_path: 'app/components/Button.tsx',
+                    url: '/isolated-components/Button?s=Default',
+                    screenshot_path: 'screenshots/row-1.png',
+                    created_at: '2024-01-01 00:00:00',
+                    updated_at: '2024-01-02 00:00:00',
+                },
+                {
+                    id: 'row-2',
+                    name: 'Scenario 2',
+                    description: null,
+                    component_name: null,
+                    component_path: null,
+                    url: '/',
+                    screenshot_path: null,
+                    created_at: '2024-01-03 00:00:00',
+                    updated_at: '2024-01-03 00:00:00',
+                },
+            ];
+            const manifest = buildManifestFromRows(rows);
+            expect(manifest.version).toBe(1);
+            expect(manifest.scenarios).toHaveLength(2);
+            expect(manifest.scenarios[0]).toEqual({
+                id: 'row-1',
+                name: 'Scenario 1',
+                description: 'A scenario',
+                componentName: 'Button',
+                componentPath: 'app/components/Button.tsx',
+                url: '/isolated-components/Button?s=Default',
+                mockDataFile: 'editor-scenarios/row-1.json',
+                screenshotFile: 'editor-scenarios/screenshots/row-1.png',
+                createdAt: '2024-01-01 00:00:00',
+                updatedAt: '2024-01-02 00:00:00',
+            });
+            expect(manifest.scenarios[1].screenshotFile).toBeNull();
+        });
+        it('should handle empty rows', () => {
+            const manifest = buildManifestFromRows([]);
+            expect(manifest.scenarios).toEqual([]);
+        });
+    });
+    describe('syncManifestToDatabase', () => {
+        it('should insert missing scenarios', async () => {
+            const tmp = makeTmpDir();
+            const entry = makeEntry({ id: 'new-1', name: 'New Scenario' });
+            addScenarioToManifest(tmp, entry);
+            const existingRows = [];
+            const insertedRows = [];
+            const updatedRows = [];
+            const result = await syncManifestToDatabase(tmp, 'project-1', existingRows, (row) => {
+                insertedRows.push(row);
+            }, (id, row) => {
+                updatedRows.push({ id, ...row });
+            });
+            expect(result.inserted).toBe(1);
+            expect(result.updated).toBe(0);
+            expect(result.skipped).toBe(0);
+            expect(insertedRows).toHaveLength(1);
+            expect(insertedRows[0].id).toBe('new-1');
+            expect(insertedRows[0].project_id).toBe('project-1');
+        });
+        it('should skip scenarios already in DB with same updatedAt', async () => {
+            const tmp = makeTmpDir();
+            const entry = makeEntry({
+                id: 'existing-1',
+                name: 'Existing',
+                updatedAt: '2024-01-01T00:00:00.000Z',
+            });
+            addScenarioToManifest(tmp, entry);
+            const existingRows = [
+                { id: 'existing-1', updated_at: '2024-01-01T00:00:00.000Z' },
+            ];
+            const insertedRows = [];
+            const updatedRows = [];
+            const result = await syncManifestToDatabase(tmp, 'project-1', existingRows, (row) => {
+                insertedRows.push(row);
+            }, (id, row) => {
+                updatedRows.push({ id, ...row });
+            });
+            expect(result.inserted).toBe(0);
+            expect(result.updated).toBe(0);
+            expect(result.skipped).toBe(1);
+        });
+        it('should update scenarios newer than DB row', async () => {
+            const tmp = makeTmpDir();
+            const entry = makeEntry({
+                id: 'existing-1',
+                name: 'Updated Name',
+                updatedAt: '2024-02-01T00:00:00.000Z',
+            });
+            addScenarioToManifest(tmp, entry);
+            const existingRows = [
+                { id: 'existing-1', updated_at: '2024-01-01T00:00:00.000Z' },
+            ];
+            const insertedRows = [];
+            const updatedRows = [];
+            const result = await syncManifestToDatabase(tmp, 'project-1', existingRows, (row) => {
+                insertedRows.push(row);
+            }, (id, row) => {
+                updatedRows.push({ id, ...row });
+            });
+            expect(result.inserted).toBe(0);
+            expect(result.updated).toBe(1);
+            expect(result.skipped).toBe(0);
+            expect(updatedRows[0].name).toBe('Updated Name');
+        });
+        it('should handle empty manifest', async () => {
+            const tmp = makeTmpDir();
+            writeManifest(tmp, { version: 1, updatedAt: '', scenarios: [] });
+            const result = await syncManifestToDatabase(tmp, 'project-1', [], () => { }, () => { });
+            expect(result.inserted).toBe(0);
+            expect(result.updated).toBe(0);
+            expect(result.skipped).toBe(0);
+        });
+        it('should handle no manifest file', async () => {
+            const tmp = makeTmpDir();
+            const result = await syncManifestToDatabase(tmp, 'project-1', [], () => { }, () => { });
+            expect(result.inserted).toBe(0);
+            expect(result.updated).toBe(0);
+            expect(result.skipped).toBe(0);
+        });
+        it('should handle mixed insert/update/skip', async () => {
+            const tmp = makeTmpDir();
+            addScenarioToManifest(tmp, makeEntry({
+                id: 'new-1',
+                name: 'New',
+                updatedAt: '2024-01-01T00:00:00.000Z',
+            }));
+            addScenarioToManifest(tmp, makeEntry({
+                id: 'updated-1',
+                name: 'Updated',
+                updatedAt: '2024-02-01T00:00:00.000Z',
+            }));
+            addScenarioToManifest(tmp, makeEntry({
+                id: 'current-1',
+                name: 'Current',
+                updatedAt: '2024-01-01T00:00:00.000Z',
+            }));
+            const existingRows = [
+                { id: 'updated-1', updated_at: '2024-01-01T00:00:00.000Z' },
+                { id: 'current-1', updated_at: '2024-01-01T00:00:00.000Z' },
+            ];
+            const result = await syncManifestToDatabase(tmp, 'project-1', existingRows, () => { }, () => { });
+            expect(result.inserted).toBe(1);
+            expect(result.updated).toBe(1);
+            expect(result.skipped).toBe(1);
+        });
+    });
+});
+//# sourceMappingURL=scenariosManifest.test.js.map

package/codeyam-cli/src/utils/__tests__/scenariosManifest.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"scenariosManifest.test.js","sourceRoot":"","sources":["../../../../../src/utils/__tests__/scenariosManifest.test.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAC7B,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,EACL,YAAY,EACZ,aAAa,EACb,qBAAqB,EACrB,0BAA0B,EAC1B,qBAAqB,EACrB,sBAAsB,GAGvB,MAAM,sBAAsB,CAAC;AAE9B,SAAS,UAAU;IACjB,OAAO,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,gBAAgB,CAAC,CAAC,CAAC;AAClE,CAAC;AAED,SAAS,SAAS,CAChB,YAA4C,EAAE;IAE9C,OAAO;QACL,EAAE,EAAE,WAAW;QACf,IAAI,EAAE,eAAe;QACrB,WAAW,EAAE,IAAI;QACjB,aAAa,EAAE,IAAI;QACnB,aAAa,EAAE,IAAI;QACnB,GAAG,EAAE,GAAG;QACR,YAAY,EAAE,iCAAiC;QAC/C,cAAc,EAAE,IAAI;QACpB,SAAS,EAAE,0BAA0B;QACrC,SAAS,EAAE,0BAA0B;QACrC,GAAG,SAAS;KACb,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,QAAQ,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACvD,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,MAAM,QAAQ,GAAsB;gBAClC,OAAO,EAAE,CAAC;gBACV,SAAS,EAAE,0BAA0B;gBACrC,SAAS,EAAE,CAAC,SAAS,EAAE,CAAC;aACzB,CAAC;YAEF,aAAa,CAAC,GAAG,EAAE,QAAQ,CAAC,CAAC;YAC7B,MAAM,MAAM,GAAG,YAAY,CAAC,GAAG,CAAC,CAAC;YAEjC,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;QACnC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,MAAM,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC;QACvC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;QACrC,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,MAAM,KAAK,GAAG,SAAS,EAAE,CAAC;YAE1B,qBAAqB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;YAElC,MAAM,QAAQ,GAAG,YAAY,CAAC,GAAG,CAAC,CAAC;YACnC,MAAM,CAAC,QAAQ,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;YAChC,MAAM,CAAC,QAAS,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC5C,MAAM,CAAC,QAAS,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QACtD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;YAC5C,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,qBAAqB,CAAC,GAAG,EAAE,SAAS,CAAC,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC;YAC9D,qBAAqB,CAAC,GAAG,EAAE,SAAS,CAAC,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC;YAE9D,MAAM,QAAQ,GAAG,YAAY,CAAC,GAAG,CAAC,CAAC;YACnC,MAAM,CAAC,QAAS,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC9C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;YACpD,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,qBAAqB,CAAC,GAAG,EAAE,SAAS,CAAC,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC;YACrE,qBAAqB,CAAC,GAAG,EAAE,SAAS,CAAC,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC;YAEpE,MAAM,QAAQ,GAAG,YAAY,CAAC,GAAG,CAAC,CAAC;YACnC,MAAM,CAAC,QAAS,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC5C,MAAM,CAAC,QAAS,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACtD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,4BAA4B,EAAE,GAAG,EAAE;QAC1C,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;YACxC,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,qBAAqB,CAAC,GAAG,EAAE,SAAS,CAAC,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC;YAC9D,qBAAqB,CAAC,GAAG,EAAE,SAAS,CAAC,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC;YAE9D,0BAA0B,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;YAErC,MAAM,QAAQ,GAAG,YAAY,CAAC,GAAG,CAAC,CAAC;YACnC,MAAM,CAAC,QAAS,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC5C,MAAM,CAAC,QAAS,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC9C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;YAC1D,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,mBAAmB;YACnB,0BAA0B,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC;QACjD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,qBAAqB,CAAC,GAAG,EAAE,SAAS,CAAC,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,CAAC,CAAC,CAAC;YAE9D,0BAA0B,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC;YAE/C,MAAM,QAAQ,GAAG,YAAY,CAAC,GAAG,CAAC,CAAC;YACnC,MAAM,CAAC,QAAS,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC9C,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;QACrC,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;YACvD,MAAM,IAAI,GAAG;gBACX;oBACE,EAAE,EAAE,OAAO;oBACX,IAAI,EAAE,YAAY;oBAClB,WAAW,EAAE,YAAY;oBACzB,cAAc,EAAE,QAAQ;oBACxB,cAAc,EAAE,2BAA2B;oBAC3C,GAAG,EAAE,uCAAuC;oBAC5C,eAAe,EAAE,uBAAuB;oBACxC,UAAU,EAAE,qBAAqB;oBACjC,UAAU,EAAE,qBAAqB;iBAClC;gBACD;oBACE,EAAE,EAAE,OAAO;oBACX,IAAI,EAAE,YAAY;oBAClB,WAAW,EAAE,IAAI;oBACjB,cAAc,EAAE,IAAI;oBACpB,cAAc,EAAE,IAAI;oBACpB,GAAG,EAAE,GAAG;oBACR,eAAe,EAAE,IAAI;oBACrB,UAAU,EAAE,qBAAqB;oBACjC,UAAU,EAAE,qBAAqB;iBAClC;aACF,CAAC;YAEF,MAAM,QAAQ,GAAG,qBAAqB,CAAC,IAAI,CAAC,CAAC;YAE7C,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YACjC,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC3C,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;gBACpC,EAAE,EAAE,OAAO;gBACX,IAAI,EAAE,YAAY;gBAClB,WAAW,EAAE,YAAY;gBACzB,aAAa,EAAE,QAAQ;gBACvB,aAAa,EAAE,2BAA2B;gBAC1C,GAAG,EAAE,uCAAuC;gBAC5C,YAAY,EAAE,6BAA6B;gBAC3C,cAAc,EAAE,wCAAwC;gBACxD,SAAS,EAAE,qBAAqB;gBAChC,SAAS,EAAE,qBAAqB;aACjC,CAAC,CAAC;YACH,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,QAAQ,EAAE,CAAC;QAC1D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0BAA0B,EAAE,GAAG,EAAE;YAClC,MAAM,QAAQ,GAAG,qBAAqB,CAAC,EAAE,CAAC,CAAC;YAC3C,MAAM,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QACzC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,wBAAwB,EAAE,GAAG,EAAE;QACtC,EAAE,CAAC,iCAAiC,EAAE,KAAK,IAAI,EAAE;YAC/C,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,MAAM,KAAK,GAAG,SAAS,CAAC,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,CAAC;YAC/D,qBAAqB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;YAElC,MAAM,YAAY,GAAU,EAAE,CAAC;YAC/B,MAAM,YAAY,GAAU,EAAE,CAAC;YAC/B,MAAM,WAAW,GAAU,EAAE,CAAC;YAE9B,MAAM,MAAM,GAAG,MAAM,sBAAsB,CACzC,GAAG,EACH,WAAW,EACX,YAAY,EACZ,CAAC,GAAG,EAAE,EAAE;gBACN,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACzB,CAAC,EACD,CAAC,EAAE,EAAE,GAAG,EAAE,EAAE;gBACV,WAAW,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,GAAG,GAAG,EAAE,CAAC,CAAC;YACnC,CAAC,CACF,CAAC;YAEF,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,YAAY,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YACrC,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;YACzC,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QACvD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;YACvE,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,MAAM,KAAK,GAAG,SAAS,CAAC;gBACtB,EAAE,EAAE,YAAY;gBAChB,IAAI,EAAE,UAAU;gBAChB,SAAS,EAAE,0BAA0B;aACtC,CAAC,CAAC;YACH,qBAAqB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;YAElC,MAAM,YAAY,GAAG;gBACnB,EAAE,EAAE,EAAE,YAAY,EAAE,UAAU,EAAE,0BAA0B,EAAE;aAC7D,CAAC;YACF,MAAM,YAAY,GAAU,EAAE,CAAC;YAC/B,MAAM,WAAW,GAAU,EAAE,CAAC;YAE9B,MAAM,MAAM,GAAG,MAAM,sBAAsB,CACzC,GAAG,EACH,WAAW,EACX,YAAY,EACZ,CAAC,GAAG,EAAE,EAAE;gBACN,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACzB,CAAC,EACD,CAAC,EAAE,EAAE,GAAG,EAAE,EAAE;gBACV,WAAW,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,GAAG,GAAG,EAAE,CAAC,CAAC;YACnC,CAAC,CACF,CAAC;YAEF,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACjC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;YACzD,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,MAAM,KAAK,GAAG,SAAS,CAAC;gBACtB,EAAE,EAAE,YAAY;gBAChB,IAAI,EAAE,cAAc;gBACpB,SAAS,EAAE,0BAA0B;aACtC,CAAC,CAAC;YACH,qBAAqB,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;YAElC,MAAM,YAAY,GAAG;gBACnB,EAAE,EAAE,EAAE,YAAY,EAAE,UAAU,EAAE,0BAA0B,EAAE;aAC7D,CAAC;YACF,MAAM,YAAY,GAAU,EAAE,CAAC;YAC/B,MAAM,WAAW,GAAU,EAAE,CAAC;YAE9B,MAAM,MAAM,GAAG,MAAM,sBAAsB,CACzC,GAAG,EACH,WAAW,EACX,YAAY,EACZ,CAAC,GAAG,EAAE,EAAE;gBACN,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACzB,CAAC,EACD,CAAC,EAAE,EAAE,GAAG,EAAE,EAAE;gBACV,WAAW,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,GAAG,GAAG,EAAE,CAAC,CAAC;YACnC,CAAC,CACF,CAAC;YAEF,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;QACnD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,8BAA8B,EAAE,KAAK,IAAI,EAAE;YAC5C,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,aAAa,CAAC,GAAG,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,SAAS,EAAE,EAAE,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC,CAAC;YAEjE,MAAM,MAAM,GAAG,MAAM,sBAAsB,CACzC,GAAG,EACH,WAAW,EACX,EAAE,EACF,GAAG,EAAE,GAAE,CAAC,EACR,GAAG,EAAE,GAAE,CAAC,CACT,CAAC;YAEF,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACjC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gCAAgC,EAAE,KAAK,IAAI,EAAE;YAC9C,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YAEzB,MAAM,MAAM,GAAG,MAAM,sBAAsB,CACzC,GAAG,EACH,WAAW,EACX,EAAE,EACF,GAAG,EAAE,GAAE,CAAC,EACR,GAAG,EAAE,GAAE,CAAC,CACT,CAAC;YAEF,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACjC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;YACtD,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;YACzB,qBAAqB,CACnB,GAAG,EACH,SAAS,CAAC;gBACR,EAAE,EAAE,OAAO;gBACX,IAAI,EAAE,KAAK;gBACX,SAAS,EAAE,0BAA0B;aACtC,CAAC,CACH,CAAC;YACF,qBAAqB,CACnB,GAAG,EACH,SAAS,CAAC;gBACR,EAAE,EAAE,WAAW;gBACf,IAAI,EAAE,SAAS;gBACf,SAAS,EAAE,0BAA0B;aACtC,CAAC,CACH,CAAC;YACF,qBAAqB,CACnB,GAAG,EACH,SAAS,CAAC;gBACR,EAAE,EAAE,WAAW;gBACf,IAAI,EAAE,SAAS;gBACf,SAAS,EAAE,0BAA0B;aACtC,CAAC,CACH,CAAC;YAEF,MAAM,YAAY,GAAG;gBACnB,EAAE,EAAE,EAAE,WAAW,EAAE,UAAU,EAAE,0BAA0B,EAAE;gBAC3D,EAAE,EAAE,EAAE,WAAW,EAAE,UAAU,EAAE,0BAA0B,EAAE;aAC5D,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,sBAAsB,CACzC,GAAG,EACH,WAAW,EACX,YAAY,EACZ,GAAG,EAAE,GAAE,CAAC,EACR,GAAG,EAAE,GAAE,CAAC,CACT,CAAC;YAEF,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACjC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/codeyam-cli/src/utils/__tests__/setupClaudeCodeSettings.test.js

@@ -6,6 +6,50 @@ jest.mock('fs');
 jest.mock('path');
 const mockFs = fs;
 const mockPath = path;
+// Shared permission lists for assertions
+const memoryPermissions = [
+    'Skill(codeyam-memory)',
+    'Skill(codeyam-new-rule)',
+    'Bash(codeyam)',
+    'Bash(codeyam:*)',
+    'Read(.codeyam/**)',
+    'Edit(.codeyam/**)',
+    'Write(.codeyam/**)',
+    'Glob(.codeyam/**)',
+];
+const editorPermissions = [
+    'Skill(codeyam-editor)',
+    'Bash(git:*)',
+    'Edit(**)',
+    'Write(**)',
+];
+const simulationPermissions = [
+    'Skill(codeyam-setup)',
+    'Skill(codeyam-sim)',
+    'Skill(codeyam-debug)',
+    'Skill(codeyam-test)',
+    'Skill(codeyam-verify)',
+    'Skill(codeyam-dev-mode)',
+    'Read(/tmp/**)',
+    'Edit(/tmp/**)',
+    'Write(/tmp/**)',
+    'Glob(/tmp/**)',
+    'Bash(npm install:*)',
+    'Bash(npm run:*)',
+    'Bash(npx:*)',
+    'Bash(node:*)',
+    'Bash(cp:*)',
+    'Bash(mkdir:*)',
+    'Bash(chmod:*)',
+    'Bash(rm:*)',
+    'Bash(curl:*)',
+    'Bash(sleep:*)',
+    'Bash(echo:*)',
+    'Bash(grep:*)',
+    'Bash(python3:*)',
+    'Bash(cat:*)',
+    'Bash(printf:*)',
+];
 describe('setupClaudeCodeSettings', () => {
     const projectRoot = '/Users/testuser/myproject';
     const claudeDir = '/Users/testuser/myproject/.claude';
@@ -23,10 +67,8 @@ describe('setupClaudeCodeSettings', () => {
     describe('directory and file creation', () => {
         test('creates .claude directory if it does not exist', () => {
             mockFs.existsSync.mockImplementation((p) => {
-                // .claude directory doesn't exist
                 if (p === claudeDir)
                     return false;
-                // settings.local.json doesn't exist either
                 if (p === settingsPath)
                     return false;
                 return false;
@@ -39,12 +81,10 @@ describe('setupClaudeCodeSettings', () => {
             });
             expect(result).toBe(true);
         });
-        test('creates new settings file with all CodeYam permissions', () => {
+        test('creates new settings file with only memory permissions by default', () => {
             mockFs.existsSync.mockImplementation((p) => {
-                // .claude directory exists
                 if (p === claudeDir)
                     return true;
-                // settings.local.json doesn't exist yet
                 if (p === settingsPath)
                     return false;
                 return false;
@@ -52,92 +92,73 @@ describe('setupClaudeCodeSettings', () => {
             mockFs.writeFileSync.mockImplementation(() => undefined);
             const result = setupClaudeCodeSettings(projectRoot);
             expect(mockFs.writeFileSync).toHaveBeenCalledWith(settingsPath, expect.any(String), 'utf8');
-            // Parse the written content to verify structure
             const writtenContent = mockFs.writeFileSync.mock
                 .calls[0][1];
             const settings = JSON.parse(writtenContent);
-            expect(settings.permissions.allow).toContain('Skill(codeyam:setup)');
-            expect(settings.permissions.allow).toContain('Bash(codeyam)');
-            expect(settings.permissions.allow).toContain('Read(.codeyam/**)');
+            // Should include memory permissions
+            for (const perm of memoryPermissions) {
+                expect(settings.permissions.allow).toContain(perm);
+            }
+            // Should NOT include simulation permissions
+            for (const perm of simulationPermissions) {
+                expect(settings.permissions.allow).not.toContain(perm);
+            }
             expect(result).toBe(true);
         });
     });
-    describe('adding permissions to existing settings', () => {
-        test('adds CodeYam permissions to existing settings without allow list', () => {
+    describe('memory mode (default)', () => {
+        test('adds only memory permissions when mode is memory', () => {
             mockFs.existsSync.mockReturnValue(true);
             mockFs.readFileSync.mockReturnValue(JSON.stringify({ otherSetting: 'value' }));
             let writtenSettings;
             mockFs.writeFileSync.mockImplementation((_path, content) => {
                 writtenSettings = JSON.parse(content);
             });
-            const result = setupClaudeCodeSettings(projectRoot);
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:setup)');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:sim)');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:debug)');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:test)');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:verify)');
-            expect(writtenSettings.permissions.allow).toContain('Bash(codeyam:*)');
+            const result = setupClaudeCodeSettings(projectRoot, { mode: 'memory' });
+            for (const perm of memoryPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
+            for (const perm of simulationPermissions) {
+                expect(writtenSettings.permissions.allow).not.toContain(perm);
+            }
             expect(writtenSettings.otherSetting).toBe('value');
             expect(result).toBe(true);
         });
-        test('adds CodeYam permissions to existing allow list', () => {
+        test('default mode (no options) behaves as memory mode', () => {
             mockFs.existsSync.mockReturnValue(true);
-            mockFs.readFileSync.mockReturnValue(JSON.stringify({
-                permissions: {
-                    allow: ['some-other-tool'],
-                },
-            }));
+            mockFs.readFileSync.mockReturnValue(JSON.stringify({}));
             let writtenSettings;
             mockFs.writeFileSync.mockImplementation((_path, content) => {
                 writtenSettings = JSON.parse(content);
             });
-            const result = setupClaudeCodeSettings(projectRoot);
-            expect(writtenSettings.permissions.allow).toContain('some-other-tool');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:setup)');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:sim)');
-            expect(writtenSettings.permissions.allow).toContain('Write(.codeyam/**)');
-            expect(result).toBe(true);
+            setupClaudeCodeSettings(projectRoot);
+            for (const perm of memoryPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
+            for (const perm of simulationPermissions) {
+                expect(writtenSettings.permissions.allow).not.toContain(perm);
+            }
         });
-        test('does not duplicate CodeYam permissions if already present', () => {
+        test('does not duplicate memory permissions if already present', () => {
             mockFs.existsSync.mockReturnValue(true);
             mockFs.readFileSync.mockReturnValue(JSON.stringify({
                 permissions: {
-                    allow: [
-                        'Skill(codeyam:setup)',
-                        'Skill(codeyam:sim)',
-                        'Skill(codeyam:debug)',
-                        'Skill(codeyam:test)',
-                        'Skill(codeyam:verify)',
-                        'Skill(codeyam:memory)',
-                        'Skill(codeyam:new-rule)',
-                        'Bash(codeyam)',
-                        'Bash(codeyam:*)',
-                        'Read(.codeyam/**)',
-                        'Edit(.codeyam/**)',
-                        'Write(.codeyam/**)',
-                        'Glob(.codeyam/**)',
-                        'Read(/tmp/codeyam/**)',
-                        'Edit(/tmp/codeyam/**)',
-                        'Write(/tmp/codeyam/**)',
-                        'Glob(/tmp/codeyam/**)',
-                        'other-tool',
-                    ],
+                    allow: [...memoryPermissions, 'other-tool'],
                 },
             }));
             const result = setupClaudeCodeSettings(projectRoot);
-            // Should not write if all permissions are already present
+            // Should not write if all memory permissions are already present
             expect(mockFs.writeFileSync).not.toHaveBeenCalled();
             expect(result).toBe(true);
         });
-        test('adds only missing CodeYam permissions', () => {
+        test('adds only missing memory permissions', () => {
             mockFs.existsSync.mockReturnValue(true);
             mockFs.readFileSync.mockReturnValue(JSON.stringify({
                 permissions: {
                     allow: [
-                        'Skill(codeyam:setup)',
-                        'Skill(codeyam:sim)',
+                        'Skill(codeyam-memory)',
                         'other-tool',
-                        // Missing: Skill(codeyam:debug), Skill(codeyam:test), Skill(codeyam:verify), all Bash and file ops
+                        // Missing: Skill(codeyam-new-rule), Bash, file ops
                     ],
                 },
             }));
@@ -146,20 +167,95 @@ describe('setupClaudeCodeSettings', () => {
                 writtenSettings = JSON.parse(content);
             });
             const result = setupClaudeCodeSettings(projectRoot);
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:setup)');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:sim)');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:debug)');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:test)');
-            expect(writtenSettings.permissions.allow).toContain('Skill(codeyam:verify)');
-            expect(writtenSettings.permissions.allow).toContain('Bash(codeyam)');
+            for (const perm of memoryPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
             expect(writtenSettings.permissions.allow).toContain('other-tool');
             // Should not duplicate existing ones
-            const setupCount = writtenSettings.permissions.allow.filter((p) => p === 'Skill(codeyam:setup)').length;
-            expect(setupCount).toBe(1);
+            const memoryCount = writtenSettings.permissions.allow.filter((p) => p === 'Skill(codeyam-memory)').length;
+            expect(memoryCount).toBe(1);
+            expect(result).toBe(true);
+        });
+    });
+    describe('full mode', () => {
+        test('adds all permissions (memory + simulation) in full mode', () => {
+            mockFs.existsSync.mockReturnValue(true);
+            mockFs.readFileSync.mockReturnValue(JSON.stringify({ otherSetting: 'value' }));
+            let writtenSettings;
+            mockFs.writeFileSync.mockImplementation((_path, content) => {
+                writtenSettings = JSON.parse(content);
+            });
+            const result = setupClaudeCodeSettings(projectRoot, { mode: 'full' });
+            for (const perm of memoryPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
+            for (const perm of simulationPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
+            expect(writtenSettings.otherSetting).toBe('value');
+            expect(result).toBe(true);
+        });
+        test('does not duplicate full permissions if already present', () => {
+            const allPermissions = [...memoryPermissions, ...simulationPermissions];
+            mockFs.existsSync.mockReturnValue(true);
+            mockFs.readFileSync.mockReturnValue(JSON.stringify({
+                permissions: {
+                    allow: [...allPermissions, 'other-tool'],
+                },
+            }));
+            const result = setupClaudeCodeSettings(projectRoot, { mode: 'full' });
+            expect(mockFs.writeFileSync).not.toHaveBeenCalled();
+            expect(result).toBe(true);
+        });
+        test('adds simulation permissions to existing memory-only settings', () => {
+            mockFs.existsSync.mockReturnValue(true);
+            mockFs.readFileSync.mockReturnValue(JSON.stringify({
+                permissions: {
+                    allow: [...memoryPermissions, 'other-tool'],
+                },
+            }));
+            let writtenSettings;
+            mockFs.writeFileSync.mockImplementation((_path, content) => {
+                writtenSettings = JSON.parse(content);
+            });
+            const result = setupClaudeCodeSettings(projectRoot, { mode: 'full' });
+            for (const perm of memoryPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
+            for (const perm of simulationPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
+            expect(writtenSettings.permissions.allow).toContain('other-tool');
+            // No duplicates
+            const memoryCount = writtenSettings.permissions.allow.filter((p) => p === 'Skill(codeyam-memory)').length;
+            expect(memoryCount).toBe(1);
+            expect(result).toBe(true);
+        });
+    });
+    describe('editor mode', () => {
+        test('adds memory + editor permissions in editor mode', () => {
+            mockFs.existsSync.mockReturnValue(true);
+            mockFs.readFileSync.mockReturnValue(JSON.stringify({}));
+            let writtenSettings;
+            mockFs.writeFileSync.mockImplementation((_path, content) => {
+                writtenSettings = JSON.parse(content);
+            });
+            const result = setupClaudeCodeSettings(projectRoot, {
+                mode: 'editor',
+            });
+            for (const perm of memoryPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
+            for (const perm of editorPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
+            // Editor mode includes simulation permissions too
+            for (const perm of simulationPermissions) {
+                expect(writtenSettings.permissions.allow).toContain(perm);
+            }
             expect(result).toBe(true);
         });
     });
-    // Note: Stop hook configuration tests removed - hook functionality is currently disabled
     describe('error handling', () => {
         test('handles malformed JSON gracefully and creates new settings', () => {
             mockFs.existsSync.mockReturnValue(true);
@@ -192,33 +288,25 @@ describe('setupClaudeCodeSettings', () => {
         });
     });
     describe('idempotency', () => {
-        test('returns true when settings are already fully configured', () => {
+        test('returns true without writing when memory settings are fully configured', () => {
             mockFs.existsSync.mockReturnValue(true);
             mockFs.readFileSync.mockReturnValue(JSON.stringify({
                 permissions: {
-                    allow: [
-                        'Skill(codeyam:setup)',
-                        'Skill(codeyam:sim)',
-                        'Skill(codeyam:debug)',
-                        'Skill(codeyam:test)',
-                        'Skill(codeyam:verify)',
-                        'Skill(codeyam:memory)',
-                        'Skill(codeyam:new-rule)',
-                        'Bash(codeyam)',
-                        'Bash(codeyam:*)',
-                        'Read(.codeyam/**)',
-                        'Edit(.codeyam/**)',
-                        'Write(.codeyam/**)',
-                        'Glob(.codeyam/**)',
-                        'Read(/tmp/codeyam/**)',
-                        'Edit(/tmp/codeyam/**)',
-                        'Write(/tmp/codeyam/**)',
-                        'Glob(/tmp/codeyam/**)',
-                    ],
+                    allow: [...memoryPermissions],
                 },
             }));
             const result = setupClaudeCodeSettings(projectRoot);
-            // Should not write if already configured
+            expect(mockFs.writeFileSync).not.toHaveBeenCalled();
+            expect(result).toBe(true);
+        });
+        test('returns true without writing when full settings are fully configured', () => {
+            mockFs.existsSync.mockReturnValue(true);
+            mockFs.readFileSync.mockReturnValue(JSON.stringify({
+                permissions: {
+                    allow: [...memoryPermissions, ...simulationPermissions],
+                },
+            }));
+            const result = setupClaudeCodeSettings(projectRoot, { mode: 'full' });
             expect(mockFs.writeFileSync).not.toHaveBeenCalled();
             expect(result).toBe(true);
         });