spice-js 2.7.4 → 2.7.5

package/build/models/SpiceModel.js

@@ -1894,11 +1894,19 @@ class SpiceModel {
 
           var modifiers = ((_this18$_serializers = _this18[_serializers]) == null ? void 0 : (_this18$_serializers$ = _this18$_serializers[type]) == null ? void 0 : _this18$_serializers$.modifiers) || [];
 
-          for (var modifier of modifiers) {
+          for (var i = 0; i < modifiers.length; i++) {
+            var modifier = modifiers[i];
+
             try {
-              data = yield modifier(data, old_data, _this18[_ctx], _this18.type);
+              var result = yield modifier(data, old_data, _this18[_ctx], _this18.type); // Guard against modifiers that return undefined
+
+              if (result !== undefined) {
+                data = result;
+              } else {
+                console.warn("Modifier #" + i + " for type=" + _this18.type + " returned undefined, keeping previous data");
+              }
             } catch (error) {
-              console.error("Modifier error in do_serialize:", error.stack);
+              console.error("Modifier error in do_serialize (type=" + _this18.type + ", modifier #" + i + "):", error instanceof Error ? error.stack : "Non-Error thrown: " + JSON.stringify(error));
             }
           } // Ensure data is always an array for consistent processing.
 
@@ -1957,7 +1965,7 @@ class SpiceModel {
 
         return yield doSerialize();
       } catch (error) {
-        console.error("Error in do_serialize:", error.stack);
+        console.error("Error in do_serialize:", error instanceof Error ? error.stack : "Non-Error thrown: " + JSON.stringify(error));
         throw error;
       }
     })();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spice-js",
-  "version": "2.7.4",
+  "version": "2.7.5",
   "description": "spice",
   "main": "build/index.js",
   "repository": {

package/src/models/SpiceModel.js

@@ -1678,11 +1678,25 @@ export default class SpiceModel {
 
       // Cache the modifiers lookup for the specified type.
       const modifiers = this[_serializers]?.[type]?.modifiers || [];
-      for (const modifier of modifiers) {
+      for (let i = 0; i < modifiers.length; i++) {
+        const modifier = modifiers[i];
         try {
-          data = await modifier(data, old_data, this[_ctx], this.type);
+          const result = await modifier(data, old_data, this[_ctx], this.type);
+          // Guard against modifiers that return undefined
+          if (result !== undefined) {
+            data = result;
+          } else {
+            console.warn(
+              `Modifier #${i} for type=${this.type} returned undefined, keeping previous data`
+            );
+          }
         } catch (error) {
-          console.error("Modifier error in do_serialize:", error.stack);
+          console.error(
+            `Modifier error in do_serialize (type=${this.type}, modifier #${i}):`,
+            error instanceof Error ?
+              error.stack
+            : `Non-Error thrown: ${JSON.stringify(error)}`
+          );
         }
       }
 
@@ -1735,7 +1749,12 @@ export default class SpiceModel {
       }
       return await doSerialize();
     } catch (error) {
-      console.error("Error in do_serialize:", error.stack);
+      console.error(
+        "Error in do_serialize:",
+        error instanceof Error ?
+          error.stack
+        : `Non-Error thrown: ${JSON.stringify(error)}`
+      );
       throw error;
     }
   }