Skip to content

build(deps-dev): bump vitest to 5.0.5 - #238

Merged
MarshallOfSound merged 1 commit into
mainfrom
vitest-5.0.5
Oct 25, 2026
Merged

MarshallOfSound merged 1 commit into
mainfrom
vitest-5.0.5

Conversation

@claude

@claude claude Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Bumps vitest from ^5.0.1 (locked at 5.0.1) to ^5.0.5. 5.0.5 carries the fix for GHSA-p458-7748-mcp3, which affects @vitest/ui and @vitest/browser. This repo doesn't depend on either of those, so it wasn't affected; this just keeps the vitest family on the patched release.


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 a review from a team as a code owner October 22, 2026 22:53
@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
Updatedvitest@​5.0.1 ⏵ 5.0.59810079 +199 +1100

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.

I reviewed this vitest bump and found no bugs. CODEOWNERS assigns every path to a code-owner team, so I'm leaving approval to one of its members.

What was reviewed:

  • yarn.lock matches package.json. vitest, @ vitest/mocker and @ vitest/spy all resolve to 5.0.5, and the peer pins were updated to match.
  • The two major bumps (obug 3, why-is-node-running 3.2.1) are used only by vitest. Dropping siginfo and stackback follows from why-is-node-running 3, which no longer depends on them.
  • tinyexec ^1.3.0 is still locked separately for another dependent, so splitting its entry is expected.
Extended reasoning...

The change bumps one devDependency range in package.json (vitest ^5.0.1 to ^5.0.5) and regenerates the matching yarn.lock entries. It changes no source code and touches no security-sensitive runtime code, since it only changes test tooling. The new transitive majors are used only inside vitest, and the repo already requires Node >=22.12.0. The deciding fact is .github/CODEOWNERS, which assigns all paths (*) to a code-owner team, so a code owner should approve rather than this automated review.

@MarshallOfSound
MarshallOfSound enabled auto-merge (squash) October 25, 2026 22:03
@MarshallOfSound
MarshallOfSound merged commit e6b3d45 into main Oct 25, 2026
9 checks passed
@MarshallOfSound
MarshallOfSound deleted the vitest-5.0.5 branch October 25, 2026 22:03
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.

3 participants