@lumenflow/cli 2.3.2 → 2.5.0

package/dist/__tests__/plan-link.test.js

@@ -0,0 +1,239 @@
+/**
+ * Tests for plan:link command (WU-1313)
+ *
+ * The plan:link command links existing plan files to WUs (via spec_refs)
+ * or initiatives (via related_plan). This replaces the initiative:plan command.
+ *
+ * TDD: These tests are written BEFORE the implementation.
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { existsSync, mkdirSync, rmSync, writeFileSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
+import { tmpdir } from 'node:os';
+import { parseYAML, stringifyYAML } from '@lumenflow/core/dist/wu-yaml.js';
+/** Test constants - avoid sonarjs/no-duplicate-string */
+const TEST_WU_DIR = 'docs/04-operations/tasks/wu';
+const TEST_INIT_DIR = 'docs/04-operations/tasks/initiatives';
+const TEST_PLANS_DIR = 'docs/04-operations/plans';
+const TEST_WU_ID = 'WU-1313';
+const TEST_INIT_ID = 'INIT-001';
+const TEST_WU_PLAN_URI = `lumenflow://plans/${TEST_WU_ID}-plan.md`;
+const TEST_INIT_PLAN_URI = `lumenflow://plans/${TEST_INIT_ID}-plan.md`;
+const TEST_LANE = 'Framework: CLI';
+const TEST_INIT_SLUG = 'test-initiative';
+const TEST_INIT_TITLE = 'Test Initiative';
+const TEST_WU_TITLE = 'Test WU';
+const TEST_STATUS_OPEN = 'open';
+const TEST_DATE = '2026-01-25';
+// Mock modules before importing
+vi.mock('@lumenflow/core/dist/git-adapter.js', () => ({
+    getGitForCwd: vi.fn(() => ({
+        branch: vi.fn().mockResolvedValue({ current: 'main' }),
+        status: vi.fn().mockResolvedValue({ isClean: () => true }),
+    })),
+}));
+vi.mock('@lumenflow/core/dist/wu-helpers.js', () => ({
+    ensureOnMain: vi.fn().mockResolvedValue(undefined),
+}));
+vi.mock('@lumenflow/core/dist/micro-worktree.js', () => ({
+    withMicroWorktree: vi.fn(async ({ execute }) => {
+        const tempDir = join(tmpdir(), `plan-link-test-${Date.now()}`);
+        mkdirSync(tempDir, { recursive: true });
+        return execute({ worktreePath: tempDir });
+    }),
+}));
+describe('plan:link command', () => {
+    let tempDir;
+    let originalCwd;
+    beforeEach(() => {
+        tempDir = join(tmpdir(), `plan-link-test-${Date.now()}`);
+        mkdirSync(tempDir, { recursive: true });
+        originalCwd = process.cwd();
+    });
+    afterEach(() => {
+        process.chdir(originalCwd);
+        if (existsSync(tempDir)) {
+            rmSync(tempDir, { recursive: true, force: true });
+        }
+        vi.clearAllMocks();
+    });
+    describe('linkPlanToWU', () => {
+        it('should add spec_refs field to WU YAML', async () => {
+            const { linkPlanToWU } = await import('../plan-link.js');
+            // Setup mock WU file
+            const wuDir = join(tempDir, ...TEST_WU_DIR.split('/'));
+            mkdirSync(wuDir, { recursive: true });
+            const wuPath = join(wuDir, `${TEST_WU_ID}.yaml`);
+            const wuDoc = {
+                id: TEST_WU_ID,
+                title: TEST_WU_TITLE,
+                lane: TEST_LANE,
+                status: 'ready',
+                type: 'feature',
+            };
+            writeFileSync(wuPath, stringifyYAML(wuDoc));
+            // Link plan
+            const changed = linkPlanToWU(tempDir, TEST_WU_ID, TEST_WU_PLAN_URI);
+            expect(changed).toBe(true);
+            // Verify the file was updated
+            const updated = parseYAML(readFileSync(wuPath, 'utf-8'));
+            expect(updated.spec_refs).toContain(TEST_WU_PLAN_URI);
+        });
+        it('should append to existing spec_refs', async () => {
+            const { linkPlanToWU } = await import('../plan-link.js');
+            // Setup mock WU file with existing spec_refs
+            const wuDir = join(tempDir, ...TEST_WU_DIR.split('/'));
+            mkdirSync(wuDir, { recursive: true });
+            const wuPath = join(wuDir, `${TEST_WU_ID}.yaml`);
+            const wuDoc = {
+                id: TEST_WU_ID,
+                title: TEST_WU_TITLE,
+                lane: TEST_LANE,
+                status: 'ready',
+                type: 'feature',
+                spec_refs: ['lumenflow://plans/existing-plan.md'],
+            };
+            writeFileSync(wuPath, stringifyYAML(wuDoc));
+            // Link additional plan
+            const changed = linkPlanToWU(tempDir, TEST_WU_ID, TEST_WU_PLAN_URI);
+            expect(changed).toBe(true);
+            // Verify both refs are present
+            const updated = parseYAML(readFileSync(wuPath, 'utf-8'));
+            expect(updated.spec_refs).toContain('lumenflow://plans/existing-plan.md');
+            expect(updated.spec_refs).toContain(TEST_WU_PLAN_URI);
+        });
+        it('should be idempotent if plan already linked', async () => {
+            const { linkPlanToWU } = await import('../plan-link.js');
+            // Setup mock WU file with plan already linked
+            const wuDir = join(tempDir, ...TEST_WU_DIR.split('/'));
+            mkdirSync(wuDir, { recursive: true });
+            const wuPath = join(wuDir, `${TEST_WU_ID}.yaml`);
+            const wuDoc = {
+                id: TEST_WU_ID,
+                title: TEST_WU_TITLE,
+                lane: TEST_LANE,
+                status: 'ready',
+                type: 'feature',
+                spec_refs: [TEST_WU_PLAN_URI],
+            };
+            writeFileSync(wuPath, stringifyYAML(wuDoc));
+            // Link same plan again
+            const changed = linkPlanToWU(tempDir, TEST_WU_ID, TEST_WU_PLAN_URI);
+            expect(changed).toBe(false);
+        });
+        it('should throw if WU not found', async () => {
+            const { linkPlanToWU } = await import('../plan-link.js');
+            expect(() => linkPlanToWU(tempDir, 'WU-9999', 'lumenflow://plans/plan.md')).toThrow();
+        });
+    });
+    describe('linkPlanToInitiative', () => {
+        it('should add related_plan field to initiative YAML', async () => {
+            const { linkPlanToInitiative } = await import('../plan-link.js');
+            // Setup mock initiative file
+            const initDir = join(tempDir, ...TEST_INIT_DIR.split('/'));
+            mkdirSync(initDir, { recursive: true });
+            const initPath = join(initDir, `${TEST_INIT_ID}.yaml`);
+            const initDoc = {
+                id: TEST_INIT_ID,
+                slug: TEST_INIT_SLUG,
+                title: TEST_INIT_TITLE,
+                status: TEST_STATUS_OPEN,
+                created: TEST_DATE,
+            };
+            writeFileSync(initPath, stringifyYAML(initDoc));
+            // Link plan
+            const changed = linkPlanToInitiative(tempDir, TEST_INIT_ID, TEST_INIT_PLAN_URI);
+            expect(changed).toBe(true);
+            // Verify the file was updated
+            const updated = parseYAML(readFileSync(initPath, 'utf-8'));
+            expect(updated.related_plan).toBe(TEST_INIT_PLAN_URI);
+        });
+        it('should be idempotent if plan already linked', async () => {
+            const { linkPlanToInitiative } = await import('../plan-link.js');
+            // Setup mock initiative file with plan already linked
+            const initDir = join(tempDir, ...TEST_INIT_DIR.split('/'));
+            mkdirSync(initDir, { recursive: true });
+            const initPath = join(initDir, `${TEST_INIT_ID}.yaml`);
+            const initDoc = {
+                id: TEST_INIT_ID,
+                slug: TEST_INIT_SLUG,
+                title: TEST_INIT_TITLE,
+                status: TEST_STATUS_OPEN,
+                created: TEST_DATE,
+                related_plan: TEST_INIT_PLAN_URI,
+            };
+            writeFileSync(initPath, stringifyYAML(initDoc));
+            // Link same plan again
+            const changed = linkPlanToInitiative(tempDir, TEST_INIT_ID, TEST_INIT_PLAN_URI);
+            expect(changed).toBe(false);
+        });
+        it('should warn but proceed if replacing existing plan', async () => {
+            const { linkPlanToInitiative } = await import('../plan-link.js');
+            const consoleSpy = vi.spyOn(console, 'warn').mockImplementation(() => { });
+            // Setup mock initiative with different plan
+            const initDir = join(tempDir, ...TEST_INIT_DIR.split('/'));
+            mkdirSync(initDir, { recursive: true });
+            const initPath = join(initDir, `${TEST_INIT_ID}.yaml`);
+            const initDoc = {
+                id: TEST_INIT_ID,
+                slug: TEST_INIT_SLUG,
+                title: TEST_INIT_TITLE,
+                status: TEST_STATUS_OPEN,
+                created: TEST_DATE,
+                related_plan: 'lumenflow://plans/old-plan.md',
+            };
+            writeFileSync(initPath, stringifyYAML(initDoc));
+            // Link new plan
+            const changed = linkPlanToInitiative(tempDir, TEST_INIT_ID, 'lumenflow://plans/new-plan.md');
+            expect(changed).toBe(true);
+            expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('Replacing existing'));
+            consoleSpy.mockRestore();
+        });
+    });
+    describe('validatePlanExists', () => {
+        it('should pass for existing plan file', async () => {
+            const { validatePlanExists } = await import('../plan-link.js');
+            const plansDir = join(tempDir, ...TEST_PLANS_DIR.split('/'));
+            mkdirSync(plansDir, { recursive: true });
+            const planPath = join(plansDir, `${TEST_WU_ID}-plan.md`);
+            writeFileSync(planPath, '# Plan');
+            expect(() => validatePlanExists(tempDir, TEST_WU_PLAN_URI)).not.toThrow();
+        });
+        it('should throw for non-existent plan file', async () => {
+            const { validatePlanExists } = await import('../plan-link.js');
+            expect(() => validatePlanExists(tempDir, 'lumenflow://plans/nonexistent.md')).toThrow();
+        });
+    });
+    describe('resolveTargetType', () => {
+        it('should detect WU IDs', async () => {
+            const { resolveTargetType } = await import('../plan-link.js');
+            expect(resolveTargetType(TEST_WU_ID)).toBe('wu');
+            expect(resolveTargetType('WU-001')).toBe('wu');
+            expect(resolveTargetType('WU-99999')).toBe('wu');
+        });
+        it('should detect initiative IDs', async () => {
+            const { resolveTargetType } = await import('../plan-link.js');
+            expect(resolveTargetType(TEST_INIT_ID)).toBe('initiative');
+            expect(resolveTargetType('INIT-TOOLING')).toBe('initiative');
+        });
+        it('should throw for invalid IDs', async () => {
+            const { resolveTargetType } = await import('../plan-link.js');
+            expect(() => resolveTargetType('invalid')).toThrow();
+            expect(() => resolveTargetType('')).toThrow();
+        });
+    });
+});
+describe('plan:link CLI exports', () => {
+    it('should export main function for CLI entry', async () => {
+        const planLink = await import('../plan-link.js');
+        expect(typeof planLink.main).toBe('function');
+    });
+    it('should export all required functions', async () => {
+        const planLink = await import('../plan-link.js');
+        expect(typeof planLink.linkPlanToWU).toBe('function');
+        expect(typeof planLink.linkPlanToInitiative).toBe('function');
+        expect(typeof planLink.validatePlanExists).toBe('function');
+        expect(typeof planLink.resolveTargetType).toBe('function');
+        expect(typeof planLink.LOG_PREFIX).toBe('string');
+    });
+});

