Skip to content

chore(deps): consolidate JavaScript dependency upgrades (May 2026)#12713

Merged
mekarpeles merged 3 commits into
masterfrom
deps/js-consolidation-2026-05-11
May 20, 2026
Merged

chore(deps): consolidate JavaScript dependency upgrades (May 2026)#12713
mekarpeles merged 3 commits into
masterfrom
deps/js-consolidation-2026-05-11

Conversation

@mekarpeles
Copy link
Copy Markdown
Member

@mekarpeles mekarpeles commented May 11, 2026

Summary

Consolidates open Renovate PRs into a single tested upgrade. All packages installed, built, and tested in Docker before opening this PR.

Note: lockfile diffs from the individual Renovate PRs could not be applied directly (master moved since they were opened). Packages were installed via targeted npm install <package>@<version> to maintain one-commit-per-package atomicity.

Packages upgraded

Package Before After Severity Closes
slick-carousel 1.6.0 1.8.1 Low — minor bump #12639, #12758
vue 3.5.33 3.5.34 Low — patch bump #12650
jest monorepo 30.0.5 30.4.2 Low — minor bump #12642, #12758

Excluded packages

Package Reason
chart.js 2.9.4 → 4.5.1 (#12761) 2-major-version jump (v2 → v4); breaking API changes expected. Needs its own PR with careful verification.
babel-loader 9 → 10 (#12760) Major version bump; needs separate PR.
@eslint/js 9 → 10 (#12675) Major version bump (Renovate artifact failure noted); needs separate PR.

Testing

  • npm install clean
  • make js: webpack compiled with 0 errors
  • npm run test:js: 388 passed, 20 test suites, 0 failures
  • docker compose restart web successful
  • ✅ HTTP 200: home page and /search?q=test
  • ✅ pre-commit clean

Copilot AI review requested due to automatic review settings May 11, 2026 19:09
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR consolidates several Renovate-driven JavaScript dependency upgrades into a single change-set, updating the dev dependency ranges and the npm lockfile to reflect the resolved versions used by Open Library’s frontend build/test tooling.

Changes:

  • Bump Jest + jest-environment-jsdom to the 30.4.x line.
  • Upgrade Vue to 3.5.34.
  • Upgrade slick-carousel to 1.8.1 and refresh package-lock.json accordingly.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Updates the declared devDependency version ranges for Jest, Vue, and slick-carousel.
package-lock.json Updates the resolved dependency tree to match the new versions (Jest/Vue/slick-carousel and transitive updates).

@mekarpeles mekarpeles self-assigned this May 11, 2026
@RayBB
Copy link
Copy Markdown
Collaborator

RayBB commented May 11, 2026

FYI, you can configure renovate to consolidate PRs for minor bumps like this, if you find that desirable.

https://stackoverflow.com/questions/66471226/renovate-combine-all-updates-to-one-branch-pr

@mekarpeles mekarpeles force-pushed the deps/js-consolidation-2026-05-11 branch from b4dd6da to 3802170 Compare May 20, 2026 00:16
@mekarpeles mekarpeles merged commit ccaf6e7 into master May 20, 2026
7 of 8 checks passed
@mekarpeles mekarpeles deleted the deps/js-consolidation-2026-05-11 branch May 20, 2026 21:40
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.

3 participants