- Notifications
You must be signed in to change notification settings - Fork899
fix: addedreason argument toget_me function#8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This pull request addresses a bug in the mark3labs/mcp-go library by adding a new "reason" argument to the get_me tool to avoid errors when no arguments are supplied and to provide additional context for debugging.
- Added a "reason" argument with an accompanying description in the tool configuration.
- Updated the tool initialization in pkg/github/server.go to include the new argument.
Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused.Learn more
62b20f7 intomainUh oh!
There was an error while loading.Please reload this page.
The
mark3labs/mcp-golibrary has a bug where it will error when there are no arguments specified for a tool. This adds areasonargument, which works around that bug but is also somewhat helpful for debugging and potentially even returns better results because the LLM can see why it decided to call the tool.