fix(ci): cure the three main reds — drop Deno CI (#160), bump governance to standards@fad242d3 (#163) - #164
Merged
Merged
Conversation
… to fad242d3 - Delete .github/workflows/deno-ci.yml and examples/web-project-deno.json. Deno is banned estate-wide (bun is the runtime) and the workflow ran on every pull request, so no PR could be fully green. Closes #160. The `deno-ci.yml: []` key leaves actions.lock with it; `gh actions-lock --no-fix` still verifies (16 workflows). - Bump governance-reusable.yml bd0df9ea → fad242d3 (standards main 2026-09-07, the pin tropical-types and nextgen-typing are green on). Cures: Allowlist Preflight (the live policy check is now a credentialed advisory job that skips without HYPATIA_SCAN_PAT), Workflow security linter (the SPDX predicate reads the whole leading comment block, so the `# managed by gh actions-lock` line 1 no longer hides line 2), and Code quality + docs (check-docs-presence.sh accepts .github/CONTRIBUTING.md, its documented canonical location). Lock-free: every inner action of the new reusable is already in actions.lock at the same SHA. Closes #163. Measured locally on this branch before push: standards' check-actions-lock-gate.sh (--verify-local) valid; exemption + debt ratchets OK; check-docs-presence.sh pass; SPDX + permissions predicate 16/16 workflows. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QYY8Gp4v4x2J7iSNn1vZ57
Contributor
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cures the three reds that sit on every push to
mainand every PR, so that a PR can be fully green again. Closes #160. Closes #163.What changes
.github/workflows/deno-ci.ymlexamples/web-project-deno.json.github/workflows/governance.ymlbd0df9ea→fad242d35291de1898242d6737ba02b74a59a2f2(standards main 2026-09-07, post-#742).github/workflows/actions.lockdeno-ci.yml: []key.github/CONTRIBUTING.mdstays where it is:check-docs-presence.shat the new pin accepts that path and names it the canonical location, so no move was needed.How each #163 red is cured (measured, not assumed)
fad242d3the live-policy probe is a separate credentialed advisory job (actions-policy-live) that emits::warning::and neverexit 1;allowlist-preflightno longer needsHYPATIA_SCAN_PAT(declaredrequired: false).# managed by gh actions-lockline 1 no longer hides theSPDX-License-Identifieron line 2. Ran the exact predicate over all 16 workflows on this branch: 0 failures. Also ran the three new steps' scripts locally (check-workflows-parse.sh,check-workflow-duplicate-keys.shatfad242d3;check-action-pins-resolve.shfrom main): all 16 parse, 16 clean, 3/3 pins resolve.check-docs-presence.shpasses with CONTRIBUTING under.github/(✅ Core documentation present).Other gates at the new pin, run locally against this branch:
check-actions-lock-gate.shwithupdate-actions-lock.sh --verify-local(theactions-lock-verifyjob):valid: true, transitive coverage verified. One pre-existing advisorysha-as-refnote onpush-email-notify.yml's bare-SHA ref, not a failure.origin/main: unchanged (.hypatia-ignore9,root-allow.txt1), OK. Debt ratchet: no Debtfile, nothing to ratchet.check-ts-allowlist.sh,check-language-policy.sh(the two new language-policy steps): both pass. Tracked*.vfiles are Coq; the V-lang rule keys onv.mod/vpkg.jsononly.Lock-free bump: every step-level action inside
governance-reusable.yml@fad242d3(checkout3d3c42e5, cache55cc8345, editorconfig840e866d, setup-beam54075bcc) is already inactions.lockat that SHA. Standards HEAD would bump editorconfig to51f63319and force a lock regeneration; deliberately not taken.Note on the
Rust CIcheckrust-ci-reusable.yml@bd0df9ea's detect job checks outrefs/pull/N/merge. When a PR is merged within seconds of opening, the merge ref is gone by the time the queued job starts and the check fails onCheckout repository(that is what happened to #158 and #159; #157, held open, passed). Holding this PR open until every check is green avoids the race; a pin bump to a reusable with thegithub.shacure is a separate follow-up.🤖 Generated with Claude Code
https://claude.ai/code/session_01QYY8Gp4v4x2J7iSNn1vZ57