@cspeach/cli 0.6.4 → 0.7.0

package/dist/projects/__tests__/email-template.test.js

@@ -1,71 +0,0 @@
-import { describe, it, expect, vi } from 'vitest';
-import { renderEmailTemplate } from '../email-template.js';
-describe('renderEmailTemplate', () => {
-    it('renders subject + body with title and filename interpolated', () => {
-        const out = renderEmailTemplate({
-            title: 'Customer Aging Report',
-            artefactType: 'spec-gap',
-            filename: 'customer-aging-report-spec-gap-a1b2-v1.cspeach.json',
-            itemCount: 27,
-            itemNoun: 'questions',
-            author: 'Laeeq',
-        });
-        expect(out).toContain('Subject: Gap analysis — Customer Aging Report');
-        expect(out).toContain('27 questions');
-        expect(out).toContain('customer-aging-report-spec-gap-a1b2-v1.cspeach.json');
-        expect(out).toContain('https://cspeach.dev/edit');
-        expect(out).toContain('Thanks,\nLaeeq');
-        expect(out).toContain('rename it to .cspeach.json.txt');
-    });
-    it('renders design subject + body with "objects to review" copy', () => {
-        const out = renderEmailTemplate({
-            title: 'Order Approval System',
-            artefactType: 'design',
-            filename: 'order-approval-system-design-c3d4-v1.cspeach.json',
-            itemCount: 11,
-            itemNoun: 'objects',
-            author: 'Laeeq',
-        });
-        expect(out).toContain('Subject: Design — Order Approval System');
-        expect(out).toContain('11 objects to review');
-        expect(out).toContain('keep / redesign / defer');
-    });
-    it('renders estimate subject + body with "components to review" copy', () => {
-        const out = renderEmailTemplate({
-            title: 'ZSD-441',
-            artefactType: 'estimate',
-            filename: 'zsd-441-estimate-e5f6-v1.cspeach.json',
-            itemCount: 8,
-            itemNoun: 'components',
-            author: 'Laeeq',
-        });
-        expect(out).toContain('Subject: Estimate — ZSD-441');
-        expect(out).toContain('8 components to review');
-        expect(out).toContain('confirm / dispute / defer');
-    });
-    it('respects CSPEACH_EDITOR_URL env override', async () => {
-        const original = process.env.CSPEACH_EDITOR_URL;
-        process.env.CSPEACH_EDITOR_URL = 'http://localhost:5173/edit';
-        try {
-            vi.resetModules();
-            const { renderEmailTemplate: renderFresh } = await import('../email-template.js');
-            const out = renderFresh({
-                title: 'Test',
-                artefactType: 'spec-gap',
-                filename: 'test-spec-gap-aaaa-v1.cspeach.json',
-                itemCount: 5,
-                itemNoun: 'questions',
-                author: 'Tester',
-            });
-            expect(out).toContain('http://localhost:5173/edit');
-            expect(out).not.toContain('https://cspeach.dev/edit');
-        }
-        finally {
-            if (original === undefined)
-                delete process.env.CSPEACH_EDITOR_URL;
-            else
-                process.env.CSPEACH_EDITOR_URL = original;
-            vi.resetModules();
-        }
-    });
-});

package/dist/projects/__tests__/expand-text-attachments.test.js

