Merge firestore bundles implementation without exposing public APIs #3594

wu-hui · 2020-08-10T19:43:11Z

No description provided.

…3073) * Manual copy proto and create bundle_proto.d.ts * IndexedDb schema change to introduce bundle object stores. * Renaming interfaces without leading I * Reordering imports * Address comments * Add totalBytes and totalDocuments

Implement a bundle reader to read from a underlying readable stream lazily, and parse the content to proto objects.

# Conflicts: # integration/messaging/test/utils/openNewTab.js # packages/firestore/src/core/component_provider.ts # packages/firestore/src/local/indexeddb_persistence.ts # packages/firestore/src/local/local_serializer.ts # packages/firestore/src/platform/platform.ts # packages/firestore/src/platform_browser/browser_platform.ts # packages/firestore/src/platform_node/node_platform.ts # packages/firestore/src/remote/serializer.ts # packages/firestore/test/unit/local/persistence_test_helpers.ts # packages/firestore/test/unit/specs/spec_test_components.ts # packages/firestore/test/util/test_platform.ts

* Renaming interfaces without leading I * Initial commit of bundle reading - for web only. * Tests only run when it is not Node. * Fix redundant imports * Fix missing textencoder * Remove generator. * Support bundle reader for Node * Fix rebase errors. * Remote 'only' * Merge branch 'wuandy/Bundles' into wuandy/BundleReaderNode # Conflicts: # packages/firestore/src/util/bundle_reader.ts # packages/firestore/test/unit/util/bundle.test.ts * Added more comments, and more tests for Node. * Implement BundleCache. * Add applyBundleDocuments to local store. * Add rest of bundle service to localstore Fix change buffer bug * Simplify change buffer get read time logic. * Fix lint errors * Add comments. * Change localstore to check for newer bundle directly. * A little more comments. * Address comments. * Address comments 2 * Make it tree-shakeable. * More comments addressing. * Another around of comments

* Fix firestore_bundle_proto rollup issue * Create lemon-steaks-draw.md

Still issues with shim.ts # Conflicts: # packages/firestore/src/core/component_provider.ts # packages/firestore/src/core/firestore_client.ts # packages/firestore/src/core/sync_engine.ts # packages/firestore/src/local/local_store.ts # packages/firestore/src/remote/serializer.ts # packages/firestore/test/unit/local/persistence_test_helpers.ts # packages/firestore/test/util/helpers.ts

* Make loadBundle work with exp build * Use legacy.LoadBundleTask

# Conflicts: # packages/firestore/exp/src/api/database.ts # packages/firestore/src/core/component_provider.ts # packages/firestore/src/core/firestore_client.ts # packages/firestore/src/core/sync_engine.ts # packages/firestore/src/platform/serializer.ts

# Conflicts: # packages/firestore/exp/src/api/database.ts

changeset-bot · 2020-08-10T19:43:14Z

🦋 Changeset detected

Latest commit: 205d1c8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages

Name	Type
@firebase/firestore
firebase	Patch
@firebase/rules-unit-testing	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

* Update query-document mapping from bundles. * Only update mapping when query read time is updated. * Fix lint errors * Better code structure. * Use serializer constructed from firestore client. * Don't inline await

# Conflicts: # packages/firestore/exp/src/api/database.ts # packages/firestore/scripts/run-tests.js # packages/firestore/src/core/component_provider.ts # packages/firestore/src/core/firestore_client.ts # packages/firestore/src/core/sync_engine.ts # packages/firestore/src/local/local_store.ts # packages/firestore/test/unit/local/local_store.test.ts

google-oss-bot · 2020-08-17T14:42:07Z

Binary Size Report

Affected SDKs

`@firebase/firestore`

Type	Base (`878a1bf`)	Head (`7dbf54f`)	Diff
browser	241 kB	254 kB	+12.8 kB (+5.3%)
esm2017	190 kB	200 kB	+9.15 kB (+4.8%)
main	475 kB	499 kB	+23.9 kB (+5.0%)
module	241 kB	254 kB	+12.8 kB (+5.3%)
react-native	190 kB	200 kB	+9.15 kB (+4.8%)

`@firebase/firestore/exp`

