From 689a2c8cbab0887dba0994dd0e9dc5515ab9d461 Mon Sep 17 00:00:00 2001
From: Fran Dios <frankdiox@gmail.com>
Date: Mon, 30 Aug 2021 09:08:18 +0200
Subject: [PATCH] fix: special handling for ssr.noExternal in mergeConfig
 (#4766)

---
 packages/playground/ssr-webworker/vite.config.js | 13 ++++++++++++-
 packages/vite/src/node/config.ts                 |  2 ++
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/packages/playground/ssr-webworker/vite.config.js b/packages/playground/ssr-webworker/vite.config.js
index d0e80179498685..82fdf7579e9752 100644
--- a/packages/playground/ssr-webworker/vite.config.js
+++ b/packages/playground/ssr-webworker/vite.config.js
@@ -8,5 +8,16 @@ module.exports = {
   ssr: {
     target: 'webworker',
     noExternal: true
-  }
+  },
+  plugins: [
+    {
+      config() {
+        return {
+          ssr: {
+            noExternal: ['this-should-not-replace-the-boolean']
+          }
+        }
+      }
+    }
+  ]
 }
diff --git a/packages/vite/src/node/config.ts b/packages/vite/src/node/config.ts
index 539b26205d0bc6..19274a725d785a 100644
--- a/packages/vite/src/node/config.ts
+++ b/packages/vite/src/node/config.ts
@@ -657,6 +657,8 @@ function mergeConfigRecursively(
       } else if (key === 'assetsInclude' && rootPath === '') {
         merged[key] = [].concat(existing, value)
         continue
+      } else if (key === 'noExternal' && existing === true) {
+        continue
       }
     }