- Notifications
You must be signed in to change notification settings - Fork899
fix: mention "me" and "my" inget_me description#40
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 PR updates the tool description for get_me to explicitly mention "me" and "my" to help users better understand the tool's functionality.
- Update description of the get_me tool to include explicit language for private repos and commits contexts.
Comments suppressed due to low confidence (1)
pkg/github/server.go:88
- Consider correcting 'include' to 'includes' for grammatical accuracy.
mcp.WithDescription(t("TOOL_GET_ME_DESCRIPTION", "Get details of the authenticated GitHub user. Use this when a request include "me", "my"...")),Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab.Learn more
dd8eddf intomainUh oh!
There was an error while loading.Please reload this page.
I was having trouble getting things like
list my private reposorlist the commits on my repo Xworking. Explicitly mentioning "me" and "my" in the tool description forget_meseems to have helped a lot.