@codeyam/codeyam-cli 0.1.10 → 0.1.12

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

@@ -3,7 +3,7 @@ import os from 'os';
 import path from 'path';
 import Database from 'better-sqlite3';
 import { Kysely, SqliteDialect } from 'kysely';
-import { deduplicateByName, generateScenarioSlug, convertIsoToSqliteTimestamp, determineCaptureUrl, resolvePreviewNavPath, clearEditorState, clearEditorUserPrompt, readDefaultScreenSize, readScreenSizes, resolveScenarioViewport, resolveViewportWithProjectDefault, upsertEditorScenario, cleanupScenarioFiles, readPreservedConfigProperties, validateStepTransition, slugifyDimension, isRowInFeatureSession, } from "../editorScenarios.js";
+import { deduplicateByName, generateScenarioSlug, convertIsoToSqliteTimestamp, determineCaptureUrl, resolvePreviewNavPath, clearEditorState, clearEditorUserPrompt, readDefaultScreenSize, readScreenSizes, resolveScenarioViewport, resolveViewportWithProjectDefault, upsertEditorScenario, cleanupScenarioFiles, readPreservedConfigProperties, validateStepTransition, slugifyDimension, isRowInFeatureSession, backfillEntityShaOnScenarios, countScenariosNeedingEntityBackfill, validateEntityLinkageForAppScenario, } from "../editorScenarios.js";
 describe('editorScenarios', () => {
     describe('deduplicateByName', () => {
         it('should keep only the last item for each key', () => {
@@ -497,7 +497,6 @@ describe('editorScenarios', () => {
                 type: null,
                 viewportWidth: 1280,
                 viewportHeight: 720,
-                dimension: null,
             });
             expect(result.isNew).toBe(true);
             expect(result.cleanedUpIds).toEqual([]);
@@ -522,7 +521,6 @@ describe('editorScenarios', () => {
                 type: null,
                 viewportWidth: 1280,
                 viewportHeight: 720,
-                dimension: null,
             });
             // Second registration with same name
             const second = await upsertEditorScenario(db, {
@@ -535,7 +533,6 @@ describe('editorScenarios', () => {
                 type: null,
                 viewportWidth: 800,
                 viewportHeight: 600,
-                dimension: null,
             });
             // Should reuse the same ID
             expect(second.scenarioId).toBe(first.scenarioId);
@@ -576,7 +573,6 @@ describe('editorScenarios', () => {
                 type: null,
                 viewportWidth: 1280,
                 viewportHeight: 720,
-                dimension: null,
             });
             expect(result.isNew).toBe(false);
             // Should return the IDs of cleaned-up duplicates
@@ -600,7 +596,6 @@ describe('editorScenarios', () => {
                 type: null,
                 viewportWidth: 1280,
                 viewportHeight: 720,
-                dimension: null,
             });
             await upsertEditorScenario(db, {
                 projectId,
@@ -612,7 +607,6 @@ describe('editorScenarios', () => {
                 type: null,
                 viewportWidth: 1280,
                 viewportHeight: 720,
-                dimension: null,
             });
             const rows = await db
                 .selectFrom('editor_scenarios')
@@ -631,7 +625,6 @@ describe('editorScenarios', () => {
                 type: 'application',
                 viewportWidth: 1280,
                 viewportHeight: 720,
-                dimension: null,
             });
             const result2 = await upsertEditorScenario(db, {
                 projectId: 'project-b',
@@ -643,7 +636,6 @@ describe('editorScenarios', () => {
                 type: 'application',
                 viewportWidth: 1280,
                 viewportHeight: 720,
-                dimension: null,
             });
             expect(result1.scenarioId).not.toBe(result2.scenarioId);
             expect(result1.isNew).toBe(true);
@@ -848,7 +840,7 @@ describe('editorScenarios', () => {
         afterEach(async () => {
             await db.destroy();
         });
-        it('should store dimensions as JSON array and set dimension to first element', async () => {
+        it('should store dimensions as JSON array', async () => {
             const result = await upsertEditorScenario(db, {
                 projectId,
                 name: 'Home Page',
@@ -859,7 +851,6 @@ describe('editorScenarios', () => {
                 type: 'application',
                 viewportWidth: 1440,
                 viewportHeight: 900,
-                dimension: 'Desktop',
                 dimensions: ['Desktop', 'Mobile'],
                 screenshotPaths: null,
             });
@@ -873,8 +864,6 @@ describe('editorScenarios', () => {
                 'Desktop',
                 'Mobile',
             ]);
-            // dimension (singular) should be set to first element for backward compat
-            expect(row.dimension).toBe('Desktop');
         });
         it('should store screenshot_paths as JSON object', async () => {
             const screenshotPaths = {
@@ -891,7 +880,6 @@ describe('editorScenarios', () => {
                 type: 'application',
                 viewportWidth: 1440,
                 viewportHeight: 900,
-                dimension: 'Desktop',
                 dimensions: ['Desktop', 'Mobile'],
                 screenshotPaths,
             });
@@ -913,7 +901,6 @@ describe('editorScenarios', () => {
                 type: 'application',
                 viewportWidth: 1440,
                 viewportHeight: 900,
-                dimension: 'Desktop',
                 dimensions: ['Desktop'],
                 screenshotPaths: { Desktop: 'screenshots/old--desktop.png' },
             });
@@ -927,7 +914,6 @@ describe('editorScenarios', () => {
                 type: 'application',
                 viewportWidth: 1440,
                 viewportHeight: 900,
-                dimension: 'Desktop',
                 dimensions: ['Desktop', 'Mobile'],
                 screenshotPaths: {
                     Desktop: 'screenshots/new--desktop.png',
@@ -960,7 +946,6 @@ describe('editorScenarios', () => {
                 type: null,
                 viewportWidth: 1280,
                 viewportHeight: 720,
-                dimension: null,
             });
             const row = await db
                 .selectFrom('editor_scenarios')
@@ -1055,5 +1040,444 @@ describe('editorScenarios', () => {
             expect(isRowInFeatureSession({ created_at: '2026-03-12 14:30:00', updated_at: null }, featureStart)).toBe(true);
         });
     });
+    describe('upsertEditorScenario with entitySha and displayName', () => {
+        let db;
+        let rawDb;
+        const projectId = 'test-project-id';
+        beforeEach(async () => {
+            rawDb = new Database(':memory:');
+            db = new Kysely({ dialect: new SqliteDialect({ database: rawDb }) });
+            await db.schema
+                .createTable('editor_scenarios')
+                .addColumn('id', 'varchar', (col) => col.primaryKey())
+                .addColumn('project_id', 'varchar', (col) => col.notNull())
+                .addColumn('name', 'varchar', (col) => col.notNull())
+                .addColumn('description', 'text')
+                .addColumn('component_name', 'varchar')
+                .addColumn('component_path', 'varchar')
+                .addColumn('url', 'varchar')
+                .addColumn('type', 'varchar')
+                .addColumn('screenshot_path', 'varchar')
+                .addColumn('viewport_width', 'integer')
+                .addColumn('viewport_height', 'integer')
+                .addColumn('dimension', 'varchar')
+                .addColumn('dimensions', 'text')
+                .addColumn('screenshot_paths', 'text')
+                .addColumn('page_file_path', 'varchar')
+                .addColumn('entity_sha', 'varchar')
+                .addColumn('display_name', 'varchar')
+                .addColumn('created_at', 'datetime', (col) => col.defaultTo(new Date().toISOString()))
+                .addColumn('updated_at', 'datetime', (col) => col.defaultTo(new Date().toISOString()))
+                .execute();
+        });
+        afterEach(async () => {
+            await db.destroy();
+        });
+        it('should store entitySha and displayName on insert', async () => {
+            const result = await upsertEditorScenario(db, {
+                projectId,
+                name: 'Home - Default',
+                description: null,
+                componentName: null,
+                componentPath: null,
+                url: '/',
+                type: 'application',
+                viewportWidth: 400,
+                viewportHeight: 600,
+                entitySha: 'abc123',
+                displayName: 'Home',
+            });
+            expect(result.isNew).toBe(true);
+            const row = rawDb
+                .prepare('SELECT entity_sha, display_name FROM editor_scenarios WHERE id = ?')
+                .get(result.scenarioId);
+            expect(row.entity_sha).toBe('abc123');
+            expect(row.display_name).toBe('Home');
+        });
+        it('should update entitySha and displayName on upsert', async () => {
+            // Insert first
+            await upsertEditorScenario(db, {
+                projectId,
+                name: 'Home - Default',
+                description: null,
+                componentName: null,
+                componentPath: null,
+                url: '/',
+                type: 'application',
+                viewportWidth: 400,
+                viewportHeight: 600,
+                entitySha: 'old-sha',
+                displayName: 'OldName',
+            });
+            // Upsert with new values
+            const result = await upsertEditorScenario(db, {
+                projectId,
+                name: 'Home - Default',
+                description: null,
+                componentName: null,
+                componentPath: null,
+                url: '/',
+                type: 'application',
+                viewportWidth: 400,
+                viewportHeight: 600,
+                entitySha: 'new-sha',
+                displayName: 'Home',
+            });
+            expect(result.isNew).toBe(false);
+            const row = rawDb
+                .prepare('SELECT entity_sha, display_name FROM editor_scenarios WHERE id = ?')
+                .get(result.scenarioId);
+            expect(row.entity_sha).toBe('new-sha');
+            expect(row.display_name).toBe('Home');
+        });
+        it('should not overwrite entitySha when not provided', async () => {
+            // Insert with SHA
+            const first = await upsertEditorScenario(db, {
+                projectId,
+                name: 'Home - Default',
+                description: null,
+                componentName: null,
+                componentPath: null,
+                url: '/',
+                type: 'application',
+                viewportWidth: 400,
+                viewportHeight: 600,
+                entitySha: 'keep-this',
+                displayName: 'Home',
+            });
+            // Upsert without entitySha/displayName
+            await upsertEditorScenario(db, {
+                projectId,
+                name: 'Home - Default',
+                description: 'updated',
+                componentName: null,
+                componentPath: null,
+                url: '/',
+                type: 'application',
+                viewportWidth: 400,
+                viewportHeight: 600,
+            });
+            const row = rawDb
+                .prepare('SELECT entity_sha, display_name FROM editor_scenarios WHERE id = ?')
+                .get(first.scenarioId);
+            expect(row.entity_sha).toBe('keep-this');
+            expect(row.display_name).toBe('Home');
+        });
+    });
+    describe('backfillEntityShaOnScenarios', () => {
+        let db;
+        let rawDb;
+        const projectId = 'test-project-id';
+        beforeEach(async () => {
+            rawDb = new Database(':memory:');
+            db = new Kysely({ dialect: new SqliteDialect({ database: rawDb }) });
+            await db.schema
+                .createTable('editor_scenarios')
+                .addColumn('id', 'varchar', (col) => col.primaryKey())
+                .addColumn('project_id', 'varchar', (col) => col.notNull())
+                .addColumn('name', 'varchar', (col) => col.notNull())
+                .addColumn('description', 'text')
+                .addColumn('component_name', 'varchar')
+                .addColumn('component_path', 'varchar')
+                .addColumn('url', 'varchar')
+                .addColumn('type', 'varchar')
+                .addColumn('screenshot_path', 'varchar')
+                .addColumn('viewport_width', 'integer')
+                .addColumn('viewport_height', 'integer')
+                .addColumn('dimension', 'varchar')
+                .addColumn('dimensions', 'text')
+                .addColumn('screenshot_paths', 'text')
+                .addColumn('page_file_path', 'varchar')
+                .addColumn('entity_sha', 'varchar')
+                .addColumn('display_name', 'varchar')
+                .addColumn('created_at', 'datetime', (col) => col.defaultTo(new Date().toISOString()))
+                .addColumn('updated_at', 'datetime', (col) => col.defaultTo(new Date().toISOString()))
+                .execute();
+        });
+        afterEach(async () => {
+            await db.destroy();
+        });
+        it('should backfill component scenario by component_path match', async () => {
+            // Insert a scenario with null entity_sha but with component_path
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, component_name, component_path, url, type, viewport_width, viewport_height)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-1', projectId, 'Header - Default', 'Header', 'src/components/Header.tsx', '/isolated-components/Header', 'component', 1280, 720);
+            const result = await backfillEntityShaOnScenarios(db, [
+                {
+                    sha: 'entity-sha-1',
+                    name: 'Header',
+                    filePath: 'src/components/Header.tsx',
+                },
+            ]);
+            expect(result.updated).toBe(1);
+            const row = rawDb
+                .prepare('SELECT entity_sha, display_name FROM editor_scenarios WHERE id = ?')
+                .get('sc-1');
+            expect(row.entity_sha).toBe('entity-sha-1');
+            expect(row.display_name).toBe('Header');
+        });
+        it('should backfill page scenario by page_file_path match and derive display_name from route', async () => {
+            // Insert a scenario with null entity_sha but with page_file_path
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, page_file_path, url, type, viewport_width, viewport_height)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-2', projectId, 'Feedback - Default', 'app/feedback/page.tsx', '/feedback', 'application', 1280, 720);
+            const result = await backfillEntityShaOnScenarios(db, [
+                {
+                    sha: 'entity-sha-2',
+                    name: 'Feedback',
+                    filePath: 'app/feedback/page.tsx',
+                },
+            ]);
+            expect(result.updated).toBe(1);
+            const row = rawDb
+                .prepare('SELECT entity_sha, display_name FROM editor_scenarios WHERE id = ?')
+                .get('sc-2');
+            expect(row.entity_sha).toBe('entity-sha-2');
+            // display_name derived from page_file_path via routeDisplayName(buildRoutePattern())
+            expect(row.display_name).toBe('Feedback');
+        });
+        it('should update scenarios with stale entity_sha to latest', async () => {
+            // Margo bug: entity code changes create new versions with new SHAs,
+            // but scenarios still point to the old SHA. Since journals preserve
+            // screenshots and everything is in git, we always want scenarios
+            // pointing to the latest entity version.
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, component_name, component_path, url, type, viewport_width, viewport_height, entity_sha, display_name)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-3', projectId, 'Header - Dark', 'Header', 'src/components/Header.tsx', '/isolated-components/Header', 'component', 1280, 720, 'old-sha', 'Header');
+            const result = await backfillEntityShaOnScenarios(db, [
+                {
+                    sha: 'latest-sha',
+                    name: 'Header',
+                    filePath: 'src/components/Header.tsx',
+                },
+            ]);
+            expect(result.updated).toBe(1);
+            const row = rawDb
+                .prepare('SELECT entity_sha FROM editor_scenarios WHERE id = ?')
+                .get('sc-3');
+            expect(row.entity_sha).toBe('latest-sha');
+        });
+        it('should not update scenarios already pointing to the latest SHA', async () => {
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, component_name, component_path, url, type, viewport_width, viewport_height, entity_sha, display_name)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-3b', projectId, 'Header - Dark', 'Header', 'src/components/Header.tsx', '/isolated-components/Header', 'component', 1280, 720, 'current-sha', 'Header');
+            const result = await backfillEntityShaOnScenarios(db, [
+                {
+                    sha: 'current-sha',
+                    name: 'Header',
+                    filePath: 'src/components/Header.tsx',
+                },
+            ]);
+            expect(result.updated).toBe(0);
+        });
+        it('should skip scenarios with no matching entity', async () => {
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, component_name, component_path, url, type, viewport_width, viewport_height)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-4', projectId, 'Footer - Default', 'Footer', 'src/components/Footer.tsx', '/isolated-components/Footer', 'component', 1280, 720);
+            // Entities list doesn't include Footer.tsx
+            const result = await backfillEntityShaOnScenarios(db, [
+                {
+                    sha: 'entity-sha-1',
+                    name: 'Header',
+                    filePath: 'src/components/Header.tsx',
+                },
+            ]);
+            expect(result.updated).toBe(0);
+            const row = rawDb
+                .prepare('SELECT entity_sha FROM editor_scenarios WHERE id = ?')
+                .get('sc-4');
+            expect(row.entity_sha).toBeNull();
+        });
+        it('should match component scenarios by name+path, not just path (multi-entity files)', async () => {
+            // A single file can export multiple components (e.g., FullPageLibrary.tsx
+            // exports FullPageLibrary, FullPageEmptyState, FullPageArticleCard).
+            // Each component scenario should get the SHA for its specific entity,
+            // not whichever entity happens to be last in the map.
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, component_name, component_path, url, type, viewport_width, viewport_height, entity_sha)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-fplib', projectId, 'FullPageLibrary - Default', 'FullPageLibrary', 'src/library/FullPageLibrary.tsx', '/isolated-components/FullPageLibrary', 'component', 1280, 720, 'old-fplib-sha');
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, component_name, component_path, url, type, viewport_width, viewport_height, entity_sha)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-fpempty', projectId, 'FullPageEmptyState - Default', 'FullPageEmptyState', 'src/library/FullPageLibrary.tsx', '/isolated-components/FullPageEmptyState', 'component', 1280, 720, 'old-fpempty-sha');
+            const result = await backfillEntityShaOnScenarios(db, [
+                {
+                    sha: 'new-fplib-sha',
+                    name: 'FullPageLibrary',
+                    filePath: 'src/library/FullPageLibrary.tsx',
+                },
+                {
+                    sha: 'new-fpempty-sha',
+                    name: 'FullPageEmptyState',
+                    filePath: 'src/library/FullPageLibrary.tsx',
+                },
+            ]);
+            expect(result.updated).toBe(2);
+            const fplib = rawDb
+                .prepare('SELECT entity_sha FROM editor_scenarios WHERE id = ?')
+                .get('sc-fplib');
+            expect(fplib.entity_sha).toBe('new-fplib-sha');
+            const fpempty = rawDb
+                .prepare('SELECT entity_sha FROM editor_scenarios WHERE id = ?')
+                .get('sc-fpempty');
+            expect(fpempty.entity_sha).toBe('new-fpempty-sha');
+        });
+    });
+    describe('countScenariosNeedingEntityBackfill', () => {
+        let db;
+        let rawDb;
+        const projectId = 'test-project-id';
+        beforeEach(async () => {
+            rawDb = new Database(':memory:');
+            db = new Kysely({ dialect: new SqliteDialect({ database: rawDb }) });
+            await db.schema
+                .createTable('editor_scenarios')
+                .addColumn('id', 'varchar', (col) => col.primaryKey())
+                .addColumn('project_id', 'varchar', (col) => col.notNull())
+                .addColumn('name', 'varchar', (col) => col.notNull())
+                .addColumn('description', 'text')
+                .addColumn('component_name', 'varchar')
+                .addColumn('component_path', 'varchar')
+                .addColumn('url', 'varchar')
+                .addColumn('type', 'varchar')
+                .addColumn('screenshot_path', 'varchar')
+                .addColumn('viewport_width', 'integer')
+                .addColumn('viewport_height', 'integer')
+                .addColumn('dimension', 'varchar')
+                .addColumn('dimensions', 'text')
+                .addColumn('screenshot_paths', 'text')
+                .addColumn('page_file_path', 'varchar')
+                .addColumn('entity_sha', 'varchar')
+                .addColumn('display_name', 'varchar')
+                .addColumn('created_at', 'datetime', (col) => col.defaultTo(new Date().toISOString()))
+                .addColumn('updated_at', 'datetime', (col) => col.defaultTo(new Date().toISOString()))
+                .execute();
+        });
+        afterEach(async () => {
+            await db.destroy();
+        });
+        it('should NOT count scenarios that already have entity_sha', async () => {
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, component_name, component_path, url, type, viewport_width, viewport_height, entity_sha)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-1', projectId, 'Header - Default', 'Header', 'src/components/Header.tsx', '/isolated-components/Header', 'component', 1280, 720, 'existing-sha');
+            const count = await countScenariosNeedingEntityBackfill(db);
+            expect(count).toBe(0);
+        });
+        it('should count scenarios with null entity_sha and page_file_path', async () => {
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, page_file_path, url, type, viewport_width, viewport_height)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-2', projectId, 'Home - Default', 'app/page.tsx', '/', 'application', 1280, 720);
+            const count = await countScenariosNeedingEntityBackfill(db);
+            expect(count).toBe(1);
+        });
+        it('should count scenarios with null entity_sha and component_path', async () => {
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, component_name, component_path, url, type, viewport_width, viewport_height)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-3', projectId, 'Footer - Default', 'Footer', 'src/components/Footer.tsx', '/isolated-components/Footer', 'component', 1280, 720);
+            const count = await countScenariosNeedingEntityBackfill(db);
+            expect(count).toBe(1);
+        });
+        it('should NOT count scenarios with null entity_sha and no file paths', async () => {
+            rawDb
+                .prepare(`INSERT INTO editor_scenarios (id, project_id, name, url, type, viewport_width, viewport_height)
+           VALUES (?, ?, ?, ?, ?, ?, ?)`)
+                .run('sc-4', projectId, 'No File Path', '/', 'application', 1280, 720);
+            const count = await countScenariosNeedingEntityBackfill(db);
+            expect(count).toBe(0);
+        });
+    });
+    describe('validateEntityLinkageForAppScenario', () => {
+        let tmpDir;
+        let glossaryPath;
+        beforeEach(() => {
+            tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'entity-linkage-'));
+            const codeyamDir = path.join(tmpDir, '.codeyam');
+            fs.mkdirSync(codeyamDir, { recursive: true });
+            glossaryPath = path.join(codeyamDir, 'glossary.json');
+        });
+        afterEach(() => {
+            fs.rmSync(tmpDir, { recursive: true, force: true });
+        });
+        it('should return error when file is not in glossary', () => {
+            // Write a glossary that does NOT include the target file
+            fs.writeFileSync(glossaryPath, JSON.stringify([
+                { name: 'Header', filePath: 'src/components/Header.tsx' },
+            ]));
+            const result = validateEntityLinkageForAppScenario({
+                lookupFilePath: 'src/App.tsx',
+                scenarioType: 'application',
+                projectRoot: tmpDir,
+            });
+            expect(result.valid).toBe(false);
+            expect(result.error).toContain('No glossary entry found');
+            expect(result.error).toContain('src/App.tsx');
+        });
+        it('should return needsAnalysis when file IS in glossary but no entity exists', () => {
+            // Write a glossary that includes the target file
+            fs.writeFileSync(glossaryPath, JSON.stringify([
+                { name: 'App', filePath: 'src/App.tsx', description: 'Main app' },
+            ]));
+            const result = validateEntityLinkageForAppScenario({
+                lookupFilePath: 'src/App.tsx',
+                scenarioType: 'application',
+                projectRoot: tmpDir,
+            });
+            expect(result.valid).toBe(true);
+            expect(result.needsAnalysis).toBe(true);
+        });
+        it('should skip validation for component-type scenarios', () => {
+            // No glossary file exists at all
+            const result = validateEntityLinkageForAppScenario({
+                lookupFilePath: 'src/components/Header.tsx',
+                scenarioType: 'component',
+                projectRoot: tmpDir,
+            });
+            expect(result.valid).toBe(true);
+            expect(result.needsAnalysis).toBeUndefined();
+        });
+        it('should skip validation when lookupFilePath is null', () => {
+            const result = validateEntityLinkageForAppScenario({
+                lookupFilePath: null,
+                scenarioType: 'application',
+                projectRoot: tmpDir,
+            });
+            expect(result.valid).toBe(true);
+            expect(result.needsAnalysis).toBeUndefined();
+        });
+        it('should handle glossary wrapped in object format', () => {
+            // LLMs sometimes write glossary as {"components": [...]}
+            fs.writeFileSync(glossaryPath, JSON.stringify({
+                components: [
+                    { name: 'App', filePath: 'src/App.tsx', description: 'Main app' },
+                ],
+            }));
+            const result = validateEntityLinkageForAppScenario({
+                lookupFilePath: 'src/App.tsx',
+                scenarioType: 'user',
+                projectRoot: tmpDir,
+            });
+            expect(result.valid).toBe(true);
+            expect(result.needsAnalysis).toBe(true);
+        });
+        it('should return error when glossary file does not exist', () => {
+            // Don't create a glossary file
+            const result = validateEntityLinkageForAppScenario({
+                lookupFilePath: 'src/App.tsx',
+                scenarioType: 'application',
+                projectRoot: tmpDir,
+            });
+            expect(result.valid).toBe(false);
+            expect(result.error).toContain('No glossary entry found');
+        });
+    });
 });
 //# sourceMappingURL=editorScenarios.test.js.map