Skip to content

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

Merged
wKovacs64 merged 2 commits intomainfrom
renovate/major-vitest-monorepo
Nov 7, 2025
Merged

chore(deps): update vitest monorepo to v4 (major)#551
wKovacs64 merged 2 commits intomainfrom
renovate/major-vitest-monorepo

Conversation

@renovate
Copy link
Contributor

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

This PR contains the following updates:

Package Change Age Confidence
@vitest/coverage-v8 (source) 3.2.4 -> 4.0.3 age confidence
vitest (source) 3.2.4 -> 4.0.3 age confidence

Release Notes

vitest-dev/vitest (@​vitest/coverage-v8)

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

   🚨 Breaking Changes
   🚀 Features
   🐞 Bug Fixes
   🏎 Performance
    View changes on GitHub

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), 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.

@changeset-bot
Copy link

changeset-bot bot commented Oct 22, 2025 •
edited
Loading

⚠️ No Changeset found

Latest commit: 8994740

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link

coderabbitai bot commented Oct 22, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Oct 22, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (1754bd5) to head (8994740).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##              main      #551    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           20        20            
  Lines          375       131   -244     
  Branches        32        45    +13     
==========================================
- Hits           375       131   -244
Files with missing lines Coverage Δ
src/api/haveibeenpwned/fetch-from-api.ts 100.00% <100.00%> (ø)
src/pwned-password.ts 100.00% <ø> (ø)

... and 18 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@renovate renovate bot force-pushed the renovate/major-vitest-monorepo branch 2 times, most recently from 37bace7 to 4a3d9e9 Compare October 24, 2025 18:48
@wKovacs64
Copy link
Owner

wKovacs64 commented Oct 27, 2025

@cursor can you either update our existing tests or propose new ones to restore our coverage to 100% after updating to Vitest v4?

@cursor
Copy link

cursor bot commented Oct 27, 2025 •
edited
Loading

I ran the suite with coverage, identified two uncovered branches, and added tests. Coverage is now 100% locally.

What I changed

  • Updated src/__tests__/pwned-password.test.ts: added a test that stubs crypto to simulate environments without Web Crypto, asserting pwnedPassword rejects with the expected error.
  • Updated src/api/haveibeenpwned/__tests__/fetch-from-api.test.ts: added a rate-limit test that omits the retry-after header and asserts RateLimitError.retryAfterSeconds is undefined.

Result

  • Coverage: 100% statements/branches/functions/lines across src.
  • No runtime code changes; lints pass.

How to verify

  • Run: npm ci && npm run test:coverage

Open in Cursor Open in Web

Learn more about Cursor Agents

@cursor
Copy link

cursor bot commented Oct 27, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@renovate
Copy link
Contributor Author

renovate bot commented Oct 27, 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.

@wKovacs64 wKovacs64 force-pushed the renovate/major-vitest-monorepo branch from 0586fbf to 878213a Compare November 7, 2025 15:11
@wKovacs64 wKovacs64 force-pushed the renovate/major-vitest-monorepo branch from 878213a to 8994740 Compare November 7, 2025 15:12
@wKovacs64 wKovacs64 merged commit 2deca1e into main Nov 7, 2025
17 checks passed
@wKovacs64 wKovacs64 deleted the renovate/major-vitest-monorepo branch November 7, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@wKovacs64