fix(pr-pipeline): scope read-runner caller secrets per caller in check-read-caller-keys
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 74/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- github-actions, javascript
Línea de trabajo
Empieza con scripts/check-read-caller-keys.mjs e inspecciona cómo lee los workflows de los llamadores y valida ALLOWED_SECRETS; usa como referencia el manejo existente del token de automatizaciones. Añade o localiza las pruebas del script y cubre el secreto propio de cada llamador, el secreto de otro lane y un llamador que no figure en el mapa. El trabajo estará terminado cuando los tres casos se comporten según lo especificado y pasen las comprobaciones.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
scripts/check-read-caller-keys.mjs holds the allowed secret names for read-runner callers in one global set, ALLOWED_SECRETS (line 30 after #8691). So any caller of pr-run-activity-read.yml may pass any allowed lane credential, for example issue_lane_claude_code_oauth_token from a caller that is not the issue lane, and the check stays green. The automations token already works this way.
Abusing this needs a workflow change that is already on main, so it is a hardening item, not a live gap.
Fix
Replace the global set with a per-caller map from caller workflow file to the secret names it may pass, for example work-resolve-issue.yml → the ISSUE_LANE_ names and pr-unblock.yml → the AUTOMATIONS_ names. A caller that passes a secret outside its own entry fails the check. Add tests:
- each caller's own secret passes;
- another lane's secret fails;
- a caller missing from the map fails.
Refs: #8691
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 7 h 57 min
- PR fusionados (30 d)
- 938
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: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#8831 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
priority: medium status: confirmed
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
melodic-software/claude-code-plugins#8801 ·
Los mantenedores suelen responder en 1 día
-
priority: low status: confirmed
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
melodic-software/claude-code-plugins#8799 ·
Los mantenedores suelen responder en 1 día
-
priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
melodic-software/claude-code-plugins#8730 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
fix(source-control): ready-hold check treats a Hold: line that says do not merge as a flip holdAbiertopriority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
melodic-software/claude-code-plugins#8708 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de melodic-software/claude-code-plugins
Issues similares
-
product-to-storybook is skipped silently by npx skills add because its frontmatter is invalid YAMLAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 83/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día
-
bug github_actions security
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
DewaldOosthuizen/tech-cheat-sheets-and-notes#388 ·
Los mantenedores suelen responder en 1 día
-
Link Checker ReportAbiertoautomated issue report
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
a2aproject/A2A#2376 ·
Los mantenedores suelen responder en 3 días
-
kftui is not on PATH in bash login shells that use ~/.bash_profilePosiblemente ocupada @hcavarsan la tomó hoy. Abiertoarea:ci kind:bug priority:p3
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día