refactor(conventional_commits): use TypedDict for answers

Author: bearomorphism

commitizen/cz/conventional_commits/conventional_commits.py

@@ -1,5 +1,6 @@
 import os
 import re
+from typing import TypedDict
 
 from commitizen import defaults
 from commitizen.cz.base import BaseCommitizen
@@ -27,6 +28,15 @@ def parse_subject(text):
     return required_validator(text, msg="Subject is required.")
 
 
+class ConventionalCommitsAnswers(TypedDict):
+    prefix: str
+    scope: str
+    subject: str
+    body: str
+    footer: str
+    is_breaking_change: bool
+
+
 class ConventionalCommitsCz(BaseCommitizen):
     bump_pattern = defaults.BUMP_PATTERN
     bump_map = defaults.BUMP_MAP
@@ -147,24 +157,22 @@ def questions(self) -> Questions:
             },
         ]
 
-    def message(self, answers: dict) -> str:
-        prefix = answers["prefix"]
-        scope = answers["scope"]
-        subject = answers["subject"]
-        body = answers["body"]
-        footer = answers["footer"]
-        is_breaking_change = answers["is_breaking_change"]
+    def message(self, answers: ConventionalCommitsAnswers) -> str:  # type: ignore[override]
+        parts: list[str] = [answers["prefix"]]
 
-        if scope:
-            scope = f"({scope})"
-        if body:
-            body = f"\n\n{body}"
-        if is_breaking_change:
+        if scope := answers["scope"]:
+            parts.append(f"({scope})")
+        parts.append(f": {answers['subject']}")
+        if body := answers["body"]:
+            parts.append(f"\n\n{body}")
+
+        footer = answers["footer"]
+        if answers["is_breaking_change"]:
             footer = f"BREAKING CHANGE: {footer}"
         if footer:
-            footer = f"\n\n{footer}"
+            parts.append(f"\n\n{footer}")
 
-        return f"{prefix}{scope}: {subject}{body}{footer}"
+        return "".join(parts)
 
     def example(self) -> str:
         return (