Skip to content

chore(ci): harden Dependabot auto-merge workflow#1189

Merged
bagelbits merged 1 commit into
mainfrom
chore/harden-dependabot-auto-merge
Jul 20, 2026
Merged

chore(ci): harden Dependabot auto-merge workflow#1189
bagelbits merged 1 commit into
mainfrom
chore/harden-dependabot-auto-merge

Conversation

@bagelbits

Copy link
Copy Markdown
Collaborator

Follow-up hardening for the auto-merge workflow added in #1188, from a security review.

Changes

  • Gate on PR author, not triggerer. github.actor reflects whoever triggered the latest event (e.g. a re-run), not the PR author. Switch to github.event.pull_request.user.login == 'dependabot[bot]' so the job only runs for PRs actually opened by Dependabot.
  • Pin dependabot/fetch-metadata to a commit SHA (25dd0e3 # v3.1.0) instead of the mutable @v3 tag, closing the supply-chain window on that step.

Not changed (by design)

  • Auto-approve/merge itself: intended feature. Code-owner review gate stays enforced (PAT approves as the owner), CI checks still required, major bumps still excluded.

🤖 Generated with Claude Code

Gate on the PR author (pull_request.user.login) instead of github.actor,
which reflects the triggering actor and not the author. Pin
dependabot/fetch-metadata to a commit SHA.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@bagelbits
bagelbits merged commit b755925 into main Jul 20, 2026
5 checks passed
@bagelbits
bagelbits deleted the chore/harden-dependabot-auto-merge branch July 20, 2026 23:03
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