pumuki-ast-hooks 5.5.3 → 5.5.5

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSASTIntelligentAnalyzer.js

@@ -4,12 +4,10 @@ const fs = require('fs');
 const path = require('path');
 const crypto = require('crypto');
 const env = require(path.join(__dirname, '../../../../config/env'));
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class iOSASTIntelligentAnalyzer {
   constructor(findings) {
     this.findings = findings;
-    this.auditLogger = new AuditLogger({ repoRoot: process.cwd() });
     this.sourceKittenPath = '/opt/homebrew/bin/sourcekitten';
     this.isAvailable = this.checkSourceKitten();
     this.hasSwiftSyntax = this.checkSwiftSyntax();
@@ -644,6 +642,23 @@ class iOSASTIntelligentAnalyzer {
   }
 
   analyzeAdditionalRules(filePath) {
+    const hasSwiftUIViewType = (this.imports || []).some(i => i && i.name === 'SwiftUI') &&
+      (this.structs || []).some(s => (s['key.inheritedtypes'] || []).some(t => t && t['key.name'] === 'View'));
+
+    if (this.fileContent.includes('pushViewController') || this.fileContent.includes('popViewController') || this.fileContent.includes('present(')) {
+      const line = this.findLineNumber('pushViewController') || this.findLineNumber('popViewController') || this.findLineNumber('present(');
+      this.pushFinding('ios.navigation.imperative_navigation', 'critical', filePath, line,
+        'Imperative navigation detected - use event-driven navigation/coordinator');
+    }
+
+    const swiftuiNavTokens = ['NavigationLink', 'NavigationStack', 'NavigationSplitView', '.navigationDestination'];
+    const hasSwiftUINavigation = swiftuiNavTokens.some(token => this.fileContent.includes(token));
+    if (hasSwiftUINavigation && !hasSwiftUIViewType) {
+      const line = this.findLineNumber('NavigationLink') || this.findLineNumber('NavigationStack') || this.findLineNumber('.navigationDestination');
+      this.pushFinding('ios.navigation.swiftui_navigation_outside_view', 'critical', filePath, line,
+        'SwiftUI navigation API detected outside View types');
+    }
+
     if (filePath.includes('ViewModel') && this.fileContent.includes('NavigationLink')) {
       const hasCoordinator = this.imports.some(i => i.name.includes('Coordinator'));
       if (!hasCoordinator) {

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSArchitectureDetector.js

@@ -1,14 +1,12 @@
 const fs = require('fs');
 const path = require('path');
 const glob = require('glob');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class iOSArchitectureDetector {
   constructor(projectRoot) {
     this.projectRoot = projectRoot;
-    this.auditLogger = new AuditLogger({ repoRoot: projectRoot });
     this.patterns = {
-      featureFirstClean: 0,
+      featureFirstClean: 0,  
       mvvm: 0,
       mvvmc: 0,
       mvp: 0,
@@ -83,7 +81,7 @@ class iOSArchitectureDetector {
     );
 
     if (hasFeaturesFolders) {
-      this.patterns.featureFirstClean += 10;
+      this.patterns.featureFirstClean += 10; 
     }
 
     if (foundCleanFolders.length >= 3) {
@@ -249,8 +247,8 @@ class iOSArchitectureDetector {
       const content = this.readFile(file);
 
       if (content.includes('protocol Coordinator') ||
-        content.includes(': Coordinator') ||
-        /func\s+start\(\)/.test(content) && /func\s+navigate/.test(content)) {
+          content.includes(': Coordinator') ||
+          /func\s+start\(\)/.test(content) && /func\s+navigate/.test(content)) {
         this.patterns.mvvmc += 2;
       }
     });
@@ -290,16 +288,16 @@ class iOSArchitectureDetector {
     const interactorFiles = files.filter(f => /Interactor\.swift$/.test(f));
 
     if (viewControllerFiles.length >= 2 &&
-      viewModelFiles.length === 0 &&
-      presenterFiles.length === 0 &&
-      interactorFiles.length === 0) {
+        viewModelFiles.length === 0 &&
+        presenterFiles.length === 0 &&
+        interactorFiles.length === 0) {
 
       viewControllerFiles.forEach(file => {
         const content = this.readFile(file);
         const lines = content.split('\n').length;
 
         if (lines > 300) {
-          this.patterns.mvc += 3;
+          this.patterns.mvc += 3; 
         } else if (lines > 150) {
           this.patterns.mvc += 2;
         } else {
@@ -325,7 +323,7 @@ class iOSArchitectureDetector {
     }
 
     if (sorted.length > 1 && sorted[1][1] >= dominantScore * 0.7) {
-      return 'MIXED';
+      return 'MIXED'; 
     }
 
     return this.normalizePatternName(dominant);
@@ -333,7 +331,7 @@ class iOSArchitectureDetector {
 
   normalizePatternName(pattern) {
     const mapping = {
-      'featureFirstClean': 'FEATURE_FIRST_CLEAN_DDD',
+      'featureFirstClean': 'FEATURE_FIRST_CLEAN_DDD', 
       'mvvm': 'MVVM',
       'mvvmc': 'MVVM-C',
       'mvp': 'MVP',

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSArchitectureRules.js

@@ -1,11 +1,9 @@
 const { pushFinding } = require('../../ast-core');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class iOSArchitectureRules {
   constructor(findings, detectedPattern) {
     this.findings = findings;
     this.pattern = detectedPattern;
-    this.auditLogger = new AuditLogger({ repoRoot: process.cwd() });
   }
 
   runRules(files) {

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSCICDRules.js

@@ -12,13 +12,11 @@ const { pushFinding } = require('../../ast-core');
 const fs = require('fs');
 const path = require('path');
 const glob = require('glob');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class iOSCICDRules {
   constructor(findings, projectRoot) {
     this.findings = findings;
     this.projectRoot = projectRoot;
-    this.auditLogger = new AuditLogger({ repoRoot: projectRoot });
   }
 
   analyze() {

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSEnterpriseAnalyzer.js

@@ -2,13 +2,11 @@ const path = require('path');
 const fs = require('fs').promises;
 const { SourceKittenParser } = require('../parsers/SourceKittenParser');
 const { pushFinding, mapToLevel } = require(path.join(__dirname, '../../ast-core'));
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class iOSEnterpriseAnalyzer {
   constructor() {
     this.parser = new SourceKittenParser();
     this.findings = [];
-    this.auditLogger = new AuditLogger({ repoRoot: process.cwd() });
   }
 
   async analyzeFile(filePath, findings) {

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSForbiddenLiteralsAnalyzer.js

@@ -1,5 +1,4 @@
 const { SyntaxKind } = require('../../ast-core');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 const FORBIDDEN_STRING_LITERALS = [
     'unknown',

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSNetworkingAdvancedRules.js

@@ -9,12 +9,10 @@
 const { pushFinding } = require('../../ast-core');
 const fs = require('fs');
 const glob = require('glob');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class iOSNetworkingAdvancedRules {
   constructor(findings, projectRoot) {
     this.findings = findings;
-    this.auditLogger = new AuditLogger({ repoRoot: projectRoot });
     this.projectRoot = projectRoot;
   }
 

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSPerformanceRules.js

@@ -1,9 +1,6 @@
-const AuditLogger = require('../../application/services/logging/AuditLogger');
-
 class iOSPerformanceRules {
   constructor(findings) {
     this.findings = findings;
-    this.auditLogger = new AuditLogger({ repoRoot: process.cwd() });
   }
 
   analyzeFile(filePath, content) {

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSSPMRules.js

@@ -13,13 +13,11 @@
 const { pushFinding } = require('../../ast-core');
 const fs = require('fs');
 const path = require('path');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class iOSSPMRules {
   constructor(findings, projectRoot) {
     this.findings = findings;
     this.projectRoot = projectRoot;
-    this.auditLogger = new AuditLogger({ repoRoot: projectRoot });
   }
 
   analyze() {

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSSwiftUIAdvancedRules.js

@@ -12,12 +12,10 @@
 
 const { pushFinding } = require('../../ast-core');
 const fs = require('fs');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class iOSSwiftUIAdvancedRules {
   constructor(findings) {
     this.findings = findings;
-    this.auditLogger = new AuditLogger({ repoRoot: process.cwd() });
   }
 
   analyzeFile(filePath, ast) {

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSTestingAdvancedRules.js

@@ -10,12 +10,10 @@ const { pushFinding } = require('../../ast-core');
 const fs = require('fs');
 const glob = require('glob');
 const path = require('path');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class iOSTestingAdvancedRules {
   constructor(findings, projectRoot) {
     this.findings = findings;
-    this.auditLogger = new AuditLogger({ repoRoot: projectRoot });
     this.projectRoot = projectRoot;
   }
 

package/scripts/hooks-system/infrastructure/ast/ios/native-bridge.js

@@ -3,7 +3,6 @@ const path = require('path');
 const fs = require('fs');
 const { pushFileFinding } = require('../ast-core');
 const env = require(path.join(__dirname, '../../../config/env'));
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 function getStagedSwiftFiles(repoRoot) {
   try {

package/scripts/hooks-system/infrastructure/ast/ios/parsers/SourceKittenParser.js

@@ -5,7 +5,6 @@ const util = require('util');
 const fs = require('fs').promises;
 const path = require('path');
 const { DomainError } = require('../../../../domain/errors');
-const AuditLogger = require('../../../../application/services/logging/AuditLogger');
 
 const execPromise = util.promisify(exec);
 
@@ -21,7 +20,6 @@ class SourceKittenParser {
   constructor() {
     this.sourceKittenPath = '/opt/homebrew/bin/sourcekitten';
     this.timeout = 30000;
-    this.auditLogger = new AuditLogger({ repoRoot: process.cwd() });
   }
 
   /**

package/scripts/hooks-system/infrastructure/logging/UnifiedLoggerFactory.js

@@ -1,6 +1,6 @@
 const fs = require('fs');
 const path = require('path');
-const env = require('../../../config/env.js');
+const env = require('../../config/env.js');
 
 const UnifiedLogger = require('../../application/services/logging/UnifiedLogger');
 

package/scripts/hooks-system/infrastructure/mcp/ast-intelligence-automation.js

@@ -38,7 +38,10 @@ function safeGitRoot(startDir) {
         });
         const root = String(out || '').trim();
         return root || null;
-    } catch {
+    } catch (error) {
+        if (process.env.DEBUG) {
+            process.stderr.write(`[MCP] safeGitRoot failed: ${error && error.message ? error.message : String(error)}\n`);
+        }
         return null;
     }
 }
@@ -54,6 +57,16 @@ function resolveRepoRoot() {
 
 const REPO_ROOT = resolveRepoRoot();
 
+try {
+    if (REPO_ROOT && typeof REPO_ROOT === 'string' && fs.existsSync(REPO_ROOT)) {
+        process.chdir(REPO_ROOT);
+    }
+} catch (error) {
+    if (process.env.DEBUG) {
+        process.stderr.write(`[MCP] Failed to chdir to REPO_ROOT: ${error && error.message ? error.message : String(error)}\n`);
+    }
+}
+
 // NO singleton lock - Windsurf manages process lifecycle
 // Each project gets its own independent MCP process
 
@@ -186,6 +199,34 @@ function resolveUpdateEvidenceScript() {
     return null;
 }
 
+async function runWithTimeout(fn, timeoutMs) {
+    const ms = Number(timeoutMs);
+    if (!Number.isFinite(ms) || ms <= 0) {
+        try {
+            return { ok: true, timedOut: false, value: await fn() };
+        } catch (error) {
+            return { ok: false, timedOut: false, error };
+        }
+    }
+
+    let timer = null;
+    try {
+        const result = await Promise.race([
+            Promise.resolve().then(fn).then(value => ({ ok: true, timedOut: false, value })),
+            new Promise(resolve => {
+                timer = setTimeout(() => resolve({ ok: false, timedOut: true, error: new Error('timeout') }), ms);
+            })
+        ]);
+        return result;
+    } catch (error) {
+        return { ok: false, timedOut: false, error };
+    } finally {
+        if (timer) {
+            clearTimeout(timer);
+        }
+    }
+}
+
 // Lazy Loading Services
 function getContextEngine() {
     return getCompositionRoot().getContextDetectionEngine();
@@ -537,75 +578,123 @@ function checkBranchChangesCoherence(branchName, uncommittedChanges) {
  * Returns BLOCKED or ALLOWED status with auto-fixes applied
  */
 async function aiGateCheck() {
-    const gitFlowService = getCompositionRoot().getGitFlowService();
-    const gitQuery = getCompositionRoot().getGitQueryAdapter();
-    const currentBranch = getCurrentGitBranch(REPO_ROOT);
-    const isProtectedBranch = ['main', 'master', 'develop'].includes(currentBranch);
+    const startedAt = Date.now();
+    const gateTimeoutMs = Number(process.env.MCP_GATE_TIMEOUT_MS || 1200);
+    const strict = process.env.MCP_GATE_STRICT === 'true';
+    const allowEvidenceAutofix = process.env.MCP_GATE_AUTOFIX_EVIDENCE === 'true';
+
+    const core = async () => {
+        const gitFlowService = getCompositionRoot().getGitFlowService();
+        const gitQuery = getCompositionRoot().getGitQueryAdapter();
+        const currentBranch = getCurrentGitBranch(REPO_ROOT);
+        const isProtectedBranch = ['main', 'master', 'develop'].includes(currentBranch);
 
-    const uncommittedChanges = gitQuery.getUncommittedChanges();
-    const hasUncommittedChanges = uncommittedChanges && uncommittedChanges.length > 0;
+        let uncommittedChanges = [];
+        try {
+            uncommittedChanges = gitQuery.getUncommittedChanges();
+        } catch (error) {
+            const msg = error && error.message ? error.message : String(error);
+            if (process.env.DEBUG) {
+                process.stderr.write(`[MCP] Gate gitQuery.getUncommittedChanges failed: ${msg}\n`);
+            }
+        }
+        const hasUncommittedChanges = Array.isArray(uncommittedChanges) && uncommittedChanges.length > 0;
 
-    const violations = [];
-    const warnings = [];
-    const autoFixes = [];
+        const violations = [];
+        const warnings = [];
+        const autoFixes = [];
+
+        const evidenceMonitor = getCompositionRoot().getEvidenceMonitor();
+        if (evidenceMonitor.isStale()) {
+            if (!allowEvidenceAutofix) {
+                violations.push('❌ EVIDENCE_STALE: Evidence is stale. Run ai-start to refresh.');
+            } else {
+                const elapsed = Date.now() - startedAt;
+                const remaining = Math.max(150, gateTimeoutMs - elapsed);
+                const refreshResult = await runWithTimeout(() => evidenceMonitor.refresh(), remaining);
+                if (refreshResult.ok) {
+                    autoFixes.push('✅ Evidence was stale - AUTO-FIXED');
+                } else if (refreshResult.timedOut) {
+                    violations.push('❌ EVIDENCE_STALE: Auto-fix timed out. Run ai-start to refresh.');
+                } else {
+                    const msg = refreshResult.error && refreshResult.error.message ? refreshResult.error.message : String(refreshResult.error);
+                    violations.push(`❌ EVIDENCE_STALE: Auto-fix failed: ${msg}`);
+                }
+            }
+        }
 
-    // 1. Evidence Freshness Check (Auto-fix included)
-    const evidenceMonitor = getCompositionRoot().getEvidenceMonitor();
-    if (evidenceMonitor.isStale()) {
         try {
-            await evidenceMonitor.refresh();
-            autoFixes.push('✅ Evidence was stale - AUTO-FIXED');
-        } catch (err) {
-            violations.push(`❌ EVIDENCE_STALE: Auto-fix failed: ${err.message}`);
+            const elapsed = Date.now() - startedAt;
+            const remaining = Math.max(120, gateTimeoutMs - elapsed);
+            const ghAvailable = await runWithTimeout(() => gitFlowService.isGitHubAvailable(), remaining);
+            if (!ghAvailable.ok || ghAvailable.timedOut || ghAvailable.value === false) {
+                warnings.push('⚠️ GitHub CLI not available - some automations may be limited');
+            }
+        } catch (error) {
+            warnings.push('⚠️ GitHub CLI not available - some automations may be limited');
         }
-    }
 
-    // 2. Git Flow Integrity
-    if (!gitFlowService.isGitHubAvailable()) {
-        warnings.push('⚠️ GitHub CLI not available - some automations may be limited');
-    }
+        if (isProtectedBranch) {
+            if (hasUncommittedChanges) {
+                violations.push(`❌ ON_PROTECTED_BRANCH: You are on '${currentBranch}' with uncommitted changes.`);
+                violations.push(`   Required: create a feature branch first.`);
+            } else {
+                warnings.push(`⚠️ ON_PROTECTED_BRANCH: You are on '${currentBranch}'. Create a feature branch before making changes.`);
+            }
+        }
 
-    if (isProtectedBranch) {
-        if (hasUncommittedChanges) {
-            violations.push(`❌ ON_PROTECTED_BRANCH: You are on '${currentBranch}' with uncommitted changes.`);
-            violations.push(`   Required: create a feature branch first.`);
-        } else {
-            warnings.push(`⚠️ ON_PROTECTED_BRANCH: You are on '${currentBranch}'. Create a feature branch before making changes.`);
+        if (strict) {
+            const elapsed = Date.now() - startedAt;
+            const remaining = Math.max(200, gateTimeoutMs - elapsed);
+            const blockCommitUseCase = getCompositionRoot().getBlockCommitUseCase();
+            const astAdapter = getCompositionRoot().getAstAdapter();
+
+            const astResult = await runWithTimeout(async () => {
+                const auditResult = await astAdapter.analyzeStagedFiles();
+                return blockCommitUseCase.execute(auditResult, { useStagedOnly: true });
+            }, remaining);
+
+            if (astResult.ok && astResult.value && astResult.value.shouldBlock) {
+                violations.push(`❌ AST_VIOLATIONS: ${astResult.value.reason}`);
+            } else if (!astResult.ok && process.env.DEBUG) {
+                const msg = astResult.error && astResult.error.message ? astResult.error.message : String(astResult.error);
+                process.stderr.write(`[MCP] Gate AST check skipped: ${astResult.timedOut ? 'timeout' : msg}\n`);
+            }
         }
-    }
 
-    // 3. Block Commit Use Case Integration
-    const blockCommitUseCase = getCompositionRoot().getBlockCommitUseCase();
-    const astAdapter = getCompositionRoot().getAstAdapter();
+        const isBlocked = violations.length > 0;
 
-    try {
-        const auditResult = await astAdapter.analyzeStagedFiles();
-        const decision = await blockCommitUseCase.execute(auditResult, {
-            useStagedOnly: true
-        });
+        return {
+            status: isBlocked ? 'BLOCKED' : 'ALLOWED',
+            timestamp: new Date().toISOString(),
+            branch: currentBranch,
+            violations,
+            warnings,
+            autoFixes,
+            summary: isBlocked
+                ? `🚫 BLOCKED: ${violations.length} violation(s). Fix before proceeding.`
+                : `🚦 ALLOWED: Gate passed.${warnings.length > 0 ? ` ${warnings.length} warning(s).` : ''}`,
+            instructions: isBlocked
+                ? 'DO NOT proceed with user task. Announce violations and fix them first.'
+                : 'You may proceed with user task.'
+        };
+    };
 
-        if (decision.shouldBlock) {
-            violations.push(`❌ AST_VIOLATIONS: ${decision.reason}`);
-        }
-    } catch (err) {
-        if (process.env.DEBUG) console.error('[MCP] AST Check failed:', err.message);
+    const result = await runWithTimeout(core, gateTimeoutMs);
+    if (result.ok) {
+        return result.value;
     }
 
-    const isBlocked = violations.length > 0;
-
+    const currentBranch = getCurrentGitBranch(REPO_ROOT);
     return {
-        status: isBlocked ? 'BLOCKED' : 'ALLOWED',
+        status: 'BLOCKED',
         timestamp: new Date().toISOString(),
         branch: currentBranch,
-        violations,
-        warnings,
-        autoFixes,
-        summary: isBlocked
-            ? `🚫 BLOCKED: ${violations.length} violation(s). Fix before proceeding.`
-            : `🚦 ALLOWED: Gate passed.${warnings.length > 0 ? ` ${warnings.length} warning(s).` : ''}`,
-        instructions: isBlocked
-            ? 'DO NOT proceed with user task. Announce violations and fix them first.'
-            : 'You may proceed with user task.'
+        violations: ['❌ GATE_TIMEOUT: AI gate check timed out. Retry or run ai-start manually.'],
+        warnings: [],
+        autoFixes: [],
+        summary: '🚫 BLOCKED: Gate check timed out.',
+        instructions: 'DO NOT proceed with user task. Retry the gate check.'
     };
 }
 
@@ -977,15 +1066,18 @@ protocolHandler.start(async (message) => {
     // Start polling loops ONLY after receiving 'initialized' notification from Windsurf
     if (!mcpInitialized && message.includes('"method":"initialized"')) {
         mcpInitialized = true;
-        process.stderr.write(`[MCP] Received 'initialized' - starting background loops\n`);
+        if (process.env.DEBUG) {
+            process.stderr.write(`[MCP] Received 'initialized' - starting background loops\n`);
+        }
         startPollingLoops();
     }
 
     return response;
 });
 
-// Log MCP ready
-process.stderr.write(`[MCP] Server ready for ${REPO_ROOT}\n`);
+if (process.env.DEBUG) {
+    process.stderr.write(`[MCP] Server ready for ${REPO_ROOT}\n`);
+}
 
 /**
  * Start polling loops for background notifications and automations

package/scripts/hooks-system/infrastructure/mcp/services/McpProtocolHandler.js

@@ -1,18 +1,21 @@
 const MCP_VERSION = '2024-11-05';
 const AuditLogger = require('../../../application/services/logging/AuditLogger');
+const env = require('../../../config/env');
 
 class McpProtocolHandler {
     constructor(inputStream, outputStream, logger) {
         this.inputStream = inputStream;
         this.outputStream = outputStream;
         this.logger = logger;
-        this.auditLogger = new AuditLogger({ repoRoot: process.cwd(), logger });
+        const repoRoot = (env.get('REPO_ROOT') || '').trim() || process.cwd();
+        this.auditLogger = new AuditLogger({ repoRoot, logger });
         this.buffer = Buffer.alloc(0);
     }
 
     start(messageHandler) {
-        // Log that we're starting
-        process.stderr.write('[MCP] Protocol handler starting...\n');
+        if (process.env.DEBUG) {
+            process.stderr.write('[MCP] Protocol handler starting...\n');
+        }
 
         this.inputStream.on('data', (chunk) => {
             const buf = Buffer.isBuffer(chunk) ? chunk : Buffer.from(String(chunk), 'utf8');
@@ -29,8 +32,9 @@ class McpProtocolHandler {
             process.exit(1);
         });
 
-        // Log that we're ready
-        process.stderr.write('[MCP] Protocol handler ready\n');
+        if (process.env.DEBUG) {
+            process.stderr.write('[MCP] Protocol handler ready\n');
+        }
     }
 
     async _handleChunk(chunk, messageHandler) {
@@ -115,8 +119,12 @@ class McpProtocolHandler {
             }
 
             const lineBuf = this.buffer.slice(0, nl);
-            this.buffer = this.buffer.slice(nl + 1);
             const line = lineBuf.toString('utf8').trim();
+            if (/^content-length:/i.test(line)) {
+                break;
+            }
+
+            this.buffer = this.buffer.slice(nl + 1);
             if (line) {
                 messages.push({ body: line, framed: false });
             }

package/scripts/hooks-system/infrastructure/repositories/CursorTokenRepository.js

@@ -1,5 +1,4 @@
 const env = require('../../config/env');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 const path = require('path');
 const ICursorTokenRepository = require('../../domain/repositories/ICursorTokenRepository');
@@ -20,7 +19,6 @@ class CursorTokenRepository extends ICursorTokenRepository {
         this.apiDataSource = new CursorApiDataSource({ apiUrl, apiToken, fetchImpl, logger });
         this.fileDataSource = new CursorFileDataSource({ usageFile, logger });
         this.logger = logger;
-        this.auditLogger = new AuditLogger({ repoRoot: process.cwd(), logger });
     }
 
     /**

package/scripts/hooks-system/infrastructure/repositories/FileFindingsRepository.js

@@ -4,7 +4,6 @@ const path = require('path');
 const IFindingsRepository = require('../../domain/repositories/IFindingsRepository');
 const Finding = require('../../domain/entities/Finding');
 const AuditResult = require('../../domain/entities/AuditResult');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class FileFindingsRepository extends IFindingsRepository {
   constructor(basePath = '.audit_tmp') {
@@ -12,7 +11,6 @@ class FileFindingsRepository extends IFindingsRepository {
     this.basePath = basePath;
     this.findingsFile = path.join(basePath, 'findings.json');
     this.auditResultFile = path.join(basePath, 'audit-result.json');
-    this.auditLogger = new AuditLogger({ repoRoot: process.cwd() });
   }
 
   async ensureDirectory() {

package/scripts/hooks-system/infrastructure/repositories/datasources/CursorApiDataSource.js

@@ -1,5 +1,4 @@
-const env = require('../../config/env');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
+const env = require('../../../config/env.js');
 
 const { DomainError } = require('../../../domain/errors');
 
@@ -14,7 +13,6 @@ class CursorApiDataSource {
         this.apiToken = apiToken;
         this.fetch = fetchImpl;
         this.logger = logger;
-        this.auditLogger = new AuditLogger({ repoRoot: process.cwd(), logger });
         this.failureCount = 0;
         this.failureThreshold = 5;
         this.circuitOpenUntil = null;

package/scripts/hooks-system/infrastructure/repositories/datasources/CursorFileDataSource.js

@@ -1,5 +1,4 @@
 const fs = require('fs');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 class CursorFileDataSource {
     constructor({
@@ -10,7 +9,6 @@ class CursorFileDataSource {
         this.usageFile = usageFile;
         this.fs = fsModule;
         this.logger = logger;
-        this.auditLogger = new AuditLogger({ repoRoot: process.cwd(), logger });
     }
 
     async readUsage() {

package/scripts/hooks-system/infrastructure/severity/severity-evaluator.js

@@ -8,7 +8,6 @@ const { ContextBuilder } = require('./context/context-builder');
 const RecommendationGenerator = require('./generators/RecommendationGenerator');
 const ContextMultiplier = require('./scorers/ContextMultiplier');
 const SeverityMapper = require('./mappers/SeverityMapper');
-const AuditLogger = require('../../application/services/logging/AuditLogger');
 
 /**
  * Main severity evaluator
@@ -23,7 +22,6 @@ class SeverityEvaluator {
     this.contextBuilder = new ContextBuilder();
     this.recommendationGenerator = new RecommendationGenerator();
     this.contextMultiplier = new ContextMultiplier();
-    this.auditLogger = new AuditLogger({ repoRoot: process.cwd() });
 
     this.weights = {
       security: 0.40,