@cosyte/dicom 0.1.4 → 0.1.5

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.5
+
+### Patch Changes
+
+- c827734: `serializeDicom()` and `deidentify()` now write a value over 65,534 bytes in a VR with a 16-bit length field as `UN`, as PS3.5 section 6.2.2 requires, instead of throwing a `RangeError`.
+
+  Under the Explicit VR transfer syntaxes, VRs such as `CS`, `DS`, `LO`, `PN`, `UI` and `US` carry a 16-bit length field, and the element encoder put every length through it unchecked. A value the reader tolerated at the odd length FFFFH, padded to 65,536 bytes on write, made `serializeDicom()` throw a `RangeError` it does not document, and so did a multi-valued UID that `deidentify()` remapped past 65,534 bytes; inside a Sequence, `deidentify()` threw it itself while rebuilding the Item. Such a value is now written in the long form with VR `UN`, Little Endian under Explicit VR Big Endian too, and reads back byte for byte. A Private Creator, for which the same section forbids `UN`, is not rewritten.
+
 ## 0.1.4
 
 ### Patch Changes

package/dist/index.cjs

@@ -11,7 +11,7 @@ var __export = (target, all) => {
 };
 
 // src/version.ts
-var VERSION = "0.1.4";
+var VERSION = "0.1.5";
 
 // src/dictionary/index.ts
 var dictionary_exports = {};
@@ -14943,6 +14943,7 @@ function toDate(value, options) {
   return Number.isNaN(instant.getTime()) ? void 0 : instant;
 }
 var UNDEFINED_LENGTH4 = 4294967295;
+var MAX_SHORT_FORM_LENGTH = 65534;
 var NULL_PAD_VRS = /* @__PURE__ */ new Set(["UI", "OB", "OW", "OF", "OD", "OL", "OV", "UN"]);
 function padValue(value, vr) {
   if (value.length % 2 === 0) return buffer.Buffer.from(value);
@@ -14961,6 +14962,19 @@ function uint32(n, littleEndian) {
   else buf.writeUInt32BE(n, 0);
   return buf;
 }
+function littleEndianValue(value, vr, littleEndian) {
+  const stride = BE_VR_STRIDE[vr];
+  if (littleEndian || stride === 0) return value;
+  const swapped = buffer.Buffer.from(value);
+  const whole = swapped.subarray(0, swapped.length - swapped.length % stride);
+  if (stride === 2) whole.swap16();
+  else if (stride === 4) whole.swap32();
+  else whole.swap64();
+  return swapped;
+}
+function isPrivateCreator(group, element) {
+  return group % 2 === 1 && element >= 16 && element <= 255;
+}
 function isFullSpanElement(el, encoding) {
   if (el.length === UNDEFINED_LENGTH4) return true;
   return el.vr === "SQ" && encoding !== "implicit";
@@ -14992,6 +15006,17 @@ function encodeDatasetElement(el, encoding, rawBytes = el.rawBytes) {
       value
     ]);
   }
+  if (value.length > MAX_SHORT_FORM_LENGTH && !isPrivateCreator(group, element)) {
+    return buffer.Buffer.concat([
+      uint16(group, littleEndian),
+      uint16(element, littleEndian),
+      buffer.Buffer.from("UN", "ascii"),
+      buffer.Buffer.from([0, 0]),
+      // reserved (PS3.5 §7.1.2)
+      uint32(value.length, littleEndian),
+      littleEndianValue(value, el.vr, littleEndian)
+    ]);
+  }
   return buffer.Buffer.concat([
     uint16(group, littleEndian),
     uint16(element, littleEndian),