ci: allow nested tests prefixes in commit validator

Signed-off-by: Eduardo Silva <eduardo@calyptia.com>

Author: edsiper

.github/scripts/commit_prefix_check.py

@@ -21,7 +21,10 @@
 repo = Repo(".")
 
 # Regex patterns
-PREFIX_RE = re.compile(r"^([a-z0-9_]+:)\s+\S", re.IGNORECASE)
+PREFIX_RE = re.compile(
+    r"^([a-z0-9_]+:(?:\s+[a-z0-9_]+:)*)\s+\S",
+    re.IGNORECASE,
+)
 SIGNED_OFF_RE = re.compile(r"Signed-off-by:", re.IGNORECASE)
 FENCED_BLOCK_RE = re.compile(
     r"""
@@ -231,6 +234,7 @@ def is_version_bump(commit):
         return False, f"Missing prefix in commit subject: '{first_line}'"
 
     subject_prefix = subject_prefix_match.group(1)
+    subject_root_prefix = subject_prefix.split()[0]
 
     # Run squash detection (but ignore multi-signoff errors)
     bad_squash, reason = detect_bad_squash(body)
@@ -282,6 +286,7 @@ def is_version_bump(commit):
 
     expected_lower = {p.lower() for p in expected}
     subj_lower = subject_prefix.lower()
+    subj_root_lower = subject_root_prefix.lower()
 
 
     # ------------------------------------------------
@@ -320,10 +325,10 @@ def is_version_bump(commit):
     # (because the corresponding file exists). Only reject if it's not in the expected list
     # or if it's an umbrella prefix that doesn't match.
     if len(non_build_prefixes) > 1:
-        if subj_lower in umbrella_prefixes:
+        if subj_root_lower in umbrella_prefixes:
             norm_paths = [p.replace(os.sep, "/") for p in files]
 
-            if subj_lower == "lib:":
+            if subj_root_lower == "lib:":
                 if not all(p.startswith("lib/") for p in norm_paths):
                     expected_list = sorted(expected)
                     expected_str = ", ".join(expected_list)
@@ -332,7 +337,7 @@ def is_version_bump(commit):
                         f"Expected one of: {expected_str}"
                     )
 
-            elif subj_lower == "tests:":
+            elif subj_root_lower == "tests:":
                 if not all(p.startswith("tests/") for p in norm_paths):
                     expected_list = sorted(expected)
                     expected_str = ", ".join(expected_list)
@@ -341,7 +346,7 @@ def is_version_bump(commit):
                         f"Expected one of: {expected_str}"
                     )
 
-            elif subj_lower == "http_server:":
+            elif subj_root_lower == "http_server:":
                 if not all(is_http_server_interface_path(p) for p in norm_paths):
                     expected_list = sorted(expected)
                     expected_str = ", ".join(expected_list)
@@ -359,7 +364,7 @@ def is_version_bump(commit):
             )
 
     # Subject prefix must be one of the expected ones
-    if subj_lower not in expected_lower:
+    if subj_lower not in expected_lower and subj_root_lower not in expected_lower:
         expected_list = sorted(expected)
         expected_str = ", ".join(expected_list)
         return False, (
@@ -370,7 +375,7 @@ def is_version_bump(commit):
 
     # If build is NOT optional and build: exists among expected,
     # then subject MUST be build:
-    if not build_optional and "build:" in expected_lower and subj_lower != "build:":
+    if not build_optional and "build:" in expected_lower and subj_root_lower != "build:":
         return False, (
             f"Subject prefix '{subject_prefix}' does not match files changed.\n"
             f"Expected one of: build:"