@converse/skeletor 3.1.1 → 3.1.3

package/dist/skeletor.js

@@ -8711,7 +8711,7 @@
           // eslint-disable-next-line @typescript-eslint/no-this-alias
           const that = this;
           async function localSync(method, model, options) {
-              let resp, errorMessage, promise, new_attributes;
+              let resp, caughtError, promise, new_attributes;
               // We get the collection (and if necessary the model attribute.
               // Waiting for storeInitialized will cause another iteration of
               // the event loop, after which the collection reference will
@@ -8768,10 +8768,12 @@
               catch (error) {
                   const storageSize = await that.getStorageSize();
                   if (error.code === 22 && storageSize === 0) {
-                      errorMessage = 'Private browsing is unsupported';
+                      caughtError = new Error('Private browsing is unsupported');
                   }
                   else {
-                      errorMessage = error.message;
+                      // Preserve a real Error (stack/name); wrap anything else so the
+                      // error path always surfaces an Error, never a bare string.
+                      caughtError = error instanceof Error ? error : new Error(error?.message ?? String(error));
                   }
               }
               if (resp) {
@@ -8785,10 +8787,21 @@
                       options.success(data, options);
                   }
               }
+              else if (method === 'read' && !caughtError) {
+                  // A read that finds no record is a normal empty result (first run /
+                  // empty storage), not an error. Resolve with `null` so the callback
+                  // and promise forms of fetch() agree and a first-run read never
+                  // rejects.
+                  if (options && options.success) {
+                      options.success(null, options);
+                  }
+              }
               else {
-                  errorMessage = errorMessage ? errorMessage : 'Record Not Found';
+                  // A genuine failure (storage threw, or a write produced no result):
+                  // surface a real Error so consumers get a stack and `instanceof Error`
+                  // holds.
                   if (options && options.error) {
-                      options.error(errorMessage);
+                      options.error(caughtError ?? new Error('Record Not Found'));
                   }
               }
           }
@@ -11460,12 +11473,6 @@
       try {
           await this.fetch({ fromStorage: true, promise: true });
       }
-      catch (e) {
-          // 'Record Not Found' is a normal first-run state (nothing stored yet);
-          // keep the initial attributes and resolve. Re-throw anything else.
-          if (e !== 'Record Not Found')
-              throw e;
-      }
       finally {
           __classPrivateFieldSet(this, _Model_state, 'ready', "f");
       }
@@ -12240,8 +12247,9 @@
   }
   _Collection_instances = new WeakSet(), _Collection_hydrate = async function _Collection_hydrate() {
       ensureUnloadListener(PersistentStorage);
-      // findAll() returns [] for an empty store, so a first-run collection
-      // resolves rather than erroring the way a model fetch-by-id would.
+      // An empty store is a normal first-run state: findAll() returns [] and a
+      // missing model record resolves to null, so hydration resolves rather
+      // than rejecting.
       await this.fetch({ fromStorage: true, promise: true });
   }, _a = Symbol.iterator;
   // This "enum" defines the three possible kinds of values which can be emitted