Skip to content

Commit 82ac788

Browse files
committed
Remove excessive mentions of event parameter
1 parent c08d50d commit 82ac788

File tree

3 files changed

+11
-5
lines changed

3 files changed

+11
-5
lines changed

pkg/github/__toolsnaps__/pull_request_review_write.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
"type": "string"
1616
},
1717
"event": {
18-
"description": "Review action to perform. If not specified, pending review is created.",
18+
"description": "Review action to perform.",
1919
"enum": [
2020
"APPROVE",
2121
"REQUEST_CHANGES",

pkg/github/instructions.go

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,11 +48,17 @@ Context management:
4848
func getToolsetInstructions(toolset string) string {
4949
switch toolset {
5050
case "pull_requests":
51-
return "## Pull Requests\n\nPR review workflow: Always use 'pull_request_review_write' with method 'create' and no 'event' parameter to create a pending review, then 'add_comment_to_pending_review' to add comments, and finally 'pull_request_review_write' with method 'submit_pending' to submit the review for complex reviews with line-specific comments."
51+
return `## Pull Requests
52+
53+
PR review workflow: Always use 'pull_request_review_write' with method 'create' to create a pending review, then 'add_comment_to_pending_review' to add comments, and finally 'pull_request_review_write' with method 'submit_pending' to submit the review for complex reviews with line-specific comments.`
5254
case "issues":
53-
return "## Issues\n\nCheck 'list_issue_types' first for organizations to use proper issue types. Use 'search_issues' before creating new issues to avoid duplicates. Always set 'state_reason' when closing issues."
55+
return `## Issues
56+
57+
Check 'list_issue_types' first for organizations to use proper issue types. Use 'search_issues' before creating new issues to avoid duplicates. Always set 'state_reason' when closing issues.`
5458
case "discussions":
55-
return "## Discussions\n\nUse 'list_discussion_categories' to understand available categories before creating discussions. Filter by category for better organization."
59+
return `## Discussions
60+
61+
Use 'list_discussion_categories' to understand available categories before creating discussions. Filter by category for better organization.`
5662
default:
5763
return ""
5864
}

pkg/github/pullrequests.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1077,7 +1077,7 @@ Available methods:
10771077
mcp.Description("Review comment text"),
10781078
),
10791079
mcp.WithString("event",
1080-
mcp.Description("Review action to perform. If not specified, pending review is created."),
1080+
mcp.Description("Review action to perform."),
10811081
mcp.Enum("APPROVE", "REQUEST_CHANGES", "COMMENT"),
10821082
),
10831083
mcp.WithString("commitID",

0 commit comments

Comments
 (0)