Clarify that pull_request_review_write create without event creates a pending review

Đang mở Phù hợp với người mới
#2,524 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một 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ệ
go
Lĩnh vực
documentation

Hướng nghiên cứu

Tìm pull_request_review_write trong định nghĩa công cụ và kiểm tra chú thích tiêu đề cùng mô tả ngắn của nó. Cập nhật cách diễn đạt hướng tới người dùng để method: "create" không có event thể hiện rõ một review đang chờ xử lý và chưa được gửi; hành vi hiện có của công cụ đơn lẻ phải được giữ nguyên.

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

Mô tả

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.

Ngôn ngữ chính
Go
Star
33.1k
Fork
5k
Merge trung bình
2 ngày 15 giờ
Pull request đã merge (30 ngày)
27

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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 github/github-mcp-server

Tất cả issue của github/github-mcp-server

Issue tương tự

Thêm issue về Go

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.