fix(vite-node): don't externalize "dist" by default

vitest-dev · May 25, 2023 · 3381bf2 · 3381bf2
1 parent 41e11da
commit 3381bf2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/docs/config/index.md b/docs/config/index.md
@@ -136,7 +136,7 @@ You will not be able to edit your `node_modules` code for debugging, since the c
 #### deps.external
 
 - **Type:** `(string | RegExp)[]`
-- **Default:** `['**/node_modules/**', '**/dist/**']`
+- **Default:** `['**/node_modules/**']`
 
 Externalize means that Vite will bypass the package to native Node. Externalized dependencies will not be applied Vite's transformers and resolvers, so they do not support HMR on reload. Typically, packages under `node_modules` are externalized.
 

diff --git a/packages/vite-node/src/externalize.ts b/packages/vite-node/src/externalize.ts
@@ -123,8 +123,7 @@ async function _shouldExternalize(
   if (matchExternalizePattern(id, depsExternal))
     return id
 
-  const isDist = id.includes('/dist/')
-  if ((isNodeModule || isDist) && await isValidNodeImport(id))
+  if (isNodeModule && await isValidNodeImport(id))
     return id
 
   return false