From 1bc0eceffcee3affebaaada36ebd4e80632456c5 Mon Sep 17 00:00:00 2001 From: JuggerGrimrod88 Date: Tue, 7 Jul 2026 21:53:19 -0400 Subject: [PATCH] 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 --- pkg/github/__toolsnaps__/pull_request_review_write.snap | 2 +- pkg/github/pullrequests.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/github/__toolsnaps__/pull_request_review_write.snap b/pkg/github/__toolsnaps__/pull_request_review_write.snap index 74ef808559..d00358640b 100644 --- a/pkg/github/__toolsnaps__/pull_request_review_write.snap +++ b/pkg/github/__toolsnaps__/pull_request_review_write.snap @@ -2,7 +2,7 @@ "annotations": { "idempotentHint": false, "readOnlyHint": false, - "title": "Write operations (create, submit, delete) on pull request reviews" + "title": "Create (pending), submit, or delete pull request reviews" }, "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", "inputSchema": { diff --git a/pkg/github/pullrequests.go b/pkg/github/pullrequests.go index 9fc8455988..b7beebdc00 100644 --- a/pkg/github/pullrequests.go +++ b/pkg/github/pullrequests.go @@ -1784,7 +1784,7 @@ Available methods: - 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. `), Annotations: &mcp.ToolAnnotations{ - Title: t("TOOL_PULL_REQUEST_REVIEW_WRITE_USER_TITLE", "Write operations (create, submit, delete) on pull request reviews"), + Title: t("TOOL_PULL_REQUEST_REVIEW_WRITE_USER_TITLE", "Create (pending), submit, or delete pull request reviews"), ReadOnlyHint: false, }, InputSchema: schema,