@lumenflow/cli 2.7.0 → 2.9.0

package/dist/__tests__/commands.test.js

@@ -0,0 +1,75 @@
+/**
+ * @file commands.test.ts
+ * Tests for lumenflow commands discovery feature (WU-1378)
+ *
+ * Tests the new commands subcommand that lists all available CLI commands
+ * grouped by category with brief descriptions.
+ */
+import { describe, it, expect } from 'vitest';
+import { getCommandsRegistry, formatCommandsOutput } from '../commands.js';
+describe('lumenflow commands', () => {
+    describe('getCommandsRegistry', () => {
+        it('should return command categories', () => {
+            const registry = getCommandsRegistry();
+            expect(registry).toBeDefined();
+            expect(Array.isArray(registry)).toBe(true);
+            expect(registry.length).toBeGreaterThan(0);
+        });
+        it('should include WU Lifecycle category with key commands', () => {
+            const registry = getCommandsRegistry();
+            const wuLifecycle = registry.find((cat) => cat.name === 'WU Lifecycle');
+            expect(wuLifecycle).toBeDefined();
+            expect(wuLifecycle.commands).toBeDefined();
+            const commandNames = wuLifecycle.commands.map((cmd) => cmd.name);
+            expect(commandNames).toContain('wu:create');
+            expect(commandNames).toContain('wu:claim');
+        });
+        it('should include Initiatives category', () => {
+            const registry = getCommandsRegistry();
+            const initiatives = registry.find((cat) => cat.name === 'Initiatives');
+            expect(initiatives).toBeDefined();
+            expect(initiatives.commands.some((cmd) => cmd.name === 'initiative:create')).toBe(true);
+        });
+        it('should include Gates & Quality category with gates command', () => {
+            const registry = getCommandsRegistry();
+            const gatesCategory = registry.find((cat) => cat.name === 'Gates & Quality');
+            expect(gatesCategory).toBeDefined();
+            expect(gatesCategory.commands.some((cmd) => cmd.name === 'gates')).toBe(true);
+        });
+        it('should have description for each command', () => {
+            const registry = getCommandsRegistry();
+            for (const category of registry) {
+                for (const cmd of category.commands) {
+                    expect(cmd.description).toBeDefined();
+                    expect(cmd.description.length).toBeGreaterThan(0);
+                }
+            }
+        });
+    });
+    describe('formatCommandsOutput', () => {
+        it('should include category headers', () => {
+            const output = formatCommandsOutput();
+            expect(output).toContain('WU Lifecycle');
+            expect(output).toContain('Initiatives');
+            expect(output).toContain('Gates & Quality');
+        });
+        it('should include command names and descriptions', () => {
+            const output = formatCommandsOutput();
+            expect(output).toContain('wu:create');
+            expect(output).toContain('wu:claim');
+            expect(output).toContain('initiative:create');
+            expect(output).toContain('gates');
+        });
+        it('should include hint to run --help for details', () => {
+            const output = formatCommandsOutput();
+            expect(output).toMatch(/--help/i);
+        });
+        it('should format output with clear grouping', () => {
+            const output = formatCommandsOutput();
+            const lines = output.split('\n');
+            // Should have multiple non-empty lines
+            const nonEmptyLines = lines.filter((line) => line.trim().length > 0);
+            expect(nonEmptyLines.length).toBeGreaterThan(10);
+        });
+    });
+});

package/dist/__tests__/doctor.test.js

