fin-infra 0.1.62__py3-none-any.whl → 0.1.69__py3-none-any.whl

fin_infra/investments/models.py

@@ -17,17 +17,17 @@ Models are provider-agnostic and normalize data from Plaid, SnapTrade, etc.
 
 from __future__ import annotations
 
-from datetime import date, datetime
+from datetime import date
 from decimal import Decimal
 from enum import Enum
-from typing import Dict, List, Optional
+from typing import TYPE_CHECKING, Dict, List, Optional
 
 from pydantic import BaseModel, ConfigDict, Field, computed_field
 
 
 class SecurityType(str, Enum):
     """Security type classification.
-    
+
     Categories:
     - equity: Common stock (AAPL, GOOGL, etc.)
     - etf: Exchange-traded fund (SPY, QQQ, etc.)
@@ -49,7 +49,7 @@ class SecurityType(str, Enum):
 
 class TransactionType(str, Enum):
     """Investment transaction type.
-    
+
     Categories:
     - buy: Purchase of security
     - sell: Sale of security
@@ -71,7 +71,7 @@ class TransactionType(str, Enum):
     fee = "fee"
     tax = "tax"
     transfer = "transfer"
-    split = "split"
+    split = "split"  # type: ignore[assignment]  # str.split() name conflict
     merger = "merger"
     cancel = "cancel"
     other = "other"
@@ -79,10 +79,10 @@ class TransactionType(str, Enum):
 
 class Security(BaseModel):
     """Security details (stock, bond, ETF, etc.).
-    
+
     Represents a tradable security with identifying information and current market data.
     Normalized across providers (Plaid, SnapTrade).
-    
+
     Example:
         >>> security = Security(
         ...     security_id="plaid_sec_123",
@@ -129,7 +129,9 @@ class Security(BaseModel):
     # Basic info
     name: str = Field(..., description="Security name")
     type: SecurityType = Field(..., description="Security type (equity, etf, bond, etc.)")
-    sector: Optional[str] = Field(None, description="Sector classification (Technology, Healthcare)")
+    sector: Optional[str] = Field(
+        None, description="Sector classification (Technology, Healthcare)"
+    )
 
     # Market data
     close_price: Optional[Decimal] = Field(None, ge=0, description="Latest closing price")
@@ -140,10 +142,10 @@ class Security(BaseModel):
 
 class Holding(BaseModel):
     """Investment holding with current value and cost basis.
-    
+
     Represents a position in a specific security within an investment account.
     Includes quantity, current value, cost basis, and calculated P&L.
