Refactor session results retrieval to return a 2D list and update export button visibility logic

src/interface/manual_input_interface.py

@@ -281,7 +281,7 @@ class ManualInputController(ProgressTrackingMixin):
         except Exception as e:
             return False, f"❌ Error saving verification: {str(e)}", {}
     
-    def get_session_results(self) -> List[Dict[str, Any]]:
+    def get_session_results(self) -> List[List[str]]:
         """
         Get all results from the current session.
         
@@ -296,18 +296,21 @@ class ManualInputController(ProgressTrackingMixin):
         if not session:
             return []
         
-        results = []
+        # Gradio Dataframe renders most reliably with a 2D list (rows) when
+        # headers are provided. Returning dicts can show up as "[object Object]"
+        # in the browser.
+        results: List[List[str]] = []
         for record in session.verifications:
-            results.append({
-                "message": record.original_message,
-                "classifier_decision": record.classifier_decision.upper(),
-                "ground_truth": record.ground_truth_label.upper(),
-                "is_correct": "✓" if record.is_correct else "✗",
-                "confidence": f"{record.classifier_confidence * 100:.1f}%",
-                "indicators": ", ".join(record.classifier_indicators),
-                "notes": record.verifier_notes,
-                "timestamp": record.timestamp.strftime("%Y-%m-%d %H:%M:%S")
-            })
+            results.append([
+                record.original_message,
+                record.classifier_decision.upper(),
+                record.ground_truth_label.upper(),
+                "✓" if record.is_correct else "✗",
+                f"{record.classifier_confidence * 100:.1f}%",
+                ", ".join(record.classifier_indicators),
+                record.verifier_notes,
+                record.timestamp.strftime("%Y-%m-%d %H:%M:%S"),
+            ])
         
         return results
     
@@ -597,16 +600,10 @@ def create_manual_input_interface() -> gr.Blocks:
                 # Export options
                 gr.Markdown("## 💾 Export Options")
                 with gr.Column():
-                    # Keep the existing buttons to show a status message,
-                    # but also provide direct downloads via DownloadButton.
-                    export_csv_btn = StandardizedComponents.create_export_button("csv")
-                    download_csv_btn = gr.DownloadButton("⬇️ Download CSV", variant="secondary")
-
-                    export_json_btn = StandardizedComponents.create_export_button("json")
-                    download_json_btn = gr.DownloadButton("⬇️ Download JSON", variant="secondary")
-
-                    export_xlsx_btn = StandardizedComponents.create_export_button("xlsx")
-                    download_xlsx_btn = gr.DownloadButton("⬇️ Download XLSX", variant="secondary")
+                    # Hidden until there's at least one verified message
+                    download_csv_btn = gr.DownloadButton("⬇️ Download CSV", variant="secondary", visible=False)
+                    download_json_btn = gr.DownloadButton("⬇️ Download JSON", variant="secondary", visible=False)
+                    download_xlsx_btn = gr.DownloadButton("⬇️ Download XLSX", variant="secondary", visible=False)
                 
                 # Complete session
                 gr.Markdown("## 🏁 Session Control")
@@ -653,6 +650,9 @@ def create_manual_input_interface() -> gr.Blocks:
                     gr.Row(visible=True),           # manual_input_section
                     gr.Row(visible=True),           # results_history_section
                     session_info,                   # session_info_display
+                    gr.DownloadButton(visible=False),  # download_csv_btn
+                    gr.DownloadButton(visible=False),  # download_json_btn
+                    gr.DownloadButton(visible=False),  # download_xlsx_btn
                     message                         # status_message
                 )
             else:
@@ -661,6 +661,9 @@ def create_manual_input_interface() -> gr.Blocks:
                     gr.Row(visible=False),          # manual_input_section
                     gr.Row(visible=False),          # results_history_section
                     "Enter your name and click 'Start Session' to begin",  # session_info_display
+                    gr.DownloadButton(visible=False),  # download_csv_btn
+                    gr.DownloadButton(visible=False),  # download_json_btn
+                    gr.DownloadButton(visible=False),  # download_xlsx_btn
                     message                         # status_message
                 )
         
@@ -707,6 +710,9 @@ def create_manual_input_interface() -> gr.Blocks:
                     gr.Row(visible=False),          # correction_section
                     stats_text,                     # session_stats_display
                     results,                        # results_display
+                    gr.DownloadButton(visible=True),   # download_csv_btn
+                    gr.DownloadButton(visible=True),   # download_json_btn
+                    gr.DownloadButton(visible=True),   # download_xlsx_btn
                     message                         # status_message
                 )
             else:
@@ -716,6 +722,9 @@ def create_manual_input_interface() -> gr.Blocks:
                     gr.Row(visible=False),          # correction_section
                     gr.Markdown(value=""),          # session_stats_display (no change)
                     gr.Dataframe(value=[]),         # results_display (no change)
+                    gr.DownloadButton(),             # download_csv_btn (no change)
+                    gr.DownloadButton(),             # download_json_btn (no change)
+                    gr.DownloadButton(),             # download_xlsx_btn (no change)
                     message                         # status_message
                 )
         
@@ -744,6 +753,9 @@ def create_manual_input_interface() -> gr.Blocks:
                     "",                             # correction_notes (clear)
                     stats_text,                     # session_stats_display
                     results,                        # results_display
+                    gr.DownloadButton(visible=True),   # download_csv_btn
+                    gr.DownloadButton(visible=True),   # download_json_btn
+                    gr.DownloadButton(visible=True),   # download_xlsx_btn
                     message                         # status_message
                 )
             else:
@@ -754,14 +766,12 @@ def create_manual_input_interface() -> gr.Blocks:
                     notes,                          # correction_notes (keep)
                     gr.Markdown(value=""),          # session_stats_display (no change)
                     gr.Dataframe(value=[]),         # results_display (no change)
+                    gr.DownloadButton(),             # download_csv_btn (no change)
+                    gr.DownloadButton(),             # download_json_btn (no change)
+                    gr.DownloadButton(),             # download_xlsx_btn (no change)
                     message                         # status_message
                 )
         
-        def on_export_results(format_type):
-            """Handle results export."""
-            success, message, file_path = controller.export_session_results(format_type)
-            return message
-
         def on_export_results_file(format_type):
             """Handle results export for DownloadButton (returns file path)."""
             success, message, file_path = controller.export_session_results(format_type)
@@ -812,6 +822,9 @@ def create_manual_input_interface() -> gr.Blocks:
                 manual_input_section,
                 results_history_section,
                 session_info_display,
+                download_csv_btn,
+                download_json_btn,
+                download_xlsx_btn,
                 status_message
             ]
         )
@@ -836,6 +849,9 @@ def create_manual_input_interface() -> gr.Blocks:
                 correction_section,
                 session_stats_display,
                 results_display,
+                download_csv_btn,
+                download_json_btn,
+                download_xlsx_btn,
                 status_message
             ]
         )
@@ -855,35 +871,23 @@ def create_manual_input_interface() -> gr.Blocks:
                 correction_notes,
                 session_stats_display,
                 results_display,
+                download_csv_btn,
+                download_json_btn,
+                download_xlsx_btn,
                 status_message
             ]
         )
         
-        export_csv_btn.click(
-            lambda: on_export_results("csv"),
-            outputs=[status_message]
-        )
-
         download_csv_btn.click(
             lambda: on_export_results_file("csv"),
             outputs=[download_csv_btn]
         )
         
-        export_json_btn.click(
-            lambda: on_export_results("json"),
-            outputs=[status_message]
-        )
-
         download_json_btn.click(
             lambda: on_export_results_file("json"),
             outputs=[download_json_btn]
         )
         
-        export_xlsx_btn.click(
-            lambda: on_export_results("xlsx"),
-            outputs=[status_message]
-        )
-
         download_xlsx_btn.click(
             lambda: on_export_results_file("xlsx"),
             outputs=[download_xlsx_btn]