Group Dependabot security updates into a single pull request - #6126
Open
j0ntz wants to merge 1 commit into
Open
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
peachbits
approved these changes
Aug 3, 2026
j0ntz
force-pushed
the
jon/dependabot-group-security-updates
branch
from
August 3, 2026 21:23
fe5c19b to
45e59ed
Compare
j0ntz
enabled auto-merge
August 3, 2026 21:24
j0ntz
force-pushed
the
jon/dependabot-group-security-updates
branch
from
August 3, 2026 21:25
45e59ed to
b946e7f
Compare
j0ntz
disabled auto-merge
August 3, 2026 21:25
j0ntz
enabled auto-merge
August 3, 2026 21:25
j0ntz
disabled auto-merge
August 3, 2026 21:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CHANGELOG
Does this branch warrant an entry to the CHANGELOG?
Dependencies
none
Requirements
If you have made any visual changes to the GUI. Make sure you have:
Description
Collapses Dependabot's security update PRs into a single grouped PR. We currently have 27 open, going back to March 2023.
Why it got this bad: this repo has no
.github/dependabot.ymlat all, so every Dependabot PR here is an alert-driven security update, and security updates are explicitly exempt fromopen-pull-requests-limit. There was no setting to turn down without adding a config file first.What the config does:
groups.all-security-updateswithapplies-to: security-updatesandpatterns: ['*']puts every security fix in one PR. Theapplies-tokey is load-bearing: without it, grouping defaults to version updates only and would change nothing here.open-pull-requests-limit: 0keeps version updates off, so adding this file does not trade 27 security PRs for a larger firehose of routine version bumps. GitHub documents this value for the security-updates-only case.schedule.intervalis required syntax but inert. Security updates fire on new advisories, not on an interval, and there are no version updates left for it to govern.What to expect after merge:
Per GitHub's docs, "When grouped security updates are first enabled, Dependabot will immediately try to create grouped pull requests. You may notice Dependabot closing old pull requests and opening new ones." So this should not wait for a scheduled tick.
Dependabot closes an old PR once the new grouped PR covers the same dependency. Anything it cannot fit into the group, most likely the transitive bumps blocked by a pinned parent, will stay open and need closing by hand. Use a plain close for those, not
@dependabot close, which suppresses recreation and would drop the dependency from future grouped PRs.Steady state after this is at most one open Dependabot PR. New advisories fold into that PR while it is open rather than opening another, so it is worth merging promptly instead of letting it sit and rebase underneath us.
Reference: Dependabot options reference.
Note
Low Risk
CI-only Dependabot configuration with no application runtime or dependency manifest changes in this diff.
Overview
Adds
.github/dependabot.ymlso npm security fixes land in one grouped PR instead of one per advisory (the repo had no config before, which left many alert-driven security PRs open).The config sets
groups.all-security-updateswithapplies-to: security-updatesandpatterns: ['*']so grouping applies to security updates (not only version bumps).open-pull-requests-limit: 0turns off routine version-update PRs while security updates still flow.schedule.interval: weeklyis required but does not drive security timing.After merge, Dependabot may close older security PRs and open a single grouped one; steady state should be at most one open Dependabot PR for npm at the repo root.
Reviewed by Cursor Bugbot for commit b946e7f. Bugbot is set up for automated code reviews on this repo. Configure here.