msgpackr 1.9.4 → 1.9.5

package/dist/test.js

@@ -1635,8 +1635,13 @@
 							if (Array.isArray(value)) {
 								packArray(value);
 							} else {
-								if (value.toJSON) // use this as an alternate mechanism for expressing how to serialize
-									return pack(value.toJSON());
+								// use this as an alternate mechanism for expressing how to serialize
+								if (value.toJSON) {
+									const json = value.toJSON();
+									// if for some reason value.toJSON returns itself it'll loop forever
+									if (json !== value)
+										return pack(json)
+								}
 								
 								// if there is a writeFunction, use it, otherwise just encode as undefined
 								if (type === 'function')
@@ -4098,6 +4103,17 @@
 			assert.deepEqual(values, [[1,0,1], [2,1,2], [3,2,3], [4,3,4]]);
 		});
 
+		test('pack toJSON returning this', () => {
+			class Serializable {
+				someData = [1, 2, 3, 4]
+				toJSON() {
+					return this
+				}
+			}
+			const serialized = pack(new Serializable);
+			const deserialized = unpack(serialized);
+			assert.deepStrictEqual(deserialized, { someData: [1, 2, 3, 4] });
+		});
 	});
 	suite('msgpackr performance tests', function(){
 		test('performance JSON.parse', function() {