From 6866d48f4e273fbf4d5c9e4d5cfa35820ce59893 Mon Sep 17 00:00:00 2001 From: Devguru Date: Fri, 27 Feb 2026 18:13:42 +0530 Subject: [PATCH] fix: Prevent silent field overwrite when mapping conflicting fields to PostLabelingDelay --- package/src/pyaslreport/modalities/asl/processor.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/package/src/pyaslreport/modalities/asl/processor.py b/package/src/pyaslreport/modalities/asl/processor.py index 4ec31318..82d63a17 100644 --- a/package/src/pyaslreport/modalities/asl/processor.py +++ b/package/src/pyaslreport/modalities/asl/processor.py @@ -242,7 +242,14 @@ def _rename_fields(self, session: Dict[str, Any]) -> None: for old_key, new_key in field_mappings.items(): if old_key in session: - session[new_key] = session[old_key] + if new_key in session and session[new_key] != session[old_key]: + import logging + logging.warning( + f"Field conflict: '{old_key}' ({session[old_key]}) maps to '{new_key}' " + f"which already has value {session[new_key]}. Retaining existing '{new_key}'." + ) + else: + session[new_key] = session[old_key] del session[old_key] # Handle NumRFBlocks special case