openkrak-mcp 1.0.16 → 1.0.18

package/dist/index.js

@@ -242241,6 +242241,66 @@ function computeStats(nodes, edges) {
     edge_kind_breakdown: edgeBreakdown
   };
 }
+var EXTERNAL_PREFIXES = [
+  "node:",
+  "fs/",
+  "path",
+  "crypto",
+  "os",
+  "http",
+  "https",
+  "stream",
+  "util",
+  "events",
+  "buffer",
+  "url",
+  "querystring",
+  "child_process",
+  "cluster",
+  "next/",
+  "react",
+  "react-dom",
+  "react/",
+  "@next/",
+  "@vercel/",
+  "@modelcontextprotocol/",
+  "@typescript-eslint/",
+  "@base-ui/",
+  "hono",
+  "hono/",
+  "zod",
+  "openai",
+  "pino",
+  "pino/",
+  "kysely",
+  "p-retry",
+  "simple-git",
+  "dorchester",
+  "framer-motion",
+  "lucide-react",
+  "mathjs",
+  "katex",
+  "tailwindcss",
+  "typescript",
+  "tsup",
+  "tsx",
+  "vitest",
+  "jest",
+  "payhip",
+  "stripe",
+  "@stripe/"
+];
+function isExternalImport(target) {
+  if (!target.includes("/") && !target.startsWith(".")) return true;
+  for (const prefix of EXTERNAL_PREFIXES) {
+    if (target.startsWith(prefix)) return true;
+  }
+  if (target.endsWith(".css") || target.endsWith(".scss") || target.endsWith(".svg")) return true;
+  return false;
+}
+function isBarrelFile(filePath) {
+  return filePath.endsWith("/index.ts") || filePath.endsWith("/index.js") || filePath === "index.ts" || filePath === "index.js";
+}
 function detectCycles(graph) {
   const adjMap = /* @__PURE__ */ new Map();
   for (const edge of graph.edges) {
@@ -242261,11 +242321,14 @@ function detectCycles(graph) {
   const visited = /* @__PURE__ */ new Set();
   const inStack = /* @__PURE__ */ new Set();
   const cyclePaths = [];
+  const MAX_CYCLES = 20;
   function dfs(nodeId, path7) {
+    if (cyclePaths.length >= MAX_CYCLES) return;
     visited.add(nodeId);
     inStack.add(nodeId);
     path7.push(nodeId);
     for (const neighbor of adjMap.get(nodeId) ?? []) {
+      if (cyclePaths.length >= MAX_CYCLES) break;
       if (!visited.has(neighbor)) {
         dfs(neighbor, [...path7]);
       } else if (inStack.has(neighbor)) {
@@ -242278,6 +242341,7 @@ function detectCycles(graph) {
     inStack.delete(nodeId);
   }
   for (const node of graph.nodes) {
+    if (cyclePaths.length >= MAX_CYCLES) break;
     if (!visited.has(node.id)) {
       dfs(node.id, []);
     }
@@ -242291,57 +242355,67 @@ function detectDeadCode(graph, fileIndex) {
     incomingCount.set(edge.to, (incomingCount.get(edge.to) ?? 0) + 1);
   }
   const testFiles = new Set(fileIndex.filter((f) => f.role === "test").map((f) => f.path));
-  const entryPoints = new Set(
-    graph.nodes.filter((n) => n.is_entry_point).map((n) => n.id)
-  );
+  const entryPoints = new Set(graph.nodes.filter((n) => n.is_entry_point).map((n) => n.id));
+  const barrelFiles = new Set(fileIndex.filter((f) => isBarrelFile(f.path)).map((f) => f.path));
   const findings = [];
+  const MAX_DEAD_CODE_PER_FILE = 3;
+  const deadCountPerFile = /* @__PURE__ */ new Map();
   for (const node of graph.nodes) {
     if (node.kind === "file") continue;
+    if (testFiles.has(node.file)) continue;
+    if (entryPoints.has(node.id)) continue;
+    if (node.is_exported && barrelFiles.has(node.file)) continue;
+    if (node.is_exported) continue;
     const incoming = incomingCount.get(node.id) ?? 0;
-    if (incoming === 0 && !entryPoints.has(node.id) && !testFiles.has(node.file)) {
-      const isExported = node.is_exported;
-      findings.push({
-        id: (0, import_crypto2.randomUUID)(),
-        type: "dead_code",
-        severity: isExported ? "low" : "medium",
-        file: node.file,
-        line_start: node.line_start,
-        line_end: node.line_end,
-        symbol: node.id,
-        title: `Dead code: ${node.name}`,
-        description: `Symbol '${node.name}' has no incoming references and is not an entry point.`,
-        raw_data: { incoming_edge_count: 0, last_referenced_at: null },
-        source_component: "deepstrike",
-        is_structural: true
-      });
-    }
+    if (incoming > 0) continue;
+    const fileCount = deadCountPerFile.get(node.file) ?? 0;
+    if (fileCount >= MAX_DEAD_CODE_PER_FILE) continue;
+    deadCountPerFile.set(node.file, fileCount + 1);
+    findings.push({
+      id: (0, import_crypto2.randomUUID)(),
+      type: "dead_code",
+      severity: "low",
+      file: node.file,
+      line_start: node.line_start,
+      line_end: node.line_end,
+      symbol: node.id,
+      title: `Dead code: ${node.name}`,
+      description: `Symbol '${node.name}' has no incoming references and is not exported or an entry point.`,
+      raw_data: { incoming_edge_count: 0, last_referenced_at: null },
+      source_component: "deepstrike",
+      is_structural: true
+    });
   }
   return findings;
 }
 function detectMissingSymbols(graph, knownNodeIds) {
   const findings = [];
+  const seen = /* @__PURE__ */ new Set();
+  const MAX_MISSING = 30;
   for (const edge of graph.edges) {
     if (edge.kind !== "import") continue;
-    if (!edge.to.includes("/") && !edge.to.startsWith(".")) continue;
-    if (!knownNodeIds.has(edge.to)) {
-      findings.push({
-        id: (0, import_crypto2.randomUUID)(),
-        type: "missing_symbol",
-        severity: "high",
-        file: edge.file,
-        line_start: edge.line,
-        line_end: edge.line,
-        symbol: null,
-        title: `Unresolved import: ${edge.to}`,
-        description: `Import target '${edge.to}' could not be resolved in the dependency graph.`,
-        raw_data: {
-          import_statement: edge.to,
-          attempted_resolution: edge.to
-        },
-        source_component: "deepstrike",
-        is_structural: true
-      });
-    }
+    if (isExternalImport(edge.to)) continue;
+    if (knownNodeIds.has(edge.to)) continue;
+    if (seen.has(edge.to)) continue;
+    if (findings.length >= MAX_MISSING) break;
+    seen.add(edge.to);
+    findings.push({
+      id: (0, import_crypto2.randomUUID)(),
+      type: "missing_symbol",
+      severity: "medium",
+      file: edge.file,
+      line_start: edge.line,
+      line_end: edge.line,
+      symbol: null,
+      title: `Unresolved import: ${edge.to}`,
+      description: `Import target '${edge.to}' could not be resolved. May indicate a missing file, broken alias, or uninstalled dependency.`,
+      raw_data: {
+        import_statement: edge.to,
+        attempted_resolution: edge.to
+      },
+      source_component: "deepstrike",
+      is_structural: true
+    });
   }
   return findings;
 }
@@ -242360,7 +242434,7 @@ function buildCycleFindings(cyclePaths, nodes) {
       line_end: null,
       symbol: cycle[0],
       title: `Circular dependency (${cycle.length} nodes)`,
-      description: `Cycle: ${cycle.slice(0, 3).join(" \u2192 ")}${cycle.length > 3 ? " ..." : ""}`,
+      description: `Cycle: ${cycle.slice(0, 4).join(" \u2192 ")}${cycle.length > 4 ? " ..." : ""}`,
       raw_data: { cycle_path: cycle, cycle_length: cycle.length },
       source_component: "deepstrike",
       is_structural: true
@@ -242442,10 +242516,11 @@ function detectSecurityPatterns(filePath, content) {
 }
 async function classifyTopology(repoRoot, allFiles) {
   const type = await detectProjectType(repoRoot);
+  const framework = await detectFramework(repoRoot);
   const entryPoints = detectEntryPoints(allFiles);
   const layers = detectLayers(allFiles);
   const modules = detectModules(allFiles);
-  return { type, entry_points: entryPoints, layers, modules };
+  return { type, entry_points: entryPoints, layers, modules, framework };
 }
 async function detectProjectType(repoRoot) {
   try {
@@ -242455,7 +242530,7 @@ async function detectProjectType(repoRoot) {
     if (pkg.private === true && Array.isArray(pkg.workspaces)) return "monorepo";
   } catch {
   }
-  for (const config of ["nx.json", "lerna.json"]) {
+  for (const config of ["nx.json", "lerna.json", "pnpm-workspace.yaml", "turbo.json"]) {
     try {
       await import_promises3.default.access(import_path4.default.join(repoRoot, config));
       return "monorepo";
@@ -242464,12 +242539,70 @@ async function detectProjectType(repoRoot) {
   }
   return "monolith";
 }
+async function detectFramework(repoRoot) {
+  try {
+    const pkgPath = import_path4.default.join(repoRoot, "package.json");
+    const pkg = JSON.parse(await import_promises3.default.readFile(pkgPath, "utf-8"));
+    const deps = { ...pkg.dependencies, ...pkg.devDependencies };
+    if (deps["next"]) return `next@${deps["next"].replace(/[\^~]/g, "")}`;
+    if (deps["nuxt"]) return `nuxt@${deps["nuxt"].replace(/[\^~]/g, "")}`;
+    if (deps["@remix-run/react"]) return "remix";
+    if (deps["astro"]) return `astro@${deps["astro"].replace(/[\^~]/g, "")}`;
+    if (deps["svelte"]) return "svelte";
+    if (deps["vue"]) return `vue@${deps["vue"].replace(/[\^~]/g, "")}`;
+    if (deps["react"]) return `react@${deps["react"].replace(/[\^~]/g, "")}`;
+    if (deps["express"]) return `express@${deps["express"].replace(/[\^~]/g, "")}`;
+    if (deps["fastify"]) return `fastify@${deps["fastify"].replace(/[\^~]/g, "")}`;
+    if (deps["hono"]) return `hono@${deps["hono"].replace(/[\^~]/g, "")}`;
+    if (deps["@nestjs/core"]) return "nestjs";
+    if (deps["@angular/core"]) return "angular";
+  } catch {
+  }
+  try {
+    await import_promises3.default.access(import_path4.default.join(repoRoot, "next.config.js"));
+    return "next";
+  } catch {
+  }
+  try {
+    await import_promises3.default.access(import_path4.default.join(repoRoot, "next.config.mjs"));
+    return "next";
+  } catch {
+  }
+  try {
+    await import_promises3.default.access(import_path4.default.join(repoRoot, "next.config.ts"));
+    return "next";
+  } catch {
+  }
+  try {
+    await import_promises3.default.access(import_path4.default.join(repoRoot, "vite.config.ts"));
+    return "vite";
+  } catch {
+  }
+  try {
+    await import_promises3.default.access(import_path4.default.join(repoRoot, "astro.config.mjs"));
+    return "astro";
+  } catch {
+  }
+  return null;
+}
 function detectEntryPoints(files) {
   const entries = [];
   for (const f of files) {
-    if (/\/(index|main|app|server|cli)\.(ts|js)x?$/.test(f)) {
-      const type = f.includes("cli") ? "cli" : f.includes("server") || f.includes("app") ? "api" : "main";
+    if (/\/(index|main|server|cli)\.(ts|js)x?$/.test(f)) {
+      const type = f.includes("cli") ? "cli" : f.includes("server") ? "api" : "main";
       entries.push({ path: f, type });
+      continue;
+    }
+    if (/^app\/.*page\.(ts|js)x?$/.test(f) || /^app\/layout\.(ts|js)x?$/.test(f)) {
+      entries.push({ path: f, type: "api" });
+      continue;
+    }
+    if (/^app\/.*route\.(ts|js)x?$/.test(f) || /^pages\/api\//.test(f)) {
+      entries.push({ path: f, type: "api" });
+      continue;
+    }
+    if (/^src\/(main|index|App)\.(ts|js)x?$/.test(f)) {
+      entries.push({ path: f, type: "main" });
     }
   }
   return entries;
@@ -243265,6 +243398,7 @@ function detectGodObjects(graph, couplingScores, fileRole) {
   for (const [file, count] of nodeCountByFile) {
     if (count <= NODE_COUNT_THRESHOLD) continue;
     if (fileRole.get(file) === "test") continue;
+    if (file.endsWith("/index.ts") || file.endsWith("/index.js") || file === "index.ts") continue;
     const coupling = couplingScores.get(file);
     if (!coupling) continue;
     if (coupling.fan_in <= FAN_IN_THRESHOLD && coupling.fan_out <= FAN_OUT_THRESHOLD) continue;
@@ -243292,7 +243426,8 @@ function aggregateHotspots(params) {
     cycleFindings,
     affectedSymbolsByFile,
     cyclePaths = [],
-    graph
+    graph,
+    changeFrequencyMap = /* @__PURE__ */ new Map()
   } = params;
   const cycleFilesFromPaths = /* @__PURE__ */ new Map();
   for (const path7 of cyclePaths) {
@@ -243344,7 +243479,10 @@ function aggregateHotspots(params) {
     const isRepository = file.includes("/repositories/") || file.includes("/repository/") || file.toLowerCase().includes("repository");
     const isFoundational = isRepository && methodCount >= 3 && (coupling?.fan_in ?? 0) >= 1;
     const foundationalBonus = isFoundational ? 0.25 : 0;
-    const rawScore = couplingScore * 0.4 + complexityScore * 0.15 + methodScore * 0.1 + violationPenalty + godObjectBonus + cyclePenalty + foundationalBonus;
+    const changeFreq = changeFrequencyMap.get(file) ?? 0;
+    const maxChangeFreq = Math.max(1, ...[...changeFrequencyMap.values()]);
+    const changeFreqScore = Math.min(1, changeFreq / maxChangeFreq);
+    const rawScore = couplingScore * 0.35 + complexityScore * 0.15 + methodScore * 0.1 + changeFreqScore * 0.1 + violationPenalty + godObjectBonus + cyclePenalty + foundationalBonus;
     const finalScore = Math.min(1, rawScore);
     const reasons = [];
     if (coupling && couplingScore > 0) {
@@ -243481,10 +243619,11 @@ function buildArchitectureViolationFindings(violations) {
   }));
 }
 var logger3 = createLogger("hotspot_registry");
-async function runHotspotRegistry(store) {
+async function runHotspotRegistry(store, opts) {
   const start = Date.now();
   const errors = [];
   const scanId = store.getMeta().scan_id;
+  const changeFrequencyMap = opts?.changeFrequencyMap ?? /* @__PURE__ */ new Map();
   const log = logger3.child({ scan_id: scanId });
   log.info({ event: "hotspot_registry.start" }, "Hotspot Registry started");
   const graph = store.getDependencyGraph();
@@ -243536,7 +243675,8 @@ async function runHotspotRegistry(store) {
     cycleFindings,
     affectedSymbolsByFile,
     cyclePaths: graph.stats.cycle_paths ?? [],
-    graph
+    graph,
+    changeFrequencyMap
   });
   const couplingFindings = buildCouplingFindings(couplingScores.values(), graph.nodes);
   const complexityFindings = buildComplexityFindings(complexityScores);
@@ -244782,8 +244922,22 @@ async function runPipeline(options) {
   try {
     logger7.info("Step 1/6: DeepStrike");
     await runDeepStrike(store, { repoRoot: options.repoPath });
+    const changeFrequencyMap = /* @__PURE__ */ new Map();
+    try {
+      const { execSync } = await import("child_process");
+      const gitLog = execSync("git log --name-only --pretty=format: -- .", {
+        cwd: options.repoPath,
+        encoding: "utf-8",
+        timeout: 8e3
+      });
+      for (const line of gitLog.split("\n")) {
+        const trimmed = line.trim().replace(/\\/g, "/");
+        if (trimmed) changeFrequencyMap.set(trimmed, (changeFrequencyMap.get(trimmed) ?? 0) + 1);
+      }
+    } catch {
+    }
     logger7.info("Step 2/6: Hotspot Registry");
-    await runHotspotRegistry(store);
+    await runHotspotRegistry(store, { repoRoot: options.repoPath, changeFrequencyMap });
     logger7.info("Step 3/6: Correlation Engine");
     await runCorrelationEngine(store);
     logger7.info("Step 4/6: Blast Radius Engine");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openkrak-mcp",
-  "version": "1.0.16",
+  "version": "1.0.18",
   "description": "OpenKrak MCP Server - AI coding intelligence via Dorchester engine",
   "mcpName": "io.github.FrnzJulianBergmann/openkrak",
   "type": "commonjs",