Repository navigation
Conversation
Upgrade Vitest and its Playwright browser provider to 5.0.5 so both browser projects use the upstream mock interceptor fix instead of the locked 5.0.0 implementation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues are identified for the Vitest dependency update.
No open issues 🎉
What changed in this PR
Updates the repository’s Vitest development dependencies to pick up the upstream mock interceptor fix, addressing startup warnings without adding warning filters or local workarounds.
Changes:
- Raises
vitestand@vitest/browser-playwrightminimum versions to^5.0.5, matching@vitest/browser. - Regenerates the lockfile with aligned versions, removing older duplicate browser and mocker packages.
- Leaves component code and published package behavior unchanged.
| File | Description |
|---|---|
package.json |
Updates Vitest and its Playwright provider to ^5.0.5. |
package-lock.json |
Locks aligned dependencies and updates their transitive packages. |
Review effort: Balanced
No linked repository issue.
Fix the two
vitest:mocks:interceptorwarnings printed when starting Vitest. The lockfile mixedvitestand@vitest/browser-playwright5.0.0 with@vitest/browser5.0.5. Both browser projects therefore loaded the old mock interceptor, which definedconfigureServereven when WebSocket registration was disabled.Align Vitest and its Playwright provider on 5.0.5 and regenerate the lockfile. The upstream fix in vitest-dev/vitest#11377 only attaches that hook when enabled. No warning filters or local plugin workarounds are needed.
Changelog
New
None.
Changed
Update the minimum versions of the
vitestand@vitest/browser-playwrightdevelopment dependencies to^5.0.5, matching the existing browser dependency and deduplicating the older browser/mocker packages.Removed
The two mock interceptor startup warnings. Other warnings remain visible.
Rollout strategy
Development dependency update only; no changeset is needed.
Testing & Reviewing
Reproduced exactly two matching warnings with the original lockfile. Verified zero matching warnings in the complete updated test run.
npm test -- --run— passed: 161 test files, 2,928 tests passed, 3 skipped, 10 todo.npm ls vitest @vitest/browser @vitest/browser-playwright @vitest/mocker --depth=1— passed: aligned 5.0.5 dependencies with no peer dependency errors.npm run build— passed.npm run type-check— passed.npm run lint— passed after rerunning once the build completed; the initial concurrent run encountered a temporary build config file that had already been removed.npm run lint:css— passed.npm run format:diff— passed.npx prettier --check package.json package-lock.json— passed.npm run formatandnpm run lint:fix— passed before committing; no unrelated changes.git diff --check— passed.Installed the missing Playwright Chromium runtime to execute the browser tests. No component or visual changes; visual regression tests were not run.