Skip to content

Commit 1bc0ece

Browse files
Clarify pull_request_review_write title notes pending review creation
Update the pull_request_review_write tool Title annotation from "Write operations (create, submit, delete) on pull request reviews" to "Create (pending), submit, or delete pull request reviews". The parenthetical scopes "pending" to the create action: method: "create" opens a pending review when the event parameter is omitted, and submits immediately when event is supplied. MCP clients that surface only the short title previously implied that create submits a review; the new title surfaces the pending-by-default behavior at a glance. The description body already documents the event-conditional behavior; only the short title was misleading. Closes #2524. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 40db5e3 commit 1bc0ece

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

pkg/github/__toolsnaps__/pull_request_review_write.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"annotations": {
33
"idempotentHint": false,
44
"readOnlyHint": false,
5-
"title": "Write operations (create, submit, delete) on pull request reviews"
5+
"title": "Create (pending), submit, or delete pull request reviews"
66
},
77
"description": "Create and/or submit, delete review of a pull request.\n\nAvailable methods:\n- create: Create a new review of a pull request. If \"event\" parameter is provided, the review is submitted. If \"event\" is omitted, a pending review is created.\n- submit_pending: Submit an existing pending review of a pull request. This requires that a pending review exists for the current user on the specified pull request. The \"body\" and \"event\" parameters are used when submitting the review.\n- delete_pending: Delete an existing pending review of a pull request. This requires that a pending review exists for the current user on the specified pull request.\n- resolve_thread: Resolve a review thread. Requires only \"threadId\" parameter with the thread's node ID (e.g., PRRT_kwDOxxx). The owner, repo, and pullNumber parameters are not used for this method. Resolving an already-resolved thread is a no-op.\n- unresolve_thread: Unresolve a previously resolved review thread. Requires only \"threadId\" parameter. The owner, repo, and pullNumber parameters are not used for this method. Unresolving an already-unresolved thread is a no-op.\n",
88
"inputSchema": {

pkg/github/pullrequests.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1784,7 +1784,7 @@ Available methods:
17841784
- unresolve_thread: Unresolve a previously resolved review thread. Requires only "threadId" parameter. The owner, repo, and pullNumber parameters are not used for this method. Unresolving an already-unresolved thread is a no-op.
17851785
`),
17861786
Annotations: &mcp.ToolAnnotations{
1787-
Title: t("TOOL_PULL_REQUEST_REVIEW_WRITE_USER_TITLE", "Write operations (create, submit, delete) on pull request reviews"),
1787+
Title: t("TOOL_PULL_REQUEST_REVIEW_WRITE_USER_TITLE", "Create (pending), submit, or delete pull request reviews"),
17881788
ReadOnlyHint: false,
17891789
},
17901790
InputSchema: schema,

0 commit comments

Comments
 (0)