Repository navigation
build(deps-dev): bump vitest to 5.0.5 - #222
Open
claude[bot] wants to merge 1 commit into
Open
claude[bot] wants to merge 1 commit into
claude[bot] wants to merge 1 commit into
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01N2vvE5XzurMvQfsX7Vz7bF
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Contributor
Author
There was a problem hiding this comment.
I reviewed this dev-only vitest bump and didn't find any bugs. Every path in the repo has CODEOWNERS (* in .github/CODEOWNERS), so a code owner still needs to sign off.
What was reviewed:
- yarn.lock matches package.json. The workspace entry and the vitest/@ vitest/coverage-v8 peer pins all move together to 5.0.5.
- The new versions of obug (3.0.0) and why-is-node-running (3.2.1) are only used inside vitest and coverage-v8. No repo source imports them.
- Nothing else in the lockfile still needs siginfo or stackback. The old
tinyexec ^1.3.0entry is still needed by another dependency, so keeping it alongside the new^1.3.1entry is correct.
Extended reasoning...
This change only touches package.json devDependencies (vitest and @ vitest/coverage-v8, 5.0.1 to 5.0.5) and the regenerated yarn.lock. No runtime dependencies or source files change. It touches no security-sensitive code, though it does pull in new third-party package versions, all with matching lockfile checksums. Approval was withheld only because .github/CODEOWNERS assigns every path to code owner teams.
This branch has not been deployed
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.
This bumps
vitestfrom ^5.0.1 to ^5.0.5 and@vitest/coverage-v8from 5.0.1 to 5.0.5 (still pinned exactly), and the lockfile moves both from 5.0.1 to 5.0.5 along with vitest's own dependencies. 5.0.5 carries the fix for GHSA-p458-7748-mcp3, which affects@vitest/uiand@vitest/browser. This repo doesn't use either of those, so it wasn't exposed, and the bump keeps the vitest packages in step. Every package that changed in the lockfile was published at least 7 days ago. Lint andtscpass locally, and all 144 tests pass with coverage.Generated by Claude Code