style: replace dict with Mapping

bearomorphism · Lee-W · commit c1eea63240c1 · 2025-06-09T23:18:04.000+09:00
diff --git a/commitizen/cz/base.py b/commitizen/cz/base.py
@@ -1,7 +1,7 @@
 from __future__ import annotations
 
 from abc import ABCMeta, abstractmethod
-from collections.abc import Iterable
+from collections.abc import Iterable, Mapping
 from typing import Any, Callable, Protocol
 
 from jinja2 import BaseLoader, PackageLoader
@@ -72,7 +72,7 @@ def questions(self) -> Iterable[CzQuestion]:
         """Questions regarding the commit message."""
 
     @abstractmethod
-    def message(self, answers: dict) -> str:
+    def message(self, answers: Mapping[str, Any]) -> str:
         """Format your git message."""
 
     @property
diff --git a/commitizen/cz/customize/customize.py b/commitizen/cz/customize/customize.py
@@ -1,6 +1,7 @@
 from __future__ import annotations
 
-from typing import TYPE_CHECKING
+from collections.abc import Mapping
+from typing import TYPE_CHECKING, Any
 
 from commitizen.question import CzQuestion
 
@@ -49,7 +50,7 @@ def __init__(self, config: BaseConfig) -> None:
     def questions(self) -> list[CzQuestion]:
         return self.custom_settings.get("questions", [{}])  # type: ignore
 
-    def message(self, answers: dict) -> str:
+    def message(self, answers: Mapping[str, Any]) -> str:
         message_template = Template(self.custom_settings.get("message_template", ""))
         if getattr(Template, "substitute", None):
             return message_template.substitute(**answers)  # type: ignore
diff --git a/commitizen/cz/jira/jira.py b/commitizen/cz/jira/jira.py
@@ -1,4 +1,5 @@
 import os
+from collections.abc import Mapping
 
 from commitizen.cz.base import BaseCommitizen
 from commitizen.question import CzQuestion
@@ -43,7 +44,7 @@ def questions(self) -> list[CzQuestion]:
             },
         ]
 
-    def message(self, answers: dict[str, str]) -> str:
+    def message(self, answers: Mapping[str, str]) -> str:
         return " ".join(
             x
             for k in ("message", "issues", "workflow", "time", "comment")
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -3,7 +3,7 @@
 import os
 import re
 import tempfile
-from collections.abc import Iterator
+from collections.abc import Iterator, Mapping
 from pathlib import Path
 
 import pytest
@@ -210,7 +210,7 @@ def questions(self) -> list:
             },
         ]
 
-    def message(self, answers: dict) -> str:
+    def message(self, answers: Mapping) -> str:
         prefix = answers["prefix"]
         subject = answers.get("subject", "default message").trim()
         return f"{prefix}: {subject}"
@@ -226,7 +226,7 @@ class MockPlugin(BaseCommitizen):
     def questions(self) -> list[CzQuestion]:
         return []
 
-    def message(self, answers: dict) -> str:
+    def message(self, answers: Mapping) -> str:
         return ""
 
 
diff --git a/tests/test_cz_base.py b/tests/test_cz_base.py
@@ -1,3 +1,5 @@
+from collections.abc import Mapping
+
 import pytest
 
 from commitizen.cz.base import BaseCommitizen
@@ -7,7 +9,7 @@ class DummyCz(BaseCommitizen):
     def questions(self):
         return [{"type": "input", "name": "commit", "message": "Initial commit:\n"}]
 
-    def message(self, answers: dict):
+    def message(self, answers: Mapping):
         return answers["commit"]
 
 

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`import os`
	`2`	`+from collections.abc import Mapping`
`2`	`3`
`3`	`4`	`from commitizen.cz.base import BaseCommitizen`
`4`	`5`	`from commitizen.question import CzQuestion`
`@@ -43,7 +44,7 @@ def questions(self) -> list[CzQuestion]:`
`43`	`44`	`},`
`44`	`45`	`]`
`45`	`46`
`46`		`- def message(self, answers: dict[str, str]) -> str:`
	`47`	`+ def message(self, answers: Mapping[str, str]) -> str:`
`47`	`48`	`return " ".join(`
`48`	`49`	`x`
`49`	`50`	`for k in ("message", "issues", "workflow", "time", "comment")`