Skip to content

Commit b2bf8bb

Browse files
committed
fix: clarify pending review creation
1 parent a378370 commit b2bf8bb

3 files changed

Lines changed: 6 additions & 6 deletions

File tree

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1157,7 +1157,7 @@ The following sets of tools are available:
11571157
- `pullNumber`: Pull request number (number, required)
11581158
- `repo`: Repository name (string, required)
11591159

1160-
- **pull_request_review_write** - Write operations (create, submit, delete) on pull request reviews
1160+
- **pull_request_review_write** - Create pending, submit, delete pull request reviews
11611161
- **Required OAuth Scopes**: `repo`
11621162
- `body`: Review comment text (string, optional)
11631163
- `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
@@ -1,8 +1,8 @@
11
{
22
"annotations": {
3-
"title": "Write operations (create, submit, delete) on pull request reviews"
3+
"title": "Create pending, submit, delete pull request reviews"
44
},
5-
"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",
5+
"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",
66
"inputSchema": {
77
"properties": {
88
"body": {

pkg/github/pullrequests.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1767,17 +1767,17 @@ func PullRequestReviewWrite(t translations.TranslationHelperFunc) inventory.Serv
17671767
ToolsetMetadataPullRequests,
17681768
mcp.Tool{
17691769
Name: "pull_request_review_write",
1770-
Description: t("TOOL_PULL_REQUEST_REVIEW_WRITE_DESCRIPTION", `Create and/or submit, delete review of a pull request.
1770+
Description: t("TOOL_PULL_REQUEST_REVIEW_WRITE_DESCRIPTION", `Create pending reviews, submit pending reviews, delete pending reviews, and resolve or unresolve review threads.
17711771
17721772
Available methods:
1773-
- 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.
1773+
- 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.
17741774
- 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.
17751775
- 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.
17761776
- 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.
17771777
- 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.
17781778
`),
17791779
Annotations: &mcp.ToolAnnotations{
1780-
Title: t("TOOL_PULL_REQUEST_REVIEW_WRITE_USER_TITLE", "Write operations (create, submit, delete) on pull request reviews"),
1780+
Title: t("TOOL_PULL_REQUEST_REVIEW_WRITE_USER_TITLE", "Create pending, submit, delete pull request reviews"),
17811781
ReadOnlyHint: false,
17821782
},
17831783
InputSchema: schema,

0 commit comments

Comments
 (0)