Skip to content

fix(test): emit NodeNext-compatible provider type specifiers - #2360

Merged
fengmk2 merged 2 commits into
voidzero-dev:mainfrom
eai04191:fix/nodenext-browser-provider-declarations
Aug 6, 2026
Merged

fix(test): emit NodeNext-compatible provider type specifiers#2360
fengmk2 merged 2 commits into
voidzero-dev:mainfrom
eai04191:fix/nodenext-browser-provider-declarations

Conversation

@eai04191

@eai04191 eai04191 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Emit .js extensions for relative shim specifiers in inlined browser-provider declarations so they resolve under NodeNext.
  • Add regression coverage using the public vite-plus/test/browser-playwright types export.

Fixes #2354

Validation

  • pnpm --filter vite-plus test src/__tests__/exports-map.spec.ts
  • vp check
  • Verified pnpm run check and pnpm run typecheck in the public reproduction using a locally packed Vite+ build

byCodex

Co-authored-by: GPT-5.6 Codex <codex@openai.com>
@netlify

netlify Bot commented Aug 6, 2026

Copy link
Copy Markdown

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit f208aba
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/6a7445865a633c00089483bd

@jong-kyung jong-kyung left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 👍

@jong-kyung

Copy link
Copy Markdown
Collaborator

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

Reviewed commit: 4dcbbae384

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@fengmk2 fengmk2 self-assigned this Aug 6, 2026
@fengmk2 fengmk2 added test: e2e Auto run e2e tests test: install-e2e run vite install e2e test test: create-e2e Run `vp create` e2e tests test: sfw labels Aug 6, 2026
@fengmk2
fengmk2 merged commit 0382b54 into voidzero-dev:main Aug 6, 2026
100 checks passed
@eai04191
eai04191 deleted the fix/nodenext-browser-provider-declarations branch August 6, 2026 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test: create-e2e Run `vp create` e2e tests test: e2e Auto run e2e tests test: install-e2e run vite install e2e test test: sfw

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vite-plus/test/browser-playwright is error-typed under NodeNext

3 participants