pyopenapi-gen 0.21.0__tar.gz → 0.21.1__tar.gz

{pyopenapi_gen-0.21.0 → pyopenapi_gen-0.21.1}/CHANGELOG.md

@@ -1,6 +1,48 @@
 # CHANGELOG
 
 
+## v0.21.1 (2025-10-24)
+
+### Bug Fixes
+
+- **codegen**: Fix enum parameter serialization in query, header, and path parameters
+  ([`be5391f`](https://github.com/mindhiveoy/pyopenapi_gen/commit/be5391ff9fde14a1826aa217c077b06a0f832f8c))
+
+Implement consistent parameter serialization using DataclassSerializer.serialize() for query,
+  header, and path parameters to handle enum-to-string conversion correctly.
+
+Technical changes: - _write_query_params() now wraps all parameter values in
+  DataclassSerializer.serialize() - _write_header_params() now wraps all parameter values in
+  DataclassSerializer.serialize() - generate_url_and_args() serializes path params before f-string
+  interpolation - Added context.add_import() calls to ensure DataclassSerializer is imported in
+  generated code
+
+Implementation approach: Query/header parameters use DataclassSerializer.serialize() in dict
+  construction: Required: "{param}": DataclassSerializer.serialize(param_var)
+
+Optional: **{{"{param}": DataclassSerializer.serialize(param_var)} if param_var is not None else {}}
+
+Path parameters are serialized before URL construction to prevent enum objects in f-strings (would
+  produce "EnumType.VALUE" instead of "value"): param_var = DataclassSerializer.serialize(param_var)
+
+Breaking changes: None API changes: None (generated code behavior corrected to match intended
+  design)
+
+Test coverage: - 5 new TDD tests validate enum serialization for all parameter types - All 1360
+  tests passing with 88.43% coverage - Integration tests confirm correct mypy validation and runtime
+  behavior
+
+Fixes issue where enum parameters (e.g., DocumentStatus.INDEXED) were passed directly to httpx
+  instead of being converted to their string values ("indexed"), causing incorrect API calls.
+
+Related: business_swagger.json list_documents endpoint with status parameter
+
+### Chores
+
+- **release**: Sync __init__.py version [skip ci]
+  ([`823dc4e`](https://github.com/mindhiveoy/pyopenapi_gen/commit/823dc4e7083f17df7323aeebd9a400fd9d7aca5d))
+
+
 ## v0.21.0 (2025-10-23)
 
 ### Bug Fixes

{pyopenapi_gen-0.21.0 → pyopenapi_gen-0.21.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyopenapi-gen
-Version: 0.21.0
+Version: 0.21.1
 Summary: Modern, async-first Python client generator for OpenAPI specifications with advanced cycle detection and unified type resolution
 Project-URL: Homepage, https://github.com/your-org/pyopenapi-gen
 Project-URL: Documentation, https://github.com/your-org/pyopenapi-gen/blob/main/README.md

{pyopenapi_gen-0.21.0 → pyopenapi_gen-0.21.1}/coverage.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" ?>
-<coverage version="7.10.7" timestamp="1761233680435" lines-valid="6518" lines-covered="5763" line-rate="0.8842" branches-covered="0" branches-valid="0" branch-rate="0" complexity="0">
+<coverage version="7.10.7" timestamp="1761323025794" lines-valid="6528" lines-covered="5773" line-rate="0.8843" branches-covered="0" branches-valid="0" branch-rate="0" complexity="0">
 	<!-- Generated by coverage.py: https://coverage.readthedocs.io/en/7.10.7 -->
 	<!-- Based on https://raw.githubusercontent.com/cobertura/web/master/htdocs/xml/coverage-04.dtd -->
 	<sources>
@@ -6019,7 +6019,7 @@
 				</class>
 			</classes>
 		</package>
-		<package name="pyopenapi_gen.visit.endpoint.generators" line-rate="0.8884" branch-rate="0" complexity="0">
+		<package name="pyopenapi_gen.visit.endpoint.generators" line-rate="0.89" branch-rate="0" complexity="0">
 			<classes>
 				<class name="__init__.py" filename="pyopenapi_gen/visit/endpoint/generators/__init__.py" complexity="0" line-rate="1" branch-rate="0">
 					<methods/>
@@ -6669,7 +6669,7 @@
 						<line number="83" hits="1"/>
 					</lines>
 				</class>
-				<class name="url_args_generator.py" filename="pyopenapi_gen/visit/endpoint/generators/url_args_generator.py" complexity="0" line-rate="0.9882" branch-rate="0">
+				<class name="url_args_generator.py" filename="pyopenapi_gen/visit/endpoint/generators/url_args_generator.py" complexity="0" line-rate="0.9895" branch-rate="0">
 					<methods/>
 					<lines>
 						<line number="5" hits="1"/>
@@ -6689,74 +6689,84 @@
 						<line number="41" hits="1"/>
 						<line number="42" hits="1"/>
 						<line number="44" hits="0"/>
-						<line number="46" hits="1"/>
 						<line number="47" hits="1"/>
-						<line number="48" hits="1"/>
 						<line number="49" hits="1"/>
+						<line number="50" hits="1"/>
 						<line number="51" hits="1"/>
 						<line number="52" hits="1"/>
+						<line number="54" hits="1"/>
 						<line number="55" hits="1"/>
 						<line number="60" hits="1"/>
-						<line number="67" hits="1"/>
-						<line number="69" hits="1"/>
-						<line number="70" hits="1"/>
-						<line number="73" hits="1"/>
-						<line number="74" hits="1"/>
+						<line number="65" hits="1"/>
+						<line number="72" hits="1"/>
+						<line number="75" hits="1"/>
 						<line number="76" hits="1"/>
-						<line number="77" hits="1"/>
+						<line number="78" hits="1"/>
+						<line number="79" hits="1"/>
 						<line number="82" hits="1"/>
-						<line number="87" hits="1"/>
+						<line number="83" hits="1"/>
+						<line number="85" hits="1"/>
+						<line number="86" hits="1"/>
+						<line number="93" hits="1"/>
 						<line number="98" hits="1"/>
-						<line number="99" hits="1"/>
-						<line number="100" hits="1"/>
-						<line number="104" hits="1"/>
-						<line number="105" hits="1"/>
-						<line number="106" hits="1"/>
-						<line number="107" hits="1"/>
-						<line number="108" hits="1"/>
-						<line number="110" hits="1"/>
-						<line number="112" hits="1"/>
 						<line number="113" hits="1"/>
+						<line number="114" hits="1"/>
 						<line number="116" hits="1"/>
 						<line number="117" hits="1"/>
 						<line number="118" hits="1"/>
 						<line number="119" hits="1"/>
 						<line number="120" hits="1"/>
 						<line number="122" hits="1"/>
+						<line number="123" hits="1"/>
 						<line number="124" hits="1"/>
-						<line number="125" hits="1"/>
+						<line number="128" hits="1"/>
 						<line number="129" hits="1"/>
+						<line number="130" hits="1"/>
 						<line number="131" hits="1"/>
-						<line number="133" hits="1"/>
+						<line number="132" hits="1"/>
 						<line number="134" hits="1"/>
-						<line number="135" hits="1"/>
 						<line number="136" hits="1"/>
 						<line number="137" hits="1"/>
-						<line number="138" hits="1"/>
+						<line number="140" hits="1"/>
+						<line number="141" hits="1"/>
 						<line number="142" hits="1"/>
+						<line number="143" hits="1"/>
+						<line number="144" hits="1"/>
 						<line number="146" hits="1"/>
-						<line number="147" hits="1"/>
 						<line number="148" hits="1"/>
 						<line number="149" hits="1"/>
-						<line number="150" hits="1"/>
-						<line number="151" hits="1"/>
-						<line number="152" hits="1"/>
 						<line number="153" hits="1"/>
 						<line number="155" hits="1"/>
+						<line number="157" hits="1"/>
+						<line number="158" hits="1"/>
 						<line number="159" hits="1"/>
 						<line number="160" hits="1"/>
 						<line number="161" hits="1"/>
 						<line number="162" hits="1"/>
-						<line number="165" hits="1"/>
-						<line number="168" hits="1"/>
-						<line number="169" hits="1"/>
+						<line number="166" hits="1"/>
+						<line number="170" hits="1"/>
+						<line number="171" hits="1"/>
+						<line number="172" hits="1"/>
 						<line number="173" hits="1"/>
 						<line number="174" hits="1"/>
 						<line number="175" hits="1"/>
-						<line number="178" hits="1"/>
-						<line number="180" hits="1"/>
-						<line number="181" hits="1"/>
+						<line number="176" hits="1"/>
+						<line number="177" hits="1"/>
+						<line number="179" hits="1"/>
 						<line number="183" hits="1"/>
+						<line number="184" hits="1"/>
+						<line number="185" hits="1"/>
+						<line number="186" hits="1"/>
+						<line number="189" hits="1"/>
+						<line number="192" hits="1"/>
+						<line number="193" hits="1"/>
+						<line number="197" hits="1"/>
+						<line number="198" hits="1"/>
+						<line number="199" hits="1"/>
+						<line number="202" hits="1"/>
+						<line number="204" hits="1"/>
+						<line number="205" hits="1"/>
+						<line number="207" hits="1"/>
 					</lines>
 				</class>
 			</classes>

{pyopenapi_gen-0.21.0 → pyopenapi_gen-0.21.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "pyopenapi-gen"
-version = "0.21.0"
+version = "0.21.1"
 description = "Modern, async-first Python client generator for OpenAPI specifications with advanced cycle detection and unified type resolution"
 authors = [{ name = "Mindhive Oy", email = "contact@mindhive.fi" }]
 maintainers = [{ name = "Ville Venäläinen | Mindhive Oy", email = "ville@mindhive.fi" }]
@@ -140,7 +140,7 @@ pyopenapi-gen = "pyopenapi_gen.cli:app"
 
 [tool.commitizen]
 name = "cz_conventional_commits"
-version = "0.21.0"
+version = "0.21.1"
 version_files = [
     "pyproject.toml:version",
     "src/pyopenapi_gen/__init__.py:__version__"

{pyopenapi_gen-0.21.0 → pyopenapi_gen-0.21.1}/src/pyopenapi_gen/__init__.py

@@ -50,7 +50,7 @@ __all__ = [
 ]
 
 # Semantic version of the generator core – automatically managed by semantic-release.
-__version__: str = "0.21.0"
+__version__: str = "0.21.1"
 
 # ---------------------------------------------------------------------------
 # Lazy-loading and autocompletion support (This part remains)

{pyopenapi_gen-0.21.0 → pyopenapi_gen-0.21.1}/src/pyopenapi_gen/visit/endpoint/generators/url_args_generator.py

@@ -43,17 +43,22 @@ class EndpointUrlArgsGenerator:
             # writer.write_line("# No query parameters to write") # Optional: for clarity during debugging
             return
 
+        # Import DataclassSerializer since we use it for parameter serialization
+        context.add_import(f"{context.core_package_name}.utils", "DataclassSerializer")
+
         for i, p in enumerate(query_params_to_write):
             param_var_name = NameSanitizer.sanitize_method_name(p["name"])  # Ensure name is sanitized
             original_param_name = p["original_name"]
             line_end = ","  # Always add comma, let formatter handle final one if needed
 
             if p.get("required", False):
-                writer.write_line(f'    "{original_param_name}": {param_var_name}{line_end}')
+                writer.write_line(
+                    f'    "{original_param_name}": DataclassSerializer.serialize({param_var_name}){line_end}'
+                )
             else:
                 # Using dict unpacking for conditional parameters
                 writer.write_line(
-                    f'    **({{"{original_param_name}": {param_var_name}}} '
+                    f'    **({{"{original_param_name}": DataclassSerializer.serialize({param_var_name})}} '
                     f"if {param_var_name} is not None else {{}}){line_end}"
                 )
 
@@ -66,6 +71,10 @@ class EndpointUrlArgsGenerator:
         # if ordered_params is the sole source of truth for method params.
         header_params_to_write = [p for p in ordered_params if p.get("param_in") == "header"]
 
+        # Import DataclassSerializer since we use it for parameter serialization
+        if header_params_to_write:
+            context.add_import(f"{context.core_package_name}.utils", "DataclassSerializer")
+
         for p_info in header_params_to_write:
             param_var_name = NameSanitizer.sanitize_method_name(
                 p_info["name"]
@@ -74,13 +83,15 @@ class EndpointUrlArgsGenerator:
             line_end = ","
 
             if p_info.get("required", False):
-                writer.write_line(f'    "{original_header_name}": {param_var_name}{line_end}')
+                writer.write_line(
+                    f'    "{original_header_name}": DataclassSerializer.serialize({param_var_name}){line_end}'
+                )
             else:
                 # Conditional inclusion for optional headers
                 # This assumes that if an optional header parameter is None, it should not be sent.
                 # If specific behavior (e.g. empty string) is needed for None, logic would adjust.
                 writer.write_line(
-                    f'    **({{"{original_header_name}": {param_var_name}}} '
+                    f'    **({{"{original_header_name}": DataclassSerializer.serialize({param_var_name})}} '
                     f"if {param_var_name} is not None else {{}}){line_end}"
                 )
 
@@ -95,6 +106,19 @@ class EndpointUrlArgsGenerator:
     ) -> bool:
         """Writes URL, query, and header parameters. Returns True if header params were written."""
         # Main logic from EndpointMethodGenerator._write_url_and_args
+
+        # Serialize path parameters before URL construction
+        # This ensures enums, dates, and other complex types are converted to strings
+        # before f-string interpolation in the URL
+        path_params = [p for p in ordered_params if p.get("param_in") == "path"]
+        if path_params:
+            # Import DataclassSerializer since we use it for parameter serialization
+            context.add_import(f"{context.core_package_name}.utils", "DataclassSerializer")
+            for p in path_params:
+                param_var_name = NameSanitizer.sanitize_method_name(p["name"])
+                writer.write_line(f"{param_var_name} = DataclassSerializer.serialize({param_var_name})")
+            writer.write_line("")  # Blank line after path param serialization
+
         url_expr = self._build_url_with_path_vars(op.path)
         writer.write_line(f"url = {url_expr}")
         writer.write_line("")  # Add a blank line for readability

{pyopenapi_gen-0.21.0 → pyopenapi_gen-0.21.1}/tests/emitters/test_endpoints_emitter.py

@@ -804,9 +804,9 @@ def test_endpoints_emitter__query_params_included_in_params_dict(
     # Read the generated code
     with open(os.path.join(out_dir, "endpoints", "tenants.py")) as f:
         content = f.read()
-    # Assert that all query params are included in the params dict
-    assert '"start_date": start_date' in content
-    assert '"end_date": end_date' in content
+    # Assert that all query params are included in the params dict with serialization
+    assert '"start_date": DataclassSerializer.serialize(start_date)' in content
+    assert '"end_date": DataclassSerializer.serialize(end_date)' in content
     # Also check that tenant_id is not in params (it's a path param)
     assert '"tenant_id": tenant_id' not in content
 

{pyopenapi_gen-0.21.0 → pyopenapi_gen-0.21.1}/tests/visit/endpoint/generators/test_url_args_generator.py

@@ -157,7 +157,7 @@ class TestEndpointUrlArgsGenerator:
             code_writer_mock.write_line.assert_any_call(f'url = f"{{self.base_url}}/items"')
             code_writer_mock.write_line.assert_any_call("params: dict[str, Any] = {")
             code_writer_mock.write_line.assert_any_call(
-                '    **({"filterBy": filter_by} if filter_by is not None else {}),'
+                '    **({"filterBy": DataclassSerializer.serialize(filter_by)} if filter_by is not None else {}),'
             )
             mock_sanitize_query.assert_any_call("filter_by")
             render_context_mock.add_import.assert_any_call("typing", "Any")
@@ -227,9 +227,9 @@ class TestEndpointUrlArgsGenerator:
 
         code_writer_mock.write_line.assert_any_call(f'url = f"{{self.base_url}}/items_with_headers"')
         code_writer_mock.write_line.assert_any_call("headers: dict[str, Any] = {")
-        code_writer_mock.write_line.assert_any_call('    "X-Request-ID": x_request_id,')
+        code_writer_mock.write_line.assert_any_call('    "X-Request-ID": DataclassSerializer.serialize(x_request_id),')
         code_writer_mock.write_line.assert_any_call(
-            '    **({"X-Client-Version": x_client_version} if x_client_version is not None else {}),'
+            '    **({"X-Client-Version": DataclassSerializer.serialize(x_client_version)} if x_client_version is not None else {}),'
         )
         code_writer_mock.write_line.assert_any_call("}")  # Closing brace for headers dict
 
@@ -371,7 +371,7 @@ class TestEndpointUrlArgsGenerator:
             # Query params assertions
             code_writer_mock.write_line.assert_any_call("params: dict[str, Any] = {")
             code_writer_mock.write_line.assert_any_call(
-                '    **({"verboseOutput": verbose_output_sanitized} if verbose_output_sanitized is not None else {}),'
+                '    **({"verboseOutput": DataclassSerializer.serialize(verbose_output_sanitized)} if verbose_output_sanitized is not None else {}),'
             )
             # _write_query_params calls sanitize_method_name on p["name"]
             mock_sanitize_method_name.assert_any_call(
@@ -382,7 +382,9 @@ class TestEndpointUrlArgsGenerator:
             code_writer_mock.write_line.assert_any_call("headers: dict[str, Any] = {")
             # Content-Type is NOT added by this generator to the headers dict
             # self.code_writer_mock.write_line.assert_any_call(f'    "Content-Type": "{primary_content_type}",')
-            code_writer_mock.write_line.assert_any_call('    "X-Correlation-ID": x_correlation_id_sanitized,')
+            code_writer_mock.write_line.assert_any_call(
+                '    "X-Correlation-ID": DataclassSerializer.serialize(x_correlation_id_sanitized),'
+            )
 
             # _write_header_params calls sanitize_method_name on p_info["name"]
             mock_sanitize_method_name.assert_any_call(
@@ -601,3 +603,281 @@ class TestEndpointUrlArgsGenerator:
 
         code_writer_mock.write_line.assert_any_call(f"bytes_body: bytes = bytes_content")
         # No specific imports added by this path in the generator
+
+    # ========== ENUM SERIALIZATION TESTS (TDD - RED PHASE) ==========
+
+    def test_write_query_params__required_enum_parameter__serializes_value(
+        self, url_args_generator: EndpointUrlArgsGenerator, render_context_mock: MagicMock
+    ) -> None:
+        """
+        Scenario: Required query parameter is an enum type
+        Expected Outcome: Generated code calls DataclassSerializer.serialize() to convert enum to string value
+
+        This test follows TDD principles:
+        - RED: Test will fail because current code doesn't call serialize()
+        - GREEN: Will pass after implementing the fix
+        - REFACTOR: Cleanup if needed
+        """
+        # Arrange
+        writer = CodeWriter()
+        operation = IROperation(
+            operation_id="list_documents",
+            summary="List documents",
+            description="List documents with status filter",
+            method=HTTPMethod.GET,
+            path="/documents",
+            tags=["documents"],
+            parameters=[
+                IRParameter(
+                    name="status",
+                    param_in="query",
+                    required=True,
+                    schema=IRSchema(type="string", is_nullable=False),
+                    description="Document status filter",
+                )
+            ],
+            request_body=None,
+            responses=[],
+        )
+        ordered_params = [
+            {
+                "name": "status",
+                "original_name": "status",
+                "param_in": "query",
+                "required": True,
+            }
+        ]
+
+        # Act
+        url_args_generator._write_query_params(writer, operation, ordered_params, render_context_mock)
+        generated_code = writer.get_code()
+
+        # Assert
+        assert "DataclassSerializer.serialize(status)" in generated_code, (
+            "Required query parameter should be serialized with DataclassSerializer.serialize() "
+            "to handle enum→string conversion"
+        )
+        assert '"status": DataclassSerializer.serialize(status)' in generated_code
+
+    def test_write_query_params__optional_enum_parameter__serializes_value(
+        self, url_args_generator: EndpointUrlArgsGenerator, render_context_mock: MagicMock
+    ) -> None:
+        """
+        Scenario: Optional query parameter is an enum type
+        Expected Outcome: Generated code calls DataclassSerializer.serialize() within None check
+
+        This ensures enums are converted to string values before being passed to httpx,
+        preventing enum objects from appearing in query strings.
+        """
+        # Arrange
+        writer = CodeWriter()
+        operation = IROperation(
+            operation_id="list_documents",
+            summary="List documents",
+            description="List documents with optional status filter",
+            method=HTTPMethod.GET,
+            path="/documents",
+            tags=["documents"],
+            parameters=[
+                IRParameter(
+                    name="status",
+                    param_in="query",
+                    required=False,
+                    schema=IRSchema(type="string", is_nullable=True),
+                    description="Optional document status filter",
+                )
+            ],
+            request_body=None,
+            responses=[],
+        )
+        ordered_params = [
+            {
+                "name": "status",
+                "original_name": "status",
+                "param_in": "query",
+                "required": False,
+            }
+        ]
+
+        # Act
+        url_args_generator._write_query_params(writer, operation, ordered_params, render_context_mock)
+        generated_code = writer.get_code()
+
+        # Assert
+        assert "DataclassSerializer.serialize(status)" in generated_code, (
+            "Optional query parameter should be serialized with DataclassSerializer.serialize() "
+            "to handle enum→string conversion"
+        )
+        assert "if status is not None" in generated_code, "Optional parameter should have None check"
+        # Verify serialization happens inside the conditional
+        assert '{"status": DataclassSerializer.serialize(status)}' in generated_code
+
+    def test_write_header_params__required_enum_parameter__serializes_value(
+        self, url_args_generator: EndpointUrlArgsGenerator, render_context_mock: MagicMock
+    ) -> None:
+        """
+        Scenario: Required header parameter is an enum type
+        Expected Outcome: Generated code calls DataclassSerializer.serialize() to convert enum to string value
+
+        Headers must be strings. This test ensures enums are properly converted.
+        """
+        # Arrange
+        writer = CodeWriter()
+        operation = IROperation(
+            operation_id="get_data",
+            summary="Get data",
+            description="Get data with API version header",
+            method=HTTPMethod.GET,
+            path="/data",
+            tags=["data"],
+            parameters=[
+                IRParameter(
+                    name="X-API-Version",
+                    param_in="header",
+                    required=True,
+                    schema=IRSchema(type="string", is_nullable=False),
+                    description="API version",
+                )
+            ],
+            request_body=None,
+            responses=[],
+        )
+        ordered_params = [
+            {
+                "name": "x_api_version",
+                "original_name": "X-API-Version",
+                "param_in": "header",
+                "required": True,
+            }
+        ]
+
+        # Act
+        url_args_generator._write_header_params(writer, operation, ordered_params, render_context_mock)
+        generated_code = writer.get_code()
+
+        # Assert
+        assert "DataclassSerializer.serialize(x_api_version)" in generated_code, (
+            "Required header parameter should be serialized with DataclassSerializer.serialize() "
+            "to handle enum→string conversion"
+        )
+        assert '"X-API-Version": DataclassSerializer.serialize(x_api_version)' in generated_code
+
+    def test_write_header_params__optional_enum_parameter__serializes_value(
+        self, url_args_generator: EndpointUrlArgsGenerator, render_context_mock: MagicMock
+    ) -> None:
+        """
+        Scenario: Optional header parameter is an enum type
+        Expected Outcome: Generated code calls DataclassSerializer.serialize() within None check
+
+        Ensures enum headers are properly converted to strings before being sent to httpx.
+        """
+        # Arrange
+        writer = CodeWriter()
+        operation = IROperation(
+            operation_id="get_data",
+            summary="Get data",
+            description="Get data with optional API version header",
+            method=HTTPMethod.GET,
+            path="/data",
+            tags=["data"],
+            parameters=[
+                IRParameter(
+                    name="X-API-Version",
+                    param_in="header",
+                    required=False,
+                    schema=IRSchema(type="string", is_nullable=True),
+                    description="Optional API version",
+                )
+            ],
+            request_body=None,
+            responses=[],
+        )
+        ordered_params = [
+            {
+                "name": "x_api_version",
+                "original_name": "X-API-Version",
+                "param_in": "header",
+                "required": False,
+            }
+        ]
+
+        # Act
+        url_args_generator._write_header_params(writer, operation, ordered_params, render_context_mock)
+        generated_code = writer.get_code()
+
+        # Assert
+        assert "DataclassSerializer.serialize(x_api_version)" in generated_code, (
+            "Optional header parameter should be serialized with DataclassSerializer.serialize() "
+            "to handle enum→string conversion"
+        )
+        assert "if x_api_version is not None" in generated_code, "Optional parameter should have None check"
+        assert '{"X-API-Version": DataclassSerializer.serialize(x_api_version)}' in generated_code
+
+    def test_generate_url_and_args__path_enum_parameter__serializes_before_url(
+        self, url_args_generator: EndpointUrlArgsGenerator, code_writer_mock: MagicMock, render_context_mock: MagicMock
+    ) -> None:
+        """
+        Scenario: Path parameter is an enum type
+        Expected Outcome: Generated code serializes path variable before URL construction
+
+        Path parameters are interpolated into f-strings. Enums must be converted to strings
+        first, otherwise f-string will produce "DocumentStatus.INDEXED" instead of "indexed".
+        """
+        # Arrange
+        operation = IROperation(
+            operation_id="get_document_by_status",
+            summary="Get document by status",
+            description="Get document using status path parameter",
+            method=HTTPMethod.GET,
+            path="/documents/{status}",
+            tags=["documents"],
+            parameters=[
+                IRParameter(
+                    name="status",
+                    param_in="path",
+                    required=True,
+                    schema=IRSchema(type="string", is_nullable=False),
+                    description="Document status",
+                )
+            ],
+            request_body=None,
+            responses=[],
+        )
+        ordered_params = [
+            {
+                "name": "status",
+                "original_name": "status",
+                "param_in": "path",
+                "required": True,
+            }
+        ]
+
+        # Act
+        url_args_generator.generate_url_and_args(
+            code_writer_mock, operation, render_context_mock, ordered_params, None, None
+        )
+
+        # Assert
+        # Get all write_line calls
+        calls = [c[0][0] for c in code_writer_mock.write_line.call_args_list]
+
+        # Find serialization and URL lines
+        serialize_line_idx = None
+        url_line_idx = None
+
+        for i, line in enumerate(calls):
+            if "DataclassSerializer.serialize(status)" in line and "status = " in line:
+                serialize_line_idx = i
+            if 'url = f"{self.base_url}' in line:
+                url_line_idx = i
+
+        # Assert serialization happens before URL construction
+        assert serialize_line_idx is not None, (
+            "Path parameter must be serialized using DataclassSerializer.serialize() "
+            "to convert enum to string before f-string interpolation"
+        )
+        assert url_line_idx is not None, "URL construction line should exist"
+        assert serialize_line_idx < url_line_idx, (
+            "Path parameter serialization must occur before URL construction "
+            "to ensure f-string receives string value, not enum object"
+        )