Don't use interface types (#3770)

Author: schmidt-sebastian

packages/firestore/exp/dependencies.json

@@ -22,7 +22,7 @@ registerFirestore();
 export { FieldPath, documentId } from '../lite/src/api/field_path';
 
 export {
-  Firestore as FirebaseFirestore,
+  FirebaseFirestore,
   initializeFirestore,
   getFirestore,
   enableIndexedDbPersistence,
@@ -31,14 +31,19 @@ export {
   waitForPendingWrites,
   disableNetwork,
   enableNetwork,
-  terminate
+  terminate,
+  Settings
 } from './src/api/database';
 
 export {
+  DocumentChange,
   DocumentSnapshot,
   QueryDocumentSnapshot,
   QuerySnapshot,
-  snapshotEqual
+  snapshotEqual,
+  SnapshotOptions,
+  FirestoreDataConverter,
+  DocumentChangeType
 } from './src/api/snapshot';
 
 export { SnapshotMetadata } from '../src/api/database';
@@ -59,7 +64,12 @@ export {
   limit,
   limitToLast,
   where,
-  orderBy
+  orderBy,
+  SetOptions,
+  QueryConstraintType,
+  DocumentData,
+  UpdateData,
+  OrderByDirection
 } from '../lite/src/api/reference';
 
 export { runTransaction, Transaction } from './src/api/transaction';
@@ -88,7 +98,7 @@ export {
   serverTimestamp
 } from '../lite/src/api/field_value';
 
-export { setLogLevel } from '../src/util/log';
+export { setLogLevel, LogLevel } from '../src/util/log';
 
 export { Bytes } from '../lite/src/api/bytes';
 
@@ -101,3 +111,9 @@ export { GeoPoint } from '../src/api/geo_point';
 export { Timestamp } from '../src/api/timestamp';
 
 export { refEqual, queryEqual } from '../lite/src/api/reference';
+
+export { SnapshotListenOptions } from './src/api/reference';
+
+export { CACHE_SIZE_UNLIMITED } from '../src/api/database';
+
+export { FirestoreErrorCode, FirestoreError } from '../src/util/error';

packages/firestore/exp/index.ts

@@ -18,7 +18,7 @@
 import { _registerComponent, registerVersion } from '@firebase/app-exp';
 import { Component, ComponentType } from '@firebase/component';
 
-import { Firestore } from './src/api/database';
+import { FirebaseFirestore } from './src/api/database';
 import { version } from '../package.json';
 
 export function registerFirestore(): void {
@@ -27,7 +27,7 @@ export function registerFirestore(): void {
       'firestore-exp',
       container => {
         const app = container.getProvider('app-exp').getImmediate()!;
-        return ((app, auth) => new Firestore(app, auth))(
+        return ((app, auth) => new FirebaseFirestore(app, auth))(
           app,
           container.getProvider('auth-internal')
         );

packages/firestore/exp/register.ts

@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-import { Firestore } from './database';
+import { FirebaseFirestore } from './database';
 import { PersistenceSettings } from '../../../src/core/firestore_client';
 import {
   MemoryOfflineComponentProvider,
@@ -45,16 +45,16 @@ export const LOG_TAG = 'ComponentProvider';
 // Instance maps that ensure that only one component provider exists per
 // Firestore instance.
 const offlineComponentProviders = new Map<
-  Firestore,
+  FirebaseFirestore,
   Promise<OfflineComponentProvider>
 >();
 const onlineComponentProviders = new Map<
-  Firestore,
+  FirebaseFirestore,
   Promise<OnlineComponentProvider>
 >();
 
 export async function setOfflineComponentProvider(
-  firestore: Firestore,
+  firestore: FirebaseFirestore,
   persistenceSettings: PersistenceSettings,
   offlineComponentProvider: OfflineComponentProvider
 ): Promise<void> {
@@ -83,7 +83,7 @@ export async function setOfflineComponentProvider(
 }
 
 export async function setOnlineComponentProvider(
-  firestore: Firestore,
+  firestore: FirebaseFirestore,
   onlineComponentProvider: OnlineComponentProvider
 ): Promise<void> {
   const onlineDeferred = new Deferred<OnlineComponentProvider>();
@@ -112,7 +112,7 @@ export async function setOnlineComponentProvider(
 }
 
 function getOfflineComponentProvider(
-  firestore: Firestore
+  firestore: FirebaseFirestore
 ): Promise<OfflineComponentProvider> {
   firestore._queue.verifyOperationInProgress();
 
@@ -129,7 +129,7 @@ function getOfflineComponentProvider(
 }
 
 function getOnlineComponentProvider(
-  firestore: Firestore
+  firestore: FirebaseFirestore
 ): Promise<OnlineComponentProvider> {
   firestore._queue.verifyOperationInProgress();
 
@@ -144,19 +144,25 @@ function getOnlineComponentProvider(
 // Note: These functions cannot be `async` since we want to throw an exception
 // when Firestore is terminated (via `getOnlineComponentProvider()`).
 
-export function getSyncEngine(firestore: Firestore): Promise<SyncEngine> {
+export function getSyncEngine(
+  firestore: FirebaseFirestore
+): Promise<SyncEngine> {
   return getOnlineComponentProvider(firestore).then(
     components => components.syncEngine
   );
 }
 
-export function getRemoteStore(firestore: Firestore): Promise<RemoteStore> {
+export function getRemoteStore(
+  firestore: FirebaseFirestore
+): Promise<RemoteStore> {
   return getOnlineComponentProvider(firestore).then(
     components => components.remoteStore
   );
 }
 
-export function getEventManager(firestore: Firestore): Promise<EventManager> {
+export function getEventManager(
+  firestore: FirebaseFirestore
+): Promise<EventManager> {
   return getOnlineComponentProvider(firestore).then(components => {
     const eventManager = components.eventManager;
     eventManager.onListen = syncEngineListen.bind(null, components.syncEngine);
@@ -168,13 +174,17 @@ export function getEventManager(firestore: Firestore): Promise<EventManager> {
   });
 }
 
-export function getPersistence(firestore: Firestore): Promise<Persistence> {
+export function getPersistence(
+  firestore: FirebaseFirestore
+): Promise<Persistence> {
   return getOfflineComponentProvider(firestore).then(
     components => components.persistence
   );
 }
 
-export function getLocalStore(firestore: Firestore): Promise<LocalStore> {
+export function getLocalStore(
+  firestore: FirebaseFirestore
+): Promise<LocalStore> {
   return getOfflineComponentProvider(firestore).then(
     provider => provider.localStore
   );
@@ -184,7 +194,9 @@ export function getLocalStore(firestore: Firestore): Promise<LocalStore> {
  * Removes all components associated with the provided instance. Must be called
  * when the Firestore instance is terminated.
  */
-export async function removeComponents(firestore: Firestore): Promise<void> {
+export async function removeComponents(
+  firestore: FirebaseFirestore
+): Promise<void> {
   const onlineComponentProviderPromise = onlineComponentProviders.get(
     firestore
   );