Type	Base (`878a1bf`)	Head (`7dbf54f`)	Diff
browser	189 kB	198 kB	+9.19 kB (+4.9%)
main	476 kB	500 kB	+24.0 kB (+5.0%)
module	189 kB	198 kB	+9.19 kB (+4.9%)
react-native	189 kB	198 kB	+9.19 kB (+4.9%)

`@firebase/firestore/lite`

Type	Base (`878a1bf`)	Head (`7dbf54f`)	Diff
browser	62.9 kB	63.1 kB	+231 B (+0.4%)
main	139 kB	139 kB	+244 B (+0.2%)
module	62.9 kB	63.1 kB	+231 B (+0.4%)
react-native	63.1 kB	63.3 kB	+231 B (+0.4%)

`@firebase/firestore/memory`

Type	Base (`878a1bf`)	Head (`7dbf54f`)	Diff
browser	177 kB	191 kB	+13.2 kB (+7.5%)
esm2017	139 kB	149 kB	+9.90 kB (+7.1%)
main	344 kB	368 kB	+24.1 kB (+7.0%)
module	177 kB	191 kB	+13.2 kB (+7.5%)
react-native	139 kB	149 kB	+9.90 kB (+7.1%)

`firebase`

Type	Base (`878a1bf`)	Head (`7dbf54f`)	Diff
firebase-firestore.js	280 kB	292 kB	+12.3 kB (+4.4%)
firebase-firestore.memory.js	218 kB	230 kB	+12.8 kB (+5.9%)
firebase.js	822 kB	835 kB	+12.3 kB (+1.5%)

Test Logs

Base (878a1bff): https://github.com/firebase/firebase-js-sdk/actions/runs/334235239
Head (7dbf54f2): https://github.com/firebase/firebase-js-sdk/actions/runs/334835011

google-cla · 2020-11-09T14:52:02Z

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

# Conflicts: # packages/firestore/exp/test/shim.ts # packages/firestore/src/api/database.ts

google-cla · 2020-11-09T15:40:24Z

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

wu-hui

This has changed quite a bit on the api/firestore_client level since last time you reviewed it.

Also, this is no longer release but hide it from public API, it is release bundles as free functions. I will hold it until we got all approvals and ready to launch.

wu-hui · 2020-11-09T15:54:33Z

.changeset/lemon-steaks-draw.md

@@ -0,0 +1,3 @@
+---


wu-hui · 2020-11-09T15:55:20Z

packages/firestore-types/index.d.ts

+export interface LoadBundleTask {
+  onProgress(
+    next?: (progress: LoadBundleTaskProgress) => any,
+    error?: (error: Error) => any,
+    complete?: () => void
+  ): void;
+
+  then<T, R>(
+    onFulfilled?: (a: LoadBundleTaskProgress) => T | PromiseLike<T>,
+    onRejected?: (a: Error) => R | PromiseLike<R>
+  ): Promise<T | R>;
+
+  catch<R>(
+    onRejected: (a: Error) => R | PromiseLike<R>
+  ): Promise<R | LoadBundleTaskProgress>;
+}
+
+export interface LoadBundleTaskProgress {
+  documentsLoaded: number;
+  totalDocuments: number;
+  bytesLoaded: number;
+  totalBytes: number;
+  taskState: TaskState;
+}
+
+export type TaskState = 'Error' | 'Running' | 'Success';


Will keep it this way, since this is now a release PR.

wu-hui · 2020-11-09T15:55:30Z

packages/firestore/exp-types/index.d.ts

+}
+
+export type TaskState = 'Error' | 'Running' | 'Success';
+


wu-hui · 2020-11-09T15:56:36Z

packages/firestore/src/local/local_store.ts

