test(source-control): make the ready case's base-merged grader check the merged file, not the command
メンテナーはふだん 1 日以内に返信
関連するプルリクエストがすでにマージされています。
- #10915 @melodic-automation-lanes による — マージ済み
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 76/100
調査の方向性
plugins/source-control/evals/pull-request-ready-merges-base-never-rebases/graders/base-merged.md とケースの scaffold.sh を読み、src/extra.ts が main に追加されることを確認してください。grader を変更して、そのファイルが両方のブランチに存在することを確認させてから、python3 plugins/evals/skills/validate/scripts/validate-cases.py plugins/source-control/evals を実行してください。grader がマージされたファイルを確認し、検証が終了コード 0 で終了したら完了です。
索引モデルが issue の本文から書いたものです。
説明
Parent: #6997 (slice 1 of 2). Runs first; slice 2 waits on it.
Runs in: lane. Grader and static validation only; no model run.
Problem
In the 2026-10-24 rerun, the base-merged grader of pull-request-ready-merges-base-never-rebases passed in a with-plugin run whose command was git fetch origin main && git merge --no-edit origin/main. The fetch failed, so the merge never ran. The grader (graders/base-merged.md: type: tool_used, tool: Bash, input_match: 'git\s+merge...') credits the attempted command, not a merge that happened. A tool_used grader matches only a call's input (plugins/evals/skills/validate/scripts/validate-cases.py:153), so it cannot see the result.
The case's scaffold gives an outcome to check: main adds src/extra.ts after the branch forks (scaffold.sh, "chore: advance main"), so the file is in the workspace only after the base really merged.
Fix
Change graders/base-merged.md to a file_exists grader on src/extra.ts (exists: true, arm: both), so it passes only when the merge landed. Keep no-rebase and base-merged-before-flip as they are. If the case description or expected_outcome names the command, reword it to the outcome. Add a none changelog fragment for source-control if only an eval case changed, per the repo's fragment rules.
Done when
-
grep -c '^type: file_exists' plugins/source-control/evals/pull-request-ready-merges-base-never-rebases/graders/base-merged.mdprints 1 -
grep -c 'src/extra.ts' plugins/source-control/evals/pull-request-ready-merges-base-never-rebases/graders/base-merged.mdprints 1 -
grep -c 'src/extra.ts' plugins/source-control/evals/pull-request-ready-merges-base-never-rebases/scaffold.shprints 1 or more (the file is still added only on main) -
python3 plugins/evals/skills/validate/scripts/validate-cases.py plugins/source-control/evalsexits 0 -
ci-statusis green on the PR head
Split by the backlog HITL session (2026-10-25, owner rule Q93: split large work before a run picks it up). This was generated by AI.
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 2時間 40分
- マージ済み PR(30日)
- 936
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
host-run priority: medium status: ready work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
melodic-software/claude-code-plugins#11191 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
host-run priority: medium status: ready work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
melodic-software/claude-code-plugins#11189 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
fix(planning): read back answers that depart from the recommendation before marking them answeredオープンhost-run priority: medium status: ready work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
melodic-software/claude-code-plugins#11187 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
host-run priority: low status: confirmed work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
melodic-software/claude-code-plugins#11147 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium status: needs-decision work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
melodic-software/claude-code-plugins#10492 · コメント 7 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
severity: low
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
luainkernel/lunatik#2221 ·
メンテナーはふだん 1 日以内に返信
-
priority middle
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
KATO-Hiro/AtCoderClans#12945 ·
メンテナーはふだん 1 日以内に返信
-
needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
mattpocock/skills#1360 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
Niko1221/Strata#4779 · コメント 2 件 ·
メンテナーはふだん 3 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
community-scripts/ProxmoxVE#18332 ·
メンテナーはふだん 1 日以内に返信