Update app.py

app.py

@@ -7,8 +7,7 @@ import requests
 import pytz
 import yaml
 
-from typing import Dict, Optional, Union
-from datetime import datetime
+from typing import Dict
 from tools.final_answer import FinalAnswerTool
 
 from Gradio_UI import GradioUI
@@ -34,26 +33,28 @@ FinalAnswerTool – A built-in tool that lets the agent provide a final answer.
 # Tools allow the agent to interact with its environment (e.g., checking time, fetching images, etc.).
 
 # ✅ Nicaragua Weather Tool
+import os
+import requests
+from typing import Dict
+
 @tool
 def get_nicaragua_weather(city: str = "Managua") -> str:
-    """
-    Fetches real-time weather information for cities in Nicaragua using WeatherAPI service.
+    """Gets current weather information for a city in Nicaragua.
 
     Args:
-        city (str): The name of the city in Nicaragua to get weather data for. Must be one of:
-                   'Managua', 'León', 'Granada', 'Matagalpa', or 'Bluefields'. If an invalid
-                   city is provided, defaults to 'Managua'.
+        city: The name of the Nicaraguan city to check weather for.
+              Valid options are: Managua, León, Granada, Matagalpa, and Bluefields.
+              If invalid city provided, defaults to Managua.
 
     Returns:
-        str: A formatted string containing current weather information including:
-             temperature in Celsius, weather description in Spanish, humidity percentage,
-             and wind speed in km/h.
-
-    Raises:
-        RequestException: If there's a network error or API request fails
-        KeyError: If the API response is missing expected data fields
+        str: Current weather conditions including temperature, description, humidity and wind speed.
     """
-    # Configuration
+    # API Configuration
+    API_KEY = os.getenv("WEATHER_API_KEY")
+    if not API_KEY:
+        return "Error: API key is missing. Please add WEATHER_API_KEY in Hugging Face Secrets."
+
+    # Valid cities and their coordinates
     VALID_CITIES = {
         "Managua": "12.1328,-86.2504",
         "León": "12.4375,-86.8833",
@@ -62,31 +63,22 @@ def get_nicaragua_weather(city: str = "Managua") -> str:
         "Bluefields": "12.0137,-83.7647"
     }
 
-    # Validate API key
-    API_KEY = os.getenv("WEATHER_API_KEY")
-    if not API_KEY:
-        return "Error: API key is missing. Please add WEATHER_API_KEY in Hugging Face Secrets."
-
-    # Validate city input (case-insensitive)
-    normalized_input = city.lower().strip()
+    # Validate city (case-insensitive)
     valid_city = next(
-        (c for c in VALID_CITIES.keys() if c.lower() == normalized_input),
+        (c for c in VALID_CITIES if c.lower() == city.lower().strip()),
         "Managua"
     )
-    
-    # Prepare API request
-    base_url = "http://api.weatherapi.com/v1/current.json"
-    url = f"{base_url}?key={API_KEY}&q={VALID_CITIES[valid_city]}&lang=es"
 
     try:
         # Make API request
+        url = f"http://api.weatherapi.com/v1/current.json?key={API_KEY}&q={VALID_CITIES[valid_city]}&lang=es"
         response = requests.get(url, timeout=10)
         response.raise_for_status()
         data = response.json()
 
-        # Extract weather information
+        # Extract weather data
         current = data["current"]
-        weather_info = {
+        weather = {
             "description": current["condition"]["text"],
             "temp": current["temp_c"],
             "humidity": current["humidity"],
@@ -95,18 +87,18 @@ def get_nicaragua_weather(city: str = "Managua") -> str:
 
         # Format response
         prefix = f"Using Managua instead of {city}. " if valid_city != city else ""
-        return (f"{prefix}{weather_info['description']}, "
-               f"{weather_info['temp']}°C, humidity: {weather_info['humidity']}%, "
-               f"wind speed: {weather_info['wind_speed']} km/h.")
+        return (f"{prefix}{weather['description']}, "
+               f"{weather['temp']}°C, humidity: {weather['humidity']}%, "
+               f"wind speed: {weather['wind_speed']} km/h.")
 
     except requests.Timeout:
-        return f"Error: Request timed out while fetching weather data for {city}"
+        return f"Error: Request timed out for {city}"
     except requests.RequestException as e:
-        return f"Error: Network error while fetching weather data for {city}: {str(e)}"
+        return f"Error: Network error for {city}: {str(e)}"
     except KeyError as e:
-        return f"Error: Invalid data received from weather API: {str(e)}"
+        return f"Error: Invalid API response: {str(e)}"
     except Exception as e:
-        return f"Error: Unexpected error while fetching weather data for {city}: {str(e)}"
+        return f"Error: Unexpected error: {str(e)}"
 
 
 # ✅ Nicaragua Biodiversity Facts