+ * This ensures that the loaded documents do not get garbage collected
+ * right away.
+ */
+export function umbrellaTarget(bundleName: string): Target {


wu-hui · 2020-11-09T16:04:03Z

packages/firestore/src/core/firestore_client.ts

+      return resultTask.catch(e => {
+        logWarn(LOG_TAG, `Loading bundle failed with ${e}`);
+      });


This looks different now, but should be fixed.

wu-hui · 2020-11-09T16:05:05Z

packages/firestore/exp/src/api/database.ts

+    const reader = createBundleReader(bundleData, newSerializer(databaseId));
+    const syncEngine = await getSyncEngine(firestore);
+
+    loadBundleSyncEngine(syncEngine, reader, resultTask);


wu-hui · 2020-11-09T16:06:40Z

packages/firestore/exp/src/api/database.ts

+    loadBundleSyncEngine(syncEngine, reader, resultTask);
+    return resultTask.catch((e: Error) => {
+      logWarn(LOG_TAG, `Loading bundle failed with ${e}`);
+    });


I think this is fixed, although this particular piece of code no longer exists.

wu-hui · 2020-11-09T16:06:46Z

packages/firestore/exp/src/api/database.ts

+      }
+
+      return null;
+      // return new Query(namedQuery.query, firestore, null);


wu-hui · 2020-11-09T16:06:52Z

packages/firestore/exp/test/shim.ts

+
+  async namedQuery(name: string): Promise<Query | null> {
+    return null;
+    // return namedQuery(this._delegate, name);


wu-hui · 2020-11-09T16:15:28Z

packages/firestore/rollup.shared.js

@@ -129,7 +129,8 @@ const manglePrivatePropertiesOptions = {
  },
  mangle: {
    properties: {
-      regex: /^__PRIVATE_/
+      regex: /^__PRIVATE_/,
+      reserved: ['do']


google-cla · 2020-11-09T16:25:41Z

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

google-cla · 2020-11-11T20:14:40Z

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

schmidt-sebastian · 2020-11-11T23:54:26Z

packages/firestore/src/api/database.ts

@@ -427,36 +424,6 @@ export function setLogLevel(level: PublicLogLevel): void {
  setClientLogLevel(level);
 }

-export function loadBundle(


You can keep these functions as long as they are not referenced.

schmidt-sebastian

Looks there there are still some stray API bits, but otherwise good. Since the size tooling seems to be broken at the moment, can you provide a rough number on the size increase?

schmidt-sebastian · 2020-11-11T23:55:14Z

.changeset/lemon-steaks-draw.md

+"@firebase/firestore": feature
+---
+
+Add support for loading Firestore Bundle Files.


This is no longer correct.

schmidt-sebastian · 2020-11-11T23:55:33Z

packages/firebase/index.d.ts

+    taskState: TaskState;
+  }
+
+  export type TaskState = 'Error' | 'Running' | 'Success';


Please remove all this.

schmidt-sebastian · 2020-11-11T23:55:53Z

packages/firestore-types/index.d.ts

+export interface LoadBundleTask {
+  onProgress(
+    next?: (progress: LoadBundleTaskProgress) => any,
+    error?: (error: Error) => any,
+    complete?: () => void
+  ): void;
+
+  then<T, R>(
+    onFulfilled?: (a: LoadBundleTaskProgress) => T | PromiseLike<T>,
+    onRejected?: (a: Error) => R | PromiseLike<R>
+  ): Promise<T | R>;
+
+  catch<R>(
+    onRejected: (a: Error) => R | PromiseLike<R>
+  ): Promise<R | LoadBundleTaskProgress>;
+}
+
+export interface LoadBundleTaskProgress {
+  documentsLoaded: number;
+  totalDocuments: number;
+  bytesLoaded: number;
+  totalBytes: number;
+  taskState: TaskState;
+}
+
+export type TaskState = 'Error' | 'Running' | 'Success';


Please remove as discussed.

schmidt-sebastian · 2020-11-11T23:56:07Z

packages/firestore/exp-types/index.d.ts

+}
+
+export type TaskState = 'Error' | 'Running' | 'Success';
+


Please remove.

schmidt-sebastian · 2020-11-11T23:56:17Z

packages/firestore/exp-types/index.d.ts

+export function namedQuery(
+  firestore: FirebaseFirestore,
+  name: string
+): Promise<Query<DocumentData> | null>;


Remove these as well.

schmidt-sebastian · 2020-11-11T23:59:45Z

packages/firestore/src/core/firestore_client.ts

+  data: ReadableStream<Uint8Array> | ArrayBuffer | string,
+  resultTask: LoadBundleTask
+): Promise<void> {
+  client.verifyNotTerminated();


The other functions in this file don't call verifyNotTerminated (done at callsite). Should we follow this pattern here as well?

Same just below.

schmidt-sebastian · 2020-11-12T00:01:13Z

packages/firestore/src/core/sync_engine.ts

+  });
+}
+
+async function loadBundleImpl(


Could this potentially be inlined? Or did we not want to make syncEngineLoadBundle async?

Yep, that was the reason.

google-cla · 2020-11-12T15:13:54Z

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

wu-hui

firebase-firestore.js 283361 -> 286235
firebase-firestore.memory.js 221571 -> 224766

wu-hui · 2020-11-12T14:32:14Z

.changeset/lemon-steaks-draw.md

+"@firebase/firestore": feature
+---
+
+Add support for loading Firestore Bundle Files.


wu-hui · 2020-11-12T14:34:23Z

packages/firebase/index.d.ts

+    taskState: TaskState;
+  }
+
+  export type TaskState = 'Error' | 'Running' | 'Success';


