Threat detection fails the run when it was skipped and `continue-on-error: false`
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 76/100
Línea de trabajo
Empieza leyendo conclude_threat_detection.sh y las condiciones y pasos de instalación del trabajo de detección; reproduce el caso con continue-on-error: false y sin salidas seguras. Comprueba cómo gestiona la conclusión RUN_DETECTION: false cuando se omitió la instalación. La tarea estará terminada cuando la detección se informe como omitida sin fallar en ese caso, mientras que un detector ausente siga generando un error cuando se esperaba que se ejecutara la detección.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
With safe-outputs.threat-detection.continue-on-error: false, a run in which the agent emits no safe output at all ends with a failed detection job:
ERR_SYSTEM: threat-detect binary not found on PATH
##[error]Process completed with exit code 1.
Nothing was there to analyze. The guard step says so ("Detection skipped: no agent outputs or patches to analyze"), and the run fails anyway, with a message that points at the detector install rather than at the agent.
This happens every time the agent stops before writing to outputs.jsonl. In our case the agent hit max-ai-credits (403 Maximum AI credits exceeded), so outputs.jsonl was never created, and the only failure shown on the run was the threat-detect one.
Seen on v0.89.21 and v0.91.5.
Repro
- A workflow with any safe output and:
safe-outputs: threat-detection: continue-on-error: false - Make the agent end without calling any safe-output tool. For instance, set a
max-ai-creditslow enough to stop it early, or use a prompt that ends without calling any tool,noopincluded. - Look at the
detectionjob:Check if detection needed:OUTPUT_TYPES:empty,HAS_PATCH: false, "Detection skipped: no agent outputs or patches to analyze".Install threat-detect binary: skipped.Conclude threat detection: env hasRUN_DETECTION: false,THREAT_DETECT_INSTALL_OUTCOME: skipped,DETECTION_AGENTIC_EXECUTION_OUTCOME: skipped, thenERR_SYSTEM: threat-detect binary not found on PATHand exit code 1.
Expected
When the guard skipped detection because there was nothing to analyze, the conclusion should report it as skipped, as the comment in conclude_threat_detection.sh describes, whatever continue-on-error is set to. A missing detector should only be reported as an error when detection was actually expected to run.
- Lenguaje dominante
- Go
- Estrellas
- 5.4k
- Forks
- 576
- Merge medio
- 7 h 40 min
- PR fusionados (30 d)
- 806
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un 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 github/gh-aw
-
automation models
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
agentic-workflows maintenance
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
github/gh-aw#34892 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Smoke Test: Kiro - 38325297603Abiertoautomation testing
Dificultad 1/5 Menos de una hora Aptitud para principiantes 25/100
Los mantenedores suelen responder en 1 día
-
agentic-workflows
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
Los mantenedores suelen responder en 1 día
-
agentic-workflows
Dificultad 4/5 3-5 días Aptitud para principiantes 15/100
Los mantenedores suelen responder en 1 día
Todos los issues de github/gh-aw
Issues similares
-
darwin zsh
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
carapace-sh/carapace-bin#3846 ·
Los mantenedores suelen responder en 1 día
-
appplication - is a typo?Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 70/100
golang-app/ecommerce#179 ·
Los mantenedores suelen responder en 1 día
-
Simulator process crashes (nil pointer panic) when the dataset returns an errorPosiblemente ocupada @EugeneVon la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
llm-d/llm-d-inference-sim#772 ·
Los mantenedores suelen responder en 1 día
-
refactor: decode numchildren with a lenient int type instead of a hand-written xProp.UnmarshalXMLAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
crazy-goat/xdbg#125 ·
Los mantenedores suelen responder en 1 día
-
150 points ci complexity:medium drips-wave help wanted medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
sorotrail/SoroBeacon#449 ·
Los mantenedores suelen responder en 1 día