refactor: split report title and status into separate lines

The report now opens with a plain '# Commit Check' title line followed
by the status line ('✅ All checks passed (N scopes)' or the failure
count), keeping the PR comment and job summary identical.

Author: shenxianpeng

main.py

@@ -19,9 +19,8 @@
 from dataclasses import dataclass, field
 from typing import Any
 
-# Constants for message titles
-SUCCESS_TITLE = "# Commit Check ✔️"
-FAILURE_TITLE = "# Commit Check ❌"
+# Constant for the report title
+REPORT_TITLE = "# Commit Check"
 COMMIT_MESSAGE_DELIMITER = "\x00"
 RULES_URL = "https://commit-check.com/rules/"
 
@@ -419,20 +418,22 @@ def _markdown_details(results: list[ScopeResult]) -> str:
 def render_report(results: list[ScopeResult], include_footer: bool = True) -> str:
     """Render the Markdown report shared by the job summary and PR comment.
 
-    All checks passing collapses to a single success line; failures render
-    a scope table with rule links plus collapsible failure details.
+    The report opens with the plain title line followed by the status line:
+    ``✅ All checks passed (N scopes)`` on success, or the failure count on
+    failure, followed by a scope table with rule links and collapsible
+    failure details.
     """
     if all(scope.status == "pass" for scope in results):
         scopes = "scope" if len(results) == 1 else "scopes"
-        return f"{SUCCESS_TITLE} All checks passed ({len(results)} {scopes})"
+        return f"{REPORT_TITLE}\n\n✅ All checks passed ({len(results)} {scopes})"
 
     failures = _failure_count(results)
     unit = "failure" if failures == 1 else "failures"
     scopes = "scope" if len(results) == 1 else "scopes"
     lines = [
-        FAILURE_TITLE,
+        REPORT_TITLE,
         "",
-        f"**{failures} {unit}** across {len(results)} {scopes}",
+        f"❌ **{failures} {unit}** across {len(results)} {scopes}",
         "",
         _markdown_table(results),
         "",
@@ -456,8 +457,8 @@ def render_pr_comment(results: list[ScopeResult]) -> str:
 def build_result_body(result_text: str | None) -> str:
     """Legacy helper kept for backward compatibility with existing callers."""
     if result_text is None:
-        return SUCCESS_TITLE
-    return f"{FAILURE_TITLE}\n```\n{result_text}\n```"
+        return REPORT_TITLE
+    return f"{REPORT_TITLE}\n```\n{result_text}\n```"
 
 
 # ---------------------------------------------------------------------------
@@ -605,7 +606,7 @@ def add_pr_comments(results: list[ScopeResult]) -> int:
         matching_comments = [
             c
             for c in comments
-            if c.body.startswith(SUCCESS_TITLE) or c.body.startswith(FAILURE_TITLE)
+            if c.body.startswith(REPORT_TITLE)
             # Match comments from older versions that used a hyphenated title.
             or c.body.startswith("# Commit-Check")
         ]

main_test.py

@@ -631,12 +631,12 @@ def test_raw_text_fallback_is_printed(self):
 class TestRenderJobSummary(unittest.TestCase):
     def test_all_pass(self):
         body = main.render_job_summary([pass_scope("Branch")])
-        self.assertTrue(body.startswith(main.SUCCESS_TITLE))
+        self.assertTrue(body.startswith(main.REPORT_TITLE))
         self.assertIn("All checks passed (1 scope)", body)
 
     def test_failure_renders_table_with_rule_links(self):
         body = main.render_job_summary([fail_scope("Commit 1/1")])
-        self.assertTrue(body.startswith(main.FAILURE_TITLE))
+        self.assertTrue(body.startswith(main.REPORT_TITLE))
         self.assertIn("**1 failure** across 1 scope", body)
         self.assertIn("| Scope | Failed checks | Result |", body)
         self.assertIn(
@@ -658,25 +658,25 @@ def test_all_pass_matches_job_summary(self):
         comment = main.render_pr_comment([pass_scope("Branch")])
         summary = main.render_job_summary([pass_scope("Branch")])
         self.assertEqual(comment, summary)
-        self.assertTrue(comment.startswith(main.SUCCESS_TITLE))
+        self.assertTrue(comment.startswith(main.REPORT_TITLE))
         self.assertIn("All checks passed (1 scope)", comment)
 
     def test_failure_matches_job_summary(self):
         comment = main.render_pr_comment([fail_scope("Commit 1/1")])
         summary = main.render_job_summary([fail_scope("Commit 1/1")])
         self.assertEqual(comment, summary)
-        self.assertTrue(comment.startswith(main.FAILURE_TITLE))
+        self.assertTrue(comment.startswith(main.REPORT_TITLE))
         self.assertIn("**1 failure** across 1 scope", comment)
         self.assertIn("| Scope | Failed checks | Result |", comment)
 
 
 class TestBuildResultBody(unittest.TestCase):
     def test_success_body(self):
-        self.assertEqual(main.build_result_body(None), main.SUCCESS_TITLE)
+        self.assertEqual(main.build_result_body(None), main.REPORT_TITLE)
 
     def test_failure_body(self):
         result = main.build_result_body("bad commit")
-        self.assertIn(main.FAILURE_TITLE, result)
+        self.assertIn(main.REPORT_TITLE, result)
         self.assertIn("bad commit", result)
 
 
@@ -800,7 +800,7 @@ def test_creates_comment_with_rendered_body(self):
         self.assertEqual(rc, 1)
         self.assertEqual(mock_pull_request.create_comment.call_count, 1)
         body = mock_pull_request.create_comment.call_args[1]["body"]
-        self.assertTrue(body.startswith(main.FAILURE_TITLE))
+        self.assertTrue(body.startswith(main.REPORT_TITLE))
         self.assertIn("| Scope | Failed checks | Result |", body)
 
     def test_updates_existing_comment_when_changed(self):