@sdsrs/code-graph 0.23.1 → 0.24.1

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

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

package/claude-plugin/scripts/adopt.js

@@ -33,9 +33,16 @@ function readAdoptedBy(filePath) {
 // scratch dirs, mixed repos). Per-type variants live in `buildIndexLine` and
 // are computed per-cwd at adopt + needsRefresh time. Adopted-project receives
 // the typed variant; everyone else falls back to this canonical line.
+// Tags MUST be ≥4 chars and topic-specific (per claudemd §11-EXT Tag-specificity).
+// Generic single-word English tags (impact / refs / overview / semantic / deps /
+// trace / route / similar) substring-match release-notes / commit-message prose
+// via the §11 read-the-file hook regex (word-boundary + 0–2 declension chars),
+// producing false-positive denies. Each tag below aligns with its MCP tool name
+// (impact_analysis / find_references / module_overview / …) so hyphenated literals
+// never collide with natural prose.
 const INDEX_LINE =
   '- [code-graph-mcp](plugin_code_graph_mcp.md) ' +
-  '[impact, callgraph, refs, overview, semantic, ast-search, dead-code, similar, deps, trace] — ' +
+  '[impact-analysis, callgraph, find-references, module-overview, semantic-search, ast-search, dead-code, find-similar-code, dependency-graph, trace-http-chain] — ' +
   '改 X 影响面/谁调用 X/X 被谁用/看 X 源码/Y 模块长啥样/概念查询 优先于 Grep；字面匹配走 Grep。' +
   '核心 7（get_call_graph/module_overview/semantic_code_search/ast_search/find_references/get_ast_node/project_map）' +
   '+ 进阶 5（impact_analysis/trace_http_chain/dependency_graph/find_similar_code/find_dead_code），决策表见全文';
@@ -235,12 +242,12 @@ function buildIndexLine(projectType = 'generic') {
     case 'web-py':
     case 'web-go':
       return prefix +
-        '[trace, route, callgraph, impact, refs, overview, semantic, deps] — ' +
+        '[trace-http-chain, http-route, callgraph, impact-analysis, find-references, module-overview, semantic-search, dependency-graph] — ' +
         'HTTP 路由→handler 链路用 trace_http_chain（或 get_call_graph route_path=）；改 handler 影响面用 impact；' +
         '其他结构化查询同上 优先于 Grep。' + coreSuffix;
     case 'frontend':
       return prefix +
-        '[refs, overview, semantic, callgraph, impact, ast-search] — ' +
+        '[find-references, module-overview, semantic-search, callgraph, impact-analysis, ast-search] — ' +
         '组件重命名/重构用 find_references（含 imports/inherits）；模块层级用 module_overview；' +
         '改 props/接口前用 impact 看下游；HTTP route 通常不适用。' + coreSuffix;
     case 'rust':
@@ -248,7 +255,7 @@ function buildIndexLine(projectType = 'generic') {
     case 'python':
     case 'node':
       return prefix +
-        '[callgraph, impact, refs, overview, semantic, ast-search, dead-code, deps] — ' +
+        '[callgraph, impact-analysis, find-references, module-overview, semantic-search, ast-search, dead-code, dependency-graph] — ' +
         '改 X 影响面/谁调用 X/Y 模块 优先于 Grep；HTTP route 追踪通常不适用（无 web 框架）；' +
         '字面匹配走 Grep。' + coreSuffix;
     case 'generic':

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

@@ -648,7 +648,7 @@ test('buildIndexLine generic returns the canonical INDEX_LINE byte-for-byte', ()
 
 test('buildIndexLine web-rs prepends route/trace tags + handler-focused lead', () => {
   const line = buildIndexLine('web-rs');
-  assert.match(line, /\[trace, route,/, 'web-rs index line should lead with trace/route tags');
+  assert.match(line, /\[trace-http-chain, http-route,/, 'web-rs index line should lead with trace-http-chain/http-route tags');
   assert.match(line, /HTTP 路由/, 'lead sentence should mention HTTP routes');
 });
 
@@ -675,6 +675,57 @@ test('adopt + needsRefresh agree on typed INDEX_LINE — no spurious refresh in
   } finally { sb.cleanup(); }
 });
 
+test('stale INDEX_LINE → adopt rewrites in place without duplicating sentinel blocks', () => {
+  // Regression for the v0.24+ tag-rename fix (feedback_adoption_tag_specificity).
+  // If a user's MEMORY.md was written by an older code-graph-mcp (pre-rename),
+  // SessionStart → maybeAutoAdopt → needsRefresh must detect the drift,
+  // stripSentinelBlock must locate the old v1 block by sentinel marker, and
+  // the rewrite must end with exactly one BEGIN/END pair. Breaks if anyone
+  // bumps SENTINEL_BEGIN without teaching stripSentinelBlock to also match
+  // the prior version — would leave an orphan v1 block plus a new v2 block.
+  const sb = makeSandbox();
+  try {
+    fs.writeFileSync(path.join(sb.cwd, 'Cargo.toml'), '[package]\nname="x"\n');
+    // Plant pre-rename MEMORY.md: well-formed v1 sentinel + an obsolete tag
+    // line that no current buildIndexLine variant produces. Exact prior bytes
+    // don't matter — only that it differs from today's desiredBlock.
+    const stalePayload =
+      '- [code-graph-mcp](plugin_code_graph_mcp.md) [obsolete-tag, another-stale] — stale lead sentence kept for drift detection.';
+    const indexPath = path.join(sb.dir, 'MEMORY.md');
+    const neighbor = '- [user_profile.md](user_profile.md) — neighbor (must survive)';
+    fs.writeFileSync(
+      indexPath,
+      `# Memory Index\n\n${neighbor}\n\n${SENTINEL_BEGIN}\n${stalePayload}\n${SENTINEL_END}\n`
+    );
+    // Plant target file so isAdopted/needsRefresh treat this as a real prior
+    // adoption (body bytewise-identical to shipped template — drift is only
+    // in MEMORY.md, not the template body).
+    const tplBody = fs.readFileSync(TEMPLATE_PATH);
+    const marker = Buffer.from(`<!-- adopted-by: ${sb.cwd} -->\n`);
+    fs.writeFileSync(path.join(sb.dir, TARGET_NAME), Buffer.concat([marker, tplBody]));
+
+    assert.strictEqual(isAdopted({ cwd: sb.cwd, home: sb.home }), true,
+      'precondition: planted state should look adopted');
+    assert.strictEqual(needsRefresh({ cwd: sb.cwd, home: sb.home }), true,
+      'tag-list drift must trigger refresh');
+
+    adopt({ cwd: sb.cwd, home: sb.home });
+
+    const after = fs.readFileSync(indexPath, 'utf8');
+    const escBegin = SENTINEL_BEGIN.replace(/[-/\\^$*+?.()|[\]{}]/g, '\\$&');
+    const escEnd = SENTINEL_END.replace(/[-/\\^$*+?.()|[\]{}]/g, '\\$&');
+    const beginHits = (after.match(new RegExp(escBegin, 'g')) || []).length;
+    const endHits = (after.match(new RegExp(escEnd, 'g')) || []).length;
+    assert.strictEqual(beginHits, 1, 'exactly one sentinel BEGIN after rewrite (no duplicate blocks)');
+    assert.strictEqual(endHits, 1, 'exactly one sentinel END after rewrite');
+    assert.ok(!after.includes('obsolete-tag'), 'stale tag list must be gone');
+    assert.ok(!after.includes('another-stale'), 'stale tag list must be gone');
+    assert.ok(after.includes(neighbor), 'neighbor entry preserved');
+    assert.strictEqual(needsRefresh({ cwd: sb.cwd, home: sb.home }), false,
+      'post-refresh needsRefresh must be false (no SessionStart refresh loop)');
+  } finally { sb.cleanup(); }
+});
+
 // 2A — false-positive hardening: comment-strip + section-aware scan.
 
 test('detectProjectType ignores commented-out web-framework deps in Cargo.toml', () => {

package/package.json

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