Troubleshooting & known limitations
@cosyte/dicom is built to be correct and honest about its edges rather than to claim more than
it delivers. Mis-reading a patient identifier, an image's signedness, or a rescale slope can cause
real clinical harm, so this page is the deliberate "do not over-trust" list: the error model, common
symptoms, and (critically) the explicit metadata-first boundary and the list of what is not
in scope. Everything here is a documented boundary, not a bug: the lenient parser never silently drops
or garbles data; where a limitation applies, the raw bytes are preserved (often with a warning), they
are simply not further interpreted.
When does it throw vs warn?
Only four unrecoverable Tier-3 conditions throw a DicomParseError; everything else is a warning
on ds.warnings.
import { parseDicom } from "@cosyte/dicom";
// Bytes that are not a Part 10 object: a structural fatal, not a tolerated quirk.
parseDicom(Buffer.from("plainly not a DICOM object, just ASCII bytes", "ascii"));
// throws DicomParseError (NOT_DICOM_PART_10)
| Fatal code (throws) | Meaning |
|---|---|
NOT_DICOM_PART_10 | No preamble/DICM and no recoverable File Meta: not a Part 10 object. |
INVALID_FILE_META | The File Meta group is present but structurally unreadable. |
UNSUPPORTED_TRANSFER_SYNTAX | The transfer syntax UID is not one of the four v1 syntaxes. |
EMPTY_INPUT | Zero-length input. |
Narrow on the caught error via err instanceof DicomParseError and err.code === FATAL_CODES.* (see
Tolerance & the warning model). Everything a real-world archive does short
of that (a missing preamble, an odd-length value, an off-spec VR, a group-length mismatch) is a
warning you triage, not an exception you catch.
Common symptoms
| Symptom | Likely cause | What to do |
|---|---|---|
ds.get("PatientName") is undefined | get takes the tag form, not a keyword | Use the tag (ds.get("00100010")), or resolve a keyword with Dictionary.byKeyword("PatientName")?.tag. |
ds.image.rescaleSlope is undefined | Rescale Slope was absent | This is by design. It is not defaulted to 1. Apply a fallback deliberately in your own code if the modality warrants it. |
ds.image.signed is undefined | Pixel Representation (0028,0103) was absent | Signedness is unknown, never guessed. Do not assume unsigned. |
A DICOM_VR_MISMATCH warning | The on-wire Explicit VR disagreed with the dictionary | The on-wire VR is used (Postel's Law: on the read path the sender's own declaration wins) and the deviation recorded; check the sender's encoding. |
A DICOM_PRIVATE_CREATOR_UNKNOWN warning | A private tag's creator is not in the active profile | The element degrades to UN; add the creator via a profile to resolve it. |
ds.get(tag)?.value is { kind: "binary" } for Pixel Data | Pixel data is exposed raw, never decoded | Expected. Decoding pixels is out of scope (see below). |
A DICOM_BURNED_IN_ANNOTATION_NOT_REMOVED warning after deidentify | The object may carry burned-in PHI in the pixels | Metadata de-id cannot clean pixels; route to a pixel-cleaning step before sharing. |
A DICOM_SQ_NOT_DESCENDED warning | A defined-length Implicit VR LE element resolved to SQ from the dictionary, but its value is not an (FFFE,E000) item stream | The bytes are kept intact on Element.rawBytes and the rest of the object parses, but Element.items is absent, so nothing can navigate inside it. deidentify() therefore empties that element rather than shipping bytes it cannot audit, unless RetainSafePrivate plus a profile vouched for it - see the next row. |
A DICOM_DEIDENT_UNDEFINED_VR_NOT_AUDITABLE warning after deidentify, and an attribute you expected is now empty | An element reached deidentify() whose on-wire VR is not one of the 34 PS3.5 §6.2 defines, so its bytes were never decoded as a Value Field | Fail-safe by design, and the file is malformed upstream: the usual cause is an under-declared Value Length earlier on, which leaves the reader mid-value so that leftover bytes are read as a Data Element header. report.undefinedVrElements names the byte offset and the byte length dropped - deliberately not a tag, because a fabricated header's tag bytes are themselves part of some element's value. Capped at 64 entries; the emptying is not. A file conformant to PS3.5 2026c never trips it, and an Implicit VR LE file cannot. |
A DICOM_DEIDENT_SEQUENCE_NOT_AUDITABLE warning after deidentify, and a sequence you expected is now empty | That SQ element reached deidentify() with no items, so its item stream could not be walked | Fail-safe by design: PS3.15 §E.1.1 obliges a de-identifier to reach listed attributes inside a Sequence of Items, and a run that cannot enumerate them must not pass them through. report.unauditableSequences names the tag and the byte length dropped (capped at 64; the emptying is not). Usually a sender-side encoding defect, but not always: a conformant file nested past this library's own NESTING_DEPTH_LIMIT of 64 is refused identically. |
A DICOM_DEIDENT_EMBEDDED_ATTRIBUTE_REMOVED warning after deidentify, and a value you expected is now empty | The sender over-declared that element's Value Length, so the element that followed it was absorbed into its value | The carrier is emptied rather than kept, because an attribute encoded inside a value is invisible to the PS3.15 action table. report.embeddedAttributes names the carrier and the tags that were hiding in it. The file is malformed at source - raise it with the sender. |
Keeping PHI out of logs
A Tier-2 warning is safe to log whole. Its message comes from a frozen registry keyed by the
code, and the only things substituted into it are structural: a tag, a VR checked against the closed
34-VR set, and input-derived numbers. No warning factory accepts a string read out of the document,
so ds.warnings holds codes, positions and registry prose, not patient data.
Two things that array does not cover:
- A
DicomParseErrorcarries asnippet: up to 16 bytes of the source as hex. Those are raw input bytes and the library does not redact them. Logerr.code,err.byteOffsetanderr.message; treaterr.snippetas PHI. - Value-decode deviations do not appear on
ds.warnings. Decode is lazy, so aDAin a legacy format or aUIwith the wrong pad surfaces on the decoded value's ownwarnings(el.value.warnings), never folded into the frozen dataset array. Those messages are built from the same registry and are equally safe, but a logger that only readsds.warningswill not see them at all.
A DeidentifyReport is safe to log except for uidMap, whose keys are the source UIDs read out
of the file. They are there so UID replacement stays consistent across a study, and a study UID is a
unique identifier: the rest of the report (tags, keywords, action codes, sequence context paths) is
composed from static tables and carries nothing.
The field-by-field split between identifiers and values is in
Tolerance.
Keep the same discipline in your own code: log w.code and w.position, not the element value.
What's not yet parsed, and what is out of scope
Depth tracks the code and never leads it. These are the deliberate boundaries, authored here so a reader never relies on something absent.
The metadata-first boundary (scope, by design)
@cosyte/dicom reads and writes DICOM metadata. These are permanent non-goals for this package.
Each is tracked as a future companion package, not a gap to be filled here:
- No pixel decoding. Pixel Data is exposed as a raw
Buffer(and, for encapsulated transfer syntaxes, its fragments) and is never decoded, decompressed, windowed, rescaled, or color-transformed. Rescale Slope/Intercept, Window Center/Width, and the LUT sequences are surfaced as metadata, but applying them to produce displayable pixels is deferred to@cosyte/dicom-pixel. - No DIMSE networking. There is no C-STORE / C-FIND / C-MOVE / C-ECHO, no SCU/SCP, no association
negotiation. This is a file/buffer library, not a PACS node. That is
@cosyte/dicom-net. - No DICOMweb. No QIDO-RS / WADO-RS / STOW-RS client or server: deferred to
@cosyte/dicomweb. - No pixel-level de-identification.
deidentifycleans metadata per PS3.15 Annex E; burned-in annotation is warned, never removed (DICOM_BURNED_IN_ANNOTATION_NOT_REMOVED). Pixel scrubbing belongs to@cosyte/dicom-pixel.
Boundaries within the metadata surface
-
Four v1 transfer syntaxes. Implicit VR LE, Explicit VR LE, Explicit VR BE, and Deflated Explicit VR LE are read and written. A compressed pixel stream inside those syntaxes is passed through byte-for-byte, never decompressed.
-
Only typed
FileMetafields round-trip.serializeDicomrecomputes a spec-clean File Meta group; File Meta elements outside the typed model are not preserved verbatim through the model. -
De-identification is metadata-only and fail-safe toward removal. Conditional Annex E codes collapse to their most-protective branch (no IOD Type-1 analysis); private attributes are removed by default unless a profile marks a creator's tags safe.
-
An element that over-declares its own Value Length hides the next element inside its value, and
deidentify()empties rather than keeps it. PS3.5 defines Value Length as the length of that element's Value Field; a sender that writes a larger number produces a file whose reading is self-consistent and in which the following element has been absorbed, header and all. Nothing on the wire says which length lied, soparseDicomreads it exactly as written and this is not recoverable at parse time. Whatdeidentify()does is narrower and fail-safe: before keeping a value, it checks whether the value's tail decodes - in the file's own transfer syntax - as whole Data Elements ending exactly at the end of the value, at least one of which this run would have acted on, and containing a byte the carrier's VR cannot legally hold (PS3.5 §6.1.3 and Table 6.1-1 permit five control characters in DICOM text; Table 6.2-1 decides which of them each VR may hold - all five inLT/ST/UT, ESC only inLO/SH/UC/PN, none elsewhere). If so the value is emptied,report.embeddedAttributesrecords the carrier and the tags found inside it, andDICOM_DEIDENT_EMBEDDED_ATTRIBUTE_REMOVEDis raised. Carriers are string VRs only, and the gap is live rather than theoretical: the identical over-declare into anOB,OW,UNorUScarrier still writes the identifier into de-identified output with no warning and no report entry. Arbitrary bytes are what those VRs are for, so no content test can tell a swallow from a legitimate value there. Treat a file that raised this warning as a file whose sender is malformed: other attributes in it may be carrying the same defect where it cannot be seen. -
An element whose on-wire VR is not a VR is emptied, and its diagnostic names no tag. Under an Explicit VR syntax the VR is two bytes the sender wrote and this parser trusts them (Postel's Law on the read path). If those two bytes are not one of the 34 PS3.5 §6.2 defines, nothing this library did to the element counts as decoding a Value Field: §6.2 requires every VR it does not yet define to be long-form with a 32-bit VL, and this parser reads an unrecognized VR short-form, so the length came from the wrong two bytes. PS3.15 §E.1.1's obligation cannot be discharged inside bytes that were never a value, so
deidentify()empties the element,report.undefinedVrElementsrecords it, andDICOM_DEIDENT_UNDEFINED_VR_NOT_AUDITABLEis raised. The record is capped at 64 entries; the emptying is not. The finding names a byte offset and no tag, uniquely among the report's findings, and that is deliberate. The header may have been fabricated out of the middle of some element's value, in which case the "tag" is four bytes of document content: on a syntheticSTcarrier holding"MR BRAIN SMITHSON"it renders as48544F53, four letters of the surname. An unrecognized VR written honestly at a correct length raises the same code with an ordinary tag, and nothing here can tell the two apart, so the tag is withheld on both routes rather than on a guess. The byte offset locates the element instead and is a position the parser counted. Where these come from: an under-declared Value Length upstream. The reader finishes the short value early, reads the remainder of the value that was actually encoded as the next Data Element header, and the element that genuinely followed is consumed as the fabricated element's value - which is how a(0010,0020)Patient ID used to reach de-identified output, silently and with a clean report. A file conformant to PS3.5 2026c never produces one, and an Implicit VR LE file cannot: there the VR comes from the dictionary.UNis one of the 34 and is unaffected. There is no exemption: unlike the sequence rule below,RetainSafePrivateplus aProfiledoes not keep such an element. If the element previously appeared inreport.embeddedAttributes(an undefined-VR carrier whose bytes happened to tile as Data Elements), it now appears inreport.undefinedVrElementsinstead. -
A sequence
deidentify()could not open is emptied, not kept, so expect data loss on a malformed file. Recursion is driven byElement.items. AnSQelement that has none is un-auditable: PS3.5 §7.5.1 says its value is "a DICOM Data Set composed of Data Elements", and PS3.15 §E.1.1 obliges an implementation claiming the Basic Profile to protect the listed attributes "whether contained in the top level Data Set or embedded in an Item of a Sequence of Items". A run that cannot enumerate them cannot discharge that, so it discharges it on the carrier: the element is replaced with a zero-item sequence,report.unauditableSequencesrecords the tag and the byte length dropped, andDICOM_DEIDENT_SEQUENCE_NOT_AUDITABLEis raised. That record is capped at 64 entries so a crafted file cannot amplify a per-element diagnostic; the emptying itself is never capped, so an array exactly 64 long means "at least 64". Before this, those bytes were written into de-identified output verbatim, identifiers and all, with a clean report. The accompanyingDICOM_SQ_NOT_DESCENDEDonds.warningssays why the parse refused, and that is usually a sender-side encoding defect worth raising with them. It is not always the sender: a conformant file whose sequences nest deeper than this library's ownNESTING_DEPTH_LIMITof 64 is refused the same way and loses that sequence too. Two shapes are exempt from the rule and still leak. A privateSQkept underRetainSafePrivateplus aProfileis kept verbatim and unaudited: the profile vouched for the element, and this rule runs after that decision. And an undefined-lengthUNvalue the CP-246 descent could not read as a sequence keepsvr === "UN"and raises nothing beyond a possibleDICOM_VR_MISMATCH. The rule cannot be extended to that one, because every ordinaryUNelement also hasitems === undefinedand applying it there would empty every unknown-VR element in every file. So for either, the reliable test is stillel.items === undefined, and a report is a record of what was reached, not a proof that everything was. -
Repeating-group rows are matched by mask, within the range the standard bounds them to.
(50xx,xxxx)Curve Data,(60xx,3000)Overlay Data and(60xx,4000)Overlay Comments are stated by the standard as a group mask rather than a single tag.deidentify()matches them in the sixteen even groups PS3.5 defines (6000-601Efor overlays,5000-501Efor curves), removes them, and records them in the report with arepeatingGroupfield naming the mask that matched. Even groups above the bound and odd groups are not overlay or curve groups and are left alone; odd groups are private and go through the private-attribute path instead. -
RetainLongitudinalTemporalmeans the standard's full-dates option, the less protective one. PS3.15 defines two longitudinal-temporal options, full dates and modified dates. This package exposes one name for both and it carries the full-dates column, so on the 169 attributes where the two columns disagree you keep the real value where modified-dates would have cleaned it. Activate it only when real dates are genuinely required; date shifting is not done at this layer.
Scope (non-goals)
- A parser + serializer + de-identifier for DICOM Part 10, metadata-first. Not a viewer, not a network stack, not a pixel toolkit.
- Pre-alpha, published on npm. The package is public on npm (
npm install @cosyte/dicom) but sits on the0.0.x-until-first-alpha ladder: pin an exact version, and expect the surface to keep moving until first alpha. For the current version, readnpm view @cosyte/dicom versionrather than a number written in a doc.
For the full public surface and the exact fields each view decodes, see the package's README.md and
the Core Concepts.