@vibecheckai/cli 3.0.10 → 3.1.1

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

@@ -1,254 +1,254 @@
-/**
- * Doctor Auto-Fix Engine
- * 
- * Executes fixes for diagnostic issues automatically when safe
- */
-
-const fs = require('fs');
-const path = require('path');
-const { execSync, spawn } = require('child_process');
-const { FIX_TYPE, SEVERITY } = require('./types');
-
-const FIX_RESULT = {
-  SUCCESS: 'success',
-  FAILED: 'failed',
-  SKIPPED: 'skipped',
-  REQUIRES_CONFIRMATION: 'requires_confirmation',
-};
-
-function canAutoFix(fix) {
-  if (!fix) return false;
-  if (fix.dangerous) return false;
-  if (fix.autoFixable === false) return false;
-  
-  // Only auto-fix commands and file operations
-  return [FIX_TYPE.COMMAND, FIX_TYPE.FILE_CREATE, FIX_TYPE.FILE_EDIT].includes(fix.type);
-}
-
-async function executeFix(fix, projectPath, options = {}) {
-  const { dryRun = false, interactive = false } = options;
-  
-  if (!fix) {
-    return { status: FIX_RESULT.SKIPPED, reason: 'No fix provided' };
-  }
-  
-  if (fix.dangerous && !options.allowDangerous) {
-    return { 
-      status: FIX_RESULT.REQUIRES_CONFIRMATION, 
-      reason: 'Fix is marked as dangerous',
-      fix,
-    };
-  }
-  
-  if (dryRun) {
-    return { 
-      status: FIX_RESULT.SKIPPED, 
-      reason: 'Dry run mode',
-      wouldExecute: fix,
-    };
-  }
-  
-  try {
-    switch (fix.type) {
-      case FIX_TYPE.COMMAND:
-        return await executeCommand(fix, projectPath, options);
-      
-      case FIX_TYPE.FILE_CREATE:
-        return await createFile(fix, projectPath);
-      
-      case FIX_TYPE.FILE_EDIT:
-        return await editFile(fix, projectPath);
-      
-      case FIX_TYPE.MANUAL:
-        return { 
-          status: FIX_RESULT.SKIPPED, 
-          reason: 'Manual fix required',
-          instructions: fix.description,
-        };
-      
-      case FIX_TYPE.LINK:
-        return { 
-          status: FIX_RESULT.SKIPPED, 
-          reason: 'External documentation',
-          url: fix.url,
-        };
-      
-      default:
-        return { 
-          status: FIX_RESULT.SKIPPED, 
-          reason: `Unknown fix type: ${fix.type}`,
-        };
-    }
-  } catch (err) {
-    return {
-      status: FIX_RESULT.FAILED,
-      error: err.message,
-      fix,
-    };
-  }
-}
-
-async function executeCommand(fix, projectPath, options = {}) {
-  const { timeout = 60000 } = options;
-  
-  if (!fix.command) {
-    return { status: FIX_RESULT.SKIPPED, reason: 'No command specified' };
-  }
-  
-  try {
-    const result = execSync(fix.command, {
-      cwd: projectPath,
-      encoding: 'utf8',
-      timeout,
-      stdio: ['pipe', 'pipe', 'pipe'],
-    });
-    
-    return {
-      status: FIX_RESULT.SUCCESS,
-      command: fix.command,
-      output: result.trim(),
-    };
-  } catch (err) {
-    return {
-      status: FIX_RESULT.FAILED,
-      command: fix.command,
-      error: err.message,
-      stderr: err.stderr,
-    };
-  }
-}
-
-async function createFile(fix, projectPath) {
-  if (!fix.path || fix.content === undefined) {
-    return { status: FIX_RESULT.SKIPPED, reason: 'No path or content specified' };
-  }
-  
-  const fullPath = path.isAbsolute(fix.path) ? fix.path : path.join(projectPath, fix.path);
-  
-  // Don't overwrite existing files
-  if (fs.existsSync(fullPath)) {
-    return { 
-      status: FIX_RESULT.SKIPPED, 
-      reason: 'File already exists',
-      path: fullPath,
-    };
-  }
-  
-  // Ensure directory exists
-  fs.mkdirSync(path.dirname(fullPath), { recursive: true });
-  
-  // Write file
-  fs.writeFileSync(fullPath, fix.content);
-  
-  return {
-    status: FIX_RESULT.SUCCESS,
-    action: 'created',
-    path: fullPath,
-  };
-}
-
-async function editFile(fix, projectPath) {
-  if (!fix.path || fix.content === undefined) {
-    return { status: FIX_RESULT.SKIPPED, reason: 'No path or content specified' };
-  }
-  
-  const fullPath = path.isAbsolute(fix.path) ? fix.path : path.join(projectPath, fix.path);
-  
-  // Backup existing file
-  if (fs.existsSync(fullPath)) {
-    const backupPath = `${fullPath}.doctor-backup`;
-    fs.copyFileSync(fullPath, backupPath);
-  }
-  
-  // Write new content
-  fs.writeFileSync(fullPath, fix.content);
-  
-  return {
-    status: FIX_RESULT.SUCCESS,
-    action: 'edited',
-    path: fullPath,
-  };
-}
-
-async function autoFixAll(diagnostics, projectPath, options = {}) {
-  const { 
-    dryRun = false, 
-    severity = [SEVERITY.CRITICAL, SEVERITY.ERROR],
-    maxFixes = 10,
-  } = options;
-  
-  const results = [];
-  let fixCount = 0;
-  
-  // Sort by severity (critical first)
-  const severityOrder = [SEVERITY.CRITICAL, SEVERITY.ERROR, SEVERITY.WARNING];
-  const sortedDiagnostics = [...diagnostics].sort((a, b) => {
-    return severityOrder.indexOf(a.severity) - severityOrder.indexOf(b.severity);
-  });
-  
-  for (const diagnostic of sortedDiagnostics) {
-    if (fixCount >= maxFixes) break;
-    if (!severity.includes(diagnostic.severity)) continue;
-    if (!diagnostic.fixes || diagnostic.fixes.length === 0) continue;
-    
-    // Try the first auto-fixable fix
-    const fix = diagnostic.fixes.find(f => canAutoFix(f));
-    if (!fix) continue;
-    
-    const result = await executeFix(fix, projectPath, { ...options, dryRun });
-    results.push({
-      diagnosticId: diagnostic.id,
-      diagnosticName: diagnostic.name,
-      ...result,
-    });
-    
-    if (result.status === FIX_RESULT.SUCCESS) {
-      fixCount++;
-    }
-  }
-  
-  return {
-    attempted: results.length,
-    succeeded: results.filter(r => r.status === FIX_RESULT.SUCCESS).length,
-    failed: results.filter(r => r.status === FIX_RESULT.FAILED).length,
-    skipped: results.filter(r => r.status === FIX_RESULT.SKIPPED).length,
-    results,
-  };
-}
-
-function getFixCommands(diagnostics, options = {}) {
-  const { severity = [SEVERITY.CRITICAL, SEVERITY.ERROR, SEVERITY.WARNING] } = options;
-  
-  const commands = [];
-  
-  for (const diagnostic of diagnostics) {
-    if (!severity.includes(diagnostic.severity)) continue;
-    if (!diagnostic.fixes) continue;
-    
-    for (const fix of diagnostic.fixes) {
-      if (fix.type === FIX_TYPE.COMMAND && fix.command) {
-        commands.push({
-          diagnosticId: diagnostic.id,
-          diagnosticName: diagnostic.name,
-          command: fix.command,
-          description: fix.description,
-          dangerous: fix.dangerous,
-          autoFixable: canAutoFix(fix),
-        });
-      }
-    }
-  }
-  
-  return commands;
-}
-
-module.exports = {
-  FIX_RESULT,
-  canAutoFix,
-  executeFix,
-  executeCommand,
-  createFile,
-  editFile,
-  autoFixAll,
-  getFixCommands,
-};
+/**
+ * Doctor Auto-Fix Engine
+ * 
+ * Executes fixes for diagnostic issues automatically when safe
+ */
+
+const fs = require('fs');
+const path = require('path');
+const { execSync, spawn } = require('child_process');
+const { FIX_TYPE, SEVERITY } = require('./types');
+
+const FIX_RESULT = {
+  SUCCESS: 'success',
+  FAILED: 'failed',
+  SKIPPED: 'skipped',
+  REQUIRES_CONFIRMATION: 'requires_confirmation',
+};
+
+function canAutoFix(fix) {
+  if (!fix) return false;
+  if (fix.dangerous) return false;
+  if (fix.autoFixable === false) return false;
+  
+  // Only auto-fix commands and file operations
+  return [FIX_TYPE.COMMAND, FIX_TYPE.FILE_CREATE, FIX_TYPE.FILE_EDIT].includes(fix.type);
+}
+
+async function executeFix(fix, projectPath, options = {}) {
+  const { dryRun = false, interactive = false } = options;
+  
+  if (!fix) {
+    return { status: FIX_RESULT.SKIPPED, reason: 'No fix provided' };
+  }
+  
+  if (fix.dangerous && !options.allowDangerous) {
+    return { 
+      status: FIX_RESULT.REQUIRES_CONFIRMATION, 
+      reason: 'Fix is marked as dangerous',
+      fix,
+    };
+  }
+  
+  if (dryRun) {
+    return { 
+      status: FIX_RESULT.SKIPPED, 
+      reason: 'Dry run mode',
+      wouldExecute: fix,
+    };
+  }
+  
+  try {
+    switch (fix.type) {
+      case FIX_TYPE.COMMAND:
+        return await executeCommand(fix, projectPath, options);
+      
+      case FIX_TYPE.FILE_CREATE:
+        return await createFile(fix, projectPath);
+      
+      case FIX_TYPE.FILE_EDIT:
+        return await editFile(fix, projectPath);
+      
+      case FIX_TYPE.MANUAL:
+        return { 
+          status: FIX_RESULT.SKIPPED, 
+          reason: 'Manual fix required',
+          instructions: fix.description,
+        };
+      
+      case FIX_TYPE.LINK:
+        return { 
+          status: FIX_RESULT.SKIPPED, 
+          reason: 'External documentation',
+          url: fix.url,
+        };
+      
+      default:
+        return { 
+          status: FIX_RESULT.SKIPPED, 
+          reason: `Unknown fix type: ${fix.type}`,
+        };
+    }
+  } catch (err) {
+    return {
+      status: FIX_RESULT.FAILED,
+      error: err.message,
+      fix,
+    };
+  }
+}
+
+async function executeCommand(fix, projectPath, options = {}) {
+  const { timeout = 60000 } = options;
+  
+  if (!fix.command) {
+    return { status: FIX_RESULT.SKIPPED, reason: 'No command specified' };
+  }
+  
+  try {
+    const result = execSync(fix.command, {
+      cwd: projectPath,
+      encoding: 'utf8',
+      timeout,
+      stdio: ['pipe', 'pipe', 'pipe'],
+    });
+    
+    return {
+      status: FIX_RESULT.SUCCESS,
+      command: fix.command,
+      output: result.trim(),
+    };
+  } catch (err) {
+    return {
+      status: FIX_RESULT.FAILED,
+      command: fix.command,
+      error: err.message,
+      stderr: err.stderr,
+    };
+  }
+}
+
+async function createFile(fix, projectPath) {
+  if (!fix.path || fix.content === undefined) {
+    return { status: FIX_RESULT.SKIPPED, reason: 'No path or content specified' };
+  }
+  
+  const fullPath = path.isAbsolute(fix.path) ? fix.path : path.join(projectPath, fix.path);
+  
+  // Don't overwrite existing files
+  if (fs.existsSync(fullPath)) {
+    return { 
+      status: FIX_RESULT.SKIPPED, 
+      reason: 'File already exists',
+      path: fullPath,
+    };
+  }
+  
+  // Ensure directory exists
+  fs.mkdirSync(path.dirname(fullPath), { recursive: true });
+  
+  // Write file
+  fs.writeFileSync(fullPath, fix.content);
+  
+  return {
+    status: FIX_RESULT.SUCCESS,
+    action: 'created',
+    path: fullPath,
+  };
+}
+
+async function editFile(fix, projectPath) {
+  if (!fix.path || fix.content === undefined) {
+    return { status: FIX_RESULT.SKIPPED, reason: 'No path or content specified' };
+  }
+  
+  const fullPath = path.isAbsolute(fix.path) ? fix.path : path.join(projectPath, fix.path);
+  
+  // Backup existing file
+  if (fs.existsSync(fullPath)) {
+    const backupPath = `${fullPath}.doctor-backup`;
+    fs.copyFileSync(fullPath, backupPath);
+  }
+  
+  // Write new content
+  fs.writeFileSync(fullPath, fix.content);
+  
+  return {
+    status: FIX_RESULT.SUCCESS,
+    action: 'edited',
+    path: fullPath,
+  };
+}
+
+async function autoFixAll(diagnostics, projectPath, options = {}) {
+  const { 
+    dryRun = false, 
+    severity = [SEVERITY.CRITICAL, SEVERITY.ERROR],
+    maxFixes = 10,
+  } = options;
+  
+  const results = [];
+  let fixCount = 0;
+  
+  // Sort by severity (critical first)
+  const severityOrder = [SEVERITY.CRITICAL, SEVERITY.ERROR, SEVERITY.WARNING];
+  const sortedDiagnostics = [...diagnostics].sort((a, b) => {
+    return severityOrder.indexOf(a.severity) - severityOrder.indexOf(b.severity);
+  });
+  
+  for (const diagnostic of sortedDiagnostics) {
+    if (fixCount >= maxFixes) break;
+    if (!severity.includes(diagnostic.severity)) continue;
+    if (!diagnostic.fixes || diagnostic.fixes.length === 0) continue;
+    
+    // Try the first auto-fixable fix
+    const fix = diagnostic.fixes.find(f => canAutoFix(f));
+    if (!fix) continue;
+    
+    const result = await executeFix(fix, projectPath, { ...options, dryRun });
+    results.push({
+      diagnosticId: diagnostic.id,
+      diagnosticName: diagnostic.name,
+      ...result,
+    });
+    
+    if (result.status === FIX_RESULT.SUCCESS) {
+      fixCount++;
+    }
+  }
+  
+  return {
+    attempted: results.length,
+    succeeded: results.filter(r => r.status === FIX_RESULT.SUCCESS).length,
+    failed: results.filter(r => r.status === FIX_RESULT.FAILED).length,
+    skipped: results.filter(r => r.status === FIX_RESULT.SKIPPED).length,
+    results,
+  };
+}
+
+function getFixCommands(diagnostics, options = {}) {
+  const { severity = [SEVERITY.CRITICAL, SEVERITY.ERROR, SEVERITY.WARNING] } = options;
+  
+  const commands = [];
+  
+  for (const diagnostic of diagnostics) {
+    if (!severity.includes(diagnostic.severity)) continue;
+    if (!diagnostic.fixes) continue;
+    
+    for (const fix of diagnostic.fixes) {
+      if (fix.type === FIX_TYPE.COMMAND && fix.command) {
+        commands.push({
+          diagnosticId: diagnostic.id,
+          diagnosticName: diagnostic.name,
+          command: fix.command,
+          description: fix.description,
+          dangerous: fix.dangerous,
+          autoFixable: canAutoFix(fix),
+        });
+      }
+    }
+  }
+  
+  return commands;
+}
+
+module.exports = {
+  FIX_RESULT,
+  canAutoFix,
+  executeFix,
+  executeCommand,
+  createFile,
+  editFile,
+  autoFixAll,
+  getFixCommands,
+};

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

