Skip to content

fix(ci): reconcile the workflows with actions.lock (gh-actions-lock) - #111

Merged
hyperpolymath merged 2 commits into
mainfrom
fix/sha-pin-actions
Sep 20, 2026
Merged

hyperpolymath merged 2 commits into
mainfrom
fix/sha-pin-actions

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

fix(ci): reconcile the workflows with actions.lock (gh-actions-lock v0.1.6)

actions.lock is authoritative: the workflows carry readable refs and the lock records the
commit each ref resolves to, which is what actually runs. Refs that stop matching the manifest
make the whole repository unstartable — startup_failure, "Invalid lockfile".

Regenerated with the official extension (github/gh-actions-lock). The hand-pinned SHA refs are
reverted to their readable form here precisely because the lockfile, not the workflow, is what
pins them.

…0.1.6)

`actions.lock` is authoritative: the workflows carry readable refs and the lock records the
commit each ref resolves to, which is what actually runs. Refs that stop matching the manifest
make the whole repository unstartable — `startup_failure`, "Invalid lockfile".

Regenerated with the official extension (`github/gh-actions-lock`). The hand-pinned SHA refs are
reverted to their readable form here precisely because the lockfile, not the workflow, is what
pins them.
@coderabbitai

coderabbitai Bot commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • .github/workflows/actions.lock is excluded by !**/*.lock

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 76ac05ef-f0b0-4ba5-9a09-e4d730039b05

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: c2e4a255-a60b-4c10-aef6-0ce2d35e7a21

📥 Commits

Reviewing files that changed from the base of the PR and between 5b92153 and 5af3bc9.

⛔ Files ignored due to path filters (1)
  • .github/workflows/actions.lock is excluded by !**/*.lock
📒 Files selected for processing (28)
  • .github/workflows/boj-build.yml
  • .github/workflows/codeql.yml
  • .github/workflows/coordination-boundary.yml
  • .github/workflows/dependabot-automerge.yml
  • .github/workflows/dogfood-gate.yml
  • .github/workflows/e2e.yml
  • .github/workflows/governance.yml
  • .github/workflows/guix-nix-policy.yml
  • .github/workflows/hypatia-scan.yml
  • .github/workflows/instant-sync.yml
  • .github/workflows/label-triage.yml
  • .github/workflows/labels.yml
  • .github/workflows/mirror.yml
  • .github/workflows/openssf-compliance.yml
  • .github/workflows/pages.yml
  • .github/workflows/push-email-notify.yml
  • .github/workflows/quality.yml
  • .github/workflows/release.yml
  • .github/workflows/rhodibot.yml
  • .github/workflows/rsr-antipattern.yml
  • .github/workflows/runtime-policy.yml
  • .github/workflows/rust-ci.yml
  • .github/workflows/scorecard.yml
  • .github/workflows/secret-scanner.yml
  • .github/workflows/security-policy.yml
  • .github/workflows/static-analysis-gate.yml
  • .github/workflows/wellknown-enforcement.yml
  • .github/workflows/workflow-linter.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details
⏰ Context from checks skipped due to timeout. (38)
  • GitHub Check: governance / Exemption ratchet
  • GitHub Check: governance / Check Workflow Staleness
  • GitHub Check: governance / Debt ratchet
  • GitHub Check: governance / Live Actions policy (credentialed advisory)
  • GitHub Check: governance / Actions lockfile verify
  • GitHub Check: governance / Trusted-base reduction policy
  • GitHub Check: governance / Licence consistency
  • GitHub Check: governance / Well-Known (RFC 9116 + RSR)
  • GitHub Check: governance / Code quality + docs
  • GitHub Check: governance / Workflow security linter
  • GitHub Check: governance / Security policy checks
  • GitHub Check: governance / Allowlist Preflight
  • GitHub Check: governance / Language / package anti-pattern policy
  • GitHub Check: scan / shell-secrets
  • GitHub Check: scan / gitleaks
  • GitHub Check: governance / Guix packaging policy (Nix retired)
  • GitHub Check: scan / Hypatia Neurosymbolic Analysis
  • GitHub Check: scan / rust-secrets
  • GitHub Check: boundary
  • GitHub Check: lint
  • GitHub Check: docs
  • GitHub Check: Validate A2ML manifests
  • GitHub Check: Groove manifest check
  • GitHub Check: analyze (actions, none)
  • GitHub Check: Validate K9 contracts
  • GitHub Check: Empty-linter (invisible characters)
  • GitHub Check: lint-workflows
  • GitHub Check: check
  • GitHub Check: panic-attack assail
  • GitHub Check: check
  • GitHub Check: Validate eclexiaiser manifest
  • GitHub Check: antipattern-check
  • GitHub Check: E2E (coordination repo — nothing to exercise)
  • GitHub Check: Hypatia neurosymbolic scan
  • GitHub Check: Runtime Policy
  • GitHub Check: Patch Bridge CVE triage
  • GitHub Check: openssf-compliance
  • GitHub Check: lint-workflows
🧰 Additional context used
🪛 GitHub Check: SonarCloud Code Analysis
.github/workflows/release.yml

[failure] 144-144: Use full commit SHA hash for this dependency.

