Make Firestore-mangled build the only Firestore build (#2640)

Author: schmidt-sebastian

integration/firestore/gulpfile.js

@@ -37,7 +37,7 @@ function copyTests() {
   const firebaseAppSdk = 'firebase/app/dist/index.esm.js';
   const firebaseFirestoreSdk = resolve(
     __dirname,
-    '../../packages/firestore/dist/index.esm.min.js'
+    '../../packages/firestore/dist/index.esm.js'
   );
   return gulp
     .src(

packages/firebase/firestore/index.min.ts

@@ -147,13 +147,6 @@ const componentBuilds = pkg.components
   })
   .reduce((a, b) => a.concat(b), []);
 
-const firestoreMinifiedBuild = {
-  input: `firestore/index.min.ts`,
-  output: createUmdOutputConfig(`firebase-firestore.min.js`),
-  plugins: [...plugins, uglify()],
-  external: ['@firebase/app']
-};
-
 /**
  * Complete Package Builds
  */
@@ -267,9 +260,4 @@ const completeBuilds = [
   }
 ];
 
-export default [
-  ...appBuilds,
-  ...componentBuilds,
-  firestoreMinifiedBuild,
-  ...completeBuilds
-];
+export default [...appBuilds, ...componentBuilds, ...completeBuilds];

packages/firebase/rollup.config.js

@@ -1,5 +1,6 @@
 {
   "externs" :  [
+    "node_modules/@types/node/base.d.ts",
     "node_modules/typescript/lib/lib.es5.d.ts",
     "node_modules/typescript/lib/lib.dom.d.ts",
     "node_modules/typescript/lib/lib.es2015.promise.d.ts",

packages/firestore/externs.json

@@ -25,11 +25,8 @@
   },
   "main": "dist/index.node.cjs.js",
   "browser": "dist/index.cjs.js",
-  "browserMinified": "dist/index.cjs.min.js",
   "module": "dist/index.esm.js",
-  "moduleMinified": "dist/index.esm.min.js",
   "esm2017": "dist/index.esm2017.js",
-  "esm2017Minified": "dist/index.esm2017.min.js",
   "license": "Apache-2.0",
   "files": [
     "dist"

packages/firestore/package.json

@@ -64,14 +64,6 @@ const terserOptions = {
  * ES5 Builds
  */
 const es5BuildPlugins = [
-  typescriptPlugin({
-    typescript,
-    cacheRoot: './.cache/es5/'
-  }),
-  json()
-];
-
-const es5MinifiedBuildPlugins = [
   typescriptPlugin({
     typescript,
     transformers,
@@ -114,35 +106,13 @@ const es5Builds = [
     ],
     plugins: es5BuildPlugins,
     external: id => deps.some(dep => id === dep || id.startsWith(`${dep}/`))
-  },
-  {
-    input: 'index.ts',
-    output: [
-      { file: pkg.browserMinified, format: 'cjs', sourcemap: true },
-      { file: pkg.moduleMinified, format: 'es', sourcemap: true }
-    ],
-    plugins: es5MinifiedBuildPlugins,
-    external: id => deps.some(dep => id === dep || id.startsWith(`${dep}/`))
   }
 ];
 
 /**
  * ES2017 Builds
  */
 const es2017BuildPlugins = [
-  typescriptPlugin({
-    typescript,
-    tsconfigOverride: {
-      compilerOptions: {
-        target: 'es2017'
-      }
-    },
-    cacheRoot: './.cache/es2017/'
-  }),
-  json({ preferConst: true })
-];
-
-const es2017MinifiedBuildPlugins = [
   typescriptPlugin({
     typescript,
     tsconfigOverride: {
@@ -170,16 +140,6 @@ const es2017Builds = [
     },
     plugins: es2017BuildPlugins,
     external: id => deps.some(dep => id === dep || id.startsWith(`${dep}/`))
-  },
-  {
-    input: 'index.ts',
-    output: {
-      file: pkg.esm2017Minified,
-      format: 'es',
-      sourcemap: true
-    },
-    plugins: es2017MinifiedBuildPlugins,
-    external: id => deps.some(dep => id === dep || id.startsWith(`${dep}/`))
   }
 ];