fix: allow inlining vite's cached dependencies (#6284)

Author: chriswheeldon-peakon

packages/vite-node/src/externalize.ts

@@ -111,11 +111,6 @@ async function _shouldExternalize(
 
   id = patchWindowsImportPath(id)
 
-  // always externalize Vite deps, they are too big to inline
-  if (options?.cacheDir && id.includes(options.cacheDir)) {
-    return id
-  }
-
   const moduleDirectories = options?.moduleDirectories || ['/node_modules/']
 
   if (matchExternalizePattern(id, moduleDirectories, options?.inline)) {
@@ -125,6 +120,12 @@ async function _shouldExternalize(
     return id
   }
 
+  // Unless the user explicitly opted to inline them, externalize Vite deps.
+  // They are too big to inline by default.
+  if (options?.cacheDir && id.includes(options.cacheDir)) {
+    return id
+  }
+
   const isLibraryModule = moduleDirectories.some(dir => id.includes(dir))
   const guessCJS = isLibraryModule && options?.fallbackCJS
   id = guessCJS ? guessCJSversion(id) || id : id

test/vite-node/test/server.test.ts

@@ -229,3 +229,55 @@ describe('server correctly caches data', () => {
     expect(webFiles).toHaveLength(1)
   })
 })
+
+describe('externalize', () => {
+  describe('by default', () => {
+    test('should externalize vite\'s cached dependencies', async () => {
+      const vnServer = new ViteNodeServer({
+        config: {
+          root: '/',
+          cacheDir: '/node_modules/.vite',
+        },
+      } as any, {})
+
+      const externalize = await vnServer.shouldExternalize('/node_modules/.vite/cached.js')
+      expect(externalize).toBeTruthy()
+    })
+  })
+
+  describe('with server.deps.inline: true', () => {
+    test('should not externalize vite\'s cached dependencies', async () => {
+      const vnServer = new ViteNodeServer({
+        config: {
+          root: '/',
+          cacheDir: '/node_modules/.vite',
+        },
+      } as any, {
+        deps: {
+          inline: true,
+        },
+      })
+
+      const externalize = await vnServer.shouldExternalize('/node_modules/.vite/cached.js')
+      expect(externalize).toBeFalsy()
+    })
+  })
+
+  describe('with server.deps.inline including the cache dir', () => {
+    test('should not externalize vite\'s cached dependencies', async () => {
+      const vnServer = new ViteNodeServer({
+        config: {
+          root: '/',
+          cacheDir: '/node_modules/.vite',
+        },
+      } as any, {
+        deps: {
+          inline: [/node_modules\/\.vite/],
+        },
+      })
+
+      const externalize = await vnServer.shouldExternalize('/node_modules/.vite/cached.js')
+      expect(externalize).toBeFalsy()
+    })
+  })
+})