Skip to content

fix(cli): dynamically scan root .d.ts files from @vitest/browser#944

Merged
fengmk2 merged 2 commits intovoidzero-dev:mainfrom
kazupon:fix/includ-vitest-browser-fully
Mar 16, 2026
Merged

fix(cli): dynamically scan root .d.ts files from @vitest/browser#944
fengmk2 merged 2 commits intovoidzero-dev:mainfrom
kazupon:fix/includ-vitest-browser-fully

Conversation

@kazupon
Copy link
Contributor

@kazupon kazupon commented Mar 16, 2026

Summary

  • Replace hardcoded list of root .d.ts files with dynamic readdir scan for @vitest/browser package
  • This ensures newly added type definition files (like aria-role.d.ts, utils.d.ts) are automatically included without requiring manual updates to the build script

related #909 in the TODO.

Test ways

  • Run node --import @oxc-node/core/register ./build.ts in packages/test and verify only @vitest/browser root .d.ts files are copied (aria-role.d.ts, context.d.ts, jest-dom.d.ts, matchers.d.ts, utils.d.ts)
  • Verify vp pack succeeds in a library project that imports UserConfig from vite-plus

@netlify
Copy link

netlify bot commented Mar 16, 2026

👷 Deploy Preview for viteplus-preview processing.

Name Link
🔨 Latest commit 7569be0
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69b7a4b9fc27e10008626809

@netlify
Copy link

netlify bot commented Mar 16, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit 6941b1e
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69b7a60ba567a30008f886b4

@kazupon kazupon changed the title fix(cli): dynamically scan root .d.ts files from @vitest/browser fix(cli): dynamically scan root .d.ts files from @vitest/browser Mar 16, 2026
@fengmk2 fengmk2 added the test: e2e Auto run e2e tests label Mar 16, 2026
@fengmk2
Copy link
Member

fengmk2 commented Mar 16, 2026

@kazupon Thanks again! 💯

@fengmk2 fengmk2 merged commit bf2bbce into voidzero-dev:main Mar 16, 2026
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test: e2e Auto run e2e tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants