@lumenflow/cli 2.20.1 → 2.21.1

package/dist/__tests__/flow-report.test.js

@@ -1,24 +0,0 @@
-/**
- * @file flow-report.test.ts
- * @description Tests for flow-report CLI command (WU-1018)
- *
- * These are smoke tests to verify the CLI module can be imported.
- * The CLI commands are wrappers around @lumenflow/metrics library functions
- * which have their own comprehensive tests.
- */
-import { describe, it, expect } from 'vitest';
-import { existsSync } from 'node:fs';
-import { join } from 'node:path';
-import { fileURLToPath } from 'node:url';
-const __dirname = fileURLToPath(new URL('.', import.meta.url));
-describe('flow-report CLI', () => {
-    it('should have the CLI source file', () => {
-        const srcPath = join(__dirname, '../flow-report.ts');
-        expect(existsSync(srcPath)).toBe(true);
-    });
-    it('should be buildable (dist file exists after build)', () => {
-        // This test verifies that tsc compiled the file successfully
-        const distPath = join(__dirname, '../../dist/flow-report.js');
-        expect(existsSync(distPath)).toBe(true);
-    });
-});

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

@@ -1,303 +0,0 @@
-/**
- * @file gates-config.test.ts
- * WU-1356: Tests for package manager and script name configuration.
- *
- * Tests configurable package_manager, gates.commands, test_runner, and build_command
- * in .lumenflow.config.yaml for framework agnosticism.
- */
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-import * as os from 'node:os';
-import * as yaml from 'yaml';
-// Import the schema and functions we're testing
-import { PackageManagerSchema, TestRunnerSchema, GatesCommandsConfigSchema, parseConfig, } from '@lumenflow/core/dist/lumenflow-config-schema.js';
-import { resolvePackageManager, resolveBuildCommand, resolveGatesCommands, resolveTestRunner, getIgnorePatterns, } from '@lumenflow/core/dist/gates-config.js';
-describe('WU-1356: Package manager and script configuration', () => {
-    let tempDir;
-    beforeEach(() => {
-        tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'lumenflow-test-'));
-    });
-    afterEach(() => {
-        fs.rmSync(tempDir, { recursive: true, force: true });
-    });
-    describe('PackageManagerSchema', () => {
-        it('accepts pnpm as default', () => {
-            const result = PackageManagerSchema.parse(undefined);
-            expect(result).toBe('pnpm');
-        });
-        it('accepts npm', () => {
-            const result = PackageManagerSchema.parse('npm');
-            expect(result).toBe('npm');
-        });
-        it('accepts yarn', () => {
-            const result = PackageManagerSchema.parse('yarn');
-            expect(result).toBe('yarn');
-        });
-        it('accepts bun', () => {
-            const result = PackageManagerSchema.parse('bun');
-            expect(result).toBe('bun');
-        });
-        it('rejects invalid package manager', () => {
-            expect(() => PackageManagerSchema.parse('invalid')).toThrow();
-        });
-    });
-    describe('TestRunnerSchema', () => {
-        it('accepts vitest as default', () => {
-            const result = TestRunnerSchema.parse(undefined);
-            expect(result).toBe('vitest');
-        });
-        it('accepts jest', () => {
-            const result = TestRunnerSchema.parse('jest');
-            expect(result).toBe('jest');
-        });
-        it('accepts mocha', () => {
-            const result = TestRunnerSchema.parse('mocha');
-            expect(result).toBe('mocha');
-        });
-        it('rejects invalid test runner', () => {
-            expect(() => TestRunnerSchema.parse('invalid')).toThrow();
-        });
-    });
-    describe('GatesCommandsConfigSchema', () => {
-        it('has sensible defaults for test commands', () => {
-            const result = GatesCommandsConfigSchema.parse({});
-            expect(result.test_full).toBeDefined();
-            expect(result.test_docs_only).toBeDefined();
-            expect(result.test_incremental).toBeDefined();
-        });
-        it('allows custom test commands', () => {
-            const config = {
-                test_full: 'npm test',
-                test_docs_only: 'npm test -- --grep docs',
-                test_incremental: 'npm test -- --changed',
-            };
-            const result = GatesCommandsConfigSchema.parse(config);
-            expect(result.test_full).toBe('npm test');
-            expect(result.test_docs_only).toBe('npm test -- --grep docs');
-            expect(result.test_incremental).toBe('npm test -- --changed');
-        });
-    });
-    describe('LumenFlowConfigSchema - package_manager field', () => {
-        it('includes package_manager with default pnpm', () => {
-            const config = parseConfig({});
-            expect(config.package_manager).toBe('pnpm');
-        });
-        it('accepts npm as package_manager', () => {
-            const config = parseConfig({ package_manager: 'npm' });
-            expect(config.package_manager).toBe('npm');
-        });
-        it('accepts yarn as package_manager', () => {
-            const config = parseConfig({ package_manager: 'yarn' });
-            expect(config.package_manager).toBe('yarn');
-        });
-        it('accepts bun as package_manager', () => {
-            const config = parseConfig({ package_manager: 'bun' });
-            expect(config.package_manager).toBe('bun');
-        });
-    });
-    describe('LumenFlowConfigSchema - test_runner field', () => {
-        it('includes test_runner with default vitest', () => {
-            const config = parseConfig({});
-            expect(config.test_runner).toBe('vitest');
-        });
-        it('accepts jest as test_runner', () => {
-            const config = parseConfig({ test_runner: 'jest' });
-            expect(config.test_runner).toBe('jest');
-        });
-        it('accepts mocha as test_runner', () => {
-            const config = parseConfig({ test_runner: 'mocha' });
-            expect(config.test_runner).toBe('mocha');
-        });
-    });
-    describe('LumenFlowConfigSchema - gates.commands section', () => {
-        it('includes gates.commands with defaults', () => {
-            const config = parseConfig({});
-            expect(config.gates.commands).toBeDefined();
-            expect(config.gates.commands.test_full).toBeDefined();
-            expect(config.gates.commands.test_docs_only).toBeDefined();
-            expect(config.gates.commands.test_incremental).toBeDefined();
-        });
-        it('allows custom gates commands configuration', () => {
-            const config = parseConfig({
-                gates: {
-                    commands: {
-                        test_full: 'npm test',
-                        test_docs_only: 'npm test -- --docs',
-                        test_incremental: 'npm test -- --changed',
-                    },
-                },
-            });
-            expect(config.gates.commands.test_full).toBe('npm test');
-            expect(config.gates.commands.test_docs_only).toBe('npm test -- --docs');
-            expect(config.gates.commands.test_incremental).toBe('npm test -- --changed');
-        });
-    });
-    describe('LumenFlowConfigSchema - build_command field', () => {
-        it('includes build_command with default for pnpm', () => {
-            const config = parseConfig({});
-            expect(config.build_command).toBe('pnpm --filter @lumenflow/cli build');
-        });
-        it('allows custom build_command', () => {
-            const config = parseConfig({ build_command: 'npm run build' });
-            expect(config.build_command).toBe('npm run build');
-        });
-    });
-    describe('resolvePackageManager', () => {
-        it('returns pnpm when no config file exists', () => {
-            const result = resolvePackageManager(tempDir);
-            expect(result).toBe('pnpm');
-        });
-        it('returns configured package manager from config file', () => {
-            const configPath = path.join(tempDir, '.lumenflow.config.yaml');
-            fs.writeFileSync(configPath, yaml.stringify({ package_manager: 'npm' }));
-            const result = resolvePackageManager(tempDir);
-            expect(result).toBe('npm');
-        });
-        it('returns yarn when configured', () => {
-            const configPath = path.join(tempDir, '.lumenflow.config.yaml');
-            fs.writeFileSync(configPath, yaml.stringify({ package_manager: 'yarn' }));
-            const result = resolvePackageManager(tempDir);
-            expect(result).toBe('yarn');
-        });
-    });
-    describe('resolveBuildCommand', () => {
-        it('returns default build command when no config file exists', () => {
-            const result = resolveBuildCommand(tempDir);
-            expect(result).toBe('pnpm --filter @lumenflow/cli build');
-        });
-        it('returns configured build_command from config file', () => {
-            const configPath = path.join(tempDir, '.lumenflow.config.yaml');
-            fs.writeFileSync(configPath, yaml.stringify({ build_command: 'npm run build' }));
-            const result = resolveBuildCommand(tempDir);
-            expect(result).toBe('npm run build');
-        });
-        it('adapts default build command for different package managers', () => {
-            const configPath = path.join(tempDir, '.lumenflow.config.yaml');
-            fs.writeFileSync(configPath, yaml.stringify({ package_manager: 'npm' }));
-            const result = resolveBuildCommand(tempDir);
-            expect(result).toContain('npm');
-        });
-    });
-    describe('resolveGatesCommands', () => {
-        it('returns default commands when no config file exists', () => {
-            const commands = resolveGatesCommands(tempDir);
-            expect(commands.test_full).toBeDefined();
-            expect(commands.test_docs_only).toBeDefined();
-            expect(commands.test_incremental).toBeDefined();
-        });
-        it('returns configured commands from config file', () => {
-            const configPath = path.join(tempDir, '.lumenflow.config.yaml');
-            fs.writeFileSync(configPath, yaml.stringify({
-                gates: {
-                    commands: {
-                        test_full: 'npm test',
-                        test_docs_only: 'npm test -- --docs',
-                        test_incremental: 'npm test -- --changed',
-                    },
-                },
-            }));
-            const commands = resolveGatesCommands(tempDir);
-            expect(commands.test_full).toBe('npm test');
-            expect(commands.test_docs_only).toBe('npm test -- --docs');
-            expect(commands.test_incremental).toBe('npm test -- --changed');
-        });
-    });
-    describe('resolveTestRunner', () => {
-        it('returns vitest when no config file exists', () => {
-            const result = resolveTestRunner(tempDir);
-            expect(result).toBe('vitest');
-        });
-        it('returns jest when configured', () => {
-            const configPath = path.join(tempDir, '.lumenflow.config.yaml');
-            fs.writeFileSync(configPath, yaml.stringify({ test_runner: 'jest' }));
-            const result = resolveTestRunner(tempDir);
-            expect(result).toBe('jest');
-        });
-    });
-    describe('getIgnorePatterns', () => {
-        it('returns .turbo pattern for vitest', () => {
-            const patterns = getIgnorePatterns('vitest');
-            expect(patterns).toContain('.turbo');
-        });
-        it('returns different pattern for jest', () => {
-            const patterns = getIgnorePatterns('jest');
-            expect(patterns).not.toContain('.turbo');
-        });
-        it('returns custom pattern from config', () => {
-            const configPath = path.join(tempDir, '.lumenflow.config.yaml');
-            fs.writeFileSync(configPath, yaml.stringify({
-                gates: {
-                    ignore_patterns: ['.nx', 'dist'],
-                },
-            }));
-            // This would be a config-aware version
-            const patterns = getIgnorePatterns('jest');
-            expect(Array.isArray(patterns)).toBe(true);
-        });
-    });
-});
-describe('WU-1356: npm+jest configuration', () => {
-    let tempDir;
-    beforeEach(() => {
-        tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'lumenflow-npm-jest-'));
-    });
-    afterEach(() => {
-        fs.rmSync(tempDir, { recursive: true, force: true });
-    });
-    it('supports npm+jest configuration', () => {
-        const configPath = path.join(tempDir, '.lumenflow.config.yaml');
-        fs.writeFileSync(configPath, yaml.stringify({
-            package_manager: 'npm',
-            test_runner: 'jest',
-            gates: {
-                commands: {
-                    test_full: 'npm test',
-                    test_docs_only: 'npm test -- --testPathPattern=docs',
-                    test_incremental: 'npm test -- --onlyChanged',
-                },
-            },
-            build_command: 'npm run build',
-        }));
-        const pkgManager = resolvePackageManager(tempDir);
-        const testRunner = resolveTestRunner(tempDir);
-        const commands = resolveGatesCommands(tempDir);
-        const buildCmd = resolveBuildCommand(tempDir);
-        expect(pkgManager).toBe('npm');
-        expect(testRunner).toBe('jest');
-        expect(commands.test_full).toBe('npm test');
-        expect(commands.test_incremental).toBe('npm test -- --onlyChanged');
-        expect(buildCmd).toBe('npm run build');
-    });
-});
-describe('WU-1356: yarn+nx configuration', () => {
-    let tempDir;
-    beforeEach(() => {
-        tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'lumenflow-yarn-nx-'));
-    });
-    afterEach(() => {
-        fs.rmSync(tempDir, { recursive: true, force: true });
-    });
-    it('supports yarn+nx configuration', () => {
-        const configPath = path.join(tempDir, '.lumenflow.config.yaml');
-        fs.writeFileSync(configPath, yaml.stringify({
-            package_manager: 'yarn',
-            test_runner: 'jest',
-            gates: {
-                commands: {
-                    test_full: 'yarn nx run-many --target=test --all',
-                    test_docs_only: 'yarn nx test docs',
-                    test_incremental: 'yarn nx affected --target=test',
-                },
-            },
-            build_command: 'yarn nx build @lumenflow/cli',
-        }));
-        const pkgManager = resolvePackageManager(tempDir);
-        const commands = resolveGatesCommands(tempDir);
-        const buildCmd = resolveBuildCommand(tempDir);
-        expect(pkgManager).toBe('yarn');
-        expect(commands.test_full).toBe('yarn nx run-many --target=test --all');
-        expect(commands.test_incremental).toBe('yarn nx affected --target=test');
-        expect(buildCmd).toBe('yarn nx build @lumenflow/cli');
-    });
-});

