Skip to content

Commit 9cafd0a

Browse files
DvirDukhanCopilot
andcommitted
feat(mcp): drop GraphRAG ask from the registered MCP tool surface
Benchmarking showed the MCP ask tool failing on every call (the spawned server env carries only FalkorDB coordinates, no LLM key) and, when keyed, returning File-level fuzzy matches rather than the structural answers the nav workflow needs — an LLM round-trip per call for no signal. Expose only the deterministic structural tools; GraphRAG ask stays on the HTTP /api/chat path. #681's ask implementation is left intact (re-enable by re-adding the import). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 338a9b8 commit 9cafd0a

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

api/mcp/tools/__init__.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,12 @@
44
``api.mcp.server``. Import this package to register all tools.
55
"""
66

7-
from . import ask, structural # noqa: F401 (registers tools on import)
7+
# NOTE: the GraphRAG-backed ``ask`` tool is intentionally NOT registered.
8+
# In benchmarking it failed on every call ("Missing GOOGLE_API_KEY/GEMINI_API_KEY")
9+
# because the spawned MCP server env carries only FalkorDB coordinates, and even
10+
# when keyed it returned File-level fuzzy matches rather than the structural
11+
# answers the nav workflow needs. Burning an LLM round-trip per call for no signal,
12+
# so we expose only the deterministic structural tools. To re-enable, add ``ask``
13+
# back to the import below (and ensure GEMINI_API_KEY is in the server env).
14+
# GraphRAG ask remains available on the HTTP /api/chat path.
15+
from . import structural # noqa: F401 (registers tools on import)

0 commit comments

Comments
 (0)