Skip to content

Commit c74723c

Browse files
committed
fix: clarify pending review creation
1 parent eb47a99 commit c74723c

4 files changed

Lines changed: 8 additions & 8 deletions

File tree

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1331,7 +1331,7 @@ For negotiated supported protocol versions `2026-07-28` or newer, the consolidat
13311331
- `pullNumber`: Pull request number (number, required)
13321332
- `repo`: Repository name (string, required)
13331333

1334-
- **pull_request_review_write** - Write operations (create, submit, delete) on pull request reviews
1334+
- **pull_request_review_write** - Create pending, submit, delete pull request reviews
13351335
- **OAuth Challenge Scopes**: `repo`
13361336
- `body`: Review comment text (string, optional)
13371337
- `commitID`: SHA of commit to review (string, optional)

‎pkg/github/__toolsnaps__/pull_request_review_write.snap‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
"annotations": {
33
"idempotentHint": false,
44
"readOnlyHint": false,
5-
"title": "Write operations (create, submit, delete) on pull request reviews"
5+
"title": "Create pending, submit, delete pull request reviews"
66
},
7-
"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",
7+
"description": "Create pending reviews, submit pending reviews, delete pending reviews, and resolve or unresolve review threads.\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 and is not submitted.\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": {
99
"properties": {
1010
"body": {

‎pkg/github/__toolsnaps__/pull_request_review_write_resolution_reason.snap‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
"annotations": {
33
"idempotentHint": false,
44
"readOnlyHint": false,
5-
"title": "Write operations (create, submit, delete) on pull request reviews"
5+
"title": "Create pending, submit, delete pull request reviews"
66
},
7-
"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",
7+
"description": "Create pending reviews, submit pending reviews, delete pending reviews, and resolve or unresolve review threads.\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 and is not submitted.\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": {
99
"properties": {
1010
"body": {

‎pkg/github/pullrequests.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2057,17 +2057,17 @@ func pullRequestReviewWrite(t translations.TranslationHelperFunc, withResolution
20572057
mcp.Tool{
20582058
Name: "pull_request_review_write",
20592059
OutputSchema: pullRequestOutputSchema[RepositoryMessageOutput](),
2060-
Description: t("TOOL_PULL_REQUEST_REVIEW_WRITE_DESCRIPTION", `Create and/or submit, delete review of a pull request.
2060+
Description: t("TOOL_PULL_REQUEST_REVIEW_WRITE_DESCRIPTION", `Create pending reviews, submit pending reviews, delete pending reviews, and resolve or unresolve review threads.
20612061
20622062
Available methods:
2063-
- 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.
2063+
- 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 and is not submitted.
20642064
- 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.
20652065
- 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.
20662066
- 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.
20672067
- 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.
20682068
`),
20692069
Annotations: &mcp.ToolAnnotations{
2070-
Title: t("TOOL_PULL_REQUEST_REVIEW_WRITE_USER_TITLE", "Write operations (create, submit, delete) on pull request reviews"),
2070+
Title: t("TOOL_PULL_REQUEST_REVIEW_WRITE_USER_TITLE", "Create pending, submit, delete pull request reviews"),
20712071
ReadOnlyHint: false,
20722072
},
20732073
InputSchema: schema,

0 commit comments

Comments
 (0)