@vibecheckai/cli 3.5.0 → 3.5.2

package/bin/runners/lib/engines/framework-adapters/index.js

@@ -1,607 +0,0 @@
-/**
- * Framework Adapters - Plugin Architecture for Framework-Specific Detection
- * 
- * This module provides a plugin system for framework-specific rules and patterns.
- * Each adapter can:
- * 
- * 1. Define framework-specific detection patterns
- * 2. Override or suppress findings based on framework conventions
- * 3. Add framework-specific checks
- * 4. Provide framework-specific fix suggestions
- * 
- * To add a new framework:
- * 1. Create a new adapter file in this directory (e.g., remix-adapter.js)
- * 2. Export it from this index.js
- * 3. The adapter will be auto-loaded based on detected framework
- */
-
-"use strict";
-
-const fs = require("fs");
-const path = require("path");
-
-// ═══════════════════════════════════════════════════════════════════════════════
-// ADAPTER INTERFACE
-// ═══════════════════════════════════════════════════════════════════════════════
-
-/**
- * Base adapter interface - all adapters should implement these methods
- */
-const AdapterInterface = {
-  // Metadata
-  name: "unknown",
-  version: "1.0.0",
-  frameworks: [], // e.g., ["nextjs", "next"]
-  
-  // Detection
-  detect: (projectPath) => false,
-  getVersion: (projectPath) => null,
-  
-  // Finding modification
-  shouldSuppressFinding: (finding, context) => ({ suppress: false, reason: null }),
-  modifyFinding: (finding, context) => finding,
-  
-  // Additional checks
-  getAdditionalChecks: () => [],
-  
-  // Fix suggestions
-  getFixSuggestion: (finding, context) => null,
-};
-
-// ═══════════════════════════════════════════════════════════════════════════════
-// BUILT-IN ADAPTERS
-// ═══════════════════════════════════════════════════════════════════════════════
-
-/**
- * Next.js Adapter - Handles Next.js specific patterns
- */
-const NextjsAdapter = {
-  name: "nextjs",
-  version: "1.0.0",
-  frameworks: ["nextjs", "next"],
-  
-  detect(projectPath) {
-    try {
-      const pkgPath = path.join(projectPath, "package.json");
-      if (fs.existsSync(pkgPath)) {
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
-        const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-        return !!deps.next;
-      }
-    } catch {}
-    return false;
-  },
-  
-  getVersion(projectPath) {
-    try {
-      const pkgPath = path.join(projectPath, "package.json");
-      if (fs.existsSync(pkgPath)) {
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
-        const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-        const version = deps.next;
-        if (version) {
-          const match = version.match(/(\d+)\.(\d+)/);
-          if (match) {
-            return { major: parseInt(match[1]), minor: parseInt(match[2]), raw: version };
-          }
-        }
-      }
-    } catch {}
-    return null;
-  },
-  
-  shouldSuppressFinding(finding, context) {
-    const { filePath, version } = context;
-    
-    // Next.js 13+ App Router: async components are valid
-    if (version?.major >= 13) {
-      if (finding.type === "async-without-await" && /\/app\//.test(filePath)) {
-        // Server components can be async without await
-        return { suppress: true, reason: "Async server components are valid in Next.js 13+ App Router" };
-      }
-      
-      if (finding.type === "top-level-await" && /\/app\//.test(filePath)) {
-        return { suppress: true, reason: "Top-level await is valid in Next.js 13+ server components" };
-      }
-    }
-    
-    // Next.js API routes: exports are expected patterns
-    if (/\/api\//.test(filePath) || /\/app\/api\//.test(filePath)) {
-      if (finding.type === "unused-export" && /^(GET|POST|PUT|DELETE|PATCH|HEAD|OPTIONS)$/.test(finding.name)) {
-        return { suppress: true, reason: "HTTP method exports are standard Next.js API route patterns" };
-      }
-    }
-    
-    // Next.js page exports
-    if (/\/pages\//.test(filePath) || /\/app\/.*page\.(tsx?|jsx?)$/.test(filePath)) {
-      if (finding.type === "unused-export" && /^(getStaticProps|getServerSideProps|getStaticPaths|generateStaticParams|generateMetadata)$/.test(finding.name)) {
-        return { suppress: true, reason: "Next.js data fetching exports are framework conventions" };
-      }
-    }
-    
-    // Server-only imports
-    if (finding.type === "import-server-only" && version?.major >= 13) {
-      return { suppress: true, reason: "server-only imports are valid in Next.js 13+ for server components" };
-    }
-    
-    return { suppress: false, reason: null };
-  },
-  
-  modifyFinding(finding, context) {
-    // Add Next.js specific context to findings
-    if (context.version?.major >= 13 && /\/app\//.test(context.filePath)) {
-      finding.frameworkContext = "Next.js App Router";
-    } else if (/\/pages\//.test(context.filePath)) {
-      finding.frameworkContext = "Next.js Pages Router";
-    }
-    return finding;
-  },
-  
-  getAdditionalChecks() {
-    return [
-      {
-        id: "nextjs-missing-metadata",
-        name: "Missing Metadata",
-        description: "Page is missing metadata export for SEO",
-        severity: "WARN",
-        check: (code, filePath) => {
-          if (!/\/app\/.*page\.(tsx?|jsx?)$/.test(filePath)) return null;
-          if (/export\s+(const|async function)\s+generateMetadata/.test(code)) return null;
-          if (/export\s+const\s+metadata/.test(code)) return null;
-          
-          return {
-            type: "nextjs-missing-metadata",
-            message: "Page is missing metadata export - important for SEO",
-            severity: "WARN",
-            confidence: 0.7,
-            fixHint: "Add `export const metadata = { title: 'Page Title' }` or `export async function generateMetadata()`",
-          };
-        },
-      },
-      {
-        id: "nextjs-client-only-in-server",
-        name: "Client-Only in Server Component",
-        description: "Using client-only features in server component",
-        severity: "BLOCK",
-        check: (code, filePath) => {
-          if (!/\/app\//.test(filePath)) return null;
-          if (/^['"]use client['"]/.test(code)) return null; // Client component
-          
-          const clientOnlyPatterns = [
-            /\buseState\s*\(/,
-            /\buseEffect\s*\(/,
-            /\buseRef\s*\(/,
-            /\bwindow\b/,
-            /\bdocument\b/,
-            /\blocalStorage\b/,
-            /\bsessionStorage\b/,
-          ];
-          
-          for (const pattern of clientOnlyPatterns) {
-            if (pattern.test(code)) {
-              return {
-                type: "nextjs-client-only-in-server",
-                message: `Using client-only feature in server component: ${pattern.source}`,
-                severity: "BLOCK",
-                confidence: 0.85,
-                fixHint: "Add 'use client' directive at the top of the file, or move client-only code to a client component",
-              };
-            }
-          }
-          
-          return null;
-        },
-      },
-    ];
-  },
-  
-  getFixSuggestion(finding, context) {
-    if (finding.type === "nextjs-missing-metadata") {
-      return {
-        description: "Add metadata export",
-        code: `export const metadata = {
-  title: 'Page Title',
-  description: 'Page description',
-};`,
-        confidence: 0.9,
-        autoApplicable: true,
-      };
-    }
-    
-    if (finding.type === "nextjs-client-only-in-server") {
-      return {
-        description: "Add 'use client' directive",
-        code: `'use client';\n\n`,
-        insertAt: "top",
-        confidence: 0.8,
-        autoApplicable: true,
-      };
-    }
-    
-    return null;
-  },
-};
-
-/**
- * React Adapter - Handles React-specific patterns
- */
-const ReactAdapter = {
-  name: "react",
-  version: "1.0.0",
-  frameworks: ["react"],
-  
-  detect(projectPath) {
-    try {
-      const pkgPath = path.join(projectPath, "package.json");
-      if (fs.existsSync(pkgPath)) {
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
-        const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-        return !!deps.react;
-      }
-    } catch {}
-    return false;
-  },
-  
-  getVersion(projectPath) {
-    try {
-      const pkgPath = path.join(projectPath, "package.json");
-      if (fs.existsSync(pkgPath)) {
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
-        const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-        const version = deps.react;
-        if (version) {
-          const match = version.match(/(\d+)\.(\d+)/);
-          if (match) {
-            return { major: parseInt(match[1]), minor: parseInt(match[2]), raw: version };
-          }
-        }
-      }
-    } catch {}
-    return null;
-  },
-  
-  shouldSuppressFinding(finding, context) {
-    const { version } = context;
-    
-    // React 19+: use() hook is valid
-    if (version?.major >= 19) {
-      if (finding.type === "unknown-hook" && finding.message?.includes("use(")) {
-        return { suppress: true, reason: "use() hook is valid in React 19+" };
-      }
-    }
-    
-    // React 18+: concurrent features are valid
-    if (version?.major >= 18) {
-      if (finding.type === "unknown-hook" && /use(Transition|DeferredValue|Id)/.test(finding.message)) {
-        return { suppress: true, reason: `${finding.message} is valid in React 18+` };
-      }
-    }
-    
-    return { suppress: false, reason: null };
-  },
-  
-  modifyFinding(finding, context) {
-    return finding;
-  },
-  
-  getAdditionalChecks() {
-    return [];
-  },
-  
-  getFixSuggestion(finding, context) {
-    return null;
-  },
-};
-
-/**
- * Express Adapter - Handles Express.js patterns
- */
-const ExpressAdapter = {
-  name: "express",
-  version: "1.0.0",
-  frameworks: ["express"],
-  
-  detect(projectPath) {
-    try {
-      const pkgPath = path.join(projectPath, "package.json");
-      if (fs.existsSync(pkgPath)) {
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
-        const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-        return !!deps.express;
-      }
-    } catch {}
-    return false;
-  },
-  
-  getVersion(projectPath) {
-    try {
-      const pkgPath = path.join(projectPath, "package.json");
-      if (fs.existsSync(pkgPath)) {
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
-        const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-        const version = deps.express;
-        if (version) {
-          const match = version.match(/(\d+)\.(\d+)/);
-          if (match) {
-            return { major: parseInt(match[1]), minor: parseInt(match[2]), raw: version };
-          }
-        }
-      }
-    } catch {}
-    return null;
-  },
-  
-  shouldSuppressFinding(finding, context) {
-    const { filePath } = context;
-    
-    // Express middleware pattern: (req, res, next) with unused next
-    if (finding.type === "unused-parameter" && finding.name === "next") {
-      if (/middleware|handler|route/i.test(filePath)) {
-        return { suppress: true, reason: "Express middleware signature requires next parameter even if unused" };
-      }
-    }
-    
-    // Error handling middleware: (err, req, res, next)
-    if (finding.type === "unused-parameter" && /^(err|error)$/.test(finding.name)) {
-      return { suppress: true, reason: "Express error middleware requires error parameter even if unused" };
-    }
-    
-    return { suppress: false, reason: null };
-  },
-  
-  modifyFinding(finding, context) {
-    return finding;
-  },
-  
-  getAdditionalChecks() {
-    return [
-      {
-        id: "express-missing-error-handler",
-        name: "Missing Error Handler",
-        description: "Express app has no error handling middleware",
-        severity: "WARN",
-        check: (code, filePath) => {
-          if (!/\bapp\s*=\s*express\(\)/.test(code)) return null;
-          if (/app\.use\([^,]+,\s*[^,]+,\s*[^,]+,\s*[^)]+\)/.test(code)) return null; // 4-param middleware
-          
-          return {
-            type: "express-missing-error-handler",
-            message: "Express app may be missing error handling middleware",
-            severity: "WARN",
-            confidence: 0.6,
-            fixHint: "Add error handling middleware: app.use((err, req, res, next) => { ... })",
-          };
-        },
-      },
-    ];
-  },
-  
-  getFixSuggestion(finding, context) {
-    return null;
-  },
-};
-
-/**
- * Prisma Adapter - Handles Prisma ORM patterns
- */
-const PrismaAdapter = {
-  name: "prisma",
-  version: "1.0.0",
-  frameworks: ["prisma"],
-  
-  detect(projectPath) {
-    try {
-      const pkgPath = path.join(projectPath, "package.json");
-      if (fs.existsSync(pkgPath)) {
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
-        const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-        return !!deps["@prisma/client"] || !!deps.prisma;
-      }
-    } catch {}
-    return false;
-  },
-  
-  getVersion(projectPath) {
-    try {
-      const pkgPath = path.join(projectPath, "package.json");
-      if (fs.existsSync(pkgPath)) {
-        const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
-        const deps = { ...pkg.dependencies, ...pkg.devDependencies };
-        const version = deps["@prisma/client"] || deps.prisma;
-        if (version) {
-          const match = version.match(/(\d+)\.(\d+)/);
-          if (match) {
-            return { major: parseInt(match[1]), minor: parseInt(match[2]), raw: version };
-          }
-        }
-      }
-    } catch {}
-    return null;
-  },
-  
-  shouldSuppressFinding(finding, context) {
-    // Prisma queries are parameterized, skip SQL injection warnings
-    if (finding.type === "sql-injection" || finding.type === "raw-sql") {
-      if (/prisma\.\$queryRaw|prisma\.\$executeRaw/.test(context.code || "")) {
-        return { suppress: true, reason: "Prisma parameterized queries are safe from SQL injection" };
-      }
-    }
-    
-    // Prisma client is generated, skip generated code warnings
-    if (finding.filePath?.includes("@prisma/client") || finding.filePath?.includes("generated/prisma")) {
-      return { suppress: true, reason: "Prisma client is generated code" };
-    }
-    
-    return { suppress: false, reason: null };
-  },
-  
-  modifyFinding(finding, context) {
-    return finding;
-  },
-  
-  getAdditionalChecks() {
-    return [];
-  },
-  
-  getFixSuggestion(finding, context) {
-    return null;
-  },
-};
-
-// ═══════════════════════════════════════════════════════════════════════════════
-// ADAPTER REGISTRY
-// ═══════════════════════════════════════════════════════════════════════════════
-
-const BUILT_IN_ADAPTERS = [
-  NextjsAdapter,
-  ReactAdapter,
-  ExpressAdapter,
-  PrismaAdapter,
-];
-
-// Custom adapters can be registered here
-const customAdapters = [];
-
-/**
- * Register a custom adapter
- */
-function registerAdapter(adapter) {
-  // Validate adapter interface
-  const required = ["name", "frameworks", "detect"];
-  for (const field of required) {
-    if (!adapter[field]) {
-      throw new Error(`Adapter missing required field: ${field}`);
-    }
-  }
-  customAdapters.push(adapter);
-}
-
-/**
- * Get all adapters (built-in + custom)
- */
-function getAllAdapters() {
-  return [...BUILT_IN_ADAPTERS, ...customAdapters];
-}
-
-/**
- * Detect which adapters are applicable for a project
- */
-function detectApplicableAdapters(projectPath) {
-  const applicable = [];
-  
-  for (const adapter of getAllAdapters()) {
-    try {
-      if (adapter.detect(projectPath)) {
-        const version = adapter.getVersion?.(projectPath) || null;
-        applicable.push({ adapter, version });
-      }
-    } catch {
-      // Skip adapters that fail detection
-    }
-  }
-  
-  return applicable;
-}
-
-/**
- * Apply all applicable adapters to a finding
- */
-function applyAdapters(finding, context = {}) {
-  const { projectPath = ".", filePath = "", code = "" } = context;
-  
-  // Get applicable adapters
-  const applicable = detectApplicableAdapters(projectPath);
-  
-  let modifiedFinding = { ...finding };
-  
-  for (const { adapter, version } of applicable) {
-    const ctx = { ...context, version, filePath, code };
-    
-    // Check if finding should be suppressed
-    const suppressResult = adapter.shouldSuppressFinding?.(modifiedFinding, ctx);
-    if (suppressResult?.suppress) {
-      return {
-        finding: null,
-        suppressed: true,
-        reason: suppressResult.reason,
-        adapter: adapter.name,
-      };
-    }
-    
-    // Modify finding
-    modifiedFinding = adapter.modifyFinding?.(modifiedFinding, ctx) || modifiedFinding;
-  }
-  
-  return {
-    finding: modifiedFinding,
-    suppressed: false,
-    reason: null,
-    adapter: null,
-  };
-}
-
-/**
- * Get all additional checks from applicable adapters
- */
-function getAdditionalChecks(projectPath) {
-  const checks = [];
-  const applicable = detectApplicableAdapters(projectPath);
-  
-  for (const { adapter } of applicable) {
-    const adapterChecks = adapter.getAdditionalChecks?.() || [];
-    for (const check of adapterChecks) {
-      checks.push({
-        ...check,
-        adapter: adapter.name,
-      });
-    }
-  }
-  
-  return checks;
-}
-
-/**
- * Get fix suggestion from applicable adapters
- */
-function getFixSuggestion(finding, context = {}) {
-  const { projectPath = "." } = context;
-  const applicable = detectApplicableAdapters(projectPath);
-  
-  for (const { adapter, version } of applicable) {
-    const ctx = { ...context, version };
-    const suggestion = adapter.getFixSuggestion?.(finding, ctx);
-    if (suggestion) {
-      return {
-        ...suggestion,
-        adapter: adapter.name,
-      };
-    }
-  }
-  
-  return null;
-}
-
-// ═══════════════════════════════════════════════════════════════════════════════
-// EXPORTS
-// ═══════════════════════════════════════════════════════════════════════════════
-
-module.exports = {
-  // Adapter management
-  registerAdapter,
-  getAllAdapters,
-  detectApplicableAdapters,
-  
-  // Finding processing
-  applyAdapters,
-  getAdditionalChecks,
-  getFixSuggestion,
-  
-  // Built-in adapters (for direct use or customization)
-  NextjsAdapter,
-  ReactAdapter,
-  ExpressAdapter,
-  PrismaAdapter,
-  
-  // Interface (for reference)
-  AdapterInterface,
-};