From 9285a5f97ec166688e65451dd941c17c80af4fab Mon Sep 17 00:00:00 2001
From: Rein Lemmens <rein@kapernikov.com>
Date: Thu, 12 Mar 2026 21:15:57 +0100
Subject: [PATCH] Revert "Extract shared named style constant and use keyword
 args"

This reverts commit 00796f39c6a08e49b78cb6d9a9c1b1fb91aeeeac.
---
 gdocs/docs_helpers.py                | 39 ++++++++++++----------------
 gdocs/managers/validation_manager.py | 11 +++++---
 2 files changed, 24 insertions(+), 26 deletions(-)

diff --git a/gdocs/docs_helpers.py b/gdocs/docs_helpers.py
index 05d7705..85354f7 100644
--- a/gdocs/docs_helpers.py
+++ b/gdocs/docs_helpers.py
@@ -10,18 +10,6 @@ from typing import Dict, Any, Optional
 
 logger = logging.getLogger(__name__)
 
-VALID_NAMED_STYLE_TYPES = (
-    "NORMAL_TEXT",
-    "TITLE",
-    "SUBTITLE",
-    "HEADING_1",
-    "HEADING_2",
-    "HEADING_3",
-    "HEADING_4",
-    "HEADING_5",
-    "HEADING_6",
-)
-
 
 def _normalize_color(
     color: Optional[str], param_name: str
@@ -149,10 +137,15 @@ def build_paragraph_style(
     fields = []
 
     if named_style_type is not None:
-        if named_style_type not in VALID_NAMED_STYLE_TYPES:
+        valid_styles = [
+            "NORMAL_TEXT", "TITLE", "SUBTITLE",
+            "HEADING_1", "HEADING_2", "HEADING_3",
+            "HEADING_4", "HEADING_5", "HEADING_6",
+        ]
+        if named_style_type not in valid_styles:
             raise ValueError(
                 f"Invalid named_style_type '{named_style_type}'. "
-                f"Must be one of: {', '.join(VALID_NAMED_STYLE_TYPES)}"
+                f"Must be one of: {', '.join(valid_styles)}"
             )
         paragraph_style["namedStyleType"] = named_style_type
         fields.append("namedStyleType")
@@ -367,15 +360,15 @@ def create_update_paragraph_style_request(
         Dictionary representing the updateParagraphStyle request, or None if no styles provided
     """
     paragraph_style, fields = build_paragraph_style(
-        heading_level=heading_level,
-        alignment=alignment,
-        line_spacing=line_spacing,
-        indent_first_line=indent_first_line,
-        indent_start=indent_start,
-        indent_end=indent_end,
-        space_above=space_above,
-        space_below=space_below,
-        named_style_type=named_style_type,
+        heading_level,
+        alignment,
+        line_spacing,
+        indent_first_line,
+        indent_start,
+        indent_end,
+        space_above,
+        space_below,
+        named_style_type,
     )
 
     if not paragraph_style:
diff --git a/gdocs/managers/validation_manager.py b/gdocs/managers/validation_manager.py
index 7b13c21..3235c1a 100644
--- a/gdocs/managers/validation_manager.py
+++ b/gdocs/managers/validation_manager.py
@@ -9,7 +9,7 @@ import logging
 from typing import Dict, Any, List, Tuple, Optional
 from urllib.parse import urlparse
 
-from gdocs.docs_helpers import validate_operation, VALID_NAMED_STYLE_TYPES
+from gdocs.docs_helpers import validate_operation
 
 logger = logging.getLogger(__name__)
 
@@ -317,10 +317,15 @@ class ValidationManager:
             )
 
         if named_style_type is not None:
-            if named_style_type not in VALID_NAMED_STYLE_TYPES:
+            valid_styles = [
+                "NORMAL_TEXT", "TITLE", "SUBTITLE",
+                "HEADING_1", "HEADING_2", "HEADING_3",
+                "HEADING_4", "HEADING_5", "HEADING_6",
+            ]
+            if named_style_type not in valid_styles:
                 return (
                     False,
-                    f"Invalid named_style_type '{named_style_type}'. Must be one of: {', '.join(VALID_NAMED_STYLE_TYPES)}",
+                    f"Invalid named_style_type '{named_style_type}'. Must be one of: {', '.join(valid_styles)}",
                 )
 
         if heading_level is not None: