fix: externalize only in ssr

vitejs · May 27, 2022 · 2923000 · 2923000
1 parent ef6e8ea
commit 2923000
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/packages/vite/src/node/plugins/index.ts b/packages/vite/src/node/plugins/index.ts
@@ -64,7 +64,7 @@ export async function resolvePlugins(
       asSrc: true,
       getDepsOptimizer: () => getDepsOptimizer(config),
       shouldExternalize:
-        isBuild && config.ssr?.target !== 'node-cjs'
+        isBuild && config.build.ssr && config.ssr?.target !== 'node-cjs'
           ? (id) => shouldExternalizeForSSR(id, config)
           : undefined
     }),

diff --git a/packages/vite/src/node/plugins/ssrRequireHook.ts b/packages/vite/src/node/plugins/ssrRequireHook.ts
@@ -12,6 +12,7 @@ import { arraify } from '../utils'
 export function ssrRequireHookPlugin(config: ResolvedConfig): Plugin | null {
   if (
     config.command !== 'build' ||
+    !config.build.ssr ||
     !config.resolve.dedupe?.length ||
     config.ssr?.noExternal === true ||
     config.ssr?.target !== 'node-cjs' ||