Skip to content

fix(ci): grant scorecard caller the reusable's job-level permissions - #343

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/scorecard-caller-perms
Aug 4, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
fix/scorecard-caller-perms

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Completes #342: scorecard-reusable's job requests security-events: write + id-token: write beyond the workflow-level actions: read/contents: read; caller grants must cover job-level requests too, so OSSF Scorecard was the one re-pinned caller still dying at startup. One file, four permission lines.

🤖 Generated with Claude Code

scorecard-reusable's job (not just its workflow header) requests
security-events: write and id-token: write; caller-side permission
validation applies at job level too, so OSSF Scorecard remained
startup_failure after #342 fixed the workflow-level grants.

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

Grants the scorecard caller the reusable's job-level permissions to fix startup failures. 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

@gitar-bot
gitar-bot Bot enabled auto-merge (squash) August 4, 2026 04:15

@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 4, 2026 21:55
@hyperpolymath
hyperpolymath merged commit a27cd3e into main Aug 4, 2026
49 of 55 checks passed
@hyperpolymath
hyperpolymath deleted the fix/scorecard-caller-perms branch August 4, 2026 21:55
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