@resolveio/server-lib 22.0.22 → 22.0.24

package/managers/subscription.manager.js

@@ -3036,7 +3036,7 @@ var SubscriptionManager = /** @class */ (function () {
     };
     SubscriptionManager.prototype.shouldInvalidateSubscription = function (sub, collection, type, documentId, document) {
         return __awaiter(this, void 0, void 0, function () {
-            var normalizedDocumentId, hasDependencyData, trackedIds, filters, filters_1, filters_1_1, filter, e_13_1;
+            var normalizedDocumentId, trackedIds, filters, hasTrackedIds, hasFilters, hasDependencyData, filters_1, filters_1_1, filter, e_13_1;
             var e_13, _a;
             var _b, _c;
             return __generator(this, function (_d) {
@@ -3044,6 +3044,10 @@ var SubscriptionManager = /** @class */ (function () {
                     case 0:
                         this.ensureDependencyContainers(sub);
                         normalizedDocumentId = this.normalizeDocumentId(documentId);
+                        trackedIds = sub.collectionDependencies.get(collection);
+                        filters = sub.collectionFilters.get(collection);
+                        hasTrackedIds = !!(trackedIds && trackedIds.size);
+                        hasFilters = !!(filters && filters.length);
                         hasDependencyData = (((_b = sub.collectionDependencies.get(collection)) === null || _b === void 0 ? void 0 : _b.size) || 0) > 0 ||
                             (((_c = sub.collectionFilters.get(collection)) === null || _c === void 0 ? void 0 : _c.length) || 0) > 0 ||
                             sub.watchAllCollections.has(collection);
@@ -3063,16 +3067,32 @@ var SubscriptionManager = /** @class */ (function () {
                             this.dependencyDebug('Invalidate due to watch-all collection', { publication: sub.publication, collection: collection, type: type });
                             return [2 /*return*/, true];
                         }
-                        trackedIds = sub.collectionDependencies.get(collection);
                         if (trackedIds && trackedIds.has(normalizedDocumentId)) {
                             this.dependencyDebug('Invalidate due to tracked id', { publication: sub.publication, collection: collection, type: type, documentId: normalizedDocumentId });
                             return [2 /*return*/, true];
                         }
                         if (type === 'delete') {
+                            if (hasFilters && !hasTrackedIds) {
+                                this.dependencyDebug('Invalidate delete due to filter-only dependency without tracked ids', {
+                                    publication: sub.publication,
+                                    collection: collection,
+                                    type: type,
+                                    documentId: normalizedDocumentId
+                                });
+                                return [2 /*return*/, true];
+                            }
                             this.dependencyDebug('Skip invalidation on delete for unknown id', { publication: sub.publication, collection: collection, type: type, documentId: normalizedDocumentId });
                             return [2 /*return*/, false];
                         }
-                        filters = sub.collectionFilters.get(collection);
+                        if (type === 'update' && hasFilters && !hasTrackedIds) {
+                            this.dependencyDebug('Invalidate update due to filter-only dependency without tracked ids', {
+                                publication: sub.publication,
+                                collection: collection,
+                                type: type,
+                                documentId: normalizedDocumentId
+                            });
+                            return [2 /*return*/, true];
+                        }
                         if (!(filters && filters.length)) return [3 /*break*/, 8];
                         _d.label = 1;
                     case 1: