[Audit S-5] message_structure (MSH-9.3) wird befüllt, aber nie konsumiert

[Pappet]

Schweregrad

Niedrig (toter/spekulativer Code).

Problem

msg.message_structure wird aus MSH-9.3 befüllt (src/hl7/parser.rs:60-64) und serialisiert (src/hl7/types.rs:59-62). Der Doc-Kommentar behauptet „Used by Milestone 3 validation to select the correct structure definition" — validate_message matcht jedoch ausschließlich auf msg.message_type (src/validation.rs:46), nie auf die Struktur.

→ Spekulativ vorab eingeführtes, derzeit totes Feld (KI-Artefakt: Lösung für ein noch nicht existierendes Problem).

Lösungsvorschlag

Entweder die struktur-basierte Validierung tatsächlich implementieren, oder den Doc-Kommentar entschärfen und das Feld als „derzeit nur zur Anzeige" markieren, bis es genutzt wird.

Teil des Vibecoding-Code-Audits (S-5).

[Pappet]

Architektonischer Lösungsvorschlag — #186

Root Cause

Dateien/Zeilen: src/hl7/parser.rs:60-64, src/hl7/types.rs:59-62, src/hl7/validation.rs:46

msg.message_structure wird in enrich_msh() zuverlässig aus MSH-9.3 befüllt (parser.rs:60-64), und der Doc-Kommentar auf dem Feld (types.rs:60) lautet "Used by Milestone 3 validation". Die Realität: validate_message (validation.rs:46) dispatched ausschließlich auf msg.message_type — message_structure wird nie abgefragt.

Das Feld ist ein klassisches KI-Artefakt: vorausschauend für Milestone 3 eingeführt, aber nie rückgekoppelt.

Lösungsvorschlag

Zwei Pfade, je nach Milestone-3-Plan:

Option A — Struktur-basierte Validierung implementieren (empfohlen)

HL7 v2.x definiert für jede Message Structure (z. B. ADT_A01, ORU_R01, MDM_T02) eine exakte Segment-Sequenz. Die Validierung könnte wie folgt auf message_structure umgestellt werden:

// Neue dispatch-Tabelle in validation.rs
pub fn validate_message(msg: &Hl7Message) -> Vec<ValidationWarning> {
    let mut warnings = Vec::new();
    validate_msh(msg, &mut warnings);

    // Struktur-basierte Validierung (falls vorhanden)
    if let Some(structure) = &msg.message_structure {
        validate_structure(msg, structure, &mut warnings);
    } else {
        // Fallback: Type-basiert (aktuelles Verhalten)
        validate_by_message_type(msg, &mut warnings);
    }

    validate_data_types(msg, &mut warnings);
    warnings
}

Dazu eine validate_structure()-Funktion, die die erwarteten Segmente testet:

fn validate_structure(msg: &Hl7Message, structure: &str, warnings: &mut Vec<ValidationWarning>) {
    // Struktur-Tabelle aus message_types.rs oder hardcoded
    let expected_segments = match structure {
        "ADT_A01" => &["MSH", "EVN", "PID", "PV1"][..],
        "ORU_R01" => &["MSH", "PID", "OBR", "OBX"][..],
        "MDM_T02" => &["MSH", "EVN", "PID", "PV1", "TXA", "OBX"][..],
        // Fallback: segments aus message_type_description ableiten
        _ => return,
    };
    // Prüfen, ob alle expected_segments in der richtigen Reihenfolge vorkommen
    // ...
}

Option B — Feld als „Display-only" deklarieren

Falls Milestone 3 nicht priorisiert wird, genügt eine minimale Änderung:

// types.rs:60 - Doc-Kommentar entschärfen
/// MSH-9.3 message structure identifier, e.g. `"ADT_A01"` or `"ORU_R01"`.
/// Currently populated for display/debugging purposes only.
/// Future: planned for structure-based validation in the message_types module.
pub message_structure: Option<String>,

Implementierungs-Reihenfolge

1. Nächstes Milestone 3: Option A implementieren — nach Issue [Audit S-4] Repetition / Sub-Komponente / Escape werden nie verarbeitet #185 (Repetition/Sub-Komponente), weil die korrekte Feld-Struktur Voraussetzung für Segment-Validierung ist.
2. Vor Milestone 3: Option B (Doc-Update) sofort, um Verwirrung zu vermeiden.

Risiken/Nebenwirkungen

• Option A: Breite der Struktur-Tabelle. Pro Structure müssen die erwarteten Segmente definiert werden. Initial beschränken auf die 6 Message-Typen, die validate_message aktuell matcht (ADT, ORU, ORM, OML, SIU, MDM).
• Option A + [Audit S-4] Repetition / Sub-Komponente / Escape werden nie verarbeitet #185: Wenn [Audit S-4] Repetition / Sub-Komponente / Escape werden nie verarbeitet #185 Hl7Field mit repetitions ändert, ändert sich auch die Feld-Zugriffs-Logik in get_field_value. Das muss koordiniert werden.
• Fallback-Verhalten: Structure-basierte Validierung sollte zusätzlich zur Type-basierten laufen, nicht stattdessen — beide decken unterschiedliche Aspekte ab (Segmente vs. Felder)."