feat: enhance commit message checks with section separators and no-banner option

Author: shenxianpeng

main.py

@@ -10,6 +10,7 @@
 SUCCESS_TITLE = "# Commit-Check ✔️"
 FAILURE_TITLE = "# Commit-Check ❌"
 COMMIT_MESSAGE_DELIMITER = "\x00"
+COMMIT_SECTION_SEPARATOR = "\n" + ("-" * 72) + "\n"
 
 # Environment variables
 MESSAGE = os.getenv("MESSAGE", "false")
@@ -156,12 +157,23 @@ def run_pr_message_checks(pr_messages: list[str], result_file: TextIO) -> int:
     total_messages = len(pr_messages)
     for index, msg in enumerate(pr_messages, start=1):
         subject = msg.splitlines()[0] if msg else "<empty commit message>"
+        command_args = ["--message"]
+        if index > 1:
+            command_args.append("--no-banner")
+
+        output_prefix = f"\n--- Commit {index}/{total_messages}: {subject}\n"
+        if index > 1:
+            output_prefix = (
+                f"{COMMIT_SECTION_SEPARATOR}"
+                f"--- Commit {index}/{total_messages}: {subject}\n"
+            )
+
         has_failure = (
             run_check_command(
-                ["--message"],
+                command_args,
                 result_file,
                 input_text=msg,
-                output_prefix=f"\n--- Commit {index}/{total_messages}: {subject}\n",
+                output_prefix=output_prefix,
             )
             != 0
             or has_failure

main_test.py

@@ -126,6 +126,39 @@ def test_empty_list(self):
         self.assertEqual(rc, 0)
         mock_run.assert_not_called()
 
+    def test_second_and_later_messages_use_no_banner(self):
+        results = [
+            MagicMock(returncode=1, stdout="Commit rejected.\n"),
+            MagicMock(returncode=1, stdout="Type subject_imperative check failed\n"),
+        ]
+        with patch("main.subprocess.run", side_effect=results) as mock_run:
+            main.run_pr_message_checks(["bad first", "bad second"], io.StringIO())
+
+        self.assertEqual(
+            mock_run.call_args_list[0][0][0], ["commit-check", "--message"]
+        )
+        self.assertEqual(
+            mock_run.call_args_list[1][0][0],
+            ["commit-check", "--message", "--no-banner"],
+        )
+
+    def test_second_message_prefix_uses_separator(self):
+        results = [
+            MagicMock(returncode=1, stdout="Commit rejected.\n"),
+            MagicMock(returncode=1, stdout="Type subject_imperative check failed\n"),
+        ]
+        result_file = io.StringIO()
+        with patch("main.subprocess.run", side_effect=results):
+            main.run_pr_message_checks(["bad first", "bad second"], result_file)
+
+        output = result_file.getvalue()
+        self.assertIn("\n--- Commit 1/2: bad first\nCommit rejected.\n", output)
+        self.assertIn(
+            f"{main.COMMIT_SECTION_SEPARATOR}--- Commit 2/2: bad second\n",
+            output,
+        )
+        self.assertIn("Type subject_imperative check failed\n", output)
+
 
 class TestRunOtherChecks(unittest.TestCase):
     def test_empty_args_returns_zero(self):