From 2326892bd8b3138737642015bfa4fa29c2b316e9 Mon Sep 17 00:00:00 2001
From: Yu-Ting Hsiung <bear890707@gmail.com>
Date: Sun, 1 Feb 2026 23:42:46 +0800
Subject: [PATCH] refactor(tags): extract version resolution method

---
 commitizen/tags.py | 47 ++++++++++++++++++++++------------------------
 1 file changed, 22 insertions(+), 25 deletions(-)

diff --git a/commitizen/tags.py b/commitizen/tags.py
index dcddbd335f..a2276526b2 100644
--- a/commitizen/tags.py
+++ b/commitizen/tags.py
@@ -154,24 +154,13 @@ def extract_version(self, tag: GitTag) -> Version:
         candidates = (
             m for regex in self.version_regexes if (m := regex.fullmatch(tag.name))
         )
-        if not (m := next(candidates, None)):
+        if not (match := next(candidates, None)):
             raise InvalidVersion(self._version_tag_error(tag.name))
-        if "version" in m.groupdict():
-            return self.scheme(m.group("version"))
 
-        parts = m.groupdict()
-        version = parts["major"]
+        if version := match.groupdict().get("version"):
+            return self.scheme(version)
 
-        if minor := parts.get("minor"):
-            version = f"{version}.{minor}"
-        if patch := parts.get("patch"):
-            version = f"{version}.{patch}"
-
-        if parts.get("prerelease"):
-            version = f"{version}-{parts['prerelease']}"
-        if parts.get("devrelease"):
-            version = f"{version}{parts['devrelease']}"
-        return self.scheme(version)
+        return self.scheme(self._extract_version_parts(match))
 
     def include_in_changelog(self, tag: GitTag) -> bool:
         """Check if a tag should be included in the changelog"""
@@ -195,19 +184,14 @@ def search_version(self, text: str, last: bool = False) -> VersionTag | None:
 
         match = matches[-1 if last else 0]
 
-        if "version" in match.groupdict():
-            return VersionTag(match.group("version"), match.group(0))
+        groups = match.groupdict()
+        if version := groups.get("version"):
+            return VersionTag(version, match.group(0))
 
-        parts = match.groupdict()
-        try:
-            version = f"{parts['major']}.{parts['minor']}.{parts['patch']}"
-        except KeyError:
+        if not all(value in groups for value in ["major", "minor", "patch"]):
             return None
 
-        if parts.get("prerelease"):
-            version = f"{version}-{parts['prerelease']}"
-        if parts.get("devrelease"):
-            version = f"{version}{parts['devrelease']}"
+        version = self._extract_version_parts(match)
         return VersionTag(version, match.group(0))
 
     def normalize_tag(
@@ -284,3 +268,16 @@ def from_settings(cls, settings: Settings) -> Self:
             ignored_tag_formats=settings["ignored_tag_formats"],
             merge_prereleases=settings["changelog_merge_prerelease"],
         )
+
+    def _extract_version_parts(self, match: re.Match[str]) -> str:
+        groups = match.groupdict()
+        parts: list[str] = [groups["major"]]
+        if minor := groups.get("minor"):
+            parts.append(f".{minor}")
+        if patch := groups.get("patch"):
+            parts.append(f".{patch}")
+        if prerelease := groups.get("prerelease"):
+            parts.append(f"-{prerelease}")
+        if devrelease := groups.get("devrelease"):
+            parts.append(devrelease)
+        return "".join(parts)