-    
+
     Example:
         >>> holding = Holding(
         ...     account_id="acct_123",
@@ -193,38 +195,61 @@ class Holding(BaseModel):
     # Position data
     quantity: Decimal = Field(..., ge=0, description="Number of shares/units held")
     institution_price: Decimal = Field(..., ge=0, description="Current price per share")
-    institution_value: Decimal = Field(..., ge=0, description="Current market value (quantity × price)")
-    cost_basis: Optional[Decimal] = Field(None, ge=0, description="Total cost basis (original purchase price)")
+    institution_value: Decimal = Field(
+        ..., ge=0, description="Current market value (quantity × price)"
+    )
+    cost_basis: Optional[Decimal] = Field(
+        None, ge=0, description="Total cost basis (original purchase price)"
+    )
 
     # Additional data
     currency: str = Field("USD", description="Currency code")
     unofficial_currency_code: Optional[str] = Field(None, description="For crypto/alt currencies")
     as_of_date: Optional[date] = Field(None, description="Date of pricing data")
 
-    @computed_field
-    @property
-    def unrealized_gain_loss(self) -> Optional[Decimal]:
-        """Calculate unrealized gain/loss (current value - cost basis)."""
-        if self.cost_basis is None:
-            return None
-        return self.institution_value - self.cost_basis
-
-    @computed_field
-    @property
-    def unrealized_gain_loss_percent(self) -> Optional[Decimal]:
-        """Calculate unrealized gain/loss percentage."""
-        if self.cost_basis is None or self.cost_basis == 0:
-            return None
-        gain_loss = self.institution_value - self.cost_basis
-        return round((gain_loss / self.cost_basis) * 100, 2)
+    if TYPE_CHECKING:
+
+        @property
+        def unrealized_gain_loss(self) -> Optional[Decimal]:
+            """Calculate unrealized gain/loss (current value - cost basis)."""
+            if self.cost_basis is None:
+                return None
+            return self.institution_value - self.cost_basis
+
+        @property
+        def unrealized_gain_loss_percent(self) -> Optional[Decimal]:
+            """Calculate unrealized gain/loss percentage."""
+            if self.cost_basis is None or self.cost_basis == 0:
+                return None
+            gain_loss = self.institution_value - self.cost_basis
+            return round((gain_loss / self.cost_basis) * 100, 2)
+
+    else:
+
+        @computed_field
+        @property
+        def unrealized_gain_loss(self) -> Optional[Decimal]:
+            """Calculate unrealized gain/loss (current value - cost basis)."""
+            if self.cost_basis is None:
+                return None
+            return self.institution_value - self.cost_basis
+
+        @computed_field
+        @property
+        def unrealized_gain_loss_percent(self) -> Optional[Decimal]:
+            """Calculate unrealized gain/loss percentage."""
+            if self.cost_basis is None or self.cost_basis == 0:
+                return None
+            gain_loss = self.institution_value - self.cost_basis
+            return round((gain_loss / self.cost_basis) * 100, 2)
 
 
 class InvestmentTransaction(BaseModel):
     """Investment transaction (buy, sell, dividend, etc.).
-    
+
     Represents a single transaction in an investment account.
     Used to calculate realized gains and track transaction history.
-    
+
     Example:
         >>> transaction = InvestmentTransaction(
         ...     transaction_id="tx_123",
@@ -276,7 +301,9 @@ class InvestmentTransaction(BaseModel):
     # Transaction details
     transaction_date: date = Field(..., alias="date", description="Transaction date")
     name: str = Field(..., description="Transaction description")
-    transaction_type: TransactionType = Field(..., alias="type", description="Transaction type (buy, sell, dividend)")
+    transaction_type: TransactionType = Field(
+        ..., alias="type", description="Transaction type (buy, sell, dividend)"
+    )
     subtype: Optional[str] = Field(None, description="Provider-specific subtype")
 
     # Amounts
@@ -292,10 +319,10 @@ class InvestmentTransaction(BaseModel):
 
 class InvestmentAccount(BaseModel):
     """Investment account with aggregated holdings and metrics.
-    
+
     Represents a complete investment account with all holdings, balances, and P&L.
     Includes calculated fields for total value, cost basis, and unrealized gains.
-    
+
     Example:
         >>> account = InvestmentAccount(
         ...     account_id="acct_123",
@@ -321,7 +348,11 @@ class InvestmentAccount(BaseModel):
                 "holdings": [
                     {
                         "account_id": "acct_abc123",
-                        "security": {"ticker_symbol": "AAPL", "name": "Apple Inc.", "type": "equity"},
+                        "security": {
+                            "ticker_symbol": "AAPL",
+                            "name": "Apple Inc.",
+                            "type": "equity",
+                        },
                         "quantity": 10.5,
                         "institution_price": 150.25,
                         "institution_value": 1577.63,
@@ -350,42 +381,77 @@ class InvestmentAccount(BaseModel):
     # Holdings
     holdings: List[Holding] = Field(default_factory=list, description="List of holdings in account")
 
-    @computed_field
-    @property
-    def total_value(self) -> Decimal:
-        """Calculate total account value (sum of holdings + cash)."""
-        holdings_value = sum(h.institution_value for h in self.holdings)
-        cash_balance = self.balances.get("current") or Decimal(0)
-        return holdings_value + cash_balance
-
-    @computed_field
-    @property
-    def total_cost_basis(self) -> Decimal:
-        """Calculate total cost basis (sum of cost_basis across holdings)."""
-        return sum(h.cost_basis for h in self.holdings if h.cost_basis is not None)
-
-    @computed_field
-    @property
-    def total_unrealized_gain_loss(self) -> Decimal:
-        """Calculate total unrealized P&L (value - cost_basis)."""
-        holdings_value = sum(h.institution_value for h in self.holdings)
-        return holdings_value - self.total_cost_basis
-
-    @computed_field
-    @property
-    def total_unrealized_gain_loss_percent(self) -> Optional[Decimal]:
-        """Calculate total unrealized P&L percentage."""
-        if self.total_cost_basis == 0:
-            return None
-        return round((self.total_unrealized_gain_loss / self.total_cost_basis) * 100, 2)
+    if TYPE_CHECKING:
+
+        @property
+        def total_value(self) -> Decimal:
+            """Calculate total account value (sum of holdings + cash)."""
+            holdings_value = sum((h.institution_value for h in self.holdings), start=Decimal(0))
+            cash_balance = self.balances.get("current") or Decimal(0)
+            return holdings_value + cash_balance
+
+        @property
+        def total_cost_basis(self) -> Decimal:
+            """Calculate total cost basis (sum of cost_basis across holdings)."""
+            return sum(
+                (h.cost_basis for h in self.holdings if h.cost_basis is not None),
+                start=Decimal(0),
+            )
+
+        @property
+        def total_unrealized_gain_loss(self) -> Decimal:
+            """Calculate total unrealized P&L (value - cost_basis)."""
+            holdings_value = sum((h.institution_value for h in self.holdings), start=Decimal(0))
+            return holdings_value - self.total_cost_basis
+
+        @property
+        def total_unrealized_gain_loss_percent(self) -> Optional[Decimal]:
+            """Calculate total unrealized P&L percentage."""
+            if self.total_cost_basis == 0:
+                return None
+            return round((self.total_unrealized_gain_loss / self.total_cost_basis) * 100, 2)
+
+    else:
+
+        @computed_field
+        @property
+        def total_value(self) -> Decimal:
+            """Calculate total account value (sum of holdings + cash)."""
+            holdings_value = sum((h.institution_value for h in self.holdings), start=Decimal(0))
+            cash_balance = self.balances.get("current") or Decimal(0)
+            return holdings_value + cash_balance
+
+        @computed_field
+        @property
+        def total_cost_basis(self) -> Decimal:
+            """Calculate total cost basis (sum of cost_basis across holdings)."""
+            return sum(
+                (h.cost_basis for h in self.holdings if h.cost_basis is not None),
+                start=Decimal(0),
+            )
+
+        @computed_field
+        @property
+        def total_unrealized_gain_loss(self) -> Decimal:
+            """Calculate total unrealized P&L (value - cost_basis)."""
+            holdings_value = sum((h.institution_value for h in self.holdings), start=Decimal(0))
+            return holdings_value - self.total_cost_basis
+
+        @computed_field
+        @property
+        def total_unrealized_gain_loss_percent(self) -> Optional[Decimal]:
+            """Calculate total unrealized P&L percentage."""
+            if self.total_cost_basis == 0:
+                return None
+            return round((self.total_unrealized_gain_loss / self.total_cost_basis) * 100, 2)
 
 
 class AssetAllocation(BaseModel):
     """Asset allocation breakdown by security type and sector.
-    
+
     Provides percentage breakdown of portfolio by security type and sector.
     Used for diversification analysis and portfolio visualization.
-    
+
     Example:
         >>> allocation = AssetAllocation(
         ...     by_security_type={

fin_infra/investments/providers/base.py

@@ -77,9 +77,7 @@ class InvestmentProvider(ABC):
         pass
 
     @abstractmethod
-    async def get_securities(
-        self, access_token: str, security_ids: List[str]
-    ) -> List[Security]:
+    async def get_securities(self, access_token: str, security_ids: List[str]) -> List[Security]:
         """Fetch security details (ticker, name, type, current price).
 
         Args:
@@ -97,9 +95,7 @@ class InvestmentProvider(ABC):
         pass
 
     @abstractmethod
-    async def get_investment_accounts(
-        self, access_token: str
-    ) -> List[InvestmentAccount]:
+    async def get_investment_accounts(self, access_token: str) -> List[InvestmentAccount]:
         """Fetch investment accounts with aggregated holdings.
 
         Args:
@@ -176,8 +172,7 @@ class InvestmentProvider(ABC):
         }
 
         by_sector_percent = {
-            sector: round((value / total_value) * 100, 2)
-            for sector, value in sector_values.items()
+            sector: round((value / total_value) * 100, 2) for sector, value in sector_values.items()
         }
 
         cash_percent = round((cash_value / total_value) * 100, 2)

fin_infra/investments/providers/plaid.py

@@ -10,7 +10,7 @@ from __future__ import annotations
 
 from datetime import date
 from decimal import Decimal
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, List, Optional, cast
 
 from plaid.api import plaid_api
 from plaid.model.investments_holdings_get_request import InvestmentsHoldingsGetRequest
@@ -31,7 +31,6 @@ from ..models import (
     InvestmentAccount,
     InvestmentTransaction,
     Security,
-    SecurityType,
     TransactionType,
 )
 from .base import InvestmentProvider
@@ -103,7 +102,7 @@ class PlaidInvestmentProvider(InvestmentProvider):
             "development": plaid.Environment.Sandbox,  # Map development to sandbox
             "production": plaid.Environment.Production,
         }
-        return hosts.get(environment.lower(), plaid.Environment.Sandbox)
+        return cast(str, hosts.get(environment.lower(), plaid.Environment.Sandbox))
 
     async def get_holdings(
         self, access_token: str, account_ids: Optional[List[str]] = None
@@ -136,9 +135,7 @@ class PlaidInvestmentProvider(InvestmentProvider):
             if account_ids:
                 request.options = {"account_ids": account_ids}
 
-            response: InvestmentsHoldingsGetResponse = (
-                self.client.investments_holdings_get(request)
-            )
+            response: InvestmentsHoldingsGetResponse = self.client.investments_holdings_get(request)
 
             # Build security lookup map
             securities_map = {
@@ -204,8 +201,8 @@ class PlaidInvestmentProvider(InvestmentProvider):
             if account_ids:
                 request.options = {"account_ids": account_ids}
 
-            response: InvestmentsTransactionsGetResponse = (
-                self.client.investments_transactions_get(request)
+            response: InvestmentsTransactionsGetResponse = self.client.investments_transactions_get(
+                request
             )
 
             # Build security lookup map
@@ -229,9 +226,7 @@ class PlaidInvestmentProvider(InvestmentProvider):
         except ApiException as e:
             raise self._transform_error(e)
 
-    async def get_securities(
-        self, access_token: str, security_ids: List[str]
-    ) -> List[Security]:
+    async def get_securities(self, access_token: str, security_ids: List[str]) -> List[Security]:
         """Fetch security details from Plaid holdings.
 
         Note: Plaid doesn't have a dedicated securities endpoint.
@@ -254,9 +249,7 @@ class PlaidInvestmentProvider(InvestmentProvider):
         """
         try:
             request = InvestmentsHoldingsGetRequest(access_token=access_token)
-            response: InvestmentsHoldingsGetResponse = (
-                self.client.investments_holdings_get(request)
-            )
+            response: InvestmentsHoldingsGetResponse = self.client.investments_holdings_get(request)
 
             # Filter securities by requested IDs
             securities = []
@@ -271,9 +264,7 @@ class PlaidInvestmentProvider(InvestmentProvider):
         except ApiException as e:
             raise self._transform_error(e)
 
-    async def get_investment_accounts(
-        self, access_token: str
-    ) -> List[InvestmentAccount]:
+    async def get_investment_accounts(self, access_token: str) -> List[InvestmentAccount]:
         """Fetch investment accounts with aggregated holdings.
 
         Returns accounts with total value, cost basis, and unrealized P&L.
@@ -295,9 +286,7 @@ class PlaidInvestmentProvider(InvestmentProvider):
         """
         try:
             request = InvestmentsHoldingsGetRequest(access_token=access_token)
-            response: InvestmentsHoldingsGetResponse = (
-                self.client.investments_holdings_get(request)
-            )
+            response: InvestmentsHoldingsGetResponse = self.client.investments_holdings_get(request)
 
             # Build security lookup map
             securities_map = {
@@ -318,11 +307,7 @@ class PlaidInvestmentProvider(InvestmentProvider):
                     if account_id not in accounts_map:
                         # Find account metadata
                         plaid_account = next(
-                            (
-                                acc
-                                for acc in response.accounts
-                                if acc.account_id == account_id
-                            ),
+                            (acc for acc in response.accounts if acc.account_id == account_id),
                             None,
                         )
                         accounts_map[account_id] = {
@@ -340,12 +325,16 @@ class PlaidInvestmentProvider(InvestmentProvider):
 
                 investment_account = InvestmentAccount(
                     account_id=account_id,
-                    name=account_dict.get("name", account_dict.get("official_name", "Unknown Account")),
+                    name=account_dict.get(
+                        "name", account_dict.get("official_name", "Unknown Account")
+                    ),
                     type=account_dict.get("type", "investment"),
                     subtype=account_dict.get("subtype"),
                     balances={
-                        "current": float(account_dict.get("balances", {}).get("current", 0)),
-                        "available": float(account_dict.get("balances", {}).get("available")),
+                        "current": Decimal(str(account_dict.get("balances", {}).get("current", 0))),
+                        "available": Decimal(
+                            str(account_dict.get("balances", {}).get("available") or 0)
+                        ),
                     },
                     holdings=holdings,
                 )
@@ -363,14 +352,14 @@ class PlaidInvestmentProvider(InvestmentProvider):
         # Handle close_price - Plaid may return None for securities without recent pricing
         close_price_raw = plaid_security.get("close_price")
         close_price = Decimal(str(close_price_raw)) if close_price_raw is not None else Decimal("0")
-        
+
         return Security(
             security_id=plaid_security["security_id"],
             cusip=plaid_security.get("cusip"),
             isin=plaid_security.get("isin"),
             sedol=plaid_security.get("sedol"),
             ticker_symbol=plaid_security.get("ticker_symbol"),
-            name=plaid_security.get("name"),
+            name=plaid_security.get("name") or "Unknown Security",
             type=self._normalize_security_type(plaid_security.get("type", "other")),
             sector=plaid_security.get("sector"),
             close_price=close_price,
@@ -379,9 +368,7 @@ class PlaidInvestmentProvider(InvestmentProvider):
             currency=plaid_security.get("iso_currency_code", "USD"),
         )
 
-    def _transform_holding(
-        self, plaid_holding: Dict[str, Any], security: Security
-    ) -> Holding:
+    def _transform_holding(self, plaid_holding: Dict[str, Any], security: Security) -> Holding:
         """Transform Plaid holding data to Holding model."""
         return Holding(
             account_id=plaid_holding["account_id"],
@@ -389,7 +376,9 @@ class PlaidInvestmentProvider(InvestmentProvider):
             quantity=Decimal(str(plaid_holding.get("quantity", 0))),
             institution_price=Decimal(str(plaid_holding.get("institution_price", 0))),
             institution_value=Decimal(str(plaid_holding.get("institution_value", 0))),
-            cost_basis=Decimal(str(plaid_holding.get("cost_basis"))) if plaid_holding.get("cost_basis") else None,
+            cost_basis=Decimal(str(plaid_holding.get("cost_basis")))
+            if plaid_holding.get("cost_basis")
+            else None,
             currency=plaid_holding.get("iso_currency_code", "USD"),
             unofficial_currency_code=plaid_holding.get("unofficial_currency_code"),
         )

fin_infra/investments/providers/snaptrade.py

@@ -11,7 +11,7 @@ from __future__ import annotations
 
 from datetime import date
 from decimal import Decimal
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, List, Optional, cast
 
 import httpx
 
@@ -77,9 +77,7 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
             ValueError: If client_id or consumer_key is missing
         """
         if not client_id or not consumer_key:
-            raise ValueError(
-                "client_id and consumer_key are required for SnapTrade provider"
-            )
+            raise ValueError("client_id and consumer_key are required for SnapTrade provider")
 
         self.client_id = client_id
         self.consumer_key = consumer_key
@@ -97,15 +95,15 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
 
     def _auth_headers(self, user_id: str, user_secret: str) -> Dict[str, str]:
         """Build authentication headers for SnapTrade API requests.
-        
+
         SECURITY: User secrets are passed in headers, NOT URL params.
         URL params are logged in access logs, browser history, and proxy logs.
         Headers are not logged by default in most web servers.
-        
+
         Args:
             user_id: SnapTrade user ID
             user_secret: SnapTrade user secret (sensitive!)
-            
+
         Returns:
             Dict with authentication headers
         """
@@ -158,9 +156,7 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
             all_holdings = []
             for account in accounts:
                 account_id = account["id"]
-                positions_url = (
-                    f"{self.base_url}/accounts/{account_id}/positions"
-                )
+                positions_url = f"{self.base_url}/accounts/{account_id}/positions"
                 pos_response = await self.client.get(positions_url, headers=auth_headers)
                 pos_response.raise_for_status()
                 positions = await pos_response.json()
@@ -226,15 +222,15 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
             all_transactions = []
             for account in accounts:
                 account_id = account["id"]
-                transactions_url = (
-                    f"{self.base_url}/accounts/{account_id}/transactions"
-                )
+                transactions_url = f"{self.base_url}/accounts/{account_id}/transactions"
                 # Date params are non-sensitive, only auth goes in headers
                 tx_params = {
                     "startDate": start_date.isoformat(),
                     "endDate": end_date.isoformat(),
                 }
-                tx_response = await self.client.get(transactions_url, params=tx_params, headers=auth_headers)
+                tx_response = await self.client.get(
+                    transactions_url, params=tx_params, headers=auth_headers
+                )
                 tx_response.raise_for_status()
                 transactions = await tx_response.json()
 
@@ -250,9 +246,7 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
         except Exception as e:
             raise ValueError(f"SnapTrade API error: {str(e)}")
 
-    async def get_securities(
-        self, access_token: str, security_ids: List[str]
-    ) -> List[Security]:
+    async def get_securities(self, access_token: str, security_ids: List[str]) -> List[Security]:
         """Fetch security details from SnapTrade positions.
 
         Note: SnapTrade doesn't have a dedicated securities endpoint.
@@ -290,9 +284,7 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
         except Exception as e:
             raise ValueError(f"SnapTrade API error: {str(e)}")
 
-    async def get_investment_accounts(
-        self, access_token: str
-    ) -> List[InvestmentAccount]:
+    async def get_investment_accounts(self, access_token: str) -> List[InvestmentAccount]:
         """Fetch investment accounts with aggregated holdings.
 
         Returns accounts with total value, cost basis, and unrealized P&L.
@@ -328,9 +320,7 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
                 account_id = account["id"]
 
                 # Get positions for this account
-                positions_url = (
-                    f"{self.base_url}/accounts/{account_id}/positions"
-                )
+                positions_url = f"{self.base_url}/accounts/{account_id}/positions"
                 pos_response = await self.client.get(positions_url, headers=auth_headers)
                 pos_response.raise_for_status()
                 positions = await pos_response.json()
@@ -354,8 +344,8 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
                     type=account.get("type", "investment"),
                     subtype=account.get("account_type"),
                     balances={
-                        "current": float(balances.get("total", {}).get("amount", 0)),
-                        "available": float(balances.get("cash", {}).get("amount", 0)),
+                        "current": Decimal(str(balances.get("total", {}).get("amount", 0))),
+                        "available": Decimal(str(balances.get("cash", {}).get("amount", 0))),
                     },
                     holdings=holdings,
                 )
@@ -368,9 +358,7 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
         except Exception as e:
             raise ValueError(f"SnapTrade API error: {str(e)}")
 
-    async def list_connections(
-        self, access_token: str
-    ) -> List[Dict[str, Any]]:
+    async def list_connections(self, access_token: str) -> List[Dict[str, Any]]:
         """List brokerage connections for a user.
 
         Returns which brokerages the user has connected (E*TRADE, Robinhood, etc.).
@@ -393,7 +381,7 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
             url = f"{self.base_url}/connections"
             response = await self.client.get(url, headers=auth_headers)
             response.raise_for_status()
-            return await response.json()
+            return cast(list[dict[str, Any]], await response.json())
 
         except httpx.HTTPStatusError as e:
             raise self._transform_error(e)
@@ -490,16 +478,12 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
             user_id, user_secret = access_token.split(":", 1)
             return user_id, user_secret
         except ValueError:
-            raise ValueError(
-                "Invalid access_token format. Expected 'user_id:user_secret'"
-            )
+            raise ValueError("Invalid access_token format. Expected 'user_id:user_secret'")
 
-    def _transform_holding(
-        self, snaptrade_position: Dict[str, Any], account_id: str
-    ) -> Holding:
+    def _transform_holding(self, snaptrade_position: Dict[str, Any], account_id: str) -> Holding:
         """Transform SnapTrade position data to Holding model."""
         symbol_data = snaptrade_position.get("symbol", {})
-        
+
         # Create Security from symbol data
         security = Security(
             security_id=symbol_data.get("id", symbol_data.get("symbol", "")),
@@ -513,9 +497,7 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
         # SnapTrade uses "average_purchase_price" for cost basis
         avg_price = snaptrade_position.get("average_purchase_price")
         quantity = Decimal(str(snaptrade_position.get("units", 0)))
-        cost_basis = (
-            Decimal(str(avg_price)) * quantity if avg_price is not None else None
-        )
+        cost_basis = Decimal(str(avg_price)) * quantity if avg_price is not None else None
 
         return Holding(
             account_id=account_id,
@@ -532,7 +514,7 @@ class SnapTradeInvestmentProvider(InvestmentProvider):
     ) -> InvestmentTransaction:
         """Transform SnapTrade transaction to InvestmentTransaction model."""
         symbol_data = snaptrade_tx.get("symbol", {})
-        
+
         # Create Security from symbol data
         security = Security(
             security_id=symbol_data.get("id", symbol_data.get("symbol", "")),