feat(dev-server): don't deepmerge exportConditions. allow user config to fully replace the default

michaelwarren1106 · michaelwarren1106 · commit d9996d2dc2e8 · 2023-10-06T14:54:01.000-04:00
diff --git a/.changeset/tidy-spoons-work.md b/.changeset/tidy-spoons-work.md
@@ -0,0 +1,5 @@
+---
+'@web/dev-server': patch
+---
+
+Fix an issue where the nodeResolve plugin wasn't accepting user configuration correctly
diff --git a/packages/dev-server/demo/export-conditions/config.mjs b/packages/dev-server/demo/export-conditions/config.mjs
@@ -0,0 +1,10 @@
+import { fileURLToPath } from 'url';
+import { resolve } from 'path';
+
+export default {
+  rootDir: resolve(fileURLToPath(import.meta.url), '..', '..', '..'),
+  appIndex: '/demo/export-conditions/index.html',
+  nodeResolve: {
+    exportConditions: ['default']
+  },
+};
diff --git a/packages/dev-server/demo/export-conditions/index.html b/packages/dev-server/demo/export-conditions/index.html
@@ -0,0 +1,31 @@
+<html>
+  <body>
+    <img width="100" src="../logo.png" />
+
+    <h1>Node resolve demo</h1>
+    <p>A demo which resolves bare module imports</p>
+
+    <div id="test"></div>
+
+    <script type="module">
+      // inline bare modules are resolved
+      import { render, html } from 'lit-html';
+
+      window.__inlineNodeResolve = !!render && !!html;
+    </script>
+
+    <script type="module">
+
+      window.__tests = {
+        // lit-html only adds this global in development mode
+        // so when the exportCondition is overwritten, it'll be undefined
+        prodExport: typeof window.litIssuedWarnings === 'undefined',
+      };
+      document.getElementById('test').innerHTML = `<pre>${JSON.stringify(
+        window.__tests,
+        null,
+        2,
+      )}</pre>`;
+    </script>
+  </body>
+</html>
diff --git a/packages/dev-server/src/plugins/nodeResolvePlugin.ts b/packages/dev-server/src/plugins/nodeResolvePlugin.ts
@@ -14,12 +14,14 @@ export function nodeResolvePlugin(
       extensions: ['.mjs', '.js', '.cjs', '.jsx', '.json', '.ts', '.tsx'],
       moduleDirectories: ['node_modules', 'web_modules'],
       // allow resolving polyfills for nodejs libs
-      preferBuiltins: false,
-      exportConditions: ['development'],
+      preferBuiltins: false
     },
     userOptionsObject,
   );
 
+  // use user config exportConditions if present. otherwise use ['development']
+  options.exportConditions = userOptionsObject.exportConditions || ['development'];
+
   return rollupAdapter(
     nodeResolve(options),
     { preserveSymlinks },
diff --git a/packages/dev-server/test/integration.test.mjs b/packages/dev-server/test/integration.test.mjs
@@ -33,6 +33,10 @@ const testCases = [
     name: 'syntax',
     tests: ['stage4', 'inlineStage4', 'importMeta', 'staticImports', 'dynamicImports'],
   },
+  {
+    name: 'export-conditions',
+    tests: ['prodExport'],
+  },
 ];
 
 describe('integration tests', () => {

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'@web/dev-server': patch
 +---
++
 +Fix an issue where the nodeResolve plugin wasn't accepting user configuration correctly