Fix test summary extracting full tracebacks

Extract only the "short test summary info" section
instead of the full FAILURES section with tracebacks.
Reduces real-world output from 45K to 343 lines.

Author: stephen-tatari

pr-status/scripts/pr_status.py

@@ -261,30 +261,32 @@ def extract_error_annotations(lines: list[str], *, context: int = 2) -> list[str
 def extract_test_summary(lines: list[str]) -> list[str]:
     """Extract test failure summaries from log lines.
 
-    Looks for pytest FAILURES sections, FAILED lines, ERROR lines,
-    and AssertionError occurrences.
+    Captures the pytest "short test summary info" section (concise)
+    and individual FAILED/ERROR/AssertionError lines. Avoids capturing
+    full tracebacks from the FAILURES section.
     """
     if not lines:
         return []
 
     result: list[str] = []
 
-    # Extract pytest FAILURES section
-    in_failures = False
+    # Extract pytest "short test summary info" section (concise one-liners)
+    in_short_summary = False
     for line in lines:
         stripped = line.strip()
-        if "=== FAILURES ===" in stripped:
-            in_failures = True
-        if in_failures:
+        if "short test summary info" in stripped:
+            in_short_summary = True
             result.append(line)
-            if stripped.startswith("===") and "passed" in stripped:
+            continue
+        if in_short_summary:
+            if stripped.startswith("===") or stripped.startswith("---"):
+                result.append(line)
                 break
-            if "=== short test summary info ===" in stripped:
-                continue
+            result.append(line)
 
     # Extract individual FAILED / ERROR lines not already captured
     seen = set(result)
-    for i, line in enumerate(lines):
+    for line in lines:
         stripped = line.strip()
         if line in seen:
             continue

pr-status/scripts/test_pr_status.py

@@ -633,18 +633,21 @@ def test_empty_input(self):
 # ---------------------------------------------------------------------------
 
 class TestExtractTestSummary:
-    def test_extracts_pytest_failures_section(self):
+    def test_extracts_short_test_summary_section(self):
         lines = [
             "collecting ...",
             "=== FAILURES ===",
-            "FAILED test_foo.py::test_bar - AssertionError",
+            "lots of traceback here",
             "=== short test summary info ===",
             "FAILED test_foo.py::test_bar",
             "=== 1 failed, 5 passed ===",
             "more stuff",
         ]
         result = extract_test_summary(lines)
+        assert any("short test summary info" in l for l in result)
         assert "FAILED test_foo.py::test_bar" in result
+        # Should NOT include the full traceback
+        assert "lots of traceback here" not in result
 
     def test_extracts_failed_lines(self):
         lines = [