wu-hui · 2020-11-12T14:34:51Z

packages/firestore-types/index.d.ts

+export interface LoadBundleTask {
+  onProgress(
+    next?: (progress: LoadBundleTaskProgress) => any,
+    error?: (error: Error) => any,
+    complete?: () => void
+  ): void;
+
+  then<T, R>(
+    onFulfilled?: (a: LoadBundleTaskProgress) => T | PromiseLike<T>,
+    onRejected?: (a: Error) => R | PromiseLike<R>
+  ): Promise<T | R>;
+
+  catch<R>(
+    onRejected: (a: Error) => R | PromiseLike<R>
+  ): Promise<R | LoadBundleTaskProgress>;
+}
+
+export interface LoadBundleTaskProgress {
+  documentsLoaded: number;
+  totalDocuments: number;
+  bytesLoaded: number;
+  totalBytes: number;
+  taskState: TaskState;
+}
+
+export type TaskState = 'Error' | 'Running' | 'Success';


wu-hui · 2020-11-12T14:41:20Z

packages/firestore/exp-types/index.d.ts

+}
+
+export type TaskState = 'Error' | 'Running' | 'Success';
+


wu-hui · 2020-11-12T14:43:21Z

packages/firestore/exp-types/index.d.ts

+export function namedQuery(
+  firestore: FirebaseFirestore,
+  name: string
+): Promise<Query<DocumentData> | null>;


wu-hui · 2020-11-12T14:51:17Z

packages/firestore/src/api/database.ts

@@ -427,36 +424,6 @@ export function setLogLevel(level: PublicLogLevel): void {
  setClientLogLevel(level);
 }

