fix: isolate CLI tests from real registry and add CODEGRAPH_REGISTRY_PATH env var

The cli.test.js afterAll called pruneRegistry() with no arguments, operating
on the real ~/.codegraph/registry.json. The run() helper also lacked HOME
isolation, risking writes to the real registry. Add CODEGRAPH_REGISTRY_PATH
env var support to registry.js, isolate all CLI spawns with a fake HOME, and
remove the bare pruneRegistry() call.

Author: github-actions[bot]

src/parser.js

@@ -3,7 +3,7 @@ import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { Language, Parser } from 'web-tree-sitter';
 import { warn } from './logger.js';
-import { loadNative } from './native.js';
+import { getNative, loadNative } from './native.js';
 
 // Re-export all extractors for backward compatibility
 export {
@@ -78,7 +78,7 @@ function resolveEngine(opts = {}) {
     const native = loadNative();
     if (native) return { name: 'native', native };
     if (pref === 'native') {
-      warn('Native engine requested but unavailable — falling back to WASM');
+      getNative(); // throws with detailed error + install instructions
     }
   }
   return { name: 'wasm', native: null };

src/registry.js

@@ -3,7 +3,8 @@ import os from 'node:os';
 import path from 'node:path';
 import { debug, warn } from './logger.js';
 
-export const REGISTRY_PATH = path.join(os.homedir(), '.codegraph', 'registry.json');
+export const REGISTRY_PATH =
+  process.env.CODEGRAPH_REGISTRY_PATH || path.join(os.homedir(), '.codegraph', 'registry.json');
 
 /** Default TTL: entries not accessed within 30 days are pruned. */
 export const DEFAULT_TTL_DAYS = 30;

tests/integration/cli.test.js

@@ -8,7 +8,6 @@ import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 import { afterAll, beforeAll, describe, expect, test } from 'vitest';
-import { pruneRegistry } from '../../src/registry.js';
 
 const CLI = path.resolve('src/cli.js');
 
@@ -37,19 +36,21 @@ export function main() {
 `.trimStart(),
 };
 
-let tmpDir, dbPath;
+let tmpDir, tmpHome, dbPath;
 
 /** Run the CLI and return stdout as a string. Throws on non-zero exit. */
 function run(...args) {
   return execFileSync('node', [CLI, ...args], {
     cwd: tmpDir,
     encoding: 'utf-8',
     timeout: 30_000,
+    env: { ...process.env, HOME: tmpHome, USERPROFILE: tmpHome },
   });
 }
 
 beforeAll(async () => {
   tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'codegraph-cli-'));
+  tmpHome = fs.mkdtempSync(path.join(os.tmpdir(), 'codegraph-clihome-'));
   for (const [name, content] of Object.entries(FIXTURE_FILES)) {
     fs.writeFileSync(path.join(tmpDir, name), content);
   }
@@ -61,7 +62,7 @@ beforeAll(async () => {
 
 afterAll(() => {
   if (tmpDir) fs.rmSync(tmpDir, { recursive: true, force: true });
-  pruneRegistry();
+  if (tmpHome) fs.rmSync(tmpHome, { recursive: true, force: true });
 });
 
 describe('CLI smoke tests', () => {

tests/parsers/unified.test.js

@@ -22,6 +22,12 @@ describe('Unified parser API', () => {
       expect(engine.name).toBe('wasm');
       expect(engine.version).toBeNull();
     });
+
+    it('throws when engine=native is explicitly requested but unavailable', () => {
+      const engine = getActiveEngine();
+      if (engine.name === 'native') return; // skip — native is available
+      expect(() => getActiveEngine({ engine: 'native' })).toThrow(/[Nn]ative/);
+    });
   });
 
   describe('parseFileAuto', () => {

tests/unit/registry.test.js

@@ -1,3 +1,4 @@
+import { execFileSync } from 'node:child_process';
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
@@ -29,9 +30,27 @@ afterEach(() => {
 // ─── REGISTRY_PATH ──────────────────────────────────────────────────
 
 describe('REGISTRY_PATH', () => {
-  it('points to ~/.codegraph/registry.json', () => {
+  it('points to ~/.codegraph/registry.json by default', () => {
     expect(REGISTRY_PATH).toBe(path.join(os.homedir(), '.codegraph', 'registry.json'));
   });
+
+  it('respects CODEGRAPH_REGISTRY_PATH env var', () => {
+    const customPath = path.join(tmpDir, 'custom', 'registry.json');
+    const result = execFileSync(
+      'node',
+      [
+        '--input-type=module',
+        '-e',
+        `import { REGISTRY_PATH } from './src/registry.js'; process.stdout.write(REGISTRY_PATH);`,
+      ],
+      {
+        cwd: path.resolve(import.meta.dirname, '..', '..'),
+        encoding: 'utf-8',
+        env: { ...process.env, CODEGRAPH_REGISTRY_PATH: customPath },
+      },
+    );
+    expect(result).toBe(customPath);
+  });
 });
 
 // ─── loadRegistry ───────────────────────────────────────────────────