agentic-qe 3.8.4 → 3.8.6

package/dist/domains/requirements-validation/services/requirements-validator.d.ts

@@ -54,12 +54,6 @@ export interface LLMRequirementsAnalysis {
     /** Whether analysis was enhanced by LLM */
     llmEnhanced: boolean;
 }
-/**
- * Requirements Validation Service Implementation
- * Validates requirement quality, detects ambiguity, and analyzes dependencies
- *
- * ADR-051: Added LLM-powered requirements analysis for enhanced validation
- */
 export declare class RequirementsValidatorService implements IRequirementsValidationService {
     private readonly config;
     private readonly memory;

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

@@ -5,6 +5,7 @@
  * ADR-051: Added LLM-powered requirements analysis for enhanced 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 { safeJsonParse } from '../../../shared/safe-json.js';
@@ -53,6 +54,7 @@ const AMBIGUOUS_TERMS = {
  *
  * ADR-051: Added LLM-powered requirements analysis for enhanced validation
  */
+const logger = LoggerFactory.create('requirements-validation/requirements-validator');
 export class RequirementsValidatorService {
     config;
     memory;
@@ -155,13 +157,13 @@ Be specific and actionable in your suggestions.`,
                 }
                 catch {
                     // JSON parse failed - use basic analysis
-                    console.warn('[RequirementsValidator] LLM response parsing failed');
+                    logger.warn('LLM response parsing failed');
                 }
             }
             return ok(this.getBasicAnalysis(requirements));
         }
         catch (error) {
-            console.warn('[RequirementsValidator] LLM analysis failed:', error);
+            logger.warn('LLM analysis failed');
             return ok(this.getBasicAnalysis(requirements));
         }
     }

package/dist/domains/security-compliance/coordinator.js

@@ -6,6 +6,7 @@
  * - DQN Algorithm: Optimizes security test prioritization
  * - QEFlashAttention: Fast vulnerability similarity matching
  */
+import { LoggerFactory } from '../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import { toError } from '../../shared/error-utils.js';
 import { ok, err, } from '../../shared/types/index.js';
@@ -47,6 +48,7 @@ const DEFAULT_CONFIG = {
 // ============================================================================
 // Security & Compliance Coordinator Implementation
 // ============================================================================
+const logger = LoggerFactory.create('security-compliance');
 export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
     memory;
     agentCoordinator;
@@ -135,16 +137,16 @@ export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
                             ...this.config.consensusEngineConfig,
                         },
                     });
-                    console.log(`[SecurityCompliance] Multi-Model Consensus initialized with ${providers.length} providers`);
+                    logger.info(`Multi-Model Consensus initialized with ${providers.length} providers`);
                 }
                 else {
-                    console.warn('[SecurityCompliance] No model providers available for consensus verification');
+                    logger.warn('No model providers available for consensus verification');
                 }
             }
             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;
         }
     }
@@ -175,18 +177,18 @@ export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
                 );
                 // For security audits, we're the scan itself, so we just log the governance check
                 if (!validation.allowed && this.securityGovernanceMixin.isStrictMode()) {
-                    console.log(`[${this.domainName}] Security audit for auth code registered with governance`);
+                    logger.info(`Security audit for auth code registered with governance`);
                 }
             }
             // Self-healing: Check if operations should be paused due to critical topology
             if (this.minCutMixin.shouldPauseOperations()) {
-                console.warn(`[${this.domainName}] Security audit paused: topology is in critical state`);
+                logger.warn(`Security audit paused: topology is in critical state`);
                 this.failWorkflow(workflowId, 'Topology is in critical state');
                 return err(new Error('Security audit paused: topology is in critical state'));
             }
             // Self-healing: Log warning and potentially reduce scope when topology is degraded
             if (this.config.enableMinCutAwareness && !this.isTopologyHealthy()) {
-                console.warn(`[${this.domainName}] Topology degraded, using conservative audit strategy`);
+                logger.warn(`Topology degraded, using conservative audit strategy`);
                 // Could reduce parallel operations, prioritize critical checks, etc.
             }
             // Check agent availability
@@ -246,13 +248,13 @@ export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
             this.startWorkflow(workflowId, 'scan');
             // Self-healing: Check if operations should be paused due to critical topology
             if (this.minCutMixin.shouldPauseOperations()) {
-                console.warn(`[${this.domainName}] Security scan paused: topology is in critical state`);
+                logger.warn(`Security scan paused: topology is in critical state`);
                 this.failWorkflow(workflowId, 'Topology is in critical state');
                 return err(new Error('Security scan paused: topology is in critical state'));
             }
             // Self-healing: Log warning when topology is degraded
             if (this.config.enableMinCutAwareness && !this.isTopologyHealthy()) {
-                console.warn(`[${this.domainName}] Topology degraded, proceeding with conservative scan strategy`);
+                logger.warn(`Topology degraded, proceeding with conservative scan strategy`);
             }
             // Run scan based on type
             let result;
@@ -289,13 +291,13 @@ export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
             this.startWorkflow(workflowId, 'compliance');
             // Self-healing: Check if operations should be paused due to critical topology
             if (this.minCutMixin.shouldPauseOperations()) {
-                console.warn(`[${this.domainName}] Compliance validation paused: topology is in critical state`);
+                logger.warn(`Compliance validation paused: topology is in critical state`);
                 this.failWorkflow(workflowId, 'Topology is in critical state');
                 return err(new Error('Compliance validation paused: topology is in critical state'));
             }
             // Self-healing: Log warning when topology is degraded
             if (this.config.enableMinCutAwareness && !this.isTopologyHealthy()) {
-                console.warn(`[${this.domainName}] Topology degraded, proceeding with compliance validation`);
+                logger.warn(`Topology degraded, proceeding with compliance validation`);
             }
             // Get standard by ID and validate
             const standards = await this.complianceValidator.getAvailableStandards();
@@ -331,13 +333,13 @@ export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
             this.startWorkflow(workflowId, 'posture');
             // Self-healing: Check if operations should be paused due to critical topology
             if (this.minCutMixin.shouldPauseOperations()) {
-                console.warn(`[${this.domainName}] Security posture assessment paused: topology is in critical state`);
+                logger.warn(`Security posture assessment paused: topology is in critical state`);
                 this.failWorkflow(workflowId, 'Topology is in critical state');
                 return err(new Error('Security posture assessment paused: topology is in critical state'));
             }
             // Self-healing: Log warning when topology is degraded
             if (this.config.enableMinCutAwareness && !this.isTopologyHealthy()) {
-                console.warn(`[${this.domainName}] Topology degraded, proceeding with security posture assessment`);
+                logger.warn(`Topology degraded, proceeding with security posture assessment`);
             }
             const result = await this.securityAuditor.getSecurityPosture();
             if (result.success) {
@@ -409,7 +411,7 @@ export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
             return scored.map((s) => s.test);
         }
         catch (error) {
-            console.error('Failed to prioritize security tests:', error);
+            logger.error('Failed to prioritize security tests:', error instanceof Error ? error : undefined);
             return tests;
         }
     }
@@ -441,10 +443,10 @@ export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
                 timestamp: new Date(),
             };
             await this.dqnAlgorithm.train(experience);
-            console.log(`[DQN] Trained security audit with reward: ${reward.toFixed(3)}`);
+            logger.info(`Trained security audit with reward: ${reward.toFixed(3)}`);
         }
         catch (error) {
-            console.error('Failed to train DQN:', error);
+            logger.error('Failed to train DQN:', error instanceof Error ? error : undefined);
         }
     }
     /**
@@ -530,12 +532,12 @@ export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
                 const outputSlice = Array.from(attentionResult.slice(i * dim, (i + 1) * dim));
                 const selfSimilarity = embeddings[i].reduce((sum, val, idx) => sum + val * outputSlice[idx], 0);
                 if (selfSimilarity > 0.7) {
-                    console.log(`[FlashAttention] High self-attention for vulnerability ${vulnerabilities[i].id}`);
+                    logger.info(`High self-attention for vulnerability ${vulnerabilities[i].id}`);
                 }
             }
         }
         catch (error) {
-            console.error('Failed to enhance vulnerability analysis:', error);
+            logger.error('Failed to enhance vulnerability analysis:', error instanceof Error ? error : undefined);
         }
     }
     /**
@@ -692,28 +694,28 @@ export class SecurityComplianceCoordinator extends BaseDomainCoordinator {
                 const result = await this.consensusMixin.verifyFinding(domainFinding);
                 if (result.success && result.value.verdict === 'verified') {
                     // Finding confirmed by multiple models
-                    console.log(`[${this.domainName}] Consensus VERIFIED: ${vuln.title} (confidence: ${(result.value.confidence * 100).toFixed(1)}%)`);
+                    logger.info(`Consensus VERIFIED: ${vuln.title} (confidence: ${(result.value.confidence * 100).toFixed(1)}%)`);
                     verified.push(vuln);
                 }
                 else if (result.success && result.value.verdict === 'disputed') {
                     // Disputed findings still included but logged
-                    console.log(`[${this.domainName}] Consensus DISPUTED: ${vuln.title} - including with lower confidence`);
+                    logger.info(`Consensus DISPUTED: ${vuln.title} - including with lower confidence`);
                     verified.push(vuln);
                 }
                 else {
                     // Rejected by consensus - likely false positive
-                    console.log(`[${this.domainName}] Consensus REJECTED: ${vuln.title} - likely false positive`);
+                    logger.info(`Consensus REJECTED: ${vuln.title} - likely false positive`);
                 }
             }
             catch (error) {
                 // On consensus error, include the finding but log warning
-                console.warn(`[${this.domainName}] Consensus error verifying ${vuln.title}, including by default:`, error);
+                logger.warn(`Consensus error verifying ${vuln.title}, including by default:`);
                 verified.push(vuln);
             }
         }
         const rejectedCount = highSeverity.length - verified.filter(v => v.severity === 'critical' || v.severity === 'high').length;
         if (rejectedCount > 0) {
-            console.log(`[${this.domainName}] Consensus filtered out ${rejectedCount} likely false positive(s)`);
+            logger.info(`Consensus filtered out ${rejectedCount} likely false positive(s)`);
         }
         return verified;
     }

package/dist/domains/security-compliance/services/scanners/dast-scanner.d.ts

@@ -4,27 +4,6 @@
  */
 import { Result } from '@shared/types/index.js';
 import type { SecurityScannerConfig, DASTResult, DASTOptions, AuthCredentials, MemoryBackend, ScanStatus } from './scanner-types.js';
-/**
- * DAST Scanner - Dynamic Application Security Testing
- * Scans running applications for security vulnerabilities
- *
- * **Capabilities:**
- * - Security header analysis (HSTS, CSP, X-Frame-Options, etc.)
- * - Cookie security (Secure, HttpOnly, SameSite flags)
- * - CORS misconfiguration detection
- * - Sensitive file exposure (/.git, /.env, etc.)
- * - Link crawling with same-origin scope
- * - XSS reflection testing (GET parameters)
- * - SQL injection error-based detection (GET parameters)
- * - Form security analysis (CSRF tokens, autocomplete, action URLs)
- *
- * **Limitations:**
- * - Injection testing: GET parameters only (POST form submission not implemented)
- * - Crawling: Same-origin only, max 10 links per page, single depth
- * - Auth flows: Header-based only, no login form automation
- * - No JavaScript execution (static response analysis only)
- * - No session management testing beyond cookie attributes
- */
 export declare class DASTScanner {
     private readonly config;
     private readonly memory;

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

@@ -2,6 +2,7 @@
  * Agentic QE v3 - DAST (Dynamic Application Security Testing) Scanner
  * Performs dynamic analysis of running applications to detect security vulnerabilities
  */
+import { LoggerFactory } from '../../../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import { ok, err } from '@shared/types/index.js';
 import { toError } from '@shared/error-utils.js';
@@ -30,6 +31,7 @@ import { analyzeSecurityHeaders, analyzeCookieSecurity, analyzeServerHeaders, sc
  * - No JavaScript execution (static response analysis only)
  * - No session management testing beyond cookie attributes
  */
+const logger = LoggerFactory.create('security-compliance/dast-scanner');
 export class DASTScanner {
     config;
     memory;
@@ -186,7 +188,7 @@ export class DASTScanner {
             }
         }
         catch (error) {
-            console.error('DAST scan error:', error);
+            logger.error('DAST scan error:', error instanceof Error ? error : undefined);
         }
         return { vulnerabilities, crawledUrls };
     }
@@ -247,7 +249,7 @@ export class DASTScanner {
             }
         }
         catch (error) {
-            console.error('Authenticated DAST scan error:', error);
+            logger.error('Authenticated DAST scan error:', error instanceof Error ? error : undefined);
         }
         return { vulnerabilities, crawledUrls };
     }

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

@@ -5,10 +5,6 @@
 import { Result } from '@shared/types/index.js';
 import type { FilePath } from '@shared/value-objects/index.js';
 import type { SecurityScannerConfig, Vulnerability, RemediationAdvice, SASTResult, RuleSet, FalsePositiveCheck, MemoryBackend, HybridRouter, ScanStatus } from './scanner-types.js';
-/**
- * SAST Scanner - Static Application Security Testing
- * Scans source code files for security vulnerabilities using pattern matching
- */
 export declare class SASTScanner {
     private readonly config;
     private readonly memory;

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

@@ -2,6 +2,7 @@
  * Agentic QE v3 - SAST (Static Application Security Testing) Scanner
  * Performs static code analysis to detect security vulnerabilities
  */
+import { LoggerFactory } from '../../../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import { ok, err } from '@shared/types/index.js';
 import { ALL_SECURITY_PATTERNS, BUILT_IN_RULE_SETS } from './security-patterns.js';
@@ -15,6 +16,7 @@ import { isSemgrepAvailable, runSemgrepWithRules, convertSemgrepFindings, } from
  * SAST Scanner - Static Application Security Testing
  * Scans source code files for security vulnerabilities using pattern matching
  */
+const logger = LoggerFactory.create('security-compliance/sast-scanner');
 export class SASTScanner {
     config;
     memory;
@@ -557,7 +559,7 @@ Provide detailed remediation advice specific to this code.`,
             }
         }
         catch (error) {
-            console.warn('[SASTScanner] LLM analysis failed:', error);
+            logger.warn('LLM analysis failed:');
         }
         return this.getDefaultRemediation(vuln);
     }

