@cosyte/dicom 0.1.2 → 0.1.4

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## 0.1.4
+
+### Patch Changes
+
+- d21f159: Decoding a string value now takes time linear in its length, so a long run of spaces, digits, or pad bytes no longer stalls `Element.value` or `parseSpecificCharacterSet()` for seconds.
+
+  Three checks on file-controlled text used patterns that backtracked quadratically on such a run: the trim of each value of a multi-valued string VR, the DS number grammar, and the trim of each Specific Character Set term. Each now scans the value once from each end and returns the same result as before. De-identification also strips every asterisk from the conditional action branch it takes, not only the first; no row of the Annex E table puts one there today, so no output changes.
+
+## 0.1.3
+
+### Patch Changes
+
+- 4a0989b: The README and the installation page now say Node.js 22 or later, tested on 22, 24, and 26, and the README's Node badge shows the exact `engines` range.
+
 ## 0.1.2
 
 ### Patch Changes

package/README.md

@@ -12,7 +12,7 @@
 [![npm version](https://img.shields.io/npm/v/@cosyte/dicom.svg)](https://www.npmjs.com/package/@cosyte/dicom)
 [![CI](https://img.shields.io/github/actions/workflow/status/cosyte/dicom/ci.yml?branch=main&label=CI)](https://github.com/cosyte/dicom/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://github.com/cosyte/dicom/blob/main/LICENSE)
-[![Node](https://img.shields.io/badge/node-%3E%3D22-brightgreen.svg)](https://nodejs.org)
+[![Node](https://img.shields.io/badge/node-%3E%3D22.0.0-brightgreen.svg)](https://nodejs.org)
 
 DICOM Part 10 parser + utility library for Node.js and TypeScript: metadata-first, vendor-quirky-tolerant, dual ESM/CJS.
 
@@ -47,7 +47,7 @@ Most software that touches medical imaging needs a handful of header fields and
 pnpm add @cosyte/dicom
 ```
 
-- **Node.js 22 or later, tested on 22 and 24.** `engines.node` is `>=22.0.0`.
+- **Node.js 22 or later, tested on 22, 24, and 26.** `engines.node` is `>=22.0.0`.
 - **Module format: dual ESM and CJS**, with per-condition type declarations, so `import` and `require` both resolve against the type that matches them.
 - **Strict TypeScript** (`noUncheckedIndexedAccess`, no `any`), with JSDoc and an `@example` on every public export feeding your editor's IntelliSense.
 

package/dist/index.cjs

@@ -11,7 +11,7 @@ var __export = (target, all) => {
 };
 
 // src/version.ts
-var VERSION = "0.1.2";
+var VERSION = "0.1.4";
 
 // src/dictionary/index.ts
 var dictionary_exports = {};
@@ -13223,7 +13223,18 @@ var MULTIBYTE_LABELS = /* @__PURE__ */ new Set([
 ]);
 function parseSpecificCharacterSet(bytes) {
   const raw = bytes.toString("latin1");
-  return raw.split("\\").map((t) => t.replace(/[ ]+$/u, "").replace(/^[ ]+/u, "")).map((t) => t === "" || TERM_TO_LABEL.has(t) ? t : WITHHELD);
+  return raw.split("\\").map(trimTermPad).map((t) => t === "" || TERM_TO_LABEL.has(t) ? t : WITHHELD);
+}
+function trimTermPad(term) {
+  let end = term.length;
+  while (end > 0) {
+    const code = term.charCodeAt(end - 1);
+    if (code !== 0 && code !== 32) break;
+    end--;
+  }
+  let start = 0;
+  while (start < end && term.charCodeAt(start) === 32) start++;
+  return term.slice(start, end);
 }
 function isKnownCharsetTerm(term) {
   return TERM_TO_LABEL.has(term);
@@ -13790,6 +13801,14 @@ function stripPad(bytes, tag, vr, position, warnings) {
   }
   return bytes.subarray(0, end);
 }
+function trimSpaces(value) {
+  let start = 0;
+  let end = value.length;
+  while (start < end && value.charCodeAt(start) === SPACE2) start++;
+  while (end > start && value.charCodeAt(end - 1) === SPACE2) end--;
+  return value.slice(start, end);
+}
+var DECIMAL_STRING = /^[+-]?(?:\d+(?:\.\d*)?|\.\d+)(?:[eE][+-]?\d+)?$/u;
 function attachWarnings(value, warnings) {
   return warnings.length > 0 ? { ...value, warnings } : value;
 }
@@ -13841,7 +13860,7 @@ function decodeElementValue(element) {
   }
   if (vr === "LO" || vr === "SH" || vr === "UC") {
     const text = decodeCharsetString(trimmed, tag, vr, charset, position, warnings);
-    const values = text.split("\\").map((v) => v.replace(/^ +| +$/gu, ""));
+    const values = text.split("\\").map(trimSpaces);
     return attachWarnings({ kind: "strings", values }, warnings);
   }
   if (vr === "LT" || vr === "ST" || vr === "UT") {
@@ -13854,7 +13873,7 @@ function decodeElementValue(element) {
   }
   if (vr === "AE" || vr === "CS" || vr === "AS" || vr === "UI") {
     const text = decodeAsciiString(trimmed, tag, vr, position, warnings);
-    const values = text.split("\\").map((v) => v.replace(/^ +| +$/gu, ""));
+    const values = text.split("\\").map(trimSpaces);
     return attachWarnings({ kind: "strings", values }, warnings);
   }
   if (vr === "DS") {
@@ -13862,7 +13881,7 @@ function decodeElementValue(element) {
     const values = text.split("\\").map((v) => {
       const t = v.trim();
       if (t.length === 0) return null;
-      if (!/^[+-]?(\d+\.?\d*|\.\d+)([eE][+-]?\d+)?$/u.test(t)) return null;
+      if (!DECIMAL_STRING.test(t)) return null;
       const n = Number(t);
       return Number.isFinite(n) ? n : null;
     });
@@ -16695,7 +16714,7 @@ function effectiveCode(action, active) {
   return action.basicProfile;
 }
 function resolveAction(code) {
-  const first = code.split("/", 1)[0]?.replace("*", "") ?? "K";
+  const first = code.split("/", 1)[0]?.replaceAll("*", "") ?? "K";
   return first;
 }
 var TEXT_DUMMIES = {