Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

[deep-report] Add --no-security-scanner flag to gh aw upgrade (CLI safety-guard parity gap missed by ADR-45573)

Cerrado Apto para principiantes
  • ¿Algo no está bien?
#63,482 0 comentarios 0 reacciones 0 asignados Ver en GitHub

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
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
github-actions, go
Área
ci-cd, cli

Línea de trabajo

Comienza en pkg/cli/upgrade_command.go:347-368 y compara el registro del flag y la llamada de compilación con la integración en pkg/cli/update_manifest.go:582-585,629-632. Traza cómo llega DisableSecurityScanner a compileWorkflowsForUpdate. Se considera terminado cuando gh aw upgrade acepta --no-security-scanner y pasa la opción por la canalización compartida para que se pueda omitir el escáner.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

automation cli code-quality cookie improvement quick-win task-mining
Description

ADR-45573 (2026-07-15) closed three of four CLI sibling-command flag-parity gaps between compile/validate/update/upgrade, and explicitly deferred --no-security-scanner on compile/validate as an intentional no-op ("those commands operate exclusively on local files where the scanner is a no-op"). That reasoning does not extend to upgrade: tracing upgrade's Step 4 compile call (compileWorkflowsForUpdate → shared CompileWorkflows pipeline, pkg/cli/upgrade_command.go:347-368) shows upgrade shares the exact same security-scan-enforcing code path as update's manifest-managed-workflow rewrite (pkg/cli/update_manifest.go:582-585,629-632), where a failing ScanMarkdownSecurity finding hard-fails with the message "...or pass --no-security-scanner to skip this check" — a flag upgrade does not register.

Any CI pipeline running gh aw upgrade on a workflow whose upstream manifest source trips a security-scan finding has no bypass short of falling back to update or manual editing — reproducing the exact UX problem ADR-45573 was written to solve, for a flag the ADR's audit missed.

Suggested fix

Register --no-security-scanner on the upgrade command and thread it into the same DisableSecurityScanner option consumed by compileWorkflowsForUpdate, mirroring update's existing flag wiring in pkg/cli/update_manifest.go.

Expected Impact

Closes a real (not hypothetical) escape-hatch gap for CI pipelines running gh aw upgrade, achieving full flag parity across compile/validate/update/upgrade per the original intent of ADR-45573.

Suggested Agent

CLI maintenance / code-quality agent.

Estimated Effort

Quick (~1h)

Data Source

DeepReport Intelligence Briefing analysis (2026-09-25, cycle 3), sourced from Repository Quality Improvement Report #63445.

Generated by 🔬 Deep Report · claude · agent · 202.3 AIC · ⌖ 9.81 AIC · ⊞ 13K · ◷

  • expires on Sep 27, 2026, 10:39 AM UTC-08:00
Lenguaje dominante
Go
Estrellas
5.4k
Forks
596
Merge medio
3 h 37 min
PR fusionados (30 d)
833

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de github/gh-aw

Todos los issues de github/gh-aw

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.