package/dist/domains/security-compliance/services/security-auditor-dast.d.ts

@@ -3,9 +3,5 @@
  * Extracted from security-auditor.ts - Dynamic Application Security Testing
  */
 import type { DASTResult } from '../interfaces.js';
-/**
- * Perform DAST scan using HTTP requests to test for vulnerabilities
- * Tests for common web vulnerabilities: XSS, SQL injection, security headers, etc.
- */
 export declare function performDASTScan(targetUrl: string): Promise<DASTResult>;
 //# sourceMappingURL=security-auditor-dast.d.ts.map

package/dist/domains/security-compliance/services/security-auditor-dast.js

@@ -2,6 +2,7 @@
  * Agentic QE v3 - Security Auditor DAST Scanner
  * Extracted from security-auditor.ts - Dynamic Application Security Testing
  */
+import { LoggerFactory } from '../../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import { toErrorMessage } from '../../../shared/error-utils.js';
 // ============================================================================
@@ -11,6 +12,7 @@ import { toErrorMessage } from '../../../shared/error-utils.js';
  * Perform DAST scan using HTTP requests to test for vulnerabilities
  * Tests for common web vulnerabilities: XSS, SQL injection, security headers, etc.
  */
+const logger = LoggerFactory.create('security-compliance/security-auditor-dast');
 export async function performDASTScan(targetUrl) {
     const scanId = uuidv4();
     const startTime = Date.now();
@@ -241,7 +243,7 @@ export async function performDASTScan(targetUrl) {
         }
     }
     catch (error) {
-        console.error('DAST scan failed:', error);
+        logger.error('DAST scan failed:', error instanceof Error ? error : undefined);
     }
     const scanDurationMs = Date.now() - startTime;
     // Calculate summary

