ci(scorecard): add job-level permissions for reusable workflow - #61
Conversation
The `scorecard-reusable.yml` reusable requires the calling `analysis` job to declare `security-events: write` and `id-token: write` — called-workflow permissions are CAPPED by the caller's block (the reusable docstring states this explicitly). Without this, every Scorecard run silently fails with `startup_failure` because ossf/scorecard-action cannot upload SARIF. Estate-wide sweep tracked at hyperpolymath/standards#282; same pattern as julia-professional-registry#19 (2026-05-27) and absolute-zero#68 (2026-05-30). Refs hyperpolymath/standards#282 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
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. Code Review ✅ ApprovedAdds job-level permissions for security events and ID tokens to the Scorecard reusable workflow to prevent silent SARIF upload failures. No issues found.
OptionsDisplay: compact → Showing less information. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
|
|
![gitar-bot[bot]](https://avatars.githubusercontent.com/in/827041?s=60&v=4){kind=small}
Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
1 participant
The
scorecard-reusable.ymlreusable requires the callinganalysisjob to declaresecurity-events: writeandid-token: write— called-workflow permissions are CAPPED by the caller's block (the reusable docstring states this explicitly).Without this, every Scorecard run silently fails with
startup_failurebecause ossf/scorecard-action cannot upload SARIF.Estate-wide sweep tracked at hyperpolymath/standards#282; same pattern as julia-professional-registry#19 (2026-05-27) and absolute-zero#68 (2026-05-30).
Refs hyperpolymath/standards#282
Summary
Closes #
Type of change
How has this been verified?
Checklist
git commit -S).SPDX-License-Identifier(code/configMPL-2.0,prose
CC-BY-SA-4.0); I did not relicense existing files.Notes for reviewers