agentic-qe 3.3.0 → 3.3.2

package/v3/dist/cli/wizards/test-wizard.js

@@ -3,103 +3,38 @@
  * ADR-041: V3 QE CLI Enhancement
  *
  * Interactive wizard for test generation with step-by-step configuration.
- * Prompts for source files, test type, coverage target, framework, and AI enhancement.
+ * Refactored to use Command Pattern for reduced complexity and better reusability.
  */
-import { createInterface } from 'readline';
 import chalk from 'chalk';
 import { existsSync, readdirSync, statSync, readFileSync } from 'fs';
 import { join, resolve, relative, extname, basename } from 'path';
+import { BaseWizard, BaseWizardCommand, SingleSelectStep, BooleanStep, NumericStep, WizardPrompt, WizardFormat, WizardSuggestions, } from './core/index.js';
 // ============================================================================
-// Wizard Implementation
+// Source Files Step (Custom)
 // ============================================================================
-export class TestGenerationWizard {
-    options;
-    cwd;
-    constructor(options = {}) {
-        this.options = options;
-        this.cwd = process.cwd();
+/**
+ * Custom step for source file selection with file resolution
+ */
+class SourceFilesStep extends BaseWizardCommand {
+    id = 'sourceFiles';
+    stepNumber = '1/6';
+    title = 'Source Files';
+    description = 'Enter file paths, glob patterns, or directory';
+    defaultSourceFiles;
+    constructor(defaultSourceFiles) {
+        super(defaultSourceFiles);
+        this.defaultSourceFiles = defaultSourceFiles;
     }
-    /**
-     * Run the interactive wizard
-     */
-    async run() {
-        // Non-interactive mode returns defaults
-        if (this.options.nonInteractive) {
-            return this.getDefaults();
-        }
-        const rl = createInterface({
-            input: process.stdin,
-            output: process.stdout,
-        });
-        try {
-            // Print header
-            this.printHeader();
-            // Step 1: Source files
-            const sourceFiles = await this.promptSourceFiles(rl);
-            if (sourceFiles.length === 0) {
-                return this.getCancelled();
-            }
-            // Step 2: Test type
-            const testType = await this.promptTestType(rl);
-            // Step 3: Coverage target
-            const coverageTarget = await this.promptCoverageTarget(rl);
-            // Step 4: Framework
-            const framework = await this.promptFramework(rl);
-            // Step 5: AI enhancement level
-            const aiLevel = await this.promptAILevel(rl);
-            // Step 6: Anti-pattern detection
-            const detectAntiPatterns = await this.promptAntiPatternDetection(rl);
-            // Print summary
-            this.printSummary({
-                sourceFiles,
-                testType,
-                coverageTarget,
-                framework,
-                aiLevel,
-                detectAntiPatterns,
-                cancelled: false,
-            });
-            // Confirm
-            const confirmed = await this.promptConfirmation(rl);
-            if (!confirmed) {
-                return this.getCancelled();
-            }
-            return {
-                sourceFiles,
-                testType,
-                coverageTarget,
-                framework,
-                aiLevel,
-                detectAntiPatterns,
-                cancelled: false,
-            };
+    async execute(context) {
+        if (context.nonInteractive) {
+            const resolved = this.resolveSourceFiles(this.defaultSourceFiles.join(', '), context.cwd);
+            return this.success(resolved.length > 0 ? resolved : this.defaultSourceFiles);
         }
-        finally {
-            rl.close();
-        }
-    }
-    /**
-     * Print wizard header
-     */
-    printHeader() {
-        console.log('');
-        console.log(chalk.blue('========================================'));
-        console.log(chalk.blue.bold('       Test Generation Wizard'));
-        console.log(chalk.blue('========================================'));
-        console.log(chalk.gray('Generate tests with AI-powered assistance'));
-        console.log(chalk.gray('Press Ctrl+C to cancel at any time'));
-        console.log('');
-    }
-    /**
-     * Step 1: Prompt for source files
-     */
-    async promptSourceFiles(rl) {
-        console.log(chalk.cyan('Step 1/6: Source Files'));
-        console.log(chalk.gray('Enter file paths, glob patterns, or directory'));
+        WizardPrompt.printStepHeader(this.stepNumber, this.title, this.description);
         console.log(chalk.gray('Examples: src/services/*.ts, ./src/utils, src/auth.ts'));
         console.log('');
-        // Show available suggestions
-        const suggestions = this.getSourceFileSuggestions();
+        // Show suggestions
+        const suggestions = WizardSuggestions.getTestSourceFiles(context.cwd);
         if (suggestions.length > 0) {
             console.log(chalk.yellow('Suggestions:'));
             suggestions.slice(0, 5).forEach((s, i) => {
@@ -107,247 +42,28 @@ export class TestGenerationWizard {
             });
             console.log('');
         }
-        const defaultValue = this.options.defaultSourceFiles?.join(', ') || '.';
-        const input = await this.prompt(rl, `Source files [${chalk.gray(defaultValue)}]: `);
+        const defaultValue = this.defaultSourceFiles.join(', ') || '.';
+        const input = await WizardPrompt.prompt(context.rl, `Source files [${chalk.gray(defaultValue)}]: `);
         const value = input.trim() || defaultValue;
         // Parse input into file list
-        return this.resolveSourceFiles(value);
-    }
-    /**
-     * Step 2: Prompt for test type
-     */
-    async promptTestType(rl) {
-        console.log('');
-        console.log(chalk.cyan('Step 2/6: Test Type'));
-        console.log(chalk.gray('Select the type of tests to generate'));
-        console.log('');
-        const options = [
-            { key: '1', value: 'unit', description: 'Unit tests - isolated component testing' },
-            { key: '2', value: 'integration', description: 'Integration tests - module interaction testing' },
-            { key: '3', value: 'e2e', description: 'End-to-end tests - full workflow testing' },
-            { key: '4', value: 'property', description: 'Property-based tests - invariant testing' },
-            { key: '5', value: 'contract', description: 'Contract tests - API contract validation' },
-        ];
-        options.forEach(opt => {
-            const marker = opt.value === this.options.defaultTestType ? chalk.green(' (default)') : '';
-            console.log(chalk.white(`  ${opt.key}. ${opt.value}${marker}`));
-            console.log(chalk.gray(`     ${opt.description}`));
-        });
-        console.log('');
-        const defaultValue = this.options.defaultTestType || 'unit';
-        const input = await this.prompt(rl, `Select test type [${chalk.gray(defaultValue)}]: `);
-        const value = input.trim();
-        if (!value)
-            return defaultValue;
-        // Check if input is a number
-        const numInput = parseInt(value, 10);
-        if (numInput >= 1 && numInput <= options.length) {
-            return options[numInput - 1].value;
-        }
-        // Check if input is a valid test type
-        const validTypes = ['unit', 'integration', 'e2e', 'property', 'contract'];
-        if (validTypes.includes(value)) {
-            return value;
-        }
-        console.log(chalk.yellow(`Invalid input, using default: ${defaultValue}`));
-        return defaultValue;
-    }
-    /**
-     * Step 3: Prompt for coverage target
-     */
-    async promptCoverageTarget(rl) {
-        console.log('');
-        console.log(chalk.cyan('Step 3/6: Coverage Target'));
-        console.log(chalk.gray('Target code coverage percentage (0-100)'));
-        console.log(chalk.gray('Recommended: 80% for new code, 60% for legacy'));
-        console.log('');
-        const defaultValue = this.options.defaultCoverageTarget || 80;
-        const input = await this.prompt(rl, `Coverage target % [${chalk.gray(defaultValue)}]: `);
-        const value = input.trim();
-        if (!value)
-            return defaultValue;
-        const numValue = parseInt(value, 10);
-        if (isNaN(numValue) || numValue < 0 || numValue > 100) {
-            console.log(chalk.yellow(`Invalid input, using default: ${defaultValue}%`));
-            return defaultValue;
-        }
-        return numValue;
-    }
-    /**
-     * Step 4: Prompt for test framework
-     */
-    async promptFramework(rl) {
-        console.log('');
-        console.log(chalk.cyan('Step 4/6: Test Framework'));
-        console.log(chalk.gray('Select the testing framework to use'));
-        console.log('');
-        const options = [
-            { key: '1', value: 'vitest', description: 'Vitest - Fast, Vite-native testing' },
-            { key: '2', value: 'jest', description: 'Jest - Feature-rich, widely adopted' },
-            { key: '3', value: 'mocha', description: 'Mocha - Flexible, configurable' },
-            { key: '4', value: 'playwright', description: 'Playwright - Browser automation (for e2e)' },
-        ];
-        // Detect framework from project
-        const detectedFramework = this.detectFramework();
-        const defaultValue = this.options.defaultFramework || detectedFramework || 'vitest';
-        options.forEach(opt => {
-            const marker = opt.value === detectedFramework ? chalk.green(' (detected)') :
-                opt.value === defaultValue ? chalk.gray(' (default)') : '';
-            console.log(chalk.white(`  ${opt.key}. ${opt.value}${marker}`));
-            console.log(chalk.gray(`     ${opt.description}`));
-        });
-        console.log('');
-        const input = await this.prompt(rl, `Select framework [${chalk.gray(defaultValue)}]: `);
-        const value = input.trim();
-        if (!value)
-            return defaultValue;
-        // Check if input is a number
-        const numInput = parseInt(value, 10);
-        if (numInput >= 1 && numInput <= options.length) {
-            return options[numInput - 1].value;
-        }
-        // Check if input is a valid framework
-        const validFrameworks = ['jest', 'vitest', 'mocha', 'playwright'];
-        if (validFrameworks.includes(value)) {
-            return value;
-        }
-        console.log(chalk.yellow(`Invalid input, using default: ${defaultValue}`));
-        return defaultValue;
-    }
-    /**
-     * Step 5: Prompt for AI enhancement level
-     */
-    async promptAILevel(rl) {
-        console.log('');
-        console.log(chalk.cyan('Step 5/6: AI Enhancement Level'));
-        console.log(chalk.gray('Select the level of AI assistance for test generation'));
-        console.log('');
-        const options = [
-            { key: '1', value: 'none', description: 'None - Template-based generation only' },
-            { key: '2', value: 'basic', description: 'Basic - Simple pattern matching' },
-            { key: '3', value: 'standard', description: 'Standard - AI-powered test suggestions' },
-            { key: '4', value: 'advanced', description: 'Advanced - Full AI with edge case generation' },
-        ];
-        const defaultValue = this.options.defaultAILevel || 'standard';
-        options.forEach(opt => {
-            const marker = opt.value === defaultValue ? chalk.green(' (recommended)') : '';
-            console.log(chalk.white(`  ${opt.key}. ${opt.value}${marker}`));
-            console.log(chalk.gray(`     ${opt.description}`));
-        });
-        console.log('');
-        const input = await this.prompt(rl, `Select AI level [${chalk.gray(defaultValue)}]: `);
-        const value = input.trim();
-        if (!value)
-            return defaultValue;
-        // Check if input is a number
-        const numInput = parseInt(value, 10);
-        if (numInput >= 1 && numInput <= options.length) {
-            return options[numInput - 1].value;
-        }
-        // Check if input is a valid level
-        const validLevels = ['none', 'basic', 'standard', 'advanced'];
-        if (validLevels.includes(value)) {
-            return value;
-        }
-        console.log(chalk.yellow(`Invalid input, using default: ${defaultValue}`));
-        return defaultValue;
-    }
-    /**
-     * Step 6: Prompt for anti-pattern detection
-     */
-    async promptAntiPatternDetection(rl) {
-        console.log('');
-        console.log(chalk.cyan('Step 6/6: Anti-Pattern Detection'));
-        console.log(chalk.gray('Enable detection and avoidance of code anti-patterns'));
-        console.log('');
-        const input = await this.prompt(rl, `Enable anti-pattern detection? [${chalk.gray('Y/n')}]: `);
-        const value = input.trim().toLowerCase();
-        if (value === 'n' || value === 'no') {
-            return false;
-        }
-        return true; // Default to yes
-    }
-    /**
-     * Prompt for final confirmation
-     */
-    async promptConfirmation(rl) {
-        console.log('');
-        const input = await this.prompt(rl, `${chalk.green('Proceed with test generation?')} [${chalk.gray('Y/n')}]: `);
-        const value = input.trim().toLowerCase();
-        if (value === 'n' || value === 'no') {
-            console.log(chalk.yellow('\nWizard cancelled.'));
-            return false;
+        const files = this.resolveSourceFiles(value, context.cwd);
+        if (files.length === 0) {
+            console.log(chalk.yellow('  No matching files found, using provided patterns'));
+            return this.success(value.split(',').map(p => p.trim()).filter(p => p.length > 0));
         }
-        return true;
-    }
-    /**
-     * Print configuration summary
-     */
-    printSummary(result) {
-        console.log('');
-        console.log(chalk.blue('========================================'));
-        console.log(chalk.blue.bold('       Configuration Summary'));
-        console.log(chalk.blue('========================================'));
-        console.log('');
-        console.log(chalk.white('  Source Files:'));
-        result.sourceFiles.slice(0, 5).forEach(f => {
-            const relativePath = relative(this.cwd, f);
-            console.log(chalk.gray(`    - ${relativePath || f}`));
-        });
-        if (result.sourceFiles.length > 5) {
-            console.log(chalk.gray(`    ... and ${result.sourceFiles.length - 5} more`));
-        }
-        console.log('');
-        console.log(chalk.white(`  Test Type:        ${chalk.cyan(result.testType)}`));
-        console.log(chalk.white(`  Coverage Target:  ${chalk.cyan(result.coverageTarget + '%')}`));
-        console.log(chalk.white(`  Framework:        ${chalk.cyan(result.framework)}`));
-        console.log(chalk.white(`  AI Enhancement:   ${chalk.cyan(result.aiLevel)}`));
-        console.log(chalk.white(`  Anti-Patterns:    ${chalk.cyan(result.detectAntiPatterns ? 'Enabled' : 'Disabled')}`));
-        console.log('');
-    }
-    /**
-     * Generic prompt helper
-     */
-    prompt(rl, question) {
-        return new Promise(resolve => {
-            rl.question(question, answer => {
-                resolve(answer);
-            });
-        });
-    }
-    /**
-     * Get source file suggestions based on project structure
-     */
-    getSourceFileSuggestions() {
-        const suggestions = [];
-        // Check common directories
-        const commonDirs = ['src', 'lib', 'app', 'packages'];
-        for (const dir of commonDirs) {
-            const dirPath = join(this.cwd, dir);
-            if (existsSync(dirPath) && statSync(dirPath).isDirectory()) {
-                suggestions.push(`${dir}/**/*.ts`);
-                suggestions.push(dir);
-            }
-        }
-        // Add specific patterns for TypeScript projects
-        if (existsSync(join(this.cwd, 'src'))) {
-            suggestions.push('src/services/**/*.ts');
-            suggestions.push('src/utils/**/*.ts');
-            suggestions.push('src/components/**/*.tsx');
-        }
-        return suggestions;
+        return this.success(files);
     }
     /**
      * Resolve source files from input (glob patterns, directories, files)
      * Security: Validates that resolved paths stay within project directory to prevent path traversal
      */
-    resolveSourceFiles(input) {
+    resolveSourceFiles(input, cwd) {
         const files = [];
         const parts = input.split(',').map(p => p.trim()).filter(p => p.length > 0);
         // Normalize cwd for consistent comparison (resolve removes trailing slashes and normalizes)
-        const normalizedCwd = resolve(this.cwd);
+        const normalizedCwd = resolve(cwd);
         for (const part of parts) {
-            const resolved = resolve(this.cwd, part);
+            const resolved = resolve(cwd, part);
             // Security: Prevent path traversal - ensure resolved path is within project directory
             if (!resolved.startsWith(normalizedCwd + '/') && resolved !== normalizedCwd) {
                 console.warn(`Warning: Skipping path outside project directory: ${part}`);
@@ -366,7 +82,7 @@ export class TestGenerationWizard {
             }
             else if (part.includes('*')) {
                 // Handle glob pattern - for now, just expand common patterns
-                const baseDir = resolve(this.cwd, part.split('*')[0]);
+                const baseDir = resolve(cwd, part.split('*')[0]);
                 // Security: Validate glob base directory is within project
                 if (!baseDir.startsWith(normalizedCwd + '/') && baseDir !== normalizedCwd) {
                     console.warn(`Warning: Skipping glob pattern outside project directory: ${part}`);
@@ -431,12 +147,64 @@ export class TestGenerationWizard {
         }
         return files;
     }
+}
+// ============================================================================
+// Framework Detection Step (Custom)
+// ============================================================================
+/**
+ * Custom step for framework selection with auto-detection
+ */
+class FrameworkSelectStep extends SingleSelectStep {
+    cwd;
+    constructor(defaultFramework, cwd) {
+        const detectedFramework = FrameworkSelectStep.detectFramework(cwd);
+        const effectiveDefault = defaultFramework || detectedFramework || 'vitest';
+        super({
+            id: 'framework',
+            stepNumber: '4/6',
+            title: 'Test Framework',
+            description: 'Select the testing framework to use',
+            options: [
+                {
+                    key: '1',
+                    value: 'vitest',
+                    label: 'vitest',
+                    description: 'Vitest - Fast, Vite-native testing',
+                    isRecommended: detectedFramework === 'vitest',
+                },
+                {
+                    key: '2',
+                    value: 'jest',
+                    label: 'jest',
+                    description: 'Jest - Feature-rich, widely adopted',
+                    isRecommended: detectedFramework === 'jest',
+                },
+                {
+                    key: '3',
+                    value: 'mocha',
+                    label: 'mocha',
+                    description: 'Mocha - Flexible, configurable',
+                    isRecommended: detectedFramework === 'mocha',
+                },
+                {
+                    key: '4',
+                    value: 'playwright',
+                    label: 'playwright',
+                    description: 'Playwright - Browser automation (for e2e)',
+                    isRecommended: detectedFramework === 'playwright',
+                },
+            ],
+            defaultValue: effectiveDefault,
+            validValues: ['jest', 'vitest', 'mocha', 'playwright'],
+        });
+        this.cwd = cwd;
+    }
     /**
      * Detect test framework from project configuration
      * Security: Uses fs.readFileSync instead of require() to prevent code execution
      */
-    detectFramework() {
-        const packageJsonPath = join(this.cwd, 'package.json');
+    static detectFramework(cwd) {
+        const packageJsonPath = join(cwd, 'package.json');
         if (!existsSync(packageJsonPath)) {
             return null;
         }
@@ -458,20 +226,123 @@ export class TestGenerationWizard {
             // Ignore errors (file read or JSON parse failures)
         }
         // Check for config files
-        if (existsSync(join(this.cwd, 'vitest.config.ts')) || existsSync(join(this.cwd, 'vitest.config.js'))) {
+        if (existsSync(join(cwd, 'vitest.config.ts')) || existsSync(join(cwd, 'vitest.config.js'))) {
             return 'vitest';
         }
-        if (existsSync(join(this.cwd, 'jest.config.ts')) || existsSync(join(this.cwd, 'jest.config.js'))) {
+        if (existsSync(join(cwd, 'jest.config.ts')) || existsSync(join(cwd, 'jest.config.js'))) {
             return 'jest';
         }
-        if (existsSync(join(this.cwd, 'playwright.config.ts')) || existsSync(join(this.cwd, 'playwright.config.js'))) {
+        if (existsSync(join(cwd, 'playwright.config.ts')) || existsSync(join(cwd, 'playwright.config.js'))) {
             return 'playwright';
         }
         return null;
     }
-    /**
-     * Get default result for non-interactive mode
-     */
+}
+// ============================================================================
+// Wizard Implementation
+// ============================================================================
+export class TestGenerationWizard extends BaseWizard {
+    constructor(options = {}) {
+        super(options);
+    }
+    getTitle() {
+        return 'Test Generation Wizard';
+    }
+    getSubtitle() {
+        return 'Generate tests with AI-powered assistance';
+    }
+    getConfirmationPrompt() {
+        return 'Proceed with test generation?';
+    }
+    isNonInteractive() {
+        return this.options.nonInteractive ?? false;
+    }
+    getCommands() {
+        return [
+            // Step 1: Source files
+            new SourceFilesStep(this.options.defaultSourceFiles || ['.']),
+            // Step 2: Test type
+            new SingleSelectStep({
+                id: 'testType',
+                stepNumber: '2/6',
+                title: 'Test Type',
+                description: 'Select the type of tests to generate',
+                options: [
+                    { key: '1', value: 'unit', label: 'unit', description: 'Unit tests - isolated component testing' },
+                    { key: '2', value: 'integration', label: 'integration', description: 'Integration tests - module interaction testing' },
+                    { key: '3', value: 'e2e', label: 'e2e', description: 'End-to-end tests - full workflow testing' },
+                    { key: '4', value: 'property', label: 'property', description: 'Property-based tests - invariant testing' },
+                    { key: '5', value: 'contract', label: 'contract', description: 'Contract tests - API contract validation' },
+                ],
+                defaultValue: this.options.defaultTestType || 'unit',
+                validValues: ['unit', 'integration', 'e2e', 'property', 'contract'],
+            }),
+            // Step 3: Coverage target
+            new NumericStep({
+                id: 'coverageTarget',
+                stepNumber: '3/6',
+                title: 'Coverage target %',
+                description: 'Target code coverage percentage (0-100). Recommended: 80% for new code, 60% for legacy.',
+                defaultValue: this.options.defaultCoverageTarget || 80,
+                min: 0,
+                max: 100,
+            }),
+            // Step 4: Framework
+            new FrameworkSelectStep(this.options.defaultFramework, this.cwd),
+            // Step 5: AI enhancement level
+            new SingleSelectStep({
+                id: 'aiLevel',
+                stepNumber: '5/6',
+                title: 'AI Enhancement Level',
+                description: 'Select the level of AI assistance for test generation',
+                options: [
+                    { key: '1', value: 'none', label: 'none', description: 'None - Template-based generation only' },
+                    { key: '2', value: 'basic', label: 'basic', description: 'Basic - Simple pattern matching' },
+                    { key: '3', value: 'standard', label: 'standard', description: 'Standard - AI-powered test suggestions', isRecommended: true },
+                    { key: '4', value: 'advanced', label: 'advanced', description: 'Advanced - Full AI with edge case generation' },
+                ],
+                defaultValue: this.options.defaultAILevel || 'standard',
+                validValues: ['none', 'basic', 'standard', 'advanced'],
+            }),
+            // Step 6: Anti-pattern detection
+            new BooleanStep({
+                id: 'detectAntiPatterns',
+                stepNumber: '6/6',
+                title: 'Enable anti-pattern detection',
+                description: 'Enable detection and avoidance of code anti-patterns',
+                defaultValue: true,
+            }),
+        ];
+    }
+    buildResult(results) {
+        return {
+            sourceFiles: results.sourceFiles,
+            testType: results.testType,
+            coverageTarget: results.coverageTarget,
+            framework: results.framework,
+            aiLevel: results.aiLevel,
+            detectAntiPatterns: results.detectAntiPatterns,
+            cancelled: false,
+        };
+    }
+    printSummary(result) {
+        WizardPrompt.printSummaryHeader();
+        console.log(chalk.white('  Source Files:'));
+        result.sourceFiles.slice(0, 5).forEach(f => {
+            const relativePath = relative(this.cwd, f);
+            console.log(chalk.gray(`    - ${relativePath || f}`));
+        });
+        if (result.sourceFiles.length > 5) {
+            console.log(chalk.gray(`    ... and ${result.sourceFiles.length - 5} more`));
+        }
+        console.log('');
+        WizardPrompt.printSummaryField('Test Type', result.testType);
+        WizardPrompt.printSummaryField('Coverage Target', WizardFormat.percentage(result.coverageTarget));
+        WizardPrompt.printSummaryField('Framework', result.framework);
+        WizardPrompt.printSummaryField('AI Enhancement', result.aiLevel);
+        WizardPrompt.printSummaryField('Anti-Patterns', WizardFormat.enabledDisabled(result.detectAntiPatterns));
+        console.log('');
+    }
     getDefaults() {
         return {
             sourceFiles: this.options.defaultSourceFiles || ['.'],
@@ -483,9 +354,6 @@ export class TestGenerationWizard {
             cancelled: false,
         };
     }
-    /**
-     * Get cancelled result
-     */
     getCancelled() {
         return {
             sourceFiles: [],