Update app.py

app.py

@@ -240,14 +240,10 @@ def _generate_summary_tables(record: dict) -> Tuple[List[List[str]], List[List[s
     for test_name, test_info in by_test_data.items():
         count = test_info.get("count", 0)
         errors = test_info.get("errors", {})
-        # Format errors with line breaks between each error type
+        # Format errors as "Nx error_message" for each unique error
         error_list = [f"{cnt}× {err}" for err, cnt in errors.items()]
-        error_str = "\n—————\n".join(error_list)  # Use separator line between errors
-        
-        # Format test name with line breaks at :: for better readability
-        formatted_test_name = test_name.replace("::", "\n::\n")
-        
-        by_test_rows.append([formatted_test_name, str(count), error_str])
+        error_str = "; ".join(error_list)
+        by_test_rows.append([test_name, str(count), error_str])
     
     # By model table
     by_model_rows = []
@@ -255,14 +251,53 @@ def _generate_summary_tables(record: dict) -> Tuple[List[List[str]], List[List[s
     for model_name, model_info in by_model_data.items():
         count = model_info.get("count", 0)
         errors = model_info.get("errors", {})
-        # Format errors with line breaks between each error type
+        # Format errors as "Nx error_message" for each unique error
         error_list = [f"{cnt}× {err}" for err, cnt in errors.items()]
-        error_str = "\n—————\n".join(error_list)  # Use separator line between errors
+        error_str = "; ".join(error_list)
         by_model_rows.append([model_name, str(count), error_str])
     
     return by_test_rows, by_model_rows
 
 
+def _generate_readable_summary(record: dict) -> str:
+    """Generate a readable text summary instead of cramped tables."""
+    
+    by_test_data = record.get("by_test", {})
+    by_model_data = record.get("by_model", {})
+    
+    summary = []
+    
+    # By Test Summary
+    if by_test_data:
+        summary.append("## 📝 Failed Tests\n")
+        for test_name, test_info in by_test_data.items():
+            count = test_info.get("count", 0)
+            errors = test_info.get("errors", {})
+            
+            summary.append(f"### {test_name}")
+            summary.append(f"**Failures:** {count}\n")
+            summary.append("**Errors:**")
+            for err, cnt in errors.items():
+                summary.append(f"- `{cnt}×` {err}")
+            summary.append("\n---\n")
+    
+    # By Model Summary
+    if by_model_data:
+        summary.append("\n## 🏷️ Failed Models\n")
+        for model_name, model_info in by_model_data.items():
+            count = model_info.get("count", 0)
+            errors = model_info.get("errors", {})
+            
+            summary.append(f"### {model_name}")
+            summary.append(f"**Failures:** {count}\n")
+            summary.append("**Errors:**")
+            for err, cnt in errors.items():
+                summary.append(f"- `{cnt}×` {err}")
+            summary.append("\n---\n")
+    
+    return "\n".join(summary)
+
+
 def _generate_markdown_summary(record: dict) -> str:
     """Generate markdown summary for copy-paste to GitHub."""
     
@@ -350,6 +385,7 @@ def _generate_pytest_commands(record: dict) -> str:
 
 def query(repo: str, pr: str, sha: str) -> Tuple[
     str,  # metadata_info
+    str,  # readable_summary
     List[List[str]],  # by_test_table
     List[List[str]],  # by_model_table
     str,  # markdown_summary
@@ -366,6 +402,7 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     if not pr:
         return (
             "**Error:** PR number is required.",
+            "",
             [],
             [],
             "",
@@ -380,6 +417,7 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     if not records:
         return (
             f"**No records found** for PR {pr}.",
+            "",
             [],
             [],
             "",
@@ -404,6 +442,9 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     ]
     metadata_info = "\n\n".join(metadata_lines)
     
+    # Generate readable summary
+    readable_summary = _generate_readable_summary(latest_record)
+    
     # Generate tables
     by_test_rows, by_model_rows = _generate_summary_tables(latest_record)
     
@@ -420,6 +461,7 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     
     return (
         metadata_info,
+        readable_summary,
         by_test_rows,
         by_model_rows,
         markdown_summary,
@@ -492,6 +534,15 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
         with gr.Tab("📊 Summary"):
             metadata_box = gr.Markdown(label="Metadata")
             
+            gr.Markdown("---")
+            
+            # Use a scrollable Markdown display instead of cramped tables
+            readable_summary = gr.Markdown(
+                label="Test Failures",
+                value="",
+            )
+        
+        with gr.Tab("📋 Tables (Compact View)"):
             gr.Markdown("### 📝 By Test")
             by_test_table = gr.Dataframe(
                 headers=["Test", "Failures", "Full error(s)"],
@@ -559,6 +610,7 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
         else:
             return (
                 "Enter a PR number and click Search",
+                "",
                 [],
                 [],
                 "",
@@ -573,6 +625,7 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
         inputs=[repo_box, pr_box, sha_box], 
         outputs=[
             metadata_box,
+            readable_summary,
             by_test_table,
             by_model_table,
             markdown_output,
@@ -594,6 +647,7 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
         inputs=[repo_box, pr_box, sha_box], 
         outputs=[
             metadata_box,
+            readable_summary,
             by_test_table,
             by_model_table,
             markdown_output,