msgpackr 1.11.13 → 1.12.0

package/dist/index-no-eval.cjs

@@ -1296,7 +1296,13 @@
 					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)
@@ -1362,7 +1368,14 @@
 							let newSharedData = prepareStructures(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;