package/dist/domains/security-compliance/services/security-auditor-sast.d.ts

@@ -14,9 +14,6 @@ interface SASTVulnerabilityPattern {
     fixExample?: string;
     cweId: string;
 }
-/**
- * Get the default SAST vulnerability patterns for JS/TS analysis
- */
 export declare function getSASTVulnerabilityPatterns(): SASTVulnerabilityPattern[];
 /**
  * Perform SAST scan using AST-based analysis for JavaScript/TypeScript

package/dist/domains/security-compliance/services/security-auditor-sast.js

@@ -2,12 +2,14 @@
  * Agentic QE v3 - Security Auditor SAST Scanner
  * Extracted from security-auditor.ts - Static Application Security Testing
  */
+import { LoggerFactory } from '../../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import * as fs from 'fs/promises';
 import * as path from 'path';
 /**
  * Get the default SAST vulnerability patterns for JS/TS analysis
  */
+const logger = LoggerFactory.create('security-compliance/security-auditor-sast');
 export function getSASTVulnerabilityPatterns() {
     return [
         // SQL Injection patterns
@@ -233,7 +235,7 @@ export async function performSASTScan(findSourceFiles, shouldExclude) {
         }
     }
     catch (error) {
-        console.error('SAST scan failed:', error);
+        logger.error('SAST scan failed:', error instanceof Error ? error : undefined);
     }
     const scanDurationMs = Date.now() - startTime;
     // Calculate summary

package/dist/domains/security-compliance/services/security-auditor-secrets.d.ts

@@ -10,9 +10,6 @@ interface SecretPattern {
     regex: RegExp;
     entropyThreshold?: number;
 }
-/**
- * Get the comprehensive set of secret detection patterns
- */
 export declare function getSecretPatterns(): SecretPattern[];
 /**
  * Calculate Shannon entropy of a string

package/dist/domains/security-compliance/services/security-auditor-secrets.js

@@ -2,10 +2,12 @@
  * Agentic QE v3 - Security Auditor Secret Scanner
  * Extracted from security-auditor.ts - Secret/credential detection
  */
+import { LoggerFactory } from '../../../logging/index.js';
 import * as fs from 'fs/promises';
 /**
  * Get the comprehensive set of secret detection patterns
  */
+const logger = LoggerFactory.create('security-compliance/security-auditor-secrets');
 export function getSecretPatterns() {
     return [
         // AWS Keys
@@ -229,7 +231,7 @@ export async function scanFileForSecrets(file) {
         }
     }
     catch (error) {
-        console.error(`Failed to scan file for secrets: ${file.value}`, error);
+        logger.error(`Failed to scan file for secrets: ${file.value}`, error instanceof Error ? error : undefined);
     }
     return secrets;
 }

package/dist/domains/security-compliance/services/security-auditor.js

@@ -9,6 +9,7 @@
  *   - security-auditor-secrets.ts: Secret/credential detection
  *   - security-auditor-reports.ts: Risk calculation, posture, recommendations
  */
+import { LoggerFactory } from '../../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import * as fs from 'fs/promises';
 import * as path from 'path';
@@ -25,6 +26,7 @@ import { calculateOverallRisk, generateRecommendations, countBySeverity, countOp
 // ============================================================================
 // Security Auditor Service Implementation
 // ============================================================================
+const logger = LoggerFactory.create('security-compliance/security-auditor');
 export class SecurityAuditorService {
     memory;
     config;
@@ -269,7 +271,7 @@ export class SecurityAuditorService {
             }
         }
         catch (error) {
-            console.error(`Failed to parse dependencies from ${manifest}:`, error);
+            logger.error(`Failed to parse dependencies from ${manifest}:`, error instanceof Error ? error : undefined);
         }
         return dependencies;
     }
@@ -293,7 +295,7 @@ export class SecurityAuditorService {
             }
         }
         catch (error) {
-            console.error(`Failed to check vulnerabilities for ${dep.name}@${dep.version}:`, error);
+            logger.error(`Failed to check vulnerabilities for ${dep.name}@${dep.version}:`, error instanceof Error ? error : undefined);
         }
         return vulnerabilities;
     }
