Remove bundles from public interfaces.

Author: wu-hui

integration/firestore/firebase_export.ts

@@ -55,16 +55,5 @@ const Timestamp = firebase.firestore.Timestamp;
 const GeoPoint = firebase.firestore.GeoPoint;
 const FieldValue = firebase.firestore.FieldValue;
 const Blob = firebase.firestore.Blob;
-const loadBundle = firebase.firestore.loadBundle;
-const namedQuery = firebase.firestore.namedQuery;
 
-export {
-  Firestore,
-  FieldValue,
-  FieldPath,
-  Timestamp,
-  Blob,
-  GeoPoint,
-  loadBundle,
-  namedQuery
-};
+export { Firestore, FieldValue, FieldPath, Timestamp, Blob, GeoPoint };

integration/firestore/firebase_export_memory.ts

@@ -55,16 +55,5 @@ const Timestamp = firebase.firestore.Timestamp;
 const GeoPoint = firebase.firestore.GeoPoint;
 const FieldValue = firebase.firestore.FieldValue;
 const Blob = firebase.firestore.Blob;
-const loadBundle = firebase.firestore.loadBundle;
-const namedQuery = firebase.firestore.namedQuery;
 
-export {
-  Firestore,
-  FieldValue,
-  FieldPath,
-  Timestamp,
-  Blob,
-  GeoPoint,
-  loadBundle,
-  namedQuery
-};
+export { Firestore, FieldValue, FieldPath, Timestamp, Blob, GeoPoint };

packages/firestore-types/index.d.ts

@@ -99,16 +99,6 @@ export class FirebaseFirestore {
   INTERNAL: { delete: () => Promise<void> };
 }
 
-export function loadBundle(
-  db: FirebaseFirestore,
-  bundleData: ArrayBuffer | ReadableStream<ArrayBuffer> | string
-): LoadBundleTask;
-
-export function namedQuery(
-  db: FirebaseFirestore,
-  name: string
-): Promise<Query<DocumentData> | null>;
-
 export interface LoadBundleTask {
   onProgress(
     next?: (progress: LoadBundleTaskProgress) => any,

packages/firestore/exp/test/shim.ts

@@ -55,10 +55,9 @@ import {
   validateSetOptions
 } from '../../src/util/input_validation';
 import { Compat } from '../../src/compat/compat';
-import { Firestore, loadBundle, namedQuery } from '../../src/api/database';
+import { Firestore } from '../../src/api/database';
 
 export { GeoPoint, Timestamp } from '../index';
-export { loadBundle, namedQuery };
 
 /* eslint-disable @typescript-eslint/no-explicit-any */
 

packages/firestore/register-module.ts

@@ -36,8 +36,6 @@ declare module '@firebase/app-types' {
       Transaction: typeof types.Transaction;
       WriteBatch: typeof types.WriteBatch;
       setLogLevel: typeof types.setLogLevel;
-      loadBundle: typeof types.loadBundle;
-      namedQuery: typeof types.namedQuery;
     };
   }
   interface FirebaseApp {

packages/firestore/src/api/database.ts

@@ -30,9 +30,7 @@ import {
   firestoreClientGetDocumentViaSnapshotListener,
   firestoreClientListen,
   firestoreClientTransaction,
-  firestoreClientWrite,
-  firestoreClientLoadBundle,
-  firestoreClientGetNamedQuery
+  firestoreClientWrite
 } from '../core/firestore_client';
 import {
   Bound,
@@ -143,7 +141,6 @@ import {
 import { newUserDataReader } from '../../lite/src/api/reference';
 import { makeDatabaseInfo } from '../../lite/src/api/database';
 import { DEFAULT_HOST } from '../../lite/src/api/components';
-import { LoadBundleTask } from './bundle';
 
 /**
  * Constant used to indicate the LRU garbage collection should be disabled.
@@ -427,36 +424,6 @@ export function setLogLevel(level: PublicLogLevel): void {
   setClientLogLevel(level);
 }
 
-export function loadBundle(
-  db: Firestore,
-  bundleData: ArrayBuffer | ReadableStream<Uint8Array> | string
-): LoadBundleTask {
-  const resultTask = new LoadBundleTask();
-  // eslint-disable-next-line @typescript-eslint/no-floating-promises
-  firestoreClientLoadBundle(
-    ensureFirestoreConfigured(db._delegate),
-    bundleData,
-    resultTask
-  );
-  return resultTask;
-}
-
-export function namedQuery(
-  db: Firestore,
-  name: string
-): Promise<PublicQuery | null> {
-  return firestoreClientGetNamedQuery(
-    ensureFirestoreConfigured(db._delegate),
-    name
-  ).then(namedQuery => {
-    if (!namedQuery) {
-      return null;
-    }
-
-    return new Query(namedQuery.query, db, null);
-  });
-}
-
 /**
  * A reference to a transaction.
  */

packages/firestore/src/config.ts

@@ -30,8 +30,6 @@ import {
   QuerySnapshot,
   Transaction,
   WriteBatch,
-  loadBundle,
-  namedQuery,
   setLogLevel
 } from './api/database';
 import { Blob } from './api/blob';
@@ -56,8 +54,6 @@ const firestoreNamespace = {
   FieldPath,
   FieldValue,
   setLogLevel,
-  loadBundle,
-  namedQuery,
   CACHE_SIZE_UNLIMITED
 };