Skip to content

Bulk dependency upgrade: React 19, Supabase 2.105, Vite 8, Vitest 4#30

Merged
brianfunk merged 2 commits into
devfrom
deps-bulk-upgrade
May 18, 2026
Merged

Bulk dependency upgrade: React 19, Supabase 2.105, Vite 8, Vitest 4#30
brianfunk merged 2 commits into
devfrom
deps-bulk-upgrade

Conversation

@brianfunk

Copy link
Copy Markdown
Collaborator

Summary

Consolidates the six grouped Dependabot PRs (#24#29) into one upgrade pass with a single CI run. All releases on this PR are direct upgrades — no breaking-change shims required.

Runtime

  • react / react-dom: 18.3.1 → 19.2.6
  • react-router-dom: 7.7.1 → 7.15.1
  • @supabase/supabase-js: 2.52.1 → 2.105.4
  • lucide-react: 0.344.0 → 0.577.0

Dev

  • @types/react / @types/react-dom: bumped to 19.x to match runtime
  • vite: 7.0.6 → 8.0.13 (rolldown-powered)
  • @vitejs/plugin-react: 4.7.0 → 6.0.2
  • vitest: 3.2.4 → 4.1.6
  • @playwright/test / playwright: 1.58.2 → 1.60.0
  • @axe-core/playwright: 4.10.2 → 4.11.3
  • @testing-library/jest-dom: 6.6.4 → 6.9.1
  • @testing-library/react: 16.3.0 → 16.3.2
  • jest-environment-jsdom: 30.0.5 → 30.4.1
  • eslint-plugin-react-refresh: 0.4.11 → 0.5.2
  • globals: 15.9.0 → 17.6.0

GitHub Actions

  • actions/checkout: v4 → v6
  • actions/setup-node: v4 → v6
  • actions/upload-artifact: v4 → v7
  • actions/dependency-review-action: v4 → v5
  • github/codeql-action: v3 → v4

Intentionally NOT bumped

  • eslint 10 + @eslint/js 10: ESLint 10 requires eslint-plugin-react-hooks 7, which adds strict React-Compiler rules (react-hooks/static-components, react-hooks/immutability) that error on ~84 currently-working patterns. Refactoring is an orthogonal change. Staying on ESLint 9 / hooks 5 keeps lint green here; the React-Compiler-friendly refactor lands in its own PR.
  • lucide-react 1.x: v1.0 dropped brand-style icons (Github, Linkedin) used by Footer/PublicLogin. 0.577.0 is the highest 0.x release with React 19 peer support and full icon set.

Local verification

  • npm install — 498 packages, 0 vulnerabilities
  • npm run typecheck — green
  • npm run lint — 0 errors (170 pre-existing warnings unchanged)
  • npm run build — 503 KB initial bundle (still ~40 % below pre-split baseline of 837 KB)
  • npm run test:ci — 28 / 28 passing on Vitest 4
  • npm run test:a11y — 13 / 13 axe routes passing on Playwright 1.60
  • npm run db:validate:fresh — full migration chain replays cleanly

After merge

The six Dependabot PRs (#24#29) auto-close when their target versions land on dev. Dependabot will resume its weekly cadence from the new baseline.

Test plan

  • CI on this PR: all required jobs green (including the new fresh-DB migration replay)
  • Smoke npm run dev on the merge commit, hit a docket detail and the landing page
  • Confirm Netlify dev deploy succeeds after merge

brianfunk added 2 commits May 17, 2026 23:13
…wright 1.60, lucide 0.577

Consolidates the six Dependabot grouped PRs (#24#29) into a single
upgrade pass with one CI run.

Runtime:
- react / react-dom: 18.3.1 -> 19.2.6
- react-router-dom: 7.7.1 -> 7.15.1
- @supabase/supabase-js: 2.52.1 -> 2.105.4
- lucide-react: 0.344.0 -> 0.577.0 (last 0.x; v1.x drops brand icons
  like Github/Linkedin that this codebase uses; staying on 0.x adds
  React 19 peer support without losing the icons)

Dev:
- @types/react / @types/react-dom: bumped to 19.x to match runtime
- vite: 7.0.6 -> 8.0.13 (now rolldown-powered)
- @vitejs/plugin-react: 4.7.0 -> 6.0.2
- vitest: 3.2.4 -> 4.1.6
- @playwright/test / playwright: 1.58.2 -> 1.60.0
- @axe-core/playwright: 4.10.2 -> 4.11.3
- @testing-library/jest-dom: 6.6.4 -> 6.9.1
- @testing-library/react: 16.3.0 -> 16.3.2
- jest-environment-jsdom: 30.0.5 -> 30.4.1
- eslint-plugin-react-refresh: 0.4.11 -> 0.5.2
- globals: 15.9.0 -> 17.6.0

Intentionally NOT bumped:
- eslint 9.x and @eslint/js 9.x kept (eslint 10 requires
  eslint-plugin-react-hooks 7, which adds strict React-Compiler rules
  that error on ~84 currently-fine patterns — orthogonal refactor).
- eslint-plugin-react-hooks 5.x kept for the same reason.

GitHub Actions:
- actions/checkout: v4 -> v6
- actions/setup-node: v4 -> v6
- actions/upload-artifact: v4 -> v7
- actions/dependency-review-action: v4 -> v5
- github/codeql-action: v3 -> v4

Verified locally:
- npm install — 0 vulnerabilities
- npm run typecheck — green
- npm run lint — 0 errors (170 pre-existing warnings unchanged)
- npm run build — 503 KB initial (still well under the pre-split 837 KB)
- npm run test:ci — 28/28 passing
- npm run test:a11y — 13/13 axe routes passing
- npm run db:validate:fresh — 17 migrations replay cleanly, 20 public tables
Vite 8 (rolldown) and a few other deps ship platform-specific native
binaries. npm install on macOS only writes the host platform's optional
deps into the lockfile, so npm ci on Linux runners fails with
'Missing: @emnapi/core@1.10.0 from lock file' even though everything
resolves correctly locally.

Trading strict ci reproducibility for the practical workaround other
repos in the org use. The lockfile is still committed and used by
npm install; --no-audit / --no-fund just keep the install output
focused on the real work.
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