@@ -1,105 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { mkdirSync, rmSync, writeFileSync } from 'node:fs';
-import { join } from 'node:path';
-import os from 'node:os';
-import { expandTextFileAttachments } from '../workspace.js';
-/**
- * These tests touch the workspace folder via the resolver, which
- * reads the user's home directory at run time. We don't want to
- * pollute the user's actual ~/CSPeach, so we set CSPEACH-style
- * config to point the workspace at a temp dir for the test process.
- *
- * The simplest reliable way to redirect ensureWorkspace's lookup
- * here is to override HOME / USERPROFILE for the duration of the
- * test — the loader's path helpers join from os.homedir(). We
- * restore the original value in afterEach.
- */
-const TEST_HOME = join(os.tmpdir(), `cspeach-expand-test-${process.pid}-${Date.now()}`);
-const TEST_WORKSPACE = join(TEST_HOME, 'CSPeach');
-function setHome(p) {
-    const prev = {
-        HOME: process.env['HOME'],
-        USERPROFILE: process.env['USERPROFILE'],
-    };
-    process.env['HOME'] = p;
-    process.env['USERPROFILE'] = p;
-    return {
-        restore: () => {
-            process.env['HOME'] = prev.HOME;
-            process.env['USERPROFILE'] = prev.USERPROFILE;
-        },
-    };
-}
-describe('expandTextFileAttachments', () => {
-    let homeGuard;
-    beforeEach(() => {
-        // 2026-05-08: default workspace is now project-rooted (`<git-root>/
-        // .cspeach/`) rather than `~/CSPeach/`. The test still uses HOME
-        // redirection to isolate from the real config, but it now ALSO
-        // pins workspace_folder via a synthetic ~/.cspeach/config.toml so
-        // ensureWorkspace returns TEST_WORKSPACE instead of the project's
-        // .cspeach/.
-        mkdirSync(TEST_WORKSPACE, { recursive: true });
-        mkdirSync(join(TEST_HOME, '.cspeach'), { recursive: true });
-        writeFileSync(join(TEST_HOME, '.cspeach', 'config.toml'), `workspace_folder = "${TEST_WORKSPACE.replace(/\\/g, '/')}"\nmigration_prompted = true\n`);
-        homeGuard = setHome(TEST_HOME);
-    });
-    afterEach(() => {
-        homeGuard.restore();
-        try {
-            rmSync(TEST_HOME, { recursive: true, force: true });
-        }
-        catch { /* best-effort */ }
-    });
-    it('substitutes @<txt-file> with an <attached> block', async () => {
-        writeFileSync(join(TEST_WORKSPACE, 'spec.txt'), 'My requirement here.');
-        const out = await expandTextFileAttachments('Please run /abap-spec-gap on @spec.txt', process.cwd());
-        expect(out).toContain('<attached file="spec.txt">');
-        expect(out).toContain('My requirement here.');
-        expect(out).toContain('</attached>');
-        expect(out).not.toContain('@spec.txt');
-    });
-    it('substitutes @<md-file> the same way as txt', async () => {
-        writeFileSync(join(TEST_WORKSPACE, 'requirement.md'), '# Requirement\n\nDo X.');
-        const out = await expandTextFileAttachments('@requirement.md please', process.cwd());
-        expect(out).toContain('<attached file="requirement.md">');
-        expect(out).toContain('# Requirement');
-        expect(out).toContain('Do X.');
-    });
-    it('leaves the message unchanged when no @-tokens are present', async () => {
-        const msg = 'Just plain prose with no attachments.';
-        const out = await expandTextFileAttachments(msg, process.cwd());
-        expect(out).toBe(msg);
-    });
-    it('leaves @<envelope> tokens alone (handled by --from / --status)', async () => {
-        writeFileSync(join(TEST_WORKSPACE, 'foo-design-aaaa-v1.cspeach.json'), JSON.stringify({ artefactType: 'design', version: 1, title: 'Foo' }));
-        const out = await expandTextFileAttachments('inspect @foo-design-aaaa-v1.cspeach.json please', process.cwd());
-        expect(out).toContain('@foo-design-aaaa-v1.cspeach.json');
-        expect(out).not.toContain('<attached');
-    });
-    it('leaves unmatched @<token> as a literal', async () => {
-        const out = await expandTextFileAttachments('reference @nothing-here for context', process.cwd());
-        expect(out).toContain('@nothing-here');
-        expect(out).not.toContain('<attached');
-    });
-    it('handles multiple @<txt> tokens in one message', async () => {
-        writeFileSync(join(TEST_WORKSPACE, 'a.txt'), 'AAA');
-        writeFileSync(join(TEST_WORKSPACE, 'b.txt'), 'BBB');
-        const out = await expandTextFileAttachments('Look at @a.txt and @b.txt together.', process.cwd());
-        expect(out).toContain('<attached file="a.txt">\nAAA\n</attached>');
-        expect(out).toContain('<attached file="b.txt">\nBBB\n</attached>');
-    });
-    it('truncates files larger than the 64 KB cap', async () => {
-        const big = 'x'.repeat(70_000);
-        writeFileSync(join(TEST_WORKSPACE, 'big.txt'), big);
-        const out = await expandTextFileAttachments('@big.txt', process.cwd());
-        expect(out).toContain('[truncated, file exceeds 64 KB]');
-        expect(out.length).toBeLessThan(70_000 + 1000);
-    });
-    it('reports attached / not-found through the optional log callback', async () => {
-        writeFileSync(join(TEST_WORKSPACE, 'real.txt'), 'r');
-        const lines = [];
-        await expandTextFileAttachments('mix @real.txt and @ghost-token', process.cwd(), (l) => lines.push(l));
-        expect(lines.some((l) => l.includes('attached real.txt'))).toBe(true);
-    });
-});

package/dist/projects/__tests__/extract-cca.test.js

@@ -1,141 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { extractCcaAssessment } from '../extract-cca.js';
-const FULL_FIXTURE = `# /abap-cca — ZFI / ZSD estate
-
-[lots of human-readable content here that the parser ignores]
-
-<!-- csforge:cca-manifest
-artefact: cca-assessment
-title: ZFI / ZSD — S/4HANA 2023 estate
-detail_path: .abapforge/cca/projects/zfi-zsd-2026-04/project.json
-project_id: zfi-zsd-2026-04
-scope: ZFI* + ZSD*
-atc_variant: S4HANA_READINESS_2023
-total_objects: 1842
-keep: 421
-fix: 738
-retire: 124
-redesign: 312
-uncategorized: 247
-classifications:
-  item-001 | ZFI_BP_SYNC      | PROG | fix      | high
-  item-002 | ZCL_FI_DOC       | CLAS | redesign | medium
-  item-003 | ZSD_LEGACY_RPT   | PROG | retire   | high
-  item-004 | ZFI_HELPER       | CLAS | keep     | low
--->
-`;
-describe('extractCcaAssessment', () => {
-    it('extracts title from manifest', () => {
-        const out = extractCcaAssessment(FULL_FIXTURE);
-        expect(out.title).toBe('ZFI / ZSD — S/4HANA 2023 estate');
-    });
-    it('extracts detailPath, projectId, scope, atcVariant', () => {
-        const out = extractCcaAssessment(FULL_FIXTURE);
-        expect(out.content.detailPath).toBe('.abapforge/cca/projects/zfi-zsd-2026-04/project.json');
-        expect(out.content.projectId).toBe('zfi-zsd-2026-04');
-        expect(out.content.scope).toBe('ZFI* + ZSD*');
-        expect(out.content.atcVariant).toBe('S4HANA_READINESS_2023');
-    });
-    it('extracts summary counts', () => {
-        const out = extractCcaAssessment(FULL_FIXTURE);
-        expect(out.content.summary).toEqual({
-            totalObjects: 1842,
-            keep: 421,
-            fix: 738,
-            retire: 124,
-            redesign: 312,
-            uncategorized: 247,
-        });
-    });
-    it('extracts classifications table with all four rows', () => {
-        const out = extractCcaAssessment(FULL_FIXTURE);
-        expect(out.content.classifications).toHaveLength(4);
-        expect(out.content.classifications[0]).toEqual({
-            id: 'item-001',
-            objectName: 'ZFI_BP_SYNC',
-            objectType: 'PROG',
-            classification: 'fix',
-            confidence: 'high',
-        });
-        expect(out.content.classifications[1]).toEqual({
-            id: 'item-002',
-            objectName: 'ZCL_FI_DOC',
-            objectType: 'CLAS',
-            classification: 'redesign',
-            confidence: 'medium',
-        });
-        expect(out.content.classifications[2]).toEqual({
-            id: 'item-003',
-            objectName: 'ZSD_LEGACY_RPT',
-            objectType: 'PROG',
-            classification: 'retire',
-            confidence: 'high',
-        });
-    });
-    it('emits one InteractionItem per classification with default status open', () => {
-        const out = extractCcaAssessment(FULL_FIXTURE);
-        expect(out.items).toHaveLength(4);
-        for (const it of out.items) {
-            expect(it.status).toBe('open');
-            expect(it.answer).toBeNull();
-            expect(it.answeredBy).toBeNull();
-            expect(it.comments).toEqual([]);
-        }
-        expect(out.items.map((i) => i.id)).toEqual(['item-001', 'item-002', 'item-003', 'item-004']);
-    });
-    it('throws when the manifest block is missing', () => {
-        expect(() => extractCcaAssessment('# Just a heading\nno manifest here')).toThrow(/No <!-- csforge:cca-manifest --> block/);
-    });
-    it('throws when artefact field disagrees', () => {
-        const wrong = `<!-- csforge:cca-manifest
-artefact: upgrade-scan
-detail_path: .abapforge/foo.json
-project_id: abc
--->`;
-        expect(() => extractCcaAssessment(wrong)).toThrow(/expected 'cca-assessment'/);
-    });
-    it('throws when detail_path is missing', () => {
-        const noDetail = `<!-- csforge:cca-manifest
-artefact: cca-assessment
-project_id: abc
--->`;
-        expect(() => extractCcaAssessment(noDetail)).toThrow(/'detail_path'/);
-    });
-    it('throws when project_id is missing', () => {
-        const noProject = `<!-- csforge:cca-manifest
-artefact: cca-assessment
-detail_path: .abapforge/foo.json
--->`;
-        expect(() => extractCcaAssessment(noProject)).toThrow(/'project_id'/);
-    });
-    it('coerces unknown classification value to safe default keep', () => {
-        const weird = `<!-- csforge:cca-manifest
-artefact: cca-assessment
-detail_path: .abapforge/foo.json
-project_id: abc
-classifications:
-  item-001 | ZFOO | PROG | refurbish | astonishing
--->`;
-        const out = extractCcaAssessment(weird);
-        expect(out.content.classifications[0]?.classification).toBe('keep');
-        expect(out.content.classifications[0]?.confidence).toBe('medium');
-    });
-    it('falls back to defaults when summary fields absent', () => {
-        const minimal = `<!-- csforge:cca-manifest
-artefact: cca-assessment
-detail_path: .abapforge/foo.json
-project_id: abc
--->`;
-        const out = extractCcaAssessment(minimal);
-        expect(out.content.summary).toEqual({
-            totalObjects: 0,
-            keep: 0,
-            fix: 0,
-            retire: 0,
-            redesign: 0,
-            uncategorized: 0,
-        });
-        expect(out.content.classifications).toEqual([]);
-        expect(out.items).toEqual([]);
-    });
-});

