Feature request: batch-fetch pull request review comment threads for an explicit PR list
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 48/100
Hướng nghiên cứu
Bắt đầu bằng việc đọc hành vi hiện có của pull_request_read và get_review_comments, sau đó xem xét các quy ước của repository đối với việc cập nhật script/generate-docs và toolsnap. Xác định và lập tài liệu về số lượng PR bị giới hạn cũng như hành vi đối với PR bị thiếu hoặc PR 404, thêm các bài kiểm thử và cập nhật tài liệu được tạo hoặc các tạo tác toolsnap. Hoàn tất khi hành vi xử lý theo lô của danh sách tường minh đã được kiểm thử và lập tài liệu.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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).
- Ngôn ngữ chính
- Go
- Star
- 33.1k
- Fork
- 5k
- Merge trung bình
- 2 ngày 1 giờ
- Pull request đã merge (30 ngày)
- 25
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của github/github-mcp-server
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
github/github-mcp-server#3235 ·
-
enhancement
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
github/github-mcp-server#3042 · 2 bình luận ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
github/github-mcp-server#3032 · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
github/github-mcp-server#2803 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
github/github-mcp-server#2740 ·
Tất cả issue của github/github-mcp-server
Issue tương tự
-
textual definition
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
geneontology/go-ontology#32653 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 75/100
-
needs design
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Priority/High ready-for-agent Severity/Major Type/Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100