Update app.py

app.py

@@ -2,12 +2,9 @@
 #
 # Universal AI Data Analyst with:
 # - Unchanged analysis & assessment logic
-# - Top: Current Assessment / Assessment History (full width, fills screen)
-# - Bottom: New Assessment (files, prompt, actions, voice)
-# - Sleek "ChatGPT-ish" UI polish (soft dark, rounded, green accents)
-# - Fixed Gradio wiring (uses gr.State for history)
-# - Triple-quoted progress strings
-# - Voice-to-Text via browser Web Speech API
+# - Fixed Gradio event wiring (uses gr.State for history)
+# - Triple-quoted progress strings (no unterminated literals)
+# - Sleek full-width UI and Voice-to-Text (browser Web Speech API)
 # - Optional HIPAA flags (fallback defaults if not present in settings.py)
 
 from __future__ import annotations
@@ -33,8 +30,7 @@ from settings import (
     COHERE_TIMEOUT_S,   # noqa: F401
     USE_OPEN_FALLBACKS  # noqa: F401
 )
-
-# Optional HIPAA flags; default to safe, non-breaking values if absent.
+# Try to import optional HIPAA flags; fall back to safe defaults if not defined.
 try:
     from settings import PHI_MODE, PERSIST_HISTORY, HISTORY_TTL_DAYS, REDACT_BEFORE_LLM, ALLOW_EXTERNAL_PHI
 except Exception:
@@ -66,7 +62,7 @@ def _sanitize_text(s: str) -> str:
     return re2.sub(r"[\p{C}--[\n\t]]+", "", s)
 
 
