msgpackr 1.7.1 → 1.7.2

package/dist/test.js

@@ -826,7 +826,7 @@
 			})
 		}
 		else
-			throw new Error('Unknown extension type ' + type)
+			throw new Error('Unknown extension type ' + type)``
 	}
 
 	var keyCache = new Array(4096);
@@ -841,7 +841,7 @@
 				return readFixedString(length)
 		} else { // not cacheable, go back and do a standard read
 			position--;
-			return read()
+			return read().toString()
 		}
 		let key = ((length << 5) ^ (length > 1 ? dataView.getUint16(position) : length > 0 ? src[position] : 0)) & 0xfff;
 		let entry = keyCache[key];
@@ -895,16 +895,7 @@
 
 	// the registration of the record definition extension (as "r")
 	const recordDefinition = (id, highByte) => {
-		let structure;
-		if (currentUnpackr.freezeData) {
-			currentUnpackr.freezeData = false;
-			try {
-				structure = read();
-			} finally {
-				currentUnpackr.freezeData = true;
-			}
-		} else
-			structure = read();
+		let structure = read().map(property => property.toString()); // ensure that all keys are strings and that the array is mutable
 		let firstByte = id;
 		if (highByte !== undefined) {
 			id = id < 32 ? -((highByte << 5) + id) : ((highByte << 5) + id);
@@ -1180,20 +1171,44 @@
 						writeStruct(value);
 					else
 						pack(value);
+					let lastBundle = bundledStrings$1;
+					if (bundledStrings$1)
+						writeBundles(start, pack, 0);
 					if (referenceMap && referenceMap.idsToInsert) {
-						let incrementPosition = referenceMap.idsToInsert.length * 6;
-						if (bundledStrings$1)
-							writeBundles(start, pack, incrementPosition);
-						position$1 += incrementPosition;
+						let idsToInsert = referenceMap.idsToInsert.sort((a, b) => a.offset > b.offset ? 1 : -1);
+						let i = idsToInsert.length;
+						let incrementPosition = -1;
+						while (lastBundle && i > 0) {
+							let insertionPoint = idsToInsert[--i].offset + start;
+							if (insertionPoint < (lastBundle.stringsPosition + start) && incrementPosition === -1)
+								incrementPosition = 0;
+							if (insertionPoint > (lastBundle.position + start)) {
+								if (incrementPosition >= 0)
+									incrementPosition += 6;
+							} else {
+								if (incrementPosition >= 0) {
+									// update the bundle reference now
+									targetView.setUint32(lastBundle.position + start,
+										targetView.getUint32(lastBundle.position + start) + incrementPosition);
+									incrementPosition = -1; // reset
+								}
+								lastBundle = lastBundle.previous;
+								i++;
+							}
+						}
+						if (incrementPosition >= 0 && lastBundle) {
+							// update the bundle reference now
+							targetView.setUint32(lastBundle.position + start,
+								targetView.getUint32(lastBundle.position + start) + incrementPosition);
+						}
+						position$1 += idsToInsert.length * 6;
 						if (position$1 > safeEnd)
 							makeRoom(position$1);
 						packr.offset = position$1;
-						let serialized = insertIds(target.subarray(start, position$1), referenceMap.idsToInsert);
+						let serialized = insertIds(target.subarray(start, position$1), idsToInsert);
 						referenceMap = null;
 						return serialized
 					}
-					if (bundledStrings$1)
-						writeBundles(start, pack, 0);
 					packr.offset = position$1; // update the offset so next serialization doesn't write over our buffer, but can continue writing to same buffer sequentially
 					if (encodeOptions & REUSE_BUFFER_MODE) {
 						target.start = start;
@@ -1251,13 +1266,15 @@
 							let maxBytes = (bundledStrings$1[0] ? bundledStrings$1[0].length * 3 + bundledStrings$1[1].length : 0) + 10;
 							if (position$1 + maxBytes > safeEnd)
 								target = makeRoom(position$1 + maxBytes);
-							if (bundledStrings$1.position) { // here we use the 0x62 extension to write the last bundle and reserve sapce for the reference pointer to the next/current bundle
+							let lastBundle;
+							if (bundledStrings$1.position) { // here we use the 0x62 extension to write the last bundle and reserve space for the reference pointer to the next/current bundle
+								lastBundle = bundledStrings$1;
 								target[position$1] = 0xc8; // ext 16
 								position$1 += 3; // reserve for the writing bundle size
 								target[position$1++] = 0x62; // 'b'
 								extStart = position$1 - start;
 								position$1 += 4; // reserve for writing bundle reference
-								writeBundles(start, pack); // write the last bundles
+								writeBundles(start, pack, 0); // write the last bundles
 								targetView.setUint16(extStart + start - 3, position$1 - start - extStart);
 							} else { // here we use the 0x62 extension just to reserve the space for the reference pointer to the bundle (will be updated once the bundle is written)
 								target[position$1++] = 0xd6; // fixext 4
@@ -1266,6 +1283,7 @@
 								position$1 += 4; // reserve for writing bundle reference
 							}
 							bundledStrings$1 = ['', '']; // create new ones
+							bundledStrings$1.previous = lastBundle;
 							bundledStrings$1.size = 0;
 							bundledStrings$1.position = extStart;
 						}
@@ -1955,7 +1973,6 @@
 		let nextId;
 		let distanceToMove = idsToInsert.length * 6;
 		let lastEnd = serialized.length - distanceToMove;
-		idsToInsert.sort((a, b) => a.offset > b.offset ? 1 : -1);
 		while (nextId = idsToInsert.pop()) {
 			let offset = nextId.offset;
 			let id = nextId.id;
@@ -1976,6 +1993,7 @@
 	function writeBundles(start, pack, incrementPosition) {
 		if (bundledStrings$1.length > 0) {
 			targetView.setUint32(bundledStrings$1.position + start, position$1 + incrementPosition - bundledStrings$1.position - start);
+			bundledStrings$1.stringsPosition = position$1 - start;
 			let writeStrings = bundledStrings$1;
 			bundledStrings$1 = null;
 			pack(writeStrings[0]);
@@ -3161,7 +3179,25 @@
 			assert.equal(deserialized.uint16Array[0], 3);
 			assert.equal(deserialized.uint16Array[1], 4);
 		});
+		test('big bundledStrings', function() {
+			const MSGPACK_OPTIONS = {bundleStrings: true};
+			const packer = new Packr$1(MSGPACK_OPTIONS);
+			const unpacker = new Unpackr$1(MSGPACK_OPTIONS);
+
+			const payload = {
+				output: [
+					{
+						url: 'https://www.example.com/',
+					},
+				],
+			};
 
+			for (let i = 0; i < 10000; i++) {
+				payload.output.push(payload.output[0]);
+			}
+			let deserialized = unpacker.unpack(packer.pack(payload));
+			assert.equal(deserialized.output[0].url, payload.output[0].url);
+		});
 		test('structured clone with bundled strings', function() {
 			const packer = new Packr$1({
 				structuredClone: true, // both options must be enabled
@@ -3170,13 +3206,19 @@
 
 			const v = {};
 
-			const shared = {
+			let shared = {
 				name1: v,
-				name2: v, // one key has to be named `data`
+				name2: v,
 			};
 
 			let deserialized = packer.unpack(packer.pack(shared));
 			assert.equal(deserialized.name1, deserialized.name2);
+
+			shared = {};
+			shared.aaaa = shared; // key length >= 4
+
+			deserialized = packer.unpack(packer.pack(shared));
+			assert.equal(deserialized.aaaa, deserialized);
 		});
 
 		test('object without prototype', function(){