fix(chutes): address PR review feedback

- Use 'rotator_library' logger for consistency (chutes_provider.py)
- Add HTTPStatusError handling in get_models() for 4xx/5xx responses
- Add timeout=30.0 to httpx.AsyncClient in run_background_job()
- Remove unused UsageManager import from chutes_quota_tracker.py

Author: b3nw

src/rotator_library/providers/chutes_provider.py

@@ -11,7 +11,7 @@
 # Create a local logger for this module
 import logging
 
-lib_logger = logging.getLogger(__name__)
+lib_logger = logging.getLogger("rotator_library")
 
 # Concurrency limit for parallel quota fetches
 QUOTA_FETCH_CONCURRENCY = 5
@@ -67,7 +67,7 @@ async def get_models(self, api_key: str, client: httpx.AsyncClient) -> List[str]
             return [
                 f"chutes/{model['id']}" for model in response.json().get("data", [])
             ]
-        except httpx.RequestError as e:
+        except (httpx.RequestError, httpx.HTTPStatusError) as e:
             lib_logger.error(f"Failed to fetch chutes.ai models: {e}")
             return []
 
@@ -138,7 +138,7 @@ async def refresh_single_credential(
                     lib_logger.warning(f"Failed to refresh Chutes quota usage: {e}")
 
         # Fetch all credentials in parallel with shared HTTP client
-        async with httpx.AsyncClient() as client:
+        async with httpx.AsyncClient(timeout=30.0) as client:
             tasks = [
                 refresh_single_credential(api_key, client) for api_key in credentials
             ]

src/rotator_library/providers/utilities/chutes_quota_tracker.py

@@ -19,13 +19,10 @@
 import logging
 import time
 from datetime import datetime, timezone, timedelta
-from typing import Any, Dict, List, Optional, Tuple, TYPE_CHECKING
+from typing import Any, Dict, List, Optional, Tuple
 
 import httpx
 
-if TYPE_CHECKING:
-    from ...usage_manager import UsageManager
-
 # Use the shared rotator_library logger
 lib_logger = logging.getLogger("rotator_library")