agentic-qe 3.8.4 → 3.8.5

package/dist/domains/quality-assessment/coordinator-rl-integration.js

@@ -4,6 +4,7 @@
  *
  * Contains: Actor-Critic RL, SONA pattern learning, Flash Attention initialization and methods
  */
+import { LoggerFactory } from '../../logging/index.js';
 import { toErrorMessage } from '../../shared/error-utils.js';
 import { ActorCriticAlgorithm } from '../../integrations/rl-suite/algorithms/actor-critic';
 import { createPersistentSONAEngine } from '../../integrations/ruvector/sona-persistence.js';
@@ -11,6 +12,7 @@ import { createQEFlashAttention, } from '../../integrations/ruvector/wrappers';
 /**
  * Initialize Actor-Critic RL for quality gate threshold tuning
  */
+const logger = LoggerFactory.create('quality-assessment/rl-integration');
 export async function initializeActorCritic() {
     try {
         const actorCritic = new ActorCriticAlgorithm({
@@ -44,12 +46,12 @@ export async function initializeQESONA() {
             minConfidence: 0.5,
             maxPatterns: 5000,
         });
-        console.log('[quality-assessment] PersistentSONAEngine initialized successfully');
+        logger.info('PersistentSONAEngine initialized successfully');
         return qesona;
     }
     catch (error) {
-        console.error('[quality-assessment] Failed to initialize PersistentSONAEngine:', error);
-        console.warn('[quality-assessment] Continuing without SONA pattern persistence');
+        logger.error('Failed to initialize PersistentSONAEngine:', error instanceof Error ? error : undefined);
+        logger.warn('Continuing without SONA pattern persistence');
         return undefined;
     }
 }
@@ -103,7 +105,7 @@ export async function tuneThresholdsWithRL(actorCritic, metrics) {
         };
     }
     catch (error) {
-        console.error('RL threshold tuning failed:', error);
+        logger.error('RL threshold tuning failed:', error instanceof Error ? error : undefined);
         return null;
     }
 }
