@sdsrs/code-graph 0.73.0 → 0.73.1

package/claude-plugin/.claude-plugin/plugin.json

@@ -4,7 +4,7 @@
   "author": {
     "name": "sdsrs"
   },
-  "version": "0.73.0",
+  "version": "0.73.1",
   "keywords": [
     "code-graph",
     "ast",

package/claude-plugin/scripts/doctor.js

@@ -330,6 +330,32 @@ function relicRepairGuard({ log = console.log, relic = undefined } = {}) {
   return true;
 }
 
+// A dev-mode rebuild must PRESERVE the existing binary's feature set. This repair
+// used to hardcode `--no-default-features`, which silently downgraded a hybrid
+// (embed-model) dev binary to FTS5-only and ping-ponged against a manual
+// `cargo build --release --features embed-model`. Probe the binary's COMPILED
+// feature via `health-check --json` → `model_available` (= cfg!(feature =
+// "embed-model"), reported even with no index) and rebuild to match. Returns
+// true/false, or null when the binary can't be probed (missing/broken) — the
+// caller then defaults to FTS5 + an explicit note, never a silent downgrade.
+// End users never reach this path (binary-stale → auto-update; binary-missing
+// non-dev → install instructions); it is purely the source-repo dev convenience.
+function detectEmbedModel(binary, run = execFileSync) {
+  if (!binary) return null;
+  try {
+    const out = run(binary, ['health-check', '--json'], {
+      timeout: 10000, encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'],
+    });
+    return JSON.parse(out).model_available === true;
+  } catch { return null; }
+}
+
+function devBuildCommand(embed) {
+  return embed
+    ? 'cargo build --release --features embed-model'
+    : 'cargo build --release --no-default-features';
+}
+
 function runRepairs(results) {
   const fixable = results.filter(r => r.fixId);
   if (fixable.length === 0) return 0;
@@ -353,14 +379,23 @@ function runRepairs(results) {
           }
           break;
         }
+        // Preserve the current binary's feature set \u2014 never silently downgrade
+        // a hybrid (embed-model) dev binary to FTS5-only (which also ping-pongs
+        // against a manual `--features embed-model` build).
+        const embed = detectEmbedModel(findBinary());
+        const buildCmd = devBuildCommand(embed === true);
         console.log('\n  Building binary...');
-        console.log('    \u2192 cargo build --release --no-default-features');
+        if (embed === null) {
+          console.log('    (could not detect current feature set \u2014 building FTS5-only;');
+          console.log('     for semantic search rebuild with `cargo build --release --features embed-model`)');
+        }
+        console.log(`    \u2192 ${buildCmd}`);
         try {
           const projectRoot = path.resolve(__dirname, '..', '..');
-          execSync('cargo build --release --no-default-features', {
+          execSync(buildCmd, {
             cwd: projectRoot,
             stdio: 'inherit',
-            timeout: 300000,
+            timeout: 600000,  // embed-model (Candle) builds exceed the old 5min
           });
           clearBinaryCache();
           console.log('  \u2705 Build complete');
@@ -374,13 +409,16 @@ function runRepairs(results) {
       case 'binary-missing': {
         console.log('\n  Installing binary...');
         if (isDevMode()) {
+          // No binary to probe \u2014 build the fast FTS5 binary, but point at the
+          // hybrid option so FTS5 isn't silently presented as the only choice.
           console.log('    \u2192 cargo build --release --no-default-features');
+          console.log('      (for semantic search: cargo build --release --features embed-model)');
           try {
             const projectRoot = path.resolve(__dirname, '..', '..');
             execSync('cargo build --release --no-default-features', {
               cwd: projectRoot,
               stdio: 'inherit',
-              timeout: 300000,
+              timeout: 600000,
             });
             clearBinaryCache();
             console.log('  \u2705 Build complete');
@@ -500,7 +538,7 @@ function runDoctor(opts = {}) {
   return { results, issueCount: issues.length };
 }
 
-module.exports = { runDiagnostics, formatReport, runRepairs, runDoctor, surveyHookCoverage, relicRepairGuard, classifyEmbeddings };
+module.exports = { runDiagnostics, formatReport, runRepairs, runDoctor, surveyHookCoverage, relicRepairGuard, classifyEmbeddings, detectEmbedModel, devBuildCommand };
 
 if (require.main === module) {
   const args = process.argv.slice(2);

package/claude-plugin/scripts/doctor.test.js

@@ -122,3 +122,30 @@ test('classifyEmbeddings OK for hybrid (partial + complete) and no-embeddable',
   assert.equal(none.status, 'ok');
   assert.match(none.detail, /no embeddable nodes/);
 });
+
+// ── dev-rebuild feature preservation (no silent hybrid→FTS5 downgrade / ping-pong) ──
+test('devBuildCommand preserves feature set: hybrid → --features embed-model, fts → --no-default-features', () => {
+  const { devBuildCommand } = require('./doctor');
+  assert.match(devBuildCommand(true), /--features embed-model/);
+  assert.doesNotMatch(devBuildCommand(true), /--no-default-features/);
+  assert.match(devBuildCommand(false), /--no-default-features/);
+  assert.doesNotMatch(devBuildCommand(false), /--features embed-model/);
+});
+
+test('detectEmbedModel reads model_available from `health-check --json`; probe failure → null (never a false downgrade signal)', () => {
+  const { detectEmbedModel } = require('./doctor');
+  // hybrid binary
+  const hybridStub = (_bin, args) => {
+    assert.deepEqual(args, ['health-check', '--json']);
+    return JSON.stringify({ model_available: true });
+  };
+  assert.equal(detectEmbedModel('/bin/cg', hybridStub), true);
+  // FTS5-only binary
+  assert.equal(detectEmbedModel('/bin/cg', () => JSON.stringify({ model_available: false })), false);
+  // probe throws (binary broken) → null (caller defaults to FTS5 + note, not a downgrade claim)
+  assert.equal(detectEmbedModel('/bin/cg', () => { throw new Error('boom'); }), null);
+  // unparseable output → null
+  assert.equal(detectEmbedModel('/bin/cg', () => 'not json'), null);
+  // no binary → null
+  assert.equal(detectEmbedModel(null), null);
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sdsrs/code-graph",
-  "version": "0.73.0",
+  "version": "0.73.1",
   "description": "MCP server that indexes codebases into an AST knowledge graph with semantic search, call graph traversal, and HTTP route tracing",
   "license": "MIT",
   "repository": {
@@ -35,10 +35,10 @@
     "node": ">=16"
   },
   "optionalDependencies": {
-    "@sdsrs/code-graph-linux-x64": "0.73.0",
-    "@sdsrs/code-graph-linux-arm64": "0.73.0",
-    "@sdsrs/code-graph-darwin-x64": "0.73.0",
-    "@sdsrs/code-graph-darwin-arm64": "0.73.0",
-    "@sdsrs/code-graph-win32-x64": "0.73.0"
+    "@sdsrs/code-graph-linux-x64": "0.73.1",
+    "@sdsrs/code-graph-linux-arm64": "0.73.1",
+    "@sdsrs/code-graph-darwin-x64": "0.73.1",
+    "@sdsrs/code-graph-darwin-arm64": "0.73.1",
+    "@sdsrs/code-graph-win32-x64": "0.73.1"
   }
 }