Fix format and types

Author: albertsola

mpt_api_client/models/model.py

@@ -30,12 +30,13 @@ class ModelList(UserList[Any]):
     """A list that automatically converts dictionaries to BaseModel objects."""
 
     def __init__(
-        self, iterable: Iterable[Any] | None = None, model_class: type["BaseModel"] | None = None
+        self,
+        iterable: Iterable[Any] | None = None,
+        model_class: type["BaseModel"] | None = None,  # noqa: WPS221
     ) -> None:
         self._model_class = model_class or BaseModel
-        super().__init__(
-            self._process_item(item) for item in iterable
-        ) if iterable is not None else super().__init__()
+        iterable = iterable or []
+        super().__init__([self._process_item(item) for item in iterable])
 
     @override
     def append(self, item: Any) -> None:
@@ -65,7 +66,7 @@ def _process_item(self, item: Any) -> Any:
 class BaseModel:
     """Base dataclass for models providing object-only access and case conversion."""
 
-    def __init__(self, **kwargs: Any) -> None:
+    def __init__(self, **kwargs: Any) -> None:  # noqa: WPS210
         """Processes resource data to convert keys and handle nested structures."""
         # Get type hints for field mapping
         hints = getattr(self, "__annotations__", {})
@@ -78,49 +79,19 @@ def __init__(self, **kwargs: Any) -> None:
             processed_value = self._process_value(value, target_class=target_class)
             object.__setattr__(self, mapped_key, processed_value)
 
-    def _process_value(self, value: Any, target_class: Any = None) -> Any:  # noqa: C901
-        """Recursively processes values to ensure nested dicts are BaseModels."""
-        if isinstance(value, dict) and not isinstance(value, BaseModel):
-            # If a target class is provided and it's a subclass of BaseModel, use it
-            if (
-                target_class
-                and isinstance(target_class, type)
-                and issubclass(target_class, BaseModel)
-            ):
-                return target_class(**value)
-            return BaseModel(**value)
-
-        if isinstance(value, (list, UserList)) and not isinstance(value, ModelList):
-            # Try to determine the model class for the list elements from type hints
-            model_class = BaseModel
-            if target_class:
-                # Handle list[ModelClass]
-
-                origin = get_origin(target_class)
-                if origin is list:
-                    args = get_args(target_class)
-                    if args and isinstance(args[0], type) and issubclass(args[0], BaseModel):
-                        model_class = args[0]
-
-            return ModelList(value, model_class=model_class)
-        # Recursively handle BaseModel if it's already one
-        if isinstance(value, BaseModel):
-            return value
-        return value
-
     def __getattr__(self, name: str) -> Any:
         # 1. Try to find the attribute in __dict__ (includes attributes set in __init__)
         if name in self.__dict__:
-            return self.__dict__[name]
+            return self.__dict__[name]  # noqa: WPS420 WPS529
 
         # 2. Check for methods or properties
         try:
             return object.__getattribute__(self, name)
         except AttributeError:
-            pass
+            pass  # noqa: WPS420
 
         raise AttributeError(
-            f"'{self.__class__.__name__}' object has no attribute '{name}'",
+            f"'{self.__class__.__name__}' object has no attribute '{name}'",  # noqa: WPS237
         )
 
     @override
@@ -162,6 +133,36 @@ def _serialize_value(self, value: Any) -> Any:
             return [self._serialize_value(item) for item in value]
         return value
 
+    def _process_value(self, value: Any, target_class: Any = None) -> Any:  # noqa: WPS231 C901
+        """Recursively processes values to ensure nested dicts are BaseModels."""
+        if isinstance(value, dict) and not isinstance(value, BaseModel):
+            # If a target class is provided and it's a subclass of BaseModel, use it
+            if (
+                target_class
+                and isinstance(target_class, type)
+                and issubclass(target_class, BaseModel)
+            ):
+                return target_class(**value)
+            return BaseModel(**value)
+
+        if isinstance(value, (list, UserList)) and not isinstance(value, ModelList):
+            # Try to determine the model class for the list elements from type hints
+            model_class = BaseModel
+            if target_class:
+                # Handle list[ModelClass]
+
+                origin = get_origin(target_class)
+                if origin is list:
+                    args = get_args(target_class)
+                    if args and isinstance(args[0], type) and issubclass(args[0], BaseModel):  # noqa: WPS221
+                        model_class = args[0]  # noqa: WPS220
+
+            return ModelList(value, model_class=model_class)
+        # Recursively handle BaseModel if it's already one
+        if isinstance(value, BaseModel):
+            return value
+        return value
+
 
 class Model(BaseModel):
     """Provides a resource to interact with api data using fluent interfaces."""