@sdsrs/code-graph 0.76.1 → 0.76.3

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

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

package/claude-plugin/scripts/auto-update.js

@@ -354,9 +354,10 @@ async function downloadAndInstall(latest, {
   exec = execFileSync,
   downloadBin = downloadBinary,
   refreshMarketplace = refreshMarketplaceClone,
+  cmdExists = commandExists,
 } = {}) {
   // Pre-flight: check required CLI tools before attempting any download
-  const missingTools = ['curl', 'tar'].filter(cmd => !commandExists(cmd));
+  const missingTools = ['curl', 'tar'].filter(cmd => !cmdExists(cmd));
   if (missingTools.length > 0) {
     console.error(`[code-graph] Auto-update skipped: missing required tools: ${missingTools.join(', ')}. Install them to enable auto-updates.`);
     return { pluginUpdated: false, binaryUpdated: false };
@@ -393,30 +394,35 @@ async function downloadAndInstall(latest, {
       pluginUpdated = true;
     }
 
-    // Update installed_plugins.json to point to new version
-    const installedPath = installedPluginsPath();
-    try {
-      const installed = readJson(installedPath);
-      if (installed && installed.plugins && installed.plugins[PLUGIN_ID]) {
-        installed.plugins[PLUGIN_ID][0].installPath = pluginDst;
-        installed.plugins[PLUGIN_ID][0].version = latest.version;
-        installed.plugins[PLUGIN_ID][0].lastUpdated = new Date().toISOString();
-        writeJsonAtomic(installedPath, installed);
-      }
-    } catch { /* not fatal */ }
-
-    // Update install manifest
-    try {
-      const manifest = readManifest();
-      manifest.version = latest.version;
-      manifest.updatedAt = new Date().toISOString();
-      writeJsonAtomic(path.join(CACHE_DIR, 'install-manifest.json'), manifest);
-    } catch { /* not fatal */ }
-
-    // Run the NEW lifecycle.js to update settings.json hooks with new paths.
-    // Without this, settings.json hooks still point to the old version directory
-    // until the next session's self-heal corrects them.
+    // Repoint state at the new version ONLY if the plugin copy actually landed.
+    // Guarding on pluginUpdated: when the copy above was skipped (pluginSrc absent, or
+    // its plugin.json version drifted from the tag — the project's version sync is known
+    // fragile), pluginDst was never created. Advancing installPath/manifest to it anyway
+    // pointed Claude Code at a nonexistent install dir while state read "up to date".
     if (pluginUpdated) {
+      // Update installed_plugins.json to point to new version
+      const installedPath = installedPluginsPath();
+      try {
+        const installed = readJson(installedPath);
+        if (installed && installed.plugins && installed.plugins[PLUGIN_ID]) {
+          installed.plugins[PLUGIN_ID][0].installPath = pluginDst;
+          installed.plugins[PLUGIN_ID][0].version = latest.version;
+          installed.plugins[PLUGIN_ID][0].lastUpdated = new Date().toISOString();
+          writeJsonAtomic(installedPath, installed);
+        }
+      } catch { /* not fatal */ }
+
+      // Update install manifest
+      try {
+        const manifest = readManifest();
+        manifest.version = latest.version;
+        manifest.updatedAt = new Date().toISOString();
+        writeJsonAtomic(path.join(CACHE_DIR, 'install-manifest.json'), manifest);
+      } catch { /* not fatal */ }
+
+      // Run the NEW lifecycle.js to update settings.json hooks with new paths.
+      // Without this, settings.json hooks still point to the old version directory
+      // until the next session's self-heal corrects them.
       try {
         const newLifecycle = path.join(pluginDst, 'scripts', 'lifecycle.js');
         if (fs.existsSync(newLifecycle)) {

package/claude-plugin/scripts/auto-update.test.js

@@ -355,6 +355,7 @@ test('downloadAndInstall wires the marketplace refresh + binary download (orches
       let binDownloads = 0;
       const result = await downloadAndInstall(latest, {
         exec,
+        cmdExists: () => true, // don't depend on host curl/tar
         refreshMarketplace: () => { refreshed++; return true; },
         downloadBin: async () => { binDownloads++; return true; },
       });
@@ -376,3 +377,58 @@ test('downloadAndInstall wires the marketplace refresh + binary download (orches
     'code-graph-mcp', 'code-graph-mcp', '9.9.9', '.claude-plugin', 'plugin.json');
   assert.equal(fs.existsSync(dst), true, 'plugin copied into sandbox plugins cache');
 });
+
+test('downloadAndInstall does NOT repoint install state when the plugin copy is skipped (version drift)', async (t) => {
+  // Guard for a silent-breakage bug: when the extracted tarball's plugin.json version
+  // doesn't match latest.version, the copy is skipped and pluginDst is never created.
+  // installed_plugins.json must NOT be advanced to that nonexistent dir, or Claude Code
+  // ends up pointed at a missing install while state reads "up to date".
+  const sandboxHome = mkDir(t, 'code-graph-dai-skip-');
+  const installedDir = path.join(sandboxHome, '.claude', 'plugins');
+  fs.mkdirSync(installedDir, { recursive: true });
+  const installedPath = path.join(installedDir, 'installed_plugins.json');
+  fs.writeFileSync(installedPath, JSON.stringify({
+    plugins: { 'code-graph-mcp@code-graph-mcp': [
+      { installPath: '/old/install/path', version: '0.0.1', lastUpdated: 'seed' },
+    ] },
+  }));
+
+  const script = `
+    const fs = require('fs');
+    const path = require('path');
+    const { downloadAndInstall } = require(${JSON.stringify(path.join(__dirname, 'auto-update.js'))});
+    const latest = { version: '9.9.9', tarballUrl: 'https://example/tar', binaryUrl: null };
+    const exec = (cmd, args) => {
+      if (cmd === 'tar') {
+        // Extract a claude-plugin/ whose version DRIFTS from latest → copy is skipped.
+        const tmpDir = args[args.indexOf('-C') + 1];
+        const mDir = path.join(tmpDir, 'claude-plugin', '.claude-plugin');
+        fs.mkdirSync(mDir, { recursive: true });
+        fs.writeFileSync(path.join(mDir, 'plugin.json'), JSON.stringify({ version: '0.0.0' }));
+      }
+    };
+    (async () => {
+      const result = await downloadAndInstall(latest, {
+        exec,
+        cmdExists: () => true, // don't depend on host curl/tar — exercise the guard deterministically
+        refreshMarketplace: () => true,
+        downloadBin: async () => true,
+      });
+      console.log(JSON.stringify({ result }));
+    })();
+  `;
+  const out = execGit(process.execPath, ['-e', script], {
+    env: { ...process.env, HOME: sandboxHome },
+    encoding: 'utf8',
+  });
+  const { result } = JSON.parse(out.trim().split('\n').pop());
+  assert.equal(result.pluginUpdated, false, 'version drift must skip the plugin copy');
+
+  // The pre-seeded record must be UNTOUCHED — not repointed to the 9.9.9 dir.
+  const rec = JSON.parse(fs.readFileSync(installedPath, 'utf8'))
+    .plugins['code-graph-mcp@code-graph-mcp'][0];
+  assert.equal(rec.installPath, '/old/install/path',
+    'installPath must not be repointed when the copy was skipped');
+  assert.equal(rec.version, '0.0.1',
+    'version must not be advanced when the copy was skipped');
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sdsrs/code-graph",
-  "version": "0.76.1",
+  "version": "0.76.3",
   "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.76.1",
-    "@sdsrs/code-graph-linux-arm64": "0.76.1",
-    "@sdsrs/code-graph-darwin-x64": "0.76.1",
-    "@sdsrs/code-graph-darwin-arm64": "0.76.1",
-    "@sdsrs/code-graph-win32-x64": "0.76.1"
+    "@sdsrs/code-graph-linux-x64": "0.76.3",
+    "@sdsrs/code-graph-linux-arm64": "0.76.3",
+    "@sdsrs/code-graph-darwin-x64": "0.76.3",
+    "@sdsrs/code-graph-darwin-arm64": "0.76.3",
+    "@sdsrs/code-graph-win32-x64": "0.76.3"
   }
 }