get_job_logs: run_id with failed_only=false should return all job logs; failed_only should be a consistent modifier
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 64/100
調査の方向性
get_job_logs の実装から始め、既存のジョブメタデータ検索を含めて、job_id、run_id、failed_only がどのように検証されているかを追跡します。issue に記載されたパラメーターの組み合わせを対象とする集中的なカバレッジを追加し、既存の成功する呼び出しが変更されないまま、提案された run レベルおよび失敗したジョブに対するリクエストが機能することを確認します。
索引モデルが issue の本文から書いたものです。
説明
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_idwithoutfailed_only=truealways errors, even though "get logs for this run" is a valid requestfailed_onlyhas no effect whenjob_idis 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.
- 主要言語
- Go
- スター
- 33.1k
- フォーク
- 5k
- 平均マージ
- 2日 1時間
- マージ済み PR（30日）
- 25
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github/github-mcp-server のほかの issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
github/github-mcp-server#3235 ·
-
enhancement
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
github/github-mcp-server#3042 · コメント 2 件 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
github/github-mcp-server#3032 · リアクション 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
github/github-mcp-server#2803 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
github/github-mcp-server#2740 ·
github/github-mcp-server の issue をすべて見る
似ている issue
-
feature-request helm
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
gravitational/teleport#69785 ·
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
crossplane/crossplane#7859 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 90/100