mirror of
https://github.com/Donkie/Spoolman.git
synced 2026-10-22 01:52:17 -05:00
Donkie
be127f2fb7
The bump alone doesn't work, it needs three supporting changes: - react-router 8 exposes its types through an exports map, which the old "moduleResolution": "Node" can't read, so tsc failed to find the module in every file importing it. Switch to "Bundler", which is what Vite has always resolved with anyway. - @refinedev/react-router peer-depends on react-router ^7.0.2, so npm kept a nested 7.18.2 next to the top-level 8.3.0. Two copies means two React contexts, and Refine's useLocation() threw "may be used only in the context of a <Router> component" inside ThemedSider, leaving a blank page. Override the adapter's react-router to ^8.3.0 so the tree dedupes. - Bundler resolution correctly rejects the deep import of NumberFieldProps from @refinedev/antd, which isn't in that package's exports map and isn't re-exported from its root. Mirror the type locally instead of taking on @refinedev/ui-types as a new dependency. react-router 8 requires Node >= 22.22.0, so the client's engines range drops Node 20. CI and the Docker build already use Node 24. Verified with the frontend Playwright suite (both tests pass, including the smoke test asserting no console errors on every page).