docs(readme): correct tool count to 14, clarify semantic_query and artifact persistence - #1315
Open
Solaris-star wants to merge 1 commit into
Open
Conversation
…tifact persistence Three factual corrections flagged in DeusData#1297: 1. Tool count is 14, not 15 — the server advertises 14 via tools/list (paginated 8+6), and the README's own MCP Tools table lists 14 rows. Fixed in the intro, feature list, manual-config verify step, and the source-layout comment. 2. semantic_query is a parameter of search_graph, not a standalone tool — calling it as a tool returns 'unknown tool'. Reworded the Search section to say 'via the semantic_query parameter of search_graph'. 3. The team-shared artifact is opt-in — persistence defaults to false (src/mcp/mcp.c:376), so the artifact is only written when 'persistence: true' is passed. Clarified the artifact paragraph. Fixes DeusData#1297 Signed-off-by: Solaris-star <820622658@qq.com>
Solaris-star
force-pushed
the
fix/1297-readme-tool-count-and-params
branch
from
July 28, 2026 06:23
140d572 to
0f985c6
Compare
DeusData
added
documentation
Owner
|
Thank you for the contribution and for grounding the README corrections in the advertised tool list and current persistence default. This is now triaged as a normal-priority documentation correction for |
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #1297
Three factual corrections to the README:
Tool count is 14, not 15 — the server advertises 14 via
tools/list(paginated 8 + 6), and the README's own MCP Tools table lists exactly 14 rows. Fixed in the intro, the feature list, the manual-config verify step, and the source-layout comment.semantic_queryis a parameter ofsearch_graph, not a standalone tool — invoking it as a tool returnsunknown tool: semantic_query. Reworded the Search section to read "via thesemantic_queryparameter ofsearch_graph".The team-shared artifact is opt-in —
persistencedefaults tofalse(src/mcp/mcp.c:376), so no.codebase-memory/graph.db.zstis written unlesspersistence: trueis passed explicitly. Clarified the artifact paragraph.Docs-only change; no code touched.