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

get_job_logs: run_id with failed_only=false should return all job logs; failed_only should be a consistent modifier

Đang mở
#2,389 0 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
64/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
github, go
Lĩnh vực
api, backend

Hướng nghiên cứu

Bắt đầu từ phần triển khai get_job_logs và theo dõi cách job_id, run_id và failed_only được xác thực, bao gồm cả việc tra cứu siêu dữ liệu job hiện có. Bổ sung phạm vi kiểm thử tập trung cho các tổ hợp tham số được nêu trong issue và xác minh rằng các lời gọi thành công hiện có vẫn không thay đổi, đồng thời các yêu cầu được đề xuất ở cấp run và đối với các job thất bại hoạt động đúng.

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

Mô tả

This came out of an analysis of github MCP server errors encountered in my past sessions across 5000 sessions.

What happened

An agent trying to retrieve all job logs for a workflow run passed:

{"owner":"danmoseley","repo":"pr-dashboard","run_id":22854416647,"return_content":true,"tail_lines":500,"failed_only":false}

And got:

job_id is required when failed_only is false

This happened 6 times across 5 sessions. The agent's intent was clear and reasonable: get all logs for this run, not just failed ones. It even explicitly set failed_only=false to signal that — and got an error.

Why the current design is confusing for agents

failed_only is used as a mode switch rather than a modifier:

  • run_id without failed_only=true always errors, even though "get logs for this run" is a valid request
  • failed_only has no effect when job_id is provided — it is silently ignored
Proposed behavior

failed_only should be a consistent modifier on whichever ID is provided:

Parameters Result Proposed
job_id + failed_only=false (or not passed) logs for that job already allowed
job_id + failed_only=true logs for that job if it failed; isError:true with status if it succeeded error -> allowed
run_id + failed_only=false (or not passed) logs for all jobs in the run already allowed
run_id + failed_only=true logs for failed jobs only error -> allowed
both job_id and run_id isError:true — provide one or the other, not both error
neither isError:true — one of job_id or run_id must be provided error

The job_id + failed_only=true case requires checking the job's conclusion before fetching logs, but the tool already fetches job metadata to get the log URL so this is a small addition.

Breaking change note

No changes to existing successful calls. This only makes certain calls that are failing begin to work.

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

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.