feat: update type checker to 'ty', enhance type hints, and improve code formatting

Author: MarcoMuellner

.github/workflows/tests.yml

@@ -38,7 +38,7 @@ jobs:
         run: poetry run pytest --cov=src/ --cov-report=xml --cov-fail-under 90 --cov-config=.coveragerc
 
       - name: 🔍 Type checking with mypy
-        run: poetry run mypy src/
+        run: poetry run ty src/
 
       - name: 🎨 Code formatting with black
         run: poetry run black --check .

mypy.ini

@@ -17,7 +17,7 @@ keywords = ["OpenAPI", "Generator", "Python", "async"]
 Changelog = "https://github.com/MarcoMuellner/openapi-python-generator/releases"
 
 [tool.poetry.dependencies]
-python = "^3.8"
+python = "^3.9"
 httpx = {extras = ["all"], version = "^0.28.0"}
 pydantic = "^2.10.2"
 orjson = "^3.9.15"
@@ -27,14 +27,15 @@ black = ">=21.10b0"
 isort = ">=5.10.1"
 openapi-pydantic = "^0.5.1"
 pyyaml = "^6.0.2"
+importlib-metadata = "^8.7.0"
 
 [tool.poetry.group.dev.dependencies]
 Pygments = ">=2.10.0"
 coverage = {extras = ["toml"], version = "^6.4.1"}
 darglint = ">=1.8.1"
 ruff = ">=0.12.12"
 furo = ">=2021.11.12"
-mypy = ">=0.930"
+ty = "^0.0.1a20"
 pep8-naming = ">=0.10.1"
 pre-commit = ">=2.16.0"
 pre-commit-hooks = ">=4.1.0"

poetry.lock

@@ -72,7 +72,7 @@ def main(
     generate_data(
         source,
         output,
-        library,
+        library if library is not None else HTTPLibrary.httpx,
         env_token_name,
         use_orjson,
         custom_template_path,

pyproject.toml

@@ -3,7 +3,7 @@
 from typing import Optional
 
 _use_orjson: bool = False
-_custom_template_path: str = None
+_custom_template_path: str | None = None
 _symbol_ascii_strip_re = re.compile(r"[^A-Za-z0-9_]")
 
 

src/openapi_python_generator/__main__.py

@@ -89,7 +89,7 @@ def type_converter(  # noqa: C901
                 conversions.append(type_converter(sub_schema, True))
             else:
                 import_type = common.normalize_symbol(sub_schema.ref.split("/")[-1])
-                if import_type == model_name:
+                if import_type == model_name and model_name is not None:
                     conversions.append(
                         TypeConversion(
                             original_type=sub_schema.ref,

src/openapi_python_generator/language_converters/python/common.py

@@ -45,7 +45,13 @@
 from openapi_python_generator.language_converters.python.model_generator import (
     type_converter,
 )
-from openapi_python_generator.models import LibraryConfig, OpReturnType, Service, ServiceOperation, TypeConversion
+from openapi_python_generator.models import (
+    LibraryConfig,
+    OpReturnType,
+    Service,
+    ServiceOperation,
+    TypeConversion,
+)
 
 
 # Helper functions for isinstance checks across OpenAPI versions
@@ -54,16 +60,18 @@ def is_response_type(obj) -> bool:
     return isinstance(obj, (Response30, Response31))
 
 
-def create_media_type_for_reference(reference_obj):
+def create_media_type_for_reference(
+    reference_obj: Response30 | Reference30 | Response31 | Reference31,
+):
     """Create a MediaType wrapper for a reference object, using the correct version"""
     # Check which version the reference object belongs to
     if isinstance(reference_obj, Reference30):
-        return MediaType30(schema=reference_obj)
+        return MediaType30(schema=reference_obj)  # type: ignore - pydantic issue with generics
     elif isinstance(reference_obj, Reference31):
-        return MediaType31(schema=reference_obj)
+        return MediaType31(schema=reference_obj)  # type: ignore - pydantic issue with generics
     else:
         # Fallback to v3.0 for generic Reference
-        return MediaType30(schema=reference_obj)
+        return MediaType30(schema=reference_obj)  # type: ignore - pydantic issue with generics
 
 
 def is_media_type(obj) -> bool:
@@ -185,13 +193,16 @@ def _generate_params_from_content(content: Any):
         if isinstance(rb_content, dict) and any(
             rb_content.get(i) is not None for i in operation_request_body_types
         ):
-            get_keyword = [i for i in operation_request_body_types if rb_content.get(i)][
-                0
-            ]
+            get_keyword = [
+                i for i in operation_request_body_types if rb_content.get(i)
+            ][0]
             content = rb_content.get(get_keyword)
             if content is not None and hasattr(content, "media_type_schema"):
                 mts = getattr(content, "media_type_schema", None)
-                if isinstance(mts, (Reference, Reference30, Reference31, Schema, Schema30, Schema31)):
+                if isinstance(
+                    mts,
+                    (Reference, Reference30, Reference31, Schema, Schema30, Schema31),
+                ):
                     params += f"{_generate_params_from_content(mts)}, "
                 else:  # pragma: no cover
                     raise Exception(
@@ -283,9 +294,8 @@ def generate_return_type(operation: Operation) -> OpReturnType:
             )
         elif is_schema_type(inner_schema):
             converted_result = type_converter(inner_schema, True)  # type: ignore
-            if (
-                "array" in converted_result.original_type
-                and isinstance(converted_result.import_types, list)
+            if "array" in converted_result.original_type and isinstance(
+                converted_result.import_types, list
             ):
                 matched = re.findall(r"List\[(.+)\]", converted_result.converted_type)
                 if len(matched) > 0:
@@ -348,23 +358,28 @@ def generate_service_operation(
                             op.parameters = []  # type: ignore
                         op.parameters.append(p)  # type: ignore
         except Exception:  # pragma: no cover
-            print(f"Error merging path-level parameters for {path_name}")  # pragma: no cover
+            print(
+                f"Error merging path-level parameters for {path_name}"
+            )  # pragma: no cover
             pass
 
         params = generate_params(op)
         # Fallback: ensure all {placeholders} in path are present as function params
         try:
-            placeholder_names = [m.group(1) for m in re.finditer(r"\{([^}/]+)\}", path_name)]
+            placeholder_names = [
+                m.group(1) for m in re.finditer(r"\{([^}/]+)\}", path_name)
+            ]
             existing_param_names = {
-                p.split(":")[0].strip()
-                for p in params.split(",") if ":" in p
+                p.split(":")[0].strip() for p in params.split(",") if ":" in p
             }
             for ph in placeholder_names:
                 norm_ph = common.normalize_symbol(ph)
                 if norm_ph not in existing_param_names and norm_ph:
                     params = f"{norm_ph}: Any, " + params
         except Exception:  # pragma: no cover
-            print(f"Error ensuring path placeholders in params for {path_name}")  # pragma: no cover
+            print(
+                f"Error ensuring path placeholders in params for {path_name}"
+            )  # pragma: no cover
             pass
         operation_id = generate_operation_id(op, http_operation, path_name)
         query_params = generate_query_params(op)

src/openapi_python_generator/language_converters/python/model_generator.py

@@ -26,7 +26,7 @@ def parse_openapi_3_0(spec_data: dict) -> OpenAPI:
     Raises:
         ValidationError: If the specification is invalid
     """
-    return OpenAPI(**spec_data)
+    return OpenAPI(**spec_data)  # type: ignore - pydantic issue with extra fields
 
 
 def generate_code_3_0(