Skip to content

fix(ci): dependabot auto-merge — block grouped major bumps#22

Merged
Exploitacious merged 1 commit intomainfrom
fix/dependabot-major-bump-guard
May 4, 2026
Merged

fix(ci): dependabot auto-merge — block grouped major bumps#22
Exploitacious merged 1 commit intomainfrom
fix/dependabot-major-bump-guard

Conversation

@Exploitacious
Copy link
Copy Markdown
Owner

Why

Old check on steps.meta.outputs.update-type only reflected the first dependency in a grouped PR. A grouped PR with one minor dep + several major-bump deps auto-merged unattended.

Incident 2026-05-04: Operational-Dashboard PR #7 (grouped runtime-deps) included astro 5→6, tailwind 3→4, TS 5→6. Auto-merged. CF Workers Build rejected the resulting peer-dep tree.

Fix

Adds steps.meta.outputs.dependency-major-versions-changed == '' check — non-empty when ANY dep in a group has a major bump. Major-bump PRs get labeled needs-review,major-version.

Bumped fetch-metadata@v2 → v3 if not already.

Test plan

  • Next dependabot PR with patch/minor only auto-merges as before
  • Next dependabot grouped PR containing a major bump does NOT auto-merge, gets labeled needs-review,major-version

Old check evaluated steps.meta.outputs.update-type which only reflects
the first dependency in a grouped PR. A grouped PR with one minor dep
+ several major-bump deps would auto-merge unattended.

Incident 2026-05-04: Operational-Dashboard PR #7 (grouped runtime-deps)
included astro 5→6, tailwind 3→4, TS 5→6. Auto-merged. CF Workers Build
rejected the resulting peer-dep tree, dashboard.umbrellaitgroup.com
went down.

New check additionally requires
  steps.meta.outputs.dependency-major-versions-changed == ''
which is non-empty when ANY dep in the group has a major bump.
Major-bump PRs get labeled 'needs-review,major-version' so they
don't sit silently.

Bumps fetch-metadata to v3 if not already.
@Exploitacious Exploitacious merged commit afea624 into main May 4, 2026
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