@firebase/firestore 4.7.1-canary.3670ab83c → 4.7.1-canary.629919ea7

package/dist/firestore/src/global_index.d.ts

@@ -4960,7 +4960,7 @@ interface SharedClientState {
      * If the target id is already associated with local client, the method simply
      * returns its `QueryTargetState`.
      */
-    addLocalQueryTarget(targetId: TargetId): QueryTargetState;
+    addLocalQueryTarget(targetId: TargetId, addToActiveTargetIds?: boolean): QueryTargetState;
     /** Removes the Query Target ID association from the local client. */
     removeLocalQueryTarget(targetId: TargetId): void;
     /** Checks whether the target is associated with the local client. */

package/dist/firestore/src/local/shared_client_state.d.ts

@@ -60,7 +60,7 @@ export interface SharedClientState {
      * If the target id is already associated with local client, the method simply
      * returns its `QueryTargetState`.
      */
-    addLocalQueryTarget(targetId: TargetId): QueryTargetState;
+    addLocalQueryTarget(targetId: TargetId, addToActiveTargetIds?: boolean): QueryTargetState;
     /** Removes the Query Target ID association from the local client. */
     removeLocalQueryTarget(targetId: TargetId): void;
     /** Checks whether the target is associated with the local client. */
@@ -232,7 +232,7 @@ export declare class WebStorageSharedClientState implements SharedClientState {
     isActiveQueryTarget(targetId: TargetId): boolean;
     addPendingMutation(batchId: BatchId): void;
     updateMutationState(batchId: BatchId, state: 'acknowledged' | 'rejected', error?: FirestoreError): void;
-    addLocalQueryTarget(targetId: TargetId): QueryTargetState;
+    addLocalQueryTarget(targetId: TargetId, addToActiveTargetIds?: boolean): QueryTargetState;
     removeLocalQueryTarget(targetId: TargetId): void;
     isLocalQueryTarget(targetId: TargetId): boolean;
     clearQueryState(targetId: TargetId): void;
@@ -296,7 +296,7 @@ export declare class MemorySharedClientState implements SharedClientState {
     sequenceNumberHandler: ((sequenceNumber: ListenSequenceNumber) => void) | null;
     addPendingMutation(batchId: BatchId): void;
     updateMutationState(batchId: BatchId, state: 'acknowledged' | 'rejected', error?: FirestoreError): void;
-    addLocalQueryTarget(targetId: TargetId): QueryTargetState;
+    addLocalQueryTarget(targetId: TargetId, addToActiveTargetIds?: boolean): QueryTargetState;
     updateQueryState(targetId: TargetId, state: QueryTargetState, error?: FirestoreError): void;
     removeLocalQueryTarget(targetId: TargetId): void;
     isLocalQueryTarget(targetId: TargetId): boolean;

package/dist/index.cjs.js

@@ -71,7 +71,7 @@ User.MOCK_USER = new User("mock-user");
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-let b = "10.13.1-canary.3670ab83c";
+let b = "10.13.1-canary.629919ea7";
 
 /**
  * @license
@@ -13325,18 +13325,20 @@ class __PRIVATE_LocalClientState {
         // in order, it is safe to delete the entry right after updating it.
         this.Qs(e);
     }
-    addLocalQueryTarget(e) {
-        let t = "not-current";
+    addLocalQueryTarget(e, t = !0) {
+        let n = "not-current";
         // Lookup an existing query state if the target ID was already registered
         // by another tab
                 if (this.isActiveQueryTarget(e)) {
-            const n = this.storage.getItem(createWebStorageQueryTargetMetadataKey(this.persistenceKey, e));
-            if (n) {
-                const r = __PRIVATE_QueryTargetMetadata.Rs(e, n);
-                r && (t = r.state);
+            const t = this.storage.getItem(createWebStorageQueryTargetMetadataKey(this.persistenceKey, e));
+            if (t) {
+                const r = __PRIVATE_QueryTargetMetadata.Rs(e, t);
+                r && (n = r.state);
             }
         }
-        return this.Ks.fs(e), this.Ns(), t;
+        // If the query is listening to cache only, the target ID should not be registered with the
+        // local Firestore client as an active watch target.
+                return t && this.Ks.fs(e), this.Ns(), n;
     }
     removeLocalQueryTarget(e) {
         this.Ks.gs(e), this.Ns();
@@ -13544,8 +13546,8 @@ class __PRIVATE_MemorySharedClientState {
     updateMutationState(e, t, n) {
         // No op.
     }
-    addLocalQueryTarget(e) {
-        return this.so.fs(e), this.oo[e] || "not-current";
+    addLocalQueryTarget(e, t = !0) {
+        return t && this.so.fs(e), this.oo[e] || "not-current";
     }
     updateQueryState(e, t, n) {
         this.oo[e] = t;
@@ -16236,7 +16238,7 @@ async function __PRIVATE_syncEngineListen(e, t, n = !0) {
 }
 
 async function __PRIVATE_allocateTargetAndMaybeListen(e, t, n, r) {
-    const i = await __PRIVATE_localStoreAllocateTarget(e.localStore, __PRIVATE_queryToTarget(t)), s = i.targetId, o = n ? e.sharedClientState.addLocalQueryTarget(s) : "not-current";
+    const i = await __PRIVATE_localStoreAllocateTarget(e.localStore, __PRIVATE_queryToTarget(t)), s = i.targetId, o = e.sharedClientState.addLocalQueryTarget(s, n);
     let _;
     return r && (_ = await __PRIVATE_initializeViewAndComputeSnapshot(e, t, s, "current" === o, i.resumeToken)), 
     e.isPrimaryClient && n && __PRIVATE_remoteStoreListen(e.remoteStore, i), _;
@@ -22233,9 +22235,9 @@ let Me = null;
         return r = Object.assign({
             useFetchStreams: t
         }, r), s._setSettings(r), s;
-    }), "PUBLIC").setMultipleInstances(!0)), app.registerVersion(S, "4.7.1-canary.3670ab83c", e), 
+    }), "PUBLIC").setMultipleInstances(!0)), app.registerVersion(S, "4.7.1-canary.629919ea7", e), 
     // BUILD_TARGET will be replaced by values like esm5, esm2017, cjs5, etc during the compilation
-    app.registerVersion(S, "4.7.1-canary.3670ab83c", "cjs2017");
+    app.registerVersion(S, "4.7.1-canary.629919ea7", "cjs2017");
 }();
 
 exports.AbstractUserDataWriter = AbstractUserDataWriter;