@cspeach/cli 0.6.4 → 0.7.0

package/dist/projects/__tests__/save-command.test.js

@@ -1,216 +0,0 @@
-import { describe, it, expect, vi } from 'vitest';
-import { mkdtempSync, rmSync, existsSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { dirname, join, resolve } from 'node:path';
-import { fileURLToPath } from 'node:url';
-import { readFileSync } from 'node:fs';
-import { runSaveCommand } from '../save-command.js';
-import { readProjectFile } from '../status.js';
-// ESM: derive __dirname from import.meta.url (see Task B2).
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const FIXTURE = readFileSync(join(__dirname, '..', '__fixtures__', 'spec-gap-skill-output.md'), 'utf8');
-const DESIGN_FIXTURE = readFileSync(resolve(__dirname, '../../../test/fixtures/abap-design-output.md'), 'utf8');
-const ESTIMATE_FIXTURE = readFileSync(resolve(__dirname, '../../../test/fixtures/abap-estimate-output.md'), 'utf8');
-describe('runSaveCommand', () => {
-    it('saves file when user confirms y', async () => {
-        const tmp = mkdtempSync(join(tmpdir(), 'cspeach-cmd-'));
-        try {
-            const log = vi.fn();
-            const path = await runSaveCommand({
-                skillOutput: FIXTURE,
-                skillName: 'abap-spec-gap',
-                skillVersion: '1.0',
-                skillInput: 'User Story: Customer Aging Report\n...',
-                tokensUsed: 7592,
-                model: 'claude-opus-4-7',
-                author: { name: 'Laeeq', role: 'consultant' },
-                cwd: tmp,
-                prompt: async () => 'y',
-                log,
-            });
-            expect(path).not.toBeNull();
-            expect(existsSync(path)).toBe(true);
-            expect(log.mock.calls.flat().join('\n')).toContain('Subject: Gap analysis');
-        }
-        finally {
-            rmSync(tmp, { recursive: true, force: true });
-        }
-    });
-    it('returns null when user declines', async () => {
-        const tmp = mkdtempSync(join(tmpdir(), 'cspeach-cmd-'));
-        try {
-            const log = vi.fn();
-            const path = await runSaveCommand({
-                skillOutput: FIXTURE,
-                skillName: 'abap-spec-gap',
-                skillVersion: '1.0',
-                skillInput: 'User Story: ...',
-                tokensUsed: 1000,
-                model: 'm',
-                author: { name: 'Laeeq', role: 'consultant' },
-                cwd: tmp,
-                prompt: async () => 'n',
-                log,
-            });
-            expect(path).toBeNull();
-        }
-        finally {
-            rmSync(tmp, { recursive: true, force: true });
-        }
-    });
-    it('returns null when skill output is not parseable as spec-gap', async () => {
-        const tmp = mkdtempSync(join(tmpdir(), 'cspeach-cmd-'));
-        try {
-            const log = vi.fn();
-            const path = await runSaveCommand({
-                skillOutput: 'Some other skill output',
-                skillName: 'abap-radar',
-                skillVersion: '1.0',
-                skillInput: '...',
-                tokensUsed: 100,
-                model: 'm',
-                author: { name: 'Laeeq', role: 'consultant' },
-                cwd: tmp,
-                prompt: async () => 'y',
-                log,
-            });
-            expect(path).toBeNull();
-        }
-        finally {
-            rmSync(tmp, { recursive: true, force: true });
-        }
-    });
-});
-describe('runSaveCommand — design dispatch', () => {
-    it('saves a /abap-design output as a cspeach-design envelope', async () => {
-        const cwd = mkdtempSync(`${tmpdir()}/cspeach-test-`);
-        try {
-            const log = [];
-            const path = await runSaveCommand({
-                skillOutput: DESIGN_FIXTURE,
-                skillName: 'abap-design',
-                skillVersion: '1.0',
-                skillInput: 'Order Approval System',
-                tokensUsed: 5000,
-                model: 'claude-opus-4-7',
-                author: { name: 'Laeeq', role: 'consultant' },
-                cwd,
-                prompt: async () => 'y',
-                log: (...lines) => log.push(...lines),
-            });
-            expect(path).toBeTruthy();
-            expect(path).toMatch(/order-approval-system-design-[a-f0-9]{4}-v1\.cspeach\.json$/);
-            const env = readProjectFile(path);
-            expect(env.artefactType).toBe('design');
-            expect(env.content.objects).toHaveLength(11);
-            const emailLines = log.join('\n');
-            expect(emailLines).toContain('11 objects');
-            expect(emailLines).toContain('design');
-        }
-        finally {
-            rmSync(cwd, { recursive: true, force: true });
-        }
-    });
-    it('returns null and prints nothing for unsupported skill names', async () => {
-        const log = [];
-        const path = await runSaveCommand({
-            skillOutput: '## Anything\n',
-            skillName: 'abap-not-a-skill',
-            skillVersion: '1.0',
-            skillInput: '',
-            tokensUsed: 0,
-            model: 'claude-opus-4-7',
-            author: { name: 'L', role: 'consultant' },
-            cwd: '.',
-            prompt: async () => 'y',
-            log: (...lines) => log.push(...lines),
-        });
-        expect(path).toBeNull();
-        expect(log).toEqual([]);
-    });
-});
-describe('runSaveCommand — estimate dispatch', () => {
-    it('saves a /abap-estimate output as a cspeach-estimate envelope', async () => {
-        const cwd = mkdtempSync(`${tmpdir()}/cspeach-test-`);
-        try {
-            const log = [];
-            const path = await runSaveCommand({
-                skillOutput: ESTIMATE_FIXTURE,
-                skillName: 'abap-estimate',
-                skillVersion: '1.0',
-                skillInput: 'ZSD-441',
-                tokensUsed: 4200,
-                model: 'claude-opus-4-7',
-                author: { name: 'Laeeq', role: 'consultant' },
-                cwd,
-                prompt: async () => 'y',
-                log: (...lines) => log.push(...lines),
-            });
-            expect(path).toBeTruthy();
-            expect(path).toMatch(/zsd-441-material-pricing-report-with-enhancement-estimate-[a-f0-9]{4}-v1\.cspeach\.json$/);
-            const env = readProjectFile(path);
-            expect(env.artefactType).toBe('estimate');
-            expect(env.content.components).toHaveLength(11);
-            const emailLines = log.join('\n');
-            expect(emailLines).toContain('11 components');
-        }
-        finally {
-            rmSync(cwd, { recursive: true, force: true });
-        }
-    });
-});
-describe('runSaveCommand — promotedFrom threading', () => {
-    it('writes promotedFrom into the saved envelope when supplied', async () => {
-        const cwd = mkdtempSync(`${tmpdir()}/cspeach-test-`);
-        try {
-            const promotedFrom = {
-                id: 'src-uuid', title: 'Source Spec',
-                artefactType: 'spec-gap', version: 3,
-                sha256: 'a'.repeat(64),
-                snapshottedAt: '2026-05-05T10:00:00Z',
-                snapshot: { items: [{ id: 'gap-001', text: 'Q', severity: 'BLOCKER', status: 'answered', answer: 'A' }] },
-            };
-            const log = [];
-            const path = await runSaveCommand({
-                skillOutput: DESIGN_FIXTURE,
-                skillName: 'abap-design',
-                skillVersion: '1.0',
-                skillInput: '',
-                tokensUsed: 0,
-                model: 'claude-opus-4-7',
-                author: { name: 'L', role: 'consultant' },
-                cwd,
-                prompt: async () => 'y',
-                log: (...l) => log.push(...l),
-                promotedFrom,
-            });
-            const env = readProjectFile(path);
-            expect(env.promotedFrom).toEqual(promotedFrom);
-        }
-        finally {
-            rmSync(cwd, { recursive: true, force: true });
-        }
-    });
-    it('writes promotedFrom: null when omitted (existing-behaviour pin)', async () => {
-        const cwd = mkdtempSync(`${tmpdir()}/cspeach-test-`);
-        try {
-            const path = await runSaveCommand({
-                skillOutput: DESIGN_FIXTURE,
-                skillName: 'abap-design',
-                skillVersion: '1.0',
-                skillInput: '',
-                tokensUsed: 0,
-                model: 'claude-opus-4-7',
-                author: { name: 'L', role: 'consultant' },
-                cwd,
-                prompt: async () => 'y',
-                log: () => { },
-            });
-            const env = readProjectFile(path);
-            expect(env.promotedFrom).toBeNull();
-        }
-        finally {
-            rmSync(cwd, { recursive: true, force: true });
-        }
-    });
-});

package/dist/projects/__tests__/save.test.js

@@ -1,41 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { mkdtempSync, rmSync, readFileSync, existsSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { saveProject } from '../save.js';
-import { buildEnvelope } from '../build.js';
-const SAMPLE_EXTRACT = {
-    title: 'Customer Aging Report',
-    content: { inputType: 'User Story', inferredDomain: 'FI-AR', sections: [] },
-    items: [],
-};
-let tmp;
-beforeEach(() => { tmp = mkdtempSync(join(tmpdir(), 'cspeach-save-')); });
-afterEach(() => { rmSync(tmp, { recursive: true, force: true }); });
-describe('saveProject', () => {
-    it('writes a file at <cwd>/<filename> and returns the path', async () => {
-        const env = buildEnvelope({
-            artefactType: 'spec-gap',
-            extract: SAMPLE_EXTRACT,
-            author: { name: 'Laeeq', role: 'consultant' },
-            source: { skill: 'abap-spec-gap', skillVersion: '1.0', input: '...', tokensUsed: 5000, model: 'claude-opus-4-7' },
-        });
-        const path = await saveProject(env, { cwd: tmp });
-        expect(existsSync(path)).toBe(true);
-        expect(path).toMatch(/customer-aging-report-spec-gap-[0-9a-f]{4}-v1\.cspeach\.json$/);
-        const written = JSON.parse(readFileSync(path, 'utf8'));
-        expect(written.title).toBe('Customer Aging Report');
-    });
-    it('refuses to overwrite an existing file (returns suffixed name)', async () => {
-        const env = buildEnvelope({
-            artefactType: 'spec-gap',
-            extract: SAMPLE_EXTRACT,
-            author: { name: 'Laeeq', role: 'consultant' },
-            source: { skill: 'abap-spec-gap', skillVersion: '1.0', input: '...', tokensUsed: 5000, model: 'claude-opus-4-7' },
-        });
-        const path1 = await saveProject(env, { cwd: tmp });
-        const path2 = await saveProject(env, { cwd: tmp });
-        expect(path1).not.toBe(path2);
-        expect(path2).toMatch(/-v1-\d+\.cspeach\.json$/);
-    });
-});

package/dist/projects/__tests__/status.test.js

@@ -1,208 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { mkdtempSync, rmSync, writeFileSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { renderStatus, readProjectFile, renderMultiFileStatus } from '../status.js';
-import { buildEnvelope } from '../build.js';
-const SAMPLE = buildEnvelope({
-    artefactType: 'spec-gap',
-    extract: {
-        title: 'Customer Aging Report',
-        content: { inputType: 'User Story', inferredDomain: 'FI-AR', sections: [] },
-        items: [
-            { id: 'gap-001', status: 'answered', answer: 'BSID', answeredBy: { name: 'Sarah', role: 'BA' }, answeredAt: '2026-05-04T09:00:00Z', comments: [] },
-            { id: 'gap-002', status: 'open', answer: null, answeredBy: null, answeredAt: null, comments: [] },
-            { id: 'gap-003', status: 'deferred', answer: null, answeredBy: null, answeredAt: null, comments: [] },
-            { id: 'gap-004', status: 'n-a', answer: null, answeredBy: null, answeredAt: null, comments: [] },
-        ],
-    },
-    author: { name: 'Laeeq', role: 'consultant' },
-    source: { skill: 'abap-spec-gap', skillVersion: '1.0', input: '...', tokensUsed: 100, model: 'm' },
-});
-describe('renderStatus', () => {
-    it('renders correct counts', () => {
-        const out = renderStatus(SAMPLE);
-        expect(out).toContain('Customer Aging Report');
-        expect(out).toContain('1 / 4 answered');
-        expect(out).toContain('1 deferred');
-        expect(out).toContain('1 n/a');
-        expect(out).toContain('1 open');
-    });
-});
-describe('readProjectFile', () => {
-    it('reads and validates a project file', () => {
-        const tmp = mkdtempSync(join(tmpdir(), 'cspeach-status-'));
-        try {
-            const path = join(tmp, 'sample.cspeach.json');
-            writeFileSync(path, JSON.stringify(SAMPLE, null, 2));
-            const env = readProjectFile(path);
-            expect(env.title).toBe('Customer Aging Report');
-        }
-        finally {
-            rmSync(tmp, { recursive: true, force: true });
-        }
-    });
-    it('throws on invalid file', () => {
-        const tmp = mkdtempSync(join(tmpdir(), 'cspeach-status-'));
-        try {
-            const path = join(tmp, 'bad.cspeach.json');
-            writeFileSync(path, '{}');
-            expect(() => readProjectFile(path)).toThrow();
-        }
-        finally {
-            rmSync(tmp, { recursive: true, force: true });
-        }
-    });
-});
-/**
- * Build two v3 envelopes that share the same id and a common ancestor (v2)
- * in their history arrays, but diverge in interaction.items at v3.
- * Spec §9: "share the same id AND their history arrays agree up to the most
- * recent entry that pre-dates the divergence".
- */
-function makeConflictPair() {
-    const sharedId = '22222222-2222-4222-8222-222222222222';
-    const sharedAncestorHistory = [
-        { at: '2026-05-03T10:00:00Z', by: { name: 'Laeeq', role: 'consultant' }, action: 'created' },
-        { at: '2026-05-04T08:00:00Z', by: { name: 'Sarah', role: 'BA' }, action: 'edited', summary: '0 gaps answered' },
-    ];
-    const baseItems = [
-        { id: 'gap-001', status: 'open', answer: null, answeredBy: null, answeredAt: null, comments: [] },
-        { id: 'gap-002', status: 'open', answer: null, answeredBy: null, answeredAt: null, comments: [] },
-    ];
-    const skeleton = {
-        schemaVersion: '1.0',
-        artefactType: 'spec-gap',
-        id: sharedId,
-        title: 'Customer Aging Report',
-        createdAt: '2026-05-03T10:00:00Z',
-        createdBy: { name: 'Laeeq', role: 'consultant' },
-        source: { skill: 'abap-spec-gap', skillVersion: '1.0', input: '...', tokensUsed: 100, model: 'm' },
-        promotedFrom: null,
-        content: { inputType: 'User Story', inferredDomain: 'FI-AR', sections: [] },
-    };
-    const fromSarah = {
-        ...skeleton,
-        version: 3,
-        lastEditedAt: '2026-05-04T09:30:00Z',
-        lastEditedBy: { name: 'Sarah', role: 'BA' },
-        history: [
-            ...sharedAncestorHistory,
-            { at: '2026-05-04T09:30:00Z', by: { name: 'Sarah', role: 'BA' }, action: 'edited', summary: '1 gap answered' },
-        ],
-        interaction: { items: [
-                { ...baseItems[0], status: 'answered', answer: 'Sarah answer', answeredBy: { name: 'Sarah', role: 'BA' }, answeredAt: '2026-05-04T09:30:00Z' },
-                baseItems[1],
-            ] },
-    };
-    const fromAhmed = {
-        ...skeleton,
-        version: 3,
-        lastEditedAt: '2026-05-04T11:00:00Z',
-        lastEditedBy: { name: 'Ahmed', role: 'BA' },
-        history: [
-            ...sharedAncestorHistory,
-            { at: '2026-05-04T11:00:00Z', by: { name: 'Ahmed', role: 'BA' }, action: 'edited', summary: '1 gap deferred' },
-        ],
-        interaction: { items: [
-                baseItems[0],
-                { ...baseItems[1], status: 'deferred' },
-            ] },
-    };
-    return [fromSarah, fromAhmed];
-}
-describe('renderMultiFileStatus', () => {
-    it('warns when two files share source ancestor but have different SHAs', async () => {
-        const [env1, env2] = makeConflictPair();
-        const out = await renderMultiFileStatus([env1, env2]);
-        expect(out).toContain('Two v3');
-        expect(out).toContain('files received');
-        expect(out).toContain('parallel edit');
-    });
-    it('does not warn when SHAs match (identical envelopes)', async () => {
-        const [env1] = makeConflictPair();
-        const out = await renderMultiFileStatus([env1, env1]);
-        expect(out).not.toContain('parallel edit');
-    });
-    it('does not warn when ids differ (different sources, no parallel edit)', async () => {
-        const [env1] = makeConflictPair();
-        const env2 = { ...env1, id: '33333333-3333-4333-8333-333333333333' };
-        const out = await renderMultiFileStatus([env1, env2]);
-        expect(out).not.toContain('parallel edit');
-    });
-});
-const baseEnv = {
-    schemaVersion: '1.0',
-    id: '11111111-1111-1111-1111-111111111111',
-    title: 'T',
-    createdAt: '2026-05-05T00:00:00Z',
-    createdBy: { name: 'A', role: 'consultant' },
-    lastEditedAt: '2026-05-05T09:00:00Z',
-    lastEditedBy: { name: 'B', role: 'arch' },
-    version: 1,
-    source: { skill: 's', skillVersion: '1.0', input: '', tokensUsed: 0, model: 'claude-opus-4-7' },
-    promotedFrom: null,
-    history: [{ at: '2026-05-05T00:00:00Z', by: { name: 'A', role: 'consultant' }, action: 'created' }],
-};
-describe('renderStatus — design', () => {
-    const env = {
-        ...baseEnv, artefactType: 'design',
-        content: { objects: [
-                { id: 'obj-001', name: 'ZCL_AGING', type: 'CLAS', package: 'ZAGING', dependsOn: [] },
-                { id: 'obj-002', name: 'ZTBL_AGING', type: 'TABL', package: 'ZAGING', dependsOn: ['obj-001'] },
-            ] },
-        interaction: { items: [
-                { id: 'obj-001', status: 'keep', answer: null, answeredBy: null, answeredAt: null, comments: [] },
-                { id: 'obj-002', status: 'redesign', answer: 'split into two', answeredBy: { name: 'B', role: 'arch' }, answeredAt: '2026-05-05T09:00:00Z', comments: [] },
-            ] },
-    };
-    it('renders status counts using design enum', () => {
-        const out = renderStatus(env);
-        expect(out).toContain('design (v1)');
-        expect(out).toContain('1 / 2 keep');
-        expect(out).toContain('1 redesign');
-    });
-    it('renders open-list as "no decisions yet" entries (status not in primary/secondary terminals shown explicitly is N/A in design — design has no "open" terminal)', () => {
-        const out = renderStatus(env);
-        expect(out).not.toContain('Open (');
-    });
-});
-describe('renderStatus — estimate', () => {
-    const env = {
-        ...baseEnv, artefactType: 'estimate',
-        content: { components: [
-                { id: 'cmp-001', name: 'Aging core', hoursOptimistic: 6, hoursRealistic: 10, hoursPessimistic: 16, risks: [] },
-                { id: 'cmp-002', name: 'Currency conversion', hoursOptimistic: 4, hoursRealistic: 8, hoursPessimistic: 14, risks: ['unclear FX rates'] },
-            ] },
-        interaction: { items: [
-                { id: 'cmp-001', status: 'confirmed', answer: 'OK', answeredBy: { name: 'B', role: 'PM' }, answeredAt: '2026-05-05T09:00:00Z', comments: [] },
-                { id: 'cmp-002', status: 'disputed', answer: 'too low', answeredBy: { name: 'B', role: 'PM' }, answeredAt: '2026-05-05T09:00:00Z', comments: [] },
-            ] },
-    };
-    it('renders status counts using estimate enum', () => {
-        const out = renderStatus(env);
-        expect(out).toContain('estimate (v1)');
-        expect(out).toContain('1 / 2 confirmed');
-        expect(out).toContain('1 disputed');
-    });
-});
-describe('renderStatus — promote hint by artefact', () => {
-    it('spec-gap with answered items recommends /abap-design --from', () => {
-        const env = {
-            ...baseEnv, artefactType: 'spec-gap',
-            content: { inputType: 'X', inferredDomain: 'FI', sections: [{ name: 'G', audience: 'business', gaps: [{ id: 'gap-001', severity: 'BLOCKER', text: 'Q' }] }] },
-            interaction: { items: [{ id: 'gap-001', status: 'answered', answer: 'A', answeredBy: { name: 'B', role: 'BA' }, answeredAt: '2026-05-05T09:00:00Z', comments: [] }] },
-        };
-        const out = renderStatus(env);
-        expect(out).toContain('/abap-design --from');
-    });
-    it('design with kept items recommends /abap-estimate --from', () => {
-        const env = {
-            ...baseEnv, artefactType: 'design',
-            content: { objects: [{ id: 'obj-001', name: 'ZCL', type: 'CLAS', package: 'P', dependsOn: [] }] },
-            interaction: { items: [{ id: 'obj-001', status: 'keep', answer: null, answeredBy: null, answeredAt: null, comments: [] }] },
-        };
-        const out = renderStatus(env);
-        expect(out).toContain('/abap-estimate --from');
-    });
-});

package/dist/projects/__tests__/types.test.js

@@ -1,41 +0,0 @@
-import { describe, it, expectTypeOf } from 'vitest';
-describe('cspeach types', () => {
-    it('exposes ArtefactType union of 11 known kinds (v0.7 adds modernize-result + test-coverage)', () => {
-        expectTypeOf().toEqualTypeOf();
-    });
-    it('exposes per-artefact status enums', () => {
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-    });
-    it('Identity is { name, role }', () => {
-        expectTypeOf().toEqualTypeOf();
-    });
-    it('CspeachEnvelope has required envelope fields', () => {
-        expectTypeOf().toMatchTypeOf();
-    });
-    it('Comment has required shape', () => {
-        expectTypeOf().toMatchTypeOf();
-    });
-    it('HistoryEntry action is the literal union', () => {
-        expectTypeOf().toEqualTypeOf();
-    });
-    it('InteractionItem.comments is Comment[]', () => {
-        expectTypeOf().toEqualTypeOf();
-    });
-    it('SpecGapContent.sections is an array', () => {
-        expectTypeOf().toBeArray();
-    });
-    it('narrows content via artefactType discriminator', () => {
-        expectTypeOf().toEqualTypeOf();
-        expectTypeOf().toEqualTypeOf();
-    });
-});

package/dist/projects/__tests__/validate.test.js

@@ -1,165 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { validateEnvelope } from '../validate.js';
-const VALID_MIN_SPEC_GAP = {
-    schemaVersion: '1.0',
-    artefactType: 'spec-gap',
-    id: '11111111-1111-4111-8111-111111111111',
-    title: 'Customer Aging Report',
-    createdAt: '2026-05-04T10:00:00Z',
-    createdBy: { name: 'Laeeq', role: 'consultant' },
-    lastEditedAt: '2026-05-04T10:00:00Z',
-    lastEditedBy: { name: 'Laeeq', role: 'consultant' },
-    version: 1,
-    source: { skill: 'abap-spec-gap', skillVersion: '1.0', input: '...', tokensUsed: 1000, model: 'claude-opus-4-7' },
-    promotedFrom: null,
-    content: { inputType: 'User Story', inferredDomain: 'FI-AR', sections: [] },
-    interaction: { items: [] },
-    history: [{ at: '2026-05-04T10:00:00Z', by: { name: 'Laeeq', role: 'consultant' }, action: 'created' }],
-};
-describe('validateEnvelope', () => {
-    it('accepts a minimal valid spec-gap envelope', () => {
-        const result = validateEnvelope(VALID_MIN_SPEC_GAP);
-        expect(result.ok).toBe(true);
-    });
-    it('rejects missing schemaVersion', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP };
-        delete bad.schemaVersion;
-        const result = validateEnvelope(bad);
-        expect(result.ok).toBe(false);
-        if (!result.ok) {
-            expect(result.error.code).toBe('MISSING_FIELD');
-            expect(result.error.field).toBe('schemaVersion');
-        }
-    });
-    it('rejects unknown artefactType', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP, artefactType: 'invalid' };
-        const result = validateEnvelope(bad);
-        expect(result.ok).toBe(false);
-        if (!result.ok)
-            expect(result.error.code).toBe('INVALID_VALUE');
-    });
-    it('rejects negative version', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP, version: -1 };
-        const result = validateEnvelope(bad);
-        expect(result.ok).toBe(false);
-    });
-    it('rejects version mismatch with history.length', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP, version: 5 }; // history has 1 entry
-        const result = validateEnvelope(bad);
-        expect(result.ok).toBe(false);
-        if (!result.ok)
-            expect(result.error.code).toBe('VERSION_MISMATCH');
-    });
-    it('rejects items without stable id', () => {
-        const bad = {
-            ...VALID_MIN_SPEC_GAP,
-            interaction: { items: [{ status: 'open', answer: null, answeredBy: null, answeredAt: null, comments: [] }] },
-        };
-        const result = validateEnvelope(bad);
-        expect(result.ok).toBe(false);
-        if (!result.ok)
-            expect(result.error.code).toBe('MISSING_FIELD');
-    });
-    it('rejects status not in spec-gap enum', () => {
-        const bad = {
-            ...VALID_MIN_SPEC_GAP,
-            interaction: { items: [{ id: 'gap-001', status: 'keep', answer: null, answeredBy: null, answeredAt: null, comments: [] }] },
-        };
-        const result = validateEnvelope(bad);
-        expect(result.ok).toBe(false);
-        if (!result.ok)
-            expect(result.error.code).toBe('INVALID_VALUE');
-    });
-    it('warns on schemaVersion newer than supported', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP, schemaVersion: '2.0' };
-        const result = validateEnvelope(bad);
-        expect(result.ok).toBe(true); // best-effort parse
-        expect(result.warnings).toContain('SCHEMA_TOO_NEW');
-    });
-    it('rejects null root', () => {
-        expect(validateEnvelope(null).ok).toBe(false);
-    });
-    it('rejects non-object root (array)', () => {
-        expect(validateEnvelope([]).ok).toBe(false);
-    });
-    it('rejects non-object root (string)', () => {
-        expect(validateEnvelope('hello').ok).toBe(false);
-    });
-    it('treats null required field as MISSING_FIELD', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP, title: null };
-        const r = validateEnvelope(bad);
-        expect(r.ok).toBe(false);
-        if (!r.ok)
-            expect(r.error.code).toBe('MISSING_FIELD');
-    });
-    it('treats empty-string required field as MISSING_FIELD', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP, title: '' };
-        const r = validateEnvelope(bad);
-        expect(r.ok).toBe(false);
-        if (!r.ok)
-            expect(r.error.code).toBe('MISSING_FIELD');
-    });
-    it('rejects malformed schemaVersion (not <int>.<int>)', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP, schemaVersion: 'hello' };
-        const r = validateEnvelope(bad);
-        expect(r.ok).toBe(false);
-        if (!r.ok)
-            expect(r.error.code).toBe('INVALID_VALUE');
-    });
-    it('accepts patch version of supported major.minor (1.0.5)', () => {
-        const ok = { ...VALID_MIN_SPEC_GAP, schemaVersion: '1.0.5' };
-        const r = validateEnvelope(ok);
-        expect(r.ok).toBe(true);
-    });
-    it('warns SCHEMA_TOO_NEW on higher minor (1.1)', () => {
-        const future = { ...VALID_MIN_SPEC_GAP, schemaVersion: '1.1' };
-        const r = validateEnvelope(future);
-        expect(r.ok).toBe(true);
-        expect(r.warnings).toContain('SCHEMA_TOO_NEW');
-    });
-    it('warns SCHEMA_TOO_NEW on higher major (2.0)', () => {
-        const future = { ...VALID_MIN_SPEC_GAP, schemaVersion: '2.0' };
-        const r = validateEnvelope(future);
-        expect(r.ok).toBe(true);
-        expect(r.warnings).toContain('SCHEMA_TOO_NEW');
-    });
-    it('rejects null in interaction.items[]', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP, interaction: { items: [null] } };
-        expect(validateEnvelope(bad).ok).toBe(false);
-    });
-    it('rejects string in interaction.items[]', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP, interaction: { items: ['oops'] } };
-        expect(validateEnvelope(bad).ok).toBe(false);
-    });
-    it('rejects duplicate item id', () => {
-        const bad = {
-            ...VALID_MIN_SPEC_GAP,
-            interaction: { items: [
-                    { id: 'gap-001', status: 'open', answer: null, answeredBy: null, answeredAt: null, comments: [] },
-                    { id: 'gap-001', status: 'open', answer: null, answeredBy: null, answeredAt: null, comments: [] },
-                ] },
-        };
-        const r = validateEnvelope(bad);
-        expect(r.ok).toBe(false);
-        if (!r.ok) {
-            expect(r.error.code).toBe('INVALID_VALUE');
-            expect(r.error.field).toMatch(/interaction\.items\[1\]\.id/);
-        }
-    });
-    it('rejects history entry with missing at', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP,
-            history: [{ by: { name: 'L', role: 'consultant' }, action: 'created' }] };
-        const r = validateEnvelope(bad);
-        expect(r.ok).toBe(false);
-        if (!r.ok)
-            expect(r.error.code).toBe('INVALID_HISTORY');
-    });
-    it('rejects history entry with invalid action', () => {
-        const bad = { ...VALID_MIN_SPEC_GAP,
-            history: [{ at: '2026-05-04T10:00:00Z', by: { name: 'L', role: 'consultant' }, action: 'deleted' }] };
-        const r = validateEnvelope(bad);
-        expect(r.ok).toBe(false);
-        if (!r.ok)
-            expect(r.error.code).toBe('INVALID_HISTORY');
-    });
-});