openkrak-mcp 1.0.16 → 1.0.17

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

package/package.json

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