Skip to content

Commit f3eac12

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 9a930e7 commit f3eac12

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

api/mcp/tools/__init__.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,11 @@
44
``api.mcp.server``. Import this package to register all tools.
55
"""
66

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

0 commit comments

Comments
 (0)