@signaldb/svelte 2.0.0-beta.7 → 2.0.0-beta.9

package/dist/base/core/src/Collection/index.js

@@ -62,13 +62,29 @@ class Collection extends EventEmitter_1.default {
         const execute = () => Collection.collections.reduce((memo, collection) => () => {
             return collection.batch(memo);
         }, callback)();
-        const maybePromise = execute();
         const afterBatch = () => {
             Collection.batchOperationInProgress = false;
         };
+        let maybePromise;
+        try {
+            maybePromise = execute();
+        }
+        catch (error) {
+            // A synchronously throwing callback must not leave the batch flag
+            // stuck at `true` — that would defer every requery forever (see the
+            // rejection branch below).
+            afterBatch();
+            throw error;
+        }
         if (maybePromise && typeof maybePromise.then === 'function') {
             return maybePromise
-                .then(() => afterBatch());
+                .then(() => afterBatch(), (error) => {
+                // Rejections need the same cleanup as fulfillment — otherwise
+                // `batchOperationInProgress` stays `true` and all deferred
+                // requeries are never flushed, silently freezing reactivity.
+                afterBatch();
+                throw error;
+            });
         }
         else {
             afterBatch();
@@ -358,14 +374,31 @@ class Collection extends EventEmitter_1.default {
         if (this.batchOperationInProgress)
             return callback();
         this.batchOperationInProgress = true;
-        const maybePromise = callback();
         const afterBatch = () => {
             this.batchOperationInProgress = false;
             this.postBatchCallbacks.forEach(callback_ => callback_());
             this.postBatchCallbacks.clear();
         };
+        let maybePromise;
+        try {
+            maybePromise = callback();
+        }
+        catch (error) {
+            // A synchronously throwing callback must not leave the batch flag
+            // stuck at `true` — that would defer every requery forever (see the
+            // rejection branch below).
+            afterBatch();
+            throw error;
+        }
         if (maybePromise && typeof maybePromise.then === 'function') {
-            return maybePromise.then(() => afterBatch());
+            return maybePromise.then(() => afterBatch(), (error) => {
+                // Rejections need the same cleanup as fulfillment — otherwise
+                // `batchOperationInProgress` stays `true`, deferred post-batch
+                // callbacks (e.g. reactive requeries) are never flushed and
+                // reactivity silently freezes for the rest of the session.
+                afterBatch();
+                throw error;
+            });
         }
         else {
             afterBatch();

package/package.json

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