Skip to content

ci(deps): group all Bun updates, ignore majors, pin vue-router#58

Merged
roziscoding merged 2 commits into
mainfrom
chore/dependabot-grouping
Jun 28, 2026
Merged

ci(deps): group all Bun updates, ignore majors, pin vue-router#58
roziscoding merged 2 commits into
mainfrom
chore/dependabot-grouping

Conversation

@roziscoding

@roziscoding roziscoding commented Jun 28, 2026

Copy link
Copy Markdown
Owner

Why

Dependabot opened 8 PRs in the last batch. Two structural problems caused it:

  1. Per-member directory scanning (/apps/*, /packages/*) on a single-lockfile Bun workspace produced isolated package.json edits that left the shared root bun.lock stale → bun install --frozen-lockfile failed, and duplicated bumps the root scan already makes. (Already fixed on main in b635516 — scan only /.)
  2. Majors are never grouped, so each major got its own PR (vue-router×2, typescript×2, antfu×1).

This branch finishes the cleanup.

Changes

  • dependabot.yml — collapse prod+dev into a single all-dependencies group (one weekly PR for the whole workspace), and ignore all major-version bumps. Security updates still apply; majors get handled deliberately by hand.
  • apps/ui vue-router pin — was ^4.5.1, but nuxt@4.x depends on vue-router ^5.1.0. The stale pin forced a second, unused v4 copy alongside Nuxt's v5. Pin ^5.1.0 to match Nuxt (now safe from surprise majors via the ignore rule). Lockfile regenerated → single vue-router@5.1.0, no v4 anywhere.

Verification

  • bun install --frozen-lockfile — clean
  • nuxt typecheck (apps/ui) — exits 0, zero errors

Follow-ups (not in this PR)

Greptile Summary

This PR tightens dependency update handling for the Bun workspace. The main changes are:

  • One grouped Dependabot PR for Bun dependency updates.
  • Scheduled major dependency updates ignored for manual handling.
  • apps/ui aligned to Nuxt's vue-router ^5.1.0 dependency.
  • The Bun lockfile regenerated for the new router resolution.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • The router pin matches Nuxt's own dependency.
  • The lockfile keeps incompatible Babel versions separated through nested resolutions.
  • The remaining note is a small Dependabot commit-label cleanup.

Fix All in Claude Code Fix All in Codex

Reviews (1): Last reviewed commit: "chore(ui): align vue-router with Nuxt's ..." | Re-trigger Greptile

Greptile also left 1 inline comment on this PR.

Context used (5)

  • Context used - packages/schemas/CLAUDE.md (source)
  • Context used - CLAUDE.md (source)
  • Context used - Use Bun instead of Node.js, npm, pnpm, or vite. (source)
  • Context used - apps/backend/CLAUDE.md (source)
  • Context used - AGENTS.md (source)

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.
Comment thread .github/dependabot.yml
@roziscoding roziscoding changed the title ci(deps): group Bun updates into one PR, ignore majors, align vue-router with Nuxt ci(deps): group all Bun updates, ignore majors, pin vue-router Jun 28, 2026
@roziscoding roziscoding merged commit 9b72506 into main Jun 28, 2026
10 checks passed
@roziscoding roziscoding deleted the chore/dependabot-grouping branch June 28, 2026 13:22
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