package/dist/__tests__/plan-promote.test.js

@@ -0,0 +1,181 @@
+/**
+ * Tests for plan:promote command (WU-1313)
+ *
+ * The plan:promote command promotes a plan from draft to approved status,
+ * or creates WUs from plan sections.
+ *
+ * TDD: These tests are written BEFORE the implementation.
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { existsSync, mkdirSync, rmSync, writeFileSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
+import { tmpdir } from 'node:os';
+/** Test constants - avoid sonarjs/no-duplicate-string */
+const TEST_PLANS_DIR = 'docs/04-operations/plans';
+const TEST_WU_ID = 'WU-1313';
+// Mock modules before importing
+vi.mock('@lumenflow/core/dist/git-adapter.js', () => ({
+    getGitForCwd: vi.fn(() => ({
+        branch: vi.fn().mockResolvedValue({ current: 'main' }),
+        status: vi.fn().mockResolvedValue({ isClean: () => true }),
+    })),
+}));
+vi.mock('@lumenflow/core/dist/wu-helpers.js', () => ({
+    ensureOnMain: vi.fn().mockResolvedValue(undefined),
+}));
+vi.mock('@lumenflow/core/dist/micro-worktree.js', () => ({
+    withMicroWorktree: vi.fn(async ({ execute }) => {
+        const tempDir = join(tmpdir(), `plan-promote-test-${Date.now()}`);
+        mkdirSync(tempDir, { recursive: true });
+        return execute({ worktreePath: tempDir });
+    }),
+}));
+describe('plan:promote command', () => {
+    let tempDir;
+    let originalCwd;
+    beforeEach(() => {
+        tempDir = join(tmpdir(), `plan-promote-test-${Date.now()}`);
+        mkdirSync(tempDir, { recursive: true });
+        originalCwd = process.cwd();
+    });
+    afterEach(() => {
+        process.chdir(originalCwd);
+        if (existsSync(tempDir)) {
+            rmSync(tempDir, { recursive: true, force: true });
+        }
+        vi.clearAllMocks();
+    });
+    describe('promotePlan', () => {
+        it('should add approved status marker to plan', async () => {
+            const { promotePlan } = await import('../plan-promote.js');
+            // Setup plan file
+            const plansDir = join(tempDir, ...TEST_PLANS_DIR.split('/'));
+            mkdirSync(plansDir, { recursive: true });
+            const planPath = join(plansDir, `${TEST_WU_ID}-plan.md`);
+            writeFileSync(planPath, `# WU-1313 Plan
+
+Created: 2026-02-01
+
+## Goal
+
+Implement plan tooling.
+`);
+            // Promote plan
+            const changed = promotePlan(planPath);
+            expect(changed).toBe(true);
+            const content = readFileSync(planPath, 'utf-8');
+            expect(content).toContain('Status: approved');
+            expect(content).toContain('Approved:');
+        });
+        it('should return false if plan already approved', async () => {
+            const { promotePlan } = await import('../plan-promote.js');
+            // Setup plan file with approved status
+            const plansDir = join(tempDir, ...TEST_PLANS_DIR.split('/'));
+            mkdirSync(plansDir, { recursive: true });
+            const planPath = join(plansDir, `${TEST_WU_ID}-plan.md`);
+            writeFileSync(planPath, `# WU-1313 Plan
+
+Created: 2026-02-01
+Status: approved
+Approved: 2026-02-01
+
+## Goal
+
+Implement plan tooling.
+`);
+            // Try to promote again
+            const changed = promotePlan(planPath);
+            expect(changed).toBe(false);
+        });
+        it('should throw if plan file not found', async () => {
+            const { promotePlan } = await import('../plan-promote.js');
+            const planPath = join(tempDir, 'nonexistent.md');
+            expect(() => promotePlan(planPath)).toThrow();
+        });
+    });
+    describe('validatePlanComplete', () => {
+        it('should pass for complete plan', async () => {
+            const { validatePlanComplete } = await import('../plan-promote.js');
+            // Setup complete plan file
+            const plansDir = join(tempDir, ...TEST_PLANS_DIR.split('/'));
+            mkdirSync(plansDir, { recursive: true });
+            const planPath = join(plansDir, `${TEST_WU_ID}-plan.md`);
+            writeFileSync(planPath, `# WU-1313 Plan
+
+Created: 2026-02-01
+
+## Goal
+
+Clear goal statement here.
+
+## Scope
+
+- In scope: A
+- Out of scope: B
+
+## Approach
+
+Step 1: Do X
+Step 2: Do Y
+`);
+            const result = validatePlanComplete(planPath);
+            expect(result.valid).toBe(true);
+            expect(result.errors).toHaveLength(0);
+        });
+        it('should fail for plan with empty sections', async () => {
+            const { validatePlanComplete } = await import('../plan-promote.js');
+            // Setup incomplete plan file
+            const plansDir = join(tempDir, ...TEST_PLANS_DIR.split('/'));
+            mkdirSync(plansDir, { recursive: true });
+            const planPath = join(plansDir, `${TEST_WU_ID}-plan.md`);
+            writeFileSync(planPath, `# WU-1313 Plan
+
+Created: 2026-02-01
+
+## Goal
+
+## Scope
+
+## Approach
+`);
+            const result = validatePlanComplete(planPath);
+            expect(result.valid).toBe(false);
+            expect(result.errors.length).toBeGreaterThan(0);
+            expect(result.errors.some((e) => e.includes('Goal'))).toBe(true);
+        });
+    });
+    describe('getPlanPath', () => {
+        it('should resolve plan path from ID', async () => {
+            const { getPlanPath } = await import('../plan-promote.js');
+            // Setup plan file
+            const plansDir = join(tempDir, ...TEST_PLANS_DIR.split('/'));
+            mkdirSync(plansDir, { recursive: true });
+            const planPath = join(plansDir, `${TEST_WU_ID}-plan.md`);
+            writeFileSync(planPath, '# Plan');
+            process.chdir(tempDir);
+            const resolved = getPlanPath('WU-1313');
+            expect(resolved).toContain(`${TEST_WU_ID}-plan.md`);
+        });
+    });
+    describe('getCommitMessage', () => {
+        it('should generate correct commit message', async () => {
+            const { getCommitMessage } = await import('../plan-promote.js');
+            expect(getCommitMessage('WU-1313')).toBe('docs: promote wu-1313 plan to approved');
+            expect(getCommitMessage('INIT-001')).toBe('docs: promote init-001 plan to approved');
+        });
+    });
+});
+describe('plan:promote CLI exports', () => {
+    it('should export main function for CLI entry', async () => {
+        const planPromote = await import('../plan-promote.js');
+        expect(typeof planPromote.main).toBe('function');
+    });
+    it('should export all required functions', async () => {
+        const planPromote = await import('../plan-promote.js');
+        expect(typeof planPromote.promotePlan).toBe('function');
+        expect(typeof planPromote.validatePlanComplete).toBe('function');
+        expect(typeof planPromote.getPlanPath).toBe('function');
+        expect(typeof planPromote.getCommitMessage).toBe('function');
+        expect(typeof planPromote.LOG_PREFIX).toBe('string');
+    });
+});

