Skip to content

ci: group Dependabot updates across mirrored packages#867

Merged
theashraf merged 5 commits into
mainfrom
ci/dependabot-grouping
May 18, 2026
Merged

ci: group Dependabot updates across mirrored packages#867
theashraf merged 5 commits into
mainfrom
ci/dependabot-grouping

Conversation

@theashraf

@theashraf theashraf commented May 14, 2026

Copy link
Copy Markdown
Member

Description

Consolidates Dependabot version-update PRs across the six mirrored /packages/* directories using v2 directories: (plural) plus per-ecosystem groups.

Why now

Across the last 50 Dependabot PRs:

  • Merged: 0
  • Closed without merge: 40
  • Still open: 10
  • Merge rate: 0%

Same dep recreates near-identical PRs per release across mirrored packages — examples from the last ~6 weeks:

Dep + version Duplicate PRs
tsdown@0.22.0 8
typescript@6.0.3 5
typescript@6.0.2 5
tsdown@0.21.9 5
tsdown@0.21.8 5
tsdown@0.21.7 5

Each round is auto-superseded by the next bump a week later and closed unmerged, so reviewers see churn instead of signal — and real security updates (e.g. #826 postcss, sitting open since 2026-04-29) get lost.

What this changes

  • Replaces six per-directory updates: entries with a single directories: (plural) entry — one grouped PR will now span all six packages.
  • Adds two groups per ecosystem: dev-dependencies and prod-dependencies, both scoped to minor + patch only.
  • Adds an github-actions group covering all action bumps.

What stays the same

  • Major bumps still get individual PRs. The update-types: ["minor", "patch"] filter excludes majors from the groups — they need individual review.
  • Security updates bypass groups automatically. Dependabot's security-update flow is not subject to the weekly schedule or groups config.
  • Weekly schedule preserved.
  • No new ecosystems added. /apps/viewer remains out of scope (already covered by security updates today).

Package(s) affected

  • CI / repo config only (no package code changes)

Type of change

  • Chore (build, CI, docs, refactor)

Checklist

  • Changes have been tested locally (YAML validated)
  • Tests have been added or updated — n/a, config-only
  • Changeset has been added — n/a, not a package change

Notes for reviewer

After merge, Dependabot will recompute its open PR set on the next scheduled run. Existing open PRs (#859#865, #846, #826, #799) will be auto-closed and replaced by grouped PRs on the next cycle. No manual cleanup required.

Last 50 Dependabot PRs: 0 merged, 40 closed without merge, 10 still open.
Same dep (e.g. tsdown) recreates 5 near-identical PRs per release across
the mirrored npm packages, drowning reviewers and masking real security
updates. Address SECURITY-AUDIT.md item #12.

Use Dependabot v2 `directories:` (plural) so groups span all six
`/packages/*` directories in a single PR. Groups limited to minor+patch
to keep majors reviewable individually; security updates bypass groups
automatically.
@theashraf theashraf requested a review from a team as a code owner May 14, 2026 04:56
@changeset-bot

changeset-bot Bot commented May 14, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 26c21d2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
@lottiefiles/dotlottie-web 27.19 KB (0%) 544 ms (0%) 16 ms (-66.28% 🔽) 560 ms
@lottiefiles/dotlottie-web WASM 484.83 KB (0%) 9.7 s (0%) 0 ms (+100% 🔺) 9.7 s
@lottiefiles/dotlottie-react 28.53 KB (0%) 571 ms (0%) 33 ms (+27.29% 🔺) 603 ms
@lottiefiles/dotlottie-vue 35.56 KB (0%) 712 ms (0%) 32 ms (-52.11% 🔽) 743 ms
@lottiefiles/dotlottie-wc 33.15 KB (0%) 664 ms (0%) 125 ms (+62.31% 🔺) 788 ms
@lottiefiles/dotlottie-svelte 41.11 KB (0%) 823 ms (0%) 66 ms (-16.42% 🔽) 888 ms
@lottiefiles/dotlottie-solid 33.92 KB (0%) 679 ms (0%) 16 ms (-44.68% 🔽) 694 ms

@theashraf theashraf merged commit c7e27e1 into main May 18, 2026
8 checks passed
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