Skip to content

ci: annotate the lockfix permissions with sentinel accepts - #144

Merged
bgard68 merged 1 commit into
mainfrom
claude/new-session-muubr7
Aug 25, 2026
Merged

ci: annotate the lockfix permissions with sentinel accepts#144
bgard68 merged 1 commit into
mainfrom
claude/new-session-muubr7

Conversation

@bgard68

@bgard68 bgard68 commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Adds the sentinel:accept GHA002 markers to the two permission lines in dependabot-lockfix.yml, with the reason on the line the scanner flags:

permissions:
  contents: write  # sentinel:accept GHA002 - pushes the regenerated lock files to the Dependabot branch
  actions: write   # sentinel:accept GHA002 - approves the PR checks GitHub creates after the bot push

Each scope is used by exactly one step — contents: write by "Push the fix if anything changed" (git push origin HEAD:$BRANCH), actions: write by "Approve the pull-request checks on the fixed commit" (POST .../actions/runs/{id}/approve) — so there is nothing for GHA002 to act on beyond recording that.

This returns the declaration to the workflow level so the markers sit on the lines GHA002 reports, and drops the job-level block added in #142, so there is one place to read the answer instead of two.

Effective permissions are unchanged. The file has a single job, which now inherits exactly what it previously overrode with:

before  workflow: {contents: read}                    job: {contents: write, actions: write}
after   workflow: {contents: write, actions: write}   job: inherits

The one thing lost is the "a second job added here starts read-only" property from #142. That only matters if this file ever gains another job, and the markers are worth more than a hypothetical.

cleanup-runs.yml still carries its actions: write at job level with no marker — say the word and it gets the same treatment on both main and dapper.

🤖 Generated with Claude Code

https://claude.ai/code/session_019h7znwAftvs36vD4YwKMRf


Generated by Claude Code

Both scopes are used by exactly one step each — contents: write by "Push the
fix if anything changed" and actions: write by "Approve the pull-request checks
on the fixed commit" — so GHA002 has nothing to act on. The sentinel:accept
markers say that to the scanner in its own syntax, with the reason on the line
it flags.

Returns the declaration to the workflow level, where the markers sit on the
lines GHA002 reports, and drops the job-level block added in #142 so there is
one place to read the answer instead of two. Effective permissions are
unchanged: the file has a single job, which inherits exactly what it used to
override with.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019h7znwAftvs36vD4YwKMRf
@bgard68
bgard68 merged commit f00e7fc into main Aug 25, 2026
10 checks passed
@bgard68
bgard68 deleted the claude/new-session-muubr7 branch August 25, 2026 06:28
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