fix: update README.md version reference from v0.9.16 to v0.9.21#619
fix: update README.md version reference from v0.9.16 to v0.9.21#619okwn wants to merge 3 commits into
Conversation
Updates docs and plugin metadata to reflect the correct MCP tool count. Files updated: - README.md: 51→53 in install instructions, tool section header, and env docs - plugin/.claude-plugin/plugin.json: 51→53 MCP tools in description - plugin/.codex-plugin/plugin.json: 51→53 MCP tools in description Ref: rohitg00#565 Docs and plugin metadata still mix 51/53 MCP tool counts
|
Someone is attempting to deploy a commit to the rohitg00's projects Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughThis PR updates project documentation and plugin manifests to reflect current statistics: test count increased to 1081+ tests, MCP tool surface expanded to 53 tools in proxy mode, and version gate for global installation prompt adjusted to v0.9.21. ChangesProject Statistics and Tool Count Updates
🎯 1 (Trivial) | ⏱️ ~3 minutes
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
README.md (1)
415-415:⚠️ Potential issue | 🟠 MajorUpdate
mcp-standalonetest to enforce exact 53-tool count
test/mcp-standalone.test.tsdoes not currently assert the total MCP tool count is 53—it only checkstools.lengthis>= 41and validates a subset of tool names. Update the test to assert the fullgetAllTools()result size (e.g.,expect(tools.length).toBe(53)) so the README/plugin docs change (51→53) is enforced.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@README.md` at line 415, The test currently only checks tools.length >= 41 and a subset of names; update test/mcp-standalone.test.ts to assert the exact total tool count returned by getAllTools() is 53 (replace the existing length assertion with expect(tools.length).toBe(53)) and keep/adjust any name checks to validate the full toolset if needed; locate the tools variable produced by getAllTools() in that file and change its expectation accordingly so the test enforces the README/plugin docs change.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@README.md`:
- Line 415: The test currently only checks tools.length >= 41 and a subset of
names; update test/mcp-standalone.test.ts to assert the exact total tool count
returned by getAllTools() is 53 (replace the existing length assertion with
expect(tools.length).toBe(53)) and keep/adjust any name checks to validate the
full toolset if needed; locate the tools variable produced by getAllTools() in
that file and change its expectation accordingly so the test enforces the
README/plugin docs change.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 62ea0b42-5978-49d9-bf59-5f4d5b6ae3c6
📒 Files selected for processing (4)
AGENTS.mdREADME.mdplugin/.claude-plugin/plugin.jsonplugin/.codex-plugin/plugin.json
Update the README.md version reference from v0.9.16 to v0.9.21 to match the current version.
Summary by CodeRabbit