Update app.py

app.py

@@ -231,44 +231,92 @@ def _filter_records(repo: str, pr: str, sha: str) -> List[dict]:
     return records[:MAX_ROWS]
 
 
-def _generate_summary_tables(record: dict) -> Tuple[List[List[str]], List[List[str]]]:
-    """Generate by_test and by_model summary tables from a record."""
+def _generate_html_tables(record: dict) -> Tuple[str, str]:
+    """Generate HTML tables with proper horizontal scrolling and full text display."""
     
     # By test table
-    by_test_rows = []
+    by_test_html = """
+    <div style="overflow-x: auto; max-width: 100%; border: 1px solid #ddd; border-radius: 8px;">
+        <table style="width: 100%; border-collapse: collapse; font-family: monospace; font-size: 13px;">
+            <thead>
+                <tr style="background-color: #f6f8fa; border-bottom: 2px solid #d0d7de;">
+                    <th style="padding: 12px; text-align: left; white-space: nowrap; min-width: 400px; position: sticky; left: 0; background-color: #f6f8fa; z-index: 10;">Test</th>
+                    <th style="padding: 12px; text-align: center; white-space: nowrap; width: 80px;">Failures</th>
+                    <th style="padding: 12px; text-align: left; white-space: nowrap; min-width: 600px;">Full error(s)</th>
+                </tr>
+            </thead>
+            <tbody>
+    """
+    
     by_test_data = record.get("by_test", {})
-    for test_name, test_info in by_test_data.items():
+    for idx, (test_name, test_info) in enumerate(by_test_data.items()):
         count = test_info.get("count", 0)
         errors = test_info.get("errors", {})
         
-        # Format errors with each on a NEW line, like GitHub table format
+        # Format errors with line breaks
         error_lines = []
         for err, cnt in errors.items():
-            error_lines.append(f"{cnt}× {err}")
+            error_lines.append(f"<div style='margin-bottom: 8px;'><strong>{cnt}×</strong> {err}</div>")
+        error_html = "".join(error_lines)
         
-        # Join with actual newlines - Gradio will render these in the table
-        error_str = "\n".join(error_lines)
+        row_bg = "#ffffff" if idx % 2 == 0 else "#f6f8fa"
         
-        by_test_rows.append([test_name, str(count), error_str])
+        by_test_html += f"""
+                <tr style="background-color: {row_bg}; border-bottom: 1px solid #d0d7de;">
+                    <td style="padding: 12px; white-space: nowrap; position: sticky; left: 0; background-color: {row_bg}; border-right: 2px solid #d0d7de; z-index: 5;">{test_name}</td>
+                    <td style="padding: 12px; text-align: center;">{count}</td>
+                    <td style="padding: 12px; white-space: normal;">{error_html}</td>
+                </tr>
+        """
+    
+    by_test_html += """
+            </tbody>
+        </table>
+    </div>
+    """
     
     # By model table
-    by_model_rows = []
+    by_model_html = """
+    <div style="overflow-x: auto; max-width: 100%; border: 1px solid #ddd; border-radius: 8px; margin-top: 20px;">
+        <table style="width: 100%; border-collapse: collapse; font-family: monospace; font-size: 13px;">
+            <thead>
+                <tr style="background-color: #f6f8fa; border-bottom: 2px solid #d0d7de;">
+                    <th style="padding: 12px; text-align: left; white-space: nowrap; min-width: 150px; position: sticky; left: 0; background-color: #f6f8fa; z-index: 10;">Model</th>
+                    <th style="padding: 12px; text-align: center; white-space: nowrap; width: 80px;">Failures</th>
+                    <th style="padding: 12px; text-align: left; white-space: nowrap; min-width: 600px;">Full error(s)</th>
+                </tr>
+            </thead>
+            <tbody>
+    """
+    
     by_model_data = record.get("by_model", {})
-    for model_name, model_info in by_model_data.items():
+    for idx, (model_name, model_info) in enumerate(by_model_data.items()):
         count = model_info.get("count", 0)
         errors = model_info.get("errors", {})
         
-        # Format errors with each on a NEW line
+        # Format errors with line breaks
         error_lines = []
         for err, cnt in errors.items():
-            error_lines.append(f"{cnt}× {err}")
+            error_lines.append(f"<div style='margin-bottom: 8px;'><strong>{cnt}×</strong> {err}</div>")
+        error_html = "".join(error_lines)
         