-export function loadBundle(


wu-hui · 2020-11-12T14:51:44Z

packages/firestore/src/core/firestore_client.ts

+  data: ReadableStream<Uint8Array> | ArrayBuffer | string,
+  resultTask: LoadBundleTask
+): Promise<void> {
+  client.verifyNotTerminated();


wu-hui · 2020-11-12T14:52:12Z

packages/firestore/src/core/sync_engine.ts

+  });
+}
+
+async function loadBundleImpl(


Yep, that was the reason.

schmidt-sebastian

Wheee!

schmidt-sebastian · 2020-11-12T17:36:51Z

packages/firestore/src/api/database.ts

+  db: Firestore,
+  bundleData: ArrayBuffer | ReadableStream<Uint8Array> | string
+): ApiLoadBundleTask {
+  const resultTask = new LoadBundleTask();


We need to figure out where to call "ensureClientConfigured". It might depend on how we expose this function in the end.

# Conflicts: # packages/firestore/src/api/database.ts # packages/firestore/test/unit/specs/spec_builder.ts

google-cla · 2020-11-12T18:43:43Z

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

wu-hui and others added 18 commits May 21, 2020 13:45

Implement a bundle reader for Web (#3097)

b0c8299

Implement a bundle reader to read from a underlying readable stream lazily, and parse the content to proto objects.

Enable bundle reader for Node. (#3167)

2f179bc

Implement BundleCache for IDB and memory. (#3170)

d3f0ab4

Merge branch 'master' into wuandy/Bundles

2e73801

Merge branch 'master' into wuandy/Bundles

fff6b1b

Fix firestore_bundle_proto rollup issue. (#3316)

3a78d7c

* Fix firestore_bundle_proto rollup issue * Create lemon-steaks-draw.md

Implement bundle loading. (#3201)

0bf9c73

Merge branch 'master' into wuandy/Bundles

2bfa185

Bundles load from secondary tabs (#3393)

9bfb995

Make loadBundle work with exp build (#3488)

8b8a097

* Make loadBundle work with exp build * Use legacy.LoadBundleTask

Bundle's named query resume. (#3395)

5ebf8c3

Merge remote-tracking branch 'origin/wuandy/Bundles' into wuandy/Bundles

2bec6d8

# Conflicts: # packages/firestore/exp/src/api/database.ts

Merge with named query change.

0f697df

wu-hui requested review from avolkovi, bojeil-google, sam-gc and scottcrossen as code owners August 10, 2020 19:43

Add missing byte_stream_reader.ts for *_lite platforms

dc02c03

wu-hui requested a review from egilmorez as a code owner August 10, 2020 20:44

wu-hui and others added 3 commits August 11, 2020 15:05

Finally builds and passes

15b96a9

Update query-document mapping from bundles. (#3620)

b4f9936

* Update query-document mapping from bundles. * Only update mapping when query read time is updated. * Fix lint errors * Better code structure. * Use serializer constructed from firestore client. * Don't inline await

wu-hui requested a review from zwu52 as a code owner August 17, 2020 14:28

wu-hui requested review from erikeldridge, jamesdaniels, yuchenshi and zijianjoy as code owners November 9, 2020 14:51

Merge branch 'master' into wuandy/Bundles

089fcdd

# Conflicts: # packages/firestore/exp/test/shim.ts # packages/firestore/src/api/database.ts

wu-hui commented Nov 9, 2020

View reviewed changes

wu-hui changed the title ~~Merge firestore bundles into master but hide it from public API~~ Release firestore bundles as free functions Nov 9, 2020

Address comments

80dc41d

Remove bundles from public interfaces.

d4ab17f

schmidt-sebastian reviewed Nov 11, 2020

View reviewed changes

schmidt-sebastian reviewed Nov 12, 2020

View reviewed changes

schmidt-sebastian assigned wu-hui and unassigned schmidt-sebastian Nov 12, 2020

Remove bundles from public interfaces take 2.

1b3d633

wu-hui commented Nov 12, 2020

View reviewed changes

wu-hui assigned schmidt-sebastian and unassigned wu-hui Nov 12, 2020

wu-hui changed the title ~~Release firestore bundles as free functions~~ Merge firestore bundles implementation without exposing public APIs Nov 12, 2020

schmidt-sebastian approved these changes Nov 12, 2020

View reviewed changes

Merge branch 'master' into wuandy/Bundles

205d1c8

# Conflicts: # packages/firestore/src/api/database.ts # packages/firestore/test/unit/specs/spec_builder.ts

wu-hui added the api: firestore label Nov 12, 2020

hsubox76 merged commit 6dffdf2 into master Nov 12, 2020

google-oss-bot mentioned this pull request Nov 12, 2020

Version Packages #4054

Merged

firebase locked and limited conversation to collaborators Dec 13, 2020

Merge firestore bundles implementation without exposing public APIs #3594

Merge firestore bundles implementation without exposing public APIs #3594

Uh oh!

Conversation

wu-hui commented Aug 10, 2020

Uh oh!

changeset-bot bot commented Aug 10, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

google-oss-bot commented Aug 17, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Binary Size Report

Affected SDKs

@firebase/firestore

@firebase/firestore/exp

@firebase/firestore/lite

@firebase/firestore/memory

firebase

Test Logs

Uh oh!

google-cla bot commented Nov 9, 2020

Uh oh!

google-cla bot commented Nov 9, 2020

Uh oh!

wu-hui left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

google-cla bot commented Nov 9, 2020

Uh oh!

google-cla bot commented Nov 11, 2020

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

schmidt-sebastian left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

changeset-bot bot commented Aug 10, 2020 •

edited

Loading

google-oss-bot commented Aug 17, 2020 •

edited

Loading

`@firebase/firestore`

`@firebase/firestore/exp`

`@firebase/firestore/lite`

`@firebase/firestore/memory`

`firebase`