agentic-qe 3.14.3 → 3.14.4

package/dist/domains/code-intelligence/services/metric-collector/test-counter.js

@@ -185,29 +185,21 @@ export function checkTestRunners(projectPath) {
  */
 function countVitestTests(projectPath, config) {
     try {
-        // Use vitest list for safe enumeration (NO execution)
-        // --reporter=json gives structured output we can parse
-        const output = execSync('npx vitest list --reporter=json', {
+        // Use vitest list for safe enumeration (NO execution). --json emits one
+        // entry per discovered test in Vitest 4 and 5; --reporter=json does not.
+        const output = execSync('npx vitest list --json', {
             cwd: projectPath,
             encoding: 'utf-8',
             timeout: Math.min(config.timeout, 30000), // Cap at 30s for listing
             maxBuffer: 10 * 1024 * 1024, // 10MB buffer (reduced from 50MB)
         });
-        // Parse JSON output from vitest list
         try {
             const data = safeJsonParse(output);
-            // vitest list --reporter=json returns array of test files with tests
-            if (Array.isArray(data)) {
-                let total = 0;
-                for (const file of data) {
-                    // Each file entry has a `tests` array
-                    if (file.tests && Array.isArray(file.tests)) {
-                        total += countTestsRecursive(file.tests);
-                    }
-                }
-                return classifyTests(total, 'vitest', projectPath);
+            // Vitest list --json returns a flat array of { name, file, location? }.
+            if (Array.isArray(data) && data.every(test => test && typeof test.name === 'string' && typeof test.file === 'string')) {
+                return classifyTests(data.length, 'vitest', projectPath);
             }
-            // Alternative format: object with numTotalTests
+            // Preserve compatibility with older structured runner output.
             if (data && typeof data.numTotalTests === 'number') {
                 return classifyTests(data.numTotalTests, 'vitest', projectPath);
             }
@@ -228,34 +220,11 @@ function countVitestTests(projectPath, config) {
         // If vitest list produced no usable output, fall back to file pattern
         return countTestsByFilePattern(projectPath, config);
     }
-    catch (error) {
+    catch {
         // vitest list failed (not installed, timeout, etc.) - fall back to file pattern
         return countTestsByFilePattern(projectPath, config);
     }
 }
-/**
- * Recursively count tests in vitest's nested test structure
- */
-function countTestsRecursive(tests) {
-    let count = 0;
-    for (const test of tests) {
-        if (typeof test === 'object' && test !== null) {
-            const t = test;
-            // Count this test if it has a name (actual test, not describe block)
-            if (t.type === 'test' || t.mode === 'run' || t.mode === 'skip') {
-                count++;
-            }
-            // Recurse into children (nested describes)
-            if (Array.isArray(t.tests)) {
-                count += countTestsRecursive(t.tests);
-            }
-            if (Array.isArray(t.children)) {
-                count += countTestsRecursive(t.children);
-            }
-        }
-    }
-    return count;
-}
 /**
  * Count tests using Jest
  * Uses `jest --listTests` for accurate test file discovery WITHOUT execution.
@@ -286,7 +255,7 @@ function countJestTests(projectPath, config) {
         // If jest --listTests produced no files, fall back to file pattern
         return countTestsByFilePattern(projectPath, config);
     }
-    catch (error) {
+    catch {
         // jest --listTests failed - fall back to file pattern
         return countTestsByFilePattern(projectPath, config);
     }

package/dist/domains/constants.d.ts

@@ -460,12 +460,7 @@ export declare const LLM_ANALYSIS_CONSTANTS: {
     /**
      * Model tier mapping.
      */
-    readonly MODEL_TIERS: {
-        readonly 1: "claude-haiku-4-5";
-        readonly 2: "claude-sonnet-4-6";
-        readonly 3: "claude-sonnet-4-6";
-        readonly 4: "claude-opus-4-7";
-    };
+    readonly MODEL_TIERS: Readonly<Record<1 | 4 | 2 | 3, string>>;
 };
 export type TestExecutionConstants = typeof TEST_EXECUTION_CONSTANTS;
 export type E2EConstants = typeof E2E_CONSTANTS;

package/dist/domains/constants.js

@@ -7,6 +7,7 @@
  *
  * @see ADR-051: LLM-Powered Analysis
  */
+import { CLAUDE_TIER_MODELS } from '../shared/llm/model-registry.js';
 // ============================================================================
 // Test Execution Constants
 // ============================================================================
@@ -493,11 +494,6 @@ export const LLM_ANALYSIS_CONSTANTS = {
     /**
      * Model tier mapping.
      */
-    MODEL_TIERS: {
-        1: 'claude-haiku-4-5',
-        2: 'claude-sonnet-4-6',
-        3: 'claude-sonnet-4-6',
-        4: 'claude-opus-4-7',
-    },
+    MODEL_TIERS: CLAUDE_TIER_MODELS,
 };
 //# sourceMappingURL=constants.js.map

package/dist/domains/contract-testing/services/contract-validator.js

@@ -8,6 +8,7 @@ import { createSafeRegex } from '../../../shared/security/regex-safety-validator
 import { CONTRACT_CONSTANTS, LLM_ANALYSIS_CONSTANTS } from '../../constants.js';
 import { toError } from '../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../shared/safe-json.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 const DEFAULT_CONFIG = {
     strictMode: true,
     validateExamples: true,
@@ -46,13 +47,7 @@ export class ContractValidatorService {
      * ADR-051: Get model name for tier
      */
     getModelForTier(tier) {
-        const models = {
-            1: 'claude-haiku-4-5',
-            2: 'claude-sonnet-4-6',
-            3: 'claude-sonnet-4-6',
-            4: 'claude-opus-4-7',
-        };
-        return models[tier] || models[2];
+        return getClaudeModelForTier(tier);
     }
     /**
      * ADR-051: LLM-powered contract analysis

package/dist/domains/coverage-analysis/services/coverage-analyzer.js

@@ -9,6 +9,7 @@ import { ok, err } from '../../../shared/types/index.js';
 import { toError } from '../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../shared/safe-json.js';
 import { writeQualityEvidence } from '../../quality-assessment/quality-evidence.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 const DEFAULT_CONFIG = {
     defaultThreshold: 80,
     enableLLMAnalysis: true, // On by default - opt-out
@@ -182,18 +183,7 @@ export class CoverageAnalyzerService {
      * Get model ID for the configured tier
      */
     getModelForTier(tier) {
-        switch (tier) {
-            case 1:
-                return 'claude-haiku-4-5-20251001';
-            case 2:
-                return 'claude-sonnet-4-6';
-            case 3:
-                return 'claude-sonnet-4-6';
-            case 4:
-                return 'claude-opus-4-7';
-            default:
-                return 'claude-sonnet-4-6';
-        }
+        return getClaudeModelForTier(tier);
     }
     /**
      * Analyze coverage gaps using LLM for intelligent insights

package/dist/domains/coverage-analysis/services/gap-detector.js

@@ -8,6 +8,7 @@ import { LoggerFactory } from '../../../logging/index.js';
 import { ok, err } from '../../../shared/types/index.js';
 import { toError } from '../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../shared/safe-json.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 const DEFAULT_CONFIG = {
     minCoverage: 80,
     enableLLMAnalysis: true, // On by default - opt-out
@@ -59,13 +60,7 @@ export class GapDetectorService {
      * Get model ID for the configured tier
      */
     getModelForTier(tier) {
-        switch (tier) {
-            case 1: return 'claude-haiku-4-5-20251001';
-            case 2: return 'claude-sonnet-4-6';
-            case 3: return 'claude-sonnet-4-6';
-            case 4: return 'claude-opus-4-7';
-            default: return 'claude-sonnet-4-6';
-        }
+        return getClaudeModelForTier(tier);
     }
     /**
      * Analyze coverage gaps using LLM for intelligent prioritization

package/dist/domains/defect-intelligence/services/defect-predictor.js

@@ -13,6 +13,7 @@ import { FileReader } from '../../../shared/io/index.js';
 import { TypeScriptParser } from '../../../shared/parsers/index.js';
 import { toError } from '../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../shared/safe-json.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 // LLMDefectAnalysis is imported from ../interfaces
 const DEFAULT_CONFIG = {
     defaultThreshold: 0.5,
@@ -111,18 +112,7 @@ export class DefectPredictorService {
      * Get model ID for the configured tier
      */
     getModelForTier(tier) {
-        switch (tier) {
-            case 1:
-                return 'claude-haiku-4-5-20251001';
-            case 2:
-                return 'claude-sonnet-4-6';
-            case 3:
-                return 'claude-sonnet-4-6';
-            case 4:
-                return 'claude-opus-4-7';
-            default:
-                return 'claude-sonnet-4-6';
-        }
+        return getClaudeModelForTier(tier);
     }
     /**
      * Analyze defect risk using LLM for enhanced insights

package/dist/domains/defect-intelligence/services/root-cause-analyzer.js

@@ -8,6 +8,7 @@ import { LoggerFactory } from '../../../logging/index.js';
 import { ok, err } from '../../../shared/types/index.js';
 import { toError } from '../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../shared/safe-json.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 const DEFAULT_CONFIG = {
     maxTimelineEvents: 20,
     maxRelatedFiles: 10,
@@ -193,13 +194,7 @@ export class RootCauseAnalyzerService {
      * Get model ID for the configured tier
      */
     getModelForTier(tier) {
-        switch (tier) {
-            case 1: return 'claude-haiku-4-5-20251001';
-            case 2: return 'claude-sonnet-4-6';
-            case 3: return 'claude-sonnet-4-6';
-            case 4: return 'claude-opus-4-7';
-            default: return 'claude-sonnet-4-6';
-        }
+        return getClaudeModelForTier(tier);
     }
     /**
      * Analyze root cause using LLM for deeper insights

package/dist/domains/learning-optimization/services/learning-coordinator.js

@@ -8,6 +8,7 @@ import { ok, err } from '../../../shared/types/index.js';
 import { rehydrateDates } from '../../../shared/utils/kv-date-rehydrate.js';
 import { toError } from '../../../shared/error-utils.js';
 import { createQEFlashAttention, } from '../../../integrations/ruvector/wrappers.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 const DEFAULT_CONFIG = {
     minExperiencesForPattern: 5,
     patternConfidenceThreshold: 0.7,
@@ -162,13 +163,7 @@ export class LearningCoordinatorService {
      * @returns Model ID string
      */
     getModelForTier(tier) {
-        const models = {
-            1: 'claude-haiku-4-5',
-            2: 'claude-sonnet-4-6',
-            3: 'claude-sonnet-4-6',
-            4: 'claude-opus-4-7',
-        };
-        return models[tier] || models[2];
+        return getClaudeModelForTier(tier);
     }
     /**
      * ADR-051: LLM-powered pattern synthesis

package/dist/domains/quality-assessment/services/deployment-advisor.js

@@ -9,6 +9,7 @@ import { v4 as uuidv4 } from 'uuid';
 import { ok, err } from '../../../shared/types/index.js';
 import { toError } from '../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../shared/safe-json.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 const DEFAULT_CONFIG = {
     riskWeights: {
         coverage: 0.15,
@@ -68,13 +69,7 @@ export class DeploymentAdvisorService {
      * Get model ID for the configured tier
      */
     getModelForTier(tier) {
-        switch (tier) {
-            case 1: return 'claude-haiku-4-5-20251001';
-            case 2: return 'claude-sonnet-4-6';
-            case 3: return 'claude-sonnet-4-6';
-            case 4: return 'claude-opus-4-7';
-            default: return 'claude-sonnet-4-6';
-        }
+        return getClaudeModelForTier(tier);
     }
     /**
      * Generate deployment advice using LLM for deeper insights

package/dist/domains/quality-assessment/services/quality-analyzer.js

@@ -8,6 +8,7 @@ import { ok, err } from '../../../shared/types/index.js';
 import { getCodeMetricsAnalyzer } from '../../../shared/metrics/index.js';
 import { toError } from '../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../shared/safe-json.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 const DEFAULT_CONFIG = {
     enableTrendAnalysis: true,
     trendDataPointsMin: 3,
@@ -77,13 +78,7 @@ export class QualityAnalyzerService {
      * Get model ID for the configured tier
      */
     getModelForTier(tier) {
-        switch (tier) {
-            case 1: return 'claude-haiku-4-5-20251001';
-            case 2: return 'claude-sonnet-4-6';
-            case 3: return 'claude-sonnet-4-6';
-            case 4: return 'claude-opus-4-7';
-            default: return 'claude-sonnet-4-6';
-        }
+        return getClaudeModelForTier(tier);
     }
     /**
      * Generate quality insights using LLM

package/dist/domains/requirements-validation/services/requirements-validator.js

@@ -9,6 +9,7 @@ import { LoggerFactory } from '../../../logging/index.js';
 import { ok, err } from '../../../shared/types/index.js';
 import { toError } from '../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../shared/safe-json.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 const DEFAULT_CONFIG = {
     enableAmbiguityDetection: true,
     minAcceptanceCriteria: 1,
@@ -83,13 +84,7 @@ export class RequirementsValidatorService {
      * Get model ID for the configured tier
      */
     getModelForTier(tier) {
-        switch (tier) {
-            case 1: return 'claude-haiku-4-5-20251001';
-            case 2: return 'claude-sonnet-4-6';
-            case 3: return 'claude-sonnet-4-6';
-            case 4: return 'claude-opus-4-7';
-            default: return 'claude-sonnet-4-6';
-        }
+        return getClaudeModelForTier(tier);
     }
     /**
      * Analyze requirements with LLM for deeper insights

package/dist/domains/security-compliance/services/scanners/sast-scanner.js

@@ -11,6 +11,7 @@ import { ok, err } from '../../../../shared/types/index.js';
 import { ALL_SECURITY_PATTERNS, BUILT_IN_RULE_SETS } from './security-patterns.js';
 import { toError } from '../../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../../shared/safe-json.js';
+import { CLAUDE_TIER_MODELS, DEFAULT_OPUS_MODEL } from '../../../../shared/llm/model-registry.js';
 import { isSemgrepAvailable, runSemgrepWithRules, convertSemgrepFindings, } from '../semgrep-integration.js';
 // ============================================================================
 // SAST Scanner Service
@@ -537,13 +538,8 @@ export class SASTScanner {
      * Get model ID for the configured tier
      */
     getModelForTier(tier) {
-        switch (tier) {
-            case 1: return 'claude-haiku-4-5-20251001';
-            case 2: return 'claude-sonnet-4-6';
-            case 3: return 'claude-sonnet-4-6';
-            case 4: return 'claude-opus-4-7';
-            default: return 'claude-opus-4-7'; // Default to Opus for security
-        }
+        // Default to Opus for security when the tier is out of range
+        return CLAUDE_TIER_MODELS[tier] ?? DEFAULT_OPUS_MODEL;
     }
     /**
      * Analyze vulnerability with LLM for deeper insights

package/dist/domains/test-execution/services/flaky-detector.js

@@ -11,6 +11,7 @@ import { ok, err } from '../../../shared/types/index.js';
 import { TEST_EXECUTION_CONSTANTS, RETRY_CONSTANTS } from '../../constants.js';
 import { toError } from '../../../shared/error-utils.js';
 import { safeJsonParse } from '../../../shared/safe-json.js';
+import { getTestRunnerExecutionError } from '../../../shared/test-runner-verdict.js';
 import { createVitestJsonReport, needsVitestJsonReportFile } from '../../../shared/vitest-json-report.js';
 import { secureRandom } from '../../../shared/utils/crypto-random.js';
 const DEFAULT_FLAKY_CONFIG = {
@@ -291,13 +292,38 @@ export class FlakyDetectorService {
             child.on('close', (code) => {
                 clearTimeout(timeout);
                 const duration = Date.now() - startTime;
-                const reportText = report ? report.read(stdout) : stdout;
+                let reportText;
+                try {
+                    reportText = report ? report.read(stdout) : stdout;
+                }
+                catch (error) {
+                    report?.cleanup();
+                    reject(toError(error));
+                    return;
+                }
                 report?.cleanup();
+                if (reportText === undefined) {
+                    reject(new Error(`The current Vitest JSON report is missing for ${file}.`));
+                    return;
+                }
                 try {
                     // Parse the test results from the JSON report (or stdout for other runners)
                     const parsedResults = this.parseTestOutput(reportText, stderr, code ?? 0, file, runId, runIndex, duration);
+                    if (report) {
+                        const verdictReport = safeJsonParse(reportText);
+                        const assertions = verdictReport.testResults?.flatMap(suite => suite.assertionResults ?? []) ?? [];
+                        const executionError = getTestRunnerExecutionError('vitest', file, code, {
+                            passed: assertions.filter(test => test.status === 'passed').length,
+                            failed: assertions.filter(test => test.status === 'failed').length,
+                            skipped: assertions.filter(test => test.status === 'skipped' || test.status === 'pending').length,
+                        }, stderr, verdictReport);
+                        if (executionError)
+                            throw executionError;
+                    }
                     // If parsing fails but we have an exit code, create a single result for the file
                     if (parsedResults.size === 0) {
+                        if (report)
+                            throw new Error(`The current Vitest JSON report has no test results for ${file}.`);
                         const testId = this.generateTestId(file, 'main');
                         results.set(testId, [
                             {
@@ -321,6 +347,10 @@ export class FlakyDetectorService {
                     resolve(results);
                 }
                 catch (parseError) {
+                    if (report) {
+                        reject(toError(parseError));
+                        return;
+                    }
                     // If we can't parse output but process completed, create result from exit code
                     const testId = this.generateTestId(file, 'main');
                     results.set(testId, [
@@ -356,7 +386,7 @@ export class FlakyDetectorService {
                 return this.parseVitestJson(parsed, file, runId, runIndex);
             }
         }
-        catch (error) {
+        catch {
             // Non-critical: not valid JSON, try other formats
             logger.debug('Vitest JSON parse failed:');
         }
@@ -370,7 +400,7 @@ export class FlakyDetectorService {
                 }
             }
         }
-        catch (error) {
+        catch {
             // Non-critical: not Jest format
             logger.debug('Jest JSON parse failed:');
         }

package/dist/domains/test-execution/services/retry-handler.js

@@ -324,7 +324,7 @@ export class RetryHandlerService {
                     return 'mocha';
             }
         }
-        catch (error) {
+        catch {
             // Non-critical: package.json read errors during test runner detection
             logger.debug('package.json read failed:');
         }
@@ -409,8 +409,16 @@ export class RetryHandlerService {
                 }
                 // Parse result based on exit code and output (Vitest writes the JSON
                 // report to the --outputFile; read it back rather than trusting stdout).
-                const result = this.parseTestResult(code, report ? report.read(stdout) : stdout, stderr);
-                settle(resolve, result);
+                try {
+                    const reportText = report ? report.read(stdout) : stdout;
+                    if (reportText === undefined) {
+                        throw new Error('The current Vitest JSON report is missing for the retry run.');
+                    }
+                    settle(resolve, this.parseTestResult(code, reportText, stderr));
+                }
+                catch (error) {
+                    settle(reject, toError(error));
+                }
             });
             proc.on('error', (err) => {
                 clearTimeout(timeoutHandle);
@@ -425,55 +433,55 @@ export class RetryHandlerService {
      * Parse test runner output to determine pass/fail status
      */
     parseTestResult(exitCode, stdout, stderr) {
-        // Exit code 0 typically means all tests passed
-        if (exitCode === 0) {
-            return { passed: true };
-        }
-        // Try to parse JSON output for more detailed error info
+        // A passing process and a passing report must agree when JSON is available.
         try {
             // Vitest JSON output
             const vitestMatch = stdout.match(/\{[\s\S]*"testResults"[\s\S]*\}/);
             if (vitestMatch) {
                 const result = safeJsonParse(vitestMatch[0]);
-                if (result.success === true || result.numFailedTests === 0) {
+                if (exitCode === 0 && result.success !== false
+                    && result.numFailedTests === 0 && result.numFailedTestSuites === 0) {
                     return { passed: true };
                 }
                 const failedTest = result.testResults?.[0]?.assertionResults?.find((r) => r.status === 'failed');
                 return {
                     passed: false,
-                    error: failedTest?.failureMessages?.join('\n') ?? `Test failed with exit code ${exitCode}`,
+                    error: failedTest?.failureMessages?.join('\n') || stderr || `Test failed with exit code ${exitCode}`,
                 };
             }
             // Jest JSON output
             const jestMatch = stdout.match(/\{[\s\S]*"numFailedTests"[\s\S]*\}/);
             if (jestMatch) {
                 const result = safeJsonParse(jestMatch[0]);
-                if (result.success === true || result.numFailedTests === 0) {
+                if (exitCode === 0 && result.success !== false
+                    && result.numFailedTests === 0 && result.numFailedTestSuites === 0) {
                     return { passed: true };
                 }
                 const failedTest = result.testResults?.[0]?.assertionResults?.find((r) => r.status === 'failed');
                 return {
                     passed: false,
-                    error: failedTest?.failureMessages?.join('\n') ?? `Test failed with exit code ${exitCode}`,
+                    error: failedTest?.failureMessages?.join('\n') || stderr || `Test failed with exit code ${exitCode}`,
                 };
             }
             // Mocha JSON output
             const mochaMatch = stdout.match(/\{[\s\S]*"stats"[\s\S]*"failures"[\s\S]*\}/);
             if (mochaMatch) {
                 const result = safeJsonParse(mochaMatch[0]);
-                if (result.stats?.failures === 0) {
+                if (exitCode === 0 && result.stats?.failures === 0) {
                     return { passed: true };
                 }
                 const failure = result.failures?.[0];
                 return {
                     passed: false,
-                    error: failure?.err?.message ?? `Test failed with exit code ${exitCode}`,
+                    error: failure?.err?.message || stderr || `Test failed with exit code ${exitCode}`,
                 };
             }
         }
         catch {
             // JSON parsing failed, fall back to simple exit code check
         }
+        if (exitCode === 0)
+            return { passed: true };
         // Non-zero exit code means failure
         const errorOutput = stderr || stdout || `Test failed with exit code ${exitCode}`;
         return {

package/dist/domains/test-execution/services/test-executor.js

@@ -15,6 +15,7 @@ import { secureRandom, secureRandomInt } from '../../../shared/utils/crypto-rand
 import { writeQualityEvidence } from '../../quality-assessment/quality-evidence.js';
 import { getTestRunnerExecutionError, TestRunnerExecutionError } from '../../../shared/test-runner-verdict.js';
 import { createVitestJsonReport } from '../../../shared/vitest-json-report.js';
+import { getClaudeModelForTier } from '../../../shared/llm/model-registry.js';
 const DEFAULT_CONFIG = {
     simulateForTesting: false,
     simulatedTestsPerFile: 5,
@@ -241,13 +242,7 @@ export class TestExecutorService {
      * ADR-051: Get model name for tier
      */
     getModelForTier(tier) {
-        const models = {
-            1: 'claude-haiku-4-5',
-            2: 'claude-sonnet-4-6',
-            3: 'claude-sonnet-4-6',
-            4: 'claude-opus-4-7',
-        };
-        return models[tier] || models[2];
+        return getClaudeModelForTier(tier);
     }
     /**
      * ADR-051: LLM-powered test failure analysis
@@ -273,7 +268,7 @@ Provide:
             });
             return response.content;
         }
-        catch (error) {
+        catch {
             logger.warn('LLM analysis failed:');
             return null;
         }
@@ -379,26 +374,38 @@ Provide:
             let stdout = '';
             let stderr = '';
             let killed = false;
-            const finish = (result) => {
-                report?.cleanup();
+            let settled = false;
+            const finish = (result, cleanup = true) => {
+                if (settled)
+                    return;
+                settled = true;
+                if (cleanup)
+                    report?.cleanup();
                 resolve(result);
             };
             // Spawn the test runner process
             // Note: shell: false (default) to prevent command injection (CWE-78)
             // Arguments are passed as array to avoid shell interpretation
-            const proc = spawn(command, args, {
-                cwd: process.cwd(),
-                env: {
-                    ...process.env,
-                    FORCE_COLOR: '0', // Disable color codes for easier parsing
-                    CI: 'true', // Enable CI mode for consistent output
-                },
-            });
+            let proc;
+            try {
+                proc = spawn(command, args, {
+                    cwd: process.cwd(),
+                    env: {
+                        ...process.env,
+                        FORCE_COLOR: '0', // Disable color codes for easier parsing
+                        CI: 'true', // Enable CI mode for consistent output
+                    },
+                });
+            }
+            catch (error) {
+                finish(err(new Error(`Failed to spawn test runner: ${toErrorMessage(error)}. Is '${command}' installed?`)));
+                return;
+            }
             // Set timeout
             const timeoutId = setTimeout(() => {
                 killed = true;
                 proc.kill('SIGTERM');
-                finish(err(new Error(`Test execution timed out after ${timeout}ms for files: ${fileLabel}`)));
+                finish(err(new Error(`Test execution timed out after ${timeout}ms for files: ${fileLabel}`)), false);
             }, timeout);
             proc.stdout?.on('data', (data) => {
                 stdout += data.toString();
@@ -409,11 +416,23 @@ Provide:
             proc.on('close', (code) => {
                 clearTimeout(timeoutId);
                 if (killed) {
-                    return; // Already handled by timeout
+                    report?.cleanup();
+                    return; // Timeout result was already returned; child has now closed.
                 }
                 // Parse results based on framework. Vitest 5 writes the JSON report to
                 // a file instead of stdout, so read it back through the report handle.
-                const reportText = report ? report.read(stdout) : stdout;
+                let reportText;
+                try {
+                    reportText = report ? report.read(stdout) : stdout;
+                }
+                catch (error) {
+                    finish(err(toError(error)));
+                    return;
+                }
+                if (reportText === undefined) {
+                    finish(err(new Error(`The current Vitest JSON report is missing for ${fileLabel}.`)));
+                    return;
+                }
                 const parseResult = this.parseTestOutput(reportText, stderr, fileLabel, framework, code);
                 // If no coverage in stdout JSON, try reading from disk
                 // (vitest/jest write coverage to coverage/coverage-summary.json)
@@ -430,6 +449,8 @@ Provide:
             });
             proc.on('error', (error) => {
                 clearTimeout(timeoutId);
+                if (killed)
+                    return;
                 finish(err(new Error(`Failed to spawn test runner: ${error.message}. Is '${command}' installed?`)));
             });
         });