Skip to content

Auto-approve and merge non-major Dependabot PRs - #53

Merged
proxymesh merged 4 commits into
mainfrom
cursor/dependabot-auto-approve-merge-c08c
Sep 3, 2026
Merged

proxymesh merged 4 commits into
mainfrom
cursor/dependabot-auto-approve-merge-c08c

Conversation

@proxymeshai

@proxymeshai proxymeshai commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

Dependabot PRs were enabling auto-merge but staying BLOCKED with REVIEW_REQUIRED because branch protection needs an approving review.

This updates dependabot-auto-merge to:

  • Auto-approve non-major Dependabot PRs
  • Enable squash auto-merge for those PRs
  • Match on PR author (dependabot[bot]) instead of github.actor, so branch updates still trigger the workflow
  • Leave major updates for human review (same policy as Dependabot grouping)
  • Restrict to same-repo heads and explicit pull_request_target types

Bugbot follow-up

Fail closed when update-type is empty/unknown (can happen after human branch updates if commit verification fails). Use skip-commit-verification so metadata still parses after merge commits, and only auto-approve/merge explicit semver-minor / semver-patch updates.

Repo setting to confirm

Settings → Actions → General → Allow GitHub Actions to create and approve pull requests must be enabled, or the approve step will fail.

After merge

Update open Dependabot PR branches (or @dependabot rebase) so they re-run against the new workflow on main.

Slack Thread

Open in Web Open in Cursor 

Branch protection requires reviews, so enabling auto-merge alone left
Dependabot PRs blocked. Restore approval, keep major updates for human
review, and match on the PR author so branch updates still trigger.

Co-authored-by: ProxyMesh AI <proxymeshai@users.noreply.github.com>
@proxymeshai
proxymeshai marked this pull request as ready for review September 3, 2026 22:21

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 0a13c4a. Configure here.

Comment thread .github/workflows/dependabot-auto-merge.yml Outdated
After a human branch update, fetch-metadata can leave update-type empty
if commit verification fails, and != semver-major would still approve.
Skip commit verification for merge commits, and only auto-approve/merge
explicit minor or patch updates.

Co-authored-by: ProxyMesh AI <proxymeshai@users.noreply.github.com>
@cursor

cursor Bot commented Sep 3, 2026

Copy link
Copy Markdown

cursor review

Cursor and others added 2 commits September 3, 2026 22:48
Co-authored-by: ProxyMesh AI <proxymeshai@users.noreply.github.com>
Restrict pull_request_target to opened/reopened/synchronize, require the
head repo to match this repository, and note that steps only call the
GitHub API (no PR code checkout).

Co-authored-by: ProxyMesh AI <proxymeshai@users.noreply.github.com>
@cursor

cursor Bot commented Sep 3, 2026

Copy link
Copy Markdown

bugbot run

@proxymesh
proxymesh merged commit af32924 into main Sep 3, 2026
9 checks passed
@proxymesh
proxymesh deleted the cursor/dependabot-auto-approve-merge-c08c branch September 3, 2026 23:32
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.

2 participants