Merge 49d9b6e into 1d10bba

Author: Feiyang1

packages-exp/firebase-exp/compat/rollup.config.release.js

@@ -137,7 +137,11 @@ const appBuilds = [
           }
         ]
       }),
-      terser()
+      terser({
+        format: {
+          comments: false
+        }
+      })
     ]
   }
 ];
@@ -198,7 +202,11 @@ const componentBuilds = compatPkg.components
               }
             ]
           }),
-          terser()
+          terser({
+            format: {
+              comments: false
+            }
+          })
         ],
         external: ['@firebase/app-compat', '@firebase/app']
       }
@@ -318,7 +326,7 @@ const completeBuilds = [
     plugins: [
       sourcemaps(),
       resolveModule({
-        mainFields: ['lite', 'module', 'main']
+        mainFields: ['lite-esm5', 'esm5', 'module']
       }),
       typescriptPluginUMD,
       json(),

packages-exp/firebase-exp/rollup.config.release.js

@@ -26,6 +26,7 @@ import rollupTypescriptPlugin from 'rollup-plugin-typescript2';
 import sourcemaps from 'rollup-plugin-sourcemaps';
 import typescript from 'typescript';
 import alias from '@rollup/plugin-alias';
+import { terser } from 'rollup-plugin-terser';
 
 // remove -exp from dependencies name
 const deps = Object.keys(pkg.dependencies || {}).map(name =>
@@ -106,7 +107,15 @@ const cdnBuilds = [
       sourcemap: true,
       format: 'es'
     },
-    plugins: [...plugins, typescriptPluginCDN]
+    plugins: [
+      ...plugins,
+      typescriptPluginCDN,
+      terser({
+        format: {
+          comments: false
+        }
+      })
+    ]
   },
   ...pkg.components
     .filter(component => component !== 'app')
@@ -131,6 +140,11 @@ const cdnBuilds = [
               '@firebase/app': FIREBASE_APP_URL,
               '@firebase/installations': '@firebase/installations-exp'
             }
+          }),
+          terser({
+            format: {
+              comments: false
+            }
           })
         ],
         external: [FIREBASE_APP_URL]