Emit a module package file for functions ESM builds (#8517)

Author: dlarocque

packages/functions-compat/package.json

@@ -4,20 +4,20 @@
   "description": "",
   "author": "Firebase <[email protected]> (https://firebase.google.com/)",
   "main": "dist/index.cjs.js",
-  "browser": "dist/index.esm2017.js",
-  "module": "dist/index.esm2017.js",
+  "browser": "dist/esm/index.esm2017.js",
+  "module": "dist/esm/index.esm2017.js",
   "exports": {
     ".": {
       "types": "./dist/src/index.d.ts",
       "node": {
         "require": "./dist/index.cjs.js",
-        "import": "./dist/index.esm2017.js"
+        "import": "./dist/esm/index.esm2017.js"
       },
       "browser": {
         "require": "./dist/index.cjs.js",
-        "import": "./dist/index.esm2017.js"
+        "import": "./dist/esm/index.esm2017.js"
       },
-      "default": "./dist/index.esm2017.js"
+      "default": "./dist/esm/index.esm2017.js"
     },
     "./package.json": "./package.json"
   },

packages/functions-compat/rollup.config.js

@@ -32,21 +32,21 @@ const buildPlugins = [
   json({ preferConst: true })
 ];
 
-const browserBuilds = [
+const builds = [
   {
     input: 'src/index.ts',
     output: {
       file: pkg.browser,
       format: 'es',
       sourcemap: true
     },
-    plugins: buildPlugins,
+    plugins: [...buildPlugins, emitModulePackageFile()],
     external: id => deps.some(dep => id === dep || id.startsWith(`${dep}/`))
   },
   {
     input: 'src/index.ts',
     output: {
-      file: './dist/index.cjs.js',
+      file: pkg.main,
       format: 'cjs',
       sourcemap: true
     },
@@ -55,4 +55,4 @@ const browserBuilds = [
   }
 ];
 
-export default [...browserBuilds];
+export default builds;

packages/functions/package.json

@@ -4,20 +4,20 @@
   "description": "",
   "author": "Firebase <[email protected]> (https://firebase.google.com/)",
   "main": "dist/index.cjs.js",
-  "browser": "dist/index.esm2017.js",
-  "module": "dist/index.esm2017.js",
+  "browser": "dist/esm/index.esm2017.js",
+  "module": "dist/esm/index.esm2017.js",
   "exports": {
     ".": {
       "types": "./dist/functions-public.d.ts",
       "node": {
-        "import": "./dist/index.esm2017.js",
+        "import": "./dist/esm/index.esm2017.js",
         "require": "./dist/index.cjs.js" 
       },
       "browser": {
         "require": "./dist/index.cjs.js",
-        "import": "./dist/index.esm2017.js"
+        "import": "./dist/esm/index.esm2017.js"
       },
-      "default": "./dist/index.esm2017.js"
+      "default": "./dist/esm/index.esm2017.js"
     },
     "./package.json": "./package.json"
   },

packages/functions/rollup.config.js

@@ -34,7 +34,7 @@ const buildPlugins = [
   json({ preferConst: true })
 ];
 
-const browserBuilds = [
+const builds = [
   {
     input: 'src/index.ts',
     output: {
@@ -45,13 +45,14 @@ const browserBuilds = [
     external: id => deps.some(dep => id === dep || id.startsWith(`${dep}/`)),
     plugins: [
       ...buildPlugins,
-      replace(generateBuildTargetReplaceConfig('esm', 2017))
+      replace(generateBuildTargetReplaceConfig('esm', 2017)),
+      emitModulePackageFile()
     ]
   },
   {
     input: 'src/index.ts',
     output: {
-      file: './dist/index.cjs.js',
+      file: pkg.main,
       format: 'cjs',
       sourcemap: true
     },
@@ -63,4 +64,4 @@ const browserBuilds = [
   }
 ];
 
-export default [...browserBuilds];
+export default builds;