fix(ai-slop): detect.sh --paths-file drops the last entry when the file has no trailing newline
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 86/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- bash
- Área
- cli
Línea de trabajo
Lee plugins/ai-slop/skills/audit/scripts/lib/resolve-targets.sh y localiza el bucle de lectura de --paths-file. Reproduce el problema con un archivo de rutas cuya última entrada no tenga un salto de línea final y luego actualiza el bucle como se describe. Añade una prueba de resolve-targets para ese caso y confirma que se procesa la última ruta.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
plugins/ai-slop/skills/audit/scripts/lib/resolve-targets.sh reads --paths-file with while IFS= read -r line; do ... done. When the file's last line has no trailing newline, read returns nonzero after assigning that line, so the loop never processes it. A one-entry file then reports that it lists no paths and exits successfully, and a longer list silently omits its final file.
Reproduce, from a directory holding a.md and b.md:
printf 'a.md\nb.md' > list
bash plugins/ai-slop/skills/audit/scripts/detect.sh --list-targets --paths-file list # lists a.md only
printf 'a.md\nb.md\n' > list2
bash plugins/ai-slop/skills/audit/scripts/detect.sh --list-targets --paths-file list2 # lists a.md and b.md
Fix: while IFS= read -r line || [[ -n $line ]]; do, with a resolve-targets test for a file with no trailing newline.
Found by Codex review on https://github.com/melodic-software/pr-pipeline-sandbox/pull/195 (a byte copy of this plugin). The pr-refine fix-docs lane is not affected, because it passes its targets in the prompt rather than through a paths file.
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 6 h 58 min
- PR fusionados (30 d)
- 951
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 status: needs-decision work-class: scoped
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
melodic-software/claude-code-plugins#9339 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
pipeline-canary priority: low status: ready work-class: mechanical
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
melodic-software/claude-code-plugins#9263 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
good first issue priority: low
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
melodic-software/claude-code-plugins#9251 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area: security needs-triage priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
melodic-software/claude-code-plugins#9211 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
melodic-software/claude-code-plugins#9202 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de melodic-software/claude-code-plugins
Issues similares
-
level/task type/enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
wazuh/wazuh-installation-assistant#1177 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
ComputeCanada/puppet-magic_castle#634 ·
Los mantenedores suelen responder en 1 día
-
source:reader type:enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
stephrobert/blog-roadmap#115 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
pi-hole/docker-pi-hole#2087 ·
-
Python Volcengine provider returns failures instead of raising themPosiblemente ocupada @Ricardo-M-L la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
agent-infra/sandbox#248 ·