Skip to content

fix(ci): replace dependabot-auto-merge workflow with Mergify rule#1310

Open
ambient-code[bot] wants to merge 1 commit intomainfrom
mergify-dependabot-auto-merge
Open

fix(ci): replace dependabot-auto-merge workflow with Mergify rule#1310
ambient-code[bot] wants to merge 1 commit intomainfrom
mergify-dependabot-auto-merge

Conversation

@ambient-code
Copy link
Copy Markdown
Contributor

@ambient-code ambient-code bot commented Apr 14, 2026

Summary

  • Deletes .github/workflows/dependabot-auto-merge.yml — the only workflow using the security-sensitive pull_request_target trigger
  • Adds a Mergify pull_request_rule to auto-approve Dependabot PRs via the native review action
  • Existing Mergify queue rules already handle merging once CI checks pass and an approval is present

Why

The pull_request_target trigger grants write permissions to forked PRs, making it a known attack vector. Mergify's review action provides the same auto-approve + auto-merge functionality without this risk.

Reference: Mergify docs — Approve Dependabot Pull Requests

Closes #862

Test plan

  • Verify Mergify config is valid (Mergify validates on push)
  • Confirm next Dependabot PR gets auto-approved by Mergify
  • Confirm auto-approved Dependabot PR enters the merge queue and merges after CI passes

🤖 Ambient Session

Remove the `pull_request_target`-based GitHub Actions workflow for
auto-merging Dependabot PRs and replace it with a Mergify
pull_request_rule that auto-approves Dependabot PRs. The existing
Mergify queue rules handle merging once CI checks pass.

This eliminates the security-sensitive `pull_request_target` trigger
while providing the same functionality through Mergify's native
review action.

Closes #862

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ambient-code ambient-code bot added the ambient-code:managed PR managed by AI automation label Apr 14, 2026
actions:
review:
type: APPROVE
message: Automatically approved by Mergify
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ambient-code this will be noisy. Don't comment on every PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ambient-code:managed PR managed by AI automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

use mergify to auto-merge dependabot changes

1 participant