@@ -425,7 +427,7 @@ export class SecurityAuditorService {
             }
         }
         catch (error) {
-            console.error(`Failed to get latest version for ${name}:`, error);
+            logger.error(`Failed to get latest version for ${name}:`, error instanceof Error ? error : undefined);
         }
         return 'unknown';
     }
@@ -439,7 +441,7 @@ export class SecurityAuditorService {
             }
         }
         catch (error) {
-            console.error(`Failed to check deprecation for ${name}:`, error);
+            logger.error(`Failed to check deprecation for ${name}:`, error instanceof Error ? error : undefined);
         }
         return false;
     }
@@ -511,7 +513,7 @@ export class SecurityAuditorService {
             return { vulnerabilities, outdatedPackages, summary: { ...baseSummary, scanDurationMs } };
         }
         catch (error) {
-            console.error('Dependency scan failed:', error);
+            logger.error('Dependency scan failed:', error instanceof Error ? error : undefined);
             return {
                 vulnerabilities: [],
                 outdatedPackages: [],
@@ -538,7 +540,7 @@ export class SecurityAuditorService {
             }
         }
         catch (error) {
-            console.error('Secret scan failed:', error);
+            logger.error('Secret scan failed:', error instanceof Error ? error : undefined);
         }
         return { secretsFound, filesScanned };
     }
