Skip to content

fix(ci): re-pin standards reusables past the lockfile boundary - #48

Merged
hyperpolymath merged 1 commit into
mainfrom
ci/repin-standards-lockfile
Aug 3, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
ci/repin-standards-lockfile

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Completes the revival begun in #46: the five standards-reusable callers (Governance, Hypatia scan, Secret scan, Mirror, Scorecard) were still startup_failure because enforcement requires a lockfile in the called repo at the pinned ref, and d135b05 predates standards' actions.lock (standards#570). This re-pins all five to 5a597720 and aligns caller permission grants with the reusables' current demands (actions: read added; scorecard's job-level block carries the full union since job blocks replace top-level).

If these five execute on this PR (or on main post-merge for the push-only ones), haec is the first estate repo fully through both lockfile stages — proving the consumer half of the template.

🤖 Generated with Claude Code

…n caller perms

The five standards-reusable callers were the last startup_failures:
enforcement demands a lockfile in the CALLED repo at the pinned ref, and
d135b05 predates standards' actions.lock (#570). Re-pin all five to
5a597720 (standards main, lockfile present).

The 186-commit jump also raised the reusables' permission demands —
governance/hypatia/mirror/scorecard now declare `actions: read`, and a
reusable requesting more than its caller grants is itself a
startup_failure. Callers now grant the exact union each reusable
requests (job-level blocks replace, not merge, so scorecard's job block
carries the full set).

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

sonarqubecloud Bot commented Aug 3, 2026

Copy link
Copy Markdown

@gitar-bot

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

Re-pins standards reusables past the lockfile boundary and aligns caller permissions to satisfy current workflow demands. 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 7 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 3, 2026 18:51

@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 3, 2026 18:51
@hyperpolymath
hyperpolymath merged commit a546fad into main Aug 3, 2026
21 checks passed
@hyperpolymath
hyperpolymath deleted the ci/repin-standards-lockfile branch August 3, 2026 18:51
hyperpolymath added a commit that referenced this pull request Aug 4, 2026
…etes #48) (#50)

#48 was merged before this branch's final two commits, so main still
lacks the pieces that actually revive the five reusable callers:

1. **Hand-added lockfile entries** for the reusable-caller workflows
(`governance`/`hypatia-scan`/`mirror`/`scorecard`/`secret-scanner`:
`[]`) — `gh actions-lock` v0.1.6 skips reusable-only workflows but
enforcement still rejects them by name ("The following workflows are
missing a lockfile: … hypatia-scan.yml").
2. **Re-pin `5a59772` → `fcb8669`** — standards #571/#572 landed after
the first re-pin, restoring its gates; `fcb8669` verified via the
commits API.

This PR's own wave is the decisive consumer-half test.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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