Skip to content

fix(ci): move the scanner pin to hypatia main 23dc74d - #951

Merged
hyperpolymath merged 5 commits into
mainfrom
claude/eager-wright-zwwbe2-pin-bump
Oct 20, 2026
Merged

hyperpolymath merged 5 commits into
mainfrom
claude/eager-wright-zwwbe2-pin-bump

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Summary

This is the last step of the baseline-gate stack. governance.yml and hypatia-scan.yml pass hypatia-ref: aa34a4c (#940), which predates the rule fixes that clear this repo's own false positives. This PR moves both to hypatia main at 23dc74d, which contains:

uses: refs are unchanged, so actions.lock is untouched.

What this PR's governance run will show

This PR's own governance / Validate Hypatia Baseline run is the measurement: it builds the scanner at 23dc74d. The jump from aa34a4c also 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:

  • RE001 (harden-runner): most of the original 18 jobs, including the 2 dispatched label workflows, which must stay action-free.
  • WH013, RE004, RE005, invalid_actions_lock: expected 0.

I'll post the measured list here once the run completes.

Type of change

  • 🔧 Build / CI / tooling

How has this been verified?

Checklist

  • My commits are signed. The session's signing shim is platform-mediated.
  • I ran the project's own checks/tests locally and they pass.
  • New files carry the correct SPDX (none added).
  • Docs are updated (the pin comment names what it carries).
  • I have not introduced a soundness hole.

Generated by Claude Code

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
@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You'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.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: fa0e27ef-0870-4a11-a73a-1ece5347e75a
📥 Commits

Reviewing files that changed from the base of the PR and between 0a44921 and 9f0937d.

📒 Files selected for processing: 2
  • .github/workflows/governance.yml
  • .github/workflows/hypatia-scan.yml
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hyperpolymath
hyperpolymath marked this pull request as ready for review October 20, 2026 14:08
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-20T14:47:34.457731Z 9f0937d New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

claude and others added 2 commits October 20, 2026 14:14
`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
@hyperpolymath
hyperpolymath enabled auto-merge (squash) October 20, 2026 14:27
@hyperpolymath
hyperpolymath merged commit 6e24760 into main Oct 20, 2026
41 checks passed
@hyperpolymath
hyperpolymath deleted the claude/eager-wright-zwwbe2-pin-bump branch October 20, 2026 14:44
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.

2 participants