Fix conversation verification export record id

src/interface/simplified_gradio_app.py

@@ -880,7 +880,10 @@ Changes apply only to your current session.
                     "start_time": verification_session.start_time.isoformat() if hasattr(verification_session, "start_time") else None,
                     "verification_records": [
                         {
-                            "record_id": r.record_id,
+                            # Conversation verification records use `exchange_id`.
+                            # Keep a `record_id` alias for backward compatibility with older exports.
+                            "exchange_id": getattr(r, "exchange_id", None),
+                            "record_id": getattr(r, "exchange_id", None),
                             "timestamp": r.timestamp.isoformat() if hasattr(r, "timestamp") else None,
                             "user_message": r.user_message,
                             "assistant_response": r.assistant_response,
@@ -2066,7 +2069,8 @@ To revert, use "Reset to Default" button.
             # If we already have dicts, build a lightweight VerificationRecord
             if isinstance(r, dict):
                 rec = VerificationRecord(
-                    record_id=r.get("record_id", ""),
+                    exchange_id=r.get("exchange_id") or r.get("record_id", ""),
+                    exchange_number=r.get("exchange_number", 0),
                     user_message=r.get("user_message", ""),
                     assistant_response=r.get("assistant_response", ""),
                     original_classification=r.get("original_classification", ""),
@@ -2099,7 +2103,9 @@ To revert, use "Reset to Default" button.
 
             records_as_dicts = [
                 {
-                    "record_id": r.record_id,
+                    "exchange_id": r.exchange_id,
+                    "exchange_number": r.exchange_number,
+                    "record_id": r.exchange_id,
                     "timestamp": r.timestamp,
                     "user_message": r.user_message,
                     "assistant_response": r.assistant_response,

tests/test_conversation_verification_export.py

@@ -0,0 +1,67 @@
+import json
+import os
+from datetime import datetime
+
+
+def test_conversation_verification_export_serializes_without_record_id(tmp_path, monkeypatch):
+    """Regression test: conversation verification records use exchange_id, not record_id."""
+
+    # Minimal stand-in objects (match the attributes used by open_verification_window export code)
+    class _Record:
+        def __init__(self):
+            self.exchange_id = "sess_1"
+            self.exchange_number = 1
+            self.timestamp = datetime(2025, 12, 12, 0, 0, 0)
+            self.user_message = "hi"
+            self.assistant_response = "hello"
+            self.original_classification = "YELLOW"
+            self.original_confidence = 0.9
+            self.original_indicators = ["stress"]
+            self.original_reasoning = "reason"
+            self.is_correct = None
+            self.correct_classification = None
+            self.correction_reason = None
+            self.verifier_notes = None
+
+    class _Session:
+        def __init__(self):
+            self.session_id = "verification_test"
+            self.patient_name = "Test"
+            self.verifier_name = "Verifier"
+            self.start_time = datetime(2025, 12, 12, 0, 0, 0)
+            self.verification_records = [_Record()]
+
+    vs = _Session()
+
+    # This mirrors the payload schema in simplified_gradio_app.open_verification_window
+    payload = {
+        "session_id": vs.session_id,
+        "patient_name": vs.patient_name,
+        "verifier_name": vs.verifier_name,
+        "start_time": vs.start_time.isoformat(),
+        "verification_records": [
+            {
+                "exchange_id": getattr(r, "exchange_id", None),
+                "record_id": getattr(r, "exchange_id", None),
+                "timestamp": r.timestamp.isoformat(),
+                "user_message": r.user_message,
+                "assistant_response": r.assistant_response,
+                "original_classification": r.original_classification,
+                "original_confidence": r.original_confidence,
+                "original_indicators": r.original_indicators,
+                "original_reasoning": r.original_reasoning,
+                "is_correct": r.is_correct,
+                "correct_classification": r.correct_classification,
+                "correction_reason": r.correction_reason,
+                "verifier_notes": r.verifier_notes,
+            }
+            for r in vs.verification_records
+        ],
+    }
+
+    out = tmp_path / "export.json"
+    out.write_text(json.dumps(payload, ensure_ascii=False, indent=2), encoding="utf-8")
+
+    loaded = json.loads(out.read_text(encoding="utf-8"))
+    assert loaded["verification_records"][0]["exchange_id"] == "sess_1"
+    assert loaded["verification_records"][0]["record_id"] == "sess_1"