msgpackr 1.9.3 → 1.9.5-debug.1

package/index.d.cts

@@ -7,13 +7,14 @@ export enum FLOAT32_OPTIONS {
 
 export interface Options {
 	useFloat32?: FLOAT32_OPTIONS
-	useRecords?: boolean | (value:any)=> boolean
+	useRecords?: boolean | ((value:any)=> boolean)
 	structures?: {}[]
 	moreTypes?: boolean
 	sequential?: boolean
 	structuredClone?: boolean
 	mapsAsObjects?: boolean
 	variableMapSize?: boolean
+	coercibleKeyAsNumber?: boolean
 	copyBuffers?: boolean
 	bundleStrings?: boolean
 	useTimestamp32?: boolean

package/index.d.ts

@@ -7,13 +7,14 @@ export enum FLOAT32_OPTIONS {
 
 export interface Options {
 	useFloat32?: FLOAT32_OPTIONS
-	useRecords?: boolean | (value:any)=> boolean
+	useRecords?: boolean | ((value:any)=> boolean)
 	structures?: {}[]
 	moreTypes?: boolean
 	sequential?: boolean
 	structuredClone?: boolean
 	mapsAsObjects?: boolean
 	variableMapSize?: boolean
+	coercibleKeyAsNumber?: boolean
 	copyBuffers?: boolean
 	bundleStrings?: boolean
 	useTimestamp32?: boolean

package/pack.js

@@ -488,8 +488,13 @@ export class Packr extends Unpackr {
 						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')
@@ -534,7 +539,7 @@ export class Packr extends Unpackr {
 			}
 		}
 
-		const writePlainObject = this.variableMapSize ? (object) => {
+		const writePlainObject = (this.variableMapSize || this.coercibleKeyAsNumber) ? (object) => {
 			// this method is slightly slower, but generates "preferred serialization" (optimally small for smaller objects)
 			let keys = Object.keys(object)
 			let length = keys.length
@@ -550,9 +555,19 @@ export class Packr extends Unpackr {
 				position += 4
 			}
 			let key
-			for (let i = 0; i < length; i++) {
-				pack(key = keys[i])
-				pack(object[key])
+			if (this.coercibleKeyAsNumber) {
+				for (let i = 0; i < length; i++) {
+					key = keys[i]
+					let num = Number(key)
+					pack(isNaN(num) ? key : num)
+					pack(object[key])
+				}
+
+			} else {
+				for (let i = 0; i < length; i++) {
+					pack(key = keys[i])
+					pack(object[key])
+				}
 			}
 		} :
 		(object, safePrototype) => {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "msgpackr",
   "author": "Kris Zyp",
-  "version": "1.9.3",
+  "version": "1.9.5-debug.1",
   "description": "Ultra-fast MessagePack implementation with extensions for records and structured cloning",
   "license": "MIT",
   "types": "./index.d.ts",

package/struct.js

@@ -717,11 +717,16 @@ function readStruct(src, position, srcEnd, unpackr) {
 				objectLiteralProperties.push('__proto__:this');
 			}
 			let toObject = (new Function(...args, 'return function(s){return{' + objectLiteralProperties.join(',') + '}}')).apply(null, properties.map(prop => prop.get));
-			Object.defineProperty(prototype, 'toJSON', {
-				value(omitUnderscoredProperties) {
-					return toObject.call(this, this[sourceSymbol]);
-				}
-			});
+			try {
+				Object.defineProperty(prototype, 'toJSON', {
+					value(omitUnderscoredProperties) {
+						return toObject.call(this, this[sourceSymbol]);
+					}
+				});
+			} catch(error) {
+				error.message += ' setting properties ' + JSON.stringify(properties);
+				throw error;
+			}
 		} else {
 			Object.defineProperty(prototype, 'toJSON', {
 				value(omitUnderscoredProperties) {