docs: sync MCP tool counts across metadata#566
Conversation
|
@yixinforu is attempting to deploy a commit to the rohitg00's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✅ Files skipped from review due to trivial changes (3)
📝 WalkthroughWalkthroughREADME, example env, plugin metadata, a proxy comment, and the website compare table were updated to consistently report a 53-tool MCP surface and a core visibility of 8 tools (previously mixed 51/53 and 7/8 references). ChangesMCP Tool Count and Visibility
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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.
Inline comments:
In `@README.md`:
- Around line 1192-1193: The README currently contradicts itself: the comment
"Tool visibility: 'core' (8 tools)" (AGENTMEMORY_TOOLS=core) doesn't match the
"Core tools (always available)" list which contains 11 entries; update one of
them to be consistent. Either change the count in the AGENTMEMORY_TOOLS comment
from 8 to 11 (update the text "core (8 tools)" to "core (11 tools)"), or prune
the "Core tools (always available)" list to remove three tools so it truly
contains 8 entries; edit the section titled "Core tools (always available)"
accordingly and ensure the AGENTMEMORY_TOOLS=core comment reflects the final
chosen count.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
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)
825-859:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winClarify the “53 total” extended-tools section to avoid count mismatch confusion.
This heading says “53 total,” but the table below does not enumerate enough tools to reconcile with
core (8)+all (53). Please either make this list exhaustive or label it as a representative subset (and point to the canonical registry/source of truth).🤖 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` around lines 825 - 859, The "Extended tools (53 total — set AGENTMEMORY_TOOLS=all)" header in README.md does not match the tool table contents; update the heading and/or table so counts are consistent: either expand the table to list all 53 tools (ensuring each tool name like memory_patterns, memory_timeline, memory_relations, etc. is included) or change the header to indicate this is a representative subset (e.g., "Representative extended tools") and add a short sentence pointing to the canonical registry/source of truth (the external list or variable that defines AGENTMEMORY_TOOLS) so readers can find the full exhaustive list.
🤖 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`:
- Around line 825-859: The "Extended tools (53 total — set
AGENTMEMORY_TOOLS=all)" header in README.md does not match the tool table
contents; update the heading and/or table so counts are consistent: either
expand the table to list all 53 tools (ensuring each tool name like
memory_patterns, memory_timeline, memory_relations, etc. is included) or change
the header to indicate this is a representative subset (e.g., "Representative
extended tools") and add a short sentence pointing to the canonical
registry/source of truth (the external list or variable that defines
AGENTMEMORY_TOOLS) so readers can find the full exhaustive list.
|
Fixed. Kept the core count at 8 because that matches getVisibleTools()/ESSENTIAL_TOOLS, and updated the README table to |
1 participant
Summary
AGENTMEMORY_TOOLSdocs with the current 8 default visible tools and 53 full toolsAGENTMEMORY_TOOLS=coresurfaceCloses #565
Summary by CodeRabbit