Skip to content

fix(cli): declare playwright peer for copied provider types - #2865

Open
liangmiQwQ wants to merge 2 commits into
voidzero-dev:mainfrom
liangmiQwQ:liang/codex/fix-browser-provider-peers
Open

liangmiQwQ wants to merge 2 commits into
voidzero-dev:mainfrom
liangmiQwQ:liang/codex/fix-browser-provider-peers

Conversation

@liangmiQwQ

Copy link
Copy Markdown
Collaborator

Close #2854

This PR adds playwright as an optional peer of vite-plus, matching the upstream provider's * range. The copied provider declarations import Playwright types directly, so isolated dependency layouts need this peer to preserve option type checking even with skipLibCheck enabled.

🤖 Generated with Codex

@liangmiQwQ
liangmiQwQ marked this pull request as ready for review October 2, 2026 00:28
@liangmiQwQ
liangmiQwQ requested review from Brooooooklyn and fengmk2 and removed request for Brooooooklyn October 2, 2026 02:53
fengmk2 pushed a commit that referenced this pull request Oct 3, 2026
The Node API snapshot can fail when Vitest's JSON and blob reporters
finish writing in a different order, as seen in [CI for
#2865](https://github.com/voidzero-dev/vite-plus/actions/runs/36882680402/job/110442181616).
Vitest invokes reporters concurrently, so their completion messages have
no fixed order.

Capture this runner's stdout through Vitest's public option and sort the
lines before printing them for the snapshot. All completion messages and
existing report-content assertions remain checked.

🤖 Generated with Codex

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vite-plus/test/browser-playwright types import playwright, which vite-plus does not declare, so provider options go unchecked under pnpm

1 participant