Skip to content

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

Merged
MarshallOfSound merged 1 commit into
mainfrom
vitest-5
Sep 23, 2026
Merged

MarshallOfSound merged 1 commit into
mainfrom
vitest-5

Conversation

@claude

@claude claude Bot commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Requested by David Sanders · Slack thread

This bumps vitest from ^4.1.11 to ^5.0.1 and @vitest/coverage-v8 from 4.1.1 to 5.0.1. The coverage plugin requires the exact same vitest version, and the old pin had drifted from the locked vitest 4.1.11, which caused a "mixed versions" warning; that warning is now gone. Vitest 5 no longer depends on vite and instead lists it as a required peer dependency, so vite is added as a devDependency at ^8.0.16, the version already in the lockfile. The minimum @types/node version is also raised from ^22.9.0 to ^22.12.0 (now resolving to 22.20.3), because the locked 22.9.0 did not satisfy the @types/node peer range that vite and vitest require, and yarn warned about it on install. No test or config changes were needed: lint, build and all 123 tests pass, and coverage reports on the same files as before.

🤖 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 September 23, 2026 21:30
@socket-security

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.1.1 ⏵ 5.0.19910079 +198100
Updatedvitest@​4.1.11 ⏵ 5.0.198 +110079 +198100
Updated@​types/​node@​22.9.0 ⏵ 22.20.3100 +110081 +196100
Addedvite@​8.0.16991008296100

View full report

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Looks good, straightforward dependency bump.

What was reviewed: the package.json diff (vitest ^4.1.11→^5.0.1, @ vitest/coverage-v8 4.1.1→5.0.1, @ types/node ^22.9.0→^22.12.0, new vite ^8.0.16 devDependency) and confirmed yarn.lock is a mechanical regeneration with no other source changes; checked CODEOWNERS (generic fallback, no special ownership concern) and confirmed no application source files are touched.

Extended reasoning...

The change is confined to package.json and yarn.lock: a devDependency-only version bump (vitest v4→v5, matching coverage-v8, an added vite peer dependency, and a raised @ types/node floor) with no application source touched and no security-sensitive surface. CODEOWNERS is just the generic wg-upgrades/wg-ecosystem fallback, no specific human owner flagged. One candidate issue (future drift between the exact-pinned coverage-v8 and caret-ranged vitest) was already investigated and ruled out per the harness notes. The diff is small, mechanical, and easily verifiable by inspection, so no human look is required.

@MarshallOfSound
MarshallOfSound enabled auto-merge (squash) September 23, 2026 23:13
@MarshallOfSound
MarshallOfSound merged commit fa6c624 into main Sep 23, 2026
6 checks passed
@MarshallOfSound
MarshallOfSound deleted the vitest-5 branch September 23, 2026 23:13
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.

4 participants