Skip to content

fix(governance): lockfile-aware SHA-pin check for consumer repos - #574

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/governance-lint-lockfile-aware
Aug 4, 2026
Merged

fix(governance): lockfile-aware SHA-pin check for consumer repos#574
hyperpolymath merged 1 commit into
mainfrom
fix/governance-lint-lockfile-aware

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

The governance-reusable greps consumer workflows for 40-hex pins, which fail-flags every repo that adopted actions.lock (the locker's canonical output is tag-style refs with SHAs held in the lockfile the runner enforces). First seen on haec: after its lockfile revival (haec#46/#48/#50), Governance metadata was its only red — from this check.

With a lockfile present the step now verifies coverage via gh actions-lock --verify-local (offline) and keeps the inline-SHA demand only for cross-repo reusable workflow calls, which stay outside lockfile scope. Repos without a lockfile keep the exact old grep.

Downstream plan: once this merges, every open estate lockfile PR gets re-pinned to the resulting SHA so their Governance checks pass on first run.

🤖 Generated with Claude Code

…epos

Repos adopting GitHub's Actions dependency lockfile carry tag-style
uses: refs — the SHA authority lives in actions.lock, which the runner
enforces. The governance-reusable's bare 40-hex grep fail-flags every
such consumer (first seen: haec, whose Governance metadata check was
the sole red after its lockfile revival).

With actions.lock present: verify coverage via gh actions-lock
--verify-local and keep the inline-SHA requirement only for cross-repo
reusable workflow calls (lockfile does not scope those). Without a
lockfile: unchanged grep.

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

sonarqubecloud Bot commented Aug 4, 2026

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath merged commit bd0df9e into main Aug 4, 2026
31 of 34 checks passed
@hyperpolymath
hyperpolymath deleted the fix/governance-lint-lockfile-aware branch August 4, 2026 04:10
@gitar-bot

gitar-bot Bot commented Aug 4, 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

Makes the governance SHA-pin check lockfile-aware for consumer repos by verifying local coverage when an actions.lock file is present. 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 6 days — upgrade now to keep code review, CI analysis, auto-apply, custom automations, and more.

Was this helpful? React with 👍 / 👎 | Gitar

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