@@ -0,0 +1,510 @@
+/**
+ * Doctor CLI Tests (WU-1386)
+ *
+ * Tests for the agent-friction checks extension to lumenflow doctor:
+ * - Managed-file dirty checks (uncommitted changes to managed files)
+ * - WU validity check (--deep flag runs wu:validate --all)
+ * - Worktree sanity check (orphan detection from wu:prune)
+ * - Exit codes: 0=healthy, 1=warnings, 2=errors
+ * - Auto-run after init (non-blocking)
+ *
+ * Note: These tests use real git operations (not mocked) to verify
+ * the actual implementation behavior.
+ */
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { join } from 'node:path';
+import { mkdtempSync, rmSync, writeFileSync, mkdirSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { execFileSync } from 'node:child_process';
+/**
+ * Import the module under test
+ */
+import { runDoctor, runDoctorForInit } from '../doctor.js';
+/**
+ * Test constants
+ */
+const HUSKY_DIR = '.husky';
+const SCRIPTS_DIR = 'scripts';
+const DOCS_TASKS_DIR = 'docs/04-operations/tasks';
+/**
+ * Test directory path
+ */
+let testDir;
+/**
+ * Helper to create a minimal valid project structure
+ */
+function setupMinimalProject(baseDir) {
+    // Create husky
+    mkdirSync(join(baseDir, HUSKY_DIR), { recursive: true });
+    writeFileSync(join(baseDir, HUSKY_DIR, 'pre-commit'), '#!/bin/sh\n', 'utf-8');
+    // Create safe-git
+    mkdirSync(join(baseDir, SCRIPTS_DIR), { recursive: true });
+    writeFileSync(join(baseDir, SCRIPTS_DIR, 'safe-git'), '#!/bin/sh\n', 'utf-8');
+    // Create AGENTS.md
+    writeFileSync(join(baseDir, 'AGENTS.md'), '# Agents\n', 'utf-8');
+    // Create .lumenflow.config.yaml
+    writeFileSync(join(baseDir, '.lumenflow.config.yaml'), 'lanes: []\n', 'utf-8');
+}
+/**
+ * Helper to initialize git in test directory
+ */
+function initGit(baseDir) {
+    execFileSync('git', ['init', '-b', 'main'], { cwd: baseDir, stdio: 'pipe' });
+    execFileSync('git', ['config', 'user.email', 'test@test.com'], {
+        cwd: baseDir,
+        stdio: 'pipe',
+    });
+    execFileSync('git', ['config', 'user.name', 'Test'], { cwd: baseDir, stdio: 'pipe' });
+}
+describe('doctor CLI (WU-1386) - Agent Friction Checks', () => {
+    beforeEach(() => {
+        testDir = mkdtempSync(join(tmpdir(), 'doctor-test-'));
+    });
+    afterEach(() => {
+        try {
+            rmSync(testDir, { recursive: true, force: true });
+        }
+        catch {
+            // Ignore cleanup errors
+        }
+    });
+    describe('managed-file dirty check', () => {
+        it('should detect uncommitted changes to .lumenflow.config.yaml', async () => {
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            // Commit initial state
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            // Modify managed file
+            writeFileSync(join(testDir, '.lumenflow.config.yaml'), 'lanes: []\nmodified: true\n', 'utf-8');
+            const result = await runDoctor(testDir);
+            expect(result.workflowHealth).toBeDefined();
+            expect(result.workflowHealth?.managedFilesDirty.passed).toBe(false);
+            expect(result.workflowHealth?.managedFilesDirty.files).toContain('.lumenflow.config.yaml');
+        });
+        it('should detect uncommitted changes to docs/04-operations/tasks/**', async () => {
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            // Create and commit a placeholder file in the tasks directory first
+            // This is needed because git shows untracked directories as just "?? docs/"
+            // but shows files in tracked directories with full paths
+            mkdirSync(join(testDir, DOCS_TASKS_DIR, 'wu'), { recursive: true });
+            writeFileSync(join(testDir, DOCS_TASKS_DIR, 'wu', 'WU-000.yaml'), 'id: WU-000\nstatus: done\nlane: Framework: CLI\n', 'utf-8');
+            // Commit initial state including the placeholder
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            // Now add a new WU file (uncommitted) - git will show full path
+            writeFileSync(join(testDir, DOCS_TASKS_DIR, 'wu', 'WU-001.yaml'), 'id: WU-001\nstatus: ready\nlane: Framework: CLI\n', 'utf-8');
+            const result = await runDoctor(testDir);
+            expect(result.workflowHealth?.managedFilesDirty.passed).toBe(false);
+            expect(result.workflowHealth?.managedFilesDirty.files).toContain('docs/04-operations/tasks/wu/WU-001.yaml');
+        });
+        it('should pass when no managed files have uncommitted changes', async () => {
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            // Commit all files
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            const result = await runDoctor(testDir);
+            expect(result.workflowHealth?.managedFilesDirty.passed).toBe(true);
+            expect(result.workflowHealth?.managedFilesDirty.files).toHaveLength(0);
+        });
+        it('should detect changes to AGENTS.md and CLAUDE.md', async () => {
+            setupMinimalProject(testDir);
+            writeFileSync(join(testDir, 'CLAUDE.md'), '# Claude\n', 'utf-8');
+            initGit(testDir);
+            // Commit initial state
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            // Modify both files
+            writeFileSync(join(testDir, 'AGENTS.md'), '# Agents\nmodified\n', 'utf-8');
+            writeFileSync(join(testDir, 'CLAUDE.md'), '# Claude\nmodified\n', 'utf-8');
+            const result = await runDoctor(testDir);
+            expect(result.workflowHealth?.managedFilesDirty.passed).toBe(false);
+            expect(result.workflowHealth?.managedFilesDirty.files).toContain('AGENTS.md');
+            expect(result.workflowHealth?.managedFilesDirty.files).toContain('CLAUDE.md');
+        });
+        it('should gracefully handle non-git directories', async () => {
+            setupMinimalProject(testDir);
+            // Don't init git
+            const result = await runDoctor(testDir);
+            // Should pass with graceful degradation message
+            expect(result.workflowHealth?.managedFilesDirty.passed).toBe(true);
+            expect(result.workflowHealth?.managedFilesDirty.message).toContain('skipped');
+        });
+    });
+    describe('worktree sanity check', () => {
+        it('should pass with graceful degradation in isolated test environment', async () => {
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            const result = await runDoctor(testDir);
+            // In isolated test env, wu:prune isn't available so it gracefully degrades
+            // The check passes with a skip message or runs successfully
+            expect(result.workflowHealth?.worktreeSanity.passed).toBe(true);
+        });
+        it('should gracefully handle non-git directories', async () => {
+            setupMinimalProject(testDir);
+            // Don't init git
+            const result = await runDoctor(testDir);
+            // Should pass with graceful degradation
+            expect(result.workflowHealth?.worktreeSanity.passed).toBe(true);
+            expect(result.workflowHealth?.worktreeSanity.message).toContain('skipped');
+        });
+    });
+    describe('--deep flag (WU validity check)', () => {
+        it('should skip WU validation by default (fast mode)', async () => {
+            setupMinimalProject(testDir);
+            const result = await runDoctor(testDir);
+            // WU validation should be undefined in default mode
+            expect(result.workflowHealth?.wuValidity).toBeUndefined();
+        });
+        it('should include wuValidity in --deep mode', async () => {
+            setupMinimalProject(testDir);
+            mkdirSync(join(testDir, DOCS_TASKS_DIR, 'wu'), { recursive: true });
+            writeFileSync(join(testDir, DOCS_TASKS_DIR, 'wu', 'WU-001.yaml'), 'id: WU-001\nstatus: ready\nlane: Framework: CLI\n', 'utf-8');
+            const result = await runDoctor(testDir, { deep: true });
+            // WU validation should be included in deep mode
+            expect(result.workflowHealth?.wuValidity).toBeDefined();
+            // WU-1387: In isolated test env without wu:validate CLI, should report failure
+            // (previously this would silently pass, now it correctly reports the CLI failure)
+            expect(typeof result.workflowHealth?.wuValidity?.passed).toBe('boolean');
+            // The message should indicate the validation ran or explain why it couldn't
+            expect(result.workflowHealth?.wuValidity?.message).toBeTruthy();
+        });
+        it('should gracefully handle missing wu:validate CLI', async () => {
+            setupMinimalProject(testDir);
+            // No WU directory - should still handle gracefully
+            const result = await runDoctor(testDir, { deep: true });
+            // No WU directory means passed=true with "No WU directory found" message
+            expect(result.workflowHealth?.wuValidity).toBeDefined();
+            expect(result.workflowHealth?.wuValidity?.passed).toBe(true);
+            expect(result.workflowHealth?.wuValidity?.message).toContain('No WU');
+        });
+    });
+    describe('exit codes', () => {
+        it('should return exitCode 0 when all checks pass', async () => {
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            const result = await runDoctor(testDir);
+            expect(result.exitCode).toBe(0);
+        });
+        it('should return exitCode 1 when warnings are present', async () => {
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            // Modify managed file to create warning
+            writeFileSync(join(testDir, '.lumenflow.config.yaml'), 'lanes: []\nmodified: true\n', 'utf-8');
+            const result = await runDoctor(testDir);
+            expect(result.exitCode).toBe(1);
+        });
+        it('should return exitCode 2 when critical errors are present', async () => {
+            // No husky hooks - critical error
+            mkdirSync(join(testDir, SCRIPTS_DIR), { recursive: true });
+            writeFileSync(join(testDir, SCRIPTS_DIR, 'safe-git'), '#!/bin/sh\n', 'utf-8');
+            writeFileSync(join(testDir, 'AGENTS.md'), '# Agents\n', 'utf-8');
+            writeFileSync(join(testDir, '.lumenflow.config.yaml'), 'lanes: []\n', 'utf-8');
+            const result = await runDoctor(testDir);
+            expect(result.exitCode).toBe(2);
+        });
+    });
+    describe('doctor result structure', () => {
+        it('should include workflowHealth section in result', async () => {
+            setupMinimalProject(testDir);
+            const result = await runDoctor(testDir);
+            // New workflowHealth section should be present
+            expect(result.workflowHealth).toBeDefined();
+            expect(result.workflowHealth?.managedFilesDirty).toBeDefined();
+            expect(result.workflowHealth?.worktreeSanity).toBeDefined();
+        });
+    });
+});
+describe('doctor auto-run after init (WU-1386)', () => {
+    beforeEach(() => {
+        testDir = mkdtempSync(join(tmpdir(), 'doctor-init-test-'));
+    });
+    afterEach(() => {
+        try {
+            rmSync(testDir, { recursive: true, force: true });
+        }
+        catch {
+            // Ignore cleanup errors
+        }
+    });
+    it('should never block init even with warnings', async () => {
+        setupMinimalProject(testDir);
+        initGit(testDir);
+        execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+        execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+        // Modify managed file to create warning
+        writeFileSync(join(testDir, '.lumenflow.config.yaml'), 'lanes: []\nmodified: true\n', 'utf-8');
+        const result = await runDoctorForInit(testDir);
+        // Should return warnings but not block
+        expect(result.blocked).toBe(false);
+        expect(result.warnings).toBeGreaterThan(0);
+    });
+    it('should print warnings but return success', async () => {
+        setupMinimalProject(testDir);
+        const result = await runDoctorForInit(testDir);
+        // Non-blocking mode should always indicate success
+        expect(result.blocked).toBe(false);
+    });
+});
+/**
+ * WU-1387 Edge Case Tests
+ *
+ * Tests for the specific edge cases identified in WU-1386 review:
+ * - AC1: Worktree sanity parsing for orphan, missing, stale, blocked, unclaimed worktrees
+ * - AC2: WU validity passes=false when CLI errors
+ * - AC3: runDoctorForInit shows accurate status including lane health and prereqs
+ * - AC4: Managed-file detection from git repo root in subdirectories
+ * - AC5: Real output parsing (not just graceful degradation)
+ */
+describe('WU-1387 Edge Cases - Worktree Sanity Parsing', () => {
+    /**
+     * These tests use a mock wu:prune module to test parsing of various output formats.
+     * The real wu:prune produces these outputs, we need to verify doctor parses them.
+     */
+    let testDir;
+    beforeEach(() => {
+        testDir = mkdtempSync(join(tmpdir(), 'doctor-1387-'));
+    });
+    afterEach(() => {
+        try {
+            rmSync(testDir, { recursive: true, force: true });
+        }
+        catch {
+            // Ignore cleanup errors
+        }
+    });
+    describe('AC1: parseWorktreePruneOutput helper', () => {
+        /**
+         * Import the parsing helper directly for unit testing
+         */
+        it('should parse orphan directory counts from summary', async () => {
+            // This test validates that the parser can extract counts from wu:prune summary
+            const sampleOutput = `[wu-prune] Summary
+[wu-prune] ========
+[wu-prune] Tracked worktrees: 3
+[wu-prune] Orphan directories: 2
+[wu-prune] Warnings: 1
+[wu-prune] Errors: 0`;
+            // Call the parsing helper (we'll need to export it or test via integration)
+            // For now, test via runDoctor which internally uses the parser
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            // The result should handle parsing when wu:prune is available
+            const result = await runDoctor(testDir);
+            expect(result.workflowHealth).toBeDefined();
+            expect(result.workflowHealth?.worktreeSanity).toBeDefined();
+        });
+    });
+});
+describe('WU-1387 Edge Cases - WU Validity Error Handling', () => {
+    let testDir;
+    beforeEach(() => {
+        testDir = mkdtempSync(join(tmpdir(), 'doctor-wuvalid-'));
+    });
+    afterEach(() => {
+        try {
+            rmSync(testDir, { recursive: true, force: true });
+        }
+        catch {
+            // Ignore cleanup errors
+        }
+    });
+    describe('AC2: CLI error handling', () => {
+        it('should handle validation gracefully when wu directory exists but validate fails', async () => {
+            setupMinimalProject(testDir);
+            mkdirSync(join(testDir, DOCS_TASKS_DIR, 'wu'), { recursive: true });
+            // Create a malformed WU file that will cause validation issues
+            writeFileSync(join(testDir, DOCS_TASKS_DIR, 'wu', 'WU-TEST.yaml'), 'id: WU-TEST\nstatus: invalid_status\nlane: Framework: CLI\n', 'utf-8');
+            const result = await runDoctor(testDir, { deep: true });
+            // Should still have a result, either gracefully degraded or actually validated
+            expect(result.workflowHealth?.wuValidity).toBeDefined();
+            // In isolated env, this will gracefully skip
+            expect(typeof result.workflowHealth?.wuValidity?.passed).toBe('boolean');
+        });
+    });
+});
+describe('WU-1387 Edge Cases - runDoctorForInit Accuracy', () => {
+    let testDir;
+    beforeEach(() => {
+        testDir = mkdtempSync(join(tmpdir(), 'doctor-init-'));
+    });
+    afterEach(() => {
+        try {
+            rmSync(testDir, { recursive: true, force: true });
+        }
+        catch {
+            // Ignore cleanup errors
+        }
+    });
+    describe('AC3: Accurate status reporting', () => {
+        it('should report all critical errors in output', async () => {
+            // Create directory without any required files
+            mkdirSync(testDir, { recursive: true });
+            const result = await runDoctorForInit(testDir);
+            // Should report errors for missing critical components
+            expect(result.errors).toBeGreaterThan(0);
+            // Output should contain specific error descriptions
+            expect(result.output).toMatch(/husky|hook/i);
+            expect(result.output).toMatch(/safe-git|script/i);
+            expect(result.output).toMatch(/AGENTS|agent/i);
+        });
+        it('should count workflow health warnings separately from errors', async () => {
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            // Modify managed file to create workflow warning
+            writeFileSync(join(testDir, '.lumenflow.config.yaml'), 'lanes: []\nmodified: true\n', 'utf-8');
+            const result = await runDoctorForInit(testDir);
+            // Should have warnings but no errors (all critical checks pass)
+            expect(result.errors).toBe(0);
+            expect(result.warnings).toBeGreaterThan(0);
+            expect(result.output).toMatch(/uncommitted|managed/i);
+        });
+    });
+});
+describe('WU-1387 Edge Cases - Managed File Detection', () => {
+    let testDir;
+    beforeEach(() => {
+        testDir = mkdtempSync(join(tmpdir(), 'doctor-managed-'));
+    });
+    afterEach(() => {
+        try {
+            rmSync(testDir, { recursive: true, force: true });
+        }
+        catch {
+            // Ignore cleanup errors
+        }
+    });
+    describe('AC4: Git repo root path resolution', () => {
+        it('should detect managed files when running from subdirectory', async () => {
+            // Setup project structure
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            // Create subdirectory structure
+            const subDir = join(testDir, 'packages', 'cli');
+            mkdirSync(subDir, { recursive: true });
+            // Commit initial state
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            // Modify managed file at repo root
+            writeFileSync(join(testDir, '.lumenflow.config.yaml'), 'lanes: []\nmodified: true\n', 'utf-8');
+            // Run doctor from subdirectory
+            const result = await runDoctor(subDir);
+            // Should still detect the modified managed file at repo root
+            expect(result.workflowHealth?.managedFilesDirty.passed).toBe(false);
+            expect(result.workflowHealth?.managedFilesDirty.files).toContain('.lumenflow.config.yaml');
+        });
+        it('should use git repo root for all path comparisons', async () => {
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            // Create deeply nested subdirectory
+            const deepSubDir = join(testDir, 'packages', 'core', 'src', 'lib');
+            mkdirSync(deepSubDir, { recursive: true });
+            mkdirSync(join(testDir, DOCS_TASKS_DIR, 'wu'), { recursive: true });
+            // Create a tracked file in the managed directory
+            writeFileSync(join(testDir, DOCS_TASKS_DIR, 'wu', 'WU-TRACK.yaml'), 'id: WU-TRACK\n', 'utf-8');
+            // Commit initial state
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            // Modify the WU file
+            writeFileSync(join(testDir, DOCS_TASKS_DIR, 'wu', 'WU-TRACK.yaml'), 'id: WU-TRACK\nmodified: true\n', 'utf-8');
+            // Run doctor from deeply nested subdirectory
+            const result = await runDoctor(deepSubDir);
+            // Should detect modified file using paths relative to repo root
+            expect(result.workflowHealth?.managedFilesDirty.passed).toBe(false);
+            expect(result.workflowHealth?.managedFilesDirty.files.some((f) => f.includes('WU-TRACK'))).toBe(true);
+        });
+    });
+});
+/**
+ * WU-1387 AC5: Real output parsing tests
+ * These unit tests verify the parsing logic directly with sample outputs
+ */
+describe('WU-1387 AC5: Real Output Parsing', () => {
+    // Import the parsing helper for direct testing
+    // Note: We test via integration since the helper is not exported
+    describe('worktree sanity parsing via integration', () => {
+        let testDir;
+        beforeEach(() => {
+            testDir = mkdtempSync(join(tmpdir(), 'doctor-parsing-'));
+        });
+        afterEach(() => {
+            try {
+                rmSync(testDir, { recursive: true, force: true });
+            }
+            catch {
+                // Ignore cleanup errors
+            }
+        });
+        it('should correctly interpret valid worktree output', async () => {
+            setupMinimalProject(testDir);
+            initGit(testDir);
+            execFileSync('git', ['add', '.'], { cwd: testDir, stdio: 'pipe' });
+            execFileSync('git', ['commit', '-m', 'initial'], { cwd: testDir, stdio: 'pipe' });
+            const result = await runDoctor(testDir);
+            // In a clean project, worktree sanity should pass
+            // Either it runs and finds no issues, or gracefully degrades
+            expect(result.workflowHealth?.worktreeSanity).toBeDefined();
+            expect(typeof result.workflowHealth?.worktreeSanity.passed).toBe('boolean');
+            expect(typeof result.workflowHealth?.worktreeSanity.orphans).toBe('number');
+            expect(typeof result.workflowHealth?.worktreeSanity.stale).toBe('number');
+        });
+    });
+    describe('WU validity parsing via integration', () => {
+        let testDir;
+        beforeEach(() => {
+            testDir = mkdtempSync(join(tmpdir(), 'doctor-wuparse-'));
+        });
+        afterEach(() => {
+            try {
+                rmSync(testDir, { recursive: true, force: true });
+            }
+            catch {
+                // Ignore cleanup errors
+            }
+        });
+        it('should return structured result with all expected fields', async () => {
+            setupMinimalProject(testDir);
+            mkdirSync(join(testDir, DOCS_TASKS_DIR, 'wu'), { recursive: true });
+            writeFileSync(join(testDir, DOCS_TASKS_DIR, 'wu', 'WU-TEST.yaml'), 'id: WU-TEST\nstatus: ready\nlane: Test\n', 'utf-8');
+            const result = await runDoctor(testDir, { deep: true });
+            // WU validity should have all expected fields
+            expect(result.workflowHealth?.wuValidity).toBeDefined();
+            const wuValidity = result.workflowHealth?.wuValidity;
+            expect(typeof wuValidity?.passed).toBe('boolean');
+            expect(typeof wuValidity?.total).toBe('number');
+            expect(typeof wuValidity?.valid).toBe('number');
+            expect(typeof wuValidity?.invalid).toBe('number');
+            expect(typeof wuValidity?.warnings).toBe('number');
+            expect(typeof wuValidity?.message).toBe('string');
+            expect(wuValidity?.message.length).toBeGreaterThan(0);
+        });
+        it('should set passed=false with clear message when CLI unavailable', async () => {
+            setupMinimalProject(testDir);
+            mkdirSync(join(testDir, DOCS_TASKS_DIR, 'wu'), { recursive: true });
+            writeFileSync(join(testDir, DOCS_TASKS_DIR, 'wu', 'WU-TEST.yaml'), 'id: WU-TEST\nstatus: ready\n', 'utf-8');
+            // In isolated test env without pnpm scripts, CLI will fail
+            const result = await runDoctor(testDir, { deep: true });
+            // WU-1387: Should report failure, not silently pass
+            expect(result.workflowHealth?.wuValidity).toBeDefined();
+            const wuValidity = result.workflowHealth?.wuValidity;
+            // Message should indicate failure reason
+            expect(wuValidity?.message).toBeTruthy();
+            // If CLI couldn't run, message should explain why
+            if (!wuValidity?.passed) {
+                expect(wuValidity?.message).toMatch(/failed|error|unavailable|not found|could not/i);
+            }
+        });
+    });
+});

package/dist/__tests__/gates-config.test.js

@@ -5,7 +5,6 @@
  * Tests configurable package_manager, gates.commands, test_runner, and build_command
  * in .lumenflow.config.yaml for framework agnosticism.
  */
-/* eslint-disable sonarjs/no-duplicate-string -- Test files intentionally repeat string literals for readability */
 import { describe, it, expect, beforeEach, afterEach } from 'vitest';
 import * as fs from 'node:fs';
 import * as path from 'node:path';