fix(plugin): don't merge the entire user vite.config.js

yext · Sep 17, 2024 · 4291e3b · 4291e3b
1 parent 1b9de9e
commit 4291e3b
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/packages/pages/src/vite-plugin/serverless-functions/plugin.ts b/packages/pages/src/vite-plugin/serverless-functions/plugin.ts
@@ -8,10 +8,7 @@ import { processEnvVariables } from "../../util/processEnvVariables.js";
 import { FunctionMetadataParser } from "../../common/src/function/internal/functionMetadataParser.js";
 import { nodePolyfills } from "vite-plugin-node-polyfills";
 import pc from "picocolors";
-import {
-  removePluginFromViteConfig,
-  scopedViteConfigPath,
-} from "../../util/viteConfig.js";
+import { scopedViteConfigPath } from "../../util/viteConfig.js";
 
 export const buildServerlessFunctions = async (
   projectStructure: ProjectStructure
@@ -98,7 +95,13 @@ export const buildServerlessFunctions = async (
     };
     await build(
       mergeConfig(
-        removePluginFromViteConfig(viteConfig.default),
+        {
+          build: {
+            rollupOptions: {
+              external: viteConfig.default.build.rollupOptions.external,
+            },
+          },
+        },
         serverlessFunctionBuildConfig
       )
     );