@vibecheckai/cli 3.2.5 → 3.3.0

package/bin/runners/lib/doctor/service.js

@@ -1,262 +1,262 @@
-/**
- * DoctorService — Enterprise-Grade Environment Diagnostics
- * 
- * Features:
- * - Pluggable diagnostic modules
- * - Parallel execution for independent checks
- * - Dependency resolution between checks
- * - Auto-fix capability
- * - JSON/Markdown report generation
- * - Health scoring with weighted categories
- * - CI-friendly exit codes
- */
-
-const path = require('path');
-const { SEVERITY, CATEGORY, CATEGORY_META } = require('./types');
-const { getAllDiagnostics } = require('./modules');
-const { createSummary, saveReports, generateJsonReport } = require('./reporter');
-const { autoFixAll, getFixCommands } = require('./autofix');
-const ui = require('./ui');
-
-class DoctorService {
-  constructor(projectPath, options = {}) {
-    this.projectPath = path.resolve(projectPath);
-    this.options = {
-      json: false,
-      markdown: false,
-      fix: false,
-      fixDryRun: false,
-      quiet: false,
-      verbose: false,
-      categories: null, // null = all categories
-      skipNetwork: false,
-      saveReport: true,
-      ...options,
-    };
-    
-    this.diagnostics = [];
-    this.results = [];
-    this.startTime = null;
-  }
-
-  async run() {
-    this.startTime = Date.now();
-    
-    // Load diagnostics
-    this.diagnostics = this.loadDiagnostics();
-    
-    // Filter by category if specified
-    if (this.options.categories) {
-      this.diagnostics = this.diagnostics.filter(d => 
-        this.options.categories.includes(d.category)
-      );
-    }
-    
-    // Skip network if requested
-    if (this.options.skipNetwork) {
-      this.diagnostics = this.diagnostics.filter(d => d.category !== CATEGORY.NETWORK);
-    }
-    
-    // Print header (unless quiet or json)
-    if (!this.options.quiet && !this.options.json) {
-      ui.printHeader();
-    }
-    
-    // Run diagnostics
-    await this.runDiagnostics();
-    
-    // Calculate summary
-    const durationMs = Date.now() - this.startTime;
-    const summary = createSummary(this.results);
-    
-    // Output results
-    if (this.options.json) {
-      // JSON output only
-      const report = generateJsonReport(this.results, this.projectPath, durationMs);
-      console.log(JSON.stringify(report, null, 2));
-    } else if (!this.options.quiet) {
-      // Print issues and summary
-      ui.printIssuesSummary(this.results);
-      ui.printFinalSummary(summary, durationMs);
-    }
-    
-    // Save reports
-    if (this.options.saveReport && !this.options.json) {
-      const outputDir = path.join(this.projectPath, '.vibecheck');
-      try {
-        const paths = saveReports(this.results, this.projectPath, durationMs, outputDir);
-        if (this.options.verbose) {
-          console.log(`${ui.c.dim}Reports saved to:${ui.c.reset}`);
-          console.log(`  ${ui.c.dim}${paths.lastJson}${ui.c.reset}`);
-          console.log(`  ${ui.c.dim}${paths.lastMd}${ui.c.reset}`);
-        }
-      } catch (err) {
-        if (this.options.verbose) {
-          console.log(`${ui.c.dim}Could not save reports: ${err.message}${ui.c.reset}`);
-        }
-      }
-    }
-    
-    // Auto-fix if requested
-    if (this.options.fix) {
-      const fixResults = await autoFixAll(this.results, this.projectPath, {
-        dryRun: this.options.fixDryRun,
-        severity: [SEVERITY.CRITICAL, SEVERITY.ERROR],
-      });
-      
-      if (!this.options.quiet && !this.options.json) {
-        ui.printAutoFixResults(fixResults);
-      }
-      
-      // Re-run affected checks after fixing
-      if (fixResults.succeeded > 0 && !this.options.fixDryRun) {
-        if (!this.options.quiet && !this.options.json) {
-          console.log(`${ui.c.cyan}Re-running diagnostics after fixes...${ui.c.reset}\n`);
-        }
-        return this.run();
-      }
-    }
-    
-    // Return exit code
-    if (summary.critical > 0) return 3;
-    if (summary.errors > 0) return 1;
-    if (summary.warnings > 0 && this.options.failOnWarn) return 2;
-    return 0;
-  }
-
-  loadDiagnostics() {
-    return getAllDiagnostics(this.projectPath);
-  }
-
-  async runDiagnostics() {
-    // Group by category for display
-    const byCategory = {};
-    for (const d of this.diagnostics) {
-      if (!byCategory[d.category]) byCategory[d.category] = [];
-      byCategory[d.category].push(d);
-    }
-    
-    // Sort categories
-    const sortedCategories = Object.keys(byCategory).sort((a, b) => {
-      const orderA = CATEGORY_META[a]?.order || 99;
-      const orderB = CATEGORY_META[b]?.order || 99;
-      return orderA - orderB;
-    });
-    
-    // Track completed diagnostic IDs for dependency resolution
-    const completed = new Set();
-    const resultMap = new Map();
-    
-    for (const category of sortedCategories) {
-      const categoryDiagnostics = byCategory[category];
-      
-      if (!this.options.quiet && !this.options.json) {
-        ui.printCategoryHeader(category);
-      }
-      
-      // Separate parallel and sequential diagnostics
-      const parallel = categoryDiagnostics.filter(d => d.parallel && !d.dependsOn?.length);
-      const sequential = categoryDiagnostics.filter(d => !d.parallel || d.dependsOn?.length);
-      
-      // Run parallel diagnostics
-      if (parallel.length > 0) {
-        const parallelPromises = parallel.map(d => this.runSingleDiagnostic(d));
-        const parallelResults = await Promise.all(parallelPromises);
-        
-        for (let i = 0; i < parallel.length; i++) {
-          const diagnostic = parallel[i];
-          const result = parallelResults[i];
-          
-          this.results.push(result);
-          completed.add(diagnostic.id);
-          resultMap.set(diagnostic.id, result);
-          
-          if (!this.options.quiet && !this.options.json) {
-            ui.printDiagnostic(result, { showFixes: this.options.verbose });
-          }
-        }
-      }
-      
-      // Run sequential diagnostics (respecting dependencies)
-      for (const diagnostic of sequential) {
-        // Check dependencies
-        if (diagnostic.dependsOn) {
-          const unmetDeps = diagnostic.dependsOn.filter(dep => !completed.has(dep));
-          if (unmetDeps.length > 0) {
-            // Skip if dependencies not met
-            const skipResult = {
-              ...diagnostic,
-              severity: SEVERITY.INFO,
-              message: `Skipped (depends on: ${unmetDeps.join(', ')})`,
-              durationMs: 0,
-            };
-            this.results.push(skipResult);
-            if (!this.options.quiet && !this.options.json) {
-              ui.printDiagnostic(skipResult);
-            }
-            continue;
-          }
-        }
-        
-        const result = await this.runSingleDiagnostic(diagnostic);
-        this.results.push(result);
-        completed.add(diagnostic.id);
-        resultMap.set(diagnostic.id, result);
-        
-        if (!this.options.quiet && !this.options.json) {
-          ui.printDiagnostic(result, { showFixes: this.options.verbose });
-        }
-      }
-      
-      if (!this.options.quiet && !this.options.json) {
-        console.log('');
-      }
-    }
-  }
-
-  async runSingleDiagnostic(diagnostic) {
-    const startTime = Date.now();
-    
-    try {
-      const checkResult = await diagnostic.check();
-      const durationMs = Date.now() - startTime;
-      
-      return {
-        id: diagnostic.id,
-        name: diagnostic.name,
-        category: diagnostic.category,
-        severity: checkResult.severity || SEVERITY.PASS,
-        message: checkResult.message || 'OK',
-        detail: checkResult.detail,
-        durationMs,
-        fixes: checkResult.fixes,
-        metadata: checkResult.metadata,
-      };
-    } catch (err) {
-      const durationMs = Date.now() - startTime;
-      
-      return {
-        id: diagnostic.id,
-        name: diagnostic.name,
-        category: diagnostic.category,
-        severity: SEVERITY.ERROR,
-        message: `Check failed: ${err.message}`,
-        durationMs,
-      };
-    }
-  }
-
-  getFixCommands() {
-    return getFixCommands(this.results);
-  }
-}
-
-async function diagnose(projectPath, options = {}) {
-  const service = new DoctorService(projectPath, options);
-  return service.run();
-}
-
-module.exports = {
-  DoctorService,
-  diagnose,
-};
+/**
+ * DoctorService — Enterprise-Grade Environment Diagnostics
+ * 
+ * Features:
+ * - Pluggable diagnostic modules
+ * - Parallel execution for independent checks
+ * - Dependency resolution between checks
+ * - Auto-fix capability
+ * - JSON/Markdown report generation
+ * - Health scoring with weighted categories
+ * - CI-friendly exit codes
+ */
+
+const path = require('path');
+const { SEVERITY, CATEGORY, CATEGORY_META } = require('./types');
+const { getAllDiagnostics } = require('./modules');
+const { createSummary, saveReports, generateJsonReport } = require('./reporter');
+const { autoFixAll, getFixCommands } = require('./autofix');
+const ui = require('./ui');
+
+class DoctorService {
+  constructor(projectPath, options = {}) {
+    this.projectPath = path.resolve(projectPath);
+    this.options = {
+      json: false,
+      markdown: false,
+      fix: false,
+      fixDryRun: false,
+      quiet: false,
+      verbose: false,
+      categories: null, // null = all categories
+      skipNetwork: false,
+      saveReport: true,
+      ...options,
+    };
+    
+    this.diagnostics = [];
+    this.results = [];
+    this.startTime = null;
+  }
+
+  async run() {
+    this.startTime = Date.now();
+    
+    // Load diagnostics
+    this.diagnostics = this.loadDiagnostics();
+    
+    // Filter by category if specified
+    if (this.options.categories) {
+      this.diagnostics = this.diagnostics.filter(d => 
+        this.options.categories.includes(d.category)
+      );
+    }
+    
+    // Skip network if requested
+    if (this.options.skipNetwork) {
+      this.diagnostics = this.diagnostics.filter(d => d.category !== CATEGORY.NETWORK);
+    }
+    
+    // Print header (unless quiet or json)
+    if (!this.options.quiet && !this.options.json) {
+      ui.printHeader();
+    }
+    
+    // Run diagnostics
+    await this.runDiagnostics();
+    
+    // Calculate summary
+    const durationMs = Date.now() - this.startTime;
+    const summary = createSummary(this.results);
+    
+    // Output results
+    if (this.options.json) {
+      // JSON output only
+      const report = generateJsonReport(this.results, this.projectPath, durationMs);
+      console.log(JSON.stringify(report, null, 2));
+    } else if (!this.options.quiet) {
+      // Print issues and summary
+      ui.printIssuesSummary(this.results);
+      ui.printFinalSummary(summary, durationMs);
+    }
+    
+    // Save reports
+    if (this.options.saveReport && !this.options.json) {
+      const outputDir = path.join(this.projectPath, '.vibecheck');
+      try {
+        const paths = saveReports(this.results, this.projectPath, durationMs, outputDir);
+        if (this.options.verbose) {
+          console.log(`${ui.c.dim}Reports saved to:${ui.c.reset}`);
+          console.log(`  ${ui.c.dim}${paths.lastJson}${ui.c.reset}`);
+          console.log(`  ${ui.c.dim}${paths.lastMd}${ui.c.reset}`);
+        }
+      } catch (err) {
+        if (this.options.verbose) {
+          console.log(`${ui.c.dim}Could not save reports: ${err.message}${ui.c.reset}`);
+        }
+      }
+    }
+    
+    // Auto-fix if requested
+    if (this.options.fix) {
+      const fixResults = await autoFixAll(this.results, this.projectPath, {
+        dryRun: this.options.fixDryRun,
+        severity: [SEVERITY.CRITICAL, SEVERITY.ERROR],
+      });
+      
+      if (!this.options.quiet && !this.options.json) {
+        ui.printAutoFixResults(fixResults);
+      }
+      
+      // Re-run affected checks after fixing
+      if (fixResults.succeeded > 0 && !this.options.fixDryRun) {
+        if (!this.options.quiet && !this.options.json) {
+          console.log(`${ui.c.cyan}Re-running diagnostics after fixes...${ui.c.reset}\n`);
+        }
+        return this.run();
+      }
+    }
+    
+    // Return exit code
+    if (summary.critical > 0) return 3;
+    if (summary.errors > 0) return 1;
+    if (summary.warnings > 0 && this.options.failOnWarn) return 2;
+    return 0;
+  }
+
+  loadDiagnostics() {
+    return getAllDiagnostics(this.projectPath);
+  }
+
+  async runDiagnostics() {
+    // Group by category for display
+    const byCategory = {};
+    for (const d of this.diagnostics) {
+      if (!byCategory[d.category]) byCategory[d.category] = [];
+      byCategory[d.category].push(d);
+    }
+    
+    // Sort categories
+    const sortedCategories = Object.keys(byCategory).sort((a, b) => {
+      const orderA = CATEGORY_META[a]?.order || 99;
+      const orderB = CATEGORY_META[b]?.order || 99;
+      return orderA - orderB;
+    });
+    
+    // Track completed diagnostic IDs for dependency resolution
+    const completed = new Set();
+    const resultMap = new Map();
+    
+    for (const category of sortedCategories) {
+      const categoryDiagnostics = byCategory[category];
+      
+      if (!this.options.quiet && !this.options.json) {
+        ui.printCategoryHeader(category);
+      }
+      
+      // Separate parallel and sequential diagnostics
+      const parallel = categoryDiagnostics.filter(d => d.parallel && !d.dependsOn?.length);
+      const sequential = categoryDiagnostics.filter(d => !d.parallel || d.dependsOn?.length);
+      
+      // Run parallel diagnostics
+      if (parallel.length > 0) {
+        const parallelPromises = parallel.map(d => this.runSingleDiagnostic(d));
+        const parallelResults = await Promise.all(parallelPromises);
+        
+        for (let i = 0; i < parallel.length; i++) {
+          const diagnostic = parallel[i];
+          const result = parallelResults[i];
+          
+          this.results.push(result);
+          completed.add(diagnostic.id);
+          resultMap.set(diagnostic.id, result);
+          
+          if (!this.options.quiet && !this.options.json) {
+            ui.printDiagnostic(result, { showFixes: this.options.verbose });
+          }
+        }
+      }
+      
+      // Run sequential diagnostics (respecting dependencies)
+      for (const diagnostic of sequential) {
+        // Check dependencies
+        if (diagnostic.dependsOn) {
+          const unmetDeps = diagnostic.dependsOn.filter(dep => !completed.has(dep));
+          if (unmetDeps.length > 0) {
+            // Skip if dependencies not met
+            const skipResult = {
+              ...diagnostic,
+              severity: SEVERITY.INFO,
+              message: `Skipped (depends on: ${unmetDeps.join(', ')})`,
+              durationMs: 0,
+            };
+            this.results.push(skipResult);
+            if (!this.options.quiet && !this.options.json) {
+              ui.printDiagnostic(skipResult);
+            }
+            continue;
+          }
+        }
+        
+        const result = await this.runSingleDiagnostic(diagnostic);
+        this.results.push(result);
+        completed.add(diagnostic.id);
+        resultMap.set(diagnostic.id, result);
+        
+        if (!this.options.quiet && !this.options.json) {
+          ui.printDiagnostic(result, { showFixes: this.options.verbose });
+        }
+      }
+      
+      if (!this.options.quiet && !this.options.json) {
+        console.log('');
+      }
+    }
+  }
+
+  async runSingleDiagnostic(diagnostic) {
+    const startTime = Date.now();
+    
+    try {
+      const checkResult = await diagnostic.check();
+      const durationMs = Date.now() - startTime;
+      
+      return {
+        id: diagnostic.id,
+        name: diagnostic.name,
+        category: diagnostic.category,
+        severity: checkResult.severity || SEVERITY.PASS,
+        message: checkResult.message || 'OK',
+        detail: checkResult.detail,
+        durationMs,
+        fixes: checkResult.fixes,
+        metadata: checkResult.metadata,
+      };
+    } catch (err) {
+      const durationMs = Date.now() - startTime;
+      
+      return {
+        id: diagnostic.id,
+        name: diagnostic.name,
+        category: diagnostic.category,
+        severity: SEVERITY.ERROR,
+        message: `Check failed: ${err.message}`,
+        durationMs,
+      };
+    }
+  }
+
+  getFixCommands() {
+    return getFixCommands(this.results);
+  }
+}
+
+async function diagnose(projectPath, options = {}) {
+  const service = new DoctorService(projectPath, options);
+  return service.run();
+}
+
+module.exports = {
+  DoctorService,
+  diagnose,
+};