-# Conservative PHI redaction patterns (applied only if PHI_MODE & REDACT_BEFORE_LLM are enabled)
+# Conservative PHI redaction patterns (only applied if PHI_MODE & REDACT_BEFORE_LLM are enabled)
 PHI_PATTERNS = [
     (re.compile(r"\b\d{3}-\d{2}-\d{4}\b"), "[REDACTED_SSN]"),
     (re.compile(r"\b\d{9}\b"), "[REDACTED_MRN]"),
@@ -92,6 +88,7 @@ def safe_log(event_name: str, meta: dict | None = None):
         meta.pop("raw", None)
         log_event(event_name, None, meta)
     except Exception:
+        # Never raise from logging
         pass
 
 
@@ -275,60 +272,61 @@ TERMS_OF_SERVICE_TEXT = load_markdown_text("terms_of_service.md")
 # ---------------------- Sleek UI assets (CSS/JS only) ----------------------
 
 SLEEK_CSS = """
-/* Global */
+/* Full-bleed, modern look */
 :root, body, #root, .gradio-container { height: 100%; }
-.gradio-container { padding: 0 !important; background: #0e1117; color: #e6e9ef; }
+.gradio-container { padding: 0 !important; }
 .block { padding: 0 !important; }
 
 /* Header */
 .header {
-  padding: 18px 22px;
-  background: linear-gradient(180deg, #11161f 0%, #0f141b 100%);
-  border-bottom: 1px solid #1e2633;
-  display: flex; align-items: center; justify-content: space-between; gap: 12px;
+  padding: 20px 28px;
+  background: linear-gradient(135deg, #0e1726, #1d2a44 60%, #243a5e);
+  color: #fff;
+  display: flex; align-items: center; justify-content: space-between;
+  gap: 16px;
 }
-.header h1 { margin: 0; font-size: 18px; font-weight: 600; letter-spacing: .2px; color: #eaf2ee; }
-.header .badge {
-  font-size: 12px; background: #1a2a1f; color: #a6e3b1; padding: 6px 10px; border-radius: 999px; border: 1px solid #23402a;
+.header h1 { margin: 0; font-size: 22px; letter-spacing: 0.3px; font-weight: 600; }
+.header .badge { font-size: 12px; opacity: 0.9; background:#ffffff22; padding:6px 10px; border-radius: 999px; }
+
+/* Main layout */
+.main {
+  display: grid;
+  grid-template-columns: 420px 1fr;
+  gap: 16px;
+  padding: 16px;
+  height: calc(100vh - 72px);
+  box-sizing: border-box;
 }
-
-/* App body: top (tabs) + bottom (input) */
-.app { display: flex; flex-direction: column; height: calc(100vh - 60px); }
-
-/* Top area */
-.top-area { flex: 1 1 auto; padding: 12px 12px 0 12px; box-sizing: border-box; }
-.top-card {
-  height: 100%; background: #0f141b; border: 1px solid #1e2633;
-  border-radius: 14px; overflow: hidden; display: flex; flex-direction: column;
-  box-shadow: 0 0 0 1px #0a0d12 inset, 0 8px 20px rgba(0,0,0,.35);
+.left, .right {
+  background: #0b1020;
+  color: #e9edf3;
+  border-radius: 16px;
+  border: 1px solid #1c2642;
 }
+.left { padding: 16px; display: flex; flex-direction: column; gap: 12px; }
+.right { padding: 0; display: flex; flex-direction: column; }
 
-/* Tabs */
-.tabs { flex: 1 1 auto; display: flex; flex-direction: column; }
-.tabitem { flex: 1 1 auto; display: flex; flex-direction: column; }
-#chatbot_container { flex: 1 1 auto; display: flex; }
-#chatbot_container .gr-chatbot { flex: 1 1 auto; height: 100%; }
-
-/* Bottom input area */
-.bottom-area { flex: 0 0 auto; padding: 10px 12px 12px 12px; box-sizing: border-box; }
-.bottom-card {
-  background: #0f141b; border: 1px solid #1e2633; border-radius: 14px; padding: 12px;
-  box-shadow: 0 0 0 1px #0a0d12 inset, 0 8px 20px rgba(0,0,0,.35);
-}
-.panel-title { font-size: 13px; font-weight: 600; color: #c9d4cf; margin-bottom: 4px; }
-.helper { font-size: 12px; color: #96a1a7; margin-bottom: 8px; }
+/* Panels */
+.panel-title { font-size: 14px; font-weight: 600; color: #aeb8cc; margin-bottom: 6px; }
+.helper { font-size: 12px; color: #97a3bb; margin-bottom: 8px; }
 
-/* Inputs & actions */
-.actions { display: flex; gap: 8px; }
+/* Sticky actions */
+.actions {
+  display: flex; gap: 8px; align-items: center; justify-content: stretch;
+}
 .actions .gr-button { flex: 1; }
-.gr-button.primary { background: #1f8a53 !important; border: 1px solid #2ea36a !important; }
-.gr-button.primary:hover { filter: brightness(1.05); }
 
-.voice-hint { font-size: 12px; color: #9fb0cc; margin-top: 4px; }
-.hr { height: 1px; background: #1a2230; margin: 10px 0; }
+/* Tabs full height */
+.right .tabs { height: 100%; display: flex; flex-direction: column; }
+.right .tabitem { flex: 1; display: flex; flex-direction: column; }
+#chatbot_container { flex: 1; }
+#chatbot_container .gr-chatbot { height: 100%; }
+
+/* Tiny separators */
+.hr { height: 1px; background: #16203b; margin: 10px 0; }
 
-/* Markdown / content */
-.markdown-body { color: #e6e9ef; }
+/* Voice hint */
+.voice-hint { font-size: 12px; color:#9fb0cc; margin-top: 4px; }
 """
 
 VOICE_STT_HTML = """
@@ -365,10 +363,10 @@ function rs_toggle_stt(elemId){
 """
 
 
-# ---------------------- UI: Top tabs (full width) + Bottom input ----------------------
+# ---------------------- Sleek UI (with fixed State wiring) ----------------------
 
 with gr.Blocks(theme=gr.themes.Soft(), css=SLEEK_CSS, fill_width=True) as demo:
-    # In-memory history component (fixes Gradio list/_id issue)
+    # Persistent in-memory history component (fixes list/_id error)
     assessment_history = gr.State([])
 
     # Header
@@ -378,65 +376,61 @@ with gr.Blocks(theme=gr.themes.Soft(), css=SLEEK_CSS, fill_width=True) as demo:
                "PHI Mode ON" if PHI_MODE else "PHI Mode OFF"
         gr.Markdown(f"<span class='badge'>{pill}</span>")
 
-    # App body: top tabs + bottom input
-    with gr.Column(elem_classes=["app"]):
-
-        # ===== TOP: Current Assessment / Assessment History =====
-        with gr.Row(elem_classes=["top-area"]):
-            with gr.Column(elem_classes=["top-card"]):
-                with gr.Tabs(elem_classes=["tabs"]):
-                    with gr.TabItem("Current Assessment", id=0, elem_classes=["tabitem"]):
-                        with gr.Column(elem_id="chatbot_container"):
-                            chat_history_output = gr.Chatbot(label="Analysis Output", type="messages")
-                    with gr.TabItem("Assessment History", id=1, elem_classes=["tabitem"]):
-                        gr.Markdown("### Review Past Assessments")
-                        history_dropdown = gr.Dropdown(label="Select an assessment to review", choices=[])
-                        history_display = gr.Markdown(label="Selected Assessment Details")
-
-        # ===== BOTTOM: New Assessment (files + prompt + actions) =====
-        with gr.Row(elem_classes=["bottom-area"]):
-            with gr.Column(elem_classes=["bottom-card"]):
-                gr.Markdown("<div class='panel-title'>New Assessment</div>")
-                gr.Markdown("<div class='helper'>Upload CSVs for analysis, or enter a prompt. Voice works in modern browsers.</div>")
-
-                files_input = gr.Files(
-                    label="Upload Data Files (.csv)",
-                    file_count="multiple",
-                    type="filepath",
-                    file_types=[".csv"],
+    # Main layout
+    with gr.Row(elem_classes=["main"]):
+        # Left panel
+        with gr.Column(elem_classes=["left"]):
+            gr.Markdown("<div class='panel-title'>New Assessment</div>")
+            gr.Markdown("<div class='helper'>Upload CSVs for analysis, or enter a prompt. Voice works in modern browsers.</div>")
+            files_input = gr.Files(
+                label="Upload Data Files (.csv)",
+                file_count="multiple",
+                type="filepath",
+                file_types=[".csv"],
+            )
+            prompt_input = gr.Textbox(
+                label="Prompt",
+                placeholder="Paste your scenario or question here...",
+                lines=12,
+                elem_id="prompt_box",
+                autofocus=True,
+            )
+
+            with gr.Row(elem_classes=["actions"]):
+                send_btn = gr.Button("▶️ Run Analysis", variant="primary")
+                clear_btn = gr.Button("🧹 Clear")
+                voice_btn = gr.Button("🎙️ Voice")
+
+            gr.Markdown("<div class='voice-hint'>Click Voice to start/stop dictation into the prompt box.</div>")
+            ping_btn = gr.Button("🔌 Ping Cohere")
+            ping_out = gr.Markdown()
+
+            gr.Markdown("<div class='hr'></div>")
+            if PHI_MODE:
+                gr.Markdown(
+                    "⚠️ **PHI Mode:** History persistence is disabled by default. Avoid unnecessary identifiers."
                 )
-                prompt_input = gr.Textbox(
-                    label="Prompt",
-                    placeholder="Paste your scenario or question here...",
-                    lines=6,
-                    elem_id="prompt_box",
-                    autofocus=True,
-                )
-
-                with gr.Row(elem_classes=["actions"]):
-                    send_btn = gr.Button("▶️ Run Analysis", variant="primary")
-                    clear_btn = gr.Button("🧹 Clear")
-                    voice_btn = gr.Button("��️ Voice")
-
-                gr.Markdown("<div class='voice-hint'>Click Voice to start/stop dictation into the prompt box.</div>")
-                ping_btn = gr.Button("🔌 Ping Cohere")
-                ping_out = gr.Markdown()
 
+            with gr.Accordion("Privacy & Terms", open=False):
+                gr.Markdown(PRIVACY_POLICY_TEXT)
                 gr.Markdown("<div class='hr'></div>")
-                if PHI_MODE:
-                    gr.Markdown(
-                        "⚠️ **PHI Mode:** History persistence is disabled by default. Avoid unnecessary identifiers."
-                    )
-
-                with gr.Accordion("Privacy & Terms", open=False):
-                    gr.Markdown(PRIVACY_POLICY_TEXT)
-                    gr.Markdown("<div class='hr'></div>")
-                    gr.Markdown(TERMS_OF_SERVICE_TEXT)
-
-    # Inject voice-to-text helper (browser-side only)
+                gr.Markdown(TERMS_OF_SERVICE_TEXT)
+
+        # Right panel
+        with gr.Column(elem_classes=["right"]):
+            with gr.Tabs(elem_classes=["tabs"]):
+                with gr.TabItem("Current Assessment", id=0, elem_classes=["tabitem"]):
+                    with gr.Column(elem_id="chatbot_container"):
+                        chat_history_output = gr.Chatbot(label="Analysis Output", type="messages")
+                with gr.TabItem("Assessment History", id=1, elem_classes=["tabitem"]):
+                    gr.Markdown("### Review Past Assessments")
+                    history_dropdown = gr.Dropdown(label="Select an assessment to review", choices=[])
+                    history_display = gr.Markdown(label="Selected Assessment Details")
+
+    # Inject voice-to-text helper
     gr.HTML(VOICE_STT_HTML)
 
-    # ================= Event logic (unchanged analysis flow) =================
+    # --------- Event logic (unchanged analysis flow) ----------
 
     def run_analysis_wrapper(prompt, files, chat_history_list, history_state_list):
         if not prompt:
@@ -444,11 +438,14 @@ with gr.Blocks(theme=gr.themes.Soft(), css=SLEEK_CSS, fill_width=True) as demo:
             yield chat_history_list, history_state_list, gr.update()
             return
 
+        # Append user's message
         chat_with_user_msg = _append_msg(chat_history_list, "user", prompt)
 
+        # Optional progress callback (not streaming in this UI)
         def dummy_update(message: str):
             pass
 
+        # Thinking bubble
         thinking_message = _append_msg(
             chat_with_user_msg,
             "assistant",
@@ -458,17 +455,21 @@ with gr.Blocks(theme=gr.themes.Soft(), css=SLEEK_CSS, fill_width=True) as demo:
         )
         yield thinking_message, history_state_list, gr.update()
 
+        # Run analysis/chat
         ai_response_text = handle(prompt, files, dummy_update)
 
+        # Append final assistant response
         final_chat = _append_msg(chat_with_user_msg, "assistant", ai_response_text)
         timestamp = datetime.now().strftime("%Y-%m-%d %H:%M:%S")
 
+        # Capture filenames (if any)
         file_names: List[str] = []
         if files:
             file_names = [
                 os.path.basename(f.name if hasattr(f, "name") else f) for f in files
             ]
 
+        # Build history record
         new_entry = {
             "id": timestamp,
             "prompt": prompt,
@@ -477,6 +478,7 @@ with gr.Blocks(theme=gr.themes.Soft(), css=SLEEK_CSS, fill_width=True) as demo:
             "chat_history": final_chat,
         }
 
+        # Respect PHI/history flags
         if PERSIST_HISTORY and (not PHI_MODE or (PHI_MODE and HISTORY_TTL_DAYS > 0)):
             updated_history: List[Dict[str, Any]] = (history_state_list or []) + [new_entry]
         else:
@@ -525,7 +527,7 @@ with gr.Blocks(theme=gr.themes.Soft(), css=SLEEK_CSS, fill_width=True) as demo:
 {chat_md}
 """
 
-    # Wire events (State component used for history)
+    # Wire events (using proper gr.State component for history)
     send_btn.click(
         run_analysis_wrapper,
         inputs=[prompt_input, files_input, chat_history_output, assessment_history],