package/dist/__tests__/gates-integration-tests.test.js

@@ -1,112 +0,0 @@
-/**
- * @file gates-integration-tests.test.ts
- * @description Tests for gates infrastructure fixes (WU-1415)
- *
- * Bug 1: vitest --include is not a valid CLI option
- * Bug 2: docs-only turbo filter uses directory names instead of package names
- */
-import { describe, it, expect } from 'vitest';
-import { extractPackagesFromCodePaths, resolveDocsOnlyTestPlan } from '../gates.js';
-describe('WU-1415: Gates infrastructure fixes', () => {
-    describe('Bug 1: vitest integration test command', () => {
-        it('should NOT use --include flag (vitest does not support it)', async () => {
-            // Import the module to inspect the command construction
-            // We need to verify that runIntegrationTests uses valid vitest syntax
-            //
-            // vitest run accepts positional glob patterns, NOT --include flags:
-            //   WRONG: vitest run --include='**/*.integration.*'
-            //   RIGHT: vitest run '**/*.integration.*'
-            //
-            // This test ensures we're using the correct vitest CLI syntax
-            const gatesModule = await import('../gates.js');
-            // The command construction happens in runIntegrationTests
-            // We can't directly test the internal function, but we can verify
-            // via the module's exported constants or by checking the implementation
-            // doesn't contain --include
-            // Read the source to verify no --include in vitest commands
-            const fs = await import('fs');
-            const path = await import('path');
-            const gatesPath = path.join(import.meta.dirname, '..', 'gates.ts');
-            const source = fs.readFileSync(gatesPath, 'utf-8');
-            // Find the runIntegrationTests function and check it doesn't use --include
-            const integrationTestMatch = source.match(/function runIntegrationTests[\s\S]*?^}/m);
-            if (integrationTestMatch) {
-                const functionBody = integrationTestMatch[0];
-                // vitest run should NOT have --include flags
-                expect(functionBody).not.toMatch(/vitest.*--include/);
-                // Instead, glob patterns should be positional args or via proper config
-                // The fix should pass patterns directly: vitest run 'pattern1' 'pattern2'
-            }
-        });
-    });
-    describe('Bug 2: docs-only turbo filter', () => {
-        describe('extractPackagesFromCodePaths', () => {
-            it('should extract scoped package names from packages/ paths', () => {
-                const codePaths = [
-                    'packages/@lumenflow/cli/src/gates.ts',
-                    'packages/@lumenflow/core/src/index.ts',
-                ];
-                const packages = extractPackagesFromCodePaths(codePaths);
-                expect(packages).toContain('@lumenflow/cli');
-                expect(packages).toContain('@lumenflow/core');
-            });
-            it('should return empty array for apps/ paths that are not real turbo packages', () => {
-                // apps/docs/ directory name is 'docs' but the turbo package might be
-                // named differently (e.g., '@lumenflow/docs' or not exist at all)
-                //
-                // The current implementation returns 'docs' which causes turbo to fail:
-                //   "No package found with name 'docs' in workspace"
-                //
-                // Fix: Either lookup actual package.json name or skip apps
-                const codePaths = ['apps/docs/src/content/docs/', 'apps/github-app/'];
-                const packages = extractPackagesFromCodePaths(codePaths);
-                // Current buggy behavior returns ['docs', 'github-app']
-                // Fixed behavior should either:
-                // - Return actual package names from package.json
-                // - Or return empty array (apps don't have turbo test tasks)
-                //
-                // For now, the fix should skip apps that aren't valid turbo packages
-                // because apps/docs has no test script and apps/github-app was deleted
-                expect(packages).not.toContain('docs');
-                expect(packages).not.toContain('github-app');
-            });
-            it('should handle mixed code_paths (packages + apps + docs)', () => {
-                const codePaths = [
-                    'packages/@lumenflow/cli/src/file.ts',
-                    'apps/docs/astro.config.mjs',
-                    'docs/DISTRIBUTION.md',
-                ];
-                const packages = extractPackagesFromCodePaths(codePaths);
-                // Should include the real package
-                expect(packages).toContain('@lumenflow/cli');
-                // Should NOT include apps (no valid turbo package)
-                expect(packages).not.toContain('docs');
-                // Should NOT include docs/ (not a package)
-                expect(packages.length).toBe(1);
-            });
-            it('should return empty array for pure docs paths', () => {
-                const codePaths = ['docs/01-product/product-lines.md', 'docs/DISTRIBUTION.md'];
-                const packages = extractPackagesFromCodePaths(codePaths);
-                expect(packages).toEqual([]);
-            });
-        });
-        describe('resolveDocsOnlyTestPlan', () => {
-            it('should return skip mode for pure documentation WUs', () => {
-                const plan = resolveDocsOnlyTestPlan({
-                    codePaths: ['docs/README.md', 'apps/docs/content/'],
-                });
-                expect(plan.mode).toBe('skip');
-                expect(plan.packages).toEqual([]);
-            });
-            it('should return filtered mode only for valid package paths', () => {
-                const plan = resolveDocsOnlyTestPlan({
-                    codePaths: ['packages/@lumenflow/cli/src/gates.ts', 'apps/docs/content/'],
-                });
-                expect(plan.mode).toBe('filtered');
-                expect(plan.packages).toContain('@lumenflow/cli');
-                // apps/docs should not be included
-                expect(plan.packages).not.toContain('docs');
-            });
-        });
-    });
-});