Update app/routes/analytics.py

app/routes/analytics.py

@@ -88,19 +88,33 @@ async def get_all_reports(
     current_user: Dict[str, Any] = Depends(require_role("doctor", "lab_tech", "admin")),
 ):
     """Returns all extracted clinical pathology reports in the EMR vault."""
+    import json
     reports = db.query(PathologyReport).order_by(PathologyReport.created_at.desc()).all()
     out = []
     for r in reports:
+        # Safely parse findings JSON
+        parsed_findings = []
+        if isinstance(r.findings, list):
+            parsed_findings = r.findings
+        elif isinstance(r.findings, str) and r.findings.strip():
+            try:
+                parsed_findings = json.loads(r.findings)
+            except Exception:
+                parsed_findings = []
+
         out.append({
             "id": str(r.id),
+            "document_id": str(r.document_id) if r.document_id else "",
             "file_id": str(r.document_id) if r.document_id else "",
             "patient_name": r.patient_name or "Unknown Patient",
             "patient_id": r.patient_id or "N/A",
             "test_type": r.test_type or "General Pathology",
-            "test_date": str(r.test_date) if r.test_date else "",
-            "diagnosis": r.diagnosis or "",
-            "findings": r.findings if isinstance(r.findings, list) else [],
-            "clinical_summary": r.clinical_summary or "",
+            "test_date": str(r.test_date) if r.test_date else "Recent",
+            "diagnosis": r.diagnosis or "Normal",
+            "findings": parsed_findings,
+            "findings_count": len(parsed_findings),
+            "summary": r.summary or "",
+            "clinical_summary": r.summary or "",
             "created_at": str(r.created_at) if r.created_at else "",
         })
     return {"status": "success", "reports": out, "total": len(out)}