toiljs 0.0.15 → 0.0.19

package/test/seo.test.ts

@@ -1,164 +1,175 @@
-import { describe, expect, it } from 'vitest';
-
-import type { ScannedRoute } from '../src/compiler/routes';
-import {
-    injectSeoHtml,
-    llmsTxt,
-    robotsTxt,
-    routeSeo,
-    seoHeadTags,
-    sitemapXml,
-} from '../src/compiler/seo';
-
-const routes: ScannedRoute[] = [
-    { file: 'a', pattern: '/' },
-    { file: 'b', pattern: '/about' },
-    { file: 'c', pattern: '/blog/:id' }, // dynamic, excluded from sitemap
-    { file: 'd', pattern: '/photo/:id', slot: 'modal', intercept: true }, // slot/intercept, excluded
-];
-
-describe('seoHeadTags', () => {
-    it('bakes description, OG, canonical, preconnect, and JSON-LD', () => {
-        const html = seoHeadTags({
-            url: 'https://x.test',
-            title: 'Home',
-            description: 'desc',
-            openGraph: { type: 'website', image: 'https://x.test/og.png' },
-            preconnect: ['https://cdn.test'],
-            jsonLd: { '@context': 'https://schema.org', '@type': 'WebSite' },
-        });
-        expect(html).toContain('<meta name="description" content="desc" />');
-        expect(html).toContain('<meta property="og:title" content="Home" />');
-        expect(html).toContain('<meta property="og:image" content="https://x.test/og.png" />');
-        expect(html).toContain('<link rel="canonical" href="https://x.test" />');
-        expect(html).toContain('<link rel="preconnect" href="https://cdn.test" />');
-        expect(html).toContain('application/ld+json');
-        expect(html).toContain('"@type":"WebSite"');
-    });
-
-    it('escapes attribute values', () => {
-        expect(seoHeadTags({ description: 'a "b" <c>' })).toContain(
-            'content="a &quot;b&quot; &lt;c&gt;"',
-        );
-    });
-
-    it('renders a full Twitter card + OG image dimensions + fb:app_id', () => {
-        const html = seoHeadTags({
-            title: 'Home',
-            description: 'd',
-            openGraph: {
-                image: 'https://x.test/og.png',
-                imageAlt: 'alt',
-                imageWidth: 1200,
-                imageHeight: 630,
-            },
-            twitter: { site: '@x' },
-            facebook: { appId: '123' },
-        });
-        expect(html).toContain('<meta name="twitter:card" content="summary_large_image" />');
-        expect(html).toContain('<meta name="twitter:site" content="@x" />');
-        expect(html).toContain('<meta name="twitter:title" content="Home" />');
-        expect(html).toContain('<meta name="twitter:image" content="https://x.test/og.png" />');
-        expect(html).toContain('<meta property="og:image:width" content="1200" />');
-        expect(html).toContain('<meta property="og:image:alt" content="alt" />');
-        expect(html).toContain('<meta property="fb:app_id" content="123" />');
-    });
-
-    it('neutralizes </script> in JSON-LD (no script breakout)', () => {
-        const html = seoHeadTags({ jsonLd: { x: '</script><img src=x onerror=alert(1)>' } });
-        expect(html).not.toContain('</script><img');
-        expect(html).toContain('\\u003c/script');
-    });
-});
-
-describe('routeSeo', () => {
-    it("overlays a route's metadata over the site defaults and points URLs at the route", () => {
-        const site = { url: 'https://x.test', title: 'Site', description: 'site desc' };
-        const out = routeSeo(site, { title: 'About', description: 'about desc' }, '/about');
-        expect(out.title).toBe('About');
-        expect(out.description).toBe('about desc');
-        expect(out.url).toBe('https://x.test/about');
-    });
-
-    it('falls back to the site defaults when a route has no metadata', () => {
-        const site = { url: 'https://x.test', title: 'Site' };
-        expect(routeSeo(site, null, '/x')).toMatchObject({
-            title: 'Site',
-            url: 'https://x.test/x',
-        });
-    });
-});
-
-describe('injectSeoHtml', () => {
-    it('replaces the title + description and inserts the rest before </head>', () => {
-        const shell =
-            '<!doctype html><html><head><title>old</title><meta name="description" content="" /></head><body></body></html>';
-        const out = injectSeoHtml(shell, {
-            title: 'New',
-            description: 'fresh',
-            url: 'https://x.test',
-        });
-        expect(out).toContain('<title>New</title>');
-        expect(out).not.toContain('<title>old</title>');
-        expect(out.match(/name="description"/g)).toHaveLength(1);
-        expect(out).toContain('content="fresh"');
-        expect(out).toContain('<link rel="canonical" href="https://x.test" />');
-    });
-});
-
-describe('robotsTxt', () => {
-    it('allows all + lists AI crawlers + links the sitemap by default', () => {
-        const txt = robotsTxt({ url: 'https://x.test' });
-        expect(txt).toContain('User-agent: *');
-        expect(txt).toContain('Allow: /');
-        expect(txt).toContain('User-agent: GPTBot');
-        expect(txt).toContain('User-agent: ClaudeBot');
-        expect(txt).toContain('Sitemap: https://x.test/sitemap.xml');
-    });
-
-    it('disallows AI crawlers when ai: "disallow"', () => {
-        const txt = robotsTxt({ url: 'https://x.test', robots: { ai: 'disallow' } });
-        expect(txt).toMatch(/User-agent: GPTBot\nDisallow: \//);
-    });
-
-    it('is empty when robots: false', () => {
-        expect(robotsTxt({ robots: false })).toBe('');
-    });
-});
-
-describe('sitemapXml', () => {
-    it('lists only static routes, absolute', () => {
-        const xml = sitemapXml({ url: 'https://x.test' }, routes);
-        expect(xml).toContain('<loc>https://x.test</loc>');
-        expect(xml).toContain('<loc>https://x.test/about</loc>');
-        expect(xml).not.toContain(':id');
-        expect(xml).not.toContain('/photo');
-    });
-
-    it('is empty without a base url', () => {
-        expect(sitemapXml({}, routes)).toBe('');
-    });
-});
-
-describe('llmsTxt', () => {
-    it('renders title, summary, instructions, and pages', () => {
-        const txt = llmsTxt(
-            {
-                url: 'https://x.test',
-                title: 'My Site',
-                description: 'a site',
-                llms: { instructions: 'Be nice.' },
-            },
-            routes,
-        );
-        expect(txt).toContain('# My Site');
-        expect(txt).toContain('> a site');
-        expect(txt).toContain('Be nice.');
-        expect(txt).toContain('[Home](https://x.test)');
-        expect(txt).toContain('[/about](https://x.test/about)');
-    });
-
-    it('is empty when llms: false', () => {
-        expect(llmsTxt({ llms: false }, routes)).toBe('');
-    });
-});
+import { describe, expect, it } from 'vitest';
+
+import type { ScannedRoute } from '../src/compiler/routes';
+import {
+    injectSeoHtml,
+    llmsTxt,
+    robotsTxt,
+    routeSeo,
+    seoHeadTags,
+    sitemapXml,
+} from '../src/compiler/seo';
+
+const routes: ScannedRoute[] = [
+    { file: 'a', pattern: '/' },
+    { file: 'b', pattern: '/about' },
+    { file: 'c', pattern: '/blog/:id' }, // dynamic, excluded from sitemap
+    { file: 'd', pattern: '/photo/:id', slot: 'modal', intercept: true }, // slot/intercept, excluded
+];
+
+describe('seoHeadTags', () => {
+    it('bakes description, OG, canonical, preconnect, and JSON-LD', () => {
+        const html = seoHeadTags({
+            url: 'https://x.test',
+            title: 'Home',
+            description: 'desc',
+            openGraph: { type: 'website', image: 'https://x.test/og.png' },
+            preconnect: ['https://cdn.test'],
+            jsonLd: { '@context': 'https://schema.org', '@type': 'WebSite' },
+        });
+        expect(html).toContain('<meta name="description" content="desc" />');
+        expect(html).toContain('<meta property="og:title" content="Home" />');
+        expect(html).toContain('<meta property="og:image" content="https://x.test/og.png" />');
+        expect(html).toContain('<link rel="canonical" href="https://x.test" />');
+        expect(html).toContain('<link rel="preconnect" href="https://cdn.test" />');
+        expect(html).toContain('application/ld+json');
+        expect(html).toContain('"@type":"WebSite"');
+    });
+
+    it('escapes attribute values', () => {
+        expect(seoHeadTags({ description: 'a "b" <c>' })).toContain(
+            'content="a &quot;b&quot; &lt;c&gt;"',
+        );
+    });
+
+    it('renders a full Twitter card + OG image dimensions + fb:app_id', () => {
+        const html = seoHeadTags({
+            title: 'Home',
+            description: 'd',
+            openGraph: {
+                image: 'https://x.test/og.png',
+                imageAlt: 'alt',
+                imageWidth: 1200,
+                imageHeight: 630,
+            },
+            twitter: { site: '@x' },
+            facebook: { appId: '123' },
+        });
+        expect(html).toContain('<meta name="twitter:card" content="summary_large_image" />');
+        expect(html).toContain('<meta name="twitter:site" content="@x" />');
+        expect(html).toContain('<meta name="twitter:title" content="Home" />');
+        expect(html).toContain('<meta name="twitter:image" content="https://x.test/og.png" />');
+        expect(html).toContain('<meta property="og:image:width" content="1200" />');
+        expect(html).toContain('<meta property="og:image:alt" content="alt" />');
+        expect(html).toContain('<meta property="fb:app_id" content="123" />');
+    });
+
+    it('neutralizes </script> in JSON-LD (no script breakout)', () => {
+        const html = seoHeadTags({ jsonLd: { x: '</script><img src=x onerror=alert(1)>' } });
+        expect(html).not.toContain('</script><img');
+        expect(html).toContain('\\u003c/script');
+    });
+});
+
+describe('routeSeo', () => {
+    it("overlays a route's metadata over the site defaults and points URLs at the route", () => {
+        const site = { url: 'https://x.test', title: 'Site', description: 'site desc' };
+        const out = routeSeo(site, { title: 'About', description: 'about desc' }, '/about');
+        expect(out.title).toBe('About');
+        expect(out.description).toBe('about desc');
+        expect(out.url).toBe('https://x.test/about');
+    });
+
+    it('falls back to the site defaults when a route has no metadata', () => {
+        const site = { url: 'https://x.test', title: 'Site' };
+        expect(routeSeo(site, null, '/x')).toMatchObject({
+            title: 'Site',
+            url: 'https://x.test/x',
+        });
+    });
+});
+
+describe('injectSeoHtml', () => {
+    it('replaces the title + description and inserts the rest before </head>', () => {
+        const shell =
+            '<!doctype html><html><head><title>old</title><meta name="description" content="" /></head><body></body></html>';
+        const out = injectSeoHtml(shell, {
+            title: 'New',
+            description: 'fresh',
+            url: 'https://x.test',
+        });
+        expect(out).toContain('<title>New</title>');
+        expect(out).not.toContain('<title>old</title>');
+        expect(out.match(/name="description"/g)).toHaveLength(1);
+        expect(out).toContain('content="fresh"');
+        expect(out).toContain('<link rel="canonical" href="https://x.test" />');
+    });
+});
+
+describe('robotsTxt', () => {
+    it('allows all + lists AI crawlers + links the sitemap by default', () => {
+        const txt = robotsTxt({ url: 'https://x.test' });
+        expect(txt).toContain('User-agent: *');
+        expect(txt).toContain('Allow: /');
+        expect(txt).toContain('User-agent: GPTBot');
+        expect(txt).toContain('User-agent: ClaudeBot');
+        expect(txt).toContain('Sitemap: https://x.test/sitemap.xml');
+    });
+
+    it('disallows AI crawlers when ai: "disallow"', () => {
+        const txt = robotsTxt({ url: 'https://x.test', robots: { ai: 'disallow' } });
+        expect(txt).toMatch(/User-agent: GPTBot\nDisallow: \//);
+    });
+
+    it('is empty when robots: false', () => {
+        expect(robotsTxt({ robots: false })).toBe('');
+    });
+});
+
+describe('sitemapXml', () => {
+    it('lists only static routes, absolute', () => {
+        const xml = sitemapXml({ url: 'https://x.test' }, routes);
+        expect(xml).toContain('<loc>https://x.test</loc>');
+        expect(xml).toContain('<loc>https://x.test/about</loc>');
+        expect(xml).not.toContain(':id');
+        expect(xml).not.toContain('/photo');
+    });
+
+    it('is empty without a base url', () => {
+        expect(sitemapXml({}, routes)).toBe('');
+    });
+});
+
+describe('llmsTxt', () => {
+    it('renders title, summary, instructions, and pages', () => {
+        const txt = llmsTxt(
+            {
+                url: 'https://x.test',
+                title: 'My Site',
+                description: 'a site',
+                llms: { instructions: 'Be nice.' },
+            },
+            routes,
+        );
+        expect(txt).toContain('# My Site');
+        expect(txt).toContain('> a site');
+        expect(txt).toContain('Be nice.');
+        expect(txt).toContain('[Home](https://x.test)');
+        expect(txt).toContain('[/about](https://x.test/about)');
+    });
+
+    it('is empty when llms: false', () => {
+        expect(llmsTxt({ llms: false }, routes)).toBe('');
+    });
+
+    it('renders a supplied page list (titles + descriptions, e.g. SSG pages) over the static paths', () => {
+        const txt = llmsTxt({ url: 'https://x.test', title: 'Docs' }, routes, [
+            { title: 'useReducer | React Hooks', url: 'https://x.test/hooks/usereducer', description: 'A reducer.' },
+            { title: 'About', url: 'https://x.test/about' },
+        ]);
+        expect(txt).toContain('[useReducer | React Hooks](https://x.test/hooks/usereducer): A reducer.');
+        expect(txt).toContain('[About](https://x.test/about)');
+        // the supplied list replaces the bare static-path fallback
+        expect(txt).not.toContain('[Home](https://x.test)');
+    });
+});

package/test/slot-layouts.test.ts

@@ -1,69 +1,69 @@
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-
-import { afterEach, describe, expect, it } from 'vitest';
-
-import { loadConfig } from '../src/compiler/config';
-import { generate } from '../src/compiler/generate';
-
-const roots: string[] = [];
-function project(files: Record<string, string>): string {
-    const root = fs.mkdtempSync(path.join(os.tmpdir(), 'toil-gen-'));
-    roots.push(root);
-    for (const [rel, content] of Object.entries(files)) {
-        const abs = path.join(root, rel);
-        fs.mkdirSync(path.dirname(abs), { recursive: true });
-        fs.writeFileSync(abs, content);
-    }
-    return root;
-}
-afterEach(() => {
-    for (const r of roots.splice(0)) fs.rmSync(r, { recursive: true, force: true });
-});
-
-const COMP = `export default function C() { return null; }\n`;
-const LAYOUT = `export default function L({ children }) { return children; }\n`;
-const HTML = `<!doctype html><html><head></head><body><div id="root"></div></body></html>\n`;
-
-describe('generate: parallel-slot layout chains', () => {
-    it('keeps the parent layout on the full-page route but drops it from the @slot route', async () => {
-        const root = project({
-            'client/public/index.html': HTML,
-            'client/routes/gallery/layout.tsx': LAYOUT,
-            'client/routes/gallery/index.tsx': COMP,
-            'client/routes/gallery/photo/[id].tsx': COMP,
-            'client/routes/gallery/@modal/(.)photo/[id].tsx': COMP,
-        });
-        const cfg = await loadConfig({ root });
-        generate(cfg);
-        const lines = fs.readFileSync(path.join(cfg.toilDir, 'routes.ts'), 'utf8').split('\n');
-
-        // The normal full-page route is wrapped by gallery/layout.
-        const mainLine = lines.find((l) => l.includes('photo/[id]') && !l.includes('@modal'));
-        expect(mainLine).toMatch(/gallery\/layout/);
-
-        // The intercepting @modal slot route is rendered INTO gallery/layout's <Slot>, so it must not
-        // re-include that layout (doing so recurses, the slot rendering itself forever).
-        const slotLine = lines.find((l) => l.includes('@modal'));
-        expect(slotLine).toContain('intercept: true');
-        expect(slotLine).toMatch(/layouts: \[\]/);
-        expect(slotLine).not.toMatch(/gallery\/layout/);
-    });
-
-    it('still applies a layout placed inside the @slot subtree', async () => {
-        const root = project({
-            'client/public/index.html': HTML,
-            'client/routes/gallery/layout.tsx': LAYOUT,
-            'client/routes/gallery/@modal/layout.tsx': LAYOUT,
-            'client/routes/gallery/@modal/(.)photo/[id].tsx': COMP,
-        });
-        const cfg = await loadConfig({ root });
-        generate(cfg);
-        const lines = fs.readFileSync(path.join(cfg.toilDir, 'routes.ts'), 'utf8').split('\n');
-        const slotLine = lines.find((l) => l.includes('@modal/(.)photo'));
-        // The slot's own layout (inside @modal) applies; the parent gallery layout does not.
-        expect(slotLine).toMatch(/@modal\/layout/);
-        expect(slotLine).not.toMatch(/gallery\/layout/);
-    });
-});
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+
+import { afterEach, describe, expect, it } from 'vitest';
+
+import { loadConfig } from '../src/compiler/config';
+import { generate } from '../src/compiler/generate';
+
+const roots: string[] = [];
+function project(files: Record<string, string>): string {
+    const root = fs.mkdtempSync(path.join(os.tmpdir(), 'toil-gen-'));
+    roots.push(root);
+    for (const [rel, content] of Object.entries(files)) {
+        const abs = path.join(root, rel);
+        fs.mkdirSync(path.dirname(abs), { recursive: true });
+        fs.writeFileSync(abs, content);
+    }
+    return root;
+}
+afterEach(() => {
+    for (const r of roots.splice(0)) fs.rmSync(r, { recursive: true, force: true });
+});
+
+const COMP = `export default function C() { return null; }\n`;
+const LAYOUT = `export default function L({ children }) { return children; }\n`;
+const HTML = `<!doctype html><html><head></head><body><div id="root"></div></body></html>\n`;
+
+describe('generate: parallel-slot layout chains', () => {
+    it('keeps the parent layout on the full-page route but drops it from the @slot route', async () => {
+        const root = project({
+            'client/public/index.html': HTML,
+            'client/routes/gallery/layout.tsx': LAYOUT,
+            'client/routes/gallery/index.tsx': COMP,
+            'client/routes/gallery/photo/[id].tsx': COMP,
+            'client/routes/gallery/@modal/(.)photo/[id].tsx': COMP,
+        });
+        const cfg = await loadConfig({ root });
+        generate(cfg);
+        const lines = fs.readFileSync(path.join(cfg.toilDir, 'routes.ts'), 'utf8').split('\n');
+
+        // The normal full-page route is wrapped by gallery/layout.
+        const mainLine = lines.find((l) => l.includes('photo/[id]') && !l.includes('@modal'));
+        expect(mainLine).toMatch(/gallery\/layout/);
+
+        // The intercepting @modal slot route is rendered INTO gallery/layout's <Slot>, so it must not
+        // re-include that layout (doing so recurses, the slot rendering itself forever).
+        const slotLine = lines.find((l) => l.includes('@modal'));
+        expect(slotLine).toContain('intercept: true');
+        expect(slotLine).toMatch(/layouts: \[\]/);
+        expect(slotLine).not.toMatch(/gallery\/layout/);
+    });
+
+    it('still applies a layout placed inside the @slot subtree', async () => {
+        const root = project({
+            'client/public/index.html': HTML,
+            'client/routes/gallery/layout.tsx': LAYOUT,
+            'client/routes/gallery/@modal/layout.tsx': LAYOUT,
+            'client/routes/gallery/@modal/(.)photo/[id].tsx': COMP,
+        });
+        const cfg = await loadConfig({ root });
+        generate(cfg);
+        const lines = fs.readFileSync(path.join(cfg.toilDir, 'routes.ts'), 'utf8').split('\n');
+        const slotLine = lines.find((l) => l.includes('@modal/(.)photo'));
+        // The slot's own layout (inside @modal) applies; the parent gallery layout does not.
+        expect(slotLine).toMatch(/@modal\/layout/);
+        expect(slotLine).not.toMatch(/gallery\/layout/);
+    });
+});

package/test/ssg.test.ts

@@ -1,36 +1,36 @@
-import { describe, expect, it } from 'vitest';
-
-import { sitemapXml, type SeoConfig } from '../src/compiler/seo';
-import { fillPattern } from '../src/compiler/ssg';
-import { type ScannedRoute } from '../src/compiler/routes';
-
-describe('fillPattern', () => {
-    it('substitutes :param and *catch-all segments', () => {
-        expect(fillPattern('/:a/:b/:c', { a: 'x', b: 'y', c: 'z' })).toBe('/x/y/z');
-        expect(fillPattern('/blog/:id', { id: '42' })).toBe('/blog/42');
-        expect(fillPattern('/docs/*slug', { slug: ['a', 'b'] })).toBe('/docs/a/b');
-        expect(fillPattern('/static', {})).toBe('/static');
-    });
-});
-
-describe('sitemapXml with SSG paths', () => {
-    const seo: SeoConfig = { url: 'https://x.dev' };
-    const routes: ScannedRoute[] = [
-        { file: 'a', pattern: '/' },
-        { file: 'b', pattern: '/about' },
-        { file: 'c', pattern: '/blog/:id' },
-    ];
-
-    it('lists static routes plus enumerated SSG URLs, deduped, never the bare pattern', () => {
-        const xml = sitemapXml(seo, routes, ['/blog/1', '/blog/2', '/about']);
-        expect(xml).toContain('https://x.dev/blog/1');
-        expect(xml).toContain('https://x.dev/blog/2');
-        expect(xml).toContain('https://x.dev/about');
-        expect(xml).not.toContain('/blog/:id'); // dynamic pattern is never listed literally
-        expect((xml.match(/<loc>[^<]*\/about<\/loc>/g) ?? []).length).toBe(1); // deduped
-    });
-
-    it('is empty without a base url', () => {
-        expect(sitemapXml({}, routes, ['/blog/1'])).toBe('');
-    });
-});
+import { describe, expect, it } from 'vitest';
+
+import { sitemapXml, type SeoConfig } from '../src/compiler/seo';
+import { fillPattern } from '../src/compiler/ssg';
+import { type ScannedRoute } from '../src/compiler/routes';
+
+describe('fillPattern', () => {
+    it('substitutes :param and *catch-all segments', () => {
+        expect(fillPattern('/:a/:b/:c', { a: 'x', b: 'y', c: 'z' })).toBe('/x/y/z');
+        expect(fillPattern('/blog/:id', { id: '42' })).toBe('/blog/42');
+        expect(fillPattern('/docs/*slug', { slug: ['a', 'b'] })).toBe('/docs/a/b');
+        expect(fillPattern('/static', {})).toBe('/static');
+    });
+});
+
+describe('sitemapXml with SSG paths', () => {
+    const seo: SeoConfig = { url: 'https://x.dev' };
+    const routes: ScannedRoute[] = [
+        { file: 'a', pattern: '/' },
+        { file: 'b', pattern: '/about' },
+        { file: 'c', pattern: '/blog/:id' },
+    ];
+
+    it('lists static routes plus enumerated SSG URLs, deduped, never the bare pattern', () => {
+        const xml = sitemapXml(seo, routes, ['/blog/1', '/blog/2', '/about']);
+        expect(xml).toContain('https://x.dev/blog/1');
+        expect(xml).toContain('https://x.dev/blog/2');
+        expect(xml).toContain('https://x.dev/about');
+        expect(xml).not.toContain('/blog/:id'); // dynamic pattern is never listed literally
+        expect((xml.match(/<loc>[^<]*\/about<\/loc>/g) ?? []).length).toBe(1); // deduped
+    });
+
+    it('is empty without a base url', () => {
+        expect(sitemapXml({}, routes, ['/blog/1'])).toBe('');
+    });
+});

package/test/update.test.ts

@@ -1,44 +1,44 @@
-import { describe, expect, it } from 'vitest';
-
-import { buildRows, classifyBump, parseNcuJson } from '../src/cli/updates';
-
-describe('classifyBump', () => {
-    it('classifies major / minor / patch from ranges', () => {
-        expect(classifyBump('^19.2.6', '^20.0.0')).toBe('major');
-        expect(classifyBump('^19.2.6', '^19.3.0')).toBe('minor');
-        expect(classifyBump('^19.2.6', '^19.2.7')).toBe('patch');
-        expect(classifyBump('1.2.3', '1.2.3')).toBe('other');
-    });
-});
-
-describe('parseNcuJson', () => {
-    it('parses the jsonUpgraded object, tolerating surrounding noise', () => {
-        expect(parseNcuJson('{"react":"^19.3.0","eslint":"^10.4.1"}')).toEqual({
-            react: '^19.3.0',
-            eslint: '^10.4.1',
-        });
-        expect(parseNcuJson('npx noise\n{"a":"1.0.0"}\nbye')).toEqual({ a: '1.0.0' });
-        expect(parseNcuJson('{}')).toEqual({});
-        expect(parseNcuJson('not json')).toEqual({});
-    });
-});
-
-describe('buildRows', () => {
-    it('joins ncu output with current ranges and sorts major-first then by name', () => {
-        const rows = buildRows(
-            { react: '^20.0.0', eslint: '^10.4.1', vite: '^8.0.15' },
-            { react: '^19.2.6', eslint: '^10.2.0', vite: '^8.0.14' },
-        );
-        expect(rows.map((r) => `${r.name}:${r.bump}`)).toEqual([
-            'react:major',
-            'eslint:minor',
-            'vite:patch',
-        ]);
-        expect(rows[0]).toMatchObject({ name: 'react', from: '^19.2.6', to: '^20.0.0' });
-    });
-
-    it('marks a package missing from current deps with a "?" source', () => {
-        const rows = buildRows({ foo: '^2.0.0' }, {});
-        expect(rows[0]).toMatchObject({ name: 'foo', from: '?', to: '^2.0.0', bump: 'major' });
-    });
-});
+import { describe, expect, it } from 'vitest';
+
+import { buildRows, classifyBump, parseNcuJson } from '../src/cli/updates';
+
+describe('classifyBump', () => {
+    it('classifies major / minor / patch from ranges', () => {
+        expect(classifyBump('^19.2.6', '^20.0.0')).toBe('major');
+        expect(classifyBump('^19.2.6', '^19.3.0')).toBe('minor');
+        expect(classifyBump('^19.2.6', '^19.2.7')).toBe('patch');
+        expect(classifyBump('1.2.3', '1.2.3')).toBe('other');
+    });
+});
+
+describe('parseNcuJson', () => {
+    it('parses the jsonUpgraded object, tolerating surrounding noise', () => {
+        expect(parseNcuJson('{"react":"^19.3.0","eslint":"^10.4.1"}')).toEqual({
+            react: '^19.3.0',
+            eslint: '^10.4.1',
+        });
+        expect(parseNcuJson('npx noise\n{"a":"1.0.0"}\nbye')).toEqual({ a: '1.0.0' });
+        expect(parseNcuJson('{}')).toEqual({});
+        expect(parseNcuJson('not json')).toEqual({});
+    });
+});
+
+describe('buildRows', () => {
+    it('joins ncu output with current ranges and sorts major-first then by name', () => {
+        const rows = buildRows(
+            { react: '^20.0.0', eslint: '^10.4.1', vite: '^8.0.15' },
+            { react: '^19.2.6', eslint: '^10.2.0', vite: '^8.0.14' },
+        );
+        expect(rows.map((r) => `${r.name}:${r.bump}`)).toEqual([
+            'react:major',
+            'eslint:minor',
+            'vite:patch',
+        ]);
+        expect(rows[0]).toMatchObject({ name: 'react', from: '^19.2.6', to: '^20.0.0' });
+    });
+
+    it('marks a package missing from current deps with a "?" source', () => {
+        const rows = buildRows({ foo: '^2.0.0' }, {});
+        expect(rows[0]).toMatchObject({ name: 'foo', from: '?', to: '^2.0.0', bump: 'major' });
+    });
+});