[ci] Inventory workflow jobs explicitly - #3576
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## Gqofuqcjxqt64bw7di2snqyunualoi3ht #3576 +/- ##
==================================================================
Coverage 91.85% 91.85%
==================================================================
Files 20 20
Lines 6093 6093
==================================================================
Hits 5597 5597
Misses 496 496 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
4aa8517 to
0c5a889
Compare
ca89a5d to
1b5b56e
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
1b5b56e to
67fc58e
Compare
0c5a889 to
057e851
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
67fc58e to
f277ad7
Compare
0be1058 to
dabba7d
Compare
f277ad7 to
453c9c9
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Swish! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Add a narrow scanner for every lower-case YAML workflow file and its canonical top-level job IDs. Compare that live inventory with a sorted, reviewed registry which assigns each job a plain-language role. Reject quoted or anchored job declarations, duplicate jobs, unfamiliar jobs keys, case-variant YAML extensions, unsorted registry rows, unknown roles, and any missing or unreviewed job. Continue using action-validator for the complete YAML and GitHub Actions schema rather than implementing another YAML interpreter in zc. This commit only establishes the handwritten-workflow inventory boundary. Its crate-private entry point is temporarily allowed to be unused outside tests; the all-input boundary removes that allowance when it begins calling the audit. Later checks use the roles to constrain generated data and keep required static validation and aggregation jobs visible. Tests: CARGO_NET_OFFLINE=true ./ci/check_tools.sh Tests: ./ci/check_fmt.sh *Authored by an agent, posting via joshlf's account* gherrit-pr-id: Ghrxpfppzzenc5ecwz3u273gd42e4psk7
453c9c9 to
cc02374
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
2 participants
Add a narrow scanner for every lower-case YAML workflow file and its
canonical top-level job IDs. Compare that live inventory with a sorted,
reviewed registry which assigns each job a plain-language role.
Reject quoted or anchored job declarations, duplicate jobs, unfamiliar
jobs keys, case-variant YAML extensions, unsorted registry rows, unknown
roles, and any missing or unreviewed job. Continue using
action-validator for the complete YAML and GitHub Actions schema rather
than implementing another YAML interpreter in zc.
This commit only establishes the handwritten-workflow inventory
boundary. Its crate-private entry point is temporarily allowed to be
unused outside tests; the all-input boundary removes that allowance when
it begins calling the audit. Later checks use the roles to constrain
generated data and keep required static validation and aggregation jobs
visible.
Tests: CARGO_NET_OFFLINE=true ./ci/check_tools.sh
Tests: ./ci/check_fmt.sh
Authored by an agent, posting via joshlf's account
Latest Update: v12 — Compare vs v11
📚 Full Patch History
Links show the diff between the row version and the column version.
⬇️ Download this PR
Branch
git fetch origin refs/heads/Ghrxpfppzzenc5ecwz3u273gd42e4psk7 && git checkout -b pr-Ghrxpfppzzenc5ecwz3u273gd42e4psk7 FETCH_HEADCheckout
git fetch origin refs/heads/Ghrxpfppzzenc5ecwz3u273gd42e4psk7 && git checkout FETCH_HEADCherry Pick
git fetch origin refs/heads/Ghrxpfppzzenc5ecwz3u273gd42e4psk7 && git cherry-pick FETCH_HEADPull
Stacked PRs enabled by GHerrit.