Skip to content

chore: bump vitest from 4.1.11 to 5.0.0 - #59

Open
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/vitest-5.0.0
Open

chore: bump vitest from 4.1.11 to 5.0.0#59
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/vitest-5.0.0

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Sep 7, 2026

Copy link
Copy Markdown
Contributor

Bumps vitest from 4.1.11 to 5.0.0.

Release notes

Sourced from vitest's releases.

v5.0.0

Vitest 5 is officially out! This release focuses on performance and brings a lot of new features while fixing long-standing bugs. See our blog post for the official announcement.

   🚨 Breaking Changes

... (truncated)

Commits
  • f441c6f chore: release v5.0.0 (#11130)
  • d46a747 fix: treat test.describe as a suite during static collection (#11128)
  • 584cf30 fix: add a warning if inline project has duplicate plugins due to unexpected ...
  • f08ce4b fix: apply queued mocks from doMock() in queue order (fixes #10706) (#11127)
  • 897f51f chore: release v5.0.0-rc.4 (#11107)
  • 1339b06 chore(deps): update all non-major dependencies (#11104)
  • 51e9494 feat!: parse files statically in vitest list by default (#11088)
  • 2122ffd fix: propagate --maxWorkers to projects (#11102)
  • dc10f5f fix(browser): report the action error when a task times out (#11101)
  • d4fe198 feat: promote clearCache out of experimental (#11086)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) from 4.1.11 to 5.0.0.
- [Release notes](https://github.com/vitest-dev/vitest/releases)
- [Changelog](https://github.com/vitest-dev/vitest/blob/main/docs/releases.md)
- [Commits](https://github.com/vitest-dev/vitest/commits/v5.0.0/packages/vitest)

---
updated-dependencies:
- dependency-name: vitest
  dependency-version: 5.0.0
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code labels Sep 7, 2026
@claude

claude Bot commented Sep 7, 2026

Copy link
Copy Markdown

Claude finished @dependabot[bot]'s task in 1m 38s —— View job


Dependency review: vitest 4.1.11 → 5.0.0 (major)

  • Gather context (diff, engines, CI matrix, lockfile)
  • Check vitest 5 breaking changes against actual repo usage
  • Verify CI result
  • Post review

Verdict: safe to merge. CI is green on both Node 22 and 24 (run), and none of vitest 5's breaking changes touch code in this repo.

What changed

Dev-only bump. package.json moves vitest ^4.1.10^5.0.0; the lockfile resolves vitest 5.0.0 and drops the now-inlined @vitest/runner / @vitest/expect packages (vitest 5 inlines expect and runner and stops publishing them). vite is unchanged at 8.2.2 — already inside vitest 5's new ^6.4.0 || ^7 || ^8 peer range, so no transitive Vite major came along.

Breaking changes checked against this codebase

vitest 5 breaking change Impact here
Requires Node 22 / Vite 6.4 Floor is already >=22, Vite is 8.2.2 — see inline note on the exact patch level
Clear mocks by default before each test No effect. Every mock is constructed per-test — inside it (test/tools/execute-action.test.ts) or inside beforeEach (test/client/kosli-client.test.ts:87) — so nothing is shared across tests. clearMocks also only clears call history, not implementations
Don't look up config from ancestor dirs No vitest.config.* exists at all; the suite runs on defaults
Remove deprecated entry points All 8 test files import from bare "vitest" only
Inline expect / @vitest/runner Neither is imported directly
Fail on un-awaited async assertions Would surface as failures — suite passes, so none present
toThrow(""), test.for/each $ titles, sequential, expect.poll, locators, benchmark/browser/coverage/reporter changes None of these APIs are used anywhere in test/

The suite uses only describe/it/expect/vi.fn/beforeEach/afterEach, which is why the blast radius is so small — the catalog-driven design keeps tests to plain assertions plus an injected fetchFn, with no module mocking, fake timers, or snapshots.

One thing to note

vitest 5's own engines is ^22.12.0 || ^24.0.0 || >=26.0.0, slightly narrower than this repo's ">=22". Dev-only and CI-safe; details in the inline comment.

Not done

I couldn't run npm ci / npm test locally in this sandbox (install requires approval), so the verification above rests on the repo's own CI run rather than a local execution.
· branch dependabot/npm_and_yarn/vitest-5.0.0

Comment thread package.json
"tsx": "^4.19.0",
"typescript": "^5.7.0",
"vitest": "^4.1.10"
"vitest": "^5.0.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Heads-up (non-blocking): vitest 5 declares engines.node: "^22.12.0 || ^24.0.0 || >=26.0.0", but this repo declares "node": ">=22" (package.json:13). A contributor on Node 22.0–22.11 — or on an odd release like 23/25 — now gets EBADENGINE on npm install, even though the repo says they're supported.

It's dev-only, so the published package and .mcpb bundle are unaffected, and CI is fine (setup-node with '22' resolves to the latest 22.x). Per CLAUDE.md, raising the real floor means moving engines, manifest.json, README.md, both workflows and @types/node together — probably not worth it just for a devDependency. Worth a line in CONTRIBUTING.md instead if it bites anyone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants