Skip to content

fix: mention "me" and "my" in get_me description - #40

Merged
toby merged 1 commit into
mainfrom
me-improvements
Mar 24, 2025
Merged

fix: mention "me" and "my" in get_me description#40
toby merged 1 commit into
mainfrom
me-improvements

Conversation

@toby

@tobytoby commented Mar 24, 2025

Copy link
Copy Markdown
Member

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.

CopilotAI review requested due to automatic review settings March 24, 2025 23:24

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

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 mainMar 24, 2025
@toby
toby deleted the me-improvements branch March 24, 2025 23:27
DaleSeo pushed a commit to DaleSeo/github-mcp-server that referenced this pull request Oct 24, 2025
Sign up for freeto 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.

2 participants

@toby