Fix

app.py

@@ -22,7 +22,14 @@ DEFAULT_API_URL = "https://agents-course-unit4-scoring.hf.space"
 
 @tool
 def serper_search(query: str) -> str:
-    """Enhanced web search using Serper API with better result processing"""
+    """Enhanced web search using Serper API with better result processing.
+    
+    Args:
+        query (str): The search query to be executed.
+        
+    Returns:
+        str: Formatted search results with relevance scoring.
+    """
     try:
         api_key = os.getenv("SERPER_API_KEY")
         if not api_key:
@@ -78,9 +85,17 @@ def serper_search(query: str) -> str:
     except Exception as e:
         return f"Search error: {str(e)}"
 
+
 @tool
 def wikipedia_search(query: str) -> str:
-    """Enhanced Wikipedia search with multiple strategies"""
+    """Enhanced Wikipedia search with multiple strategies.
+    
+    Args:
+        query (str): Wikipedia search query to look up.
+        
+    Returns:
+        str: Comprehensive Wikipedia information.
+    """
     try:
         results = []
         clean_query = query.replace(" ", "_")
@@ -157,9 +172,17 @@ def wikipedia_search(query: str) -> str:
     except Exception as e:
         return f"Wikipedia search error: {str(e)}"
 
+
 @tool
 def youtube_analyzer(url: str) -> str:
-    """Enhanced YouTube video analyzer with transcript extraction"""
+    """Enhanced YouTube video analyzer with transcript extraction.
+    
+    Args:
+        url (str): YouTube video URL to analyze.
+        
+    Returns:
+        str: Comprehensive video analysis.
+    """
     try:
         video_id_match = re.search(r'(?:v=|/|youtu\.be/)([A-Za-z0-9_-]{11})', url)
         if not video_id_match:
@@ -238,9 +261,19 @@ def youtube_analyzer(url: str) -> str:
     except Exception as e:
         return f"YouTube analysis error: {str(e)}"
 
+
 @tool
 def text_processor(text: str, operation: str = "analyze") -> str:
-    """Advanced text processing for various linguistic operations"""
+    """Advanced text processing for various linguistic operations.
+    
+    Args:
+        text (str): Text to process.
+        operation (str, optional): Operation type (reverse, parse, analyze, extract_numbers, decode). 
+                                  Defaults to "analyze".
+        
+    Returns:
+        str: Processed text results.
+    """
     try:
         if operation == "reverse":
             return text[::-1]
@@ -305,9 +338,17 @@ def text_processor(text: str, operation: str = "analyze") -> str:
     except Exception as e:
         return f"Text processing error: {str(e)}"
 
+
 @tool
 def math_solver(problem: str) -> str:
-    """Advanced mathematical problem solver with multiple strategies"""
+    """Advanced mathematical problem solver with multiple strategies.
+    
+    Args:
+        problem (str): Mathematical problem or structure to analyze.
+        
+    Returns:
+        str: Mathematical analysis and solution approach.
+    """
     try:
         problem_lower = problem.lower()
         
@@ -388,9 +429,19 @@ pattern recognition, or formula application"""
     except Exception as e:
         return f"Math solver error: {str(e)}"
 
+
 @tool
 def data_extractor(source: str, target: str, context: str = "") -> str:
-    """Enhanced data extraction with context awareness"""
+    """Enhanced data extraction with context awareness.
+    
+    Args:
+        source (str): Source text/data to extract from.
+        target (str): What to extract from the source.
+        context (str, optional): Additional context for extraction. Defaults to "".
+        
+    Returns:
+        str: Extracted and processed data.
+    """
     try:
         target_lower = target.lower()
         source_lower = source.lower()
@@ -476,9 +527,17 @@ def data_extractor(source: str, target: str, context: str = "") -> str:
     except Exception as e:
         return f"Data extraction error: {str(e)}"
 
+
 @tool
 def web_page_fetcher(url: str) -> str:
-    """Fetch and extract text content from web pages"""
+    """Fetch and extract text content from web pages.
+    
+    Args:
+        url (str): URL to fetch content from.
+        
+    Returns:
+        str: Extracted text content.
+    """
     try:
         headers = {
             'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.124 Safari/537.36'
@@ -508,9 +567,17 @@ def web_page_fetcher(url: str) -> str:
     except Exception as e:
         return f"Web fetch error: {str(e)}"
 
+
 @tool  
 def calculator_tool(expression: str) -> str:
-    """Safe calculator for mathematical expressions"""
+    """Safe calculator for mathematical expressions.
+    
+    Args:
+        expression (str): Mathematical expression to evaluate.
+        
+    Returns:
+        str: Calculation result.
+    """
     try:
         expression = expression.strip()