Merge pull request #135 from naveedkhan1998/revert-134-copilot/add-options-viewing-graph

Revert "feat: Options chain viewer on instrument graph page"

Author: naveedkhan1998

backend/apps/core/services/alpaca_service.py

@@ -159,59 +159,5 @@ def get_historic_bars(
             url = f"{self.data_base_url}/v2/stocks/{symbol}/bars"
             return self._make_request("GET", url, params=params)
 
-    def get_option_snapshots(
-        self,
-        underlying_symbol: str,
-        feed: str = "indicative",
-        expiration_date_gte: str | None = None,
-        expiration_date_lte: str | None = None,
-        option_type: Literal["call", "put"] | None = None,
-        limit: int = 200,
-        page_token: str | None = None,
-    ) -> dict:
-        """Fetch option chain snapshots (quotes + greeks) for an underlying symbol."""
-        params: dict = {"feed": feed, "limit": limit}
-        if expiration_date_gte:
-            params["expiration_date_gte"] = expiration_date_gte
-        if expiration_date_lte:
-            params["expiration_date_lte"] = expiration_date_lte
-        if option_type:
-            params["type"] = option_type
-        if page_token:
-            params["page_token"] = page_token
-
-        url = f"{self.data_base_url}/v1beta1/options/snapshots/{underlying_symbol}"
-        return self._make_request("GET", url, params=params)
-
-    def get_option_bars(
-        self,
-        symbols: str | list[str],
-        timeframe: str = "1Day",
-        start: str | None = None,
-        end: str | None = None,
-        limit: int = 1000,
-        page_token: str | None = None,
-        sort: Literal["asc", "desc"] = "asc",
-    ) -> dict:
-        """Fetch historical OHLCV bars for option contract symbols."""
-        if isinstance(symbols, list):
-            symbols = ",".join(symbols)
-
-        params: dict = {
-            "symbols": symbols,
-            "timeframe": timeframe,
-            "limit": limit,
-            "sort": sort,
-        }
-        if start:
-            params["start"] = start
-        if end:
-            params["end"] = end
-        if page_token:
-            params["page_token"] = page_token
-
-        url = f"{self.data_base_url}/v1beta1/options/bars"
-        return self._make_request("GET", url, params=params)
-
 
 alpaca_service = AlpacaService()

backend/apps/core/views/main.py

@@ -605,119 +605,6 @@ def candles_v3(self, request, pk=None):
         """
         return get_candles_v3(self, request, pk)
 
-    @staticmethod
-    def _parse_limit(request, default: int, maximum: int) -> int:
-        """Parse and clamp the 'limit' query parameter."""
-        try:
-            return min(int(request.query_params.get("limit", default)), maximum)
-        except (ValueError, TypeError):
-            return default
-
-    @action(detail=True, methods=["get"], url_path="option_chain")
-    def option_chain(self, request, pk=None):
-        """
-        Fetch the options chain (snapshots with quotes and greeks) for an underlying equity asset.
-
-        Query Parameters:
-            expiration_date_gte: Filter options expiring on or after this date (YYYY-MM-DD).
-            expiration_date_lte: Filter options expiring on or before this date (YYYY-MM-DD).
-            type: Option type filter – 'call' or 'put'.
-            limit: Maximum number of contracts to return (default 200, max 1000).
-            page_token: Pagination token from a previous response.
-
-        Returns:
-            {
-                "msg": "Ok",
-                "data": {
-                    "snapshots": { "<symbol>": { "details": {...}, "greeks": {...}, ... } },
-                    "next_page_token": null
-                }
-            }
-        """
-        asset = self.get_object()
-
-        if asset.asset_class != "us_equity":
-            return Response(
-                {"msg": "Options chain is only available for equity assets"},
-                status=status.HTTP_400_BAD_REQUEST,
-            )
-
-        expiration_date_gte = request.query_params.get("expiration_date_gte")
-        expiration_date_lte = request.query_params.get("expiration_date_lte")
-        option_type = request.query_params.get("type")
-        limit = self._parse_limit(request, default=200, maximum=1000)
-        page_token = request.query_params.get("page_token")
-
-        try:
-            data = alpaca_service.get_option_snapshots(
-                underlying_symbol=asset.symbol,
-                expiration_date_gte=expiration_date_gte,
-                expiration_date_lte=expiration_date_lte,
-                option_type=option_type,
-                limit=limit,
-                page_token=page_token,
-            )
-            return Response({"msg": "Ok", "data": data}, status=status.HTTP_200_OK)
-        except Exception as e:
-            logger.error(
-                f"Error fetching option chain for {asset.symbol}: {e}", exc_info=True
-            )
-            return Response(
-                {"msg": "Error fetching option chain"},
-                status=status.HTTP_500_INTERNAL_SERVER_ERROR,
-            )
-
-    @action(detail=False, methods=["get"], url_path="option_bars")
-    def option_bars(self, request):
-        """
-        Fetch historical OHLCV bars for a specific option contract.
-
-        Query Parameters:
-            symbol: Option contract symbol (required), e.g. AAPL240119C00180000.
-            timeframe: Bar timeframe – '1Min', '5Min', '1Hour', '1Day' (default '1Day').
-            start: Start date/time in ISO 8601 format.
-            end: End date/time in ISO 8601 format.
-            limit: Maximum number of bars to return (default 1000, max 10000).
-
-        Returns:
-            {
-                "msg": "Ok",
-                "data": {
-                    "bars": { "<symbol>": [{"t": ..., "o": ..., "h": ..., "l": ..., "c": ..., "v": ...}] },
-                    "next_page_token": null
-                }
-            }
-        """
-        symbol = request.query_params.get("symbol")
-        if not symbol:
-            return Response(
-                {"msg": "symbol parameter is required"},
-                status=status.HTTP_400_BAD_REQUEST,
-            )
-
-        timeframe = request.query_params.get("timeframe", "1Day")
-        start = request.query_params.get("start")
-        end = request.query_params.get("end")
-        limit = self._parse_limit(request, default=1000, maximum=10000)
-
-        try:
-            data = alpaca_service.get_option_bars(
-                symbols=symbol,
-                timeframe=timeframe,
-                start=start,
-                end=end,
-                limit=limit,
-            )
-            return Response({"msg": "Ok", "data": data}, status=status.HTTP_200_OK)
-        except Exception as e:
-            logger.error(
-                f"Error fetching option bars for {symbol}: {e}", exc_info=True
-            )
-            return Response(
-                {"msg": "Error fetching option bars"},
-                status=status.HTTP_500_INTERNAL_SERVER_ERROR,
-            )
-
 
 class WatchListViewSet(PublicReadOnlyMixin, viewsets.ModelViewSet):
     """

frontend/src/features/graphs/components/controls/ChartToolbar.tsx

@@ -42,13 +42,11 @@ import {
   LayoutTemplate,
   Settings2,
   FunctionSquare,
-  TrendingUp,
 } from 'lucide-react';
 import type { SeriesType } from 'lightweight-charts';
 import { timeframeOptions } from '@/lib/constants';
 import { useIndicatorUI } from '../../context';
 import { useRequireAuth } from '@/hooks/useRequireAuth';
-import type { Asset } from '@/types/common-types';
 
 const chartTypeOptions: {
   value: SeriesType;
@@ -99,13 +97,7 @@ const presetOptions = [
   },
 ];
 
-export default function ChartToolbar({
-  asset,
-  onOptionsClick,
-}: {
-  asset?: Asset;
-  onOptionsClick?: () => void;
-}) {
+export default function ChartToolbar() {
   const dispatch = useAppDispatch();
   const timeframe = useAppSelector(selectTimeframe);
   const chartType = useAppSelector(selectChartType);
@@ -267,21 +259,6 @@ export default function ChartToolbar({
             )}
           </Button>
 
-          {/* Options Button (equity only) */}
-          {asset?.asset_class === 'us_equity' && onOptionsClick && (
-            <Button
-              variant="ghost"
-              size="sm"
-              className="h-8 px-2 gap-1.5 text-muted-foreground hover:text-foreground"
-              onClick={onOptionsClick}
-            >
-              <TrendingUp className="w-4 h-4" />
-              <span className="hidden text-xs font-medium sm:inline">
-                Options
-              </span>
-            </Button>
-          )}
-
           {/* Settings Popover */}
           <Popover>
             <PopoverTrigger asChild>