Skip to content

chore(deps): bulk-update npm packages and group Renovate updates - #61

Merged
fredclausen merged 1 commit into
masterfrom
chore/bulk-npm-update-and-renovate-groups
Jul 5, 2026
Merged

chore(deps): bulk-update npm packages and group Renovate updates#61
fredclausen merged 1 commit into
masterfrom
chore/bulk-npm-update-and-renovate-groups

Conversation

@fredclausen

@fredclausen fredclausen commented Jul 5, 2026

Copy link
Copy Markdown
Member

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-dom 19.2.6 -> 19.2.7, react-router 7.18.0 -> 8.1.0
  • vite 8.0.13 -> 8.1.3, vitest + @vitest/coverage-v8 4.1.6 -> 4.1.9
  • @playwright/test 1.60.0 -> 1.61.1, @axe-core/playwright 4.11.3 -> 4.12.1
  • @types/node 25 -> 26, @types/react 19.2.14 -> 19.2.17, @vitejs/plugin-react 6.0.1 -> 6.0.3
  • sass 1.99.0 -> 1.101.0

Why 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/artifacts failure) and tsc could not resolve types (Lint failure). Here react and react-router are bumped together and package-lock.json is regenerated in one resolution.

Other changes

  • tsconfig: drop deprecated baseUrl (removed in TS 7.0); paths now resolve relative to the tsconfig directory.
  • tests: fix four pre-existing failing tests whose expectations had drifted from the UI copy (project-description separator, HomePage tagline/bio, ProjectsPage list accessible label). All 154 tests now pass.
  • renovate.json: add packageRules grouping 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 passed
  • tsc --noEmit: clean
  • biome check .: clean
  • pre-commit hooks: all passed

Summary by CodeRabbit

  • Bug Fixes

    • Improved project and profile text expectations so the app’s displayed content matches the current wording more reliably.
    • Updated project list labeling to better reflect the current site identity in accessible text.
  • Chores

    • Upgraded several app, testing, and build dependencies.
    • Adjusted automated update rules to keep related packages grouped together.

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.
@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b18c13cc-52c1-4a9e-bf3d-f196c1aae9a1

📥 Commits

Reviewing files that changed from the base of the PR and between e0628ee and b36cc46.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (6)
  • package.json
  • renovate.json
  • src/components/ProjectCard/__tests__/ProjectCard.test.tsx
  • src/pages/HomePage/__tests__/HomePage.test.tsx
  • src/pages/ProjectsPage/__tests__/ProjectsPage.test.tsx
  • tsconfig.json

📝 Walkthrough

Walkthrough

This PR bumps package dependency versions, adds Renovate grouping rules for react/playwright/vitest packages, updates the tsconfig @/* path alias to a relative path, and updates several test files to match revised UI text and accessible labels.

Changes

Dependency and tooling configuration

Layer / File(s) Summary
Dependency version bumps and Renovate grouping
package.json, renovate.json
Bumps react, react-dom, react-router, and various devDependencies (Playwright, testing tools, Vite/Vitest, Sass); adds Renovate packageRules grouping updates by "react ecosystem", "playwright", and "vitest".
tsconfig path alias adjustment
tsconfig.json
Changes the @/* path mapping from src/* to ./src/*.

Test assertion updates for content changes

Layer / File(s) Summary
ProjectCard description assertion update
src/components/ProjectCard/__tests__/ProjectCard.test.tsx
Switches description assertion to a regex substring match, adding comments about the leading separator.
HomePage tagline and bio assertion updates
src/pages/HomePage/__tests__/HomePage.test.tsx
Updates expected tagline text to "Open source contributor" and narrows the bio match to a specific phrase.
ProjectsPage accessible label update
src/pages/ProjectsPage/__tests__/ProjectsPage.test.tsx
Updates expected accessible list label from "github projects" to "fredsystems projects" in two assertions.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 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 accurately summarizes the main change: a bulk npm dependency update plus Renovate grouping rules.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bulk-npm-update-and-renovate-groups

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

@fredclausen
fredclausen merged commit 07d2269 into master Jul 5, 2026
2 checks passed
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