@@ -1,37 +1,37 @@
-/**
- * Doctor Service — Enterprise Environment Diagnostics
- * 
- * Main entry point for the Doctor service
- */
-
-const { DoctorService, diagnose } = require('./service');
-const { SEVERITY, CATEGORY, FIX_TYPE, SEVERITY_WEIGHT, CATEGORY_META } = require('./types');
-const modules = require('./modules');
-const reporter = require('./reporter');
-const autofix = require('./autofix');
-const ui = require('./ui');
-
-module.exports = {
-  // Main service
-  DoctorService,
-  diagnose,
-  
-  // Types and constants
-  SEVERITY,
-  CATEGORY,
-  FIX_TYPE,
-  SEVERITY_WEIGHT,
-  CATEGORY_META,
-  
-  // Modules
-  modules,
-  
-  // Reporter
-  reporter,
-  
-  // Auto-fix
-  autofix,
-  
-  // UI utilities
-  ui,
-};
+/**
+ * Doctor Service — Enterprise Environment Diagnostics
+ * 
+ * Main entry point for the Doctor service
+ */
+
+const { DoctorService, diagnose } = require('./service');
+const { SEVERITY, CATEGORY, FIX_TYPE, SEVERITY_WEIGHT, CATEGORY_META } = require('./types');
+const modules = require('./modules');
+const reporter = require('./reporter');
+const autofix = require('./autofix');
+const ui = require('./ui');
+
+module.exports = {
+  // Main service
+  DoctorService,
+  diagnose,
+  
+  // Types and constants
+  SEVERITY,
+  CATEGORY,
+  FIX_TYPE,
+  SEVERITY_WEIGHT,
+  CATEGORY_META,
+  
+  // Modules
+  modules,
+  
+  // Reporter
+  reporter,
+  
+  // Auto-fix
+  autofix,
+  
+  // UI utilities
+  ui,
+};