Clarify that pull_request_review_write create without event creates a pending review
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 72/100
- Type d'issue
- Documentation
- Clarté
- Clairement spécifiée
- Activité
- Calme
- Stack technique
- go
- Domaine
- documentation
Piste de recherche
Recherchez pull_request_review_write dans la définition de l'outil et examinez son annotation de titre ainsi que sa courte description. Mettez à jour la formulation destinée à l'utilisateur afin que method: "create" sans event signifie clairement une review en attente et non soumise ; le comportement existant de l'outil unique doit rester inchangé.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Go
- Étoiles
- 33.1k
- Forks
- 5k
- Merge moyen
- 2 j 15 h
- PR mergées (30 j)
- 27
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de github/github-mcp-server
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
github/github-mcp-server#3235 ·
-
enhancement
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
github/github-mcp-server#3042 · 2 commentaires ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
github/github-mcp-server#3032 · 1 réaction ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
github/github-mcp-server#2803 · 1 commentaire ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
github/github-mcp-server#2740 ·
Toutes les issues de github/github-mcp-server
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 84/100
-
enhancement needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
kind/cleanup
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
kubernetes-sigs/kueue#15947 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
sympozium-ai/sympozium#627 ·