@vibecheckai/cli 3.2.5 → 3.3.0

package/bin/runners/lib/doctor/modules/runtime.js

@@ -1,224 +1,224 @@
-/**
- * Runtime Diagnostics Module
- * 
- * Checks Node.js, npm, package managers, and runtime environment
- */
-
-const { execSync } = require('child_process');
-const { SEVERITY, CATEGORY, FIX_TYPE } = require('../types');
-
-const MODULE_ID = 'runtime';
-
-function createDiagnostics(projectPath) {
-  return [
-    {
-      id: `${MODULE_ID}.node_version`,
-      name: 'Node.js Version',
-      category: CATEGORY.RUNTIME,
-      parallel: true,
-      check: async () => {
-        const version = process.version;
-        const major = parseInt(version.slice(1).split('.')[0]);
-        const minor = parseInt(version.split('.')[1]);
-        
-        const metadata = { version, major, minor };
-        
-        if (major < 18) {
-          return {
-            severity: SEVERITY.ERROR,
-            message: `${version} — requires Node 18+`,
-            detail: 'vibecheck uses modern JavaScript features that require Node 18+',
-            metadata,
-            fixes: [
-              {
-                type: FIX_TYPE.COMMAND,
-                description: 'Install Node 20 via nvm',
-                command: 'nvm install 20 && nvm use 20',
-                autoFixable: false,
-              },
-              {
-                type: FIX_TYPE.LINK,
-                description: 'Download Node.js',
-                url: 'https://nodejs.org/',
-              },
-            ],
-          };
-        }
-        
-        if (major < 20) {
-          return {
-            severity: SEVERITY.WARNING,
-            message: `${version} — recommend Node 20+`,
-            detail: 'Node 20+ has better performance and LTS support',
-            metadata,
-            fixes: [{
-              type: FIX_TYPE.COMMAND,
-              description: 'Upgrade to Node 20',
-              command: 'nvm install 20 && nvm use 20',
-              autoFixable: false,
-            }],
-          };
-        }
-        
-        return {
-          severity: SEVERITY.PASS,
-          message: version,
-          metadata,
-        };
-      },
-    },
-    {
-      id: `${MODULE_ID}.npm_version`,
-      name: 'npm Version',
-      category: CATEGORY.RUNTIME,
-      parallel: true,
-      check: async () => {
-        try {
-          const version = execSync('npm --version', { 
-            encoding: 'utf8',
-            timeout: 10000,
-            stdio: ['pipe', 'pipe', 'pipe'],
-          }).trim();
-          
-          const major = parseInt(version.split('.')[0]);
-          const metadata = { version, major };
-          
-          if (major < 8) {
-            return {
-              severity: SEVERITY.WARNING,
-              message: `${version} — recommend npm 9+`,
-              metadata,
-              fixes: [{
-                type: FIX_TYPE.COMMAND,
-                description: 'Update npm',
-                command: 'npm install -g npm@latest',
-                autoFixable: true,
-              }],
-            };
-          }
-          
-          return {
-            severity: SEVERITY.PASS,
-            message: version,
-            metadata,
-          };
-        } catch (err) {
-          return {
-            severity: SEVERITY.ERROR,
-            message: 'npm not found',
-            detail: err.message,
-            fixes: [{
-              type: FIX_TYPE.LINK,
-              description: 'Install Node.js (includes npm)',
-              url: 'https://nodejs.org/',
-            }],
-          };
-        }
-      },
-    },
-    {
-      id: `${MODULE_ID}.package_manager`,
-      name: 'Package Manager',
-      category: CATEGORY.RUNTIME,
-      parallel: true,
-      check: async () => {
-        const managers = [
-          { name: 'pnpm', preferred: true },
-          { name: 'yarn', preferred: false },
-          { name: 'npm', preferred: false },
-        ];
-        
-        const detected = [];
-        
-        for (const mgr of managers) {
-          try {
-            const version = execSync(`${mgr.name} --version`, {
-              encoding: 'utf8',
-              timeout: 5000,
-              stdio: ['pipe', 'pipe', 'pipe'],
-            }).trim();
-            detected.push({ ...mgr, version });
-          } catch {
-            // Not installed
-          }
-        }
-        
-        if (detected.length === 0) {
-          return {
-            severity: SEVERITY.ERROR,
-            message: 'No package manager found',
-            fixes: [{
-              type: FIX_TYPE.COMMAND,
-              description: 'Install pnpm (recommended)',
-              command: 'npm install -g pnpm',
-              autoFixable: true,
-            }],
-          };
-        }
-        
-        const primary = detected[0];
-        const metadata = { detected, primary: primary.name };
-        
-        return {
-          severity: SEVERITY.PASS,
-          message: `${primary.name} ${primary.version}${detected.length > 1 ? ` (+${detected.length - 1} more)` : ''}`,
-          metadata,
-        };
-      },
-    },
-    {
-      id: `${MODULE_ID}.node_env`,
-      name: 'NODE_ENV',
-      category: CATEGORY.RUNTIME,
-      parallel: true,
-      check: async () => {
-        const nodeEnv = process.env.NODE_ENV;
-        const metadata = { nodeEnv };
-        
-        if (!nodeEnv) {
-          return {
-            severity: SEVERITY.INFO,
-            message: 'Not set (defaults to development)',
-            metadata,
-          };
-        }
-        
-        if (nodeEnv === 'production') {
-          return {
-            severity: SEVERITY.WARNING,
-            message: 'production — may affect diagnostic behavior',
-            detail: 'Some checks behave differently in production mode',
-            metadata,
-          };
-        }
-        
-        return {
-          severity: SEVERITY.PASS,
-          message: nodeEnv,
-          metadata,
-        };
-      },
-    },
-    {
-      id: `${MODULE_ID}.path_env`,
-      name: 'PATH Configuration',
-      category: CATEGORY.RUNTIME,
-      parallel: true,
-      check: async () => {
-        const pathEnv = process.env.PATH || process.env.Path || '';
-        const paths = pathEnv.split(process.platform === 'win32' ? ';' : ':');
-        
-        const hasNodeModulesBin = paths.some(p => p.includes('node_modules/.bin') || p.includes('node_modules\\.bin'));
-        const metadata = { pathCount: paths.length, hasNodeModulesBin };
-        
-        return {
-          severity: SEVERITY.PASS,
-          message: `${paths.length} paths configured`,
-          metadata,
-        };
-      },
-    },
-  ];
-}
-
-module.exports = { MODULE_ID, createDiagnostics };
+/**
+ * Runtime Diagnostics Module
+ * 
+ * Checks Node.js, npm, package managers, and runtime environment
+ */
+
+const { execSync } = require('child_process');
+const { SEVERITY, CATEGORY, FIX_TYPE } = require('../types');
+
+const MODULE_ID = 'runtime';
+
+function createDiagnostics(projectPath) {
+  return [
+    {
+      id: `${MODULE_ID}.node_version`,
+      name: 'Node.js Version',
+      category: CATEGORY.RUNTIME,
+      parallel: true,
+      check: async () => {
+        const version = process.version;
+        const major = parseInt(version.slice(1).split('.')[0]);
+        const minor = parseInt(version.split('.')[1]);
+        
+        const metadata = { version, major, minor };
+        
+        if (major < 18) {
+          return {
+            severity: SEVERITY.ERROR,
+            message: `${version} — requires Node 18+`,
+            detail: 'vibecheck uses modern JavaScript features that require Node 18+',
+            metadata,
+            fixes: [
+              {
+                type: FIX_TYPE.COMMAND,
+                description: 'Install Node 20 via nvm',
+                command: 'nvm install 20 && nvm use 20',
+                autoFixable: false,
+              },
+              {
+                type: FIX_TYPE.LINK,
+                description: 'Download Node.js',
+                url: 'https://nodejs.org/',
+              },
+            ],
+          };
+        }
+        
+        if (major < 20) {
+          return {
+            severity: SEVERITY.WARNING,
+            message: `${version} — recommend Node 20+`,
+            detail: 'Node 20+ has better performance and LTS support',
+            metadata,
+            fixes: [{
+              type: FIX_TYPE.COMMAND,
+              description: 'Upgrade to Node 20',
+              command: 'nvm install 20 && nvm use 20',
+              autoFixable: false,
+            }],
+          };
+        }
+        
+        return {
+          severity: SEVERITY.PASS,
+          message: version,
+          metadata,
+        };
+      },
+    },
+    {
+      id: `${MODULE_ID}.npm_version`,
+      name: 'npm Version',
+      category: CATEGORY.RUNTIME,
+      parallel: true,
+      check: async () => {
+        try {
+          const version = execSync('npm --version', { 
+            encoding: 'utf8',
+            timeout: 10000,
+            stdio: ['pipe', 'pipe', 'pipe'],
+          }).trim();
+          
+          const major = parseInt(version.split('.')[0]);
+          const metadata = { version, major };
+          
+          if (major < 8) {
+            return {
+              severity: SEVERITY.WARNING,
+              message: `${version} — recommend npm 9+`,
+              metadata,
+              fixes: [{
+                type: FIX_TYPE.COMMAND,
+                description: 'Update npm',
+                command: 'npm install -g npm@latest',
+                autoFixable: true,
+              }],
+            };
+          }
+          
+          return {
+            severity: SEVERITY.PASS,
+            message: version,
+            metadata,
+          };
+        } catch (err) {
+          return {
+            severity: SEVERITY.ERROR,
+            message: 'npm not found',
+            detail: err.message,
+            fixes: [{
+              type: FIX_TYPE.LINK,
+              description: 'Install Node.js (includes npm)',
+              url: 'https://nodejs.org/',
+            }],
+          };
+        }
+      },
+    },
+    {
+      id: `${MODULE_ID}.package_manager`,
+      name: 'Package Manager',
+      category: CATEGORY.RUNTIME,
+      parallel: true,
+      check: async () => {
+        const managers = [
+          { name: 'pnpm', preferred: true },
+          { name: 'yarn', preferred: false },
+          { name: 'npm', preferred: false },
+        ];
+        
+        const detected = [];
+        
+        for (const mgr of managers) {
+          try {
+            const version = execSync(`${mgr.name} --version`, {
+              encoding: 'utf8',
+              timeout: 5000,
+              stdio: ['pipe', 'pipe', 'pipe'],
+            }).trim();
+            detected.push({ ...mgr, version });
+          } catch {
+            // Not installed
+          }
+        }
+        
+        if (detected.length === 0) {
+          return {
+            severity: SEVERITY.ERROR,
+            message: 'No package manager found',
+            fixes: [{
+              type: FIX_TYPE.COMMAND,
+              description: 'Install pnpm (recommended)',
+              command: 'npm install -g pnpm',
+              autoFixable: true,
+            }],
+          };
+        }
+        
+        const primary = detected[0];
+        const metadata = { detected, primary: primary.name };
+        
+        return {
+          severity: SEVERITY.PASS,
+          message: `${primary.name} ${primary.version}${detected.length > 1 ? ` (+${detected.length - 1} more)` : ''}`,
+          metadata,
+        };
+      },
+    },
+    {
+      id: `${MODULE_ID}.node_env`,
+      name: 'NODE_ENV',
+      category: CATEGORY.RUNTIME,
+      parallel: true,
+      check: async () => {
+        const nodeEnv = process.env.NODE_ENV;
+        const metadata = { nodeEnv };
+        
+        if (!nodeEnv) {
+          return {
+            severity: SEVERITY.INFO,
+            message: 'Not set (defaults to development)',
+            metadata,
+          };
+        }
+        
+        if (nodeEnv === 'production') {
+          return {
+            severity: SEVERITY.WARNING,
+            message: 'production — may affect diagnostic behavior',
+            detail: 'Some checks behave differently in production mode',
+            metadata,
+          };
+        }
+        
+        return {
+          severity: SEVERITY.PASS,
+          message: nodeEnv,
+          metadata,
+        };
+      },
+    },
+    {
+      id: `${MODULE_ID}.path_env`,
+      name: 'PATH Configuration',
+      category: CATEGORY.RUNTIME,
+      parallel: true,
+      check: async () => {
+        const pathEnv = process.env.PATH || process.env.Path || '';
+        const paths = pathEnv.split(process.platform === 'win32' ? ';' : ':');
+        
+        const hasNodeModulesBin = paths.some(p => p.includes('node_modules/.bin') || p.includes('node_modules\\.bin'));
+        const metadata = { pathCount: paths.length, hasNodeModulesBin };
+        
+        return {
+          severity: SEVERITY.PASS,
+          message: `${paths.length} paths configured`,
+          metadata,
+        };
+      },
+    },
+  ];
+}
+
+module.exports = { MODULE_ID, createDiagnostics };