diff --git a/pkg/github/actions.go b/pkg/github/actions.go index ace9d7288..680475e08 100644 --- a/pkg/github/actions.go +++ b/pkg/github/actions.go @@ -32,11 +32,9 @@ func ListWorkflows(getClient GetClientFn, t translations.TranslationHelperFunc) }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), WithPagination(), ), @@ -92,11 +90,9 @@ func ListWorkflowRuns(getClient GetClientFn, t translations.TranslationHelperFun }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithString("workflow_id", mcp.Required(), @@ -231,11 +227,9 @@ func RunWorkflow(getClient GetClientFn, t translations.TranslationHelperFunc) (t }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithString("workflow_id", mcp.Required(), @@ -330,11 +324,9 @@ func GetWorkflowRun(getClient GetClientFn, t translations.TranslationHelperFunc) }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("run_id", mcp.Required(), @@ -386,11 +378,9 @@ func GetWorkflowRunLogs(getClient GetClientFn, t translations.TranslationHelperF }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("run_id", mcp.Required(), @@ -452,11 +442,9 @@ func ListWorkflowJobs(getClient GetClientFn, t translations.TranslationHelperFun }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("run_id", mcp.Required(), @@ -540,11 +528,9 @@ func GetJobLogs(getClient GetClientFn, t translations.TranslationHelperFunc, con }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("job_id", mcp.Description("The unique identifier of the workflow job (required for single job logs)"), @@ -788,11 +774,9 @@ func RerunWorkflowRun(getClient GetClientFn, t translations.TranslationHelperFun }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("run_id", mcp.Required(), @@ -851,11 +835,9 @@ func RerunFailedJobs(getClient GetClientFn, t translations.TranslationHelperFunc }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("run_id", mcp.Required(), @@ -914,11 +896,9 @@ func CancelWorkflowRun(getClient GetClientFn, t translations.TranslationHelperFu }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("run_id", mcp.Required(), @@ -979,11 +959,9 @@ func ListWorkflowRunArtifacts(getClient GetClientFn, t translations.TranslationH }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("run_id", mcp.Required(), @@ -1048,11 +1026,9 @@ func DownloadWorkflowRunArtifact(getClient GetClientFn, t translations.Translati }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("artifact_id", mcp.Required(), @@ -1114,11 +1090,9 @@ func DeleteWorkflowRunLogs(getClient GetClientFn, t translations.TranslationHelp }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("run_id", mcp.Required(), @@ -1177,11 +1151,9 @@ func GetWorkflowRunUsage(getClient GetClientFn, t translations.TranslationHelper }), mcp.WithString("owner", mcp.Required(), - mcp.Description(DescriptionRepositoryOwner), ), mcp.WithString("repo", mcp.Required(), - mcp.Description(DescriptionRepositoryName), ), mcp.WithNumber("run_id", mcp.Required(), diff --git a/pkg/github/code_scanning.go b/pkg/github/code_scanning.go index 47eaa4be0..f788f89ca 100644 --- a/pkg/github/code_scanning.go +++ b/pkg/github/code_scanning.go @@ -23,11 +23,9 @@ func GetCodeScanningAlert(getClient GetClientFn, t translations.TranslationHelpe }), mcp.WithString("owner", mcp.Required(), - mcp.Description("The owner of the repository."), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("The name of the repository."), ), mcp.WithNumber("alertNumber", mcp.Required(), @@ -89,11 +87,9 @@ func ListCodeScanningAlerts(getClient GetClientFn, t translations.TranslationHel }), mcp.WithString("owner", mcp.Required(), - mcp.Description("The owner of the repository."), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("The name of the repository."), ), mcp.WithString("state", mcp.Description("Filter code scanning alerts by state. Defaults to open"), diff --git a/pkg/github/dependabot.go b/pkg/github/dependabot.go index 7b327cd77..44e738f3e 100644 --- a/pkg/github/dependabot.go +++ b/pkg/github/dependabot.go @@ -24,11 +24,9 @@ func GetDependabotAlert(getClient GetClientFn, t translations.TranslationHelperF }), mcp.WithString("owner", mcp.Required(), - mcp.Description("The owner of the repository."), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("The name of the repository."), ), mcp.WithNumber("alertNumber", mcp.Required(), @@ -91,11 +89,9 @@ func ListDependabotAlerts(getClient GetClientFn, t translations.TranslationHelpe }), mcp.WithString("owner", mcp.Required(), - mcp.Description("The owner of the repository."), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("The name of the repository."), ), mcp.WithString("state", mcp.Description("Filter dependabot alerts by state. Defaults to open"), diff --git a/pkg/github/discussions.go b/pkg/github/discussions.go index dc26063fd..db9b6bc70 100644 --- a/pkg/github/discussions.go +++ b/pkg/github/discussions.go @@ -126,7 +126,6 @@ func ListDiscussions(getGQLClient GetGQLClientFn, t translations.TranslationHelp }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Description("Repository name. If not provided, discussions will be queried at the organisation level."), @@ -265,11 +264,9 @@ func GetDiscussion(getGQLClient GetGQLClientFn, t translations.TranslationHelper }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("discussionNumber", mcp.Required(), @@ -340,8 +337,8 @@ func GetDiscussionComments(getGQLClient GetGQLClientFn, t translations.Translati Title: t("TOOL_GET_DISCUSSION_COMMENTS_USER_TITLE", "Get discussion comments"), ReadOnlyHint: ToBoolPtr(true), }), - mcp.WithString("owner", mcp.Required(), mcp.Description("Repository owner")), - mcp.WithString("repo", mcp.Required(), mcp.Description("Repository name")), + mcp.WithString("owner", mcp.Required()), + mcp.WithString("repo", mcp.Required()), mcp.WithNumber("discussionNumber", mcp.Required(), mcp.Description("Discussion Number")), WithCursorPagination(), ), @@ -450,7 +447,6 @@ func ListDiscussionCategories(getGQLClient GetGQLClientFn, t translations.Transl }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Description("Repository name. If not provided, discussion categories will be queried at the organisation level."), diff --git a/pkg/github/issues.go b/pkg/github/issues.go index 01ce7b42e..29154697b 100644 --- a/pkg/github/issues.go +++ b/pkg/github/issues.go @@ -154,11 +154,9 @@ func GetIssue(getClient GetClientFn, t translations.TranslationHelperFunc) (tool }), mcp.WithString("owner", mcp.Required(), - mcp.Description("The owner of the repository"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("The name of the repository"), ), mcp.WithNumber("issue_number", mcp.Required(), @@ -217,7 +215,6 @@ func ListIssueTypes(getClient GetClientFn, t translations.TranslationHelperFunc) }), mcp.WithString("owner", mcp.Required(), - mcp.Description("The organization owner of the repository"), ), ), func(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) { @@ -263,11 +260,9 @@ func AddIssueComment(getClient GetClientFn, t translations.TranslationHelperFunc }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("issue_number", mcp.Required(), @@ -337,11 +332,9 @@ func AddSubIssue(getClient GetClientFn, t translations.TranslationHelperFunc) (t }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("issue_number", mcp.Required(), @@ -425,18 +418,15 @@ func ListSubIssues(getClient GetClientFn, t translations.TranslationHelperFunc) }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("issue_number", mcp.Required(), mcp.Description("Issue number"), ), mcp.WithNumber("page", - mcp.Description("Page number for pagination (default: 1)"), ), mcp.WithNumber("per_page", mcp.Description("Number of results per page (max 100, default: 30)"), @@ -519,11 +509,9 @@ func RemoveSubIssue(getClient GetClientFn, t translations.TranslationHelperFunc) }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("issue_number", mcp.Required(), @@ -598,11 +586,9 @@ func ReprioritizeSubIssue(getClient GetClientFn, t translations.TranslationHelpe }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("issue_number", mcp.Required(), @@ -756,11 +742,9 @@ func CreateIssue(getClient GetClientFn, t translations.TranslationHelperFunc) (t }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("title", mcp.Required(), @@ -896,11 +880,9 @@ func ListIssues(getGQLClient GetGQLClientFn, t translations.TranslationHelperFun }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("state", mcp.Description("Filter by state, by default both open and closed issues are returned when not provided"), @@ -1108,11 +1090,9 @@ func UpdateIssue(getClient GetClientFn, t translations.TranslationHelperFunc) (t }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("issue_number", mcp.Required(), @@ -1271,11 +1251,9 @@ func GetIssueComments(getClient GetClientFn, t translations.TranslationHelperFun }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("issue_number", mcp.Required(), @@ -1385,11 +1363,9 @@ func AssignCopilotToIssue(getGQLClient GetGQLClientFn, t translations.Translatio }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("issueNumber", mcp.Required(), diff --git a/pkg/github/notifications.go b/pkg/github/notifications.go index 0ee5a6b14..2adb2180a 100644 --- a/pkg/github/notifications.go +++ b/pkg/github/notifications.go @@ -448,7 +448,6 @@ func ManageRepositoryNotificationSubscription(getClient GetClientFn, t translati ), mcp.WithString("repo", mcp.Required(), - mcp.Description("The name of the repository."), ), mcp.WithString("action", mcp.Required(), diff --git a/pkg/github/pullrequests.go b/pkg/github/pullrequests.go index d7547519d..0fd132028 100644 --- a/pkg/github/pullrequests.go +++ b/pkg/github/pullrequests.go @@ -27,11 +27,9 @@ func GetPullRequest(getClient GetClientFn, t translations.TranslationHelperFunc) }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -93,11 +91,9 @@ func CreatePullRequest(getClient GetClientFn, t translations.TranslationHelperFu }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("title", mcp.Required(), @@ -217,11 +213,9 @@ func UpdatePullRequest(getClient GetClientFn, getGQLClient GetGQLClientFn, t tra }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -493,11 +487,9 @@ func ListPullRequests(getClient GetClientFn, t translations.TranslationHelperFun }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("state", mcp.Description("Filter by state"), @@ -605,11 +597,9 @@ func MergePullRequest(getClient GetClientFn, t translations.TranslationHelperFun }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -743,11 +733,9 @@ func GetPullRequestFiles(getClient GetClientFn, t translations.TranslationHelper }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -818,11 +806,9 @@ func GetPullRequestStatus(getClient GetClientFn, t translations.TranslationHelpe }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -903,11 +889,9 @@ func UpdatePullRequestBranch(getClient GetClientFn, t translations.TranslationHe }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -985,11 +969,9 @@ func GetPullRequestComments(getClient GetClientFn, t translations.TranslationHel }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -1057,11 +1039,9 @@ func GetPullRequestReviews(getClient GetClientFn, t translations.TranslationHelp }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -1125,11 +1105,9 @@ func CreateAndSubmitPullRequestReview(getGQLClient GetGQLClientFn, t translation // internally for now. mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -1228,11 +1206,9 @@ func CreatePendingPullRequestReview(getGQLClient GetGQLClientFn, t translations. // internally for now. mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -1326,11 +1302,9 @@ func AddCommentToPendingReview(getGQLClient GetGQLClientFn, t translations.Trans // ), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -1487,11 +1461,9 @@ func SubmitPendingPullRequestReview(getGQLClient GetGQLClientFn, t translations. // the latest review from a user, since only one can be active at a time. mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -1621,11 +1593,9 @@ func DeletePendingPullRequestReview(getGQLClient GetGQLClientFn, t translations. // the latest pending review from a user, since only one can be active at a time. mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -1736,11 +1706,9 @@ func GetPullRequestDiff(getClient GetClientFn, t translations.TranslationHelperF }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), @@ -1804,11 +1772,9 @@ func RequestCopilotReview(getClient GetClientFn, t translations.TranslationHelpe }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithNumber("pullNumber", mcp.Required(), diff --git a/pkg/github/repositories.go b/pkg/github/repositories.go index cef227ba5..de9c20a04 100644 --- a/pkg/github/repositories.go +++ b/pkg/github/repositories.go @@ -27,11 +27,9 @@ func GetCommit(getClient GetClientFn, t translations.TranslationHelperFunc) (too }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("sha", mcp.Required(), @@ -114,11 +112,9 @@ func ListCommits(getClient GetClientFn, t translations.TranslationHelperFunc) (t }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("sha", mcp.Description("Commit SHA, branch or tag name to list commits of. If not provided, uses the default branch of the repository. If a commit SHA is provided, will list commits up to that SHA."), @@ -210,11 +206,9 @@ func ListBranches(getClient GetClientFn, t translations.TranslationHelperFunc) ( }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), WithPagination(), ), @@ -287,11 +281,9 @@ func CreateOrUpdateFile(getClient GetClientFn, t translations.TranslationHelperF }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner (username or organization)"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("path", mcp.Required(), @@ -490,11 +482,9 @@ func GetFileContents(getClient GetClientFn, getRawClient raw.GetRawClientFn, t t }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner (username or organization)"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("path", mcp.Description("Path to file/directory (directories must end with a slash '/')"), @@ -685,12 +675,9 @@ func ForkRepository(getClient GetClientFn, t translations.TranslationHelperFunc) ReadOnlyHint: ToBoolPtr(false), }), mcp.WithString("owner", - mcp.Required(), - mcp.Description("Repository owner"), - ), + mcp.Required(), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("organization", mcp.Description("Organization to fork to"), @@ -772,11 +759,9 @@ func DeleteFile(getClient GetClientFn, t translations.TranslationHelperFunc) (to }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner (username or organization)"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("path", mcp.Required(), @@ -942,11 +927,9 @@ func CreateBranch(getClient GetClientFn, t translations.TranslationHelperFunc) ( }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("branch", mcp.Required(), @@ -1043,11 +1026,9 @@ func PushFiles(getClient GetClientFn, t translations.TranslationHelperFunc) (too }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("branch", mcp.Required(), @@ -1215,11 +1196,9 @@ func ListTags(getClient GetClientFn, t translations.TranslationHelperFunc) (tool }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), WithPagination(), ), @@ -1284,11 +1263,9 @@ func GetTag(getClient GetClientFn, t translations.TranslationHelperFunc) (tool m }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("tag", mcp.Required(), @@ -1371,11 +1348,9 @@ func ListReleases(getClient GetClientFn, t translations.TranslationHelperFunc) ( }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), WithPagination(), ), @@ -1436,11 +1411,9 @@ func GetLatestRelease(getClient GetClientFn, t translations.TranslationHelperFun }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), ), func(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) { @@ -1490,11 +1463,9 @@ func GetReleaseByTag(getClient GetClientFn, t translations.TranslationHelperFunc }), mcp.WithString("owner", mcp.Required(), - mcp.Description("Repository owner"), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("Repository name"), ), mcp.WithString("tag", mcp.Required(), diff --git a/pkg/github/secret_scanning.go b/pkg/github/secret_scanning.go index c140c34ad..e45929363 100644 --- a/pkg/github/secret_scanning.go +++ b/pkg/github/secret_scanning.go @@ -24,11 +24,9 @@ func GetSecretScanningAlert(getClient GetClientFn, t translations.TranslationHel }), mcp.WithString("owner", mcp.Required(), - mcp.Description("The owner of the repository."), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("The name of the repository."), ), mcp.WithNumber("alertNumber", mcp.Required(), @@ -91,11 +89,9 @@ func ListSecretScanningAlerts(getClient GetClientFn, t translations.TranslationH }), mcp.WithString("owner", mcp.Required(), - mcp.Description("The owner of the repository."), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("The name of the repository."), ), mcp.WithString("state", mcp.Description("Filter by state"), diff --git a/pkg/github/security_advisories.go b/pkg/github/security_advisories.go index 6eaeebe47..92caffda8 100644 --- a/pkg/github/security_advisories.go +++ b/pkg/github/security_advisories.go @@ -191,11 +191,9 @@ func ListRepositorySecurityAdvisories(getClient GetClientFn, t translations.Tran }), mcp.WithString("owner", mcp.Required(), - mcp.Description("The owner of the repository."), ), mcp.WithString("repo", mcp.Required(), - mcp.Description("The name of the repository."), ), mcp.WithString("direction", mcp.Description("Sort direction."), diff --git a/pkg/github/server.go b/pkg/github/server.go index 16d28643c..a48196c75 100644 --- a/pkg/github/server.go +++ b/pkg/github/server.go @@ -13,8 +13,19 @@ import ( // NewServer creates a new GitHub MCP server with the specified GH client and logger. func NewServer(version string, opts ...server.ServerOption) *server.MCPServer { + const instructions = `GitHub MCP Server - Provides tools for GitHub operations for things like pull requests, issues, repositories, and more. + When using this MCP Server, keep in mind these common arguments used across many tools: + • "owner" - Repository owner (username or organization) + • "repo" - Repository name + • "page" - Page number for pagination (min: 1) + • "perPage" - Results per page (min: 1, max: 100) + + Also keep in mind the following: + - If you need information about the user to use a tool and you are not sure what to provide, you can use the "get_current_user" tool to get details about the authenticated user. + ` // Add default options defaultOpts := []server.ServerOption{ + server.WithInstructions(instructions), server.WithToolCapabilities(true), server.WithResourceCapabilities(true, true), server.WithLogging(), @@ -194,12 +205,10 @@ func OptionalStringArrayParam(r mcp.CallToolRequest, p string) ([]string, error) func WithPagination() mcp.ToolOption { return func(tool *mcp.Tool) { mcp.WithNumber("page", - mcp.Description("Page number for pagination (min 1)"), mcp.Min(1), )(tool) mcp.WithNumber("perPage", - mcp.Description("Results per page for pagination (min 1, max 100)"), mcp.Min(1), mcp.Max(100), )(tool) @@ -211,12 +220,10 @@ func WithPagination() mcp.ToolOption { func WithUnifiedPagination() mcp.ToolOption { return func(tool *mcp.Tool) { mcp.WithNumber("page", - mcp.Description("Page number for pagination (min 1)"), mcp.Min(1), )(tool) mcp.WithNumber("perPage", - mcp.Description("Results per page for pagination (min 1, max 100)"), mcp.Min(1), mcp.Max(100), )(tool) @@ -231,7 +238,6 @@ func WithUnifiedPagination() mcp.ToolOption { func WithCursorPagination() mcp.ToolOption { return func(tool *mcp.Tool) { mcp.WithNumber("perPage", - mcp.Description("Results per page for pagination (min 1, max 100)"), mcp.Min(1), mcp.Max(100), )(tool)