@@ -168,7 +170,7 @@ export async function trainActorCritic(actorCritic, request, result) {
         await actorCritic.train(experience);
     }
     catch (error) {
-        console.error('Actor-Critic training failed:', error);
+        logger.error('Actor-Critic training failed:', error instanceof Error ? error : undefined);
     }
 }
 /**
@@ -203,7 +205,7 @@ export async function storeQualityPattern(qesona, request, result, domainName) {
         });
     }
     catch (error) {
-        console.error('Failed to store quality pattern in SONA:', error);
+        logger.error('Failed to store quality pattern in SONA:', error instanceof Error ? error : undefined);
     }
 }
 /**
@@ -236,7 +238,7 @@ export async function storeQualityAnalysisPattern(qesona, request, report, domai
         });
     }
     catch (error) {
-        console.error('Failed to store quality analysis pattern in SONA:', error);
+        logger.error('Failed to store quality analysis pattern in SONA:', error instanceof Error ? error : undefined);
     }
 }
 /**
@@ -276,7 +278,7 @@ export async function enhanceWithSimilarityPatterns(report, flashAttention, qeso
         };
     }
     catch (error) {
-        console.error('Failed to enhance with similarity patterns:', error);
+        logger.error('Failed to enhance with similarity patterns:', error instanceof Error ? error : undefined);
         return null;
     }
 }

package/dist/domains/quality-assessment/coordinator.d.ts

@@ -63,21 +63,6 @@ export interface CoordinatorConfig extends BaseDomainCoordinatorConfig {
     borderlineMargin: number;
 }
 type QualityWorkflowType = 'gate-evaluation' | 'quality-analysis' | 'deployment-advice' | 'complexity-analysis';
-/**
- * Quality Assessment Coordinator
- * Orchestrates quality assessment workflows and coordinates with agents
- *
- * Integrations (per ADR-040):
- * - ActorCritic RL: Quality gate threshold tuning
- * - QESONA: Quality pattern learning
- * - QEFlashAttention: Similarity computations for quality reports
- *
- * V3 Integrations (ADR-047, CONSENSUS-MIXIN-001):
- * - MinCutAwareDomainMixin: Topology-aware routing and health monitoring
- * - ConsensusEnabledMixin: Multi-model consensus for high-stakes quality decisions
- *
- * CQ-002: Extends BaseDomainCoordinator
- */
 export declare class QualityAssessmentCoordinator extends BaseDomainCoordinator<CoordinatorConfig, QualityWorkflowType> implements IQualityAssessmentCoordinator {
     private readonly memory;
     private readonly agentCoordinator;

package/dist/domains/quality-assessment/coordinator.js

@@ -13,6 +13,7 @@
  *
  * CQ-002: Extends BaseDomainCoordinator for lifecycle deduplication
  */
+import { LoggerFactory } from '../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import { ok, err } from '../../shared/types';
 import { toError, toErrorMessage } from '../../shared/error-utils.js';
@@ -65,6 +66,7 @@ const DEFAULT_CONFIG = {
  *
  * CQ-002: Extends BaseDomainCoordinator
  */
+const logger = LoggerFactory.create('quality-assessment');
 export class QualityAssessmentCoordinator extends BaseDomainCoordinator {
     memory;
     agentCoordinator;
@@ -178,14 +180,14 @@ export class QualityAssessmentCoordinator extends BaseDomainCoordinator {
             this.startWorkflow(workflowId, 'gate-evaluation');
             // Self-healing: Check if operations should be paused due to critical topology
             if (this.minCutMixin.shouldPauseOperations()) {
-                console.warn('[quality-assessment] Quality gate evaluation paused: topology is in critical state');
+                logger.warn('Quality gate evaluation paused: topology is in critical state');
                 this.failWorkflow(workflowId, 'Topology is in critical state');
                 return err(new Error('Quality gate evaluation paused: topology is in critical state'));
             }
             // V3 Integration: Check topology health before proceeding (ADR-047)
             // Apply stricter thresholds when topology is degraded
             if (!this.isTopologyHealthy()) {
-                console.warn('[quality-assessment] Topology degraded - applying stricter thresholds for quality gate');
+                logger.warn('Topology degraded - applying stricter thresholds for quality gate');
                 // Continue evaluation but with heightened caution - quality gates are critical
             }
             // Spawn quality gate agent if available
@@ -285,14 +287,14 @@ export class QualityAssessmentCoordinator extends BaseDomainCoordinator {
             this.startWorkflow(workflowId, 'quality-analysis');
             // Self-healing: Check if operations should be paused due to critical topology
             if (this.minCutMixin.shouldPauseOperations()) {
-                console.warn('[quality-assessment] Quality analysis paused: topology is in critical state');
+                logger.warn('Quality analysis paused: topology is in critical state');
                 this.failWorkflow(workflowId, 'Topology is in critical state');
                 return err(new Error('Quality analysis paused: topology is in critical state'));
             }
             // V3 Integration: Check topology health and adjust behavior (ADR-047)
             const topologyHealthy = this.isTopologyHealthy();
             if (!topologyHealthy) {
-                console.warn('[quality-assessment] Topology degraded during quality analysis');
+                logger.warn('Topology degraded during quality analysis');
                 // Could adjust analysis depth or timeouts in degraded state
                 // For now, we proceed but could be extended to reduce analysis scope
             }
@@ -348,13 +350,13 @@ export class QualityAssessmentCoordinator extends BaseDomainCoordinator {
             this.startWorkflow(workflowId, 'deployment-advice');
             // Self-healing: Check if operations should be paused due to critical topology
             if (this.minCutMixin.shouldPauseOperations()) {
-                console.warn('[quality-assessment] Deployment advice paused: topology is in critical state');
+                logger.warn('Deployment advice paused: topology is in critical state');
                 this.failWorkflow(workflowId, 'Topology is in critical state');
                 return err(new Error('Deployment advice paused: topology is in critical state'));
             }
             // V3 Integration: Check topology health (ADR-047)
             if (!this.isTopologyHealthy()) {
-                console.warn('[quality-assessment] Topology degraded during deployment advice generation');
+                logger.warn('Topology degraded during deployment advice generation');
             }
             // Spawn deployment advisor agent
             const agentResult = await this.spawnDeploymentAdvisorAgent(workflowId, request);
@@ -414,13 +416,13 @@ export class QualityAssessmentCoordinator extends BaseDomainCoordinator {
             this.startWorkflow(workflowId, 'complexity-analysis');
             // Self-healing: Check if operations should be paused due to critical topology
             if (this.minCutMixin.shouldPauseOperations()) {
-                console.warn('[quality-assessment] Complexity analysis paused: topology is in critical state');
+                logger.warn('Complexity analysis paused: topology is in critical state');
                 this.failWorkflow(workflowId, 'Topology is in critical state');
                 return err(new Error('Complexity analysis paused: topology is in critical state'));
             }
             // V3 Integration: Check topology health (ADR-047)
             if (!this.isTopologyHealthy()) {
-                console.warn('[quality-assessment] Topology degraded during complexity analysis');
+                logger.warn('Topology degraded during complexity analysis');
             }
             // Spawn complexity analyzer agent
             const agentResult = await this.spawnComplexityAnalyzerAgent(workflowId, request);
@@ -652,7 +654,7 @@ export class QualityAssessmentCoordinator extends BaseDomainCoordinator {
         if (relevantInsights.length === 0) {
             return;
         }
-        console.log(`[${this.domain}] Received ${relevantInsights.length} relevant dream insights from cycle ${cycleId}`);
+        logger.info(`[${this.domain}] Received ${relevantInsights.length} relevant dream insights from cycle ${cycleId}`);
         // Apply high-confidence actionable insights
         for (const insight of relevantInsights) {
             if (insight.confidenceScore > 0.7 && insight.actionable) {
@@ -677,7 +679,7 @@ export class QualityAssessmentCoordinator extends BaseDomainCoordinator {
      * @param cycleId - The dream cycle ID for tracking
      */
     async applyDreamInsight(insight, cycleId) {
-        console.log(`[${this.domain}] Applying dream insight: ${insight.description.slice(0, 100)}...`);
+        logger.info(`[${this.domain}] Applying dream insight: ${insight.description.slice(0, 100)}...`);
         // Store as a learned pattern via SONA if available
         if (this.qesona) {
             try {
@@ -708,10 +710,10 @@ export class QualityAssessmentCoordinator extends BaseDomainCoordinator {
                     suggestedAction: insight.suggestedAction,
                     dreamDerived: true,
                 });
-                console.log(`[${this.domain}] Created SONA pattern from dream insight ${insight.id}`);
+                logger.info(`[${this.domain}] Created SONA pattern from dream insight ${insight.id}`);
             }
             catch (error) {
-                console.error(`[${this.domain}] Failed to store dream insight pattern:`, error);
+                logger.error(`[${this.domain}] Failed to store dream insight pattern`, error instanceof Error ? error : undefined);
             }
         }
         // Store insight in memory for downstream usage

package/dist/domains/quality-assessment/services/deployment-advisor.d.ts

@@ -61,16 +61,6 @@ interface DecisionThresholds {
     warning: number;
     blocked: number;
 }
-/**
- * Deployment Advisor Service Implementation
- * Uses ML-based risk scoring to provide deployment recommendations
- *
- * ADR-051: Added LLM integration for AI-powered deployment advice including:
- * - Go/no-go recommendations with reasoning
- * - Risk mitigation strategies
- * - Rollback plan suggestions
- * - Post-deployment monitoring recommendations
- */
 export declare class DeploymentAdvisorService implements IDeploymentAdvisorService {
     private config;
     private readonly memory;

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

@@ -4,6 +4,7 @@
  *
  * ADR-051: Added LLM integration for AI-powered deployment advice
  */
+import { LoggerFactory } from '../../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import { ok, err } from '../../../shared/types';
 import { toError } from '../../../shared/error-utils.js';
@@ -38,6 +39,7 @@ const DEFAULT_CONFIG = {
  * - Rollback plan suggestions
  * - Post-deployment monitoring recommendations
  */
+const logger = LoggerFactory.create('quality-assessment/deployment-advisor');
 export class DeploymentAdvisorService {
     config;
     memory;
@@ -143,12 +145,12 @@ Provide deployment advice specific to these metrics.`,
                 }
                 catch {
                     // JSON parse failed - return empty enhancement
-                    console.warn('[DeploymentAdvisor] Failed to parse LLM response JSON');
+                    logger.warn('Failed to parse LLM response JSON');
                 }
             }
         }
         catch (error) {
-            console.warn('[DeploymentAdvisor] LLM advice generation failed:', error);
+            logger.warn('LLM advice generation failed:');
         }
         return {};
     }

package/dist/domains/quality-assessment/services/quality-analyzer.d.ts

@@ -50,12 +50,6 @@ export interface QualityAnalyzerDependencies {
     /** ADR-051: Optional LLM router for AI-enhanced quality insights */
     llmRouter?: HybridRouter;
 }
-/**
- * Quality Analyzer Service Implementation
- * Analyzes code quality metrics and provides actionable insights
- *
- * ADR-051: Added LLM enhancement for AI-powered quality insights
- */
 export declare class QualityAnalyzerService implements IQualityAnalyzerService {
     private readonly config;
     private readonly metricsAnalyzer;

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

@@ -2,6 +2,7 @@
  * Agentic QE v3 - Quality Analyzer Service
  * Comprehensive quality metrics analysis with trends and recommendations
  */
+import { LoggerFactory } from '../../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import { ok, err } from '../../../shared/types';
 import { getCodeMetricsAnalyzer } from '../../../shared/metrics';
@@ -35,6 +36,7 @@ const RATING_THRESHOLDS = {
  *
  * ADR-051: Added LLM enhancement for AI-powered quality insights
  */
+const logger = LoggerFactory.create('quality-assessment/quality-analyzer');
 export class QualityAnalyzerService {
     config;
     metricsAnalyzer;
@@ -136,7 +138,7 @@ Focus on:
             return null;
         }
         catch (error) {
-            console.warn('[QualityAnalyzer] LLM insights generation failed:', error);
+            logger.warn('LLM insights generation failed:');
             return null;
         }
     }
@@ -198,7 +200,7 @@ Focus on:
             return null;
         }
         catch (error) {
-            console.warn('[QualityAnalyzer] Failed to parse LLM insights response:', error);
+            logger.warn('Failed to parse LLM insights response:');
             return null;
         }
     }

package/dist/domains/requirements-validation/coordinator.d.ts

@@ -37,9 +37,6 @@ export interface WorkflowStatus {
     error?: string;
 }
 type RequirementsWorkflowType = 'analyze' | 'generate-artifacts' | 'validate-sprint';
-/**
- * CQ-002: Extends BaseDomainCoordinator
- */
 export declare class RequirementsValidationCoordinator extends BaseDomainCoordinator<CoordinatorConfig, RequirementsWorkflowType> implements IRequirementsValidationCoordinator {
     private readonly memory;
     private readonly agentCoordinator;

package/dist/domains/requirements-validation/coordinator.js

@@ -7,6 +7,7 @@
  * - QESONA: Learns and adapts requirement patterns for improved validation
  */
 import { v4 as uuidv4 } from 'uuid';
+import { LoggerFactory } from '../../logging/index.js';
 import { ok, err } from '../../shared/types/index.js';
 import { toError } from '../../shared/error-utils.js';
 import { createEvent } from '../../shared/events/domain-events.js';
@@ -90,6 +91,7 @@ class InMemoryRequirementRepository {
 /**
  * CQ-002: Extends BaseDomainCoordinator
  */
+const logger = LoggerFactory.create('requirements-validation');
 export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
     memory;
     agentCoordinator;
@@ -158,10 +160,10 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
                         maxPatterns: 5000,
                         minConfidence: 0.6,
                     });
-                    console.log('[RequirementsValidation] PersistentSONAEngine initialized for pattern learning');
+                    logger.info('PersistentSONAEngine initialized for pattern learning');
                 }
                 catch (error) {
-                    console.error('[RequirementsValidation] Failed to initialize PersistentSONAEngine:', error);
+                    logger.error('Failed to initialize PersistentSONAEngine', error instanceof Error ? error : undefined);
                     // Continue without SONA - it's optional
                     this.sonaEngine = undefined;
                 }
@@ -169,7 +171,7 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
             this.rlInitialized = true;
         }
         catch (error) {
-            console.error('Failed to initialize RL integrations:', error);
+            logger.error('Failed to initialize RL integrations', error instanceof Error ? error : undefined);
             throw error;
         }
     }
@@ -186,7 +188,7 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
                 this.sonaEngine = undefined;
             }
             catch (error) {
-                console.error('[RequirementsValidation] Error closing SONA engine:', error);
+                logger.error('Error closing SONA engine', error instanceof Error ? error : undefined);
             }
         }
     }
@@ -208,7 +210,7 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
             this.startWorkflow(workflowId, 'analyze');
             // V3: Check topology health before proceeding (ADR-047)
             if (this.config.enableMinCutAwareness && !this.isTopologyHealthy()) {
-                console.warn('[RequirementsValidation] Topology degraded, using conservative strategy');
+                logger.warn('Topology degraded, using conservative strategy');
             }
             // V3: Pause operations if topology is critical and configured to pause
             if (this.minCutMixin.shouldPauseOperations()) {
@@ -223,7 +225,7 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
             if (this.config.enableSONA && this.sonaEngine) {
                 const pattern = await this.adaptRequirementPattern(requirement);
                 if (pattern.success && pattern.pattern) {
-                    console.log(`[SONA] Adapted pattern with ${pattern.similarity.toFixed(3)} similarity`);
+                    logger.info(`[SONA] Adapted pattern with ${pattern.similarity.toFixed(3)} similarity`);
                 }
             }
             // Spawn analysis agent
@@ -313,7 +315,7 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
                 const prediction = await this.optimizeScenarioGeneration(requirement);
                 if (prediction.success && prediction.value) {
                     optimizedScenarioCount = this.extractScenarioCount(prediction.value);
-                    console.log(`[PPO] Optimized scenario count to ${optimizedScenarioCount}`);
+                    logger.info(`[PPO] Optimized scenario count to ${optimizedScenarioCount}`);
                 }
             }
             // Generate BDD scenarios with optimized count
@@ -508,7 +510,7 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
             return scored.map((s) => s.scenario);
         }
         catch (error) {
-            console.error('Failed to optimize scenario ordering:', error);
+            logger.error('Failed to optimize scenario ordering', error instanceof Error ? error : undefined);
             return scenarios;
         }
     }
@@ -539,10 +541,10 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
                 done: true,
             };
             await this.ppoAlgorithm.train(experience);
-            console.log(`[PPO] Trained with reward: ${reward.toFixed(3)}`);
+            logger.info(`[PPO] Trained with reward: ${reward.toFixed(3)}`);
         }
         catch (error) {
-            console.error('Failed to train PPO:', error);
+            logger.error('Failed to train PPO', error instanceof Error ? error : undefined);
         }
     }
     /**
@@ -615,7 +617,7 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
             };
         }
         catch (error) {
-            console.error('Failed to adapt requirement pattern:', error);
+            logger.error('Failed to adapt requirement pattern', error instanceof Error ? error : undefined);
             return { success: false, pattern: null, similarity: 0 };
         }
     }
@@ -646,10 +648,10 @@ export class RequirementsValidationCoordinator extends BaseDomainCoordinator {
                 validationErrorCount: analysis.validationErrors.length,
                 ambiguityScore: analysis.ambiguityReport.overallScore,
             });
-            console.log(`[SONA] Stored pattern ${pattern.id} for requirement ${requirement.id}`);
+            logger.info(`[SONA] Stored pattern ${pattern.id} for requirement ${requirement.id}`);
         }
         catch (error) {
-            console.error('Failed to store requirement pattern:', error);
+            logger.error('Failed to store requirement pattern', error instanceof Error ? error : undefined);
         }
     }
     // ============================================================================

package/dist/domains/requirements-validation/services/product-factors-assessment/code-intelligence/codebase-analyzer.d.ts

@@ -28,11 +28,6 @@ export interface CodebaseAnalyzerConfig {
     /** Exclude patterns */
     excludePatterns?: string[];
 }
-/**
- * Codebase Analyzer
- *
- * Provides automated codebase analysis for SFDIPOT assessment.
- */
 export declare class CodebaseAnalyzer {
     private config;
     constructor(config: CodebaseAnalyzerConfig);

package/dist/domains/requirements-validation/services/product-factors-assessment/code-intelligence/codebase-analyzer.js

@@ -12,6 +12,7 @@
  */
 import * as path from 'path';
 import * as fs from 'fs/promises';
+import { LoggerFactory } from '../../../../../logging/index.js';
 import { HTSMCategory, Priority, AutomationFitness, generateTestId, } from '../types/index.js';
 import { safeJsonParse } from '@shared/safe-json.js';
 const DEFAULT_CONFIG = {
@@ -75,6 +76,7 @@ const EXTERNAL_SYSTEM_PATTERNS = {
  *
  * Provides automated codebase analysis for SFDIPOT assessment.
  */
+const logger = LoggerFactory.create('requirements-validation/codebase-analyzer');
 export class CodebaseAnalyzer {
     config;
     constructor(config) {
@@ -101,13 +103,13 @@ export class CodebaseAnalyzer {
         try {
             // Phase 1: Detect external systems from package.json
             if (this.config.detectExternalSystems) {
-                console.log('[CodebaseAnalyzer] Phase 1: Detecting external systems...');
+                logger.info('Phase 1: Detecting external systems...');
                 result.externalSystems = await this.detectExternalSystems();
                 result.metadata.externalSystemsDetected = result.externalSystems.length;
             }
             // Phase 2: Analyze component boundaries
             if (this.config.analyzeComponents) {
-                console.log('[CodebaseAnalyzer] Phase 2: Analyzing components...');
+                logger.info('Phase 2: Analyzing components...');
                 const componentResult = await this.analyzeComponents();
                 result.components = componentResult.components;
                 result.relationships = componentResult.relationships;
@@ -115,19 +117,19 @@ export class CodebaseAnalyzer {
             }
             // Phase 3: Analyze module coupling
             if (this.config.analyzeCoupling && result.components.length > 0) {
-                console.log('[CodebaseAnalyzer] Phase 3: Analyzing coupling...');
+                logger.info('Phase 3: Analyzing coupling...');
                 result.couplingAnalysis = this.analyzeCoupling(result.components, result.relationships);
             }
             // Generate C4 diagrams
             if (this.config.generateC4Diagrams) {
-                console.log('[CodebaseAnalyzer] Generating C4 diagrams...');
+                logger.info('Generating C4 diagrams...');
                 result.c4Diagrams = await this.generateC4Diagrams(result);
             }
             result.metadata.analysisTimeMs = Date.now() - startTime;
-            console.log(`[CodebaseAnalyzer] Analysis complete in ${result.metadata.analysisTimeMs}ms`);
+            logger.info(`Analysis complete in ${result.metadata.analysisTimeMs}ms`);
         }
         catch (error) {
-            console.error('[CodebaseAnalyzer] Analysis failed:', error);
+            logger.error('Analysis failed', error instanceof Error ? error : undefined);
             result.metadata.analysisTimeMs = Date.now() - startTime;
         }
         return result;
@@ -147,7 +149,7 @@ export class CodebaseAnalyzer {
                 await fs.access(packageJsonPath);
             }
             catch {
-                console.log('[CodebaseAnalyzer] No package.json found, skipping external system detection');
+                logger.info('No package.json found, skipping external system detection');
                 return systems;
             }
             const content = await fs.readFile(packageJsonPath, 'utf-8');
@@ -171,7 +173,7 @@ export class CodebaseAnalyzer {
             }
         }
         catch (error) {
-            console.error('[CodebaseAnalyzer] External system detection failed:', error);
+            logger.error('External system detection failed', error instanceof Error ? error : undefined);
         }
         return systems;
     }
@@ -193,7 +195,7 @@ export class CodebaseAnalyzer {
                 analyzeDir = srcDir;
             }
             catch {
-                console.log('[CodebaseAnalyzer] No src directory, analyzing root');
+                logger.info('No src directory, analyzing root');
             }
             // Get top-level directories as components
             const entries = await fs.readdir(analyzeDir, { withFileTypes: true });
@@ -232,7 +234,7 @@ export class CodebaseAnalyzer {
             }
         }
         catch (error) {
-            console.error('[CodebaseAnalyzer] Component analysis failed:', error);
+            logger.error('Component analysis failed', error instanceof Error ? error : undefined);
         }
         return { components, relationships };
     }
@@ -262,7 +264,7 @@ export class CodebaseAnalyzer {
             }
             catch (error) {
                 // Non-critical: permission errors when scanning directories
-                console.debug('[CodebaseAnalyzer] Directory scan error:', error instanceof Error ? error.message : error);
+                logger.debug(`Directory scan error: ${error instanceof Error ? error.message : error}`);
             }
         };
         await scan(dir, 0);
@@ -388,7 +390,7 @@ export class CodebaseAnalyzer {
             }
         }
         catch (error) {
-            console.error('[CodebaseAnalyzer] Coupling analysis failed:', error);
+            logger.error('Coupling analysis failed', error instanceof Error ? error : undefined);
         }
         return couplingInfo;
     }
@@ -427,7 +429,7 @@ export class CodebaseAnalyzer {
             }
         }
         catch (error) {
-            console.error('[CodebaseAnalyzer] C4 diagram generation failed:', error);
+            logger.error('C4 diagram generation failed', error instanceof Error ? error : undefined);
         }
         return diagrams;
     }

package/dist/domains/requirements-validation/services/product-factors-assessment/product-factors-service.d.ts

@@ -25,12 +25,6 @@ export interface ProductFactorsServiceConfig {
     /** Output directory for generated reports */
     outputDir?: string;
 }
-/**
- * Product Factors Assessment Service
- *
- * Implements SFDIPOT-based test idea generation without requiring BaseAgent.
- * This is a stateless service suitable for v3's domain-driven architecture.
- */
 export declare class ProductFactorsService {
     private readonly config;
     private readonly userStoryParser;

package/dist/domains/requirements-validation/services/product-factors-assessment/product-factors-service.js

@@ -7,6 +7,7 @@
  * This is a standalone service that can be used directly or through the
  * requirements-validation domain coordinator.
  */
+import { LoggerFactory } from '../../../../logging/index.js';
 import { HTSMCategory, Priority, AutomationFitness, SFDIPOT_SUBCATEGORIES, generateTestId, } from './types/index.js';
 import { UserStoryParser, DocumentParser, ArchitectureParser } from './parsers/index.js';
 import { SFDIPOTAnalyzer } from './analyzers/index.js';
@@ -21,6 +22,7 @@ import { domainPatternRegistry } from './patterns/index.js';
  * Implements SFDIPOT-based test idea generation without requiring BaseAgent.
  * This is a stateless service suitable for v3's domain-driven architecture.
  */
+const logger = LoggerFactory.create('requirements-validation/product-factors');
 export class ProductFactorsService {
     config;
     // Modular components
@@ -66,7 +68,7 @@ export class ProductFactorsService {
      * Main entry point - Generate full Product Factors assessment
      */
     async assess(input) {
-        console.log('[ProductFactorsService] Starting Product Factors assessment...');
+        logger.info('Starting Product Factors assessment...');
         // Step 1: Parse input documents
         const parsedInput = await this.parseInput(input);
         // Step 2: Detect project context
@@ -75,7 +77,7 @@ export class ProductFactorsService {
         let requirementsQualityScore;
         let requirementsQualityData;
         if (this.config.enableBrutalHonesty && parsedInput.rawContent.length > 100) {
-            console.log('[ProductFactorsService] Running Bach mode BS detection on requirements...');
+            logger.info('Running Bach mode BS detection on requirements...');
             const reqAnalysis = this.sfdipotAnalyzer
                 .getBrutalHonestyAnalyzer()
                 .analyzeRequirements(parsedInput.rawContent, context.detectedDomains);
@@ -85,10 +87,10 @@ export class ProductFactorsService {
                 const domainInfo = context.detectedDomains
                     .map((d) => `${d.displayName} (${(d.confidence * 100).toFixed(0)}%)`)
                     .join(', ');
-                console.log(`[ProductFactorsService] Detected domains: ${domainInfo}`);
+                logger.info(`Detected domains: ${domainInfo}`);
             }
             if (reqAnalysis.findings.length > 0) {
-                console.log(`[ProductFactorsService] Requirements quality: ${reqAnalysis.score}/100 - ${reqAnalysis.verdict}`);
+                logger.info(`Requirements quality: ${reqAnalysis.score}/100 - ${reqAnalysis.verdict}`);
             }
         }
         // Step 3: Analyze using SFDIPOT framework
@@ -97,7 +99,7 @@ export class ProductFactorsService {
         let codeIntelligenceResult;
         let c4Diagrams;
         if (input.codebaseRootDir && input.enableCodeIntelligence !== false) {
-            console.log(`[ProductFactorsService] Running code intelligence analysis on ${input.codebaseRootDir}...`);
+            logger.info(`Running code intelligence analysis on ${input.codebaseRootDir}...`);
             try {
                 const codebaseAnalyzer = new CodebaseAnalyzer({
                     rootDir: input.codebaseRootDir,
@@ -122,7 +124,7 @@ export class ProductFactorsService {
                 }
             }
             catch (error) {
-                console.warn('[ProductFactorsService] Code intelligence analysis failed:', error);
+                logger.warn('Code intelligence analysis failed');
             }
         }
         // Step 4: Generate clarifying questions for gaps
@@ -160,7 +162,7 @@ export class ProductFactorsService {
             codeIntelligence: codeIntelligenceResult,
             c4Diagrams,
         };
-        console.log(`[ProductFactorsService] Assessment complete: ${testIdeas.length} test ideas, ${allQuestions.length} questions`);
+        logger.info(`Assessment complete: ${testIdeas.length} test ideas, ${allQuestions.length} questions`);
         return result;
     }
     /**