@codeyam/codeyam-cli 0.1.0-staging.ae0de75 → 0.1.0-staging.b147f46

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

@@ -0,0 +1,294 @@
+import * as fs from 'fs';
+import * as path from 'path';
+import * as os from 'os';
+import { extractImageUrls, resolveImageUrl, parseVerifyImagesInput, buildVerifyImagesReport, extractImageUrlsFromScenarioFiles, } from "../editorImageVerifier.js";
+describe('extractImageUrls', () => {
+    it('should extract src from img tags', () => {
+        const html = '<img src="https://example.com/photo.jpg" alt="test">';
+        expect(extractImageUrls(html)).toContain('https://example.com/photo.jpg');
+    });
+    it('should extract src with single quotes', () => {
+        const html = "<img src='https://example.com/photo.jpg'>";
+        expect(extractImageUrls(html)).toContain('https://example.com/photo.jpg');
+    });
+    it('should extract multiple img tags', () => {
+        const html = `
+      <img src="https://example.com/a.jpg">
+      <p>some text</p>
+      <img src="https://example.com/b.png">
+    `;
+        const urls = extractImageUrls(html);
+        expect(urls).toContain('https://example.com/a.jpg');
+        expect(urls).toContain('https://example.com/b.png');
+    });
+    it('should extract relative src paths', () => {
+        const html = '<img src="/images/logo.png">';
+        expect(extractImageUrls(html)).toContain('/images/logo.png');
+    });
+    it('should extract URLs ending in image extensions from JSON data', () => {
+        // Next.js SSR pages embed data as JSON in <script> tags
+        const html = `
+      <script id="__NEXT_DATA__">
+        {"props":{"pageProps":{"drink":{"imageUrl":"https://images.unsplash.com/photo-123?w=400","name":"Earl Grey"}}}}
+      </script>
+    `;
+        const urls = extractImageUrls(html);
+        expect(urls).toContain('https://images.unsplash.com/photo-123?w=400');
+    });
+    it('should extract URLs with common image extensions from inline data', () => {
+        const html = `{"avatar":"https://i.pravatar.cc/150?u=sarah","bgImage":"https://cdn.example.com/bg.webp"}`;
+        const urls = extractImageUrls(html);
+        expect(urls).toContain('https://i.pravatar.cc/150?u=sarah');
+        expect(urls).toContain('https://cdn.example.com/bg.webp');
+    });
+    it('should deduplicate URLs', () => {
+        const html = `
+      <img src="https://example.com/photo.jpg">
+      <img src="https://example.com/photo.jpg">
+    `;
+        const urls = extractImageUrls(html);
+        expect(urls.filter((u) => u === 'https://example.com/photo.jpg')).toHaveLength(1);
+    });
+    it('should skip data: URIs', () => {
+        const html = '<img src="data:image/png;base64,iVBOR...">';
+        expect(extractImageUrls(html)).toHaveLength(0);
+    });
+    it('should skip empty src', () => {
+        const html = '<img src="">';
+        expect(extractImageUrls(html)).toHaveLength(0);
+    });
+    it('should handle srcset attribute', () => {
+        const html = '<img srcset="https://example.com/small.jpg 1x, https://example.com/large.jpg 2x">';
+        const urls = extractImageUrls(html);
+        expect(urls).toContain('https://example.com/small.jpg');
+        expect(urls).toContain('https://example.com/large.jpg');
+    });
+    it('should handle img tags with attributes before src', () => {
+        const html = '<img class="rounded" loading="lazy" src="https://example.com/photo.jpg">';
+        expect(extractImageUrls(html)).toContain('https://example.com/photo.jpg');
+    });
+    it('should return empty array for HTML with no images', () => {
+        const html = '<div><p>No images here</p></div>';
+        expect(extractImageUrls(html)).toEqual([]);
+    });
+    it('should extract pravatar-style URLs without file extension', () => {
+        // These don't end in .jpg/.png but are image URLs referenced in JSON
+        const html = `{"userImage":"https://i.pravatar.cc/150?u=marcus"}`;
+        expect(extractImageUrls(html)).toContain('https://i.pravatar.cc/150?u=marcus');
+    });
+});
+describe('resolveImageUrl', () => {
+    it('should return absolute URLs unchanged', () => {
+        expect(resolveImageUrl('https://example.com/photo.jpg', 'http://localhost:3000')).toBe('https://example.com/photo.jpg');
+    });
+    it('should resolve root-relative paths', () => {
+        expect(resolveImageUrl('/images/logo.png', 'http://localhost:3000')).toBe('http://localhost:3000/images/logo.png');
+    });
+    it('should resolve relative paths', () => {
+        expect(resolveImageUrl('photo.jpg', 'http://localhost:3000/page')).toBe('http://localhost:3000/photo.jpg');
+    });
+    it('should handle protocol-relative URLs', () => {
+        expect(resolveImageUrl('//cdn.example.com/img.png', 'http://localhost:3000')).toBe('http://cdn.example.com/img.png');
+    });
+});
+describe('parseVerifyImagesInput', () => {
+    it('should return defaults when no argument provided', () => {
+        const result = parseVerifyImagesInput('');
+        expect(result.paths).toEqual(['/']);
+        expect(result.imageUrls).toEqual([]);
+    });
+    it('should parse paths only', () => {
+        const result = parseVerifyImagesInput('{"paths":["/","/about"]}');
+        expect(result.paths).toEqual(['/', '/about']);
+        expect(result.imageUrls).toEqual([]);
+    });
+    it('should parse imageUrls only', () => {
+        const result = parseVerifyImagesInput('{"imageUrls":["https://images.unsplash.com/photo-123?w=400"]}');
+        expect(result.paths).toEqual(['/']);
+        expect(result.imageUrls).toEqual([
+            'https://images.unsplash.com/photo-123?w=400',
+        ]);
+    });
+    it('should parse both paths and imageUrls', () => {
+        const result = parseVerifyImagesInput('{"paths":["/drinks"],"imageUrls":["https://i.pravatar.cc/150?u=alice","https://images.unsplash.com/photo-456"]}');
+        expect(result.paths).toEqual(['/drinks']);
+        expect(result.imageUrls).toEqual([
+            'https://i.pravatar.cc/150?u=alice',
+            'https://images.unsplash.com/photo-456',
+        ]);
+    });
+    it('should ignore non-array imageUrls', () => {
+        const result = parseVerifyImagesInput('{"imageUrls":"not-an-array"}');
+        expect(result.imageUrls).toEqual([]);
+    });
+    it('should throw on invalid JSON', () => {
+        expect(() => parseVerifyImagesInput('not json')).toThrow();
+    });
+});
+describe('buildVerifyImagesReport', () => {
+    it('should report zero images when none found', () => {
+        const report = buildVerifyImagesReport({
+            pagesChecked: 2,
+            imageUrls: [],
+            results: [],
+        });
+        expect(report.pagesChecked).toBe(2);
+        expect(report.imagesFound).toBe(0);
+        expect(report.imagesVerified).toBe(0);
+        expect(report.failures).toEqual([]);
+    });
+    it('should count verified images', () => {
+        const results = [
+            { url: 'https://example.com/a.jpg', status: 200, ok: true },
+            { url: 'https://example.com/b.png', status: 200, ok: true },
+        ];
+        const report = buildVerifyImagesReport({
+            pagesChecked: 1,
+            imageUrls: ['https://example.com/a.jpg', 'https://example.com/b.png'],
+            results,
+        });
+        expect(report.imagesFound).toBe(2);
+        expect(report.imagesVerified).toBe(2);
+        expect(report.failures).toEqual([]);
+    });
+    it('should include failures with status and error', () => {
+        const results = [
+            { url: 'https://example.com/good.jpg', status: 200, ok: true },
+            { url: 'https://example.com/missing.jpg', status: 404, ok: false },
+            {
+                url: 'https://example.com/down.jpg',
+                status: null,
+                ok: false,
+                error: 'ECONNREFUSED',
+            },
+        ];
+        const report = buildVerifyImagesReport({
+            pagesChecked: 1,
+            imageUrls: results.map((r) => r.url),
+            results,
+        });
+        expect(report.imagesFound).toBe(3);
+        expect(report.imagesVerified).toBe(1);
+        expect(report.failures).toHaveLength(2);
+        expect(report.failures[0]).toEqual({
+            url: 'https://example.com/missing.jpg',
+            status: 404,
+            error: undefined,
+        });
+        expect(report.failures[1]).toEqual({
+            url: 'https://example.com/down.jpg',
+            status: null,
+            error: 'ECONNREFUSED',
+        });
+    });
+    it('should include scenarioFilesScanned when provided', () => {
+        const report = buildVerifyImagesReport({
+            pagesChecked: 1,
+            imageUrls: ['https://example.com/a.jpg'],
+            results: [{ url: 'https://example.com/a.jpg', status: 200, ok: true }],
+            scenarioFilesScanned: 3,
+        });
+        expect(report.scenarioFilesScanned).toBe(3);
+    });
+    it('should omit scenarioFilesScanned when not provided', () => {
+        const report = buildVerifyImagesReport({
+            pagesChecked: 1,
+            imageUrls: [],
+            results: [],
+        });
+        expect(report.scenarioFilesScanned).toBeUndefined();
+    });
+});
+function makeTmpDir() {
+    return fs.mkdtempSync(path.join(os.tmpdir(), 'image-verifier-test-'));
+}
+describe('extractImageUrlsFromScenarioFiles', () => {
+    it('should extract image URLs from scenario JSON with routes', () => {
+        const dir = makeTmpDir();
+        fs.writeFileSync(path.join(dir, 'scenario-1.json'), JSON.stringify({
+            routes: [
+                {
+                    path: '/api/drinks',
+                    response: {
+                        body: [
+                            {
+                                name: 'Earl Grey',
+                                imageUrl: 'https://images.unsplash.com/photo-123?w=400',
+                            },
+                        ],
+                    },
+                },
+            ],
+        }));
+        const result = extractImageUrlsFromScenarioFiles(dir);
+        expect(result.urls).toContain('https://images.unsplash.com/photo-123?w=400');
+        expect(result.filesScanned).toBe(1);
+    });
+    it('should extract from multiple scenario files', () => {
+        const dir = makeTmpDir();
+        fs.writeFileSync(path.join(dir, 'a.json'), JSON.stringify({
+            avatar: 'https://i.pravatar.cc/150?u=alice',
+        }));
+        fs.writeFileSync(path.join(dir, 'b.json'), JSON.stringify({
+            image: 'https://images.unsplash.com/photo-456?w=800',
+        }));
+        const result = extractImageUrlsFromScenarioFiles(dir);
+        expect(result.urls).toContain('https://i.pravatar.cc/150?u=alice');
+        expect(result.urls).toContain('https://images.unsplash.com/photo-456?w=800');
+        expect(result.filesScanned).toBe(2);
+    });
+    it('should deduplicate URLs across files', () => {
+        const dir = makeTmpDir();
+        const sharedUrl = 'https://images.unsplash.com/photo-same?w=400';
+        fs.writeFileSync(path.join(dir, 'a.json'), JSON.stringify({ image: sharedUrl }));
+        fs.writeFileSync(path.join(dir, 'b.json'), JSON.stringify({ image: sharedUrl }));
+        const result = extractImageUrlsFromScenarioFiles(dir);
+        expect(result.urls.filter((u) => u === sharedUrl)).toHaveLength(1);
+    });
+    it('should return empty when directory does not exist', () => {
+        const result = extractImageUrlsFromScenarioFiles('/tmp/nonexistent-dir-xyz-12345');
+        expect(result.urls).toEqual([]);
+        expect(result.filesScanned).toBe(0);
+    });
+    it('should skip non-JSON files', () => {
+        const dir = makeTmpDir();
+        fs.writeFileSync(path.join(dir, 'notes.txt'), 'https://images.unsplash.com/photo-txt?w=400');
+        fs.writeFileSync(path.join(dir, 'scenario.json'), JSON.stringify({ image: 'https://images.unsplash.com/photo-json?w=400' }));
+        const result = extractImageUrlsFromScenarioFiles(dir);
+        expect(result.urls).toContain('https://images.unsplash.com/photo-json?w=400');
+        expect(result.urls).not.toContain('https://images.unsplash.com/photo-txt?w=400');
+        expect(result.filesScanned).toBe(1);
+    });
+    it('should handle scenario files with no image URLs', () => {
+        const dir = makeTmpDir();
+        fs.writeFileSync(path.join(dir, 'empty.json'), JSON.stringify({ name: 'Test', count: 42 }));
+        const result = extractImageUrlsFromScenarioFiles(dir);
+        expect(result.urls).toEqual([]);
+        expect(result.filesScanned).toBe(1);
+    });
+    it('should handle deeply nested image URLs', () => {
+        const dir = makeTmpDir();
+        fs.writeFileSync(path.join(dir, 'deep.json'), JSON.stringify({
+            routes: [
+                {
+                    response: {
+                        body: {
+                            data: {
+                                items: [
+                                    {
+                                        nested: {
+                                            avatar: 'https://i.pravatar.cc/150?u=deep',
+                                        },
+                                    },
+                                ],
+                            },
+                        },
+                    },
+                },
+            ],
+        }));
+        const result = extractImageUrlsFromScenarioFiles(dir);
+        expect(result.urls).toContain('https://i.pravatar.cc/150?u=deep');
+    });
+});
+//# sourceMappingURL=editorImageVerifier.test.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"editorImageVerifier.test.js","sourceRoot":"","sources":["../../../../../src/utils/__tests__/editorImageVerifier.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,gBAAgB,EAChB,eAAe,EACf,sBAAsB,EACtB,uBAAuB,EACvB,iCAAiC,GAClC,MAAM,wBAAwB,CAAC;AAGhC,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;QAC1C,MAAM,IAAI,GAAG,sDAAsD,CAAC;QACpE,MAAM,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,SAAS,CAAC,+BAA+B,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAC/C,MAAM,IAAI,GAAG,2CAA2C,CAAC;QACzD,MAAM,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,SAAS,CAAC,+BAA+B,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;QAC1C,MAAM,IAAI,GAAG;;;;KAIZ,CAAC;QACF,MAAM,IAAI,GAAG,gBAAgB,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,2BAA2B,CAAC,CAAC;QACpD,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,2BAA2B,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,IAAI,GAAG,8BAA8B,CAAC;QAC5C,MAAM,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;IAC/D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,GAAG,EAAE;QACvE,wDAAwD;QACxD,MAAM,IAAI,GAAG;;;;KAIZ,CAAC;QACF,MAAM,IAAI,GAAG,gBAAgB,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,6CAA6C,CAAC,CAAC;IACxE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mEAAmE,EAAE,GAAG,EAAE;QAC3E,MAAM,IAAI,GAAG,4FAA4F,CAAC;QAC1G,MAAM,IAAI,GAAG,gBAAgB,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,mCAAmC,CAAC,CAAC;QAC5D,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,iCAAiC,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yBAAyB,EAAE,GAAG,EAAE;QACjC,MAAM,IAAI,GAAG;;;KAGZ,CAAC;QACF,MAAM,IAAI,GAAG,gBAAgB,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,CACJ,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,+BAA+B,CAAC,CAC1D,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IACpB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wBAAwB,EAAE,GAAG,EAAE;QAChC,MAAM,IAAI,GAAG,4CAA4C,CAAC;QAC1D,MAAM,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uBAAuB,EAAE,GAAG,EAAE;QAC/B,MAAM,IAAI,GAAG,cAAc,CAAC;QAC5B,MAAM,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,GAAG,EAAE;QACxC,MAAM,IAAI,GACR,mFAAmF,CAAC;QACtF,MAAM,IAAI,GAAG,gBAAgB,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,+BAA+B,CAAC,CAAC;QACxD,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,+BAA+B,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC3D,MAAM,IAAI,GACR,0EAA0E,CAAC;QAC7E,MAAM,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,SAAS,CAAC,+BAA+B,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC3D,MAAM,IAAI,GAAG,kCAAkC,CAAC;QAChD,MAAM,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,qEAAqE;QACrE,MAAM,IAAI,GAAG,oDAAoD,CAAC;QAClE,MAAM,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,SAAS,CACtC,oCAAoC,CACrC,CAAC;IACJ,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAC/C,MAAM,CACJ,eAAe,CAAC,+BAA+B,EAAE,uBAAuB,CAAC,CAC1E,CAAC,IAAI,CAAC,+BAA+B,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,CAAC,eAAe,CAAC,kBAAkB,EAAE,uBAAuB,CAAC,CAAC,CAAC,IAAI,CACvE,uCAAuC,CACxC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+BAA+B,EAAE,GAAG,EAAE;QACvC,MAAM,CAAC,eAAe,CAAC,WAAW,EAAE,4BAA4B,CAAC,CAAC,CAAC,IAAI,CACrE,iCAAiC,CAClC,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC9C,MAAM,CACJ,eAAe,CAAC,2BAA2B,EAAE,uBAAuB,CAAC,CACtE,CAAC,IAAI,CAAC,gCAAgC,CAAC,CAAC;IAC3C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,wBAAwB,EAAE,GAAG,EAAE;IACtC,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,MAAM,GAAG,sBAAsB,CAAC,EAAE,CAAC,CAAC;QAC1C,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;QACpC,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yBAAyB,EAAE,GAAG,EAAE;QACjC,MAAM,MAAM,GAAG,sBAAsB,CAAC,0BAA0B,CAAC,CAAC;QAClE,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,GAAG,EAAE,QAAQ,CAAC,CAAC,CAAC;QAC9C,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,MAAM,GAAG,sBAAsB,CACnC,+DAA+D,CAChE,CAAC;QACF,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;QACpC,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC;YAC/B,6CAA6C;SAC9C,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;QAC/C,MAAM,MAAM,GAAG,sBAAsB,CACnC,iHAAiH,CAClH,CAAC;QACF,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC;QAC1C,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC;YAC/B,mCAAmC;YACnC,uCAAuC;SACxC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,MAAM,GAAG,sBAAsB,CAAC,8BAA8B,CAAC,CAAC;QACtE,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8BAA8B,EAAE,GAAG,EAAE;QACtC,MAAM,CAAC,GAAG,EAAE,CAAC,sBAAsB,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;IAC7D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;IACvC,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,MAAM,MAAM,GAAG,uBAAuB,CAAC;YACrC,YAAY,EAAE,CAAC;YACf,SAAS,EAAE,EAAE;YACb,OAAO,EAAE,EAAE;SACZ,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACpC,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACnC,MAAM,CAAC,MAAM,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACtC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8BAA8B,EAAE,GAAG,EAAE;QACtC,MAAM,OAAO,GAAwB;YACnC,EAAE,GAAG,EAAE,2BAA2B,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,EAAE,IAAI,EAAE;YAC3D,EAAE,GAAG,EAAE,2BAA2B,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,EAAE,IAAI,EAAE;SAC5D,CAAC;QACF,MAAM,MAAM,GAAG,uBAAuB,CAAC;YACrC,YAAY,EAAE,CAAC;YACf,SAAS,EAAE,CAAC,2BAA2B,EAAE,2BAA2B,CAAC;YACrE,OAAO;SACR,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACnC,MAAM,CAAC,MAAM,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACtC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,MAAM,OAAO,GAAwB;YACnC,EAAE,GAAG,EAAE,8BAA8B,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,EAAE,IAAI,EAAE;YAC9D,EAAE,GAAG,EAAE,iCAAiC,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,EAAE,KAAK,EAAE;YAClE;gBACE,GAAG,EAAE,8BAA8B;gBACnC,MAAM,EAAE,IAAI;gBACZ,EAAE,EAAE,KAAK;gBACT,KAAK,EAAE,cAAc;aACtB;SACF,CAAC;QACF,MAAM,MAAM,GAAG,uBAAuB,CAAC;YACrC,YAAY,EAAE,CAAC;YACf,SAAS,EAAE,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC;YACpC,OAAO;SACR,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACnC,MAAM,CAAC,MAAM,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACtC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;YACjC,GAAG,EAAE,iCAAiC;YACtC,MAAM,EAAE,GAAG;YACX,KAAK,EAAE,SAAS;SACjB,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;YACjC,GAAG,EAAE,8BAA8B;YACnC,MAAM,EAAE,IAAI;YACZ,KAAK,EAAE,cAAc;SACtB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC3D,MAAM,MAAM,GAAG,uBAAuB,CAAC;YACrC,YAAY,EAAE,CAAC;YACf,SAAS,EAAE,CAAC,2BAA2B,CAAC;YACxC,OAAO,EAAE,CAAC,EAAE,GAAG,EAAE,2BAA2B,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC;YACtE,oBAAoB,EAAE,CAAC;SACxB,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC5D,MAAM,MAAM,GAAG,uBAAuB,CAAC;YACrC,YAAY,EAAE,CAAC;YACf,SAAS,EAAE,EAAE;YACb,OAAO,EAAE,EAAE;SACZ,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAC,aAAa,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,SAAS,UAAU;IACjB,OAAO,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,sBAAsB,CAAC,CAAC,CAAC;AACxE,CAAC;AAED,QAAQ,CAAC,mCAAmC,EAAE,GAAG,EAAE;IACjD,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;QACzB,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,iBAAiB,CAAC,EACjC,IAAI,CAAC,SAAS,CAAC;YACb,MAAM,EAAE;gBACN;oBACE,IAAI,EAAE,aAAa;oBACnB,QAAQ,EAAE;wBACR,IAAI,EAAE;4BACJ;gCACE,IAAI,EAAE,WAAW;gCACjB,QAAQ,EAAE,6CAA6C;6BACxD;yBACF;qBACF;iBACF;aACF;SACF,CAAC,CACH,CAAC;QACF,MAAM,MAAM,GAAG,iCAAiC,CAAC,GAAG,CAAC,CAAC;QACtD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAC3B,6CAA6C,CAC9C,CAAC;QACF,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;QACzB,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,QAAQ,CAAC,EACxB,IAAI,CAAC,SAAS,CAAC;YACb,MAAM,EAAE,mCAAmC;SAC5C,CAAC,CACH,CAAC;QACF,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,QAAQ,CAAC,EACxB,IAAI,CAAC,SAAS,CAAC;YACb,KAAK,EAAE,6CAA6C;SACrD,CAAC,CACH,CAAC;QACF,MAAM,MAAM,GAAG,iCAAiC,CAAC,GAAG,CAAC,CAAC;QACtD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,mCAAmC,CAAC,CAAC;QACnE,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAC3B,6CAA6C,CAC9C,CAAC;QACF,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC9C,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;QACzB,MAAM,SAAS,GAAG,8CAA8C,CAAC;QACjE,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,QAAQ,CAAC,EACxB,IAAI,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,SAAS,EAAE,CAAC,CACrC,CAAC;QACF,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,QAAQ,CAAC,EACxB,IAAI,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,SAAS,EAAE,CAAC,CACrC,CAAC;QACF,MAAM,MAAM,GAAG,iCAAiC,CAAC,GAAG,CAAC,CAAC;QACtD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,SAAS,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IACrE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC3D,MAAM,MAAM,GAAG,iCAAiC,CAC9C,gCAAgC,CACjC,CAAC;QACF,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;QACpC,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;QACzB,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,WAAW,CAAC,EAC3B,6CAA6C,CAC9C,CAAC;QACF,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,eAAe,CAAC,EAC/B,IAAI,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,8CAA8C,EAAE,CAAC,CAC1E,CAAC;QACF,MAAM,MAAM,GAAG,iCAAiC,CAAC,GAAG,CAAC,CAAC;QACtD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAC3B,8CAA8C,CAC/C,CAAC;QACF,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,SAAS,CAC/B,6CAA6C,CAC9C,CAAC;QACF,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;QACzB,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,YAAY,CAAC,EAC5B,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,EAAE,EAAE,CAAC,CAC5C,CAAC;QACF,MAAM,MAAM,GAAG,iCAAiC,CAAC,GAAG,CAAC,CAAC;QACtD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,GAAG,GAAG,UAAU,EAAE,CAAC;QACzB,EAAE,CAAC,aAAa,CACd,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,WAAW,CAAC,EAC3B,IAAI,CAAC,SAAS,CAAC;YACb,MAAM,EAAE;gBACN;oBACE,QAAQ,EAAE;wBACR,IAAI,EAAE;4BACJ,IAAI,EAAE;gCACJ,KAAK,EAAE;oCACL;wCACE,MAAM,EAAE;4CACN,MAAM,EAAE,kCAAkC;yCAC3C;qCACF;iCACF;6BACF;yBACF;qBACF;iBACF;aACF;SACF,CAAC,CACH,CAAC;QACF,MAAM,MAAM,GAAG,iCAAiC,CAAC,GAAG,CAAC,CAAC;QACtD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,kCAAkC,CAAC,CAAC;IACpE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

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

