ci(deps): group all Bun updates, ignore majors, pin vue-router#58
Merged
Conversation
Collapse prod+dev bumps into a single weekly group PR and stop raising major-version updates (security updates still apply). This is what would have kept the recent batch to one PR instead of eight.
apps/ui pinned vue-router ^4.5.1, but nuxt@4.x depends on vue-router ^5.1.0 — the pin forced a second, unused v4 copy alongside Nuxt's v5. Pin ^5.1.0 to match Nuxt (now safe from surprise majors via dependabot's ignore rule). Nuxt typecheck passes.
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.
Why
Dependabot opened 8 PRs in the last batch. Two structural problems caused it:
/apps/*,/packages/*) on a single-lockfile Bun workspace produced isolatedpackage.jsonedits that left the shared rootbun.lockstale →bun install --frozen-lockfilefailed, and duplicated bumps the root scan already makes. (Already fixed onmaininb635516— scan only/.)This branch finishes the cleanup.
Changes
dependabot.yml— collapse prod+dev into a singleall-dependenciesgroup (one weekly PR for the whole workspace), andignoreall major-version bumps. Security updates still apply; majors get handled deliberately by hand.apps/uivue-router pin — was^4.5.1, butnuxt@4.xdepends onvue-router ^5.1.0. The stale pin forced a second, unused v4 copy alongside Nuxt's v5. Pin^5.1.0to match Nuxt (now safe from surprise majors via the ignore rule). Lockfile regenerated → singlevue-router@5.1.0, no v4 anywhere.Verification
bun install --frozen-lockfile— cleannuxt typecheck(apps/ui) — exits 0, zero errorsFollow-ups (not in this PR)
github-actionsDependabot block left untouched (already one grouped PR; its majors are SHA-pinned bumps you generally want).Greptile Summary
This PR tightens dependency update handling for the Bun workspace. The main changes are:
apps/uialigned to Nuxt'svue-router ^5.1.0dependency.Confidence Score: 5/5
This looks safe to merge.
Reviews (1): Last reviewed commit: "chore(ui): align vue-router with Nuxt's ..." | Re-trigger Greptile
Context used (5)