Skip to content

docs(npm): fix stale MCP tool count in package README - #1268

Open
nhtkylc91 wants to merge 1 commit into
DeusData:mainfrom
nhtkylc91:fix/npm-readme-tool-count
Open

docs(npm): fix stale MCP tool count in package README#1268
nhtkylc91 wants to merge 1 commit into
DeusData:mainfrom
nhtkylc91:fix/npm-readme-tool-count

Conversation

@nhtkylc91

Copy link
Copy Markdown

Summary

  • `pkg/npm/README.md` claimed 14 MCP tools, but the actual `TOOLS[]` registry in `src/mcp/mcp.c` defines 15 (verified by counting entries in the array), matching the count already stated in the root `README.md`.
  • Fixes both occurrences in the npm package README so it's consistent with the source of truth and the other docs.

Test plan

  • Counted tool definitions in `src/mcp/mcp.c`'s `TOOLS[]` array: 15.
  • Confirmed root `README.md` and `pkg/npm/README.md` now agree (15).
  • Docs-only change, no code/behavior affected.

pkg/npm/README.md said "14 MCP tools" while the actual TOOLS[] registry
in src/mcp/mcp.c defines 15, matching the count already used in the
root README.md.
@nhtkylc91
nhtkylc91 requested a review from DeusData as a code owner July 25, 2026 18:42
@DeusData DeusData added documentation Improvements or additions to documentation ux/behavior Display bugs, docs, adoption UX labels Jul 28, 2026
@DeusData DeusData added this to the 0.9.1-rc milestone Jul 28, 2026
@DeusData DeusData added the priority/normal Standard review queue; useful PR with ordinary maintainer urgency. label Jul 28, 2026
@DeusData

DeusData commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Thank you for the contribution and for checking the package README against the tool registry. This is now triaged as a normal-priority documentation correction for 0.9.1-rc. The tool-count point overlaps the newer #1297 and #1315 report and will need to be reconciled against the actual tools/list response during review. Our community PR queue is currently quite full, so it may take a little time before we can complete that review. We are doing our best to support community contributions and will return with code-grounded feedback as capacity opens.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation priority/normal Standard review queue; useful PR with ordinary maintainer urgency. ux/behavior Display bugs, docs, adoption UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants