From d6311ff5e3abe19b2c4184ea138d127a94cb7a5d Mon Sep 17 00:00:00 2001 From: Sam Morrow Date: Mon, 19 May 2025 12:58:15 +0200 Subject: [PATCH 1/3] alter the description of get-me to reduce redundant repeat calls --- pkg/github/context_tools.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/github/context_tools.go b/pkg/github/context_tools.go index f0d3ad8b1..7017effb0 100644 --- a/pkg/github/context_tools.go +++ b/pkg/github/context_tools.go @@ -15,7 +15,7 @@ import ( // GetMe creates a tool to get details of the authenticated user. func GetMe(getClient GetClientFn, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) { return mcp.NewTool("get_me", - mcp.WithDescription(t("TOOL_GET_ME_DESCRIPTION", "Get details of the authenticated GitHub user. Use this when a request include \"me\", \"my\"...")), + mcp.WithDescription(t("TOOL_GET_ME_DESCRIPTION", "Get details of the authenticated GitHub user. Use this when a request include \"me\", \"my\"... the output will not change unless the user changes their profile, so only call this once.")), mcp.WithToolAnnotation(mcp.ToolAnnotation{ Title: t("TOOL_GET_ME_USER_TITLE", "Get my user profile"), ReadOnlyHint: toBoolPtr(true), From fa806749badfc19488bde10ec463b7a395e6bc21 Mon Sep 17 00:00:00 2001 From: Sam Morrow Date: Mon, 19 May 2025 13:39:26 +0200 Subject: [PATCH 2/3] Update context_tools.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- pkg/github/context_tools.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/github/context_tools.go b/pkg/github/context_tools.go index 7017effb0..b71d87efd 100644 --- a/pkg/github/context_tools.go +++ b/pkg/github/context_tools.go @@ -15,7 +15,7 @@ import ( // GetMe creates a tool to get details of the authenticated user. func GetMe(getClient GetClientFn, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) { return mcp.NewTool("get_me", - mcp.WithDescription(t("TOOL_GET_ME_DESCRIPTION", "Get details of the authenticated GitHub user. Use this when a request include \"me\", \"my\"... the output will not change unless the user changes their profile, so only call this once.")), + mcp.WithDescription(t("TOOL_GET_ME_DESCRIPTION", "Get details of the authenticated GitHub user. Use this when a request includes \"me\", \"my\"... the output will not change unless the user changes their profile, so only call this once.")), mcp.WithToolAnnotation(mcp.ToolAnnotation{ Title: t("TOOL_GET_ME_USER_TITLE", "Get my user profile"), ReadOnlyHint: toBoolPtr(true), From eefc39a8f7253af30b975dc310bde80ce0fe2410 Mon Sep 17 00:00:00 2001 From: Sam Morrow Date: Mon, 19 May 2025 14:19:58 +0200 Subject: [PATCH 3/3] enact second piece of copilot feedback --- pkg/github/context_tools.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/github/context_tools.go b/pkg/github/context_tools.go index b71d87efd..180f32dd4 100644 --- a/pkg/github/context_tools.go +++ b/pkg/github/context_tools.go @@ -15,7 +15,7 @@ import ( // GetMe creates a tool to get details of the authenticated user. func GetMe(getClient GetClientFn, t translations.TranslationHelperFunc) (tool mcp.Tool, handler server.ToolHandlerFunc) { return mcp.NewTool("get_me", - mcp.WithDescription(t("TOOL_GET_ME_DESCRIPTION", "Get details of the authenticated GitHub user. Use this when a request includes \"me\", \"my\"... the output will not change unless the user changes their profile, so only call this once.")), + mcp.WithDescription(t("TOOL_GET_ME_DESCRIPTION", "Get details of the authenticated GitHub user. Use this when a request includes \"me\", \"my\". The output will not change unless the user changes their profile, so only call this once.")), mcp.WithToolAnnotation(mcp.ToolAnnotation{ Title: t("TOOL_GET_ME_USER_TITLE", "Get my user profile"), ReadOnlyHint: toBoolPtr(true),