codeguardian-mcp 1.3.8 → 1.4.0

package/dist/agent/autoValidator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"autoValidator.d.ts","sourceRoot":"","sources":["../../src/agent/autoValidator.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAgDH,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,KAAK,CAAC;QACZ,IAAI,EAAE,MAAM,CAAC;QACb,QAAQ,EAAE,MAAM,CAAC;QACjB,OAAO,EAAE,MAAM,CAAC;QAChB,UAAU,CAAC,EAAE,MAAM,CAAC;QACpB,IAAI,CAAC,EAAE,MAAM,CAAC;QACd,IAAI,CAAC,EAAE,MAAM,CAAC;KACf,CAAC,CAAC;IACH,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,OAAO,CAAC;CACzB;AAED,MAAM,MAAM,SAAS,GAAG,MAAM,GAAG,UAAU,GAAG,QAAQ,CAAC;AAEvD,qBAAa,aAAa;IACxB,OAAO,CAAC,OAAO,CAAc;IAC7B,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,OAAO,CAAmD;IAClE,OAAO,CAAC,cAAc,CAA+B;IACrD,OAAO,CAAC,IAAI,CAAqB;IACjC,OAAO,CAAC,gBAAgB,CAAa;IACrC,OAAO,CAAC,eAAe,CAA0B;IACjD,OAAO,CAAC,QAAQ,CAAqC;IACrD,OAAO,CAAC,aAAa,CAAuC;IAC5D,OAAO,CAAC,SAAS,CAAS;IAC1B,OAAO,CAAC,gBAAgB,CAAC,CAA0C;IACnE,OAAO,CAAC,WAAW,CAAkB;IACrC,OAAO,CAAC,UAAU,CAAqC;IACvD,OAAO,CAAC,UAAU,CAAqC;IACvD,OAAO,CAAC,gBAAgB,CAAwC;IAChE,OAAO,CAAC,yBAAyB,CAAa;IAC9C,OAAO,CAAC,0BAA0B,CAA+B;IACjE,OAAO,CAAC,6BAA6B,CAAuB;IAC5D,OAAO,CAAC,6BAA6B,CAAkB;IACvD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,wBAAwB,CAAO;IACvD,OAAO,CAAC,qBAAqB,CAAa;IAC1C,OAAO,CAAC,eAAe,CAA0B;IACjD,OAAO,CAAC,kBAAkB,CAAa;IACvC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,0BAA0B,CAAK;IACvD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,wBAAwB,CAAK;IAIrD,OAAO,CAAC,aAAa,CAAkB;IACvC,OAAO,CAAC,cAAc,CAA2C;IAGjE,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,qBAAqB,CAAK;IAClD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,wBAAwB,CAAM;IAGtD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,iBAAiB,CAGvC;IACF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,gBAAgB,CAGtC;IACF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,eAAe,CAErC;IAEF;;OAEG;IACH,OAAO,CAAC,MAAM,CAAC,kBAAkB;gBAiB/B,WAAW,EAAE,MAAM,EACnB,QAAQ,GAAE,MAAqB,EAC/B,IAAI,GAAE,SAAkB,EACxB,SAAS,GAAE,MAAuB;IAapC,eAAe,CAAC,OAAO,EAAE,CAAC,KAAK,EAAE,eAAe,KAAK,IAAI,GAAG,IAAI;IAI1D,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAwG5B,OAAO,CAAC,iBAAiB;IASzB;;;OAGG;YACW,sBAAsB;IA0FpC;;;;;OAKG;IACH,OAAO,CAAC,sBAAsB;IAyB9B;;;OAGG;IACH,OAAO,CAAC,qBAAqB;IAK7B;;;OAGG;IACH,OAAO,CAAC,gBAAgB;IAsBxB;;;;;OAKG;IACH,OAAO,CAAC,sBAAsB;IAU9B;;;OAGG;IACH,OAAO,CAAC,eAAe;IA6CvB;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IAqE3B;;;;;OAKG;IACH,MAAM,CAAC,yBAAyB,CAAC,SAAS,EAAE,GAAG,EAAE,GAAG,GAAG,EAAE;IAgBzD,OAAO,CAAC,4BAA4B;YActB,qBAAqB;IAwOnC,OAAO,CAAC,wBAAwB;IAIhC,OAAO,CAAC,wBAAwB;IAmChC,OAAO,CAAC,wBAAwB;IAuChC,OAAO,CAAC,sBAAsB;IAI9B,OAAO,CAAC,2BAA2B;IAoCnC,OAAO,CAAC,6BAA6B;YAiBvB,8BAA8B;YAyB9B,wBAAwB;IAyBtC,IAAI,IAAI,IAAI;IAYZ,OAAO,CAAC,gBAAgB;IAmFxB;;;;OAIG;YACW,gBAAgB;IAe9B;;;;OAIG;IACH,OAAO,CAAC,iBAAiB;YAQX,aAAa;IAU3B,OAAO,CAAC,sBAAsB;YAUhB,YAAY;YA0QZ,WAAW;YA2BX,gBAAgB;IAqC9B,SAAS,IAAI;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,WAAW,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE;CAMtE"}
+{"version":3,"file":"autoValidator.d.ts","sourceRoot":"","sources":["../../src/agent/autoValidator.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AA0DH,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,KAAK,CAAC;QACZ,IAAI,EAAE,MAAM,CAAC;QACb,QAAQ,EAAE,MAAM,CAAC;QACjB,OAAO,EAAE,MAAM,CAAC;QAChB,UAAU,CAAC,EAAE,MAAM,CAAC;QACpB,IAAI,CAAC,EAAE,MAAM,CAAC;QACd,IAAI,CAAC,EAAE,MAAM,CAAC;KACf,CAAC,CAAC;IACH,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,OAAO,CAAC;CACzB;AAED,MAAM,MAAM,SAAS,GAAG,MAAM,GAAG,UAAU,GAAG,QAAQ,CAAC;AAEvD,qBAAa,aAAa;IACxB,OAAO,CAAC,OAAO,CAAc;IAC7B,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,OAAO,CAAmD;IAClE,OAAO,CAAC,cAAc,CAA+B;IACrD,OAAO,CAAC,IAAI,CAAqB;IACjC,OAAO,CAAC,gBAAgB,CAAa;IACrC,OAAO,CAAC,eAAe,CAA0B;IACjD,OAAO,CAAC,QAAQ,CAAqC;IACrD,OAAO,CAAC,aAAa,CAAuC;IAC5D,OAAO,CAAC,SAAS,CAAS;IAC1B,OAAO,CAAC,gBAAgB,CAAC,CAA0C;IACnE,OAAO,CAAC,WAAW,CAAkB;IACrC,OAAO,CAAC,UAAU,CAAqC;IACvD,OAAO,CAAC,UAAU,CAAqC;IACvD,OAAO,CAAC,gBAAgB,CAAwC;IAChE,OAAO,CAAC,yBAAyB,CAAa;IAC9C,OAAO,CAAC,0BAA0B,CAA+B;IACjE,OAAO,CAAC,6BAA6B,CAAuB;IAC5D,OAAO,CAAC,6BAA6B,CAAkB;IACvD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,wBAAwB,CAAO;IACvD,OAAO,CAAC,qBAAqB,CAAa;IAC1C,OAAO,CAAC,eAAe,CAA0B;IACjD,OAAO,CAAC,kBAAkB,CAAa;IACvC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,0BAA0B,CAAK;IACvD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,wBAAwB,CAAK;IAIrD,OAAO,CAAC,aAAa,CAAkB;IACvC,OAAO,CAAC,cAAc,CAA2C;IAGjE,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,qBAAqB,CAAK;IAClD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,wBAAwB,CAAM;IAGtD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,iBAAiB,CAWvC;IACF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,gBAAgB,CAUtC;IACF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,eAAe,CAMrC;IAEF;;OAEG;IACH,OAAO,CAAC,MAAM,CAAC,kBAAkB;gBAiB/B,WAAW,EAAE,MAAM,EACnB,QAAQ,GAAE,MAAqB,EAC/B,IAAI,GAAE,SAAkB,EACxB,SAAS,GAAE,MAAuB;IAapC,eAAe,CAAC,OAAO,EAAE,CAAC,KAAK,EAAE,eAAe,KAAK,IAAI,GAAG,IAAI;IAI1D,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAyH5B,OAAO,CAAC,iBAAiB;IASzB;;;OAGG;YACW,sBAAsB;IA0GpC;;;;;OAKG;IACH,OAAO,CAAC,sBAAsB;IA4B9B;;;OAGG;IACH,OAAO,CAAC,qBAAqB;IAK7B;;;OAGG;IACH,OAAO,CAAC,gBAAgB;IA0BxB;;;;;OAKG;IACH,OAAO,CAAC,sBAAsB;IAU9B;;;OAGG;IACH,OAAO,CAAC,eAAe;IAgDvB;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IA0E3B;;;;;OAKG;IACH,MAAM,CAAC,yBAAyB,CAAC,SAAS,EAAE,GAAG,EAAE,GAAG,GAAG,EAAE;IAiBzD,OAAO,CAAC,4BAA4B;YActB,qBAAqB;IAgSnC,OAAO,CAAC,wBAAwB;IAIhC,OAAO,CAAC,wBAAwB;IA4ChC,OAAO,CAAC,wBAAwB;IAsDhC,OAAO,CAAC,sBAAsB;IAI9B,OAAO,CAAC,2BAA2B;IA8CnC,OAAO,CAAC,6BAA6B;YAiBvB,8BAA8B;YAyB9B,wBAAwB;IA6BtC,IAAI,IAAI,IAAI;IAYZ,OAAO,CAAC,gBAAgB;IA8FxB;;;;OAIG;YACW,gBAAgB;IAkB9B;;;;OAIG;IACH,OAAO,CAAC,iBAAiB;YAUX,aAAa;IAU3B,OAAO,CAAC,sBAAsB;YAahB,YAAY;YA4UZ,WAAW;YAmCX,gBAAgB;IAmD9B,SAAS,IAAI;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,WAAW,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE;CAMtE"}

package/dist/agent/autoValidator.js

@@ -10,17 +10,17 @@ import * as fs from "fs/promises";
 import * as path from "path";
 import { FileWatcher } from "./fileWatcher.js";
 import { orchestrateContext } from "../context/contextOrchestrator.js";
-import { refreshFileContext, markGuardianActive, markGuardianInactive } from "../context/projectContext.js";
+import { refreshFileContext, markGuardianActive, markGuardianInactive, } from "../context/projectContext.js";
 import { extractUsagesAST, extractImportsAST, extractImportsASTWithOptions, extractTypeReferencesAST, } from "../tools/validation/extractors/index.js";
 import { loadManifestDependencies, loadPythonModuleExports, } from "../tools/validation/manifest.js";
-import { extractSymbolsAST, } from "../tools/validation/extractors/index.js";
+import { extractSymbolsAST } from "../tools/validation/extractors/index.js";
 import { validateManifest, validateSymbols, buildSymbolTable, validateUsagePatterns, } from "../tools/validation/validation.js";
-import { detectDeadCode, detectUnusedLocals, shouldSkipFrameworkPattern } from "../tools/validation/deadCode.js";
+import { detectDeadCode, detectUnusedLocals, detectUnusedClassMethods, shouldSkipFrameworkPattern, } from "../tools/validation/deadCode.js";
 import { impactAnalyzer } from "../analyzers/impactAnalyzer.js";
 import { logger } from "../utils/logger.js";
 import { shouldExcludeFile } from "../utils/fileFilter.js";
 import { PROMPT_PATTERNS, VALIDATION_CONSTRAINTS } from "../prompts/library.js";
-import { generateAntiPatternContext, enrichIssuesWithAntiPatterns } from "../analyzers/antiPatterns.js";
+import { generateAntiPatternContext, enrichIssuesWithAntiPatterns, } from "../analyzers/antiPatterns.js";
 import { verifyFindingsAutomatically, getConfirmedFindings, } from "../analyzers/findingVerifier.js";
 import { validateApiContracts, } from "../api-contract/index.js";
 export class AutoValidator {
@@ -48,8 +48,8 @@ export class AutoValidator {
     activeValidationCount = 0;
     validationQueue = new Set();
     activeRefreshCount = 0;
-    static MAX_CONCURRENT_VALIDATIONS = 2;
-    static MAX_CONCURRENT_REFRESHES = 3;
+    static MAX_CONCURRENT_VALIDATIONS = 4;
+    static MAX_CONCURRENT_REFRESHES = 5;
     // During guardian initialization, we start the watcher early but buffer events.
     // This prevents missing changes that occur while context/manifest scans are running.
     isInitialized = false;
@@ -59,15 +59,34 @@ export class AutoValidator {
     static LEARNING_MODE_FILE_COUNT = 10; // After 10 files created, switch to strict
     // Common path patterns for scope detection
     static FRONTEND_PATTERNS = [
-        '/frontend/', '/client/', '/web/', '/app/', '/src/',
-        '/components/', '/pages/', '/views/', '/hooks/', '/services/'
+        "/frontend/",
+        "/client/",
+        "/web/",
+        "/app/",
+        "/src/",
+        "/components/",
+        "/pages/",
+        "/views/",
+        "/hooks/",
+        "/services/",
     ];
     static BACKEND_PATTERNS = [
-        '/backend/', '/server/', '/api/', '/services/',
-        '/routes/', '/routers/', '/controllers/', '/models/', '/db/'
+        "/backend/",
+        "/server/",
+        "/api/",
+        "/services/",
+        "/routes/",
+        "/routers/",
+        "/controllers/",
+        "/models/",
+        "/db/",
     ];
     static SHARED_PATTERNS = [
-        '/shared/', '/common/', '/types/', '/interfaces/', '/utils/'
+        "/shared/",
+        "/common/",
+        "/types/",
+        "/interfaces/",
+        "/utils/",
     ];
     /**
      * Detect language from file extension for per-file validation
@@ -147,7 +166,8 @@ export class AutoValidator {
                 this.pyManifest = await loadManifestDependencies(pyRoot, "python");
                 this.pythonExports = await loadPythonModuleExports(pyRoot);
                 // Keep this.manifest as the "primary" for backward compat
-                this.manifest = this.language === "python" ? this.pyManifest : this.tsManifest;
+                this.manifest =
+                    this.language === "python" ? this.pyManifest : this.tsManifest;
             }
             else {
                 this.manifest = await loadManifestDependencies(this.projectPath, this.language);
@@ -246,7 +266,9 @@ export class AutoValidator {
                     dirHasPython = true;
                     break;
                 }
-                catch { /* Not found */ }
+                catch {
+                    /* Not found */
+                }
             }
             for (const m of ["package.json", "tsconfig.json"]) {
                 try {
@@ -254,7 +276,9 @@ export class AutoValidator {
                     dirHasTS = true;
                     break;
                 }
-                catch { /* Not found */ }
+                catch {
+                    /* Not found */
+                }
             }
             if (dirHasPython && dirHasTS)
                 return "both";
@@ -408,7 +432,7 @@ export class AutoValidator {
         for (const pattern of AutoValidator.FRONTEND_PATTERNS) {
             if (normalizedPath.includes(pattern)) {
                 // But make sure it's not in a backend directory
-                const isBackend = AutoValidator.BACKEND_PATTERNS.some(p => normalizedPath.includes(p));
+                const isBackend = AutoValidator.BACKEND_PATTERNS.some((p) => normalizedPath.includes(p));
                 if (!isBackend) {
                     return "frontend";
                 }
@@ -421,12 +445,13 @@ export class AutoValidator {
             }
         }
         // Detect by file extension and content patterns
-        if (filePath.endsWith('.tsx') || filePath.endsWith('.jsx')) {
+        if (filePath.endsWith(".tsx") || filePath.endsWith(".jsx")) {
             return "frontend";
         }
-        if (filePath.endsWith('.py') && !filePath.includes('test')) {
+        if (filePath.endsWith(".py") && !filePath.includes("test")) {
             // Check if it's clearly a backend file
-            if (normalizedPath.includes('main.py') || normalizedPath.includes('app.py')) {
+            if (normalizedPath.includes("main.py") ||
+                normalizedPath.includes("app.py")) {
                 return "backend";
             }
         }
@@ -442,57 +467,59 @@ export class AutoValidator {
             // PLUS keep issue types that the later lenient filter explicitly preserves:
             // - unusedImport, unusedFunction, unusedExport (per-file code hygiene)
             // - dependencyHallucination, missingDependency (build-breaking or suspicious)
-            return issues.filter(issue => issue.severity === 'critical' ||
-                issue.severity === 'high' ||
-                issue.type === 'unusedImport' ||
-                issue.type === 'unusedFunction' ||
-                issue.type === 'unusedExport' ||
-                issue.type === 'dependencyHallucination' ||
-                issue.type === 'missingDependency');
+            return issues.filter((issue) => issue.severity === "critical" ||
+                issue.severity === "high" ||
+                issue.type === "unusedImport" ||
+                issue.type === "unusedFunction" ||
+                issue.type === "unusedExport" ||
+                issue.type === "dependencyHallucination" ||
+                issue.type === "missingDependency");
         }
-        return issues.filter(issue => {
+        return issues.filter((issue) => {
             // Always show critical issues
-            if (issue.severity === 'critical')
+            if (issue.severity === "critical")
                 return true;
             // Always show API contract mismatches (they affect both sides)
-            if (issue.type === 'apiContractMismatch')
+            if (issue.type === "apiContractMismatch")
                 return true;
             // Always show dead-code hygiene findings (local unuseds) regardless of scope.
             // These are high-signal and should not be dropped by scope heuristics.
-            if (issue.type === 'unusedFunction' || issue.type === 'unusedExport' || issue.type === 'orphanedFile') {
+            if (issue.type === "unusedFunction" ||
+                issue.type === "unusedExport" ||
+                issue.type === "orphanedFile") {
                 return true;
             }
             // For high severity, show if relevant to scope
-            if (issue.severity === 'high') {
+            if (issue.severity === "high") {
                 // If we can't determine scope, show all high severity
-                if (fileScope === 'unknown')
+                if (fileScope === "unknown")
                     return true;
                 // Dead code in shared files affects everyone
-                if (issue.type === 'deadCode' && fileScope === 'shared')
+                if (issue.type === "deadCode" && fileScope === "shared")
                     return true;
                 // Unused imports are always relevant
-                if (issue.type === 'unusedImport')
+                if (issue.type === "unusedImport")
                     return true;
                 return true; // Show other high severity issues
             }
             // For medium/low severity, be more selective
-            if (fileScope === 'frontend') {
+            if (fileScope === "frontend") {
                 // In frontend files, prioritize frontend-specific issues
-                if (issue.type === 'unusedImport')
+                if (issue.type === "unusedImport")
                     return true;
-                if (issue.type === 'nonExistentFunction')
+                if (issue.type === "nonExistentFunction")
                     return true;
-                if (issue.type === 'nonExistentMethod')
+                if (issue.type === "nonExistentMethod")
                     return true;
                 return false; // Filter out less relevant issues
             }
-            if (fileScope === 'backend') {
+            if (fileScope === "backend") {
                 // In backend files, prioritize backend-specific issues
-                if (issue.type === 'unusedImport')
+                if (issue.type === "unusedImport")
                     return true;
-                if (issue.type === 'nonExistentFunction')
+                if (issue.type === "nonExistentFunction")
                     return true;
-                if (issue.type === 'nonExistentMethod')
+                if (issue.type === "nonExistentMethod")
                     return true;
                 return false;
             }
@@ -544,7 +571,9 @@ export class AutoValidator {
                         message: issue.message,
                         suggestion: issue.suggestion,
                         line: issue.line,
-                        file: issue.file ? path.relative(this.projectPath, issue.file) : undefined,
+                        file: issue.file
+                            ? path.relative(this.projectPath, issue.file)
+                            : undefined,
                     })),
                     timestamp: Date.now(),
                     llmMessage: this.createApiContractMessage(apiContractResult),
@@ -591,7 +620,9 @@ export class AutoValidator {
                         message: dc.message,
                         suggestion: dc.suggestion,
                         line: dc.line,
-                        file: dc.file ? path.relative(this.projectPath, dc.file) : undefined,
+                        file: dc.file
+                            ? path.relative(this.projectPath, dc.file)
+                            : undefined,
                     })),
                     timestamp: Date.now(),
                     llmMessage: this.createInitialScanMessage(confirmedDeadCode),
@@ -663,9 +694,11 @@ export class AutoValidator {
                     // Tier 1: Symbol validation — catches hallucinated bare function calls
                     // (e.g., reportInventoryMetricsToCloud(), dispatchSystemDiagnostics())
                     // These are NOT imports, so validateManifest can't catch them.
-                    const usages = extractUsagesAST(content, fileLang, imports, { filePath });
-                    const typeReferences = fileLang === "typescript" || fileLang === "javascript" ?
-                        extractTypeReferencesAST(content, fileLang, { filePath })
+                    const usages = extractUsagesAST(content, fileLang, imports, {
+                        filePath,
+                    });
+                    const typeReferences = fileLang === "typescript" || fileLang === "javascript"
+                        ? extractTypeReferencesAST(content, fileLang, { filePath })
                         : [];
                     const symbolIssues = validateSymbols(usages, symbolTable, content, fileLang, strictModeForInitialScan, imports, this.pythonExports, context, filePath, undefined, // missingPackages
                     typeReferences);
@@ -676,6 +709,12 @@ export class AutoValidator {
                     // Tier 2: Local dead code — catches unused non-exported functions/constants
                     const localDeadCode = detectUnusedLocals(content, filePath);
                     perFileDeadCode.push(...localDeadCode);
+                    // Tier 2b: Class method dead code — catches methods on exported singletons
+                    // that are never called across the project (e.g. spoonacularService.getRecipeNutrition).
+                    // detectUnusedLocals hard-skips sym.type === "method"; detectDeadCode only tracks
+                    // top-level exported symbols — so this cross-file instance-method check fills the gap.
+                    const classMethodDeadCode = await detectUnusedClassMethods(content, filePath, context);
+                    perFileDeadCode.push(...classMethodDeadCode);
                 }
                 catch (err) {
                     logger.debug(`Skipping initial scan of ${filePath}: ${err}`);
@@ -688,7 +727,10 @@ export class AutoValidator {
                 logger.debug(`Verifying ${allPerFileFindings.length} per-file findings...`);
                 const perFileVerification = await verifyFindingsAutomatically(perFileHallucinations, perFileDeadCode, context, this.projectPath, verifyLang);
                 const confirmedPerFile = getConfirmedFindings(perFileVerification);
-                const confirmedPerFileAll = [...confirmedPerFile.hallucinations, ...confirmedPerFile.deadCode];
+                const confirmedPerFileAll = [
+                    ...confirmedPerFile.hallucinations,
+                    ...confirmedPerFile.deadCode,
+                ];
                 logger.debug(`Per-file verification: ${confirmedPerFileAll.length} confirmed (filtered ${perFileVerification.stats.falsePositiveCount} false positives)`);
                 if (confirmedPerFileAll.length > 0) {
                     const perFileAlert = {
@@ -699,7 +741,9 @@ export class AutoValidator {
                             message: issue.message || "",
                             suggestion: issue.suggestion,
                             line: issue.line,
-                            file: issue.file ? path.relative(this.projectPath, issue.file) : undefined,
+                            file: issue.file
+                                ? path.relative(this.projectPath, issue.file)
+                                : undefined,
                         })),
                         timestamp: Date.now(),
                         llmMessage: this.createPerFileScanMessage(confirmedPerFile.hallucinations, confirmedPerFile.deadCode),
@@ -765,7 +809,9 @@ export class AutoValidator {
         lines.push("");
         if (result.summary.critical > 0) {
             lines.push("**Critical Issues (Fix Immediately):**");
-            const critical = result.issues.filter((i) => i.severity === "critical").slice(0, 3);
+            const critical = result.issues
+                .filter((i) => i.severity === "critical")
+                .slice(0, 3);
             critical.forEach((issue) => {
                 lines.push(`- ${issue.message}`);
                 lines.push(`  Suggestion: ${issue.suggestion}`);
@@ -855,7 +901,9 @@ export class AutoValidator {
                 message: issue.message,
                 suggestion: issue.suggestion,
                 line: issue.line,
-                file: issue.file ? path.relative(this.projectPath, issue.file) : undefined,
+                file: issue.file
+                    ? path.relative(this.projectPath, issue.file)
+                    : undefined,
             })),
             timestamp: Date.now(),
             llmMessage: this.createApiContractMessage(apiContractResult),
@@ -905,7 +953,8 @@ export class AutoValidator {
             logger.debug(`New file detected: ${event.path} (${fileLang}) - refreshing context...`);
             refreshPromise = this.throttledRefresh(event.path, refreshLang);
             // Check if we should exit learning mode
-            if (this.mode === "auto" && this.projectFileCount >= AutoValidator.LEARNING_MODE_FILE_COUNT) {
+            if (this.mode === "auto" &&
+                this.projectFileCount >= AutoValidator.LEARNING_MODE_FILE_COUNT) {
                 // Only switch if we were previously in learning mode (implied by auto + threshold)
                 // But for simplicity, we just let the next detectProjectMode call handle it
             }
@@ -947,7 +996,7 @@ export class AutoValidator {
         const timer = setTimeout(() => {
             this.enqueueValidation(event.path);
             this.debounceTimers.delete(event.path);
-        }, 500);
+        }, 150);
         this.debounceTimers.set(event.path, timer);
     }
     /**
@@ -958,7 +1007,7 @@ export class AutoValidator {
     async throttledRefresh(filePath, language) {
         // Wait if too many refreshes are already running
         while (this.activeRefreshCount >= AutoValidator.MAX_CONCURRENT_REFRESHES) {
-            await new Promise(resolve => setTimeout(resolve, 50));
+            await new Promise((resolve) => setTimeout(resolve, 50));
         }
         this.activeRefreshCount++;
         try {
@@ -995,7 +1044,8 @@ export class AutoValidator {
     }
     processValidationQueue() {
         // Drain queue up to concurrency limit
-        while (this.validationQueue.size > 0 && this.activeValidationCount < AutoValidator.MAX_CONCURRENT_VALIDATIONS) {
+        while (this.validationQueue.size > 0 &&
+            this.activeValidationCount < AutoValidator.MAX_CONCURRENT_VALIDATIONS) {
             const next = this.validationQueue.values().next().value;
             if (!next)
                 break;
@@ -1005,12 +1055,17 @@ export class AutoValidator {
     }
     async validateFile(filePath) {
         try {
-            // Wait for ALL pending context refreshes to complete before validating.
-            // This prevents race conditions where validation runs on stale/partial context
-            // (e.g., reverseImportGraph missing entries because refreshFileContext hasn't finished).
-            if (this.pendingRefreshes.size > 0) {
-                logger.debug(`Waiting for ${this.pendingRefreshes.size} pending context refreshes...`);
-                await Promise.all(this.pendingRefreshes.values());
+            // Wait only for THIS file's own context refresh, not all pending refreshes.
+            // Waiting for every in-flight refresh (e.g. 5 simultaneous agent edits) was
+            // the primary head-of-line-blocking bottleneck in multi-agent vibe-coding:
+            // file E's validation would stall until files A-D finished refreshing, even
+            // though those refreshes updated unrelated parts of the context.
+            // Each file's reverseImportGraph entry is updated by its own refresh, so
+            // awaiting only the current file's refresh is sufficient for correctness.
+            const ownRefresh = this.pendingRefreshes.get(filePath);
+            if (ownRefresh) {
+                logger.debug(`Waiting for own context refresh: ${path.relative(this.projectPath, filePath)}`);
+                await ownRefresh;
             }
             const isNewFile = this.newFilesTracked.has(filePath);
             const mode = this.detectProjectMode();
@@ -1040,7 +1095,9 @@ export class AutoValidator {
             // Use orchestrateContext with "all" for full-stack, or the FILE's language
             // so that TSX changes are validated against a TypeScript-aware context
             // even if the guardian was started with a different language.
-            const contextLanguage = this.isFullStack ? "all" : this.resolveContextLanguage(fileLang);
+            const contextLanguage = this.isFullStack
+                ? "all"
+                : this.resolveContextLanguage(fileLang);
             const orchestration = await orchestrateContext({
                 projectPath: this.projectPath,
                 language: contextLanguage,
@@ -1051,12 +1108,14 @@ export class AutoValidator {
             const imports = extractImportsASTWithOptions(content, fileLang, {
                 filePath,
             });
-            const usedSymbols = extractUsagesAST(content, fileLang, imports, { filePath });
+            const usedSymbols = extractUsagesAST(content, fileLang, imports, {
+                filePath,
+            });
             const symbolTable = buildSymbolTable(context, orchestration.relevantSymbols);
             // Extract type references for unused import detection
             // This is essential for TypeScript where imports might only be used as types
-            const typeReferences = fileLang === "typescript" || fileLang === "javascript" ?
-                extractTypeReferencesAST(content, fileLang, { filePath })
+            const typeReferences = fileLang === "typescript" || fileLang === "javascript"
+                ? extractTypeReferencesAST(content, fileLang, { filePath })
                 : [];
             // Tier 0: Check manifest dependencies (use the correct manifest for this file's language)
             let manifestIssues = [];
@@ -1104,14 +1163,14 @@ export class AutoValidator {
             // We only run this for the changed file to avoid full project scans.
             const exportedDeadCodeIssues = [];
             const fileSymbols = extractSymbolsAST(content, filePath, fileLang);
-            const exportedSymbols = fileSymbols.filter(s => s.isExported);
+            const exportedSymbols = fileSymbols.filter((s) => s.isExported);
             if (exportedSymbols.length > 0 && context.reverseImportGraph) {
                 for (const sym of exportedSymbols) {
                     // Skip React components and framework patterns
                     if (shouldSkipFrameworkPattern(sym.name))
                         continue;
                     // Skip type exports (interfaces, types) - they might be used as type annotations
-                    if (sym.type === 'interface' || sym.type === 'type')
+                    if (sym.type === "interface" || sym.type === "type")
                         continue;
                     // Check if this exported symbol is imported anywhere
                     const importers = context.reverseImportGraph.get(filePath);
@@ -1133,7 +1192,8 @@ export class AutoValidator {
                             const importerInfo = context.files.get(importerPath);
                             if (importerInfo) {
                                 for (const imp of importerInfo.imports) {
-                                    if (imp.namedImports.includes(sym.name) || imp.defaultImport === sym.name) {
+                                    if (imp.namedImports.includes(sym.name) ||
+                                        imp.defaultImport === sym.name) {
                                         isImported = true;
                                         break;
                                     }
@@ -1156,8 +1216,21 @@ export class AutoValidator {
                 }
             }
             deadCodeIssues.push(...exportedDeadCodeIssues);
+            // Tier 2c: Class method dead code — catches instance methods on exported singletons
+            // that are never called anywhere in the project
+            // (e.g. spoonacularService.getRecipeNutrition(), spoonacularService.autocompleteIngredient()).
+            // detectUnusedLocals hard-skips sym.type === "method"; detectDeadCode only walks
+            // top-level exported symbols — this cross-file, instance-origin-tracked check fills that gap.
+            const classMethodDeadCode = await detectUnusedClassMethods(content, filePath, context);
+            deadCodeIssues.push(...classMethodDeadCode);
             // Combine all issues for verification
-            let allIssues = [...manifestIssues, ...symbolIssues, ...patternIssues, ...deadCodeIssues, ...impactIssues];
+            let allIssues = [
+                ...manifestIssues,
+                ...symbolIssues,
+                ...patternIssues,
+                ...deadCodeIssues,
+                ...impactIssues,
+            ];
             // === SMART SCOPE FILTERING (NEW) ===
             // Detect file scope and filter issues by relevance
             const fileScope = this.detectFileScope(filePath);
@@ -1168,26 +1241,29 @@ export class AutoValidator {
             // === AUTOMATED VERIFICATION (eliminates false positives) ===
             if (allIssues.length > 0) {
                 logger.debug(`Verifying ${allIssues.length} findings to eliminate false positives...`);
-                const verificationResult = await verifyFindingsAutomatically(allIssues.filter(i => i.type !== 'deadCode' && i.type !== 'unusedExport' && i.type !== 'unusedFunction' && i.type !== 'orphanedFile'), allIssues.filter(i => i.type === 'deadCode' || i.type === 'unusedExport' || i.type === 'unusedFunction' || i.type === 'orphanedFile'), context, this.projectPath, fileLang);
+                const verificationResult = await verifyFindingsAutomatically(allIssues.filter((i) => i.type !== "deadCode" &&
+                    i.type !== "unusedExport" &&
+                    i.type !== "unusedFunction" &&
+                    i.type !== "orphanedFile"), allIssues.filter((i) => i.type === "deadCode" ||
+                    i.type === "unusedExport" ||
+                    i.type === "unusedFunction" ||
+                    i.type === "orphanedFile"), context, this.projectPath, fileLang);
                 // Replace with confirmed findings only
                 const confirmed = getConfirmedFindings(verificationResult);
                 // Reconstruct allIssues with verified findings
-                allIssues = [
-                    ...confirmed.hallucinations,
-                    ...confirmed.deadCode,
-                ];
+                allIssues = [...confirmed.hallucinations, ...confirmed.deadCode];
                 logger.debug(`Verification complete: ${allIssues.length} confirmed (filtered ${verificationResult.stats.falsePositiveCount} false positives)`);
             }
             // === SMART MODE FILTERING ===
             if (isLenient) {
                 // In learning/new file mode:
                 // 1. Ignore architectural deviations (we are learning patterns)
-                allIssues = allIssues.filter(i => i.type !== 'architecturalDeviation');
+                allIssues = allIssues.filter((i) => i.type !== "architecturalDeviation");
                 // 2. Ignore PROJECT-WIDE dead code in new/changing files (orphaned files, etc.)
                 // but KEEP per-file local dead code (unusedFunction, unusedExport from detectUnusedLocals)
                 // because local unused functions/constants are genuine code hygiene issues
                 // that are cheap to detect and valuable to report.
-                allIssues = allIssues.filter(i => i.type !== 'deadCode' && i.type !== 'orphanedFile');
+                allIssues = allIssues.filter((i) => i.type !== "deadCode" && i.type !== "orphanedFile");
                 // 3. Filter out "Medium" severity symbol issues
                 // BUT keep unusedImport, unusedFunction, and unusedExport warnings
                 // - unusedImport: the #1 vibecoder mistake
@@ -1250,7 +1326,8 @@ export class AutoValidator {
         // Include validation constraints from the library
         const constrainedMsg = PROMPT_PATTERNS.withConstraints(roleMsg, VALIDATION_CONSTRAINTS);
         // Add specific issues
-        const issuesList = issues.map((i, idx) => {
+        const issuesList = issues
+            .map((i, idx) => {
             let item = `${idx + 1}. [${i.severity.toUpperCase()}] ${i.type}: ${i.message}`;
             if (i.line)
                 item += ` (Line ${i.line})`;
@@ -1261,7 +1338,8 @@ export class AutoValidator {
                 item += `\n   📚 ${i.antiPattern.id}: ${i.antiPattern.name} - ${i.antiPattern.description}`;
             }
             return item;
-        }).join("\n");
+        })
+            .join("\n");
         // Generate anti-pattern context for LLM
         const antiPatternContext = await generateAntiPatternContext(issues, fileLang);
         let message = `${constrainedMsg}\n\nDETECTED ISSUES:\n${issuesList}`;