chore(deps): bulk-update npm packages and group Renovate updates - #61
Merged
Merged
Conversation
Update all npm dependencies to their latest versions in a single change rather than merging the individual Renovate PRs (#49-#56, #59): - react/react-dom 19.2.6 -> 19.2.7, react-router 7 -> 8 - vite 8.0.13 -> 8.1.3, vitest + coverage 4.1.6 -> 4.1.9 - @playwright/test 1.60 -> 1.61.1, @axe-core/playwright 4.11.3 -> 4.12.1 - @types/node 25 -> 26, @types/react, @vitejs/plugin-react 6.0.1 -> 6.0.3 - sass 1.99 -> 1.101 react-router 8 requires react >= 19.2.7, so react-router and react are bumped together and package-lock.json is regenerated in one resolution. This is why the standalone react-router PR (#59) failed its renovate/artifacts and lint checks: it bumped react-router alone against react 19.2.6, so npm could not resolve a lockfile and tsc could not resolve types. Also: - Drop deprecated tsconfig baseUrl (removed in TS 7.0); paths now resolve relative to the tsconfig directory. - Fix four pre-existing tests whose expectations had drifted from the UI copy (project description separator, HomePage tagline/bio, ProjectsPage list accessible label). - Add Renovate packageRules grouping peer-coupled deps (react ecosystem, playwright, vitest) so future updates resolve together and cannot reproduce the #59 failure.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (6)
📝 WalkthroughWalkthroughThis PR bumps package dependency versions, adds Renovate grouping rules for react/playwright/vitest packages, updates the tsconfig ChangesDependency and tooling configuration
Test assertion updates for content changes
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
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
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.
Summary
Replaces the stack of individual Renovate npm PRs (#49-#56, #59) with a single coordinated update of all npm dependencies to their latest versions, plus supporting fixes.
Dependency updates
react/react-dom19.2.6 -> 19.2.7,react-router7.18.0 -> 8.1.0vite8.0.13 -> 8.1.3,vitest+@vitest/coverage-v84.1.6 -> 4.1.9@playwright/test1.60.0 -> 1.61.1,@axe-core/playwright4.11.3 -> 4.12.1@types/node25 -> 26,@types/react19.2.14 -> 19.2.17,@vitejs/plugin-react6.0.1 -> 6.0.3sass1.99.0 -> 1.101.0Why PR #59 failed (and why this doesn't)
react-router 8 requires
react >= 19.2.7. PR #59 bumped react-router alone against react 19.2.6, so npm could not resolve a lockfile (renovate/artifactsfailure) andtsccould not resolve types (Lintfailure). Here react and react-router are bumped together andpackage-lock.jsonis regenerated in one resolution.Other changes
baseUrl(removed in TS 7.0);pathsnow resolve relative to the tsconfig directory.packageRulesgrouping peer-coupled deps (react ecosystem, playwright, vitest) so future updates resolve together and cannot reproduce the fix(deps): update dependency react-router to v8 - autoclosed #59 failure.Verification
vitest: 154 passedtsc --noEmit: cleanbiome check .: cleanSummary by CodeRabbit
Bug Fixes
Chores