Fix major mypy type checking issues

Author: parman-devotel

src/devo_global_comms_python/exceptions.py

@@ -423,29 +423,29 @@ def create_exception_from_response(response: requests.Response) -> DevoAPIExcept
         request_id = None
 
     # Extract rate limit headers
-    retry_after = None
-    limit = None
-    remaining = None
+    retry_after: Optional[int] = None
+    limit: Optional[int] = None
+    remaining: Optional[int] = None
 
     if status_code == 429:
-        retry_after = response.headers.get("Retry-After")
-        if retry_after:
+        retry_after_str = response.headers.get("Retry-After")
+        if retry_after_str:
             try:
-                retry_after = int(retry_after)
+                retry_after = int(retry_after_str)
             except ValueError:
                 retry_after = None
 
-        limit = response.headers.get("X-RateLimit-Limit")
-        if limit:
+        limit_str = response.headers.get("X-RateLimit-Limit")
+        if limit_str:
             try:
-                limit = int(limit)
+                limit = int(limit_str)
             except ValueError:
                 limit = None
 
-        remaining = response.headers.get("X-RateLimit-Remaining")
-        if remaining:
+        remaining_str = response.headers.get("X-RateLimit-Remaining")
+        if remaining_str:
             try:
-                remaining = int(remaining)
+                remaining = int(remaining_str)
             except ValueError:
                 remaining = None
 

src/devo_global_comms_python/models/contact_groups.py

@@ -9,8 +9,8 @@ class ContactsGroup(BaseModel):
     Contact group model representing a collection of contacts.
     """
 
-    id: str = Field(..., description="Unique identifier for the contact group")
-    name: str = Field(..., description="Name of the contact group")
+    id: str = Field(description="Unique identifier for the contact group")
+    name: str = Field(description="Name of the contact group")
     description: Optional[str] = Field(None, description="Description of the contact group")
     contacts_count: Optional[int] = Field(None, description="Number of contacts in the group")
     created_at: Optional[datetime] = Field(None, description="Creation timestamp")
@@ -24,7 +24,7 @@ class CreateContactsGroupDto(BaseModel):
     DTO for creating a new contact group.
     """
 
-    name: str = Field(..., description="Name of the contact group", min_length=1, max_length=255)
+    name: str = Field(description="Name of the contact group", min_length=1, max_length=255)
     description: Optional[str] = Field(None, description="Description of the contact group", max_length=1000)
     contact_ids: Optional[List[str]] = Field(None, description="List of contact IDs to add to the group")
     metadata: Optional[Dict[str, Any]] = Field(None, description="Additional metadata")
@@ -46,7 +46,7 @@ class DeleteContactsGroupsDto(BaseModel):
     DTO for bulk deleting contact groups.
     """
 
-    group_ids: List[str] = Field(..., description="List of contact group IDs to delete", min_items=1)
+    group_ids: List[str] = Field(description="List of contact group IDs to delete", min_length=1)
     transfer_contacts_to: Optional[str] = Field(None, description="Group ID to transfer contacts to before deletion")
 
 
@@ -55,8 +55,8 @@ class ContactsGroupListResponse(BaseModel):
     Response model for listing contact groups with pagination.
     """
 
-    groups: List[ContactsGroup] = Field(..., description="List of contact groups")
-    total: int = Field(..., description="Total number of groups")
-    page: int = Field(..., description="Current page number")
-    limit: int = Field(..., description="Number of items per page")
-    total_pages: int = Field(..., description="Total number of pages")
+    groups: List[ContactsGroup] = Field(description="List of contact groups")
+    total: int = Field(description="Total number of groups")
+    page: int = Field(description="Current page number")
+    limit: int = Field(description="Number of items per page")
+    total_pages: int = Field(description="Total number of pages")

src/devo_global_comms_python/models/contacts.py

@@ -9,7 +9,7 @@ class Contact(BaseModel):
     Contact model representing a contact in the Devo Global Communications API.
     """
 
-    id: str = Field(..., description="Unique identifier for the contact")
+    id: str = Field(description="Unique identifier for the contact")
     account_id: Optional[str] = Field(None, description="Account identifier")
     user_id: Optional[str] = Field(None, description="User identifier")
     phone_number: Optional[str] = Field(None, description="Contact phone number in E.164 format")
@@ -160,7 +160,7 @@ class DeleteContactsDto(BaseModel):
     Data transfer object for deleting contacts.
     """
 
-    contact_ids: List[str] = Field(..., min_items=1, description="List of contact IDs to delete")
+    contact_ids: List[str] = Field(description="List of contact IDs to delete", min_length=1)
 
     @validator("contact_ids")
     def validate_contact_ids(cls, v):
@@ -175,8 +175,8 @@ class AssignToContactsGroupDto(BaseModel):
     Data transfer object for assigning/unassigning contacts to/from groups.
     """
 
-    contact_ids: List[str] = Field(..., min_items=1, description="List of contact IDs")
-    contacts_group_id: str = Field(..., description="Contact group ID")
+    contact_ids: List[str] = Field(description="List of contact IDs", min_length=1)
+    contacts_group_id: str = Field(description="Contact group ID")
 
     @validator("contact_ids")
     def validate_contact_ids(cls, v):
@@ -198,7 +198,7 @@ class CreateContactsFromCsvDto(BaseModel):
     Data transfer object for importing contacts from CSV.
     """
 
