`add_issue_comment`, `update_issue`, and `get_issue` reject valid integer `issue_number`

Đang mở
#2,807 2 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ó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
68/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
go
Lĩnh vực
api, backend

Hướng nghiên cứu

Bắt đầu với các phần triển khai và các luồng xác thực request của add_issue_comment, update_issue và get_issue, tập trung vào việc nội suy issue_number vào các đường dẫn URL. Tái hiện các lệnh gọi với một số nguyên và so sánh chúng với một tham số số nguyên đang hoạt động như milestone. Được xem là hoàn tất khi các giá trị issue_number dạng số nguyên vượt qua bước xác thực và đến được các endpoint GitHub REST tương ứng, với phạm vi kiểm thử hồi quy cho cả ba công cụ.

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

Mô tả

bug request ai review
Summary

Three endpoints that take issue_number as a required parameter consistently fail when invoked with an integer, even though the tool schema declares issue_number as number. Other endpoints on the same server that take integer parameters work correctly, and the failure survives reconnecting the MCP server.

Affected tools
  • add_issue_comment
  • update_issue
  • get_issue
Unaffected tools (for comparison, same session, same repo, same auth)
  • create_issue — succeeds (does not take issue_number)
  • list_issues, search_issues — succeed
  • create_branch, create_or_update_file, push_files, create_pull_request — all succeed, including calls with integer fields like milestone
Reproduction
  1. Any repo the caller can write issues on.
  2. Call, e.g.:
    {
      "tool": "add_issue_comment",
      "arguments": {
        "owner": "<owner>",
        "repo": "<repo>",
        "issue_number": 11,
        "body": "test"
      }
    }
    
  3. Call fails. Retrying multiple times, including after fully reconnecting the MCP server, reproduces the failure.
  4. create_issue against the same repo in the same session succeeds, ruling out auth, permissions, and rate limiting.
  5. Reproduced across at least two independent sessions on the same day.
Expected

Comment is posted / issue is updated / issue is fetched, matching a direct REST call to POST /repos/{owner}/{repo}/issues/{issue_number}/comments (etc.).

Actual

Call is rejected before it reaches GitHub. The failure pattern is consistent with issue_number being serialised as a string somewhere in the request pipeline and then rejected by the server's own integer validation.

[TODO fill in with error string on next occurrence]

Hypothesis

The three broken endpoints are exactly the ones where issue_number is templated into a URL path segment (/repos/{owner}/{repo}/issues/{issue_number}[/comments]), whereas the working endpoints pass their integers in the request body or query string. Likely culprits:

  • A path-parameter substitution step stringifies the value before an internal validator runs.
  • A schema-validation layer runs after path substitution and sees a string where a number is required.

Anywhere issue_number is interpolated into a URL is worth checking first.

Affected version
  • MCP server: GitHub MCP http server
  • Transport: stdio.
  • Client: custom provider

Please run docker run -i --rm ghcr.io/github/github-mcp-server ./github-mcp-server --version and paste the output below

Workaround

None from the client side; the caller cannot influence how the server serialises the parameter. Affected users have to fall back to the GitHub REST API directly or edit the issue in the web UI.

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.