aws-samples / aws-samples/sample-autonomous-cloud-coding-agents

25 pre-existing masking findings on main fail the pre-push gate, forcing --no-verify for every contributor

Aperta
#766 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
bug ci-cd tooling
Lingua principale
TypeScript
Stelle
143
Fork
46
Merge medio
3g 10h
PR unite (30g)
24

Descrizione

## Problem

The masking-findings gate fails on **pristine `main`** with 25 pre-existing findings (up from 15 when the #763 work was briefed). Because the gate runs pre-push, every contributor must push with `--no-verify` to land anything — which also skips every *other* pre-push check, so the gate's growth is silently eroding the whole hook.

## Evidence

- Reproduced on a clean checkout of `main` (no branch changes applied): 25 findings.
- PR #763 shipped with `--no-verify` for this gate only; **no `nosemgrep` suppressions were added**, so the findings remain visible.
- The count grew 15 → 25 between the #763 brief and merge, so the backlog is actively accumulating.

## Suggested resolution

1. Triage the 25 findings on `main`: fix the real ones, suppress false positives with justified `nosemgrep` annotations.
2. Once `main` is clean, the gate becomes meaningful again and `--no-verify` stops being the default workflow.
3. Consider a baseline/ratchet mechanism (fail only on **new** findings) if a full cleanup can't land soon, so the gate blocks regressions without blocking unrelated work.

## Origin

Found during PR #763 — see the merge-guidance comment: https://github.com/aws-samples/sample-autonomous-cloud-coding-agents/pull/763#issuecomment-5289889688

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia riproducendo il gate di masking-findings su un checkout pulito di main e analizza i suoi 25 findings segnalati, usando come contesto il commento di merge-guidance di PR #763. Classifica ogni finding come reale o false positive, quindi verifica che il gate passi su un main pulito senza --no-verify; se la pulizia non può essere integrata, valuta una baseline o un ratchet che blocchi solo i nuovi findings.

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

Valutazione

Ambito
security, tooling
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
48/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.