Skip to content

build(deps-dev): upgrade vitest to v5 - #179

Open
claude[bot] wants to merge 2 commits into
mainfrom
vitest-5
Open

claude[bot] wants to merge 2 commits into
mainfrom
vitest-5

Conversation

@claude

@claude claude Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

This moves the test suite to Vitest 5 (vitest and @vitest/coverage-v8 to ^5.0.1). Vitest 5 needs Node 22.12+, so the engines floor goes from 22 to 22.12.0 and the CI, check-dist and lint workflows now run on Node 24, which also matches the action's node24 runtime. Vite is now a required peer of Vitest rather than a transitive dependency, so it is added as a devDependency at ^8.3.0, reusing the 8.3.0 already in the lockfile. @types/node goes from ^20 to ^24 to line up with the runtime and with Vite's peer range; that bump needed no source changes. No tests or config used APIs Vitest 5 removed, there are no snapshots, and the rebuilt dist/ is unchanged. Install, format check, lint, typecheck, tests with coverage, and the dist comparison all pass locally.

🤖 Generated with Claude Code

https://claude.ai/code/session_01N2vvE5XzurMvQfsX7Vz7bF


Generated by Claude Code

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01N2vvE5XzurMvQfsX7Vz7bF
@claude
claude Bot requested review from a team as code owners October 7, 2026 18:01
@socket-security

socket-security Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​vitest/​coverage-v8@​4.0.5 ⏵ 5.0.39910079 +1199 +2100
Updatedvitest@​4.1.11 ⏵ 5.0.39810079 +199 +1100
Updated@​types/​node@​20.19.23 ⏵ 24.19.010010081 +196 +1100
Addedvite@​8.3.0991008297100

View full report

@dsanders11 dsanders11 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@claude, lets create a .nvmrc file here in the repo and point CI to it.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01N2vvE5XzurMvQfsX7Vz7bF
@claude

claude Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

Added .nvmrc (Node 24) and switched the CI, check-dist and lint workflows to node-version-file: .nvmrc in cb172a6.


Generated by Claude Code

@claude
claude Bot requested a review from dsanders11 October 7, 2026 18:18
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.

2 participants