Remove Shanghai Silver website scraping

- Removed goldsilver.ai price fetching code
- Removed get_shanghai_silver_price() and _extract_price_after()
- Removed SSILVER handling from get_price()
- Removed SHANGHAISILVER from DISPLAY_NAME_MAP
- Using working commit 4f9e415 as base

Author: buzzkillb

price_service.py

@@ -1,6 +1,7 @@
 """
 Price fetching service using Pyth Network API.
 """
+
 import logging
 import os
 import re
@@ -11,85 +12,37 @@
 logger = logging.getLogger(__name__)
 
 HERMES_API_URL = "https://hermes.pyth.network/api/latest_price_feeds"
-GOLDSILVER_AI_URL = "https://goldsilver.ai/metal-prices/shanghai-silver-price"
 
 
 class PriceService:
     def __init__(self):
         self.feeds = self._load_feeds()
         self.session: Optional[aiohttp.ClientSession] = None
-    
+
     def _load_feeds(self) -> dict:
         """Load feed IDs from environment."""
         feeds_str = os.environ.get(
             "CRYPTO_FEEDS",
             "BTC:0xe62df6c8b4a85fe1a67db44dc12de5db330f7ac66b72dc658afedf0f4a415b43,"
             "ETH:0xff61491a931112ddf1bd8147cd1b641375f79f5825126d665480874634fd0ace,"
-            "SOL:0xef0d8b6fda2ceba41da15d4095d1da392a0d2f8ed0c6c7bc0f4cfac8c280b56d"
+            "SOL:0xef0d8b6fda2ceba41da15d4095d1da392a0d2f8ed0c6c7bc0f4cfac8c280b56d",
         )
-        
+
         feeds = {}
         for pair in feeds_str.split(","):
             if ":" in pair:
                 name, feed_id = pair.split(":", 1)
                 feeds[name.strip().upper()] = feed_id.strip()
-        
+
         logger.info(f"Loaded {len(feeds)} price feeds")
         return feeds
-    
+
     async def _get_session(self) -> aiohttp.ClientSession:
         if self.session is None or self.session.closed:
             timeout = aiohttp.ClientTimeout(total=15)
             self.session = aiohttp.ClientSession(timeout=timeout)
         return self.session
-    
-    async def get_shanghai_silver_price(self) -> Optional[float]:
-        """Fetch Shanghai Silver price from goldsilver.ai."""
-        try:
-            session = await self._get_session()
-            headers = {
-                "User-Agent": "RustyMcPriceface/1.0 (crypto price bot)",
-                "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8",
-                "Accept-Language": "en-US,en;q=0.9",
-            }
-            async with session.get(GOLDSILVER_AI_URL, headers=headers) as resp:
-                if resp.status != 200:
-                    logger.warning(f"goldsilver.ai returned {resp.status}")
-                    return None
-                
-                text = await resp.text()
-                
-                # Extract number after "Shanghai Spot" and "$"
-                shanghai_price = self._extract_price_after(text, "Shanghai Spot")
-                if shanghai_price and shanghai_price > 10:
-                    logger.info(f"Shanghai Silver: ${shanghai_price}")
-                    return shanghai_price
-                
-                logger.warning("Could not extract valid Shanghai price")
-                return None
-                
-        except Exception as e:
-            logger.error(f"Failed to fetch Shanghai Silver: {e}")
-            return None
-    
-    def _extract_price_after(self, html: str, prefix: str) -> Optional[float]:
-        """Extract dollar amount after a prefix."""
-        pos = html.find(prefix)
-        if pos == -1:
-            return None
-        
-        after = html[pos:pos+200]
-        
-        # Find $ followed by number
-        match = re.search(r'\$([0-9,]+\.?[0-9]*)', after)
-        if match:
-            price_str = match.group(1).replace(",", "")
-            try:
-                return float(price_str)
-            except ValueError:
-                return None
-        return None
-    
+
     async def get_yahoo_price(self, ticker: str) -> Optional[float]:
         """Fetch price from Yahoo Finance API."""
         try:
@@ -102,79 +55,75 @@ async def get_yahoo_price(self, ticker: str) -> Optional[float]:
                 if resp.status != 200:
                     logger.warning(f"Yahoo returned {resp.status} for {ticker}")
                     return None
-                
+
                 data = await resp.json()
-                
+
                 # Extract price from Yahoo Finance JSON structure
                 result = data.get("chart", {}).get("result", [])
                 if not result:
                     logger.warning(f"No result from Yahoo for {ticker}")
                     return None
-                
+
                 meta = result[0].get("meta", {})
                 price = meta.get("regularMarketPrice")
-                
+
                 if price:
                     logger.info(f"Yahoo {ticker}: {price}")
                     return float(price)
-                
+
                 logger.warning(f"No price in Yahoo response for {ticker}")
                 return None
-                
+
         except Exception as e:
             logger.error(f"Failed to fetch {ticker} from Yahoo: {e}")
             return None
-    
+
     async def get_price(self, crypto: str) -> Optional[float]:
         """Get price for a single cryptocurrency."""
         crypto = crypto.upper()
-        
-        # Special handling for Shanghai Silver (not in Pyth feeds)
-        if crypto == "SSILVER":
-            return await self.get_shanghai_silver_price()
-        
+
         # Special handling for DXY (Yahoo Finance)
         if crypto == "DXY":
             return await self.get_yahoo_price("DX-Y.NYB")
-        
+
         if crypto not in self.feeds:
             logger.warning(f"No feed ID for {crypto}")
             return None
-        
+
         feed_id = self.feeds[crypto]
         url = f"{HERMES_API_URL}?ids[]={feed_id}"
-        
+
         try:
             session = await self._get_session()
             async with session.get(url) as resp:
                 if resp.status != 200:
                     logger.warning(f"Pyth API returned {resp.status} for {crypto}")
                     return None
-                
+
                 data = await resp.json()
                 if not data or not isinstance(data, list):
                     return None
-                
+
                 price_data = data[0].get("price", {})
                 price_str = price_data.get("price")
                 expo = price_data.get("expo", 0)
-                
+
                 if price_str is None:
                     return None
-                
-                price = int(price_str) * (10 ** expo)
-                
+
+                price = int(price_str) * (10**expo)
+
                 if price <= 0:
                     logger.warning(f"Invalid price {price} for {crypto}")
                     return None
-                
+
                 logger.debug(f"Fetched {crypto} price: ${price}")
                 return float(price)
-                
+
         except Exception as e:
             logger.error(f"Failed to fetch {crypto} price: {e}")
             return None
-    
+
     async def get_all_prices(self) -> dict:
         """Get prices for all configured cryptocurrencies."""
         results = {}
@@ -183,7 +132,7 @@ async def get_all_prices(self) -> dict:
             if price:
                 results[crypto] = price
         return results
-    
+
     async def close(self):
         """Close the HTTP session."""
         if self.session and not self.session.closed: