Update app.py

app.py

@@ -3,7 +3,9 @@ import datetime
 import requests
 import pytz
 import yaml
-import requests, xml.etree.ElementTree as ET
+import requests
+import xml.etree.ElementTree as ET
+from requests.exceptions import RequestException
 
 
 from tools.final_answer import FinalAnswerTool
@@ -35,34 +37,68 @@ def coin_price_and_news(coin: str) -> str:
         str: A summary of the coin's price, 24h change, and the latest related news.
     """
     
+    
     coin = coin.lower()
+    price_info = "Price data unavailable"
+    news_info = "News data unavailable"
+    
+    # Fetch price data
     try:
-        # Fetch price data from CoinGecko
-        cg = requests.get("https://api.coingecko.com/api/v3/simple/price",
-                          params={"ids": coin, "vs_currencies": "usd", "include_24hr_change": "true"},
-                          timeout=5)
+        cg = requests.get(
+            "https://api.coingecko.com/api/v3/simple/price",
+            params={
+                "ids": coin,
+                "vs_currencies": "usd",
+                "include_24hr_change": "true"
+            },
+            timeout=10
+        )
+        cg.raise_for_status()  # Raise exception for HTTP errors
         data = cg.json()
-        if coin not in data:
-            return f"No data found for {coin}."
-        price = data[coin]["usd"]
-        change = data[coin]["usd_24hr_change"]
-
-        # Fetch Cointelegraph RSS feed and search for news mentioning the coin
-        rss = requests.get("https://cointelegraph.com/rss", timeout=5)
+        
+        if coin in data:
+            price = data[coin].get("usd", "N/A")
+            change = data[coin].get("usd_24h_change")
+            change_str = f"{change:.2f}%" if change is not None else "N/A"
+            price_info = f"{coin.capitalize()} Price: ${price:.2f} USD, Change: {change_str}"
+        else:
+            price_info = f"No price data found for {coin}."
+    except RequestException as e:
+        price_info = f"Error fetching price: {str(e)}"
+    except (ValueError, KeyError) as e:
+        price_info = f"Error processing price data: {str(e)}"
+        
+    # Fetch news
+    try:
+        rss = requests.get("https://cointelegraph.com/rss", timeout=10)
+        rss.raise_for_status()
         root = ET.fromstring(rss.content)
-        news = "No news found."
+        
+        news_items = []
         for item in root.findall(".//item"):
-            text = ((item.find("title").text or "") + " " + (item.find("description").text or "")).lower()
-            if coin in text:
-                title = item.find("title").text or "No title"
-                link = item.find("link").text or "No link"
-                desc = item.find("description").text or "No description"
-                news = f"Title: {title}\nLink: {link}\nDescription: {desc}"
-                break
-
-        return f"{coin.capitalize()} Price: ${price:.2f} USD, Change: {change:.2f}%\n\nNews:\n{news}"
+            title = item.find("title")
+            desc = item.find("description")
+            link = item.find("link")
+            
+            title_text = title.text if title is not None and title.text else "No title"
+            desc_text = desc.text if desc is not None and desc.text else "No description"
+            link_text = link.text if link is not None and link.text else "No link"
+            
+            if coin in title_text.lower() or coin in desc_text.lower():
+                news_items.append(f"Title: {title_text}\nLink: {link_text}\nDescription: {desc_text}")
+                if len(news_items) >= 3:  # Limit to 3 news items
+                    break
+        
+        if news_items:
+            news_info = "\n\n".join(news_items)
+        else:
+            news_info = f"No recent news found for {coin}."
+    except RequestException as e:
+        news_info = f"Error fetching news: {str(e)}"
     except Exception as e:
-        return f"Error: {str(e)}"
+        news_info = f"Error processing news data: {str(e)}"
+    
+    return f"{price_info}\n\nNews:\n{news_info}"