MPT-16304 Ensure correct model is used in services (#157)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Refactor**
* Enhanced type consistency across account and module service classes
for improved type safety and better IDE support.
* Aligned generic type parameters in service mixins to accurately match
their specific resource types.

* **Chores**
* Added type annotations to improve static code analysis in seed
utilities.

<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Author: d3rky

mpt_api_client/resources/accounts/account.py

@@ -39,11 +39,11 @@ class AccountsServiceConfig:
 
 
 class AccountsService(
-    CreateFileMixin[Model],
-    UpdateFileMixin[Model],
-    ActivatableMixin[Model],
-    EnablableMixin[Model],
-    ValidateMixin[Model],
+    CreateFileMixin[Account],
+    UpdateFileMixin[Account],
+    ActivatableMixin[Account],
+    EnablableMixin[Account],
+    ValidateMixin[Account],
     GetMixin[Account],
     CollectionMixin[Account],
     Service[Account],
@@ -59,11 +59,11 @@ def users(self, account_id: str) -> AccountsUsersService:
 
 
 class AsyncAccountsService(
-    AsyncCreateFileMixin[Model],
-    AsyncUpdateFileMixin[Model],
-    AsyncActivatableMixin[Model],
-    AsyncEnablableMixin[Model],
-    AsyncValidateMixin[Model],
+    AsyncCreateFileMixin[Account],
+    AsyncUpdateFileMixin[Account],
+    AsyncActivatableMixin[Account],
+    AsyncEnablableMixin[Account],
+    AsyncValidateMixin[Account],
     AsyncGetMixin[Account],
     AsyncCollectionMixin[Account],
     AsyncService[Account],

mpt_api_client/resources/accounts/buyers.py

@@ -38,11 +38,11 @@ class BuyersServiceConfig:
 
 
 class BuyersService(
-    CreateFileMixin[Model],
-    UpdateFileMixin[Model],
-    ActivatableMixin[Model],
-    EnablableMixin[Model],
-    ValidateMixin[Model],
+    CreateFileMixin[Buyer],
+    UpdateFileMixin[Buyer],
+    ActivatableMixin[Buyer],
+    EnablableMixin[Buyer],
+    ValidateMixin[Buyer],
     GetMixin[Buyer],
     DeleteMixin,
     CollectionMixin[Buyer],
@@ -71,11 +71,11 @@ def transfer(self, resource_id: str, resource_data: ResourceData | None = None)
 
 
 class AsyncBuyersService(
-    AsyncCreateFileMixin[Model],
-    AsyncUpdateFileMixin[Model],
-    AsyncActivatableMixin[Model],
-    AsyncEnablableMixin[Model],
-    AsyncValidateMixin[Model],
+    AsyncCreateFileMixin[Buyer],
+    AsyncUpdateFileMixin[Buyer],
+    AsyncActivatableMixin[Buyer],
+    AsyncEnablableMixin[Buyer],
+    AsyncValidateMixin[Buyer],
     AsyncGetMixin[Buyer],
     AsyncDeleteMixin,
     AsyncCollectionMixin[Buyer],

mpt_api_client/resources/accounts/licensees.py

@@ -33,9 +33,9 @@ class LicenseesServiceConfig:
 
 
 class LicenseesService(
-    CreateFileMixin[Model],
-    UpdateFileMixin[Model],
-    EnablableMixin[Model],
+    CreateFileMixin[Licensee],
+    UpdateFileMixin[Licensee],
+    EnablableMixin[Licensee],
     GetMixin[Licensee],
     DeleteMixin,
     CollectionMixin[Licensee],
@@ -46,9 +46,9 @@ class LicenseesService(
 
 
 class AsyncLicenseesService(
-    AsyncCreateFileMixin[Model],
-    AsyncUpdateFileMixin[Model],
-    AsyncEnablableMixin[Model],
+    AsyncCreateFileMixin[Licensee],
+    AsyncUpdateFileMixin[Licensee],
+    AsyncEnablableMixin[Licensee],
     AsyncGetMixin[Licensee],
     AsyncDeleteMixin,
     AsyncCollectionMixin[Licensee],

mpt_api_client/resources/accounts/modules.py

@@ -27,6 +27,6 @@ class ModulesService(
 
 
 class AsyncModulesService(
-    AsyncGetMixin[Module], AsyncCollectionMixin[Model], AsyncService[Module], ModulesServiceConfig
+    AsyncGetMixin[Module], AsyncCollectionMixin[Module], AsyncService[Module], ModulesServiceConfig
 ):
     """Asynchronous Modules Service."""

seed/accounts/buyer.py

@@ -78,7 +78,7 @@ async def init_buyer(
             context["accounts.buyer.id"] = created.id
             logger.info("Buyer created: %s", created.id)
             return created
-        logger.warning("Buyer creation failed")
+        logger.warning("Buyer creation failed")  # type: ignore[unreachable]
         raise ValueError("Buyer creation failed")
     logger.info("Buyer found: %s", buyer.id)
     return buyer

seed/accounts/licensee.py

@@ -89,7 +89,7 @@ async def init_licensee(
             context["accounts.licensee.id"] = created.id
             logger.info("Licensee created: %s", created.id)
             return created
-        logger.warning("Licensee creation failed")
+        logger.warning("Licensee creation failed")  # type: ignore[unreachable]
         raise ValueError("Licensee creation failed")
     logger.info("Licensee found: %s", licensee.id)
     return licensee

seed/accounts/module.py

@@ -50,7 +50,7 @@ async def refresh_module(
                 context["accounts.module.id"] = first_module.id
                 context.set_resource("accounts.module", first_module)
                 return first_module
-            logger.warning("First module is not a Module instance.")
+            logger.warning("First module is not a Module instance.")  # type: ignore[unreachable]
             return None
         logger.warning("Module 'Access Management' not found.")
         return None