package/dist/projects/__tests__/extract-design.test.js

@@ -1,62 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { readFileSync } from 'node:fs';
-import { fileURLToPath } from 'node:url';
-import { dirname, resolve } from 'node:path';
-import { extractDesign } from '../extract-design.js';
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const FIXTURE_PATH = resolve(__dirname, '../../../test/fixtures/abap-design-output.md');
-describe('extractDesign', () => {
-    it('extracts title from heading', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractDesign(md);
-        expect(out.title).toBe('Order Approval System');
-    });
-    it('extracts all 11 objects from the table', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractDesign(md);
-        expect(out.content.objects).toHaveLength(11);
-        expect(out.content.objects[0]).toMatchObject({
-            name: 'ZDOM_APPROVAL_STATUS',
-            type: 'Domain',
-            package: 'ZDEV_TR001',
-        });
-        expect(out.content.objects[8]).toMatchObject({
-            name: 'ZCL_APPROVAL_BEHAVIOR',
-            type: 'Class',
-        });
-    });
-    it('assigns sequential ids obj-001..obj-N', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractDesign(md);
-        expect(out.content.objects.map((o) => o.id)).toEqual([
-            'obj-001', 'obj-002', 'obj-003', 'obj-004', 'obj-005',
-            'obj-006', 'obj-007', 'obj-008', 'obj-009', 'obj-010', 'obj-011',
-        ]);
-    });
-    it('populates dependsOn from the Dependency Sequence section', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractDesign(md);
-        const byName = new Map(out.content.objects.map((o) => [o.name, o]));
-        expect(byName.get('ZDE_APPROVAL_STATUS').dependsOn).toContain('obj-001');
-        expect(byName.get('ZAPPR_REQUEST').dependsOn).toEqual(expect.arrayContaining(['obj-003', 'obj-004']));
-        expect(byName.get('ZBND_APPROVAL').dependsOn).toEqual(['obj-010']);
-    });
-    it('emits one InteractionItem per object with default status keep', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractDesign(md);
-        expect(out.items).toHaveLength(11);
-        for (const it of out.items) {
-            expect(it.status).toBe('keep');
-            expect(it.answer).toBeNull();
-            expect(it.answeredBy).toBeNull();
-            expect(it.comments).toEqual([]);
-        }
-        expect(out.items.map((i) => i.id)).toEqual(out.content.objects.map((o) => o.id));
-    });
-    it('throws when the heading is missing', () => {
-        expect(() => extractDesign('## Some Other Heading\n\nbody')).toThrow(/not a design output/);
-    });
-    it('throws when the Objects to Create table is missing', () => {
-        expect(() => extractDesign('## ABAP Design — X\n\n(no table)')).toThrow(/Objects to Create/);
-    });
-});

