Repository navigation
fix(ci): move the scanner pin to hypatia main 23dc74d - #951
Conversation
The governance and hypatia-scan callers pass hypatia-ref aa34a4c (#940), which predates the rule fixes that clear this repository's own false positives: - #942: WH013 judges writes per job and skips PAT and app-token writes (estate-rescan, inbox-steward-intake). - #947: RE005 skips trap handlers, and job extraction survives comments between jobs. Moving the pin to main 23dc74d puts them in the baseline gate. The jump also brings #931's new rules (WF026, WF027, PIN-003); this PR's governance run is the first measurement of them on this repository. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CYR5bf5FBLS7cXQK72aVNM
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 7 minutes. View limit details
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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
`governance / Actions lockfile verify` fails on this PR, and on main since #945. Of the verifier's findings, only the three `step-security/harden-runner@v2.22.2` "not-pinned" records are not accepted by standards' wrapper. #948 removes those steps. This is the same three-file change; it no-ops once #948 (or a lock regeneration) lands on main. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CYR5bf5FBLS7cXQK72aVNM
Summary
This is the last step of the baseline-gate stack.
governance.ymlandhypatia-scan.ymlpasshypatia-ref: aa34a4c(#940), which predates the rule fixes that clear this repo's own false positives. This PR moves both to hypatiamainat23dc74d, which contains:aa34a4c);estate-rescan.yml;inbox-steward-intake.ymlwas also repaired in fix(ci): inbox-steward-intake publishes through PRs and fails loudly #943;traphandlers, and job extraction survives comments between jobs.uses:refs are unchanged, soactions.lockis untouched.What this PR's governance run will show
This PR's own
governance / Validate Hypatia Baselinerun is the measurement: it builds the scanner at23dc74d. The jump fromaa34a4calso brings #931's new rules (WF026, WF027, PIN-003), which haven't been measured on this repo yet. hex.pm is blocked in my session, so I can't build the escript to pre-run them.Expected remaining findings, by class:
invalid_actions_lock: expected 0.I'll post the measured list here once the run completes.
Type of change
How has this been verified?
actionlint1.7.7 is clean on both files.23dc74disorigin/mainat the time of writing.git log aa34a4c..23dc74dcontains fix(rules): judge WH013 per job, and skip writes made with their own credential #942, fix(rules): close four gaps Codex found in #942 after it merged #947, fix(ci): pin test service and job containers by digest (RE004) #945, feat(rules): detect the three workflow-pin defect classes behind the estate's dead Rust CI #931, test(actions-lock): closure contract corpus shared with standards #941 and Claude/new session th4e9w lockregen #936.Checklist
Generated by Claude Code