Skip to content

fix(ci): regenerate actions.lock — workflow refs had drifted ahead of it - #49

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/regenerate-actions-lock
Aug 7, 2026
Merged

fix(ci): regenerate actions.lock — workflow refs had drifted ahead of it#49
hyperpolymath merged 1 commit into
mainfrom
fix/regenerate-actions-lock

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Workflows reference action versions this lockfile does not pin. Actions were
bumped in the workflows and gh actions-lock was never re-run, so the lockfile
still records the older ref — and lockfile enforcement rejects a workflow whose
refs it cannot resolve, before any job starts.

Regenerated with gh actions-lock (v0.1.6) rather than hand-edited. Every
entry needs a resolved commit SHA plus owner_id and repo_id, and hand-authoring
those is exactly how the sibling defect happened — a caller entry written into
the dependencies: map instead of workflows:, which made the whole file
unparseable and killed every workflow in the repo at once.

Verified before pushing, three ways: the hand-authored reusable-caller entries
survive the regeneration (checked by count, because the tool does not generate
them and deleting them would re-break the repo); every key under
dependencies: still matches owner/repo@ref; and gh actions-lock --no-fix
passes its own verification afterwards.

One of 26 repos of 52 carrying a lockfile — half the estate — found by scanning
after metadatastician/688-attack-hub#18.

Co-Authored-By: Claude Opus 5 noreply@anthropic.com

Workflows reference action versions this lockfile does not pin. Actions were
bumped in the workflows and `gh actions-lock` was never re-run, so the lockfile
still records the older ref — and lockfile enforcement rejects a workflow whose
refs it cannot resolve, before any job starts.

Regenerated with `gh actions-lock` (v0.1.6) rather than hand-edited. Every
entry needs a resolved commit SHA plus owner_id and repo_id, and hand-authoring
those is exactly how the sibling defect happened — a caller entry written into
the `dependencies:` map instead of `workflows:`, which made the whole file
unparseable and killed every workflow in the repo at once.

Verified before pushing, three ways: the hand-authored reusable-caller entries
survive the regeneration (checked by count, because the tool does not generate
them and deleting them would re-break the repo); every key under
`dependencies:` still matches owner/repo@ref; and `gh actions-lock --no-fix`
passes its own verification afterwards.

One of 26 repos of 52 carrying a lockfile — half the estate — found by scanning
after metadatastician/688-attack-hub#18.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

@gitar-bot

gitar-bot Bot commented Aug 7, 2026

Copy link
Copy Markdown

Note

Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime.
Learn more

Code Review ✅ Approved

Regenerated the actions.lock file to align with updated workflow action references, resolving the lockfile enforcement check failure. No issues found.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Important

Your trial ends in 3 days — upgrade now to keep code review, CI analysis, auto-apply, custom automations, and more.

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot
gitar-bot Bot enabled auto-merge (squash) August 7, 2026 13:23
@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Aug 7, 2026

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR and enabled auto-merge (configure)

@hyperpolymath
hyperpolymath disabled auto-merge August 7, 2026 15:09
@hyperpolymath
hyperpolymath merged commit 59d90fe into main Aug 7, 2026
44 of 46 checks passed
@hyperpolymath
hyperpolymath deleted the fix/regenerate-actions-lock branch August 7, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant