@firebase/firestore 3.4.1-canary.c2fff0c6a → 3.4.2-canary.044a8d7f9

package/dist/index.node.mjs

@@ -2,7 +2,7 @@ import { _getProvider, getApp, _removeServiceInstance, _registerComponent, regis
 import { Component } from '@firebase/component';
 import { Logger, LogLevel } from '@firebase/logger';
 import { inspect, TextEncoder, TextDecoder } from 'util';
-import { createMockUserToken, getModularInstance, deepEqual, getUA, isIndexedDBAvailable, isSafari } from '@firebase/util';
+import { FirebaseError, createMockUserToken, getModularInstance, deepEqual, getUA, isIndexedDBAvailable, isSafari } from '@firebase/util';
 import { randomBytes as randomBytes$1 } from 'crypto';
 import { credentials, loadPackageDefinition, Metadata } from '@grpc/grpc-js';
 import { dirname, resolve, join } from 'path';
@@ -10,7 +10,7 @@ import { fileURLToPath } from 'url';
 import { loadSync } from '@grpc/proto-loader';
 
 const name = "@firebase/firestore";
-const version$1 = "3.4.1-canary.c2fff0c6a";
+const version$1 = "3.4.2-canary.044a8d7f9";
 
 /**
  * @license
@@ -63,7 +63,7 @@ User.GOOGLE_CREDENTIALS = new User('google-credentials-uid');
 User.FIRST_PARTY = new User('first-party-uid');
 User.MOCK_USER = new User('mock-user');
 
-const version = "9.6.1-canary.c2fff0c6a";
+const version = "9.6.2-canary.044a8d7f9";
 
 /**
  * @license
@@ -383,7 +383,7 @@ const Code = {
     DATA_LOSS: 'data-loss'
 };
 /** An error returned by a Firestore operation. */
-class FirestoreError extends Error {
+class FirestoreError extends FirebaseError {
     /** @hideconstructor */
     constructor(
     /**
@@ -394,11 +394,9 @@ class FirestoreError extends Error {
      * A custom error description.
      */
     message) {
-        super(message);
+        super(code, message);
         this.code = code;
         this.message = message;
-        /** The custom name for all FirestoreErrors. */
-        this.name = 'FirebaseError';
         // HACK: We write a toString property directly because Error is not a real
         // class and so inheritance does not work correctly. We could alternatively
         // do the same "back-door inheritance" trick that FirebaseError does.
@@ -3161,6 +3159,19 @@ class ByteString {
         const binaryString = binaryStringFromUint8Array(array);
         return new ByteString(binaryString);
     }
+    [Symbol.iterator]() {
+        let i = 0;
+        return {
+            next: () => {
+                if (i < this.binaryString.length) {
+                    return { value: this.binaryString.charCodeAt(i++), done: false };
+                }
+                else {
+                    return { value: undefined, done: true };
+                }
+            }
+        };
+    }
     toBase64() {
         return encodeBase64(this.binaryString);
     }
@@ -3521,6 +3532,9 @@ function typeOrder(value) {
 }
 /** Tests `left` and `right` for equality based on the backend semantics. */
 function valueEquals(left, right) {
+    if (left === right) {
+        return true;
+    }
     const leftType = typeOrder(left);
     const rightType = typeOrder(right);
     if (leftType !== rightType) {
@@ -3611,6 +3625,9 @@ function arrayValueContains(haystack, needle) {
     return ((haystack.values || []).find(v => valueEquals(v, needle)) !== undefined);
 }
 function valueCompare(left, right) {
+    if (left === right) {
+        return 0;
+    }
     const leftType = typeOrder(left);
     const rightType = typeOrder(right);
     if (leftType !== rightType) {
@@ -11919,18 +11936,20 @@ function localStoreApplyRemoteEventToLocalCache(localStore, remoteEvent) {
                 .next(() => {
                 return localStoreImpl.targetCache.addMatchingKeys(txn, change.addedDocuments, targetId);
             }));
-            const resumeToken = change.resumeToken;
-            // Update the resume token if the change includes one.
-            if (resumeToken.approximateByteSize() > 0) {
-                const newTargetData = oldTargetData
-                    .withResumeToken(resumeToken, remoteVersion)
-                    .withSequenceNumber(txn.currentSequenceNumber);
-                newTargetDataByTargetMap = newTargetDataByTargetMap.insert(targetId, newTargetData);
-                // Update the target data if there are target changes (or if
-                // sufficient time has passed since the last update).
-                if (shouldPersistTargetData(oldTargetData, newTargetData, change)) {
-                    promises.push(localStoreImpl.targetCache.updateTargetData(txn, newTargetData));
-                }
+            let newTargetData = oldTargetData.withSequenceNumber(txn.currentSequenceNumber);
+            if (remoteEvent.targetMismatches.has(targetId)) {
+                newTargetData = newTargetData
+                    .withResumeToken(ByteString.EMPTY_BYTE_STRING, SnapshotVersion.min())
+                    .withLastLimboFreeSnapshotVersion(SnapshotVersion.min());
+            }
+            else if (change.resumeToken.approximateByteSize() > 0) {
+                newTargetData = newTargetData.withResumeToken(change.resumeToken, remoteVersion);
+            }
+            newTargetDataByTargetMap = newTargetDataByTargetMap.insert(targetId, newTargetData);
+            // Update the target data if there are target changes (or if
+            // sufficient time has passed since the last update).
+            if (shouldPersistTargetData(oldTargetData, newTargetData, change)) {
+                promises.push(localStoreImpl.targetCache.updateTargetData(txn, newTargetData));
             }
         });
         let changedDocs = mutableDocumentMap();
@@ -12030,7 +12049,6 @@ documentVersions) {
  * too frequent.
  */
 function shouldPersistTargetData(oldTargetData, newTargetData, change) {
-    hardAssert(newTargetData.resumeToken.approximateByteSize() > 0);
     // Always persist target data if we don't already have a resume token.
     if (oldTargetData.resumeToken.approximateByteSize() === 0) {
         return true;