@maxgfr/codeindex 2.19.0 → 2.19.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@maxgfr/codeindex",
-  "version": "2.19.0",
+  "version": "2.19.1",
   "description": "Self-contained, deterministic repo-indexing engine: walk + language detection + symbol/import extraction (tree-sitter AST with regex fallback) + import resolution + typed cross-file link-graph + analytics. Ships as a single zero-dependency engine.mjs that consumer tools vendor.",
   "type": "module",
   "packageManager": "pnpm@10.33.0",

package/scripts/engine.d.mts

@@ -1,4 +1,4 @@
-declare const ENGINE_VERSION = "2.19.0";
+declare const ENGINE_VERSION = "2.19.1";
 declare const SCHEMA_VERSION = 4;
 declare const EXTRACTOR_VERSION = 10;
 type FileKind = "code" | "doc" | "config" | "asset" | "other";
@@ -347,7 +347,7 @@ declare function extractAst(rel: string, ext: string, content: string, opts?: {
     imports?: boolean;
 }): AstResult | undefined;
 
-declare const DEFAULT_GRAMMARS_URL = "https://github.com/maxgfr/codeindex/releases/download/v2.19.0/grammars-2.19.0.tar.gz";
+declare const DEFAULT_GRAMMARS_URL = "https://github.com/maxgfr/codeindex/releases/download/v2.19.1/grammars-2.19.1.tar.gz";
 interface GrammarsPullTarget {
     url: string;
     sha256Url?: string;

package/scripts/engine.mjs

@@ -14,7 +14,7 @@ var ENGINE_VERSION, SCHEMA_VERSION, EXTRACTOR_VERSION;
 var init_types = __esm({
   "src/types.ts"() {
     "use strict";
-    ENGINE_VERSION = "2.19.0";
+    ENGINE_VERSION = "2.19.1";
     SCHEMA_VERSION = 4;
     EXTRACTOR_VERSION = 10;
   }
@@ -12539,6 +12539,7 @@ function neighborsOf(graph, target, depth = 1, kinds) {
 // src/delta.ts
 init_git();
 init_sort();
+init_walk();
 init_util();
 var RISK_WEIGHTS = {
   exportedChange: 25,
@@ -12648,6 +12649,16 @@ function computeDelta(graph, symbols, diff, depth = DEFAULT_DELTA_DEPTH) {
   }
   const changedRels = new Set(changes.filter((c2) => c2.status !== "deleted").map((c2) => c2.path));
   const dangling = graph.fileEdges.filter((e) => e.dangling && (e.kind === "import" || e.kind === "doc-link") && changedRels.has(e.from)).map((e) => ({ from: e.from, spec: e.to, reason: e.reason ?? "unknown" })).sort((a, b) => byStr(a.from, b.from) || byStr(a.spec, b.spec));
+  const intoIgnoredTree = (from, spec) => {
+    if (!spec.startsWith(".")) return false;
+    const segs = from.split("/").slice(0, -1);
+    for (const part of spec.split("/")) {
+      if (part === "." || part === "") continue;
+      if (part === "..") segs.pop();
+      else segs.push(part);
+    }
+    return segs.some((seg) => IGNORE_DIRS.has(seg));
+  };
   const byModule = /* @__PURE__ */ new Map();
   for (const c2 of changes) {
     if (c2.status === "deleted" || !c2.module) continue;
@@ -12717,7 +12728,9 @@ function computeDelta(graph, symbols, diff, depth = DEFAULT_DELTA_DEPTH) {
       score += RISK_WEIGHTS.surprise;
       reasons.push(`cross-community edge to ${sup.from === slug ? sup.to : sup.from} (surprising)`);
     }
-    const moduleDangling = dangling.filter((d) => moduleChanges.some((c2) => c2.path === d.from));
+    const moduleDangling = dangling.filter(
+      (d) => moduleChanges.some((c2) => c2.path === d.from) && !intoIgnoredTree(d.from, d.spec)
+    );
     if (moduleDangling.length) {
       score += RISK_WEIGHTS.dangling;
       const first = moduleDangling[0];

package/src/delta.ts

@@ -9,6 +9,7 @@ import type { Graph, ModuleNode, SymbolIndex } from "./types.js";
 import type { DiffFile, DiffSpec, Hunk } from "./git.js";
 import { isGitWorktree, resolveBaseRef, diffFiles, diffHunks, untrackedFiles } from "./git.js";
 import { byStr } from "./sort.js";
+import { IGNORE_DIRS } from "./walk.js";
 import { have, sh } from "./util.js";
 import { impactOf } from "./traverse.js";
 
@@ -208,6 +209,24 @@ export function computeDelta(
     .map((e) => ({ from: e.from, spec: e.to, reason: e.reason ?? "unknown" }))
     .sort((a, b) => byStr(a.from, b.from) || byStr(a.spec, b.spec));
 
+  // A relative import whose target lands in a directory the walker ignores
+  // (vendor/, dist/, build/ …) is reported as dangling because the resolver only
+  // looks inside the scan — but the file is there and the import works. It is a
+  // blind spot in the graph, not a broken reference, so it must NOT carry the
+  // dangling RISK weight: a repo that vendors a dependency would otherwise take
+  // a permanent penalty on every change to the file that imports it. Still
+  // listed above, so the blind spot stays visible.
+  const intoIgnoredTree = (from: string, spec: string): boolean => {
+    if (!spec.startsWith(".")) return false;
+    const segs = from.split("/").slice(0, -1);
+    for (const part of spec.split("/")) {
+      if (part === "." || part === "") continue;
+      if (part === "..") segs.pop();
+      else segs.push(part);
+    }
+    return segs.some((seg) => IGNORE_DIRS.has(seg));
+  };
+
   // Group by module and score.
   const byModule = new Map<string, DeltaChange[]>();
   for (const c of changes) {
@@ -296,7 +315,9 @@ export function computeDelta(
     }
 
     // 6. Dangling imports from this module's changed files.
-    const moduleDangling = dangling.filter((d) => moduleChanges.some((c) => c.path === d.from));
+    const moduleDangling = dangling.filter(
+      (d) => moduleChanges.some((c) => c.path === d.from) && !intoIgnoredTree(d.from, d.spec),
+    );
     if (moduleDangling.length) {
       score += RISK_WEIGHTS.dangling;
       const first = moduleDangling[0]!;

package/src/types.ts

@@ -1,7 +1,7 @@
 // Single source of truth for the engine version the bundle reports. Kept in
 // lockstep with package.json by the release pipeline. Do not edit by hand
 // outside a release.
-export const ENGINE_VERSION = "2.19.0";
+export const ENGINE_VERSION = "2.19.1";
 
 // Bumped whenever the on-disk artifact shape changes, so a consumer can reject
 // an index written by an incompatible engine instead of misreading it. The