Skip to content

chore(deps-dev): bump vitest to 4.1.0 and align companion packages#9233

Open
dependabot[bot] wants to merge 3 commits into
mainfrom
dependabot/npm_and_yarn/vitest-4.1.0
Open

chore(deps-dev): bump vitest to 4.1.0 and align companion packages#9233
dependabot[bot] wants to merge 3 commits into
mainfrom
dependabot/npm_and_yarn/vitest-4.1.0

Conversation

@dependabot
Copy link
Copy Markdown
Contributor

@dependabot dependabot Bot commented on behalf of github Jun 1, 2026 •
edited by Copilot AI
Loading

Commit Type

  • feature - New functionality
  • fix - Bug fix
  • refactor - Code restructuring without behavior change
  • perf - Performance improvement
  • docs - Documentation update
  • test - Test-related changes
  • chore - Maintenance/tooling

Risk Level

  • Low - Minor changes, limited scope
  • Medium - Moderate changes, some user impact
  • High - Major changes, significant user/system impact

What & Why

Upgrades vitest from 3.2.4 to 4.1.0 and aligns companion packages (@vitest/coverage-istanbul, @vitest/ui) to compatible 4.x versions to avoid mixed-version runtime issues.
Includes follow-up compatibility fixes from review feedback (snapshot updates and chat test stability) and resolves merge conflicts by merging main into this branch with lockfile reconciliation.

Impact of Change

  • Users: No direct end-user feature changes.
  • Developers: Test tooling is now consistently on Vitest 4.x; snapshots updated where serialization changed.
  • System: Dependency graph and lockfile updated; branch now includes latest main conflict-resolution merge.

Test Plan

  • Unit tests added/updated
  • E2E tests added/updated
  • Manual testing completed
  • Tested in: targeted Vitest runs for @microsoft/designer-ui (addActionCard.spec.tsx) and @microsoft/logic-apps-chat (test:lib), plus lockfile integrity check with pnpm install --frozen-lockfile

Contributors

Screenshots/Videos

N/A

@dependabot
chore(deps-dev): bump vitest from 3.2.4 to 4.1.0
ddea18c 
Bumps [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) from 3.2.4 to 4.1.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/v4.1.0/packages/vitest)

---
updated-dependencies:
- dependency-name: vitest
  dependency-version: 4.1.0
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added the dependencies Pull requests that update a dependency file label Jun 1, 2026
Copilot AI review requested due to automatic review settings June 1, 2026 21:46
@dependabot dependabot Bot added javascript Pull requests that update javascript code dependencies Pull requests that update a dependency file labels Jun 1, 2026
Copilot AI reviewed Jun 1, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026 •
edited
Loading

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

✅ PR Title

  • Current: chore(deps-dev): bump vitest to 4.1.0 and align companion packages
  • Issue: None. The title is specific, accurate, and clearly describes the dependency update.
  • Recommendation: No change needed.

✅ Commit Type

  • Properly selected (chore).
  • Only one commit type is selected, which is correct.

✅ Risk Level

  • The selected risk level is Medium, which is reasonable for a dependency/tooling upgrade with lockfile churn and test updates.
  • Advised risk from diff: Medium.

✅ What & Why

  • Current: Upgrades vitest from 3.2.4 to 4.1.0 and aligns companion packages...
  • Issue: None. This is clear and matches the diff well.
  • Recommendation: No change needed.

✅ Impact of Change

  • The section is filled out appropriately and matches a dev-tooling update.
  • Recommendation:
    • Users: No direct end-user impact is correct.
    • Developers: Correctly notes Vitest 4.x alignment and snapshot updates.
    • System: Correctly mentions lockfile and dependency graph updates.

✅ Test Plan

  • The PR includes updated unit tests in the diff, so the test plan passes.
  • The additional manual/targeted run notes are helpful, but not required for pass/fail.

✅ Contributors

  • Contributors are listed appropriately.
  • No issues found.

✅ Screenshots/Videos

  • N/A is appropriate for a tooling/dependency change.

Summary Table

Section Status Recommendation
Title ✅
Commit Type ✅
Risk Level ✅
What & Why ✅
Impact of Change ✅
Test Plan ✅
Contributors ✅
Screenshots/Videos ✅

Overall: this PR passes review for title/body compliance. The advised risk level remains Medium, matching the submitter’s selection.

Last updated: Tue, 02 Jun 2026 21:06:21 GMT

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 1, 2026 •
edited
Loading

📊 Coverage check completed. See workflow run for details.

@rllyy97
Copy link
Copy Markdown
Contributor

rllyy97 commented Jun 2, 2026

Dependency Review: vitest 3.2.4 -> 4.1.0

Verdict: Needs companion package updates + test fixes

This is a major version bump with breaking changes. The PR only updates vitest but leaves companion packages at v3:

Issues found:

  1. Version mismatch: @vitest/coverage-istanbul (3.2.4) and @vitest/ui (3.2.4) need to be updated to 4.x to match vitest 4.1.0. Running mixed versions causes errors.
  2. Snapshot failures: 5 snapshot tests failing in addActionCard.spec.tsx - vitest 4.x may serialize snapshots differently
  3. Chat test failures: @microsoft/logic-apps-chat:test:lib failing

vitest 4.x breaking changes to be aware of:

  • basic reporter removed (use default with summary: false)
  • Coverage options changed (coverage.all, coverage.extensions removed)
  • Default exclude patterns changed (only node_modules and .git excluded by default)
  • V8 coverage uses AST-based remapping

Fix needed: Bump @vitest/coverage-istanbul and @vitest/ui to 4.x, update vitest config if needed, update snapshots, and fix chat tests. Will push a fix.

@rllyy97
fix: update vitest companion packages to v4 and fix test compatibility
e1c3b7e 
- Update @vitest/coverage-istanbul and @vitest/ui to ^4.1.0 in root and a2a-core
- Update @vitest/coverage-v8 to ^4.1.0 in iframe-app
- Add virtual-store-dir-max-length=80 to .npmrc for Windows path length compatibility
- Fix coverage include patterns from src/**/* to src/**/*.{ts,tsx,js,jsx} to avoid
  instrumenting non-code files (graphify .md files) in vitest 4.x
- Fix constructor mock patterns: use function() syntax instead of arrow functions
  for class/constructor mocks (vitest 4.x breaking change)
- Increase test timeout for locale loading test (vitest 4.x API change)
- Use vitest 4.x test options API: { timeout } as second arg instead of third arg
- Update snapshots for designer-ui

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@rllyy97
Copy link
Copy Markdown
Contributor

rllyy97 commented Jun 2, 2026

@copilot resolve the merge conflicts in this pull request

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 2, 2026

@copilot resolve the merge conflicts in this pull request

Done in c8d08c8: merged origin/main into this branch and resolved the merge conflicts (including pnpm-lock.yaml), then validated lockfile integrity and reran targeted tests.

Copilot AI changed the title chore(deps-dev): bump vitest from 3.2.4 to 4.1.0 chore(deps-dev): bump vitest to 4.1.0 and align companion packages Jun 2, 2026
Copilot AI requested a review from rllyy97 June 2, 2026 20:48
@rllyy97 rllyy97 enabled auto-merge (squash) June 2, 2026 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@rllyy97 rllyy97 Awaiting requested review from rllyy97

Assignees

No one assigned

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code pr-validated

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rllyy97