style: auto-fix lint violations from enhanced ruff rules

7 files changed, 21 insertions, 19 deletions

diff --git a/services/api/src/trading_api/main.py b/services/api/src/trading_api/main.py
index 090b110..05c6d2f 100644
--- a/services/api/src/trading_api/main.py
+++ b/services/api/src/trading_api/main.py
@@ -11,7 +11,6 @@ from slowapi.util import get_remote_address
 
 from shared.config import Settings
 from shared.db import Database
-
 from trading_api.dependencies.auth import verify_token
 from trading_api.routers import orders, portfolio, strategies
 
diff --git a/services/api/src/trading_api/routers/orders.py b/services/api/src/trading_api/routers/orders.py
index 217efef..b664e2a 100644
--- a/services/api/src/trading_api/routers/orders.py
+++ b/services/api/src/trading_api/routers/orders.py
@@ -3,12 +3,13 @@
 import logging
 
 from fastapi import APIRouter, HTTPException, Query, Request
-from shared.sa_models import OrderRow, SignalRow
 from slowapi import Limiter
 from slowapi.util import get_remote_address
 from sqlalchemy import select
 from sqlalchemy.exc import OperationalError
 
+from shared.sa_models import OrderRow, SignalRow
+
 logger = logging.getLogger(__name__)
 
 router = APIRouter()
@@ -42,10 +43,10 @@ async def get_orders(request: Request, limit: int = Query(50, ge=1, le=1000)):
             ]
     except OperationalError as exc:
         logger.error("Database error fetching orders: %s", exc)
-        raise HTTPException(status_code=503, detail="Database unavailable")
+        raise HTTPException(status_code=503, detail="Database unavailable") from exc
     except Exception as exc:
         logger.error("Failed to get orders: %s", exc, exc_info=True)
-        raise HTTPException(status_code=500, detail="Failed to retrieve orders")
+        raise HTTPException(status_code=500, detail="Failed to retrieve orders") from exc
 
 
 @router.get("/signals")
@@ -73,7 +74,7 @@ async def get_signals(request: Request, limit: int = Query(50, ge=1, le=1000)):
             ]
     except OperationalError as exc:
         logger.error("Database error fetching signals: %s", exc)
-        raise HTTPException(status_code=503, detail="Database unavailable")
+        raise HTTPException(status_code=503, detail="Database unavailable") from exc
     except Exception as exc:
         logger.error("Failed to get signals: %s", exc, exc_info=True)
-        raise HTTPException(status_code=500, detail="Failed to retrieve signals")
+        raise HTTPException(status_code=500, detail="Failed to retrieve signals") from exc
diff --git a/services/api/src/trading_api/routers/portfolio.py b/services/api/src/trading_api/routers/portfolio.py
index fde90cb..56bee7c 100644
--- a/services/api/src/trading_api/routers/portfolio.py
+++ b/services/api/src/trading_api/routers/portfolio.py
@@ -3,10 +3,11 @@
 import logging
 
 from fastapi import APIRouter, HTTPException, Query, Request
-from shared.sa_models import PositionRow
 from sqlalchemy import select
 from sqlalchemy.exc import OperationalError
 
+from shared.sa_models import PositionRow
+
 logger = logging.getLogger(__name__)
 
 router = APIRouter()
@@ -32,10 +33,10 @@ async def get_positions(request: Request):
             ]
     except OperationalError as exc:
         logger.error("Database error fetching positions: %s", exc)
-        raise HTTPException(status_code=503, detail="Database unavailable")
+        raise HTTPException(status_code=503, detail="Database unavailable") from exc
     except Exception as exc:
         logger.error("Failed to get positions: %s", exc, exc_info=True)
-        raise HTTPException(status_code=500, detail="Failed to retrieve positions")
+        raise HTTPException(status_code=500, detail="Failed to retrieve positions") from exc
 
 
 @router.get("/snapshots")
@@ -55,7 +56,7 @@ async def get_snapshots(request: Request, days: int = Query(30, ge=1, le=365)):
         ]
     except OperationalError as exc:
         logger.error("Database error fetching snapshots: %s", exc)
-        raise HTTPException(status_code=503, detail="Database unavailable")
+        raise HTTPException(status_code=503, detail="Database unavailable") from exc
     except Exception as exc:
         logger.error("Failed to get snapshots: %s", exc, exc_info=True)
-        raise HTTPException(status_code=500, detail="Failed to retrieve snapshots")
+        raise HTTPException(status_code=500, detail="Failed to retrieve snapshots") from exc
diff --git a/services/api/src/trading_api/routers/strategies.py b/services/api/src/trading_api/routers/strategies.py
index 5db7320..157094c 100644
--- a/services/api/src/trading_api/routers/strategies.py
+++ b/services/api/src/trading_api/routers/strategies.py
@@ -44,7 +44,7 @@ async def list_strategies():
         ]
     except (ImportError, FileNotFoundError) as exc:
         logger.error("Strategy loading error: %s", exc)
-        raise HTTPException(status_code=503, detail="Strategy engine unavailable")
+        raise HTTPException(status_code=503, detail="Strategy engine unavailable") from exc
     except Exception as exc:
         logger.error("Failed to list strategies: %s", exc, exc_info=True)
-        raise HTTPException(status_code=500, detail="Failed to list strategies")
+        raise HTTPException(status_code=500, detail="Failed to list strategies") from exc
diff --git a/services/api/tests/test_api.py b/services/api/tests/test_api.py
index 669143b..f3b0a47 100644
--- a/services/api/tests/test_api.py
+++ b/services/api/tests/test_api.py
@@ -1,6 +1,7 @@
 """Tests for the REST API."""
 
 from unittest.mock import AsyncMock, patch
+
 from fastapi.testclient import TestClient
 
 
diff --git a/services/api/tests/test_orders_router.py b/services/api/tests/test_orders_router.py
index 0658619..52252c5 100644
--- a/services/api/tests/test_orders_router.py
+++ b/services/api/tests/test_orders_router.py
@@ -1,10 +1,10 @@
 """Tests for orders API router."""
 
-import pytest
 from unittest.mock import AsyncMock, MagicMock
-from fastapi.testclient import TestClient
-from fastapi import FastAPI
 
+import pytest
+from fastapi import FastAPI
+from fastapi.testclient import TestClient
 from trading_api.routers.orders import router
 
 
diff --git a/services/api/tests/test_portfolio_router.py b/services/api/tests/test_portfolio_router.py
index 3bd1b2c..8cd8ff8 100644
--- a/services/api/tests/test_portfolio_router.py
+++ b/services/api/tests/test_portfolio_router.py
@@ -1,11 +1,11 @@
 """Tests for portfolio API router."""
 
-import pytest
 from decimal import Decimal
 from unittest.mock import AsyncMock, MagicMock
-from fastapi.testclient import TestClient
-from fastapi import FastAPI
 
+import pytest
+from fastapi import FastAPI
+from fastapi.testclient import TestClient
 from trading_api.routers.portfolio import router