fix firebase packaging (#5368)

* fix firebase packaging

* update index.d.ts

* interop fixes

* remove redundant build

* fix script

Author: Feiyang1

common/api-review/database.api.md

@@ -44,6 +44,8 @@ export class DataSnapshot {
     val(): any;
 }
 
+export { EmulatorMockTokenOptions }
+
 // @public
 export function enableLogging(enabled: boolean, persistent?: boolean): any;
 

common/api-review/storage.api.md

@@ -29,6 +29,8 @@ export function _dataFromString(format: StringFormat, stringData: string): Strin
 // @public
 export function deleteObject(ref: StorageReference): Promise<void>;
 
+export { EmulatorMockTokenOptions }
+
 // @internal (undocumented)
 export class _FbsBlob {
     constructor(data: Blob | Uint8Array | ArrayBuffer, elideCopy?: boolean);

config/webpack.test.js

@@ -82,7 +82,7 @@ module.exports = {
       {
         test: /\.js$/,
         include: function (modulePath) {
-          const match = /node_modules\/@firebase.*/.test(modulePath) && !modulePath.includes('-compat');
+          const match = /node_modules\/@firebase.*/.test(modulePath);
           return match;
         },
         use: {

packages/database-compat/package.json

@@ -21,7 +21,7 @@
     "test:ci": "node ../../scripts/run_tests_in_ci.js -s test",
     "test:browser": "karma start --single-run",
     "test:node": "TS_NODE_FILES=true TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'test/{,!(browser)/**/}*.test.ts' --file src/index.node.ts --config ../../config/mocharc.node.js",
-    "add-compat-overloads": "ts-node-script ../../scripts/exp/create-overloads.ts -i ../database/dist/public.d.ts -o dist/database-compat/src/index.d.ts -a -r FirebaseDatabase:types.FirebaseDatabase -r Query:types.Query -r Reference:types.Reference -r FirebaseApp:FirebaseAppCompat --moduleToEnhance @firebase/database"
+    "add-compat-overloads": "ts-node-script ../../scripts/exp/create-overloads.ts -i ../database/dist/public.d.ts -o dist/database-compat/src/index.d.ts -a -r Database:types.FirebaseDatabase -r Query:types.Query -r DatabaseReference:types.Reference -r FirebaseApp:FirebaseAppCompat --moduleToEnhance @firebase/database"
   },
   "peerDependencies": {
     "@firebase/app-compat": "0.x"

packages/database/src/api.ts

@@ -17,6 +17,7 @@
 
 export {
   Database,
+  EmulatorMockTokenOptions,
   enableLogging,
   getDatabase,
   goOffline,

packages/database/src/api/Database.ts

@@ -49,6 +49,7 @@
 
  import { ReferenceImpl } from './Reference_impl';
 
+ export { EmulatorMockTokenOptions } from '@firebase/util';
  /**
   * This variable is also defined in the firebase Node.js Admin SDK. Before
   * modifying this definition, consult the definition in:

packages/firebase/analytics/package.json

@@ -2,6 +2,6 @@
   "name": "firebase/analytics",
   "main": "dist/index.cjs.js",
   "browser": "dist/index.esm.js",
-  "module": "dist/index.mjs",
+  "module": "dist/index.esm.js",
   "typings": "dist/analytics/index.d.ts"
 }

packages/firebase/app-check/package.json

@@ -2,6 +2,6 @@
   "name": "firebase/app-check",
   "main": "dist/index.cjs.js",
   "browser": "dist/index.esm.js",
-  "module": "dist/index.mjs",
+  "module": "dist/index.esm.js",
   "typings": "dist/app-check/index.d.ts"
 }

packages/firebase/app/package.json

@@ -2,6 +2,6 @@
   "name": "firebase/app",
   "main": "dist/index.cjs.js",
   "browser": "dist/index.esm.js",
-  "module": "dist/index.mjs",
+  "module": "dist/index.esm.js",
   "typings": "dist/app/index.d.ts"
 }

packages/firebase/auth/cordova/package.json

@@ -2,6 +2,6 @@
   "name": "firebase/auth/cordova",
   "main": "dist/index.cjs.js",
   "browser": "dist/index.esm.js",
-  "module": "dist/index.mjs",
+  "module": "dist/index.esm.js",
   "typings": "dist/auth/cordova/index.d.ts"
 }