Skip to content

fix(ci): repin Hypatia wrapper past the warn-severity validator bug - #72

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/hypatia-pin-warn-severity
Sep 21, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
fix/hypatia-pin-warn-severity

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

The Hypatia job on main fails inside the reusable's findings validator:

::error::Hypatia did not produce one valid findings array   (exit 2)

Cause: the pinned reusable revision (fad242d35291…) accepts only critical|high|medium|low|info|informational, but the scanner emits warn for its research rules — defiant's scan produced 67 findings, 12 of them warn.

Standards fixed the vocabulary in 591cea1b07 (2026-09-10, "reconcile native action pins and retain Hypatia warning severity"). This repins the wrapper to standards main head 9379fb71…, which carries that fix plus the cache-integrity/SARIF follow-ons.

The published check name is unchanged: caller job id hypatia + reusable job name Hypatia Neurosymbolic Analysis at both pins, so the required context hypatia / Hypatia Neurosymbolic Analysis still matches.

@hyperpolymath
hyperpolymath merged commit 0bad773 into main Sep 21, 2026
1 check failed
@hyperpolymath
hyperpolymath deleted the fix/hypatia-pin-warn-severity branch September 21, 2026 10:44
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: b23bd752-a8ed-4eac-8f39-c721af4d5a16

📥 Commits

Reviewing files that changed from the base of the PR and between 1b136e1 and bd73489.

⛔ Files ignored due to path filters (1)
  • .github/workflows/actions.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • .github/workflows/hypatia-scan.yml
 ______________________________________________________________
< Security by obscurity? I can see right through the curtains. >
 --------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ

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 added a commit that referenced this pull request Sep 21, 2026
…ck (#73)

The Hypatia repin (#72) regenerated the lockfile with a helper of mine
that wrote the 8 transitive pins in the wrong shape: `ref` + `commit`
but no `owner_id`/`repo_id`, which the validator requires.

lockfile is unreadable: .github/workflows/actions.lock: line 139, column
5: missing required action field "owner_id" for dependency
"actions/cache@55cc8345..."

This adds the 8 missing pairs, fetched from the API. `gh actions-lock
--verify` now passes locally (rc=0, 13 workflows scanned). The helper
has been fixed so it cannot emit this shape again.
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.

1 participant