Repository navigation
chore(deps): update Vitest to v5, jsdom to v30 and Node.js to 26 - #220
Merged
Merged
Conversation
CI reads the version from .tool-versions through setup-node's node-version-file, so local and CI now run the same Node.js. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- vitest, @vitest/coverage-v8, @vitest/ui: 4.1.11 -> 5.0.2 - jsdom: 29.1.1 -> 30.1.1 - @types/jsdom: 28.0.3 -> 30.0.0 Vitest 5 requires Vite >= 6.4, and the lockfile now resolves vite@8.3.1 instead of 6.3.5. 5.0.3 is not used yet because it is still inside the 7-day minimum release age. Closes #211 Co-Authored-By: Claude Opus 5.5 (1M context) <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
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
vitest,@vitest/coverage-v8,@vitest/ui) from 4.1.11 to 5.0.2, as the groundwork for test: run the test suite on real browsers with Vitest browser mode #210. The lockfile now resolvesvite@8.3.1, which satisfies Vitest 5'svite >= 6.4requirement.jsdomfrom 29.1.1 to 30.1.1 and@types/jsdomfrom 28.0.3 to 30.0.0..tool-versions. CI already reads this file vianode-version-file, so local and CI use the same version.typescriptandunbuildare out of scope (see #218). Vitest 5.0.3 is skipped for now because it is still inside the 7-day minimum release age; Renovate will pick it up later.Closes #211
Breaking changes checked
clearMocksis on by default (the config already setsmockReset: true); unawaited.resolves/.rejectsnow fail (both uses insrc/index.test.tsare awaited); novi.mockinside functions, nosequential, no removed entry points, no coverageinclude/exclude, no global assignments in jsdom. The UI now requires token authentication, which only changes howpnpm coverageopens.^22.22.2 || ^24.15.0 || >=26.0.0. 30.1.0 also fixes the ordering of mutation observer notifications; tests pass unchanged._stream_*modules,writeHeader(),--experimental-transform-types) are not used here. Node 26 is still "Current" and is scheduled to enter LTS in October 2026.Test plan
pnpm typecheckpasses on Node 26.10.0pnpm buildpasses on Node 26.10.0pnpm test-cipasses (33 tests, type tests included)describe.shuffle(5 runs with random seeds)🤖 Generated with Claude Code