msgpackr 1.8.0 → 1.8.1

package/dist/index.js

@@ -1268,6 +1268,23 @@
 						position = start;
 				}
 			};
+			const packArray = (value) => {
+				var length = value.length;
+				if (length < 0x10) {
+					target[position++] = 0x90 | length;
+				} else if (length < 0x10000) {
+					target[position++] = 0xdc;
+					target[position++] = length >> 8;
+					target[position++] = length & 0xff;
+				} else {
+					target[position++] = 0xdd;
+					targetView.setUint32(position, length);
+					position += 4;
+				}
+				for (let i = 0; i < length; i++) {
+					pack(value[i]);
+				}
+			};
 			const pack = (value) => {
 				if (position > safeEnd)
 					target = makeRoom(position);
@@ -1450,22 +1467,8 @@
 						let constructor = value.constructor;
 						if (constructor === Object) {
 							writeObject(value, true);
-						} else if (constructor === Array || Array.isArray(value)) {
-							length = value.length;
-							if (length < 0x10) {
-								target[position++] = 0x90 | length;
-							} else if (length < 0x10000) {
-								target[position++] = 0xdc;
-								target[position++] = length >> 8;
-								target[position++] = length & 0xff;
-							} else {
-								target[position++] = 0xdd;
-								targetView.setUint32(position, length);
-								position += 4;
-							}
-							for (let i = 0; i < length; i++) {
-								pack(value[i]);
-							}
+						} else if (constructor === Array) {
+							packArray(value);
 						} else if (constructor === Map) {
 							length = value.size;
 							if (length < 0x10) {
@@ -1494,7 +1497,16 @@
 											target[position++] = extension.type;
 											target[position++] = 0;
 										}
-										pack(extension.write.call(this, value));
+										let writeResult = extension.write.call(this, value);
+										if (writeResult === value) { // avoid infinite recursion
+											if (Array.isArray(value)) {
+												packArray(value);
+											} else {
+												writeObject(value);
+											}
+										} else {
+											pack(writeResult);
+										}
 										return
 									}
 									let currentTarget = target;
@@ -1531,8 +1543,13 @@
 									return
 								}
 							}
-							// no extension found, write as object
-							writeObject(value, !value.hasOwnProperty); // if it doesn't have hasOwnProperty, don't do hasOwnProperty checks
+							// check isArray after extensions, because extensions can extend Array
+							if (Array.isArray(value)) {
+								packArray(value);
+							} else {
+								// no extension found, write as object
+								writeObject(value, !value.hasOwnProperty); // if it doesn't have hasOwnProperty, don't do hasOwnProperty checks
+							}
 						}
 					}
 				} else if (type === 'boolean') {