feat(source-control): review instruction-surface diffs in pull-request ready
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
- 68/100
Línea de trabajo
Lee plugins/source-control/skills/pull-request/reference/ready-for-review.md, especialmente la sección 2.5.3 y sus condiciones para cambios que solo afectan a la documentación. Actualiza la regla para que los diffs de superficies de instrucciones como AGENTS.md, CLAUDE.md, .claude/rules/ y los cuerpos de skills o agentes reciban la revisión de seguridad antes de pasar a ready. Se considerará terminado cuando esa revisión se ejecute para esos diffs y los diffs ordinarios que solo afectan a la documentación conserven el comportamiento indicado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Refs: #7346
/source-control:pull-request ready skips both the code and security review for a docs-only diff (plugins/source-control/skills/pull-request/reference/ready-for-review.md 2.5.3, the "for any diff that is not docs-only" conditions). A diff that changes an instruction surface (AGENTS.md, CLAUDE.md, .claude/rules/, skill and agent bodies) is docs-only by file type but changes what agents do. Under the carve-out #7346 proposes, an agent could flip such a PR it opened without any local review.
Mitigating today: the flip starts the CI review lanes, and merging stays separately gated.
Proposed: treat an instruction-surface diff as not docs-only for 2.5.3, so at least the security review runs before the flip.
Found by the claude-review lane on #7346 and by a local security review during the ready flow.
🤖 Generated with Claude Code
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 8 h 41 min
- PR fusionados (30 d)
- 892
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
-
good first issue needs-human priority: low work-class: mechanical
Dificultad 2/5 1-3 horas Aptitud para principiantes 92/100
melodic-software/claude-code-plugins#8023 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
needs-human work-class: mechanical
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#8021 ·
Los mantenedores suelen responder en 1 día
-
needs-human work-class: scoped
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
melodic-software/claude-code-plugins#8018 ·
Los mantenedores suelen responder en 1 día
-
docs(planning): point the interview surface context at the surface README instead of restating itAbiertoneeds-human work-class: mechanical
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
melodic-software/claude-code-plugins#8015 ·
Los mantenedores suelen responder en 1 día
-
needs-human work-class: mechanical
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
melodic-software/claude-code-plugins#8004 ·
Los mantenedores suelen responder en 1 día
Todos los issues de melodic-software/claude-code-plugins
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
-
good first issue packaging
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
-
Let's document: tailscale funnelAbiertohelp wanted let's document new command
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
tldr-pages/tldr#24773 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
Los mantenedores suelen responder en 1 día