msgpackr 1.11.13 → 1.12.0

package/dist/node.cjs

@@ -1362,7 +1362,13 @@ class Packr extends Unpackr {
 				hasSharedUpdate = false;
 			let encodingError;
 			try {
-				if (packr.randomAccessStructure && value && typeof value === 'object') {
+				// readOnlyStructures: skip the random-access struct write path so NO new struct is
+				// minted. randomAccessStructure stays true (the struct READ path and the struct-safe
+				// integer boundary are preserved, so existing struct data still decodes), but objects
+				// fall through to the normal pack()->writeObject->writeRecord path and are written as
+				// classic shared-structure records (byte range 0x40-0x7f, disjoint from struct headers
+				// at 0x20-0x3f) — the bounded, width-agnostic encoding used before struct mode.
+				if (packr.randomAccessStructure && !packr.readOnlyStructures && value && typeof value === 'object') {
 					if (value.constructor === Object) writeStruct(value); // simple object
 					else if (value.constructor !== Map && !Array.isArray(value) && !extensionClasses.some(extClass => value instanceof extClass)) {
 						// allow user classes, if they don't need special handling (but do use toJSON if available)
@@ -1428,7 +1434,14 @@ class Packr extends Unpackr {
 						let newSharedData = prepareStructures$1(structures, packr);
 						if (!encodingError) { // TODO: If there is an encoding error, should make the structures as uninitialized so they get rebuilt next time
 							if (packr.saveStructures(newSharedData, newSharedData.isCompatible) === false) {
-								// get updated structures and try again if the update failed
+								// The save was declined (a concurrent writer updated the shared structures,
+								// or the store transaction did not durably commit). Our in-memory
+								// structures + transition trie may now reference record ids that were
+								// never persisted; re-packing as-is would re-emit the same record pointing
+								// at an unpersisted structure (-> "Record id is not defined" on decode).
+								// Mark structures uninitialized so the re-pack reloads durable structures
+								// via getStructures, rebuilds the transition trie, and re-mints + re-saves.
+								structures.uninitialized = true;
 								return packr.pack(value, encodeOptions)
 							}
 							packr.lastNamedStructuresLength = sharedLength;