Repository navigation
chore: drop Node 20 and refresh all lockfile entries - #186
Merged
Merged
Conversation
The earlier upgrade only touched named top-level JS packages, leaving transitive entries stale (e.g. source-map-js 1.2.1 despite 1.2.2 being available). Run a full in-range `yarn upgrade` and `uv lock --upgrade`. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Node 20 is end-of-life. Move the CI matrix to Node 22 and 24 and upgrade the held-back dev dependencies: vitest and coverage-v8 5, jest-dom 7, jsdom 30. Verified install, tests and build on Node 22.23.3 and 24.21.0. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
react-router-dom's latest tag stays on 7.18, so v8 is consumed from react-router directly. Swap the dependency and update imports. Tests and build pass on Node 22 and 24. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #184, which merged before these two commits landed. Supersedes #185 (source-map-js).
Refresh transitive dependencies
yarn upgradeanduv lock --upgradeacross both lockfiles. chore: update all dependencies #184 only upgraded named top-level JS packages, so transitive entries were stale (e.g.source-map-js1.2.1 although 1.2.2 was available, which is what Bump source-map-js from 1.2.1 to 1.2.2 in /fussel/web #185 bumps).Drop Node 20 (end-of-life) and take the held-back test tooling
@vitest/coverage-v84 → 5,@testing-library/jest-dom6 → 7, jsdom 29 → 30. These require Node >= 22 (jsdom 30: >= 22.22.2).Move to react-router 8
react-router-dom'slatesttag stays on 7.18, so v8 is taken fromreact-routerdirectly: swapped the dependency and the imports (react-router-dom→react-router).After this,
yarn outdatedlists nothing anduv lock --upgradechanges nothing.Verified: ruff format/check, pytest (117), and clean frozen
yarn install, JS tests (27) andyarn buildon Node 22.23.3 and 24.21.0.🤖 Generated with Claude Code