Update app.py

app.py

@@ -240,9 +240,15 @@ 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 as "Nx error_message" for each unique error
-        error_list = [f"{cnt}× {err}" for err, cnt in errors.items()]
-        error_str = "; ".join(error_list)
+        
+        # Format errors with each on a NEW line, like GitHub table format
+        error_lines = []
+        for err, cnt in errors.items():
+            error_lines.append(f"{cnt}× {err}")
+        
+        # Join with actual newlines - Gradio will render these in the table
+        error_str = "\n".join(error_lines)
+        
         by_test_rows.append([test_name, str(count), error_str])
     
     # By model table
@@ -251,52 +257,20 @@ 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 as "Nx error_message" for each unique error
-        error_list = [f"{cnt}× {err}" for err, cnt in errors.items()]
-        error_str = "; ".join(error_list)
+        
+        # Format errors with each on a NEW line
+        error_lines = []
+        for err, cnt in errors.items():
+            error_lines.append(f"{cnt}× {err}")
+        
+        # Join with actual newlines
+        error_str = "\n".join(error_lines)
+        
         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."""
@@ -385,7 +359,6 @@ 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
@@ -402,7 +375,6 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     if not pr:
         return (
             "**Error:** PR number is required.",
-            "",
             [],
             [],
             "",
@@ -417,7 +389,6 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     if not records:
         return (
             f"**No records found** for PR {pr}.",
-            "",
             [],
             [],
             "",
@@ -442,9 +413,6 @@ 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)
     
@@ -461,7 +429,6 @@ def query(repo: str, pr: str, sha: str) -> Tuple[
     
     return (
         metadata_info,
-        readable_summary,
         by_test_rows,
         by_model_rows,
         markdown_summary,
@@ -535,26 +502,23 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
             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)"],
                 wrap=True,
                 interactive=False,
+                row_count=(5, "dynamic"),  # Start with 5 rows, expand as needed
             )
             
+            gr.Markdown("---")
             gr.Markdown("### 🏷️ By Model")
+            
             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
             )
         
         with gr.Tab("📋 Copy for GitHub"):
@@ -610,7 +574,6 @@ with gr.Blocks(title="CircleCI Test Results Viewer") as demo:
         else:
             return (
                 "Enter a PR number and click Search",
-                "",
                 [],
                 [],
                 "",
@@ -625,7 +588,6 @@ 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,
@@ -647,7 +609,6 @@ 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,