@vibecheckai/cli 3.2.5 → 3.3.0

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

@@ -1,312 +1,312 @@
-/**
- * Project Diagnostics Module
- * 
- * Checks project structure, configuration, and build capability
- */
-
-const fs = require('fs');
-const path = require('path');
-const { execSync } = require('child_process');
-const { SEVERITY, CATEGORY, FIX_TYPE } = require('../types');
-
-const MODULE_ID = 'project';
-
-function createDiagnostics(projectPath) {
-  return [
-    {
-      id: `${MODULE_ID}.package_json`,
-      name: 'package.json',
-      category: CATEGORY.PROJECT,
-      parallel: true,
-      check: async () => {
-        const pkgPath = path.join(projectPath, 'package.json');
-        
-        if (!fs.existsSync(pkgPath)) {
-          return {
-            severity: SEVERITY.ERROR,
-            message: 'Not found',
-            detail: 'This doesn\'t appear to be a Node.js project',
-            fixes: [
-              {
-                type: FIX_TYPE.COMMAND,
-                description: 'Initialize a new Node.js project',
-                command: 'npm init -y',
-                autoFixable: true,
-              },
-            ],
-          };
-        }
-        
-        try {
-          const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
-          const metadata = {
-            name: pkg.name,
-            version: pkg.version,
-            hasScripts: !!pkg.scripts,
-            hasDeps: !!(pkg.dependencies || pkg.devDependencies),
-          };
-          
-          return {
-            severity: SEVERITY.PASS,
-            message: `${pkg.name || 'unnamed'}@${pkg.version || '0.0.0'}`,
-            metadata,
-          };
-        } catch (err) {
-          return {
-            severity: SEVERITY.ERROR,
-            message: 'Invalid JSON',
-            detail: err.message,
-            fixes: [{
-              type: FIX_TYPE.MANUAL,
-              description: 'Fix JSON syntax errors in package.json',
-              autoFixable: false,
-            }],
-          };
-        }
-      },
-    },
-    {
-      id: `${MODULE_ID}.node_modules`,
-      name: 'Dependencies Installed',
-      category: CATEGORY.PROJECT,
-      parallel: true,
-      dependsOn: [`${MODULE_ID}.package_json`],
-      check: async () => {
-        const nodeModulesPath = path.join(projectPath, 'node_modules');
-        
-        if (!fs.existsSync(nodeModulesPath)) {
-          return {
-            severity: SEVERITY.WARNING,
-            message: 'node_modules not found',
-            detail: 'Run your package manager install command',
-            fixes: [
-              {
-                type: FIX_TYPE.COMMAND,
-                description: 'Install dependencies with pnpm',
-                command: 'pnpm install',
-                autoFixable: true,
-              },
-              {
-                type: FIX_TYPE.COMMAND,
-                description: 'Install dependencies with npm',
-                command: 'npm install',
-                autoFixable: true,
-              },
-            ],
-          };
-        }
-        
-        try {
-          const entries = fs.readdirSync(nodeModulesPath).filter(f => !f.startsWith('.'));
-          const count = entries.length;
-          
-          // Check for common issues
-          const lockFiles = ['pnpm-lock.yaml', 'package-lock.json', 'yarn.lock']
-            .filter(f => fs.existsSync(path.join(projectPath, f)));
-          
-          const metadata = { packageCount: count, lockFiles };
-          
-          if (lockFiles.length > 1) {
-            return {
-              severity: SEVERITY.WARNING,
-              message: `${count} packages (multiple lock files detected)`,
-              detail: `Found: ${lockFiles.join(', ')}`,
-              metadata,
-              fixes: [{
-                type: FIX_TYPE.MANUAL,
-                description: 'Remove extra lock files to avoid conflicts',
-                autoFixable: false,
-              }],
-            };
-          }
-          
-          return {
-            severity: SEVERITY.PASS,
-            message: `${count} packages installed`,
-            metadata,
-          };
-        } catch {
-          return {
-            severity: SEVERITY.WARNING,
-            message: 'Cannot read node_modules',
-          };
-        }
-      },
-    },
-    {
-      id: `${MODULE_ID}.typescript`,
-      name: 'TypeScript',
-      category: CATEGORY.PROJECT,
-      parallel: true,
-      check: async () => {
-        const tsConfigPath = path.join(projectPath, 'tsconfig.json');
-        
-        if (!fs.existsSync(tsConfigPath)) {
-          return {
-            severity: SEVERITY.INFO,
-            message: 'Not configured (JavaScript project)',
-          };
-        }
-        
-        try {
-          const tsConfig = JSON.parse(fs.readFileSync(tsConfigPath, 'utf8'));
-          const strict = tsConfig.compilerOptions?.strict === true;
-          
-          const metadata = {
-            strict,
-            target: tsConfig.compilerOptions?.target,
-            module: tsConfig.compilerOptions?.module,
-          };
-          
-          if (!strict) {
-            return {
-              severity: SEVERITY.INFO,
-              message: 'Configured (strict mode disabled)',
-              metadata,
-            };
-          }
-          
-          return {
-            severity: SEVERITY.PASS,
-            message: 'Configured with strict mode',
-            metadata,
-          };
-        } catch {
-          return {
-            severity: SEVERITY.WARNING,
-            message: 'Invalid tsconfig.json',
-            fixes: [{
-              type: FIX_TYPE.COMMAND,
-              description: 'Regenerate TypeScript config',
-              command: 'npx tsc --init',
-              autoFixable: false,
-            }],
-          };
-        }
-      },
-    },
-    {
-      id: `${MODULE_ID}.source_structure`,
-      name: 'Source Structure',
-      category: CATEGORY.PROJECT,
-      parallel: true,
-      check: async () => {
-        const structures = [
-          { dir: 'src', type: 'standard' },
-          { dir: 'app', type: 'next-app-router' },
-          { dir: 'pages', type: 'next-pages-router' },
-          { dir: 'lib', type: 'library' },
-          { dir: 'packages', type: 'monorepo' },
-        ];
-        
-        const found = structures.filter(s => 
-          fs.existsSync(path.join(projectPath, s.dir))
-        );
-        
-        if (found.length === 0) {
-          return {
-            severity: SEVERITY.INFO,
-            message: 'No standard source directory detected',
-            detail: 'Consider organizing code in src/, app/, or pages/',
-          };
-        }
-        
-        const metadata = { structures: found.map(f => f.type) };
-        const primary = found[0];
-        
-        return {
-          severity: SEVERITY.PASS,
-          message: `${primary.dir}/ (${primary.type})`,
-          metadata,
-        };
-      },
-    },
-    {
-      id: `${MODULE_ID}.framework`,
-      name: 'Framework Detection',
-      category: CATEGORY.PROJECT,
-      parallel: true,
-      dependsOn: [`${MODULE_ID}.package_json`],
-      check: async () => {
-        const pkgPath = path.join(projectPath, 'package.json');
-        
-        try {
-          const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
-          const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-          
-          const frameworks = [
-            { name: 'Next.js', packages: ['next'] },
-            { name: 'Remix', packages: ['@remix-run/node', '@remix-run/react'] },
-            { name: 'Fastify', packages: ['fastify'] },
-            { name: 'Express', packages: ['express'] },
-            { name: 'Hono', packages: ['hono'] },
-            { name: 'NestJS', packages: ['@nestjs/core'] },
-            { name: 'React', packages: ['react'] },
-            { name: 'Vue', packages: ['vue'] },
-            { name: 'Svelte', packages: ['svelte'] },
-          ];
-          
-          const detected = frameworks.filter(fw =>
-            fw.packages.some(p => deps[p])
-          );
-          
-          if (detected.length === 0) {
-            return {
-              severity: SEVERITY.INFO,
-              message: 'No known framework detected',
-            };
-          }
-          
-          const metadata = { frameworks: detected.map(f => f.name) };
-          
-          return {
-            severity: SEVERITY.PASS,
-            message: detected.map(f => f.name).join(' + '),
-            metadata,
-          };
-        } catch {
-          return {
-            severity: SEVERITY.INFO,
-            message: 'Could not detect framework',
-          };
-        }
-      },
-    },
-    {
-      id: `${MODULE_ID}.build`,
-      name: 'Build Script',
-      category: CATEGORY.PROJECT,
-      parallel: true,
-      dependsOn: [`${MODULE_ID}.package_json`],
-      check: async () => {
-        const pkgPath = path.join(projectPath, 'package.json');
-        
-        try {
-          const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
-          
-          if (!pkg.scripts?.build) {
-            return {
-              severity: SEVERITY.INFO,
-              message: 'No build script defined',
-            };
-          }
-          
-          const metadata = { buildCommand: pkg.scripts.build };
-          
-          return {
-            severity: SEVERITY.PASS,
-            message: `Defined: ${pkg.scripts.build.substring(0, 40)}${pkg.scripts.build.length > 40 ? '...' : ''}`,
-            metadata,
-          };
-        } catch {
-          return {
-            severity: SEVERITY.INFO,
-            message: 'Could not check build script',
-          };
-        }
-      },
-    },
-  ];
-}
-
-module.exports = { MODULE_ID, createDiagnostics };
+/**
+ * Project Diagnostics Module
+ * 
+ * Checks project structure, configuration, and build capability
+ */
+
+const fs = require('fs');
+const path = require('path');
+const { execSync } = require('child_process');
+const { SEVERITY, CATEGORY, FIX_TYPE } = require('../types');
+
+const MODULE_ID = 'project';
+
+function createDiagnostics(projectPath) {
+  return [
+    {
+      id: `${MODULE_ID}.package_json`,
+      name: 'package.json',
+      category: CATEGORY.PROJECT,
+      parallel: true,
+      check: async () => {
+        const pkgPath = path.join(projectPath, 'package.json');
+        
+        if (!fs.existsSync(pkgPath)) {
+          return {
+            severity: SEVERITY.ERROR,
+            message: 'Not found',
+            detail: 'This doesn\'t appear to be a Node.js project',
+            fixes: [
+              {
+                type: FIX_TYPE.COMMAND,
+                description: 'Initialize a new Node.js project',
+                command: 'npm init -y',
+                autoFixable: true,
+              },
+            ],
+          };
+        }
+        
+        try {
+          const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
+          const metadata = {
+            name: pkg.name,
+            version: pkg.version,
+            hasScripts: !!pkg.scripts,
+            hasDeps: !!(pkg.dependencies || pkg.devDependencies),
+          };
+          
+          return {
+            severity: SEVERITY.PASS,
+            message: `${pkg.name || 'unnamed'}@${pkg.version || '0.0.0'}`,
+            metadata,
+          };
+        } catch (err) {
+          return {
+            severity: SEVERITY.ERROR,
+            message: 'Invalid JSON',
+            detail: err.message,
+            fixes: [{
+              type: FIX_TYPE.MANUAL,
+              description: 'Fix JSON syntax errors in package.json',
+              autoFixable: false,
+            }],
+          };
+        }
+      },
+    },
+    {
+      id: `${MODULE_ID}.node_modules`,
+      name: 'Dependencies Installed',
+      category: CATEGORY.PROJECT,
+      parallel: true,
+      dependsOn: [`${MODULE_ID}.package_json`],
+      check: async () => {
+        const nodeModulesPath = path.join(projectPath, 'node_modules');
+        
+        if (!fs.existsSync(nodeModulesPath)) {
+          return {
+            severity: SEVERITY.WARNING,
+            message: 'node_modules not found',
+            detail: 'Run your package manager install command',
+            fixes: [
+              {
+                type: FIX_TYPE.COMMAND,
+                description: 'Install dependencies with pnpm',
+                command: 'pnpm install',
+                autoFixable: true,
+              },
+              {
+                type: FIX_TYPE.COMMAND,
+                description: 'Install dependencies with npm',
+                command: 'npm install',
+                autoFixable: true,
+              },
+            ],
+          };
+        }
+        
+        try {
+          const entries = fs.readdirSync(nodeModulesPath).filter(f => !f.startsWith('.'));
+          const count = entries.length;
+          
+          // Check for common issues
+          const lockFiles = ['pnpm-lock.yaml', 'package-lock.json', 'yarn.lock']
+            .filter(f => fs.existsSync(path.join(projectPath, f)));
+          
+          const metadata = { packageCount: count, lockFiles };
+          
+          if (lockFiles.length > 1) {
+            return {
+              severity: SEVERITY.WARNING,
+              message: `${count} packages (multiple lock files detected)`,
+              detail: `Found: ${lockFiles.join(', ')}`,
+              metadata,
+              fixes: [{
+                type: FIX_TYPE.MANUAL,
+                description: 'Remove extra lock files to avoid conflicts',
+                autoFixable: false,
+              }],
+            };
+          }
+          
+          return {
+            severity: SEVERITY.PASS,
+            message: `${count} packages installed`,
+            metadata,
+          };
+        } catch {
+          return {
+            severity: SEVERITY.WARNING,
+            message: 'Cannot read node_modules',
+          };
+        }
+      },
+    },
+    {
+      id: `${MODULE_ID}.typescript`,
+      name: 'TypeScript',
+      category: CATEGORY.PROJECT,
+      parallel: true,
+      check: async () => {
+        const tsConfigPath = path.join(projectPath, 'tsconfig.json');
+        
+        if (!fs.existsSync(tsConfigPath)) {
+          return {
+            severity: SEVERITY.INFO,
+            message: 'Not configured (JavaScript project)',
+          };
+        }
+        
+        try {
+          const tsConfig = JSON.parse(fs.readFileSync(tsConfigPath, 'utf8'));
+          const strict = tsConfig.compilerOptions?.strict === true;
+          
+          const metadata = {
+            strict,
+            target: tsConfig.compilerOptions?.target,
+            module: tsConfig.compilerOptions?.module,
+          };
+          
+          if (!strict) {
+            return {
+              severity: SEVERITY.INFO,
+              message: 'Configured (strict mode disabled)',
+              metadata,
+            };
+          }
+          
+          return {
+            severity: SEVERITY.PASS,
+            message: 'Configured with strict mode',
+            metadata,
+          };
+        } catch {
+          return {
+            severity: SEVERITY.WARNING,
+            message: 'Invalid tsconfig.json',
+            fixes: [{
+              type: FIX_TYPE.COMMAND,
+              description: 'Regenerate TypeScript config',
+              command: 'npx tsc --init',
+              autoFixable: false,
+            }],
+          };
+        }
+      },
+    },
+    {
+      id: `${MODULE_ID}.source_structure`,
+      name: 'Source Structure',
+      category: CATEGORY.PROJECT,
+      parallel: true,
+      check: async () => {
+        const structures = [
+          { dir: 'src', type: 'standard' },
+          { dir: 'app', type: 'next-app-router' },
+          { dir: 'pages', type: 'next-pages-router' },
+          { dir: 'lib', type: 'library' },
+          { dir: 'packages', type: 'monorepo' },
+        ];
+        
+        const found = structures.filter(s => 
+          fs.existsSync(path.join(projectPath, s.dir))
+        );
+        
+        if (found.length === 0) {
+          return {
+            severity: SEVERITY.INFO,
+            message: 'No standard source directory detected',
+            detail: 'Consider organizing code in src/, app/, or pages/',
+          };
+        }
+        
+        const metadata = { structures: found.map(f => f.type) };
+        const primary = found[0];
+        
+        return {
+          severity: SEVERITY.PASS,
+          message: `${primary.dir}/ (${primary.type})`,
+          metadata,
+        };
+      },
+    },
+    {
+      id: `${MODULE_ID}.framework`,
+      name: 'Framework Detection',
+      category: CATEGORY.PROJECT,
+      parallel: true,
+      dependsOn: [`${MODULE_ID}.package_json`],
+      check: async () => {
+        const pkgPath = path.join(projectPath, 'package.json');
+        
+        try {
+          const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
+          const deps = { ...pkg.dependencies, ...pkg.devDependencies };
+          
+          const frameworks = [
+            { name: 'Next.js', packages: ['next'] },
+            { name: 'Remix', packages: ['@remix-run/node', '@remix-run/react'] },
+            { name: 'Fastify', packages: ['fastify'] },
+            { name: 'Express', packages: ['express'] },
+            { name: 'Hono', packages: ['hono'] },
+            { name: 'NestJS', packages: ['@nestjs/core'] },
+            { name: 'React', packages: ['react'] },
+            { name: 'Vue', packages: ['vue'] },
+            { name: 'Svelte', packages: ['svelte'] },
+          ];
+          
+          const detected = frameworks.filter(fw =>
+            fw.packages.some(p => deps[p])
+          );
+          
+          if (detected.length === 0) {
+            return {
+              severity: SEVERITY.INFO,
+              message: 'No known framework detected',
+            };
+          }
+          
+          const metadata = { frameworks: detected.map(f => f.name) };
+          
+          return {
+            severity: SEVERITY.PASS,
+            message: detected.map(f => f.name).join(' + '),
+            metadata,
+          };
+        } catch {
+          return {
+            severity: SEVERITY.INFO,
+            message: 'Could not detect framework',
+          };
+        }
+      },
+    },
+    {
+      id: `${MODULE_ID}.build`,
+      name: 'Build Script',
+      category: CATEGORY.PROJECT,
+      parallel: true,
+      dependsOn: [`${MODULE_ID}.package_json`],
+      check: async () => {
+        const pkgPath = path.join(projectPath, 'package.json');
+        
+        try {
+          const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
+          
+          if (!pkg.scripts?.build) {
+            return {
+              severity: SEVERITY.INFO,
+              message: 'No build script defined',
+            };
+          }
+          
+          const metadata = { buildCommand: pkg.scripts.build };
+          
+          return {
+            severity: SEVERITY.PASS,
+            message: `Defined: ${pkg.scripts.build.substring(0, 40)}${pkg.scripts.build.length > 40 ? '...' : ''}`,
+            metadata,
+          };
+        } catch {
+          return {
+            severity: SEVERITY.INFO,
+            message: 'Could not check build script',
+          };
+        }
+      },
+    },
+  ];
+}
+
+module.exports = { MODULE_ID, createDiagnostics };