ci: default maturity evidence to all profile#96595
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 24, 2026, 7:41 PM ET / 23:41 UTC. Summary PR surface: Tests +3, Config 0. Total +3 across 3 files. Reproducibility: not applicable. this is a CI workflow/admin contract change, not a runtime bug. Source inspection confirms current main still uses Review metrics: 1 noteworthy metric.
Stored data model Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Risk before merge
Maintainer options:
Next step before merge
Security Review detailsBest possible solution: Land the branch only after maintainers explicitly accept the full-taxonomy evidence contract; keep or add release-profile artifact compatibility if that operator path is still needed. Do we have a high-confidence way to reproduce the issue? Not applicable: this is a CI workflow/admin contract change, not a runtime bug. Source inspection confirms current main still uses Is this the best way to solve the issue? Yes, with maintainer acceptance. The diff keeps caller, validators, generated copy, and guard tests aligned; the safer alternative is to preserve release-profile artifact compatibility if maintainers still need that path. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 3ab8d6aa609a. Label changesLabel justifications:
Evidence reviewedPR surface: Tests +3, Config 0. Total +3 across 3 files. View PR surface stats
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
What Problem This Solves
The QA Profile Evidence workflow defaulted manual dispatches to the narrower
releaseprofile, while the maturity scorecard refresh path is intended to use full taxonomy evidence.Why This Change Was Made
all.allprofile evidence.User Impact
Maintainers refreshing the maturity scorecard get full taxonomy QA evidence by default instead of accidentally rendering from the narrower release profile.
Evidence
node scripts/run-vitest.mjs test/scripts/ci-workflow-guards.test.tspnpm exec oxfmt --check --threads=1 test/scripts/ci-workflow-guards.test.tsactionlint .github/workflows/qa-profile-evidence.yml .github/workflows/maturity-scorecard.ymlgit diff --check