Feature request: batch-fetch pull request review comment threads for an explicit PR list
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 48/100
Piste de recherche
Commencez par lire le comportement existant de pull_request_read et get_review_comments, puis examinez les conventions du dépôt pour les mises à jour de script/generate-docs et toolsnap. Définissez et documentez le nombre borné de PR ainsi que le comportement pour les PR manquantes ou en 404, ajoutez des tests et mettez à jour la documentation générée ou les artefacts toolsnap. C'est terminé lorsque le comportement par lot de la liste explicite est testé et documenté.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
Today, pulling inline review threads for multiple PRs means N separate pull_request_read calls (get_review_comments per PR). For agent workflows (team playbooks, review digests, distilling recurring conventions into docs or rules), that is slow, error-prone, and uses more API round-trips than necessary.
Proposed solution (v1)
Add a tool (name up to maintainers) that accepts:
owner,repopull_numbers: array of integers (bounded, e.g. configurable max such as 20–50)- Optional pagination parameters consistent with existing
get_review_commentsbehavior
Behavior: For each PR number, return the same structured data as today’s per-PR get_review_comments (threads + metadata such as resolved/outdated), keyed by pull_number.
Explicit non-goals (v1):
- No semantic ranking, clustering, or “top comments” inside the server.
- No implicit full-repo scan; the caller supplies an explicit list (e.g. from
search_pull_requests/list_pull_requests/ git).
Why this matters
- Documents a single supported pattern for agents: select PRs → one batch read → summarize offline.
- Reduces N+1 calls for engineering hygiene (guidelines derived from real review text).
Acceptance criteria (suggestion)
- Documented max PR count and behavior for missing/404 PRs (partial success vs fail-all—pick one and document).
- Tests +
script/generate-docs/ toolsnap updates per repo conventions.
Related issues
- Reply to review comments (still one thread at a time today): https://github.com/github/github-mcp-server/issues/1323
- Copilot suggestion changesets not in API (platform/API gap): https://github.com/github/github-mcp-server/issues/2235
- Companion (batch PR metadata for changelogs): https://github.com/github/github-mcp-server/issues/2302
Note
We are happy to prototype a PR if maintainers agree on v1 scope (explicit pull_numbers[], caps, and error semantics).
- Langage dominant
- Go
- Étoiles
- 33.1k
- Forks
- 5k
- Merge moyen
- 2 j 1 h
- PR mergées (30 j)
- 25
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
-
area/dev-productivity area/disaster-recovery area/ipcei kind/enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
kind/bug status/0-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
🤔 refinement needed
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
equinor/radix-operator#1979 ·