Update ui/insights_ui_generator.py

ui/insights_ui_generator.py

@@ -27,7 +27,7 @@ def format_report_for_display(report_data: Optional[pd.Series]) -> Dict[str, str
             <div class="empty-state-icon">📄</div>
             <div class="empty-state-title">Report Not Available</div>
             <div class="empty-state-description">
-                The selected report could not be loaded. Please try selecting a different report 
+                The selected report could not be loaded. Please try selecting a different report
                 or refresh the page.
             </div>
         </div>
@@ -43,7 +43,7 @@ def format_report_for_display(report_data: Optional[pd.Series]) -> Dict[str, str
     # Generate dynamic title and subtitle
     title = "Comprehensive Analysis Report"
     subtitle = ""
-    
+
     try:
         if report_type == 'Quarter':
             title = "📈 Quarterly Insights Report"
@@ -65,13 +65,26 @@ def format_report_for_display(report_data: Optional[pd.Series]) -> Dict[str, str
         {f'<p style="font-size: 1.1rem; color: #6b6b6b; margin-bottom: 0;">{subtitle}</p>' if subtitle else ''}
     </div>
     """
-    
+
     # The report_text itself should be the raw Markdown for the body
     body_markdown = report_text.strip()
-    
+
     return {'header_html': header_html, 'body_markdown': body_markdown}
 
-# The following functions (extract_key_results_for_selection, format_single_okr_for_display)
-# are no longer needed as their functionality is now handled by the new `ui/okr_ui_generator.py`
-# which directly formats the full OKR structure into HTML.
-# The 'format_report_to_markdown' function was also removed as it was redundant and not used.
+
+def format_report_to_markdown(report_string: Optional[str]) -> str:
+    """
+    This function was previously for general Markdown formatting.
+    Given the new structure where 'format_report_for_display' handles the split,
+    this function might become redundant or repurposed.
+    Keeping it for now but noting its potential redundancy depending on upstream calls.
+    """
+    if not report_string or not report_string.strip():
+        return "## Comprehensive Analysis Report\n\n*No analysis report was generated, or an error occurred during its generation.*"
+
+    # Simple formatting for now. Could be enhanced (e.g., looking for patterns like "Section X:" to make them H3)
+    formatted_report = f"## Comprehensive Analysis Report\n\n{report_string.strip()}"
+    return formatted_report
+
+# REMOVED: extract_key_results_for_selection function
+# REMOVED: format_single_okr_for_display function