msgpackr 1.9.1 → 1.9.2

package/dist/node.cjs

@@ -122,10 +122,10 @@ class Unpackr {
 			let size = source.length;
 			let value = this ? this.unpack(source, size) : defaultUnpackr.unpack(source, size);
 			if (forEach) {
-				if (forEach(value) === false) return;
+				if (forEach(value, lastPosition, position$1) === false) return;
 				while(position$1 < size) {
 					lastPosition = position$1;
-					if (forEach(checkedRead()) === false) {
+					if (forEach(checkedRead(), lastPosition, position$1) === false) {
 						return
 					}
 				}
@@ -199,6 +199,10 @@ function checkedRead(options) {
 			position$1 = bundledStrings$1.postBundlePosition;
 			bundledStrings$1 = null;
 		}
+		if (sequentialMode)
+			// we only need to restore the structures if there was an error, but if we completed a read,
+			// we can clear this out and keep the structures we read
+			currentStructures.restoreStructures = null;
 
 		if (position$1 == srcEnd) {
 			// finished reading this source, cleanup references
@@ -971,7 +975,10 @@ const recordDefinition = (id, highByte) => {
 		structure.highByte = highByte;
 	}
 	let existingStructure = currentStructures[id];
-	if (existingStructure && existingStructure.isShared) {
+	// If it is a shared structure, we need to restore any changes after reading.
+	// Also in sequential mode, we may get incomplete reads and thus errors, and we need to restore
+	// to the state prior to an incomplete read in order to properly resume.
+	if (existingStructure && (existingStructure.isShared || sequentialMode)) {
 		(currentStructures.restoreStructures || (currentStructures.restoreStructures = []))[id] = existingStructure;
 	}
 	currentStructures[id] = structure;
@@ -1319,7 +1326,7 @@ class Packr extends Unpackr {
 					if (serializationsSinceTransitionRebuild < 10)
 						serializationsSinceTransitionRebuild++;
 					let sharedLength = structures.sharedLength || 0;
-					if (structures.length > sharedLength)
+					if (structures.length > sharedLength && !isSequential)
 						structures.length = sharedLength;
 					if (transitionsCount > 10000) {
 						// force a rebuild occasionally after a lot of transitions so it can get cleaned up
@@ -1954,7 +1961,10 @@ extensions = [{
 	}
 }, {
 	pack(set, allocateForWrite, pack) {
-		if (this.setAsEmptyObject) return pack({})
+		if (this.setAsEmptyObject) {
+			allocateForWrite(0);
+			return pack({})
+		}
 		let array = Array.from(set);
 		let { target, position} = allocateForWrite(this.moreTypes ? 3 : 0);
 		if (this.moreTypes) {