Skip to content

ci: upgrade to vitest 5 - #127

Merged
rolandjitsu merged 2 commits into
mainfrom
deps/vitest-5
Sep 17, 2026
Merged

rolandjitsu merged 2 commits into
mainfrom
deps/vitest-5

Conversation

@rolandjitsu

Copy link
Copy Markdown
Collaborator

What kind of change does this PR introduce?

  • Bug Fix
  • Feature
  • Refactoring
  • Style
  • Build
  • Chore
  • Documentation
  • CI

Did you add tests for your changes?

  • Yes, my code is well tested
  • Not relevant

If relevant, did you update the documentation?

  • Yes, I've updated the documentation
  • Not relevant

Summary

Upgrade vitest to v5.

Does this PR introduce a breaking change?

No.

Other information

Both were provided transitively by vitest's vite: vite@8 pulled rolldown
to the top level, satisfying tsdown's rolldown-plugin-dts peer import, and
vitest depended on vite directly. legacy-peer-deps hides these, so the
upcoming vitest 5 bump (vite becomes a peer, rolldown is no longer hoisted)
would break the build and test runner. Declare both directly to decouple
the build from vitest's dependency tree.

Assisted-by: Claude:claude-opus-4-8
Combines dependabot #126 and #125, which must move in lockstep. vite and
rolldown are already declared directly (previous commit), so the v5 tree
resolves cleanly under legacy-peer-deps. Also drops the @vitest/mocker path
traversal advisory (GHSA-82fw-gwwq-j7x9) that shipped with vitest 4.

Assisted-by: Claude:claude-opus-4-8
@rolandjitsu
rolandjitsu merged commit 423d49d into main Sep 17, 2026
6 checks passed
@rolandjitsu
rolandjitsu deleted the deps/vitest-5 branch September 17, 2026 09:26
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