Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Incomplete PR feedback API surface in github-code skill

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#359 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
72/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
github, graphql, markdown, shell
Lĩnh vực
api, developer-experience, documentation

Hướng nghiên cứu

Bắt đầu với api-surface.md của skill github-code và so sánh các endpoint phản hồi PR được tài liệu hóa trong đó với các lệnh gọi REST và GraphQL được liệt kê trong issue. Thêm phần Complete PR Feedback Retrieval, hướng dẫn về phân trang, các hạn chế của gh pr view, ghi chú về các thay đổi được đề xuất và phạm vi bao phủ của reactions cũng như tra cứu trực tiếp. Hoàn thành khi tất cả các nguồn phản hồi được liệt kê và các chi tiết về trạng thái thread được tài liệu hóa chính xác.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

The github-code skill's api-surface.md only documents 2 of ~8 endpoints needed for complete PR feedback retrieval. This causes the agent to miss inline review comments, review thread state, and conversation comments when inspecting PRs.

Gaps

  • No reviewThreads GraphQL query — REST cannot expose isResolved, isOutdated, or resolvedBy. This is GraphQL-only and completely missing from the surface.
  • No warning that gh pr view --json reviews omits inline review comments — --json comments returns conversation comments only, --json reviews returns review summaries only. Inline code review comments require /pulls/{number}/comments or GraphQL reviewThreads/reviews.comments.
  • Missing /issues/{number}/comments — conversation comments live at the issues endpoint, not /pulls/{number}/comments (which is inline review comments only). REST equivalent is undocumented.
  • Missing review-scoped comments endpoint — GET /pulls/{number}/reviews/{review_id}/comments for walking from a review to its inline comments.
  • Missing direct lookup endpoints — single review comment, single review, single issue comment by ID.
  • Missing pagination guidance — no --paginate documentation; PRs with heavy review activity silently truncate.
  • Missing suggested changes note — suggested changes are embedded in review comment body as markdown suggestion fences, not a separate endpoint.
  • Missing reactions endpoints — /pulls/comments/{comment_id}/reactions and /issues/comments/{comment_id}/reactions.

Recommended Fix

Add a "Complete PR Feedback Retrieval" section with:

REST minimum (3 paginated calls):

gh api --paginate repos/{owner}/{repo}/issues/{number}/comments    # conversation comments
gh api --paginate repos/{owner}/{repo}/pulls/{number}/reviews      # review summaries
gh api --paginate repos/{owner}/{repo}/pulls/{number}/comments     # inline review comments

GraphQL for thread state (1 call):

pullRequest(number: N) {
  comments(first: 100) { ... }
  reviews(first: 100) { ... }
  reviewThreads(first: 100) {
    nodes { isResolved isOutdated path line comments(first: 100) { ... } }
  }
}

Add explicit warning about gh pr view --json field limitations.

Full analysis: https://sentry.slack.com/docs/T024ZCV9U/F0B44A567QE

Action taken on behalf of David Cramer.

Ngôn ngữ chính
TypeScript
Star
367
Fork
41
Merge trung bình
6 giờ 56 phút
Pull request đã merge (30 ngày)
168

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của getsentry/junior

Tất cả issue của getsentry/junior

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.