Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

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

Chiusa Adatta ai principianti
  • Qualcosa non va?
#63,482 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
github-actions, go
Ambito
ci-cd, cli

Direzione di ricerca

Inizia da pkg/cli/upgrade_command.go:347-368 e confronta la registrazione del flag e la chiamata di compilazione con il wiring in pkg/cli/update_manifest.go:582-585,629-632. Traccia come DisableSecurityScanner raggiunge compileWorkflowsForUpdate. Il lavoro è completato quando gh aw upgrade accetta --no-security-scanner e passa l’opzione attraverso la pipeline condivisa, in modo da poter bypassare lo scanner.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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
Lingua principale
Go
Stelle
5.4k
Fork
596
Merge medio
3h 51m
PR unite (30g)
831

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di github/gh-aw

Tutte le issue di github/gh-aw

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.