agentic-qe 3.13.0 → 3.13.2

package/dist/coordination/result-saver.js

@@ -177,14 +177,24 @@ export class ResultSaver {
     }
     async saveCoverage(result, prefix, options) {
         const files = [];
+        // #569: branch/function coverage are nullable — `null` means the metric was
+        // never collected, which the report must render as "n/a" rather than crash
+        // on (or, worse, print as 0.0%).
         const data = result;
         const coverageDir = path.join(this.resultsDir, 'coverage');
         // Save JSON
         const jsonPath = path.join(coverageDir, `${prefix}_coverage.json`);
         await fs.writeFile(jsonPath, JSON.stringify(data, null, 2));
         files.push(await this.createFileEntry(jsonPath, 'json'));
-        // Save LCOV format
-        if (options.includeSecondary !== false) {
+        // Save LCOV format — ONLY for genuinely measured coverage.
+        //
+        // #569/ADR-126: LCOV has nowhere to record provenance. Emitting it for an
+        // estimated (or absent) result hands downstream tools a file that is
+        // indistinguishable from instrumented output — `null` would serialize as
+        // `DA:1,0 / LF:100 / LH:0`, asserting a measured 0% over 100 lines that
+        // nothing ever measured. The JSON artifact above keeps the full result with
+        // its provenance; the lossy format is simply not written when it would lie.
+        if (options.includeSecondary !== false && data.measured === true) {
             const lcovPath = path.join(coverageDir, `${prefix}_coverage.lcov`);
             const lcovContent = this.generateLcov(data);
             await fs.writeFile(lcovPath, lcovContent);
@@ -299,12 +309,13 @@ export class ResultSaver {
     // ==========================================================================
     generateLcov(data) {
         // Simplified LCOV - real implementation would have per-file data
-        return `TN:agentic-qe-coverage
-SF:summary
-DA:1,${Math.round(data.lineCoverage)}
-LF:100
-LH:${Math.round(data.lineCoverage)}
-end_of_record
+        const line = Math.round(data.lineCoverage ?? 0);
+        return `TN:agentic-qe-coverage
+SF:summary
+DA:1,${line}
+LF:100
+LH:${line}
+end_of_record
 `;
     }
     generateSarif(data) {
@@ -334,104 +345,115 @@ end_of_record
         }, null, 2);
     }
     generateTestReport(data, options) {
-        return `# Test Generation Report
-
-**Generated:** ${new Date().toISOString()}
-**Language:** ${options.language || 'typescript'}
-**Framework:** ${options.framework || 'vitest'}
-
-## Summary
-
-| Metric | Value |
-|--------|-------|
-| Tests Generated | ${data.testsGenerated} |
-| Coverage Estimate | ${data.coverageEstimate}% |
-| Patterns Used | ${data.patternsUsed.length} |
-
-## Patterns Applied
-
-${data.patternsUsed.map(p => `- ${p}`).join('\n')}
-
-## Generated Tests
-
-${data.tests.map(t => `### ${t.name}
-- **File:** ${t.file}
-- **Type:** ${t.type}
-`).join('\n')}
+        return `# Test Generation Report
+
+**Generated:** ${new Date().toISOString()}
+**Language:** ${options.language || 'typescript'}
+**Framework:** ${options.framework || 'vitest'}
+
+## Summary
+
+| Metric | Value |
+|--------|-------|
+| Tests Generated | ${data.testsGenerated} |
+| Coverage Estimate | ${data.coverageEstimate}% |
+| Patterns Used | ${data.patternsUsed.length} |
+
+## Patterns Applied
+
+${data.patternsUsed.map(p => `- ${p}`).join('\n')}
+
+## Generated Tests
+
+${data.tests.map(t => `### ${t.name}
+- **File:** ${t.file}
+- **Type:** ${t.type}
+`).join('\n')}
 `;
     }
     generateCoverageReport(data) {
-        return `# Coverage Analysis Report
-
-**Generated:** ${new Date().toISOString()}
-**Algorithm:** Sublinear O(log n)
-
-## Summary
-
-| Metric | Coverage |
-|--------|----------|
-| Line Coverage | ${data.lineCoverage.toFixed(1)}% |
-| Branch Coverage | ${data.branchCoverage.toFixed(1)}% |
-| Function Coverage | ${data.functionCoverage.toFixed(1)}% |
-| Statement Coverage | ${data.statementCoverage.toFixed(1)}% |
-| Total Files | ${data.totalFiles} |
-
-## Coverage Gaps
-
-${data.gaps.length === 0 ? 'No significant gaps detected.' : data.gaps.map(g => `### ${g.file}
-- **Risk:** ${g.risk}
-- **Uncovered Lines:** ${g.lines.join(', ')}
-`).join('\n')}
+        // #569: a metric that was never collected renders as "not collected", not
+        // as a number. Rendering `null` as 0.0% is how "no branch data" became
+        // "0% branch coverage" — and the inverse formula made it 100%.
+        const metric = (value) => value === null ? '_not collected_' : `${value.toFixed(1)}%`;
+        const provenanceBanner = data.estimated
+            ? `\n> ⚠️ **ESTIMATED — NOT MEASURED.** No instrumentation ran; these figures are a\n` +
+                `> static estimate derived from source shape. Do not treat them as coverage\n` +
+                `> measurements or act on individual gaps without running real instrumentation.\n` +
+                (data.warning ? `>\n> ${data.warning}\n` : '')
+            : '';
+        return `# Coverage Analysis Report
+
+**Generated:** ${new Date().toISOString()}
+**Algorithm:** Sublinear O(log n)
+**Method:** ${data.coverageMethod ?? 'unknown'}${provenanceBanner}
+
+## Summary
+
+| Metric | Coverage |
+|--------|----------|
+| Line Coverage | ${metric(data.lineCoverage)} |
+| Branch Coverage | ${metric(data.branchCoverage)} |
+| Function Coverage | ${metric(data.functionCoverage)} |
+| Statement Coverage | ${metric(data.statementCoverage)} |
+| Total Files | ${data.totalFiles} |
+
+## Coverage Gaps
+
+${data.gaps.length === 0 ? 'No significant gaps detected.' : data.gaps.map(g => `### ${g.file}
+- **Risk:** ${g.risk}
+- **Uncovered Lines:** ${g.lines.join(', ')}
+`).join('\n')}
 `;
     }
     generateSecurityReport(data) {
-        return `# Security Scan Report
-
-**Generated:** ${new Date().toISOString()}
-**Scanner:** Agentic QE v3 Security
-
-## Summary
-
-| Severity | Count |
-|----------|-------|
-| Critical | ${data.critical} |
-| High | ${data.high} |
-| Medium | ${data.medium} |
-| Low | ${data.low} |
-| **Total** | **${data.vulnerabilities}** |
-
-## Top Vulnerabilities
-
-${data.topVulnerabilities.map(v => `### ${v.type}
-- **Severity:** ${v.severity.toUpperCase()}
-- **File:** ${v.file}
-- **Line:** ${v.line}
-`).join('\n')}
-
-## Recommendations
-
-${data.recommendations.map((r, i) => `${i + 1}. ${r}`).join('\n')}
+        return `# Security Scan Report
+
+**Generated:** ${new Date().toISOString()}
+**Scanner:** Agentic QE v3 Security
+
+## Summary
+
+| Severity | Count |
+|----------|-------|
+| Critical | ${data.critical} |
+| High | ${data.high} |
+| Medium | ${data.medium} |
+| Low | ${data.low} |
+| **Total** | **${data.vulnerabilities}** |
+
+## Top Vulnerabilities
+
+${data.topVulnerabilities.map(v => `### ${v.type}
+- **Severity:** ${v.severity.toUpperCase()}
+- **File:** ${v.file}
+- **Line:** ${v.line}
+`).join('\n')}
+
+## Recommendations
+
+${data.recommendations.map((r, i) => `${i + 1}. ${r}`).join('\n')}
 `;
     }
     generateQualityReport(data) {
-        return `# Quality Assessment Report
-
-**Generated:** ${new Date().toISOString()}
-**Status:** ${data.passed ? 'PASSED' : 'FAILED'}
-
-## Quality Score
-
-**${data.qualityScore.toFixed(1)}** / 100
-
-## Metrics
-
-| Metric | Score |
-|--------|-------|
-${Object.entries(data.metrics).map(([k, v]) => `| ${k} | ${v.toFixed(1)} |`).join('\n')}
-
-## Recommendations
-
-${data.recommendations.length === 0 ? 'No recommendations - all quality gates passed.' : data.recommendations.map((r, i) => `${i + 1}. ${r}`).join('\n')}
+        return `# Quality Assessment Report
+
+**Generated:** ${new Date().toISOString()}
+**Status:** ${data.passed ? 'PASSED' : 'FAILED'}
+
+## Quality Score
+
+**${data.qualityScore.toFixed(1)}** / 100
+
+## Metrics
+
+| Metric | Score |
+|--------|-------|
+${Object.entries(data.metrics).map(([k, v]) => `| ${k} | ${v.toFixed(1)} |`).join('\n')}
+
+## Recommendations
+
+${data.recommendations.length === 0 ? 'No recommendations - all quality gates passed.' : data.recommendations.map((r, i) => `${i + 1}. ${r}`).join('\n')}
 `;
     }
     // ==========================================================================

package/dist/coordination/task-executor.d.ts

@@ -53,6 +53,8 @@ export declare class DomainTaskExecutor implements TaskHandlerContext {
     private coverageAnalyzer;
     private securityScanner;
     private testGenerator;
+    /** #567: router-free instance for callers that opt out of LLM enhancement. */
+    private deterministicTestGenerator;
     private knowledgeGraph;
     private qualityAnalyzer;
     private agentBooster;
@@ -72,7 +74,30 @@ export declare class DomainTaskExecutor implements TaskHandlerContext {
     setQualityFeedbackLoop(loop: QualityFeedbackLoop | null): void;
     getCoverageAnalyzer(): CoverageAnalyzerService;
     getSecurityScanner(): SecurityScannerService;
-    getTestGenerator(): TestGeneratorService;
+    /**
+     * Issue #567: async so the ADR-043 LLM router can be resolved before the
+     * generator is constructed. `TestGeneratorService` captures `llmRouter` in its
+     * constructor, so a router obtained after construction would never be seen —
+     * the service must not be built until we know whether one exists.
+     *
+     * Router resolution order:
+     *   1. `kernel.llmRouter` — set during `QEKernelImpl.initialize()`.
+     *   2. the shared MCP singleton — same instance the kernel registers via
+     *      `setSharedLLMRouter()`, and the lazily-built one in standalone MCP mode.
+     *
+     * Both may be absent (no provider keys, no llm-config.json, or the
+     * `AQE_LLM_ROUTER_DISABLED` kill-switch). That is not an error: the generator
+     * falls back to deterministic template generation exactly as before.
+     */
+    getTestGenerator(useLLM?: boolean): Promise<TestGeneratorService>;
+    /**
+     * Best-effort LLM router lookup. Never throws — a missing or failed router
+     * degrades to `undefined` so deterministic generation still runs.
+     *
+     * The import is dynamic: `mcp/tools/base` transitively reaches back into
+     * coordination, so a static import here would be circular.
+     */
+    private resolveLLMRouter;
     getKnowledgeGraph(): KnowledgeGraphService;
     getQualityAnalyzer(): QualityAnalyzerService;
     private registerHandlers;

package/dist/coordination/task-executor.js

@@ -40,9 +40,9 @@ function resolveSecurityScannerService(memory) {
     const factory = DomainServiceRegistry.resolve(ServiceKeys.SecurityScannerService);
     return factory(memory);
 }
-function resolveTestGeneratorService(memory) {
+function resolveTestGeneratorService(memory, llmRouter) {
     const factory = DomainServiceRegistry.resolve(ServiceKeys.createTestGeneratorService);
-    return factory(memory);
+    return factory(memory, llmRouter);
 }
 function resolveKnowledgeGraphService(memory) {
     const factory = DomainServiceRegistry.resolve(ServiceKeys.KnowledgeGraphService);
@@ -119,6 +119,8 @@ export class DomainTaskExecutor {
     coverageAnalyzer = null;
     securityScanner = null;
     testGenerator = null;
+    /** #567: router-free instance for callers that opt out of LLM enhancement. */
+    deterministicTestGenerator = null;
     knowledgeGraph = null;
     qualityAnalyzer = null;
     // ADR-051: Lazy-initialized Agent Booster and Task Router (instance-level)
@@ -175,12 +177,58 @@ export class DomainTaskExecutor {
         }
         return this.securityScanner;
     }
-    getTestGenerator() {
+    /**
+     * Issue #567: async so the ADR-043 LLM router can be resolved before the
+     * generator is constructed. `TestGeneratorService` captures `llmRouter` in its
+     * constructor, so a router obtained after construction would never be seen —
+     * the service must not be built until we know whether one exists.
+     *
+     * Router resolution order:
+     *   1. `kernel.llmRouter` — set during `QEKernelImpl.initialize()`.
+     *   2. the shared MCP singleton — same instance the kernel registers via
+     *      `setSharedLLMRouter()`, and the lazily-built one in standalone MCP mode.
+     *
+     * Both may be absent (no provider keys, no llm-config.json, or the
+     * `AQE_LLM_ROUTER_DISABLED` kill-switch). That is not an error: the generator
+     * falls back to deterministic template generation exactly as before.
+     */
+    async getTestGenerator(useLLM = true) {
+        // Two cached instances, because the router is a constructor-captured
+        // dependency: callers passing `aiEnhancement: false` must get a service
+        // that was never handed a router, not one that merely declines to use it.
+        if (!useLLM) {
+            if (!this.deterministicTestGenerator) {
+                this.deterministicTestGenerator = resolveTestGeneratorService(this.kernel.memory);
+            }
+            return this.deterministicTestGenerator;
+        }
         if (!this.testGenerator) {
-            this.testGenerator = resolveTestGeneratorService(this.kernel.memory);
+            this.testGenerator = resolveTestGeneratorService(this.kernel.memory, await this.resolveLLMRouter());
         }
         return this.testGenerator;
     }
+    /**
+     * Best-effort LLM router lookup. Never throws — a missing or failed router
+     * degrades to `undefined` so deterministic generation still runs.
+     *
+     * The import is dynamic: `mcp/tools/base` transitively reaches back into
+     * coordination, so a static import here would be circular.
+     */
+    async resolveLLMRouter() {
+        if (this.kernel.llmRouter) {
+            return this.kernel.llmRouter;
+        }
+        try {
+            const { getSharedLLMRouter } = await import('../mcp/tools/base.js');
+            return (await getSharedLLMRouter()) ?? undefined;
+        }
+        catch (error) {
+            logger.debug('LLM router unavailable; using deterministic generation', {
+                error: toErrorMessage(error),
+            });
+            return undefined;
+        }
+    }
     getKnowledgeGraph() {
         if (!this.knowledgeGraph) {
             this.knowledgeGraph = resolveKnowledgeGraphService(this.kernel.memory);

package/dist/domains/code-intelligence/services/knowledge-graph.js

@@ -384,6 +384,10 @@ Be precise and only report high-confidence findings.`,
                 model: modelId,
                 maxTokens: this.config.llmMaxTokens ?? 2048,
                 temperature: 0.2, // Low temperature for consistent analysis
+                // Issue #568: per-agent routing rules (including the user's on-disk
+                // `agentOverrides`) match on `agentType`; without it an override for
+                // this agent never fires.
+                agentType: 'qe-code-intelligence',
             });
             if (response.content) {
                 return this.parseLLMRelationshipResponse(response.content);
@@ -472,6 +476,10 @@ Return JSON: { "rankedIds": ["id1", "id2", ...], "insights": ["insight1", "insig
                 model: modelId,
                 maxTokens: 1024,
                 temperature: 0.3,
+                // Issue #568: per-agent routing rules (including the user's on-disk
+                // `agentOverrides`) match on `agentType`; without it an override for
+                // this agent never fires.
+                agentType: 'qe-code-intelligence',
             });
             if (response.content) {
                 try {

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

@@ -245,6 +245,10 @@ Provide thoughtful, specific analysis based on the coverage data. Do not include
                 model: modelId,
                 maxTokens: this.config.llmMaxTokens ?? 2048,
                 temperature: 0.3, // Low temperature for consistent analysis
+                // Issue #568: per-agent routing rules (including the user's on-disk
+                // `agentOverrides`) match on `agentType`; without it an override for
+                // this agent never fires.
+                agentType: 'qe-coverage-specialist',
             });
             if (response.content && response.content.length > 0) {
                 return this.parseInsightsResponse(response.content);

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

@@ -116,6 +116,10 @@ Return JSON in this exact format:
                 model: modelId,
                 maxTokens: this.config.llmMaxTokens ?? 2048,
                 temperature: 0.3, // Low temperature for consistent analysis
+                // Issue #568: per-agent routing rules (including the user's on-disk
+                // `agentOverrides`) match on `agentType`; without it an override for
+                // this agent never fires.
+                agentType: 'qe-gap-detector',
             });
             if (response.content) {
                 const analysis = this.parseGapAnalysisResponse(response.content, gaps);

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

@@ -160,6 +160,10 @@ Be specific and actionable. Focus on concrete issues, not generic advice.`,
                 model: modelId,
                 maxTokens: this.config.llmMaxTokens ?? 2048,
                 temperature: 0.3, // Low temperature for consistent analysis
+                // Issue #568: per-agent routing rules (including the user's on-disk
+                // `agentOverrides`) match on `agentType`; without it an override for
+                // this agent never fires.
+                agentType: 'qe-defect-predictor',
             });
             if (response.content && response.content.length > 0) {
                 return this.parseLLMDefectAnalysis(response.content);

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

@@ -243,6 +243,10 @@ Return your analysis as JSON with this structure:
                 model: modelId,
                 maxTokens: this.config.llmMaxTokens ?? 2048,
                 temperature: 0.3, // Low temperature for consistent analysis
+                // Issue #568: per-agent routing rules (including the user's on-disk
+                // `agentOverrides`) match on `agentType`; without it an override for
+                // this agent never fires.
+                agentType: 'qe-root-cause-analyzer',
             });
             if (response.content && response.content.length > 0) {
                 return this.parseLLMAnalysisResponse(response.content);

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

@@ -134,6 +134,10 @@ Provide deployment advice specific to these metrics.`,
                 model: modelId,
                 maxTokens: 2048,
                 temperature: 0.3, // Low temperature for consistent advice
+                // Issue #568: per-agent routing rules (including the user's on-disk
+                // `agentOverrides`) match on `agentType`; without it an override for
+                // this agent never fires.
+                agentType: 'qe-deployment-advisor',
             });
             if (response.content) {
                 try {

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

@@ -131,6 +131,10 @@ Focus on:
                 model: modelId,
                 maxTokens: this.config.llmMaxTokens ?? 2048,
                 temperature: 0.3, // Low temperature for consistent analysis
+                // Issue #568: per-agent routing rules (including the user's on-disk
+                // `agentOverrides`) match on `agentType`; without it an override for
+                // this agent never fires.
+                agentType: 'qe-quality-gate',
             });
             if (response.content && response.content.length > 0) {
                 return this.parseQualityInsightsResponse(response.content);

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

@@ -538,6 +538,10 @@ Provide detailed remediation advice specific to this code.`,
                 model: modelId,
                 maxTokens: 1500,
                 temperature: 0.2,
+                // Issue #568: per-agent routing rules (including the user's on-disk
+                // `agentOverrides`) match on `agentType`; without it an override for
+                // this agent never fires.
+                agentType: 'qe-security-scanner',
             });
             if (response.content) {
                 try {

package/dist/domains/test-generation/index.js

@@ -29,6 +29,21 @@ export { TestGenerationCoherenceGate, createTestGenerationCoherenceGate, Coheren
 // so coordination/ can resolve them without importing from domains/.
 // ============================================================================
 import { DomainServiceRegistry, ServiceKeys } from '../../shared/domain-service-registry';
-import { createTestGeneratorService as _createTestGeneratorService } from './services/test-generator';
-DomainServiceRegistry.register(ServiceKeys.createTestGeneratorService, (memory) => _createTestGeneratorService(memory));
+import { createTestGeneratorService as _createTestGeneratorService, createTestGeneratorServiceWithDependencies as _createTestGeneratorServiceWithDependencies, } from './services/test-generator';
+/**
+ * Issue #567: the factory MUST accept an optional `llmRouter`.
+ *
+ * The flat MCP tool `test_generate_enhanced` and the CLI both reach the
+ * generator through the task-executor, which resolves it from this registry.
+ * When the factory dropped `llmRouter` on the floor,
+ * `TestGeneratorService.isLLMEnhancementAvailable()` was permanently false and
+ * the ADR-051 LLM branch was unreachable on those paths — the tool silently
+ * returned generic template scaffolding no matter how the provider was
+ * configured. Keep the second parameter optional so the router stays a
+ * degradation-friendly dependency: no router => deterministic templates, same
+ * as before.
+ */
+DomainServiceRegistry.register(ServiceKeys.createTestGeneratorService, (memory, llmRouter) => llmRouter
+    ? _createTestGeneratorServiceWithDependencies({ memory, llmRouter })
+    : _createTestGeneratorService(memory));
 //# sourceMappingURL=index.js.map

package/dist/domains/test-generation/services/test-generator.js

@@ -12,6 +12,7 @@ import * as fs from 'fs';
 import * as path from 'path';
 import ts from 'typescript';
 import { ok, err } from '../../../shared/types';
+import { toErrorMessage } from '../../../shared/error-utils.js';
 import { TestGeneratorFactory } from '../factories/test-generator-factory';
 import { TDDGeneratorService } from './tdd-generator';
 import { PropertyTestGeneratorService } from './property-test-generator';
@@ -128,7 +129,7 @@ export class TestGeneratorService {
      */
     async enhanceTestWithLLM(testCode, sourceCode, analysis, context) {
         if (!this.llmRouter)
-            return testCode;
+            return { code: testCode, enhanced: false };
         try {
             let prompt = this.buildTestEnhancementPrompt(testCode, sourceCode, analysis);
             // Prepend historical edge case patterns if injector is available (loki-mode Item 5)
@@ -189,6 +190,10 @@ Return ONLY the enhanced test code, no explanations.`,
                 model: modelId,
                 maxTokens: this.config.llmMaxTokens,
                 temperature: 0.3, // Low temperature for consistent code generation
+                // Issue #568: the router's per-agent routing rules (including the user's
+                // on-disk `agentOverrides`) match on `agentType`. Without it, an override
+                // for `qe-test-architect` silently never fires on this call.
+                agentType: 'qe-test-architect',
             });
             if (response.content && response.content.length > 0) {
                 // Extract code from response, handling potential markdown fences
@@ -197,13 +202,22 @@ Return ONLY the enhanced test code, no explanations.`,
                 if (codeMatch) {
                     enhancedCode = codeMatch[1].trim();
                 }
-                return enhancedCode || testCode;
+                // #567: only claim enhancement when the model actually returned code.
+                // An empty/unusable response falls back to the template, and the caller
+                // must be told that is what happened.
+                return enhancedCode
+                    ? { code: enhancedCode, enhanced: true }
+                    : { code: testCode, enhanced: false };
             }
-            return testCode;
+            return { code: testCode, enhanced: false };
         }
         catch (error) {
-            logger.warn('LLM enhancement failed, using original:');
-            return testCode;
+            // #567: a configured-but-broken provider (401, timeout, wrong model) used
+            // to fall back to the template while the result still reported
+            // `llmEnhanced: true` — the exact "silently receives non-LLM scaffolding"
+            // failure this issue is about, just one layer deeper. Report the truth.
+            logger.warn(`LLM enhancement failed, using deterministic template: ${toErrorMessage(error)}`);
+            return { code: testCode, enhanced: false };
         }
     }
     /**
@@ -261,6 +275,8 @@ Return a JSON array of test suggestions, each with: { "name": "test name", "desc
                 model: modelId,
                 maxTokens: 1024,
                 temperature: 0.5,
+                // Issue #568: see above — required for per-agent routing to apply.
+                agentType: 'qe-test-architect',
             });
             if (response.content) {
                 try {
@@ -443,9 +459,15 @@ Return a JSON array of test suggestions, each with: { "name": "test name", "desc
             similarCode,
         };
         let testCode = generator.generateTests(context);
-        // ADR-051: Enhance with LLM if enabled and available
+        // ADR-051: Enhance with LLM if enabled and available.
+        // #567: `llmEnhanced` reflects whether the LLM actually produced the code,
+        // not merely whether a router was configured — a broken provider must not
+        // report AI-enhanced output it did not produce.
+        let llmEnhanced = false;
         if (this.isLLMEnhancementAvailable() && sourceContent) {
-            testCode = await this.enhanceTestWithLLM(testCode, sourceContent, codeAnalysis, context);
+            const result = await this.enhanceTestWithLLM(testCode, sourceContent, codeAnalysis, context);
+            testCode = result.code;
+            llmEnhanced = result.enhanced;
         }
         const test = {
             id: uuidv4(),
@@ -458,8 +480,8 @@ Return a JSON array of test suggestions, each with: { "name": "test name", "desc
             // ADR-078: Include detected language and framework
             language: effectiveLanguage,
             framework: framework,
-            // ADR-051: Mark if LLM-enhanced
-            llmEnhanced: this.isLLMEnhancementAvailable(),
+            // ADR-051 / #567: true only when the LLM actually produced this code.
+            llmEnhanced,
         };
         // ADR-077: Compilation validation if requested
         if (originalRequest?.compileValidation && effectiveLanguage) {

package/dist/init/codex-installer.d.ts

@@ -18,9 +18,13 @@ export interface CodexInstallResult {
     success: boolean;
     mcpConfigured: boolean;
     agentsMdInstalled: boolean;
+    hooksConfigured: boolean;
+    skillsInstalled: number;
     errors: string[];
     configPath: string;
     agentsMdPath: string;
+    hooksPath: string;
+    skillsPath: string;
 }
 export declare class CodexInstaller {
     private projectRoot;
@@ -29,6 +33,15 @@ export declare class CodexInstaller {
     private generator;
     constructor(options: CodexInstallerOptions);
     install(): Promise<CodexInstallResult>;
+    /**
+     * Install Codex-native lifecycle adapters and merge AQE/ruflo hook groups
+     * without removing user-defined hooks.
+     */
+    private installCodexHooks;
+    /** Install the curated repo-scoped AQE skills Codex discovers automatically. */
+    private installCodexSkills;
+    private copyDirectory;
+    private resolvePackageRoot;
     /**
      * Merge AQE MCP server config into existing TOML.
      * Simple approach: if [mcp_servers.agentic-qe] already exists, skip.