@@ -563,7 +565,7 @@ export class SecurityAuditorService {
             }
         }
         catch (error) {
-            console.debug('[SecurityAuditor] Directory read error:', error instanceof Error ? error.message : error);
+            logger.debug('Directory read error:');
         }
         return files;
     }
@@ -597,7 +599,7 @@ export class SecurityAuditorService {
             return Math.max(0, oldCount - newCount);
         }
         catch (error) {
-            console.error('Failed to count resolved vulnerabilities:', error);
+            logger.error('Failed to count resolved vulnerabilities:', error instanceof Error ? error : undefined);
             return 0;
         }
     }
@@ -639,7 +641,7 @@ export class SecurityAuditorService {
             return Math.round(sum / resolutionTimes.length);
         }
         catch (error) {
-            console.error('Failed to calculate average resolution time:', error);
+            logger.error('Failed to calculate average resolution time:', error instanceof Error ? error : undefined);
             return 72;
         }
     }

package/dist/domains/test-execution/coordinator.js

@@ -2,6 +2,7 @@
  * Agentic QE v3 - Test Execution Coordinator
  * Orchestrates test execution workflow and publishes domain events
  */
+import { LoggerFactory } from '../../logging/index.js';
 import { v4 as uuidv4 } from 'uuid';
 import { ok, err } from '../../shared/types';
 import { TestExecutionEvents, createEvent, } from '../../shared/events';
@@ -34,6 +35,7 @@ const DEFAULT_COORDINATOR_CONFIG = {
 // ============================================================================
 // Test Execution Coordinator
 // ============================================================================
+const logger = LoggerFactory.create('test-execution');
 export class TestExecutionCoordinator extends BaseDomainCoordinator {
     executor;
     flakyDetector;
@@ -273,7 +275,7 @@ export class TestExecutionCoordinator extends BaseDomainCoordinator {
         try {
             // ADR-047: Check topology health before expensive parallel operations
             if (this.config.enableMinCutAwareness && !this.isTopologyHealthy()) {
-                console.warn(`[${this.domainName}] Topology degraded, reducing parallel workers`);
+                logger.warn(`Topology degraded, reducing parallel workers`);
                 // Reduce workers when topology is unhealthy
                 request = { ...request, workers: Math.max(1, Math.floor(request.workers / 2)) };
             }
@@ -385,7 +387,7 @@ export class TestExecutionCoordinator extends BaseDomainCoordinator {
                     }, Math.min(0.9, flaky.failureRate + 0.3) // Confidence based on failure rate
                     );
                     if (!isVerified) {
-                        console.log(`[${this.domainName}] Flaky test '${flaky.testId}' not verified, skipping publication`);
+                        logger.info(`Flaky test '${flaky.testId}' not verified, skipping publication`);
                         continue; // Skip unverified flaky detections
                     }
                 }
@@ -442,7 +444,7 @@ export class TestExecutionCoordinator extends BaseDomainCoordinator {
             }, 0.7 + (testsToRetry.length / 100) // Higher confidence for larger batches
             );
             if (!isStrategyVerified) {
-                console.warn(`[${this.domainName}] Retry strategy not verified, proceeding with reduced retries`);
+                logger.warn(`Retry strategy not verified, proceeding with reduced retries`);
                 request = { ...request, maxRetries: Math.max(1, Math.floor(request.maxRetries / 2)) };
             }
         }