-        # Join with actual newlines
-        error_str = "\n".join(error_lines)
+        row_bg = "#ffffff" if idx % 2 == 0 else "#f6f8fa"
         
-        by_model_rows.append([model_name, str(count), error_str])
+        by_model_html += f"""
+                <tr style="background-color: {row_bg}; border-bottom: 1px solid #d0d7de;">
+                    <td style="padding: 12px; white-space: nowrap; position: sticky; left: 0; background-color: {row_bg}; border-right: 2px solid #d0d7de; z-index: 5;">{model_name}</td>
+                    <td style="padding: 12px; text-align: center;">{count}</td>
+                    <td style="padding: 12px; white-space: normal;">{error_html}</td>
+                </tr>
+        """
     
-    return by_test_rows, by_model_rows
+    by_model_html += """
+            </tbody>
+        </table>
+    </div>
+    """
+    
+    return by_test_html, by_model_html
 
 
 
@@ -359,8 +407,8 @@ def _generate_pytest_commands(record: dict) -> str:
 
 def query(repo: str, pr: str, sha: str) -> Tuple[
     str,  # metadata_info
-    List[List[str]],  # by_test_table
-    List[List[str]],  # by_model_table
+    str,  # by_test_html
+    str,  # by_model_html
     str,  # markdown_summary
     str,  # pytest_commands
     str,  # raw_json
@@ -375,8 +423,8 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     if not pr:
         return (
             "**Error:** PR number is required.",
-            [],
-            [],
+            "",
+            "",
             "",
             "",
             json.dumps({"error": "PR number is required."}, indent=2),
@@ -389,8 +437,8 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     if not records:
         return (
             f"**No records found** for PR {pr}.",
-            [],
-            [],
+            "",
+            "",
             "",
             "",
             json.dumps({"error": "No records found."}, indent=2),
@@ -413,8 +461,8 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     ]
     metadata_info = "\n\n".join(metadata_lines)
     
-    # Generate tables
-    by_test_rows, by_model_rows = _generate_summary_tables(latest_record)
+    # Generate HTML tables
+    by_test_html, by_model_html = _generate_html_tables(latest_record)
     
     # Generate markdown summary
     markdown_summary = _generate_markdown_summary(latest_record)
@@ -429,8 +477,8 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     
     return (
         metadata_info,
-        by_test_rows,
-        by_model_rows,
+        by_test_html,
+        by_model_html,
         markdown_summary,
         pytest_commands,
         raw_json,
@@ -503,23 +551,15 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
             
             gr.Markdown("---")
             gr.Markdown("### 📝 By Test")
+            gr.Markdown("*Scroll horizontally to see full test names and errors*")
             
-            by_test_table = gr.Dataframe(
-                headers=["Test", "Failures", "Full error(s)"],
-                wrap=True,
-                interactive=False,
-                row_count=(5, "dynamic"),  # Start with 5 rows, expand as needed
-            )
+            by_test_html = gr.HTML(label="Test Failures")
             
             gr.Markdown("---")
             gr.Markdown("### 🏷️ By Model")
+            gr.Markdown("*Scroll horizontally to see full error messages*")
             
-            by_model_table = gr.Dataframe(
-                headers=["Model", "Failures", "Full error(s)"],
-                wrap=True,
-                interactive=False,
-                row_count=(3, "dynamic"),  # Start with 3 rows, expand as needed
-            )
+            by_model_html = gr.HTML(label="Model Failures")
         
         with gr.Tab("📋 Copy for GitHub"):
             gr.Markdown(
@@ -574,8 +614,8 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
         else:
             return (
                 "Enter a PR number and click Search",
-                [],
-                [],
+                "",
+                "",
                 "",
                 "",
                 "",
@@ -588,8 +628,8 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
         inputs=[repo_box, pr_box, sha_box], 
         outputs=[
             metadata_box,
-            by_test_table,
-            by_model_table,
+            by_test_html,
+            by_model_html,
             markdown_output,
             pytest_output,
             json_view,
@@ -609,8 +649,8 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
         inputs=[repo_box, pr_box, sha_box], 
         outputs=[
             metadata_box,
-            by_test_table,
-            by_model_table,
+            by_test_html,
+            by_model_html,
             markdown_output,
             pytest_output,
             json_view,