Fix circular imports and flake8 linting issues

Author: ChristianJStarr

projectx_sdk/client.py

@@ -223,10 +223,18 @@ def request(
 
             # Check for API-level errors
             if not data.get("success", True):
-                error_code = data.get("errorCode", 0) if data else 0  # type: ignore
-                error_message = data.get("errorMessage", "Unknown error") if data else "Unknown error"  # type: ignore
+                # Handle response data that might be None
+                if data is None:
+                    error_code = 0
+                    error_message = "Unknown error"
+                else:
+                    error_code = data.get("errorCode", 0)
+                    error_message = data.get("errorMessage", "Unknown error")
+                
                 raise ProjectXError(
-                    f"API error {error_code}: {error_message}", error_code=error_code, response=data
+                    f"API error {error_code}: {error_message}", 
+                    error_code=error_code, 
+                    response=data
                 )
 
             return data  # type: ignore

projectx_sdk/endpoints/__init__.py

@@ -2,13 +2,6 @@
 
 from abc import ABC
 
-from projectx_sdk.endpoints.account import AccountService
-from projectx_sdk.endpoints.contract import ContractService
-from projectx_sdk.endpoints.history import HistoryService, TimeUnit
-from projectx_sdk.endpoints.order import OrderService
-from projectx_sdk.endpoints.position import PositionService
-from projectx_sdk.endpoints.trade import TradeService
-
 
 class BaseService(ABC):
     """
@@ -27,7 +20,16 @@ def __init__(self, client):
         self._client = client
 
 
+# Import service classes after BaseService is defined to avoid circular imports
+from projectx_sdk.endpoints.account import AccountService  # noqa: E402
+from projectx_sdk.endpoints.contract import ContractService  # noqa: E402
+from projectx_sdk.endpoints.history import HistoryService, TimeUnit  # noqa: E402
+from projectx_sdk.endpoints.order import OrderService  # noqa: E402
+from projectx_sdk.endpoints.position import PositionService  # noqa: E402
+from projectx_sdk.endpoints.trade import TradeService  # noqa: E402
+
 __all__ = [
+    "BaseService",
     "AccountService",
     "ContractService",
     "HistoryService",

projectx_sdk/models/account.py

@@ -1,6 +1,6 @@
 """Account models for the ProjectX Gateway API."""
 
-from typing import List, Optional
+from typing import List
 
 from projectx_sdk.models import BaseModel
 from projectx_sdk.models.base import BaseResponse
@@ -62,7 +62,7 @@ def to_dict(self):
 
     def __repr__(self):
         """
-        String representation of the account.
+        Return string representation of the account.
 
         Returns:
             str: String representation

projectx_sdk/models/base.py

@@ -1,9 +1,6 @@
 """Base model implementations for ProjectX SDK."""
 
-import json
-from dataclasses import dataclass, field
-from datetime import datetime, timezone
-from typing import Any, Dict, List, Optional, Type, TypeVar
+from typing import Optional
 
 from pydantic import BaseModel, Field
 

projectx_sdk/models/contract.py

@@ -2,7 +2,7 @@
 
 from typing import List
 
-from pydantic import BaseModel, Field, model_validator
+from pydantic import BaseModel, Field
 
 from projectx_sdk.models.base import BaseResponse
 

projectx_sdk/realtime/__init__.py

@@ -66,7 +66,10 @@ def is_connected(self) -> bool:
         Returns:
             True if both user and market connections are active
         """
-        return self._user_connection.is_connected() and self._market_connection.is_connected()  # type: ignore
+        # Cast the result to bool to satisfy mypy
+        user_connected = bool(self._user_connection.is_connected())
+        market_connected = bool(self._market_connection.is_connected())
+        return user_connected and market_connected
 
     def reconnect_subscriptions(self):
         """