Skip to content

Commit d1423bb

Browse files
fix(ci): grant the secret-scanner reusable its required job permissions (#35)
Every `Secret Scanner` run in this repo has been ending in **`startup_failure`** — meaning **secret scanning has never actually executed** here. ### Root cause A called reusable workflow may only request permissions **equal to or more restrictive than its caller**. This is enforced when the workflow file is *parsed*, before a runner is allocated — which is why it surfaces as `startup_failure` with no logs and no annotations. `secret-scanner-reusable.yml` declares this on its `gitleaks` job: ```yaml permissions: contents: read pull-requests: write # gitleaks-action posts the PR summary comment actions: read # workflow-run metadata / PR-files endpoint ``` This caller granted only the file-level `permissions: contents: read`, so the reusable was asking for more than the caller had, and GitHub refused the run outright. ### Evidence Two repos pinning the **identical** reusable SHA behave differently based only on this block: `modshells` (grants the superset) runs fine; `verisimdb` (does not) `startup_failure`s. Estate-wide, 176 callers grant it and run; 26 did not and were all dead. ### Fix Grant the superset at job level — byte-identical to the canonical template. **No SHA pin is changed**, so the scanned content and supply-chain posture are unaffected; this only lets the existing pinned scanner start. ### Verified On `hyperpolymath/vext`, this exact change moved `Secret Scanner` from `startup_failure` to **success**, with `gitleaks`, `rust-secrets` and `shell-secrets` all allocating runners and passing. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 54bc1d9 commit d1423bb

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

.github/workflows/secret-scanner.yml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,12 @@ permissions:
1515

1616
jobs:
1717
scan:
18+
# The reusable's gitleaks job requests pull-requests: write (PR summary
19+
# comment) and actions: read (workflow-run metadata) at job level; the
20+
# caller must grant at least that or the run startup-fails.
21+
permissions:
22+
contents: read
23+
pull-requests: write
24+
actions: read
1825
uses: hyperpolymath/standards/.github/workflows/secret-scanner-reusable.yml@c65436ee3351cd6b0fa14b142938b195efc77586
19-
secrets: inherit
26+
secrets: inherit

0 commit comments

Comments
 (0)