Skip to content

fix(ci): admit exact panic-attack subjects to lock recovery - #86

Merged
hyperpolymath merged 1 commit into
mainfrom
repair/actions-lock-recovery-panic-20261104
Nov 4, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
repair/actions-lock-recovery-panic-20261104

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Why

panic-attack PR258 repairs the scanner's input-coverage contract. Its hosted checks also exposed three inherited reusable callers that fail with zero jobs at an old standards owner/ref. Those callers now point to the reviewed canonical source, so their native actions.lock needs the existing official regeneration route.

The recovery runner currently admits four repositories. This adds only hyperpolymath/panic-attack as the fifth exact subject, following the merged #85 pattern.

Changes

  • Add the exact choice and inline admission case in actions-lock-recovery.yml.
  • Add the same exact repository to the generator's independent admission check.
  • Extend the real workflow-block and generator boundary tests for the new subject, wrong owner, suffixed name, legacy panic-attacker alias, and missing/floating/malformed revisions.
  • Update the four-to-five repository documentation.

The default and fixed push subject remain Hypatia. The canonical standards d3e4b485… regeneration function, verified official v0.2.0 binary identity, read-only token, separated acquisition job and candidate-only output remain unchanged. No subject code is executed and no writeback or Actions-policy permission is added.

Validation

From current main b81ec53c8207bc5b8a8bd8d828834491e36f745a:

  • bash tests/actions-lock-recovery.sh: 29 boundary groups passed (was 27; both the workflow and generator now admit the fifth exact subject).
  • actionlint on the modified workflow: passed.
  • Bash syntax and whitespace checks: passed.
  • Exactly four changed files; no action refs, lockfiles, tool pins or security permissions changed.

These controls are not evidence that the real generator has run on panic-attack. After review/merge, dispatch against the then-current immutable PR258 head, verify the candidate receipt and digests, apply only with an expected-head check, and require genuine target startup/governance results. The current proposed subject is ff278a373b523c69f93e99590112f5fac205b083; if it moves, use the newly reviewed exact head.

Add panic-attack as the fifth explicit repository in both the dispatch
and generator admission boundaries. Keep the fixed Hypatia push subject,
read-only token, canonical tool/source pins and candidate-only behavior.

Extend wrong-owner, suffix, legacy-alias and malformed-revision controls;
all 29 local boundary groups and workflow actionlint pass.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 4, 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-11-04T20:15:49.349690Z e10a33e PR opened
ℹ️ 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.

@coderabbitai

coderabbitai Bot commented Nov 4, 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 11 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
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: acf590ca-cb40-4b2d-ba42-5d6ed0269141
Commits

Reviewing files that changed from the base of the PR and between b81ec53 and e10a33e.

Files selected for processing: 4
  • .github/workflows/actions-lock-recovery.yml
  • docs/ACTIONS-LOCK-RECOVERY.adoc
  • scripts/generate-lock-recovery-candidate.sh
  • tests/actions-lock-recovery.sh
  • 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

@sonarqubecloud

sonarqubecloud Bot commented Nov 4, 2026

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath merged commit b931e8e into main Nov 4, 2026
20 checks passed
@hyperpolymath
hyperpolymath deleted the repair/actions-lock-recovery-panic-20261104 branch November 4, 2026 20:17
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