fix remove unused mcp function args comment#139
Conversation
|
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 (6)
💤 Files with no reviewable changes (6)
📝 WalkthroughWalkthroughThis PR removes extraneous ChangesMCP Tool Docstring Cleanup
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
Description
Some LLM models see the function description, require the workspace_slug in arguments, and substitute it when executing the functions. However, the functions themselves don't accept this argument.
LLM models try to pass the workspace_slug because they see it in the comment and assume it's needed.
Type of Change
Test Scenarios
Test MCP on small models like Gemma-4-e2b in LM Studio
Summary by CodeRabbit