@@ -776,10 +778,10 @@ export class TestExecutionCoordinator extends BaseDomainCoordinator {
         if (this.consensusMixin.requiresConsensus(finding)) {
             const result = await this.consensusMixin.verifyFinding(finding);
             if (result.success && result.value.verdict === 'verified') {
-                console.log(`[${this.domainName}] Flaky test '${test.testId}' classification verified by consensus`);
+                logger.info(`Flaky test '${test.testId}' classification verified by consensus`);
                 return true;
             }
-            console.warn(`[${this.domainName}] Flaky test '${test.testId}' classification NOT verified: ${result.success ? result.value.verdict : result.error.message}`);
+            logger.warn(`Flaky test '${test.testId}' classification NOT verified: ${result.success ? result.value.verdict : result.error.message}`);
             return false;
         }
         return true; // No consensus needed
@@ -805,10 +807,10 @@ export class TestExecutionCoordinator extends BaseDomainCoordinator {
         if (this.consensusMixin.requiresConsensus(finding)) {
             const result = await this.consensusMixin.verifyFinding(finding);
             if (result.success && result.value.verdict === 'verified') {
-                console.log(`[${this.domainName}] Retry strategy verified by consensus`);
+                logger.info(`Retry strategy verified by consensus`);
                 return true;
             }
-            console.warn(`[${this.domainName}] Retry strategy NOT verified: ${result.success ? result.value.verdict : result.error.message}`);
+            logger.warn(`Retry strategy NOT verified: ${result.success ? result.value.verdict : result.error.message}`);
             return false;
         }
         return true; // No consensus needed
@@ -834,10 +836,10 @@ export class TestExecutionCoordinator extends BaseDomainCoordinator {
         if (this.consensusMixin.requiresConsensus(finding)) {
             const result = await this.consensusMixin.verifyFinding(finding);
             if (result.success && result.value.verdict === 'verified') {
-                console.log(`[${this.domainName}] Failure analysis for '${analysis.testId}' verified by consensus`);
+                logger.info(`Failure analysis for '${analysis.testId}' verified by consensus`);
                 return true;
             }
-            console.warn(`[${this.domainName}] Failure analysis NOT verified: ${result.success ? result.value.verdict : result.error.message}`);
+            logger.warn(`Failure analysis NOT verified: ${result.success ? result.value.verdict : result.error.message}`);
             return false;
         }
         return true; // No consensus needed

package/dist/domains/test-execution/services/auth-state-manager.d.ts

@@ -47,9 +47,6 @@ export interface AuthStateManagerConfig {
     /** Whether to auto-cleanup expired states (default: true) */
     autoCleanup?: boolean;
 }
-/**
- * Auth state manager errors
- */
 export declare class AuthStateError extends Error {
     readonly code: string;
     readonly cause?: Error | undefined;

package/dist/domains/test-execution/services/auth-state-manager.js

@@ -5,12 +5,14 @@
  * Uses agent-browser state save/load to persist cookies, localStorage,
  * sessionStorage, and other browser state to skip login flows in tests.
  */
+import { LoggerFactory } from '../../../logging/index.js';
 import { mkdir, access, readdir, stat, unlink } from 'fs/promises';
 import { join, dirname } from 'path';
 import { safeJsonParse } from '../../../shared/safe-json.js';
 /**
  * Auth state manager errors
  */
+const logger = LoggerFactory.create('test-execution/auth-state-manager');
 export class AuthStateError extends Error {
     code;
     cause;
@@ -228,7 +230,7 @@ export class AuthStateManager {
         }
         catch (error) {
             // Non-critical: file may not exist
-            console.debug('[AuthStateManager] State file removal skipped:', error instanceof Error ? error.message : error);
+            logger.debug('State file removal skipped:');
         }
         // Remove metadata file
         try {
@@ -236,7 +238,7 @@ export class AuthStateManager {
         }
         catch (error) {
             // Non-critical: file may not exist
-            console.debug('[AuthStateManager] Metadata file removal skipped:', error instanceof Error ? error.message : error);
+            logger.debug('Metadata file removal skipped:');
         }
         // Clear cache
         this.metadataCache.delete(name);