Skip to content

Fix stale MCP tool-count references (78/81 -> 84)#361

Open
unggulyoga-prog wants to merge 1 commit into
tradesdontlie:mainfrom
unggulyoga-prog:fix/tool-count-docs
Open

Fix stale MCP tool-count references (78/81 -> 84)#361
unggulyoga-prog wants to merge 1 commit into
tradesdontlie:mainfrom
unggulyoga-prog:fix/tool-count-docs

Conversation

@unggulyoga-prog

Copy link
Copy Markdown

Summary

  • README.md, CLAUDE.md, RESEARCH.md, and src/server.js all cited outdated MCP tool counts (78 in three places, 81 in CLAUDE.md).
  • Verified the current count by grepping server.tool( registrations across all files in src/tools/*.js: 84 tools.
  • Updated each reference to 84 for consistency.

Left src/cli/index.js's "70 MCP tools accessible via CLI" comment untouched — that's a different metric (CLI command/subcommand coverage), not total MCP tool count, and I didn't want to guess at that number without verifying it separately.

Test plan

  • npm run lint — no new warnings introduced (pre-existing warnings in unrelated files)
  • npm run test:unit — same 2 pre-existing failures before and after this change (network-dependent Pine compile tests + a pre-existing Windows path bug in sanitization.test.js), confirmed via git stash comparison
  • Doc-only change, no runtime behavior affected

Docs and the server instructions string cited outdated tool counts
(78 in README/RESEARCH/server.js, 81 in CLAUDE.md). Verified the
current count via grep across src/tools/*.js: 84 registered tools.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant