feat: reduce the report table to failed scopes

Passing scopes no longer pad the failure table with dash rows; the
table lists only the failed scopes with their checked value and rule
links, while the collapsible details block keeps the full pass/fail
picture in step-log style.

Author: shenxianpeng

main.py

@@ -382,26 +382,22 @@ def _failure_count(results: list[ScopeResult]) -> int:
 
 
 def _markdown_table(results: list[ScopeResult]) -> str:
-    """Render the scope/result table shared by summary and PR comment.
+    """Render the failure table shared by summary and PR comment.
 
-    The checked-value column only fills failed scopes: pass scopes keep a
-    dash so the failure stands out, and the full per-scope values live in
-    the collapsible details block.
+    Only failed scopes appear in the table so the failure stands out; the
+    full pass/fail picture lives in the collapsible details block.
     """
     rows = [
         "| Scope | Checked value | Failed checks | Result |",
         "|---|---|---|---|",
     ]
     for scope in results:
         if scope.status == "pass":
-            rows.append(f"| {scope.label} | \u2014 | \u2014 | \u2705 |")
-        else:
-            value = _scope_value(scope)
-            value_display = f"`{value}`" if value else "\u2014"
-            links = " \u00b7 ".join(
-                _rule_markdown_link(check) for check in scope.failures
-            )
-            rows.append(f"| {scope.label} | {value_display} | {links} | \u274c |")
+            continue
+        value = _scope_value(scope)
+        value_display = f"`{value}`" if value else "\u2014"
+        links = " \u00b7 ".join(_rule_markdown_link(check) for check in scope.failures)
+        rows.append(f"| {scope.label} | {value_display} | {links} | \u274c |")
     return "\n".join(rows)
 
 

main_test.py

@@ -695,7 +695,11 @@ def test_failure_details_show_all_scopes_and_values(self):
 
     def test_pass_scope_renders_checkmark_without_value(self):
         body = main.render_job_summary([pass_scope("Branch"), fail_scope("Commit 1/1")])
-        self.assertIn("| Branch | — | — | ✅ |", body)
+        # Pass scopes stay out of the table; the details block carries them.
+        table = body.split("<details>")[0]
+        self.assertNotIn("| Branch |", table)
+        self.assertIn("| Commit 1/1 | `bad message` |", table)
+        self.assertIn("✔ Branch", body)
 
 
 class TestRenderPrComment(unittest.TestCase):