fix(knowledge): list-promotion-candidates prints a stack trace on a malformed claim inventory
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
- 82/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript, nodejs
- Área
- cli
Línea de trabajo
Empieza con list-promotion-candidates.js en plugins/knowledge/skills/video-digest/extraction/watch/: el throw de cero sesiones está alrededor de las líneas 45-48, y el bloque de CLI alrededor de las líneas 114-121 no tiene catch. Copia el manejo de errores del archivo hermano vision-gated-promote.js (líneas 93-107), que escribe el mensaje en stderr y sale con código 1. Se considera terminado cuando un inventario malformado imprime solo el mensaje de una línea con la pista de formato y termina con un código distinto de cero, y una prueba de CLI verifica ambas cosas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
Run as a CLI, list-promotion-candidates.js calls listPromotionCandidates with no error handling, so its documented failure (no session parsed from research/claim-inventory.md) and any missing-file error escape as an uncaught exception with a Node stack trace. The sibling vision-gated-promote.js catches, prints the message to stderr and exits 1.
Failure scenario
Expected: a malformed claim inventory prints the one-line message with the format hint to stderr and exits with a nonzero code, with no stack trace.
Actual: the function throws at the zero-sessions check, and the CLI block does not catch it, so Node prints the stack trace. The useful part (SESSION_FORMAT_HINT) is buried under it.
Evidence
- The throw: https://github.com/melodic-software/claude-code-plugins/blob/7787be41e98e30cd6e3b82e99494886d77e30895/plugins/knowledge/skills/video-digest/extraction/watch/list-promotion-candidates.js#L45-L48
- CLI block with no catch: https://github.com/melodic-software/claude-code-plugins/blob/7787be41e98e30cd6e3b82e99494886d77e30895/plugins/knowledge/skills/video-digest/extraction/watch/list-promotion-candidates.js#L114-L121
- The sibling's pattern to match: https://github.com/melodic-software/claude-code-plugins/blob/7787be41e98e30cd6e3b82e99494886d77e30895/plugins/knowledge/skills/video-digest/extraction/watch/vision-gated-promote.js#L93-L107
Suggested fix
Wrap the call in try/catch, write error.message with writeStderr, and process.exit(1), as vision-gated-promote.js does. Add a CLI test with a claim inventory that parses to zero sessions asserting a one-line stderr and a nonzero exit.
Related
- #6813 (session-coverage gates and a zero-session inventory)
🤖 Generated with Claude Code
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 8 h 27 min
- PR fusionados (30 d)
- 957
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
-
agent-ready priority: high work-class: scoped
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
melodic-software/claude-code-plugins#7312 · 3 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
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
e2e-failure ready-to-code
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
redhat-developer/rhdh-plugin-export-overlays#4391 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
graygarbage/HermesKey#20 ·
-
Version-drift constraint evaluator (#40): validate the candidate version and run its unit test in CIAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
n8n-io/terraform-google-n8n#41 ·
Los mantenedores suelen responder en 5 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100