-    csv_data: str = Field(..., description="CSV data as string")
+    csv_data: str = Field(description="CSV data as string")
     contacts_group_id: Optional[str] = Field(None, description="Contact group ID to assign imported contacts")
     skip_duplicates: Optional[bool] = Field(True, description="Skip duplicate contacts")
     update_existing: Optional[bool] = Field(False, description="Update existing contacts")
@@ -216,10 +216,10 @@ class CreateContactsFromCsvRespDto(BaseModel):
     Response data transfer object for CSV import operation.
     """
 
-    total_processed: int = Field(..., description="Total number of contacts processed")
-    successfully_created: int = Field(..., description="Number of contacts successfully created")
-    skipped_duplicates: int = Field(..., description="Number of duplicate contacts skipped")
-    failed_imports: int = Field(..., description="Number of failed imports")
+    total_processed: int = Field(description="Total number of contacts processed")
+    successfully_created: int = Field(description="Number of contacts successfully created")
+    skipped_duplicates: int = Field(description="Number of duplicate contacts skipped")
+    failed_imports: int = Field(description="Number of failed imports")
     errors: Optional[List[str]] = Field(None, description="List of error messages")
 
 
@@ -228,11 +228,11 @@ class GetContactsSerializer(BaseModel):
     Serializer for paginated contacts list response.
     """
 
-    contacts: List[ContactSerializer] = Field(..., description="List of contacts")
-    total: int = Field(..., description="Total number of contacts")
-    page: int = Field(..., description="Current page number")
-    limit: int = Field(..., description="Number of contacts per page")
-    total_pages: int = Field(..., description="Total number of pages")
+    contacts: List[ContactSerializer] = Field(description="List of contacts")
+    total: int = Field(description="Total number of contacts")
+    page: int = Field(description="Current page number")
+    limit: int = Field(description="Number of contacts per page")
+    total_pages: int = Field(description="Total number of pages")
 
     @validator("page")
     def validate_page(cls, v):
@@ -255,9 +255,9 @@ class CustomField(BaseModel):
     Custom field model.
     """
 
-    id: str = Field(..., description="Unique identifier for the custom field")
-    name: str = Field(..., description="Custom field name")
-    field_type: str = Field(..., description="Field type (text, number, date, boolean, etc.)")
+    id: str = Field(description="Unique identifier for the custom field")
+    name: str = Field(description="Custom field name")
+    field_type: str = Field(description="Field type (text, number, date, boolean, etc.)")
     description: Optional[str] = Field(None, description="Custom field description")
     is_required: Optional[bool] = Field(False, description="Whether the field is required")
     default_value: Optional[Any] = Field(None, description="Default value for the field")
@@ -280,8 +280,8 @@ class CreateCustomFieldDto(BaseModel):
     Data transfer object for creating a custom field.
     """
 
-    name: str = Field(..., description="Custom field name")
-    field_type: str = Field(..., description="Field type (text, number, date, boolean, etc.)")
+    name: str = Field(description="Custom field name")
+    field_type: str = Field(description="Field type (text, number, date, boolean, etc.)")
     description: Optional[str] = Field(None, description="Custom field description")
     is_required: Optional[bool] = Field(False, description="Whether the field is required")
     default_value: Optional[Any] = Field(None, description="Default value for the field")
@@ -337,11 +337,11 @@ class GetCustomFieldsSerializer(BaseModel):
     Serializer for paginated custom fields list response.
     """
 
-    custom_fields: List[CustomFieldSerializer] = Field(..., description="List of custom fields")
-    total: int = Field(..., description="Total number of custom fields")
-    page: int = Field(..., description="Current page number")
-    limit: int = Field(..., description="Number of custom fields per page")
-    total_pages: int = Field(..., description="Total number of pages")
+    custom_fields: List[CustomFieldSerializer] = Field(description="List of custom fields")
+    total: int = Field(description="Total number of custom fields")
+    page: int = Field(description="Current page number")
+    limit: int = Field(description="Number of custom fields per page")
+    total_pages: int = Field(description="Total number of pages")
 
     @validator("page")
     def validate_page(cls, v):
@@ -363,7 +363,7 @@ class CommonDeleteDto(BaseModel):
     Common data transfer object for delete operations.
     """
 
-    ids: List[str] = Field(..., min_items=1, description="List of IDs to delete")
+    ids: List[str] = Field(description="List of IDs to delete", min_length=1)
 
     @validator("ids")
     def validate_ids(cls, v):

src/devo_global_comms_python/resources/contacts.py

@@ -1,4 +1,4 @@
-from typing import TYPE_CHECKING, List, Optional
+from typing import TYPE_CHECKING, Any, Dict, List, Optional
 
 from ..utils import validate_required_string
 from .base import BaseResource
@@ -70,7 +70,7 @@ def list(
         Returns:
             GetContactsSerializer: Paginated list of contacts
         """
-        params = {"page": page, "limit": limit}
+        params: Dict[str, Any] = {"page": page, "limit": limit}
 
         if contacts_group_ids:
             params["contacts_group_ids"] = contacts_group_ids
@@ -221,7 +221,7 @@ def list_custom_fields(
         Returns:
             GetCustomFieldsSerializer: Paginated list of custom fields
         """
-        params = {"page": page, "limit": limit}
+        params: Dict[str, Any] = {"page": page, "limit": limit}
 
         if id:
             params["id"] = id