Skip to content

fix(ci): grant secret-scanner caller the permissions its reusable requests - #63

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/secret-scanner-permissions
Jul 2, 2026
Merged

fix(ci): grant secret-scanner caller the permissions its reusable requests#63
hyperpolymath merged 1 commit into
mainfrom
fix/secret-scanner-permissions

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Secret Scanner was the last startup_failure on main: the gitleaks job in secret-scanner-reusable requests pull-requests: write + actions: read, and a called reusable workflow cannot escalate past the caller's grant. Same mechanism as the scorecard fix in #61 — this caller was missed because it wasn't among the sampled failing runs. One-line permissions grant.

🤖 Generated with Claude Code

…uests

The gitleaks job inside standards' secret-scanner-reusable requests
pull-requests: write + actions: read; a called reusable workflow cannot
escalate past the caller's grant, so the whole run died at startup
(same class as the scorecard fix in #61, missed for this workflow).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@hyperpolymath
hyperpolymath enabled auto-merge (squash) July 2, 2026 04:11
@hyperpolymath
hyperpolymath merged commit 7706b09 into main Jul 2, 2026
24 of 26 checks passed
@hyperpolymath
hyperpolymath deleted the fix/secret-scanner-permissions branch July 2, 2026 04:12
hyperpolymath added a commit that referenced this pull request Jul 2, 2026
…#64)

Last red on Secret Scanner after #63: the SHA-pinned trufflehog action
(v3.95.6) injects `--fail` internally, so the repo job's `extra_args:
--only-verified --fail` dies with `flag 'fail' cannot be repeated` (the
old unpinned `@main` predated the injection). `--results=verified`
replaces the deprecated `--only-verified`; fail-on-verified-findings
behavior is unchanged.

🤖 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