docs(npm): align README language and MCP tool counts with the root README - #1765
Merged
Conversation
…ADME pkg/npm/README.md claimed 159 languages and 14 MCP tools while the root README says 158 languages and 15 MCP tools. The tool count is verifiably wrong: the TOOLS[] table in src/mcp/mcp.c has 15 entries. The language count is aligned to the root README, which states 158 consistently in five places. Each stale figure appeared twice; both occurrences of each are fixed, so the package README no longer contradicts itself or the root README. Distilled from two PRs that each caught one half: #967 by ketpatil77 (language count) and #1268 by nhtkylc91 (tool count). Neither fixed all four occurrences, and both touched the same file, so they are combined here with credit to both. Note for whoever refreshes these next: 159 grammar directories exist under internal/cbm/vendored/grammars/, which does not reconcile trivially with either figure (some languages share a grammar; some enums have no vendored grammar). This change deliberately does not re-derive the number, only stops the two READMEs disagreeing. Co-authored-by: ketpatil77 <ket.patil77@gmail.com> Co-authored-by: nhtkylc91 <nhtkylc@hotmail.com> Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
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
1 participant
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.
Supersedes #967 (@ketpatil77) and #1268 (@nhtkylc91), each of which caught one half of this. Both credited via
Co-authored-by.pkg/npm/README.mdclaimed 159 languages / 14 MCP tools; the root README says 158 languages / 15 MCP tools. So the package README contradicted both the root README and, after a partial fix, itself.TOOLS[]table insrc/mcp/mcp.chas exactly 15 entries (index_repository,search_graph,query_graph,trace_path,get_code_snippet,get_graph_schema,get_architecture,search_code,list_projects,delete_project,index_status,check_index_coverage,detect_changes,manage_adr,ingest_traces).Each stale figure appeared twice in the file (line 10 carries both), which is why neither original PR was complete on its own — and since they touch the same file, merging both would have required a rebase and still left line 10 wrong.
Worth flagging for a future docs pass
There are 159 grammar directories under
internal/cbm/vendored/grammars/, which doesn't reconcile trivially with either figure — some languages share a grammar, and someCBM_LANG_*enums have no vendored grammar at all. If 158 is itself stale, that's a separate fix that should update the root README too; this PR deliberately doesn't touch it.