package/dist/projects/__tests__/extract-estimate.test.js

@@ -1,58 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { readFileSync } from 'node:fs';
-import { fileURLToPath } from 'node:url';
-import { dirname, resolve } from 'node:path';
-import { extractEstimate } from '../extract-estimate.js';
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const FIXTURE_PATH = resolve(__dirname, '../../../test/fixtures/abap-estimate-output.md');
-describe('extractEstimate', () => {
-    it('extracts title from heading', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractEstimate(md);
-        expect(out.title).toBe('ZSD-441 Material Pricing Report with Enhancement');
-    });
-    it('extracts all 11 components, skipping the TOTAL row', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractEstimate(md);
-        expect(out.content.components).toHaveLength(11);
-        expect(out.content.components[0]).toMatchObject({
-            name: 'CDS interface view — pricing conditions',
-            hoursOptimistic: 3,
-            hoursRealistic: 5,
-            hoursPessimistic: 9,
-        });
-        expect(out.content.components[5]).toMatchObject({
-            name: 'Interface for test doubles',
-            hoursOptimistic: 0.5,
-            hoursRealistic: 1,
-            hoursPessimistic: 1.5,
-        });
-    });
-    it('assigns sequential ids cmp-001..cmp-N', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractEstimate(md);
-        expect(out.content.components[0].id).toBe('cmp-001');
-        expect(out.content.components[10].id).toBe('cmp-011');
-    });
-    it('captures Notes column into risks array', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractEstimate(md);
-        expect(out.content.components[3].risks).toEqual(['BAdI behavior not fully documented']);
-        expect(out.content.components[5].risks).toEqual(['Interface design follows from class structure']);
-    });
-    it('emits one InteractionItem per component with default status confirmed', () => {
-        const md = readFileSync(FIXTURE_PATH, 'utf8');
-        const out = extractEstimate(md);
-        expect(out.items).toHaveLength(11);
-        for (const it of out.items) {
-            expect(it.status).toBe('confirmed');
-            expect(it.answer).toBeNull();
-        }
-    });
-    it('throws when the heading is missing', () => {
-        expect(() => extractEstimate('## Some Other Heading\n')).toThrow(/not an estimate output/);
-    });
-    it('throws when the Component Breakdown table is missing', () => {
-        expect(() => extractEstimate('## Effort Estimate — X\n\n(no table)')).toThrow(/Component Breakdown/);
-    });
-});

package/dist/projects/__tests__/extract-modernize.test.js

