test: respect commonjs options in playgrounds (#13273)

vitejs · May 22, 2023 · 19e8c68 · 19e8c68
1 parent 8013a66
commit 19e8c68
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/packages/vite/src/node/config.ts b/packages/vite/src/node/config.ts
@@ -457,7 +457,12 @@ export async function resolveConfig(
   const userPlugins = [...prePlugins, ...normalPlugins, ...postPlugins]
   config = await runConfigHook(config, userPlugins, configEnv)
 
-  if (process.env.VITE_TEST_WITHOUT_PLUGIN_COMMONJS) {
+  // If there are custom commonjsOptions, don't force optimized deps for this test
+  // even if the env var is set as it would interfere with the playground specs.
+  if (
+    !config.build?.commonjsOptions &&
+    process.env.VITE_TEST_WITHOUT_PLUGIN_COMMONJS
+  ) {
     config = mergeConfig(config, {
       optimizeDeps: { disabled: false },
       ssr: { optimizeDeps: { disabled: false } },