test(source-control): put the eval gh fixture on PATH without relying on zsh's .zshenv
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
Start with plugins/source-control/evals/fixtures/gh-forge/install.sh and the case.yaml environment limits described in the issue; review the findings from #7164 and the runner capability options. The change depends on an upstream runner capability request and a kept-trace run on a bash-SHELL host. Done means ready cases reach gh pr ready through the stand-in and install.sh no longer writes ~/.zshenv.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Context
Follow-up to #7164 (#6997). The owner accepted zsh-only coverage for the offline gh fixture so that PR could merge; this issue tracks the shell-independent fix.
Problem
plugins/source-control/evals/fixtures/gh-forge/install.sh puts the stand-in gh on PATH only by appending an export to the run's ~/.zshenv. On a host whose SHELL is bash (a default CI runner, for example), the stand-in is never on PATH: the run falls through to the real gh, which has no login inside the run and no network under the sandbox, so the ready cases stop at the PR lookup again.
Evidence from #7164 (Claude Code 2.1.296 probes):
- A case cannot set
PATH:case.yamlenvkeys are limited toEVAL_*. - The scaffold's
HOMEis the run's temporary home, and the eval run does not source~/.zshrc,~/.bashrcor~/.profile: functions and variables defined there were missing from the run's Bash. - Only
~/.zshenvtook effect, because zsh reads it on everyzsh -c. That is undocumented runner behavior. - No run was made on a bash-
SHELLhost; the fall-through there is inferred from the above.
Ask
The clean fix needs a capability in claude plugin eval itself, for example:
- a case-level
PATHprefix (acase.yamlkey naming a directory under the scaffolded workspace to prepend), or - a scaffold-written env file the runner applies to the run's shell.
This repository cannot add either; record the request upstream through the owner's normal channel, then switch gh-forge/install.sh to the new mechanism and drop the .zshenv append. An operator-level dispatcher gh on PATH (option 3 in #7164) works on any shell but adds a convention to every invocation; it stays the fallback if upstream declines.
Acceptance
- The
pull-requestready cases reachgh pr readythrough the stand-in on a bash-SHELLhost, shown by a kept-trace run. install.shno longer writes~/.zshenv.
Related: #6997, #7164
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 7 h 42 min
- PR fusionados (30 d)
- 923
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de melodic-software/claude-code-plugins
-
priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
melodic-software/claude-code-plugins#7351 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
priority: high
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
melodic-software/claude-code-plugins#7312 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#7288 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
knowledge: docpage-digest dual verification defines no verdict vocabulary or pass/fail thresholdAbiertoneeds-human
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
melodic-software/claude-code-plugins#7286 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#7282 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de melodic-software/claude-code-plugins
Issues similares
-
Laptop display stays disabled after unplugging external monitor: FALLBACK output counted as externalAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
-
[Bug]: Three default prefs with JSON values are never set (override.sh escapes their quotes twice)Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Floorp-Projects/Floorp#2913 ·
Los mantenedores suelen responder en 1 día
-
agent-butler-finding agent-research-recommend bug ready-for-agent
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
jordansmall/spindrift#5258 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Copy as CLI command writes words.json with a heredoc that silently overwrites an existing fileAbiertoenhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día