package/dist/__tests__/wu-create-strict.test.js

@@ -0,0 +1,118 @@
+/**
+ * @file wu-create-strict.test.ts
+ * Test suite for wu:create strict validation behavior (WU-1329)
+ *
+ * WU-1329: Make wu:create run strict validation by default
+ *
+ * Tests:
+ * - Strict validation runs by default (validates code_paths/test_paths exist)
+ * - --no-strict flag bypasses strict validation
+ * - --no-strict usage is logged
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+// Import test utilities (WU-1329)
+import { validateCreateSpec } from '../wu-create.js';
+// WU-1329: Constants for test file paths
+const NON_EXISTENT_CODE_PATH = 'non-existent/file.ts';
+const NON_EXISTENT_TEST_PATH = 'non-existent/test.test.ts';
+describe('wu:create strict validation (WU-1329)', () => {
+    describe('validateCreateSpec strict mode', () => {
+        const baseOpts = {
+            id: 'WU-9999',
+            lane: 'Framework: CLI',
+            title: 'Test WU',
+            priority: 'P2',
+            type: 'feature',
+            opts: {
+                description: 'Context: test context.\nProblem: test problem.\nSolution: test solution that exceeds minimum length requirement.',
+                acceptance: ['Acceptance criterion 1'],
+                codePaths: ['packages/@lumenflow/cli/src/wu-create.ts'],
+                testPathsUnit: ['packages/@lumenflow/cli/src/__tests__/wu-create-strict.test.ts'],
+                exposure: 'backend-only',
+                specRefs: ['lumenflow://plans/WU-9999-plan.md'],
+            },
+        };
+        it('should pass validation with valid spec in non-strict mode', () => {
+            const result = validateCreateSpec({
+                ...baseOpts,
+                opts: {
+                    ...baseOpts.opts,
+                    strict: false,
+                },
+            });
+            expect(result.valid).toBe(true);
+            expect(result.errors).toHaveLength(0);
+        });
+        // WU-1329: This test verifies that strict validation is the default
+        it('should validate code_paths existence by default (strict=true)', () => {
+            const result = validateCreateSpec({
+                ...baseOpts,
+                opts: {
+                    ...baseOpts.opts,
+                    codePaths: [NON_EXISTENT_CODE_PATH],
+                    // strict is not explicitly set - should default to true
+                },
+            });
+            // In strict mode, non-existent paths should be caught
+            expect(result.valid).toBe(false);
+            expect(result.errors.some((e) => e.includes('code_paths') || e.includes('not found'))).toBe(true);
+        });
+        // WU-1329: This test verifies --no-strict bypasses path validation
+        it('should skip code_paths existence check when strict=false', () => {
+            const result = validateCreateSpec({
+                ...baseOpts,
+                opts: {
+                    ...baseOpts.opts,
+                    codePaths: [NON_EXISTENT_CODE_PATH],
+                    strict: false,
+                },
+            });
+            // In non-strict mode, non-existent paths should not fail validation
+            // (other schema validation may still fail)
+            expect(result.errors.every((e) => !e.includes('not found') && !e.includes('does not exist'))).toBe(true);
+        });
+        // WU-1329: This test verifies test_paths validation in strict mode
+        it('should validate test_paths existence by default (strict=true)', () => {
+            const result = validateCreateSpec({
+                ...baseOpts,
+                opts: {
+                    ...baseOpts.opts,
+                    testPathsUnit: [NON_EXISTENT_TEST_PATH],
+                },
+            });
+            // In strict mode, non-existent test paths should be caught
+            expect(result.valid).toBe(false);
+            expect(result.errors.some((e) => e.includes('test') || e.includes('not found'))).toBe(true);
+        });
+    });
+    describe('--no-strict logging', () => {
+        let consoleSpy;
+        beforeEach(() => {
+            consoleSpy = vi.spyOn(console, 'warn').mockImplementation(() => { });
+        });
+        afterEach(() => {
+            consoleSpy.mockRestore();
+        });
+        // WU-1329: This test verifies --no-strict usage is logged
+        it('should log warning when --no-strict is used', () => {
+            validateCreateSpec({
+                id: 'WU-9999',
+                lane: 'Framework: CLI',
+                title: 'Test WU',
+                priority: 'P2',
+                type: 'feature',
+                opts: {
+                    description: 'Context: test context.\nProblem: test problem.\nSolution: test solution that exceeds minimum.',
+                    acceptance: ['Acceptance criterion'],
+                    codePaths: [NON_EXISTENT_CODE_PATH],
+                    testPathsUnit: [NON_EXISTENT_TEST_PATH],
+                    exposure: 'backend-only',
+                    specRefs: ['lumenflow://plans/WU-9999-plan.md'],
+                    strict: false,
+                },
+            });
+            // Should log that strict validation was bypassed
+            expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('strict validation bypassed'));
+        });
+    });
+});

package/dist/__tests__/wu-edit-strict.test.js

@@ -0,0 +1,109 @@
+/**
+ * @file wu-edit-strict.test.ts
+ * Test suite for wu:edit strict validation behavior (WU-1329)
+ *
+ * WU-1329: Make wu:edit run strict validation by default
+ *
+ * Tests:
+ * - applyEdits function works correctly (unit test for edit logic)
+ * - validateDoneWUEdits properly validates done WU edits
+ * - The main() function validates paths in strict mode (covered by e2e tests)
+ */
+import { describe, it, expect } from 'vitest';
+// Import test utilities (WU-1329)
+import { applyEdits, validateDoneWUEdits, validateExposureValue } from '../wu-edit.js';
+// WU-1329: Constants for test file paths
+const TEST_NEW_FILE_PATH = 'new/file.ts';
+const TEST_NEW_TEST_PATH = 'new/test.test.ts';
+describe('wu:edit strict validation (WU-1329)', () => {
+    describe('applyEdits code_paths handling', () => {
+        const baseWU = {
+            id: 'WU-9999',
+            title: 'Test WU',
+            lane: 'Framework: CLI',
+            type: 'feature',
+            status: 'ready',
+            priority: 'P2',
+            description: 'Context: test context.\nProblem: test problem.\nSolution: test solution that exceeds minimum length.',
+            acceptance: ['Existing criterion'],
+            code_paths: ['packages/@lumenflow/cli/src/wu-edit.ts'],
+            tests: {
+                unit: ['packages/@lumenflow/cli/src/__tests__/wu-edit-strict.test.ts'],
+                manual: [],
+                e2e: [],
+            },
+        };
+        // WU-1329: applyEdits transforms WU object - path validation is done separately
+        it('should apply code_paths edits correctly', () => {
+            const result = applyEdits(baseWU, {
+                codePaths: [TEST_NEW_FILE_PATH],
+                replaceCodePaths: true,
+            });
+            // applyEdits transforms the WU, validation happens later in main()
+            expect(result.code_paths).toContain(TEST_NEW_FILE_PATH);
+            expect(result.code_paths).toHaveLength(1);
+        });
+        it('should append code_paths by default', () => {
+            const result = applyEdits(baseWU, {
+                codePaths: [TEST_NEW_FILE_PATH],
+            });
+            expect(result.code_paths).toContain('packages/@lumenflow/cli/src/wu-edit.ts');
+            expect(result.code_paths).toContain(TEST_NEW_FILE_PATH);
+            expect(result.code_paths).toHaveLength(2);
+        });
+        it('should apply test_paths edits correctly', () => {
+            const result = applyEdits(baseWU, {
+                testPathsUnit: [TEST_NEW_TEST_PATH],
+            });
+            // Should append test paths
+            expect(result.tests.unit).toContain('packages/@lumenflow/cli/src/__tests__/wu-edit-strict.test.ts');
+            expect(result.tests.unit).toContain(TEST_NEW_TEST_PATH);
+        });
+    });
+    describe('validateDoneWUEdits', () => {
+        // WU-1329: Done WUs only allow initiative/phase/exposure edits
+        it('should allow exposure edits on done WUs', () => {
+            const result = validateDoneWUEdits({ exposure: 'backend-only' });
+            expect(result.valid).toBe(true);
+            expect(result.disallowedEdits).toHaveLength(0);
+        });
+        it('should disallow code_paths edits on done WUs', () => {
+            const result = validateDoneWUEdits({ codePaths: [TEST_NEW_FILE_PATH] });
+            expect(result.valid).toBe(false);
+            expect(result.disallowedEdits).toContain('--code-paths');
+        });
+        it('should disallow description edits on done WUs', () => {
+            const result = validateDoneWUEdits({ description: 'new description' });
+            expect(result.valid).toBe(false);
+            expect(result.disallowedEdits).toContain('--description');
+        });
+    });
+    describe('validateExposureValue', () => {
+        it('should accept valid exposure values', () => {
+            expect(validateExposureValue('ui').valid).toBe(true);
+            expect(validateExposureValue('api').valid).toBe(true);
+            expect(validateExposureValue('backend-only').valid).toBe(true);
+            expect(validateExposureValue('documentation').valid).toBe(true);
+        });
+        it('should reject invalid exposure values', () => {
+            const result = validateExposureValue('invalid-exposure');
+            expect(result.valid).toBe(false);
+            expect(result.error).toContain('Invalid exposure value');
+        });
+    });
+    describe('noStrict option support', () => {
+        // WU-1329: Verify CLI option parsing pattern
+        it('should support noStrict option in CLI argument pattern', () => {
+            // CLI uses --no-strict which Commander.js parses as noStrict
+            // The main() function converts this to strict: !noStrict
+            const cliArgs = { noStrict: true };
+            const strict = !cliArgs.noStrict;
+            expect(strict).toBe(false);
+        });
+        it('should default to strict=true when noStrict is undefined', () => {
+            const cliArgs = { noStrict: undefined };
+            const strict = !cliArgs.noStrict;
+            expect(strict).toBe(true);
+        });
+    });
+});