Update api/routes.py

api/routes.py

@@ -135,6 +135,18 @@ def escape_latex_special_chars(text: str) -> str:
         text = text.replace(char, escape)
     return text
 
+def hyphenate_long_strings(text: str, max_length: int = 10) -> str:
+    """Insert LaTeX hyphenation points into long strings to prevent overfull hboxes."""
+    if not isinstance(text, str) or len(text) <= max_length:
+        return text
+    # Insert a discretionary hyphen (\-) every max_length characters
+    result = ""
+    for i, char in enumerate(text):
+        result += char
+        if (i + 1) % max_length == 0 and (i + 1) != len(text):
+            result += "\\-"
+    return result
+
 async def process_synthea_patient(bundle: dict, file_path: str) -> Optional[dict]:
     logger.debug(f"Processing patient from file: {file_path}")
     patient_data = {}
@@ -569,9 +581,8 @@ async def add_note(
 
 @router.get("/ehr/patients/{patient_id}/pdf", response_class=Response)
 async def generate_patient_pdf(patient_id: str, current_user: dict = Depends(get_current_user)):
-    # Temporarily enable logging to debug LaTeX compilation issues
-    original_log_level = logger.level
-    logger.setLevel(logging.DEBUG)
+    # Suppress logging for this route
+    logger.setLevel(logging.CRITICAL)
 
     try:
         if current_user.get('role') not in ['doctor', 'admin']:
@@ -588,45 +599,48 @@ async def generate_patient_pdf(patient_id: str, current_user: dict = Depends(get
         if not patient:
             raise HTTPException(status_code=404, detail="Patient not found")
 
-        # Format rows safely using escape_latex_special_chars with additional safety for None values
+        # Format rows safely, ensuring proper column counts and hyphenation
         note_rows = " \\\\\n".join(
             "{} & {} & {}".format(
-                escape_latex_special_chars(n.get("date", "") or ""),
-                escape_latex_special_chars(n.get("type", "") or ""),
-                escape_latex_special_chars(n.get("text", "") or "")
+                escape_latex_special_chars(hyphenate_long_strings(n.get("date", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(n.get("type", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(n.get("text", "") or ""))
             )
             for n in patient.get("notes", [])
-        )
+        ) or "No notes available \\\\"
+
         condition_rows = " \\\\\n".join(
             "{} & {} & {} & {} & {}".format(
-                escape_latex_special_chars(c.get("id", "") or ""),
-                escape_latex_special_chars(c.get("code", "") or ""),
-                escape_latex_special_chars(c.get("status", "") or ""),
-                escape_latex_special_chars(c.get("onset_date", "") or ""),
-                escape_latex_special_chars(c.get("verification_status", "") or "")
+                escape_latex_special_chars(hyphenate_long_strings(c.get("id", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(c.get("code", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(c.get("status", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(c.get("onset_date", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(c.get("verification_status", "") or ""))
             )
             for c in patient.get("conditions", [])
-        )
+        ) or "No conditions available \\\\"
+
         medication_rows = " \\\\\n".join(
             "{} & {} & {} & {} & {}".format(
-                escape_latex_special_chars(m.get("id", "") or ""),
-                escape_latex_special_chars(m.get("name", "") or ""),
-                escape_latex_special_chars(m.get("status", "") or ""),
-                escape_latex_special_chars(m.get("prescribed_date", "") or ""),
-                escape_latex_special_chars(m.get("dosage", "") or "")
+                escape_latex_special_chars(hyphenate_long_strings(m.get("id", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(m.get("name", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(m.get("status", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(m.get("prescribed_date", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(m.get("dosage", "") or ""))
             )
             for m in patient.get("medications", [])
-        )
+        ) or "No medications available \\\\"
+
         encounter_rows = " \\\\\n".join(
             "{} & {} & {} & {} & {}".format(
-                escape_latex_special_chars(e.get("id", "") or ""),
-                escape_latex_special_chars(e.get("type", "") or ""),
-                escape_latex_special_chars(e.get("status", "") or ""),
-                escape_latex_special_chars(e.get("period", {}).get("start", "") or ""),
-                escape_latex_special_chars(e.get("service_provider", "") or "")
+                escape_latex_special_chars(hyphenate_long_strings(e.get("id", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(e.get("type", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(e.get("status", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(e.get("period", {}).get("start", "") or "")),
+                escape_latex_special_chars(hyphenate_long_strings(e.get("service_provider", "") or ""))
             )
             for e in patient.get("encounters", [])
-        )
+        ) or "No encounters available \\\\"
 
         # Use Template for safe insertion
         latex_template = Template(r"""
@@ -636,6 +650,8 @@ async def generate_patient_pdf(patient_id: str, current_user: dict = Depends(get
 \usepackage{geometry}
 \geometry{margin=1in}
 \usepackage{booktabs,longtable,fancyhdr}
+\usepackage{array}  % Added for better table column formatting
+\setlength{\headheight}{14.5pt}  % Fix fancyhdr warning
 \pagestyle{fancy}
 \fancyhf{}
 \fancyhead[L]{Patient Report}
@@ -663,7 +679,7 @@ async def generate_patient_pdf(patient_id: str, current_user: dict = Depends(get
 \end{itemize}
 
 \section*{Clinical Notes}
-\begin{longtable}{p{3cm}p{3cm}p{7cm}}
+\begin{longtable}{>{\raggedright\arraybackslash}p{3cm}>{\raggedright\arraybackslash}p{3cm}>{\raggedright\arraybackslash}p{7cm}}
 \toprule
 \textbf{Date} & \textbf{Type} & \textbf{Text} \\
 \midrule
@@ -673,7 +689,7 @@ $notes
 \end{longtable}
 
 \section*{Conditions}
-\begin{longtable}{p{2cm}p{3cm}p{2cm}p{2cm}p{3cm}}
+\begin{longtable}{>{\raggedright\arraybackslash}p{2cm}>{\raggedright\arraybackslash}p{3cm}>{\raggedright\arraybackslash}p{2cm}>{\raggedright\arraybackslash}p{2cm}>{\raggedright\arraybackslash}p{3cm}}
 \toprule
 \textbf{ID} & \textbf{Code} & \textbf{Status} & \textbf{Onset} & \textbf{Verification} \\
 \midrule
@@ -683,7 +699,7 @@ $conditions
 \end{longtable}
 
 \section*{Medications}
-\begin{longtable}{p{2cm}p{4cm}p{2cm}p{2cm}p{2cm}}
+\begin{longtable}{>{\raggedright\arraybackslash}p{2cm}>{\raggedright\arraybackslash}p{4cm}>{\raggedright\arraybackslash}p{2cm}>{\raggedright\arraybackslash}p{2cm}>{\raggedright\arraybackslash}p{2cm}}
 \toprule
 \textbf{ID} & \textbf{Name} & \textbf{Status} & \textbf{Date} & \textbf{Dosage} \\
 \midrule
@@ -693,7 +709,7 @@ $medications
 \end{longtable}
 
 \section*{Encounters}
-\begin{longtable}{p{2cm}p{4cm}p{2cm}p{3cm}p{3cm}}
+\begin{longtable}{>{\raggedright\arraybackslash}p{2cm}>{\raggedright\arraybackslash}p{4cm}>{\raggedright\arraybackslash}p{2cm}>{\raggedright\arraybackslash}p{3cm}>{\raggedright\arraybackslash}p{3cm}}
 \toprule
 \textbf{ID} & \textbf{Type} & \textbf{Status} & \textbf{Start} & \textbf{Provider} \\
 \midrule
@@ -705,12 +721,12 @@ $encounters
 \end{document}
 """)
 
-        # Set the generated_on date to 03:46 PM CET, May 16, 2025
-        generated_on = datetime.strptime("2025-05-16 15:46:00+01:00", "%Y-%m-%d %H:%M:%S%z").strftime("%A, %B %d, %Y at %I:%M %p")
+        # Set the generated_on date to 03:52 PM CET, May 16, 2025
+        generated_on = datetime.strptime("2025-05-16 15:52:00+01:00", "%Y-%m-%d %H:%M:%S%z").strftime("%A, %B %d, %Y at %I:%M %p")
 
         latex_filled = latex_template.substitute(
             generated_on=generated_on,
-            fhir_id=escape_latex_special_chars(patient.get("fhir_id", "") or ""),
+            fhir_id=escape_latex_special_chars(hyphenate_long_strings(patient.get("fhir_id", "") or "")),
             full_name=escape_latex_special_chars(patient.get("full_name", "") or ""),
             gender=escape_latex_special_chars(patient.get("gender", "") or ""),
             dob=escape_latex_special_chars(patient.get("date_of_birth", "") or ""),
@@ -724,10 +740,10 @@ $encounters
             ]))),
             marital_status=escape_latex_special_chars(patient.get("marital_status", "") or ""),
             language=escape_latex_special_chars(patient.get("language", "") or ""),
-            notes=note_rows or "No notes available \\\\",
-            conditions=condition_rows or "No conditions available \\\\",
-            medications=medication_rows or "No medications available \\\\",
-            encounters=encounter_rows or "No encounters available \\\\"
+            notes=note_rows,
+            conditions=condition_rows,
+            medications=medication_rows,
+            encounters=encounter_rows
         )
 
         # Compile LaTeX in a temporary directory
@@ -739,25 +755,20 @@ $encounters
                 f.write(latex_filled)
 
             try:
-                # Added -f to force compilation for debugging
-                result = subprocess.run(
-                    ["latexmk", "-pdf", "-interaction=nonstopmode", "-f", tex_path],
+                subprocess.run(
+                    ["latexmk", "-pdf", "-interaction=nonstopmode", tex_path],
                     cwd=tmpdir,
                     check=True,
                     capture_output=True,
                     text=True
                 )
-                logger.debug(f"latexmk stdout: {result.stdout}")
-                logger.debug(f"latexmk stderr: {result.stderr}")
             except subprocess.CalledProcessError as e:
-                logger.error(f"LaTeX compilation failed: stdout={e.stdout}, stderr={e.stderr}")
                 raise HTTPException(
                     status_code=500,
                     detail=f"LaTeX compilation failed: stdout={e.stdout}, stderr={e.stderr}"
                 )
 
             if not os.path.exists(pdf_path):
-                logger.error("PDF file was not generated")
                 raise HTTPException(
                     status_code=500,
                     detail="PDF file was not generated"
@@ -774,17 +785,15 @@ $encounters
             return response
 
     except HTTPException as http_error:
-        logger.error(f"HTTP exception in generate_patient_pdf: {str(http_error.detail)}")
         raise http_error
     except Exception as e:
-        logger.error(f"Unexpected error generating PDF: {str(e)}", exc_info=True)
         raise HTTPException(
             status_code=500,
             detail=f"Unexpected error generating PDF: {str(e)}"
         )
     finally:
         # Restore the logger level for other routes
-        logger.setLevel(original_log_level)
+        logger.setLevel(logging.INFO)
 
 @router.post("/signup", status_code=status.HTTP_201_CREATED)
 async def signup(data: SignupForm):