@@ -1,4 +1,4 @@
-import { groupEntriesByDate, groupScreenshotsByComponent, generateJournalScreenshotFilename, buildJournalMarkdown, findJournalEntryByTime, applyJournalEntryUpdates, insertCommitIntoMarkdown, } from "../editorJournal.js";
+import { groupEntriesByDate, groupScreenshotsByComponent, generateJournalScreenshotFilename, buildJournalMarkdown, findJournalEntryByTime, applyJournalEntryUpdates, insertCommitIntoMarkdown, isJournalEntryCommitted, computeTimeframeSummary, } from "../editorJournal.js";
 describe('editorJournal', () => {
     describe('groupEntriesByDate', () => {
         it('should group entries by date with newest first', () => {
@@ -21,7 +21,7 @@ describe('editorJournal', () => {
         });
     });
     describe('groupScreenshotsByComponent', () => {
-        it('should group by component prefix (before " - ")', () => {
+        it('should group by component prefix (before " - ") for legacy data', () => {
             const screenshots = [
                 { name: 'DrinkCard - Default', path: 'a.png' },
                 { name: 'DrinkCard - Dark Mode', path: 'b.png' },
@@ -35,6 +35,40 @@ describe('editorJournal', () => {
             expect(result[1][0]).toBe('DrinkCard');
             expect(result[1][1]).toHaveLength(2);
         });
+        it('should use componentName field when available', () => {
+            const screenshots = [
+                {
+                    name: 'DrinkCard - Default',
+                    path: 'a.png',
+                    componentName: 'DrinkCard',
+                    url: '/isolated-components/DrinkCard',
+                },
+                {
+                    name: 'Full Catalog',
+                    path: 'b.png',
+                    componentName: null,
+                    url: '/',
+                },
+                {
+                    name: 'Drink Detail - With Reviews',
+                    path: 'c.png',
+                    componentName: null,
+                    url: '/drinks/34',
+                },
+            ];
+            const result = groupScreenshotsByComponent(screenshots);
+            // componentName: null → "App", componentName: 'DrinkCard' → 'DrinkCard'
+            expect(result).toHaveLength(2);
+            expect(result[0][0]).toBe('App');
+            // Both null-componentName screenshots go to App (not parsed by " - ")
+            expect(result[0][1]).toHaveLength(2);
+            expect(result[0][1].map((s) => s.name)).toEqual([
+                'Full Catalog',
+                'Drink Detail - With Reviews',
+            ]);
+            expect(result[1][0]).toBe('DrinkCard');
+            expect(result[1][1]).toHaveLength(1);
+        });
         it('should sort with "App" first, then alphabetical', () => {
             const screenshots = [
                 { name: 'Zebra - Default', path: 'z.png' },
@@ -242,6 +276,219 @@ describe('editorJournal', () => {
             ]);
         });
     });
+    describe('isJournalEntryCommitted', () => {
+        it('should return true when entry has a commitSha', () => {
+            expect(isJournalEntryCommitted({ commitSha: 'abc1234' })).toBe(true);
+        });
+        it('should return false when commitSha is null', () => {
+            expect(isJournalEntryCommitted({ commitSha: null })).toBe(false);
+        });
+        it('should return false when commitSha is undefined', () => {
+            expect(isJournalEntryCommitted({})).toBe(false);
+        });
+    });
+    describe('computeTimeframeSummary', () => {
+        // Helper to create a journal entry for tests
+        function makeEntry(overrides) {
+            return {
+                date: overrides.date,
+                time: overrides.time,
+                title: 'Test Entry',
+                type: 'feature',
+                description: 'Test description',
+                scenarios: [],
+                screenshot: null,
+                scenarioScreenshots: overrides.scenarioScreenshots || [],
+                commitSha: (overrides.commitSha ?? null),
+                commitMessage: (overrides.commitSha ? 'commit msg' : null),
+            };
+        }
+        // Use a fixed "now" for deterministic tests
+        const now = new Date('2026-03-06T12:00:00Z');
+        it('should count unique commits in the period', () => {
+            const entries = [
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T10:00:00Z',
+                    commitSha: 'aaa',
+                }),
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T11:00:00Z',
+                    commitSha: 'bbb',
+                }),
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T11:30:00Z',
+                    commitSha: 'aaa',
+                }), // duplicate
+                makeEntry({
+                    date: '2026-03-05',
+                    time: '2026-03-05T10:00:00Z',
+                    commitSha: 'ccc',
+                }),
+            ];
+            const result = computeTimeframeSummary(entries, '1d', now);
+            expect(result.commitCount).toBe(3); // aaa, bbb, ccc (all within 1d of March 6)
+        });
+        it('should filter entries by period', () => {
+            const entries = [
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T10:00:00Z',
+                    commitSha: 'aaa',
+                }),
+                makeEntry({
+                    date: '2026-03-01',
+                    time: '2026-03-01T10:00:00Z',
+                    commitSha: 'bbb',
+                }),
+                makeEntry({
+                    date: '2026-02-10',
+                    time: '2026-02-10T10:00:00Z',
+                    commitSha: 'ccc',
+                }),
+            ];
+            // 3-day window from March 6: cutoff is March 3
+            const result3d = computeTimeframeSummary(entries, '3d', now);
+            expect(result3d.commitCount).toBe(1); // only 'aaa'
+            expect(result3d.entryCount).toBe(1);
+            // 7-day window: cutoff is Feb 27
+            const result7d = computeTimeframeSummary(entries, '7d', now);
+            expect(result7d.commitCount).toBe(2); // aaa, bbb
+            expect(result7d.entryCount).toBe(2);
+            // 30-day window: cutoff is Feb 4
+            const result30d = computeTimeframeSummary(entries, '30d', now);
+            expect(result30d.commitCount).toBe(3); // all three
+            expect(result30d.entryCount).toBe(3);
+        });
+        it('should group app-level scenarios separately from component scenarios', () => {
+            const entries = [
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T10:00:00Z',
+                    scenarioScreenshots: [
+                        { name: 'Home page default', path: 'screenshots/home.png' },
+                        { name: 'DrinkCard - Default', path: 'screenshots/dc-default.png' },
+                        { name: 'DrinkCard - Dark Mode', path: 'screenshots/dc-dark.png' },
+                        { name: 'ReviewList - Empty', path: 'screenshots/rl-empty.png' },
+                    ],
+                }),
+            ];
+            const result = computeTimeframeSummary(entries, '1d', now);
+            // App scenario: "Home page default" (no " - ")
+            expect(result.appScenarios).toHaveLength(1);
+            expect(result.appScenarios[0].name).toBe('Home page default');
+            // Component groups: DrinkCard (2 scenarios), ReviewList (1 scenario)
+            expect(result.componentGroups.size).toBe(2);
+            expect(result.componentGroups.get('DrinkCard')).toHaveLength(2);
+            expect(result.componentGroups.get('ReviewList')).toHaveLength(1);
+        });
+        it('should collect screenshot timeline for a scenario across entries', () => {
+            const entries = [
+                makeEntry({
+                    date: '2026-03-04',
+                    time: '2026-03-04T10:00:00Z',
+                    scenarioScreenshots: [
+                        { name: 'DrinkCard - Default', path: 'screenshots/dc-v1.png' },
+                    ],
+                }),
+                makeEntry({
+                    date: '2026-03-05',
+                    time: '2026-03-05T14:00:00Z',
+                    scenarioScreenshots: [
+                        { name: 'DrinkCard - Default', path: 'screenshots/dc-v2.png' },
+                    ],
+                }),
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T09:00:00Z',
+                    scenarioScreenshots: [
+                        { name: 'DrinkCard - Default', path: 'screenshots/dc-v3.png' },
+                    ],
+                }),
+            ];
+            const result = computeTimeframeSummary(entries, '7d', now);
+            const dcGroup = result.componentGroups.get('DrinkCard');
+            expect(dcGroup).toHaveLength(1);
+            const timeline = dcGroup[0];
+            expect(timeline.screenshots).toHaveLength(3);
+            // Should be sorted chronologically
+            expect(timeline.screenshots[0].path).toBe('screenshots/dc-v1.png');
+            expect(timeline.screenshots[1].path).toBe('screenshots/dc-v2.png');
+            expect(timeline.screenshots[2].path).toBe('screenshots/dc-v3.png');
+        });
+        it('should deduplicate screenshots by path', () => {
+            // Same scenario, same screenshot path across two entries
+            const entries = [
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T10:00:00Z',
+                    scenarioScreenshots: [
+                        { name: 'DrinkCard - Default', path: 'screenshots/dc.png' },
+                    ],
+                }),
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T11:00:00Z',
+                    scenarioScreenshots: [
+                        { name: 'DrinkCard - Default', path: 'screenshots/dc.png' }, // same path
+                    ],
+                }),
+            ];
+            const result = computeTimeframeSummary(entries, '1d', now);
+            const dcGroup = result.componentGroups.get('DrinkCard');
+            expect(dcGroup[0].screenshots).toHaveLength(1); // deduped
+        });
+        it('should return zero counts for empty entries', () => {
+            const result = computeTimeframeSummary([], '7d', now);
+            expect(result.commitCount).toBe(0);
+            expect(result.entryCount).toBe(0);
+            expect(result.totalScenarios).toBe(0);
+            expect(result.appScenarios).toHaveLength(0);
+            expect(result.componentGroups.size).toBe(0);
+        });
+        it('should not count null commitSha values', () => {
+            const entries = [
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T10:00:00Z',
+                    commitSha: null,
+                }),
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T11:00:00Z',
+                    commitSha: 'aaa',
+                }),
+            ];
+            const result = computeTimeframeSummary(entries, '1d', now);
+            expect(result.commitCount).toBe(1);
+        });
+        it('should report totalScenarios as the count of unique scenario names', () => {
+            const entries = [
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T10:00:00Z',
+                    scenarioScreenshots: [
+                        { name: 'Home', path: 'screenshots/home-v1.png' },
+                        { name: 'DrinkCard - Default', path: 'screenshots/dc-v1.png' },
+                    ],
+                }),
+                makeEntry({
+                    date: '2026-03-06',
+                    time: '2026-03-06T11:00:00Z',
+                    scenarioScreenshots: [
+                        { name: 'Home', path: 'screenshots/home-v2.png' },
+                        { name: 'DrinkCard - Default', path: 'screenshots/dc-v2.png' },
+                        { name: 'DrinkCard - Dark', path: 'screenshots/dc-dark.png' },
+                    ],
+                }),
+            ];
+            const result = computeTimeframeSummary(entries, '1d', now);
+            // 3 unique scenario names: Home, DrinkCard - Default, DrinkCard - Dark
+            expect(result.totalScenarios).toBe(3);
+        });
+    });
     describe('insertCommitIntoMarkdown', () => {
         it('should insert commit line before next section separator', () => {
             const content = [