Skip to content

fix: mention "me" and "my" in get_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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 24, 2025
Merged

Conversation

toby
Copy link
Member

@toby toby commented Mar 24, 2025

I was having trouble getting things like list my private repos or list the commits on my repo X working. Explicitly mentioning "me" and "my" in the tool description for get_me seems to have helped a lot.

@Copilot Copilot AI review requested due to automatic review settings March 24, 2025 23:24
Copy link
Contributor

@Copilot Copilot AI left a 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

@toby toby merged commit dd8eddf into main Mar 24, 2025
11 checks passed
@toby toby deleted the me-improvements branch March 24, 2025 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant