Skip to content

Exploration: using Server Instructions #1052

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.

Sign up for GitHub

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

Closed
wants to merge 6 commits into from
Closed

Exploration: using Server Instructions #1052

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
0321374
add temporary instructions
tommaso-moro Sep 5, 2025
aaa2d45
remove description for repo and owner
tommaso-moro Sep 5, 2025
5fae712
Merge branch 'main' into tommy/server-instructions
tommaso-moro Sep 5, 2025
8a5aa6f
remove "page" and "perPage" descriptions
tommaso-moro Sep 5, 2025
8627885
improve instructions
tommaso-moro Sep 5, 2025
36688fd
improve description
tommaso-moro Sep 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 0 additions & 28 deletions pkg/github/actions.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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(),
),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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)"),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
Expand Down
4 changes: 0 additions & 4 deletions pkg/github/code_scanning.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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(),
Expand Down Expand Up @@ -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"),
Expand Down
4 changes: 0 additions & 4 deletions pkg/github/dependabot.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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(),
Expand Down Expand Up @@ -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"),
Expand Down
8 changes: 2 additions & 6 deletions pkg/github/discussions.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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."),
Expand Down Expand Up @@ -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(),
Expand Down Expand Up @@ -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(),
),
Expand Down Expand Up @@ -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."),
Expand Down
Loading
Loading