Skip to content

chore(deps): upgrade G4 — design system & UI primitives - #50

Closed
bravew wants to merge 6 commits into
mainfrom
chore/upgrade-G4-design-system-2026-09-27
Closed

bravew wants to merge 6 commits into
mainfrom
chore/upgrade-G4-design-system-2026-09-27

Conversation

@bravew

@bravew bravew commented Sep 27, 2026 •

Copy link
Copy Markdown
Owner

Summary

Group 4 of the 2026-09-27 dependency sweep (dev-doc/plan/_module-lib-upgrade.md rev 4).

Package Before After
lucide-react ^1.33.0 ^1.48.0
tailwind-merge ^3.6.0 ^3.7.0

Test plan

Checks that already fail on main (typecheck 1, format 3 fixtures, API lint 4) produce identical results here. No lucide icon imports broke.

  • pnpm install: clean, 0 peer warnings
  • pnpm typecheck:all, lint, API lint, format:check: same as baseline
  • pnpm test:fast: 1453 frontend + 3533 API tests passed (7 skipped)
  • pnpm check:design-system-flags: passed
  • pnpm build: PWA precache 26,836.92 → 26,852.13 KiB (+0.06%)
  • Headless smoke of /, /library, /automation, /dashboard, /projects, /design, /video, /approvals against a worktree API with a throwaway HOME: no error boundaries or page errors. The only failed request is a 404 for the unset assets.catalog_enabled setting on a fresh database.
  • pnpm audit --audit-level moderate: 7, unchanged; signatures 2148/2148 verified

Summary by CodeRabbit

  • Chores
    • Updated supporting package versions.

@playwright/test and playwright 1.62.1 → 1.63.0, @testing-library/react
16.3.2 → 16.3.3, @testing-library/user-event 14.6.5 → 14.6.7, jsdom
30.0.1 → 30.1.1. Refreshes the transitive
@csstools/css-parser-algorithms to 4.0.1 so jsdom's css-calc peer is met.

Held: vitest 5. @testing-library/jest-dom 7.0.1 still augments
Assertion<T>, which no longer merges with Vitest 5's Assertion<R, T>, so
~680 jest-dom matcher type errors appear (jest-dom #738, fix PR #742
open). @copilotkit/channels-core's latest release also peers vitest ^4.

Gate: install clean, no peer warnings; typecheck/lint/format identical
to the main baseline; test:fast 1453 + 3533 passed; browser and API e2e
failures identical to the baseline (23 and 1, pre-existing); audit
unchanged at 7; signatures 2148/2148 verified.
vite 8.2.2 → 8.3.1, @vitejs/plugin-react 6.1.0 → 6.1.1.
vite-plugin-pwa has no newer release, so the workbox-* override stays.

Gate: install clean, no peer warnings; typecheck/lint/format identical
to the main baseline; test:fast 1453 + 3533 passed; pnpm build green,
precache 26836.92 → 26836.04 KiB (540 entries, unchanged); preview
serves index and sw.js; audit unchanged at 7; signatures 2150/2150.
lucide-react 1.33.0 → 1.48.0, tailwind-merge 3.6.0 → 3.7.0.

Gate: install clean, no peer warnings; typecheck/lint/format identical
to the main baseline (no removed-icon type errors); test:fast 1453 +
3533 passed; check:design-system-flags passed; build precache
26836.92 → 26852.13 KiB (+0.06%); headless smoke of 8 routes against
an isolated API renders without crashes; audit unchanged at 7;
signatures 2148/2148.
@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Warning

Review limit reached

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 5a65a029-574a-4358-b3f9-ddc847951e79

📥 Commits

Reviewing files that changed from the base of the PR and between 2cabe36 and 6546c38.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • package.json
  • src-video/package.json

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 63c6bd7b-7045-4adc-b62d-ea783a3df198

📥 Commits

Reviewing files that changed from the base of the PR and between 7b26dc2 and 2cabe36.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The dependency ranges for lucide-react and tailwind-merge changed in package.json.

Changes

Dependency range updates

Layer / File(s) Summary
Dependency version ranges
package.json
The declared range for lucide-react changed from ^1.33.0 to ^1.48.0. The declared range for tailwind-merge changed from ^3.6.0 to ^3.7.0.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 2cabe

The dependency updates match the lockfile used by the repository’s supported install and build workflows. No concrete merge-blocking risk is evident.

Architecture Summary

Architecture risk: 🔵 Low · up to 2cabe

The change affects 1 system.

Changed systems: package.json

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — package.json (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in package.json: Updated the lucide-react dependency range from ^1.33.0 to ^1.48.0.
  • observed — Modified behavior in package.json: Updated the tailwind-merge dependency range from ^3.6.0 to ^3.7.0.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies a dependency upgrade and accurately describes the affected design-system and UI-primitives packages.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@bravew bravew mentioned this pull request Sep 27, 2026
2 of 3 tasks
@bravew

bravew commented Sep 27, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #61, which lands G2–G14 as one commit.

@bravew bravew closed this Sep 27, 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