feat(mcp): trim tools/list to 7 core tools, 5 hidden but callable (v0.10.0)

sdsrss · claude · sdsrss · commit f87cffb4d4f3 · 2026-04-17T03:05:47.000+08:00
Shrink tools/list catalog from 12 → 7 to reduce session-startup tokens
and decision fatigue. The 5 removed tools remain fully callable by name
(backward-compatible aliases in the dispatcher) and via CLI subcommands.

Core 7 (exposed):
- semantic_code_search, get_call_graph, get_ast_node,
- module_overview, project_map, find_references, ast_search

Hidden but callable (aliases preserved):
- impact_analysis, trace_http_chain, dependency_graph,
- find_similar_code, find_dead_code

For the common blast-radius use case, prefer
`get_ast_node symbol_name=X include_impact=true` which is in the core 7.

Tests: 335 passed, 0 failed (default features + embed-model).

Co-Authored-By: Claude Opus 4.7 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/.claude-plugin/marketplace.json b/.claude-plugin/marketplace.json
@@ -5,14 +5,14 @@
   },
   "metadata": {
     "description": "AST knowledge graph plugin for Claude Code — semantic search, call graph, HTTP tracing, impact analysis",
-    "version": "0.9.1"
+    "version": "0.10.0"
   },
   "plugins": [
     {
       "name": "code-graph-mcp",
       "source": "./claude-plugin",
       "description": "AST knowledge graph for intelligent code navigation — auto-indexes your codebase and provides semantic search, call graph traversal, HTTP route tracing, and impact analysis via MCP tools",
-      "version": "0.9.1",
+      "version": "0.10.0",
       "author": {
         "name": "sdsrs"
       },
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,23 @@
 # Changelog
 
+## v0.10.0 — tools/list surface trimmed to 7 core tools
+
+### Migration note
+
+MCP `tools/list` now advertises 7 tools instead of 12. The 5 hidden tools remain fully callable by name (aliases preserved) — only their visibility to the LLM at session start is removed, to shrink tools/list payload (~40% reduction) and cut decision fatigue in daily coding flows.
+
+**Core 7 (exposed in tools/list)**:
+`semantic_code_search`, `get_call_graph`, `get_ast_node`, `module_overview`, `project_map`, `find_references`, `ast_search`.
+
+**Hidden but callable by name / CLI (backward-compatible aliases)**:
+`impact_analysis`, `trace_http_chain`, `dependency_graph`, `find_similar_code`, `find_dead_code`.
+
+**Rationale**: these 5 are niche (cleanup, duplicate detection, HTTP routing, file-level imports, blast-radius pre-check) — high value when needed, low daily frequency. For the primary blast-radius use case, prefer `get_ast_node symbol_name=X include_impact=true` which is in the core 7.
+
+**Reverse / opt-out**: call any hidden tool by name via MCP `tools/call` or the matching `code-graph-mcp <subcommand>` CLI. All handlers, schemas, and CLI paths unchanged — only the tools/list catalog shrunk.
+
+**Memory sync**: projects that auto-adopted v0.9.x will see updated `plugin_code_graph_mcp.md` decision tables on next session.
+
 ## v0.9.1 — Rust 1.95 clippy cleanup
 
 CI-only cleanup; no runtime behavior changes, no user-visible differences. Fixes 9 clippy errors surfaced by Rust 1.95.0's stricter lints (pre-existing since ~v0.8.1, was shipping with red CI):
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "code-graph-mcp"
-version = "0.9.1"
+version = "0.10.0"
 edition = "2021"
 
 [features]
diff --git a/claude-plugin/.claude-plugin/plugin.json b/claude-plugin/.claude-plugin/plugin.json
@@ -4,7 +4,7 @@
   "author": {
     "name": "sdsrs"
   },
-  "version": "0.9.1",
+  "version": "0.10.0",
   "keywords": [
     "code-graph",
     "ast",
diff --git a/npm/darwin-arm64/package.json b/npm/darwin-arm64/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sdsrs/code-graph-darwin-arm64",
-  "version": "0.9.1",
+  "version": "0.10.0",
   "description": "code-graph-mcp binary for macOS ARM64",
   "license": "MIT",
   "repository": {
diff --git a/npm/darwin-x64/package.json b/npm/darwin-x64/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sdsrs/code-graph-darwin-x64",
-  "version": "0.9.1",
+  "version": "0.10.0",
   "description": "code-graph-mcp binary for macOS x64",
   "license": "MIT",
   "repository": {
diff --git a/npm/linux-arm64/package.json b/npm/linux-arm64/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sdsrs/code-graph-linux-arm64",
-  "version": "0.9.1",
+  "version": "0.10.0",
   "description": "code-graph-mcp binary for Linux ARM64",
   "license": "MIT",
   "repository": {
diff --git a/npm/linux-x64/package.json b/npm/linux-x64/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sdsrs/code-graph-linux-x64",
-  "version": "0.9.1",
+  "version": "0.10.0",
   "description": "code-graph-mcp binary for Linux x64",
   "license": "MIT",
   "repository": {
diff --git a/npm/win32-x64/package.json b/npm/win32-x64/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sdsrs/code-graph-win32-x64",
-  "version": "0.9.1",
+  "version": "0.10.0",
   "description": "code-graph-mcp binary for Windows x64",
   "license": "MIT",
   "repository": {
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@sdsrs/code-graph",
-  "version": "0.9.1",
+  "version": "0.10.0",
   "description": "MCP server that indexes codebases into an AST knowledge graph with semantic search, call graph traversal, and HTTP route tracing",
   "license": "MIT",
   "repository": {
@@ -34,10 +34,10 @@
     "node": ">=16"
   },
   "optionalDependencies": {
-    "@sdsrs/code-graph-linux-x64": "0.9.1",
-    "@sdsrs/code-graph-linux-arm64": "0.9.1",
-    "@sdsrs/code-graph-darwin-x64": "0.9.1",
-    "@sdsrs/code-graph-darwin-arm64": "0.9.1",
-    "@sdsrs/code-graph-win32-x64": "0.9.1"
+    "@sdsrs/code-graph-linux-x64": "0.10.0",
+    "@sdsrs/code-graph-linux-arm64": "0.10.0",
+    "@sdsrs/code-graph-darwin-x64": "0.10.0",
+    "@sdsrs/code-graph-darwin-arm64": "0.10.0",
+    "@sdsrs/code-graph-win32-x64": "0.10.0"
   }
 }
diff --git a/src/mcp/tools.rs b/src/mcp/tools.rs
@@ -4,9 +4,13 @@ use serde_json::json;
 /// Expected tool count — update this when adding/removing tools.
 /// Management tools (start_watch, stop_watch, get_index_status, rebuild_index)
 /// are still callable via tools/call but hidden from tools/list to save tokens.
+/// Niche tools (trace_http_chain, impact_analysis, dependency_graph,
+/// find_similar_code, find_dead_code) are also hidden — still callable by name
+/// for CLI / advanced use, but omitted from the default LLM surface to shrink
+/// tools/list token overhead (~40% reduction in listing payload).
 /// Merged tools (find_http_route → trace_http_chain, read_snippet → get_ast_node)
 /// remain callable as aliases for backward compatibility.
-pub const TOOL_COUNT: usize = 12;
+pub const TOOL_COUNT: usize = 7;
 
 pub struct ToolRegistry {
     tools: Vec<ToolDefinition>,
@@ -53,19 +57,6 @@ impl ToolRegistry {
                     "required": ["symbol_name"]
                 }),
             },
-            ToolDefinition {
-                name: "trace_http_chain".into(),
-                description: "Trace HTTP route to handler and downstream calls. Use when: debugging API endpoints or finding which handler serves a route. depth=1 for handler only.".into(),
-                input_schema: json!({
-                    "type": "object",
-                    "properties": {
-                        "route_path": { "type": "string", "description": "Route e.g. '/api/users' or 'POST /api/login'" },
-                        "depth": { "type": "number", "description": "Call chain depth (default 3, use 1 for handler only)" },
-                        "include_middleware": { "type": "boolean", "description": "Include middleware (default true)" }
-                    },
-                    "required": ["route_path"]
-                }),
-            },
             ToolDefinition {
                 name: "get_ast_node".into(),
                 description: "Get symbol details: type, signature, code, references, impact. Use when: inspecting a function/class before editing it. Accepts symbol_name, node_id, or file_path+symbol_name.".into(),
@@ -95,20 +86,6 @@ impl ToolRegistry {
                     "required": []
                 }),
             },
-            ToolDefinition {
-                name: "impact_analysis".into(),
-                description: "Blast radius before modifying code. Use when: about to change/rename/remove a function — shows risk level, affected callers, routes, and files.".into(),
-                input_schema: json!({
-                    "type": "object",
-                    "properties": {
-                        "symbol_name": { "type": "string", "description": "Symbol to analyze" },
-                        "file_path": { "type": "string", "description": "Disambiguate same-name symbols" },
-                        "change_type": { "type": "string", "enum": ["signature", "behavior", "remove"], "description": "Change type (default 'behavior')" },
-                        "depth": { "type": "number", "description": "Max depth (default 3)" }
-                    },
-                    "required": ["symbol_name"]
-                }),
-            },
             ToolDefinition {
                 name: "module_overview".into(),
                 description: "Module structure and symbols. Use when: exploring a directory/module you haven't seen, or finding the right file to edit. Shows exports, hot paths, files.".into(),
@@ -121,34 +98,6 @@ impl ToolRegistry {
                     "required": ["path"]
                 }),
             },
-            ToolDefinition {
-                name: "dependency_graph".into(),
-                description: "File-level import/dependency map. Use when: understanding dependencies before splitting/moving files, checking import chains, or finding circular deps.".into(),
-                input_schema: json!({
-                    "type": "object",
-                    "properties": {
-                        "file_path": { "type": "string", "description": "File to analyze" },
-                        "direction": { "type": "string", "enum": ["outgoing", "incoming", "both"], "description": "Direction (default 'both')" },
-                        "depth": { "type": "number", "description": "Max depth (default 2)" },
-                        "compact": { "type": "boolean", "description": "Compact mode: paths+counts only, no symbol details (saves tokens)" }
-                    },
-                    "required": ["file_path"]
-                }),
-            },
-            ToolDefinition {
-                name: "find_similar_code".into(),
-                description: "Find semantically similar functions. Use when: looking for duplicate logic to extract, consistent refactoring, or related patterns. Requires embeddings.".into(),
-                input_schema: json!({
-                    "type": "object",
-                    "properties": {
-                        "symbol_name": { "type": "string", "description": "Symbol name (provide this OR node_id)" },
-                        "node_id": { "type": "number", "description": "Node ID (provide this OR symbol_name)" },
-                        "top_k": { "type": "number", "description": "Results count (default 5)" },
-                        "max_distance": { "type": "number", "description": "Max distance (default 0.8)" }
-                    },
-                    "required": []
-                }),
-            },
             ToolDefinition {
                 name: "ast_search".into(),
                 description: "Structural code search by type/return/params. Use when: finding all functions returning a type, or querying code structure that grep can't express.".into(),
@@ -178,21 +127,6 @@ impl ToolRegistry {
                     "required": ["symbol_name"]
                 }),
             },
-            ToolDefinition {
-                name: "find_dead_code".into(),
-                description: "Find unused code (orphans, exported-unused). Use when: cleaning up codebase, finding safe-to-delete functions, or reviewing for unused exports.".into(),
-                input_schema: json!({
-                    "type": "object",
-                    "properties": {
-                        "path": { "type": "string", "description": "Directory/file path filter (e.g. 'src/auth/')" },
-                        "node_type": { "type": "string", "enum": ["fn", "class", "struct", "enum", "interface", "type", "const", "var"], "description": "Filter by node type" },
-                        "include_tests": { "type": "boolean", "description": "Include test symbols (default false)" },
-                        "min_lines": { "type": "integer", "description": "Minimum lines to report (default 3)" },
-                        "compact": { "type": "boolean", "description": "Compact mode: name+file+line only, no code (default true)" }
-                    },
-                    "required": []
-                }),
-            },
         ];
 
         debug_assert_eq!(tools.len(), TOOL_COUNT,
@@ -221,10 +155,10 @@ mod tests {
         let registry = ToolRegistry::new();
         let names: Vec<&str> = registry.list_tools().iter().map(|t| t.name.as_str()).collect();
         for expected in [
-            "semantic_code_search", "get_call_graph", "trace_http_chain",
-            "get_ast_node", "project_map", "impact_analysis",
-            "module_overview", "dependency_graph", "find_similar_code",
-            "ast_search", "find_references", "find_dead_code",
+            "semantic_code_search", "get_call_graph",
+            "get_ast_node", "project_map",
+            "module_overview",
+            "ast_search", "find_references",
         ] {
             assert!(names.contains(&expected), "missing tool: {}", expected);
         }
@@ -236,6 +170,12 @@ mod tests {
         assert!(!names.contains(&"stop_watch"));
         assert!(!names.contains(&"get_index_status"));
         assert!(!names.contains(&"rebuild_index"));
+        // Niche tools hidden from tools/list (callable by name)
+        assert!(!names.contains(&"trace_http_chain"));
+        assert!(!names.contains(&"impact_analysis"));
+        assert!(!names.contains(&"dependency_graph"));
+        assert!(!names.contains(&"find_similar_code"));
+        assert!(!names.contains(&"find_dead_code"));
     }
 
     #[test]

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@sdsrs/code-graph-darwin-arm64",`
`3`		`- "version": "0.9.1",`
	`3`	`+ "version": "0.10.0",`
`4`	`4`	`"description": "code-graph-mcp binary for macOS ARM64",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"repository": {`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@sdsrs/code-graph-darwin-x64",`
`3`		`- "version": "0.9.1",`
	`3`	`+ "version": "0.10.0",`
`4`	`4`	`"description": "code-graph-mcp binary for macOS x64",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"repository": {`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@sdsrs/code-graph-linux-arm64",`
`3`		`- "version": "0.9.1",`
	`3`	`+ "version": "0.10.0",`
`4`	`4`	`"description": "code-graph-mcp binary for Linux ARM64",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"repository": {`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@sdsrs/code-graph-linux-x64",`
`3`		`- "version": "0.9.1",`
	`3`	`+ "version": "0.10.0",`
`4`	`4`	`"description": "code-graph-mcp binary for Linux x64",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"repository": {`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@sdsrs/code-graph-win32-x64",`
`3`		`- "version": "0.9.1",`
	`3`	`+ "version": "0.10.0",`
`4`	`4`	`"description": "code-graph-mcp binary for Windows x64",`
`5`	`5`	`"license": "MIT",`
`6`	`6`	`"repository": {`