See more on https://sonarcloud.io/project/issues?id=hyperpolymath_nextgen-typing&issues=AaC8kqLO6TtzMQZH_t1_&open=AaC8kqLO6TtzMQZH_t1_&pullRequest=111

.github/workflows/quality.yml

[failure] 34-34: Use full commit SHA hash for this dependency.

See more on https://sonarcloud.io/project/issues?id=hyperpolymath_nextgen-typing&issues=AaC8kqNo6TtzMQZH_t2C&open=AaC8kqNo6TtzMQZH_t2C&pullRequest=111


[failure] 51-51: Use full commit SHA hash for this dependency.

See more on https://sonarcloud.io/project/issues?id=hyperpolymath_nextgen-typing&issues=AaC8kqNo6TtzMQZH_t2D&open=AaC8kqNo6TtzMQZH_t2D&pullRequest=111

.github/workflows/instant-sync.yml

[failure] 35-35: Use full commit SHA hash for this dependency.

See more on https://sonarcloud.io/project/issues?id=hyperpolymath_nextgen-typing&issues=AaC8kqJo6TtzMQZH_t1-&open=AaC8kqJo6TtzMQZH_t1-&pullRequest=111

.github/workflows/dependabot-automerge.yml

[failure] 60-60: Use full commit SHA hash for this dependency.

See more on https://sonarcloud.io/project/issues?id=hyperpolymath_nextgen-typing&issues=AaC8kqMy6TtzMQZH_t2B&open=AaC8kqMy6TtzMQZH_t2B&pullRequest=111

.github/workflows/static-analysis-gate.yml

[failure] 160-160: Use full commit SHA hash for this dependency.

See more on https://sonarcloud.io/project/issues?id=hyperpolymath_nextgen-typing&issues=AaC8kqL96TtzMQZH_t2A&open=AaC8kqL96TtzMQZH_t2A&pullRequest=111

🔇 Additional comments (2)
.github/workflows/dogfood-gate.yml (1)

30-30: LGTM!

Also applies to: 43-43, 75-75, 92-92, 125-125, 190-190, 249-249, 315-315

.github/workflows/rhodibot.yml (1)

37-37: LGTM!


📝 Summary

Summary by CodeRabbit

  • Chores
    • Updated workflow automation configuration across build, testing, security, policy, deployment and notification processes.
    • Added or standardised workflow management metadata.
    • Refreshed automation action version references while preserving existing workflow behaviour.
    • Strengthened release automation reproducibility by using immutable action references.
    • Cleaned up minor workflow formatting, with no changes to end-user functionality.

Walkthrough

The pull request updates GitHub Actions workflow headers and action references. Most workflows now use version tags instead of commit SHAs. The release workflow changes its references to immutable commit SHAs. One leading blank line was removed.

Changes

Workflow reference updates

Layer / File(s) Summary
Management headers and header normalisation
.github/workflows/{boj-build,codeql,coordination-boundary,dependabot-automerge,e2e,governance,guix-nix-policy,hypatia-scan,instant-sync,label-triage,labels,mirror,openssf-compliance,pages,push-email-notify,quality,release,rsr-antipattern,runtime-policy,rust-ci,scorecard,secret-scanner,security-policy,static-analysis-gate,wellknown-enforcement,workflow-linter}.yml
Management comments were added or duplicated. The leading blank line was removed from scorecard.yml.
Mutable action references
.github/workflows/{boj-build,codeql,coordination-boundary,dependabot-automerge,dogfood-gate,guix-nix-policy,instant-sync,openssf-compliance,pages,push-email-notify,quality,rhodibot,rsr-antipattern,runtime-policy,security-policy,static-analysis-gate,wellknown-enforcement,workflow-linter}.yml
Action references changed from commit SHAs to version tags or branch references. The stated action versions remain unchanged where specified.
Release workflow action pins
.github/workflows/release.yml
Five release action references changed from version tags to matching immutable commit SHAs. Version comments were added or retained.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~15 minutes

Change: Bug fix

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the purpose and main change, but it does not follow the required template. It omits the required Changes, RSR Quality Checklist, Testing, and Screenshots sections, including a… Add the required template sections. List the key changes under Changes, complete the RSR Quality Checklist, describe the testing performed under Testing, and add Screenshots output or state that screenshots are not applicable.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the CI workflow reconciliation with actions.lock and names the gh-actions-lock tool. It is concise and directly related to the main change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

The description explains the purpose and main change, but it does not follow the required template. It omits the required Changes, RSR Quality Checklist, Testing, and Screenshots sections, including all required checklist items.


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

A rabbit checks the workflow trail
Tags hop lightly through each file
Release pins settle firm and bright
Headers mark the paths tonight
One blank line slips from sight

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

coderabbitai[bot]
coderabbitai Bot previously approved these changes Sep 20, 2026
@hyperpolymath
hyperpolymath enabled auto-merge (squash) September 20, 2026 02:19
Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
@hyperpolymath
hyperpolymath merged commit 5c8e2cd into main Sep 20, 2026
40 of 41 checks passed
@hyperpolymath
hyperpolymath deleted the fix/sha-pin-actions branch September 20, 2026 02:21
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@@ -1,3 +1,4 @@
# This workflow is managed by gh actions-lock.
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