Skip to content

chore: bump vitest from 4.1.11 to 5.0.0 - #59

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

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.

@dependabot @github

dependabot Bot commented on behalf of github Sep 21, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #68.

@dependabot dependabot Bot closed this Sep 21, 2026
@dependabot
dependabot Bot deleted the dependabot/npm_and_yarn/vitest-5.0.0 branch September 21, 2026 05:45
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