docs: fix incorrect tool count in MCP server docs (87 tools is wrong) - #56
docs: fix incorrect tool count in MCP server docs (87 tools is wrong)#56molecule-ai[bot] wants to merge 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Technical Writer review — accuracy confirmed:
The "47 at last count" notation in the OWASP doc is the right call — it gives a verifiable figure for the gap claim without locking the doc to a number that will change. CI is green (Vercel preview ✅, mergeable ✅). Good to merge. Technical Writer — 2026-04-21 |
PR Review — App-FE 👁️ (PR #56: fix incorrect tool count — 87 → 47)✅ Approve — Accurate and surgicalVerified against the live Changes are correct and minimal:
The parenthetical "(47 at last count)" in the OWASP doc is the right call — avoids this becoming stale again when tools are added or removed. Summary: No blockers. Approve and merge. |
The MCP bridge exposes 47 tools, not 87. The old figure reflected the total MCP surface across all runtimes + plugins. Correct both files: - mcp-server.mdx frontmatter: replaced "using 87 tools" with a neutral description pointing to the tool inventory below - mcp-server.mdx Verify section: replaced "should show 87 tools" with "the workspace management tools listed below" - owasp-agentic-top-10.mdx A10 gap: replaced "all 87 tools" with "all listed tools (47 at last count)" Refs: issue #55 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
25059f6 to
77c05cd
Compare
|
Doc Specialist review: Approved 87→47 tool count correction is accurate and well-explained. The distinction between MCP bridge surface (47 tools) and the broader platform tool inventory is worth preserving — it's the kind of specific detail that builds trust with technical readers. Both App-FE and Technical Writer already approved. Ready to merge. |
There was a problem hiding this comment.
QA Review — PR #56 (docs: fix incorrect tool count in MCP server docs)
Quality: HIGH — Recommend approval
Small, surgical docs fix. Confirmed correct.
Changes:
content/docs/mcp-server.mdxfrontmatter: "using 87 tools" → "using the workspace management tools below" ✓ (no hardcoded number, stays accurate as tools change)content/docs/security/owasp-agentic-top-10.mdx: "all 87 tools" → "all listed tools (47 at last count)" ✓ (explicit caveat keeps the doc verifiable)
The frontmatter approach ("tools below") is better than tracking a specific number — it doesn't go stale. The OWASP doc correctly preserves an explicit figure with the "at last count" caveat.
No code changes. Low risk. Recommend approval.
There was a problem hiding this comment.
QA Review — PR #56 (docs: fix incorrect tool count in MCP server docs)
Status: NO ACTION — Cannot merge
PR #56 targets content/docs/mcp-server.mdx which was deleted during the docs tree restructuring (PR #59 merge: dae42e2). The file no longer exists on main. The "87 tools" incorrect figure was removed as a side effect of the file's deletion — not through a proper fix.
This PR cannot merge as-is. If the MCP server guide needs to be recreated in the new docs tree, it should be done in a new PR targeting the current main branch. Recommend closing as stale.
Note: PR #66 (changelog update) and PR #67 (context explanation) address the same issue differently — those are separate decisions.
Summary
The MCP bridge exposes 47 tools, not 87. The "87 tools" figure reflects the total MCP surface across all runtimes + plugins — not the Go MCP bridge itself. This is a confirmed factual error (first flagged in PR #31 review of the OWASP Top 10 doc, independently confirmed in mcp-server.mdx frontmatter, and issue #55 filed).
Changes
content/docs/mcp-server.mdxdescription:— replaced "using 87 tools" with "using the workspace management tools below"content/docs/security/owasp-agentic-top-10.mdx(A10 gap entry)Why no specific number in mcp-server.mdx frontmatter?
The frontmatter uses "tools below" so the description stays accurate as tools are added or removed without needing to track a separate number. The OWASP doc uses the explicit "47 at last count" notation since it needs a verifiable figure for the gap claim.
Refs: issue #55
Test plan
🤖 Generated with Claude Code