esbuild: prioritize "module" over "browser" (#37052)

* Update esbuild mainFields for better tree shaking * also update test runner
ampproject · Nov 24, 2021 · b78e878 · b78e878
1 parent 4345078
commit b78e878
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/build-system/tasks/helpers.js b/build-system/tasks/helpers.js
@@ -456,6 +456,7 @@ async function esbuildCompile(srcDir, srcFilename, destDir, options) {
         incremental: !!options.watch,
         logLevel: 'silent',
         external: options.externalDependencies,
+        mainFields: ['module', 'browser', 'main'],
         write: false,
       });
     } else {

diff --git a/build-system/tasks/runtime-test/runtime-test-base.js b/build-system/tasks/runtime-test/runtime-test-base.js
@@ -236,6 +236,7 @@ class RuntimeTestConfig {
         'process.env.NODE_ENV': '"test"',
       },
       plugins: [importPathPlugin, babelPlugin],
+      mainFields: ['module', 'browser', 'main'],
       sourcemap: 'inline',
     };
   }