Skip to content

chore(deps): update vitest monorepo to v4 (major) #7509

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

chore(deps): update vitest monorepo to v4 (major) #7509

wants to merge 12 commits into from

Conversation

@renovate
Copy link
Contributor

@renovate renovate bot commented Oct 25, 2025 •
edited
Loading

This PR contains the following updates:

Package Change Age Confidence
@vitest/browser (source) 3.2.4 -> 4.0.6 age confidence
@vitest/coverage-istanbul (source) 3.2.4 -> 4.0.6 age confidence
@vitest/coverage-v8 (source) 3.2.4 -> 4.0.6 age confidence
@vitest/ui (source) 3.2.4 -> 4.0.6 age confidence
vitest (source) 3.2.4 -> 4.0.6 age confidence

Release Notes

vitest-dev/vitest (@​vitest/browser)

v4.0.6

Compare Source

   🐞 Bug Fixes
    View changes on GitHub

v4.0.5

Compare Source

   🐞 Bug Fixes
   🏎 Performance
    View changes on GitHub

v4.0.4

Compare Source

   🐞 Bug Fixes
   🏎 Performance
    View changes on GitHub

v4.0.3

Compare Source

   🐞 Bug Fixes
    View changes on GitHub

v4.0.2

Compare Source

   🐞 Bug Fixes
    View changes on GitHub

v4.0.1

Compare Source

   🐞 Bug Fixes
    View changes on GitHub

v4.0.0

Compare Source

Vitest 4.0 is out!

To stay updated, read our blog post and check the migration guide.

   🚨 Breaking Changes
   🚀 Features
   🐞 Bug Fixes
   🏎 Performance

Configuration

📅 Schedule: Branch creation - "before 3am" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.

  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot added dependencies PRs or issues relating to dependencies renovate labels Oct 25, 2025
@renovate renovate bot force-pushed the renovate/major-vitest-monorepo branch 2 times, most recently from a0f2900 to 0164fbe Compare October 25, 2025 06:18
@renovate renovate bot force-pushed the renovate/major-vitest-monorepo branch 2 times, most recently from 4de523f to 4d19b0f Compare October 27, 2025 14:22
@renovate renovate bot changed the title chore(deps): update vitest monorepo to v4 (major) Update vitest monorepo to v4 (major) Oct 28, 2025
@renovate renovate bot force-pushed the renovate/major-vitest-monorepo branch from 4d19b0f to 385890d Compare October 28, 2025 12:16
@renovate renovate bot changed the title Update vitest monorepo to v4 (major) chore(deps): update vitest monorepo to v4 (major) Oct 28, 2025
@renovate renovate bot force-pushed the renovate/major-vitest-monorepo branch from 385890d to 5998bc2 Compare October 29, 2025 13:35
@renovate renovate bot force-pushed the renovate/major-vitest-monorepo branch from 5998bc2 to 921a37f Compare November 1, 2025 12:11
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Nov 1, 2025
@renovate
Copy link
Contributor Author

renovate bot commented Nov 1, 2025

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@github-actions
Copy link

github-actions bot commented Nov 1, 2025 •
edited
Loading

📚 Website preview is ready!

🔗 Preview URL: https://pr-7509.trilium-homepage.pages.dev
📖 Production URL: https://triliumnotes.org

✅ All checks passed

This preview will be updated automatically with new commits.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Nov 1, 2025
@eliandoran
Copy link
Contributor

eliandoran commented Nov 5, 2025

/gemini review

gemini-code-assist[bot]
gemini-code-assist bot reviewed Nov 5, 2025
View reviewed changes
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the vitest monorepo to version 4, a major upgrade. The changes primarily address breaking changes introduced in vitest v4, including updates to mock implementations to use function instead of arrow functions for constructors, and modifications to the browser testing configuration. The dependency versions are consistently updated across the monorepo. I've found one critical syntax error in a test file that needs to be fixed for the tests to pass. Otherwise, the changes are correct and necessary for the upgrade.

apps/server/src/services/llm/ai_service_manager.spec.ts Outdated
OpenAIService: vi.fn().mockImplementation(function () {
this.isAvailable = vi.fn().mockReturnValue(true);
this.generateChatCompletion = vi.fn();
};
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot Nov 5, 2025

Choose a reason for hiding this comment

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

critical

This semicolon introduces a syntax error. The function body should be closed with a } and the mockImplementation call with a ). This will cause the test to fail.

Suggested change
};
})

Copilot AI and others added 2 commits November 5, 2025 17:46
Initial plan
27cc022
@eliandoran
Fix LLM streaming test race conditions after Vite update
5eb791f 
Added waits for async streaming operations in tests and reduced concurrent request count to 2 for reliability.

Co-authored-by: eliandoran <[email protected]>
Copilot AI and others added 5 commits November 5, 2025 18:16
@eliandoran
Fix syntax errors in LLM service test mocks
18a1984 
Fixed vi.mock() syntax errors that were preventing tests from running.

Co-authored-by: eliandoran <[email protected]>
@eliandoran
Improve test reliability by using vi.waitFor() for async checks
730e2da 
Replaced hardcoded timeouts with vi.waitFor() for better test reliability.

Co-authored-by: eliandoran <[email protected]>
@eliandoran
Complete vi.waitFor() migration for all async streaming tests
993d53e 
Replaced all remaining setTimeout calls with vi.waitFor() for consistency and reliability.

Co-authored-by: eliandoran <[email protected]>
@eliandoran
Fix LLM streaming test race conditions after Vite update (#7629)
dbe51cc
@eliandoran
Merge remote-tracking branch 'origin/main' into renovate/major-vitest…
2154a5e 
…-monorepo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

dependencies PRs or issues relating to dependencies merge-conflicts renovate size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@eliandoran