Clarify that pull_request_review_write create without event creates a pending review

Open Beginner friendly
#2,524 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
72/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Quiet
Tech stack
go
Domain
documentation

Research direction

Search the tool definition for pull_request_review_write and inspect its title annotation and short description. Update the user-facing wording so method: "create" without event clearly means a pending, unsubmitted review; the existing single-tool behavior should remain unchanged.

Written by the indexing model from the issue text.

Description

request ai review

Problem

When using the pull_request_review_write tool from a client like Claude Code to "create a review," the server correctly creates a pending review if the event parameter is omitted. However, the user-facing title/description does not clearly indicate that this call only creates a pending review and does not submit it.

Current behavior
  • Tool name: pull_request_review_write
  • Title: "Write operations (create, submit, delete) on pull request reviews"
  • Description (in code) already says:

    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.

Clients often show only the short title or a condensed description, so users just see that a review is being "created," not that it's pending only.

This is confusing when watching MCP tool calls in the UI: the first call to pull_request_review_write looks like it "created a review," but in GitHub it is actually just opening a pending review, which then gets line comments via add_comment_to_pending_review, and later is submitted with pull_request_review_write method: "submit_pending".

The underlying workflow is correct; the wording is misleading.

Requested change (minimal, backwards compatible)

Update the Title annotation for pull_request_review_write to make "pending" explicit, e.g.:

  • "Create pending / submit / delete pull request reviews", or
  • "Pending review create / submit / delete for pull requests"

Optionally, emphasize in the short description that method: "create" without event creates a pending review and does not submit it.

This keeps the existing single-tool design (no new tools, no API change) but makes the behavior much clearer to users of MCP clients that only surface the short title/description.

Dominant language
Go
Stars
33.1k
Forks
5k
Avg merge
2d 15h
Merged PRs (30d)
27

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from github/github-mcp-server

All issues in github/github-mcp-server

Similar issues

More Go issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.