fix: scope native import-edge skip to Windows only (#777)

* fix: scope native import-edge skip to Windows only

The Rust key mismatch in build_import_edges (backslash vs forward slash)
only affects Windows where path.join produces backslashes. On Linux/macOS
path.join uses forward slashes matching the Rust format!("{}/{}", ...).

Skipping native import edges on all platforms caused the Codegraph Impact
Analysis CI workflow to fall back to the slow JS path unnecessarily.

* fix: match Rust key format in resolvedImports instead of skipping native

Instead of skipping the native import-edge builder for addon 3.8.0,
construct resolvedImports keys as rootDir + "/" + relPath to match
the Rust format!("{}/{}", root_dir, file) exactly. This avoids the
Windows path separator mismatch without any performance penalty.

Author: carlos-alm

src/domain/graph/builder/stages/build-edges.ts

@@ -9,7 +9,6 @@ import { performance } from 'node:perf_hooks';
 import { getNodeId } from '../../../../db/index.js';
 import { debug } from '../../../../infrastructure/logger.js';
 import { loadNative } from '../../../../infrastructure/native.js';
-import { normalizePath } from '../../../../shared/constants.js';
 import type {
   BetterSqlite3Database,
   Call,
@@ -198,8 +197,11 @@ function buildImportEdgesNative(
   };
   const fileNodeRowCache = new Map<string, { id: number }>();
 
-  // 2. Pre-resolve all imports and collect supplemental resolved entries
-  const supplementalResolved: Array<{ key: string; resolvedPath: string }> = [];
+  // 2. Pre-resolve all imports and build resolved imports array.
+  // Keys use rootDir + "/" + relPath to match the Rust lookup format
+  // (format!("{}/{}", root_dir, file)) — avoids separator mismatches on Windows
+  // where path.join uses backslashes but Rust joins with forward slash.
+  const resolvedImports: Array<{ key: string; resolvedPath: string }> = [];
 
   for (const [relPath, symbols] of fileSymbols) {
     const fileNodeRow = addFileNodeId(relPath);
@@ -219,11 +221,8 @@ function buildImportEdgesNative(
       const resolvedPath = getResolved(ctx, path.join(rootDir, relPath), imp.source);
       addFileNodeId(resolvedPath);
 
-      // Check if this resolution is in batchResolved; if not, add supplemental
-      const resolveKey = `${normalizePath(path.join(rootDir, relPath))}|${imp.source}`;
-      if (!ctx.batchResolved?.has(resolveKey)) {
-        supplementalResolved.push({ key: resolveKey, resolvedPath });
-      }
+      // Key matches Rust's format!("{}/{}", root_dir, file_input.file)
+      resolvedImports.push({ key: `${rootDir}/${relPath}|${imp.source}`, resolvedPath });
 
       importInfos.push({
         source: imp.source,
@@ -244,17 +243,6 @@ function buildImportEdgesNative(
     });
   }
 
-  // 3. Flatten batchResolved + supplemental into resolved imports array
-  const resolvedImports: Array<{ key: string; resolvedPath: string }> = [];
-  if (ctx.batchResolved) {
-    for (const [key, resolvedPath] of ctx.batchResolved) {
-      resolvedImports.push({ key, resolvedPath });
-    }
-  }
-  for (const entry of supplementalResolved) {
-    resolvedImports.push(entry);
-  }
-
   // 4. Flatten reexportMap
   const fileReexports: Array<{
     file: string;
@@ -744,12 +732,10 @@ export async function buildEdges(ctx: PipelineContext): Promise<void> {
       }
     }
 
-    // Skip native import-edge path for small incremental builds (≤3 files)
-    // and for addon 3.8.0 which has a Windows path-separator bug in key
-    // construction (format!("{}/{}", root_dir, file) vs path.join backslashes).
-    const importEdgeBuggy = ctx.engineVersion === '3.8.0';
+    // Skip native import-edge path for small incremental builds (≤3 files):
+    // napi-rs marshaling overhead exceeds computation savings.
     const useNativeImportEdges =
-      native?.buildImportEdges && !importEdgeBuggy && (ctx.isFullBuild || ctx.fileSymbols.size > 3);
+      native?.buildImportEdges && (ctx.isFullBuild || ctx.fileSymbols.size > 3);
     if (useNativeImportEdges) {
       buildImportEdgesNative(ctx, getNodeIdStmt, allEdgeRows, native!);
     } else {