@@ -1,127 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { extractModernizeResult } from '../extract-modernize.js';
-describe('extractModernizeResult', () => {
-    const FULL_FIXTURE = `# /abap-modernize — ZDPR_PAYMENTS
-
-[lots of human-readable prose the parser ignores]
-
-<!-- csforge:modernize-manifest
-artefact: modernize-result
-title: ZDPR_PAYMENTS — modernization
-detail_path: .abapforge/modernize/zdpr-payments/project.json
-baseline_path: .cspeach/zdpr-payments-cca-assessment-3467-v1.cspeach.json
-baseline_sha256: a7Qj8ibfLTj49kQRTn1DOE_UAExuoBgjhaXxf33+JPY=
-transport: S4HK903400
-applied: 2
-skipped: 1
-failed: 0
-pending: 1
-modernizations:
-  item-001 | ZBDC_VENDOR_PAYMENTS_CLOUD | FUNC | bdc-against-f53        | fm-based-posting        | applied
-  item-002 | ZPAYMENTS                  | PROG | select-from-bsig       | cds-i_journalentryitem | applied
-  item-003 | ZFGPAYMENTS                | FUGR | classic-procedural     | rap-behavior            | skipped
-  item-004 | ZPAYMENTS_OLD              | PROG | bdc-against-f53        | retire                  | pending
--->
-`;
-    it('extracts title from manifest', () => {
-        const out = extractModernizeResult(FULL_FIXTURE);
-        expect(out.title).toBe('ZDPR_PAYMENTS — modernization');
-    });
-    it('extracts detailPath + baselineRef', () => {
-        const out = extractModernizeResult(FULL_FIXTURE);
-        expect(out.content.detailPath).toBe('.abapforge/modernize/zdpr-payments/project.json');
-        expect(out.content.baselineRef.path).toBe('.cspeach/zdpr-payments-cca-assessment-3467-v1.cspeach.json');
-        expect(out.content.baselineRef.sha256).toBe('a7Qj8ibfLTj49kQRTn1DOE_UAExuoBgjhaXxf33+JPY=');
-    });
-    it('extracts transport id', () => {
-        const out = extractModernizeResult(FULL_FIXTURE);
-        expect(out.content.transport).toBe('S4HK903400');
-    });
-    it('parses transport=- as null', () => {
-        const xml = FULL_FIXTURE.replace('transport: S4HK903400', 'transport: -');
-        const out = extractModernizeResult(xml);
-        expect(out.content.transport).toBeNull();
-    });
-    it('extracts summary counts', () => {
-        const out = extractModernizeResult(FULL_FIXTURE);
-        expect(out.content.summary).toEqual({
-            applied: 2,
-            skipped: 1,
-            failed: 0,
-            pending: 1,
-        });
-    });
-    it('extracts modernizations table with from/to architecture', () => {
-        const out = extractModernizeResult(FULL_FIXTURE);
-        expect(out.content.modernizations).toHaveLength(4);
-        expect(out.content.modernizations[0]).toEqual({
-            id: 'item-001',
-            objectName: 'ZBDC_VENDOR_PAYMENTS_CLOUD',
-            objectType: 'FUNC',
-            fromArchitecture: 'bdc-against-f53',
-            toArchitecture: 'fm-based-posting',
-            status: 'applied',
-        });
-        expect(out.content.modernizations[2]?.status).toBe('skipped');
-        expect(out.content.modernizations[3]?.status).toBe('pending');
-    });
-    it('emits one InteractionItem per modernization with matching status', () => {
-        const out = extractModernizeResult(FULL_FIXTURE);
-        expect(out.items).toHaveLength(4);
-        expect(out.items.map((i) => i.status)).toEqual(['applied', 'applied', 'skipped', 'pending']);
-        for (const it of out.items) {
-            expect(it.answer).toBeNull();
-            expect(it.comments).toEqual([]);
-        }
-    });
-    it('coerces unknown status values to pending', () => {
-        const weird = `<!-- csforge:modernize-manifest
-artefact: modernize-result
-detail_path: .abapforge/foo.json
-baseline_path: bar.json
-modernizations:
-  item-001 | ZFOO | PROG | a | b | maybe-applied
--->`;
-        const out = extractModernizeResult(weird);
-        expect(out.content.modernizations[0]?.status).toBe('pending');
-    });
-    it('throws when artefact field disagrees', () => {
-        const wrong = `<!-- csforge:modernize-manifest
-artefact: upgrade-scan
-detail_path: x
-baseline_path: y
--->`;
-        expect(() => extractModernizeResult(wrong)).toThrow(/expected 'modernize-result'/);
-    });
-    it('throws when detail_path is missing', () => {
-        const noDetail = `<!-- csforge:modernize-manifest
-artefact: modernize-result
-baseline_path: y
--->`;
-        expect(() => extractModernizeResult(noDetail)).toThrow(/'detail_path'/);
-    });
-    it('throws when baseline_path is missing', () => {
-        const noBaseline = `<!-- csforge:modernize-manifest
-artefact: modernize-result
-detail_path: x
--->`;
-        expect(() => extractModernizeResult(noBaseline)).toThrow(/'baseline_path'/);
-    });
-    it('throws when no manifest block present', () => {
-        expect(() => extractModernizeResult('plain markdown')).toThrow(/No <!-- csforge:modernize-manifest --> block/);
-    });
-    it('handles a manifest with zero modernizations gracefully', () => {
-        const empty = `<!-- csforge:modernize-manifest
-artefact: modernize-result
-detail_path: x
-baseline_path: y
-applied: 0
-skipped: 0
-failed: 0
-pending: 0
--->`;
-        const out = extractModernizeResult(empty);
-        expect(out.content.modernizations).toEqual([]);
-        expect(out.items).toEqual([]);
-    });
-});