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

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

@@ -1,6 +1,93 @@
 # CHANGELOG
 
 
+## v0.21.0 (2025-10-23)
+
+### Bug Fixes
+
+- **serialization**: Respect BaseSchema field mappings in DataclassSerializer
+  ([`ef5d365`](https://github.com/mindhiveoy/pyopenapi_gen/commit/ef5d3656008417bbae17e6370e33e7b0c12cefd6))
+
+BREAKING CHANGE: DataclassSerializer now correctly maps Python snake_case field names to API
+  camelCase field names when serializing BaseSchema instances.
+
+Problem: DataclassSerializer.serialize() was using Python field names (snake_case) directly as
+  dictionary keys, ignoring the field name mappings defined in
+  BaseSchema.Meta.key_transform_with_load. This caused API requests to send incorrect field names,
+  breaking communication with camelCase APIs.
+
+Solution: Modified DataclassSerializer._serialize_with_tracking() to detect BaseSchema instances and
+  use their to_dict() method, which properly handles field name mapping. Falls back to original
+  behavior for plain dataclasses to maintain backwards compatibility.
+
+Changes: - Updated DataclassSerializer._serialize_with_tracking() in core/utils.py - Added 5
+  comprehensive unit tests for field mapping scenarios - Added 2 end-to-end integration tests with
+  business_swagger.json - All existing tests pass with 88.42% coverage (exceeds 85% requirement)
+
+Example: Before: {"data_source_id": "123"} ❌
+
+After: {"dataSourceId": "123"} ✅
+
+Fixes critical bug affecting all generated clients with camelCase APIs.
+
+### Chores
+
+- **release**: Sync __init__.py version [skip ci]
+  ([`446d496`](https://github.com/mindhiveoy/pyopenapi_gen/commit/446d4963a1e8428da7b6476088104aac599dae8f))
+
+### Breaking Changes
+
+- **serialization**: Dataclassserializer now correctly maps Python snake_case field names to API
+  camelCase field names when serializing BaseSchema instances.
+
+
+## v0.20.1 (2025-10-21)
+
+### Bug Fixes
+
+- **codegen**: Resolve method naming and file handling in overloaded endpoints
+  ([`190e581`](https://github.com/mindhiveoy/pyopenapi_gen/commit/190e5819e99edf37d14722b075069851d03c4c3b))
+
+This commit fixes two critical issues in code generation for operations with multiple content types
+  (overloaded methods):
+
+1. **Method Naming**: Overloaded methods were using camelCase from OpenAPI operationId instead of
+  converting to Python's snake_case convention. - Fixed: Added NameSanitizer.sanitize_method_name()
+  in overload_generator.py - Example: updateDocument -> update_document
+
+2. **File Upload Handling**: Multipart/form-data file uploads were incorrectly being passed through
+  DataclassSerializer.serialize(), which is designed for dataclass-to-dict conversion, not file I/O.
+  - Fixed: Pass file dict directly to httpx transport in endpoint_method_generator.py - Rationale:
+  httpx expects raw IO objects, not serialized data
+
+Both issues only affected operations with multiple content types (JSON + multipart). Standard
+  single-content-type operations were unaffected.
+
+Changes: - src/pyopenapi_gen/visit/endpoint/generators/overload_generator.py: Added NameSanitizer
+  import and method name sanitisation in two locations (overload signatures and implementation
+  signatures)
+
+- src/pyopenapi_gen/visit/endpoint/generators/endpoint_method_generator.py: Removed
+  DataclassSerializer.serialize() for file parameters in multipart handling, passing files directly
+  to transport
+
+- tests/visit/endpoint/generators/test_overload_generator.py: Added unit tests for snake_case
+  conversion in overload and implementation signatures
+
+- tests/visit/endpoint/generators/test_endpoint_method_generator.py: Added test for file handling
+  without serialisation
+
+- tests/generation_issues/test_overload_naming_issues.py: Added integration tests verifying both
+  fixes in full code generation pipeline
+
+All tests pass (1348 tests) with full quality checks (format, lint, typecheck, security).
+
+### Chores
+
+- **release**: Sync __init__.py version [skip ci]
+  ([`8047e53`](https://github.com/mindhiveoy/pyopenapi_gen/commit/8047e53df11d8985d95f89dbf3b36046af1e6f1a))
+
+
 ## v0.20.0 (2025-10-19)
 
 ### Bug Fixes

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyopenapi-gen
-Version: 0.20.0
+Version: 0.21.0
 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.20.0 → pyopenapi_gen-0.21.0}/coverage.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" ?>
-<coverage version="7.10.7" timestamp="1760875420300" lines-valid="6505" lines-covered="5750" line-rate="0.8839" branches-covered="0" branches-valid="0" branch-rate="0" complexity="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">
 	<!-- 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>
@@ -652,7 +652,7 @@
 				</class>
 			</classes>
 		</package>
-		<package name="pyopenapi_gen.core" line-rate="0.8327" branch-rate="0" complexity="0">
+		<package name="pyopenapi_gen.core" line-rate="0.836" branch-rate="0" complexity="0">
 			<classes>
 				<class name="__init__.py" filename="pyopenapi_gen/core/__init__.py" complexity="0" line-rate="1" branch-rate="0">
 					<methods/>
@@ -1053,7 +1053,7 @@
 						<line number="69" hits="1"/>
 					</lines>
 				</class>
-				<class name="utils.py" filename="pyopenapi_gen/core/utils.py" complexity="0" line-rate="0.9353" branch-rate="0">
+				<class name="utils.py" filename="pyopenapi_gen/core/utils.py" complexity="0" line-rate="0.9392" branch-rate="0">
 					<methods/>
 					<lines>
 						<line number="6" hits="1"/>
@@ -1173,59 +1173,70 @@
 						<line number="313" hits="1"/>
 						<line number="320" hits="1"/>
 						<line number="321" hits="1"/>
-						<line number="339" hits="1"/>
 						<line number="341" hits="1"/>
-						<line number="342" hits="1"/>
+						<line number="343" hits="1"/>
 						<line number="344" hits="1"/>
-						<line number="347" hits="1"/>
-						<line number="348" hits="1"/>
-						<line number="351" hits="1"/>
-						<line number="352" hits="1"/>
+						<line number="346" hits="1"/>
+						<line number="349" hits="1"/>
+						<line number="350" hits="1"/>
+						<line number="353" hits="1"/>
 						<line number="354" hits="1"/>
-						<line number="355" hits="1"/>
-						<line number="356" hits="0"/>
-						<line number="359" hits="1"/>
-						<line number="360" hits="1"/>
-						<line number="363" hits="1"/>
-						<line number="364" hits="1"/>
-						<line number="367" hits="1"/>
-						<line number="368" hits="1"/>
-						<line number="369" hits="1"/>
+						<line number="356" hits="1"/>
+						<line number="357" hits="1"/>
+						<line number="358" hits="0"/>
+						<line number="361" hits="1"/>
+						<line number="362" hits="1"/>
+						<line number="365" hits="1"/>
+						<line number="366" hits="1"/>
 						<line number="370" hits="1"/>
 						<line number="371" hits="1"/>
 						<line number="372" hits="1"/>
 						<line number="374" hits="1"/>
-						<line number="375" hits="1"/>
 						<line number="376" hits="1"/>
 						<line number="377" hits="1"/>
 						<line number="378" hits="1"/>
+						<line number="379" hits="1"/>
 						<line number="380" hits="1"/>
+						<line number="381" hits="1"/>
 						<line number="383" hits="1"/>
-						<line number="384" hits="1"/>
+						<line number="386" hits="1"/>
 						<line number="387" hits="1"/>
 						<line number="388" hits="1"/>
 						<line number="389" hits="1"/>
 						<line number="390" hits="1"/>
 						<line number="391" hits="1"/>
-						<line number="392" hits="1"/>
 						<line number="393" hits="1"/>
+						<line number="394" hits="1"/>
+						<line number="395" hits="1"/>
 						<line number="396" hits="1"/>
 						<line number="397" hits="1"/>
-						<line number="400" hits="1"/>
+						<line number="399" hits="1"/>
 						<line number="402" hits="1"/>
 						<line number="403" hits="1"/>
-						<line number="404" hits="1"/>
-						<line number="405" hits="1"/>
 						<line number="406" hits="1"/>
 						<line number="407" hits="1"/>
 						<line number="408" hits="1"/>
 						<line number="409" hits="1"/>
 						<line number="410" hits="1"/>
 						<line number="411" hits="1"/>
-						<line number="413" hits="1"/>
-						<line number="416" hits="0"/>
-						<line number="417" hits="0"/>
-						<line number="419" hits="0"/>
+						<line number="412" hits="1"/>
+						<line number="415" hits="1"/>
+						<line number="416" hits="1"/>
+						<line number="419" hits="1"/>
+						<line number="421" hits="1"/>
+						<line number="422" hits="1"/>
+						<line number="423" hits="1"/>
+						<line number="424" hits="1"/>
+						<line number="425" hits="1"/>
+						<line number="426" hits="1"/>
+						<line number="427" hits="1"/>
+						<line number="428" hits="1"/>
+						<line number="429" hits="1"/>
+						<line number="430" hits="1"/>
+						<line number="432" hits="1"/>
+						<line number="435" hits="0"/>
+						<line number="436" hits="0"/>
+						<line number="438" hits="0"/>
 					</lines>
 				</class>
 				<class name="warning_collector.py" filename="pyopenapi_gen/core/warning_collector.py" complexity="0" line-rate="1" branch-rate="0">
@@ -6008,7 +6019,7 @@
 				</class>
 			</classes>
 		</package>
-		<package name="pyopenapi_gen.visit.endpoint.generators" line-rate="0.8881" branch-rate="0" complexity="0">
+		<package name="pyopenapi_gen.visit.endpoint.generators" line-rate="0.8884" 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/>
@@ -6085,7 +6096,7 @@
 						<line number="123" hits="1"/>
 					</lines>
 				</class>
-				<class name="endpoint_method_generator.py" filename="pyopenapi_gen/visit/endpoint/generators/endpoint_method_generator.py" complexity="0" line-rate="0.9291" branch-rate="0">
+				<class name="endpoint_method_generator.py" filename="pyopenapi_gen/visit/endpoint/generators/endpoint_method_generator.py" complexity="0" line-rate="0.9286" branch-rate="0">
 					<methods/>
 					<lines>
 						<line number="1" hits="1"/>
@@ -6188,7 +6199,6 @@
 						<line number="181" hits="1"/>
 						<line number="182" hits="1"/>
 						<line number="183" hits="1"/>
-						<line number="184" hits="1"/>
 						<line number="185" hits="1"/>
 						<line number="186" hits="1"/>
 						<line number="187" hits="1"/>
@@ -6228,93 +6238,96 @@
 						<line number="10" hits="1"/>
 						<line number="11" hits="1"/>
 						<line number="12" hits="1"/>
-						<line number="14" hits="1"/>
-						<line number="17" hits="1"/>
-						<line number="26" hits="1"/>
+						<line number="13" hits="1"/>
+						<line number="15" hits="1"/>
+						<line number="18" hits="1"/>
 						<line number="27" hits="1"/>
 						<line number="28" hits="1"/>
 						<line number="29" hits="1"/>
 						<line number="30" hits="1"/>
-						<line number="32" hits="1"/>
-						<line number="42" hits="1"/>
+						<line number="31" hits="1"/>
+						<line number="33" hits="1"/>
 						<line number="43" hits="1"/>
 						<line number="44" hits="1"/>
-						<line number="46" hits="1"/>
-						<line number="60" hits="1"/>
+						<line number="45" hits="1"/>
+						<line number="47" hits="1"/>
 						<line number="61" hits="1"/>
-						<line number="64" hits="1"/>
+						<line number="62" hits="1"/>
 						<line number="65" hits="1"/>
 						<line number="66" hits="1"/>
 						<line number="67" hits="1"/>
-						<line number="69" hits="1"/>
-						<line number="72" hits="1"/>
-						<line number="76" hits="1"/>
+						<line number="68" hits="1"/>
+						<line number="70" hits="1"/>
+						<line number="73" hits="1"/>
 						<line number="77" hits="1"/>
 						<line number="78" hits="1"/>
-						<line number="80" hits="1"/>
-						<line number="82" hits="1"/>
-						<line number="103" hits="1"/>
-						<line number="106" hits="1"/>
-						<line number="109" hits="1"/>
-						<line number="112" hits="1"/>
-						<line number="115" hits="1"/>
+						<line number="79" hits="1"/>
+						<line number="81" hits="1"/>
+						<line number="83" hits="1"/>
+						<line number="104" hits="1"/>
+						<line number="107" hits="1"/>
+						<line number="110" hits="1"/>
+						<line number="113" hits="1"/>
 						<line number="116" hits="1"/>
-						<line number="118" hits="1"/>
-						<line number="120" hits="1"/>
+						<line number="117" hits="1"/>
+						<line number="119" hits="1"/>
 						<line number="121" hits="1"/>
 						<line number="122" hits="1"/>
 						<line number="123" hits="1"/>
-						<line number="126" hits="1"/>
-						<line number="129" hits="1"/>
-						<line number="132" hits="1"/>
-						<line number="135" hits="1"/>
-						<line number="138" hits="1"/>
+						<line number="124" hits="1"/>
+						<line number="127" hits="1"/>
+						<line number="130" hits="1"/>
+						<line number="133" hits="1"/>
+						<line number="136" hits="1"/>
 						<line number="139" 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="145" hits="1"/>
+						<line number="146" hits="1"/>
 						<line number="147" hits="1"/>
-						<line number="159" hits="1"/>
-						<line number="161" hits="1"/>
-						<line number="164" hits="1"/>
-						<line number="166" hits="1"/>
-						<line number="167" hits="1"/>
-						<line number="169" hits="1"/>
+						<line number="149" hits="1"/>
+						<line number="151" hits="1"/>
+						<line number="163" hits="1"/>
+						<line number="165" hits="1"/>
+						<line number="168" hits="1"/>
+						<line number="170" hits="1"/>
 						<line number="171" hits="1"/>
 						<line number="173" hits="1"/>
 						<line number="175" hits="1"/>
+						<line number="177" hits="1"/>
 						<line number="179" hits="1"/>
-						<line number="180" hits="1"/>
-						<line number="182" hits="1"/>
-						<line number="197" hits="1"/>
-						<line number="200" hits="1"/>
-						<line number="203" hits="1"/>
+						<line number="183" hits="1"/>
+						<line number="184" hits="1"/>
+						<line number="186" hits="1"/>
+						<line number="201" hits="1"/>
 						<line number="204" hits="1"/>
-						<line number="206" hits="1"/>
+						<line number="207" hits="1"/>
 						<line number="208" hits="1"/>
-						<line number="209" hits="1"/>
 						<line number="210" hits="1"/>
-						<line number="211" hits="1"/>
+						<line number="212" hits="1"/>
+						<line number="213" hits="1"/>
 						<line number="214" hits="1"/>
-						<line number="217" hits="1"/>
+						<line number="215" hits="1"/>
 						<line number="218" hits="1"/>
-						<line number="220" hits="1"/>
 						<line number="221" hits="1"/>
+						<line number="222" hits="1"/>
 						<line number="224" hits="1"/>
 						<line number="225" hits="1"/>
-						<line number="226" hits="1"/>
-						<line number="227" hits="1"/>
+						<line number="228" hits="1"/>
+						<line number="229" hits="1"/>
 						<line number="230" hits="1"/>
-						<line number="233" hits="1"/>
-						<line number="236" hits="1"/>
+						<line number="231" hits="1"/>
+						<line number="234" hits="1"/>
 						<line number="237" hits="1"/>
-						<line number="238" hits="1"/>
-						<line number="239" hits="1"/>
 						<line number="240" hits="1"/>
-						<line number="241" hits="1"/>
 						<line number="243" hits="1"/>
+						<line number="244" hits="1"/>
+						<line number="245" hits="1"/>
+						<line number="246" hits="1"/>
+						<line number="247" hits="1"/>
+						<line number="248" hits="1"/>
+						<line number="250" hits="1"/>
 					</lines>
 				</class>
 				<class name="request_generator.py" filename="pyopenapi_gen/visit/endpoint/generators/request_generator.py" complexity="0" line-rate="0.9535" branch-rate="0">

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

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "pyopenapi-gen"
-version = "0.20.0"
+version = "0.21.0"
 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.20.0"
+version = "0.21.0"
 version_files = [
     "pyproject.toml:version",
     "src/pyopenapi_gen/__init__.py:__version__"

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

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

{pyopenapi_gen-0.20.0 → pyopenapi_gen-0.21.0}/src/pyopenapi_gen/core/utils.py

@@ -328,10 +328,12 @@ class DataclassSerializer:
             The serialized object with dataclasses converted to dictionaries.
 
         Handles:
-        - Dataclass instances: Converted to dictionaries
+        - BaseSchema instances: Uses to_dict() method with field name mapping (e.g., snake_case -> camelCase)
+        - Regular dataclass instances: Converted to dictionaries using field names
         - Lists: Recursively serialize each item
         - Dictionaries: Recursively serialize values
         - datetime: Convert to ISO format string
+        - Enums: Convert to their value
         - Primitives: Return unchanged
         - None values: Excluded from output
         """
@@ -363,7 +365,24 @@ class DataclassSerializer:
         if isinstance(obj, Enum) and not isinstance(obj, type):
             return obj.value
 
-        # Handle dataclass instances
+        # Handle BaseSchema instances (respects field name mappings)
+        # Check for BaseSchema by looking for both to_dict and _get_field_mappings methods
+        if hasattr(obj, "to_dict") and hasattr(obj, "_get_field_mappings") and callable(obj.to_dict):
+            visited.add(obj_id)
+            try:
+                # Use BaseSchema's to_dict() which handles field name mapping
+                result_dict = obj.to_dict(exclude_none=True)
+                # Recursively serialize nested objects in the result
+                serialized_result = {}
+                for key, value in result_dict.items():
+                    serialized_value = DataclassSerializer._serialize_with_tracking(value, visited)
+                    if serialized_value is not None:
+                        serialized_result[key] = serialized_value
+                return serialized_result
+            finally:
+                visited.discard(obj_id)
+
+        # Handle regular dataclass instances (no field mapping)
         if dataclasses.is_dataclass(obj) and not isinstance(obj, type):
             visited.add(obj_id)
             try:

{pyopenapi_gen-0.20.0 → pyopenapi_gen-0.21.0}/src/pyopenapi_gen/visit/endpoint/generators/endpoint_method_generator.py

@@ -181,12 +181,12 @@ class EndpointMethodGenerator:
                 writer.dedent()
                 writer.write_line(")")
             elif content_type == "multipart/form-data":
-                writer.write_line(f"files_data = DataclassSerializer.serialize({param_info['name']})")
+                # Files dict is already in correct format for httpx - pass directly
                 writer.write_line("response = await self._transport.request(")
                 writer.indent()
                 writer.write_line(f'"{op.method.value.upper()}", url,')
                 writer.write_line("params=None,")
-                writer.write_line("files=files_data,")
+                writer.write_line(f"files={param_info['name']},")
                 writer.write_line("headers=None")
                 writer.dedent()
                 writer.write_line(")")

{pyopenapi_gen-0.20.0 → pyopenapi_gen-0.21.0}/src/pyopenapi_gen/visit/endpoint/generators/overload_generator.py

@@ -6,6 +6,7 @@ from typing import Any
 from pyopenapi_gen import IROperation
 
 from ....context.render_context import RenderContext
+from ....core.utils import NameSanitizer
 from ....core.writers.code_writer import CodeWriter
 from ..processors.parameter_processor import EndpointParameterProcessor
 from .docstring_generator import EndpointDocstringGenerator
@@ -134,9 +135,12 @@ class OverloadMethodGenerator:
         # Get return type from response strategy
         return_type = response_strategy.return_type
 
+        # Sanitize method name to snake_case
+        method_name = NameSanitizer.sanitize_method_name(op.operation_id)
+
         # Write signature
         params_str = ",\n    ".join(param_parts)
-        writer.write_line(f"async def {op.operation_id}(")
+        writer.write_line(f"async def {method_name}(")
         writer.indent()
         writer.write_line(params_str)
         writer.dedent()
@@ -232,9 +236,12 @@ class OverloadMethodGenerator:
         # Get return type
         return_type = response_strategy.return_type
 
+        # Sanitize method name to snake_case
+        method_name = NameSanitizer.sanitize_method_name(op.operation_id)
+
         # Write signature
         params_str = ",\n    ".join(param_parts)
-        writer.write_line(f"async def {op.operation_id}(")
+        writer.write_line(f"async def {method_name}(")
         writer.indent()
         writer.write_line(params_str)
         writer.dedent()