test(source-control): give the pull-request eval cases a gh fixture so ready runs reach the flip
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 25/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- git, github, shell
- Bereich
- devtools, testing-qa
Rechercherichtung
Start with plugins/source-control/evals/pull-request-ready-hands-pr-to-explain-change/scaffold.sh and the ready phase's gh pr view lookup (§2.5.1) in the pull-request skill. The mechanism is undecided: choose between a gh shim the scaffold puts on PATH (check the sandbox's PATH first) and a fixture convention in the /evals:plugin-eval case-authoring reference. Done when the ready cases reach gh pr ready in a kept-trace run.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Context
Found in review of #6987 (Codex finding on plugins/source-control/evals/pull-request-ready-hands-pr-to-explain-change/scaffold.sh).
Problem
The source-control plugin-eval cases for /source-control:pull-request (pull-request-ready-merges-base-never-rebases, pull-request-ready-hands-pr-to-explain-change, and likely the merge and dirty-worktree cases) scaffold only a local git repository: no remote, no pull request, and no gh fixture. The skill's ready phase first resolves the PR (gh pr view, §2.5.1) and stops when that fails, so a correct run never reaches the steps these cases grade (the base merge, gh pr ready, the explain-change hand-off). The graders can then pass only on a run that ignores a failed lookup.
No case in the repository stubs gh today, and a case cannot set PATH (env keys are limited to EVAL_*), so the fix needs a decided mechanism.
Suggested fix
Decide how a plugin-eval case presents a forge to the skill: a gh shim the scaffold installs where the run's PATH finds it (verify the sandbox's PATH first), or a documented fixture convention in /evals:plugin-eval's case-authoring reference. Then apply it to every pull-request case that calls gh, and confirm with a kept-trace run that the ready cases reach gh pr ready.
Related: #6107, #6987
- Vorherrschende Sprache
- Shell
- Sterne
- 22
- Forks
- 2
- Ø Merge
- 8 Std. 25 Min.
- Gemergte PRs (30 T.)
- 944
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus melodic-software/claude-code-plugins
-
good first issue priority: low
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
melodic-software/claude-code-plugins#7530 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
needs-human priority: medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
melodic-software/claude-code-plugins#7455 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
priority: medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
melodic-software/claude-code-plugins#7351 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
agent-ready priority: high work-class: scoped
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
melodic-software/claude-code-plugins#7312 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
priority: medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
melodic-software/claude-code-plugins#7288 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in melodic-software/claude-code-plugins
Ähnliche Issues
-
area: compat area: parsing bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
reubeno/brush#1526 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Micke-K/IntuneManagement#441 ·
-
review-2026-10 Trad En
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 70/100
zigbeefordomoticz/wiki#302 ·
-
help wanted page edit
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 62/100
tldr-pages/tldr#24736 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
refactoring
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
UnionInternationalCheminsdeFer/OSDM#1647 ·
Maintainer antworten meist innerhalb von 1 Tag