@signaldb/svelte 2.0.0-beta.15 → 2.0.0-beta.17

package/dist/base/core/src/WorkerDataAdapter.js

@@ -148,11 +148,33 @@ class WorkerDataAdapter {
             // holding something else — a message lost, a query re-registered underneath, a host and an
             // adapter that disagree — applying it anyway would leave a result that silently drifts from
             // the store. Refusing it keeps the last coherent result instead, and the next full answer
-            // puts things right.
-            if (!(0, queryDelta_1.canApplyQueryDelta)(query.items, delta))
-                return;
-            if ((0, queryDelta_1.isEmptyQueryDelta)(delta))
+            // puts things right. A delta describing no change at all is refused for the simpler reason
+            // that there is nothing to apply.
+            //
+            // The *state* the message carries is not part of that judgement, and used to be dropped
+            // along with the delta. A host that cannot answer a query from its previous result announces
+            // `'active'`, re-executes, and answers with a delta — which is empty whenever the write left
+            // this query's result as it was. The query then stayed `'active'` here for good, because no
+            // further message was coming to correct it: `Cursor#isLoading()` reported a first result
+            // still pending forever, and every screen gated on it showed its loading state forever.
+            const canApply = (0, queryDelta_1.canApplyQueryDelta)(query.items, delta);
+            if (!canApply || (0, queryDelta_1.isEmptyQueryDelta)(delta)) {
+                if (state === query.state)
+                    return;
+                this.updateQuery(collectionName, {
+                    selector: query.selector,
+                    options: query.options,
+                }, { state, error });
+                const settled = collectionQueries.get(id);
+                if (!settled)
+                    return;
+                // An empty delta is passed on as it came: it settles the query without asking anyone to
+                // recompute a result that did not change. One that could not be applied is not, so the
+                // consumer re-reads and the disagreement ends there.
+                settled.stateChangeCallbacks
+                    .forEach(callback => (0, queryDelta_1.callWithDelta)(callback, state, canApply ? delta : undefined));
                 return;
+            }
             const pendingBefore = this.flattenPendingWrites(collectionName);
             const servedBefore = pendingBefore == null
                 ? null

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@signaldb/svelte",
   "type": "module",
-  "version": "2.0.0-beta.15",
+  "version": "2.0.0-beta.17",
   "scripts": {
     "build": "tsc -d --noEmit false",
     "analyze-bundle": "bundle-analyzer ./dist --upload-token=$BUNDLE_ANALYZER_UPLOAD_TOKEN --bundle-name=@signaldb/svelte",