Skip to content

chore(quality): rebaseline zizmorFindings 152→155 (nightly-release-green.yml drift) - #4669

Closed
diegosouzapw wants to merge 3 commits into
release/v3.8.37from
chore/qg-zizmor-rebaseline-155
Closed

diegosouzapw wants to merge 3 commits into
release/v3.8.37from
chore/qg-zizmor-rebaseline-155

Conversation

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Job vermelho que isto conserta

Quality Gates (Extended) no CI run #27950299650 — [check-workflows] REGRESSION — 155 zizmor finding(s) > baseline 152.

Causa

O ciclo v3.8.34 adicionou um único workflow novo: .github/workflows/nightly-release-green.yml (#4622, Solution C+D). Ele traz 3 unpinned-uses:

  • actions/checkout@v7
  • actions/setup-node@v6
  • actions/upload-artifact@v4

Esses seguem a convenção @vN deliberada do repo (137+ unpinned-uses já existentes, marcados INTOCADA na nota _scanner_harden_workflows_2026_06_16 do baseline). Não há template-injection (todos os ${{...}} já vão via env:) e o checkout usa persist-credentials: false (sem artipacked).

Ação

Rebaseline metrics.zizmorFindings.value 152 → 155 + nota de justificativa. Drift legítimo de novo workflow, consistente com o precedente documentado.

Verificação

  • Local com zizmor 1.25.2: [check-workflows] --ratchet OK — 155, baseline 155 (no regression) (exit 0).
  • Reproduz o valor do CI (155).
  • JSON validado (jq empty).
  • Apenas config/quality/quality-baseline.json alterado.

Diego Rodrigues de Sa e Souza and others added 3 commits June 22, 2026 04:35
…C+D) (#4622)

C — scripts/quality/validate-release-green.mjs (npm run check:release-green):
reproduces the release-equivalent validation (typecheck, eslint, db-rules,
public-creds, full unit, vitest, ratchets, optional --with-build package-artifact)
against the current working tree and classifies each red as HARD (real defect,
exit 1) vs DRIFT (ratchet — reported, never affects exit / never blocks). Pure
helpers exported + orchestration behind a direct-run guard; unit-tested.

D — .github/workflows/nightly-release-green.yml: runs C on the active release
branch nightly (and on workflow_dispatch) and opens/updates a single tracking
issue on HARD failures. Never a required check, never touches a contributor PR.

Closes the gap where the full gate (ci.yml) only ran on the release PR, so reds
accrued silently on release/** and surfaced in 40-min layers at release time.
Non-blocking by construction; drift is the maintainer's to rebaseline at release.

Co-authored-by: Diego Rodrigues de Sa e Souza <diego.souza@cdwasolutions.com.br>
…reen.yml drift)

O novo workflow .github/workflows/nightly-release-green.yml (#4622, Solution C+D do
ciclo v3.8.34) traz 3 unpinned-uses (actions/checkout@v7, setup-node@v6,
upload-artifact@v4) seguindo a convencao @vn deliberada do repo (137+ unpinned
existentes, INTOCADA). Sem template-injection (todos os ${{...}} via env:), checkout
com persist-credentials:false. Medido localmente com zizmor 1.25.2 = 155 e no CI run
#27950299650 (job Quality Gates Extended) = 155 > baseline 152. Drift legitimo de novo
workflow -> rebaseline, consistente com _scanner_harden_workflows_2026_06_16.
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@diegosouzapw
diegosouzapw changed the base branch from release/v3.8.34 to release/v3.8.35 June 23, 2026 06:49
@diegosouzapw
diegosouzapw changed the base branch from release/v3.8.35 to release/v3.8.36 June 23, 2026 21:35
@diegosouzapw

Copy link
Copy Markdown
Collaborator Author

Skipped this batch: superseded. The release baseline already moved past this rebaseline — zizmorFindings.value is now 159 on release/v3.8.36 (rebaselined 155→159 by _rebaseline_2026_06_23_fastpath_gates, after this PR's 152→155). Merging would set it back to 155, regressing the ratchet (measured 159 > baseline 155 → FAIL). The +3 drift this PR documents (nightly-release-green.yml) is already folded into the current 159. Recommend closing as obsolete. The other 11 PRs in the batch are merged.

@diegosouzapw
diegosouzapw changed the base branch from release/v3.8.36 to release/v3.8.37 June 25, 2026 22:01
@diegosouzapw

Copy link
Copy Markdown
Collaborator Author

Fechando: stale. O baseline zizmorFindings no release/v3.8.37 já está em 159 (config/quality/quality-baseline.json), e o salto 152→155 desta PR já foi absorvido na história até 159. Aplicá-la regrediria o gate (155 < 159).

@diegosouzapw
diegosouzapw deleted the chore/qg-zizmor-rebaseline-155 branch June 26, 2026 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant