agno 2.3.4__py3-none-any.whl → 2.3.6__py3-none-any.whl

agno/tools/function.py

@@ -897,7 +897,7 @@ class FunctionCall(BaseModel):
                 return FunctionExecutionResult(status="success", result=cached_result)
 
         # Execute function
-        execution_result = None
+        execution_result: FunctionExecutionResult
         exception_to_raise = None
 
         try:
@@ -939,6 +939,7 @@ class FunctionCall(BaseModel):
             log_debug(f"{e.__class__.__name__}: {e}")
             self.error = str(e)
             exception_to_raise = e
+            execution_result = FunctionExecutionResult(status="failure", error=str(e))
         except Exception as e:
             log_warning(f"Could not run function {self.get_call_str()}")
             log_exception(e)
@@ -948,10 +949,10 @@ class FunctionCall(BaseModel):
         finally:
             self._handle_post_hook()
 
-            if exception_to_raise is not None:
-                raise exception_to_raise
+        if exception_to_raise is not None:
+            raise exception_to_raise
 
-            return execution_result  # type: ignore[return-value]
+        return execution_result
 
     async def _handle_pre_hook_async(self):
         """Handles the async pre-hook for the function call."""
@@ -1117,7 +1118,7 @@ class FunctionCall(BaseModel):
                 return FunctionExecutionResult(status="success", result=cached_result)
 
         # Execute function
-        execution_result = None
+        execution_result: FunctionExecutionResult
         exception_to_raise = None
 
         try:
@@ -1159,6 +1160,7 @@ class FunctionCall(BaseModel):
             log_debug(f"{e.__class__.__name__}: {e}")
             self.error = str(e)
             exception_to_raise = e
+            execution_result = FunctionExecutionResult(status="failure", error=str(e))
         except Exception as e:
             log_warning(f"Could not run function {self.get_call_str()}")
             log_exception(e)
@@ -1171,10 +1173,10 @@ class FunctionCall(BaseModel):
             else:
                 self._handle_post_hook()
 
-            if exception_to_raise is not None:
-                raise exception_to_raise
+        if exception_to_raise is not None:
+            raise exception_to_raise
 
-            return execution_result  # type: ignore[return-value]
+        return execution_result
 
 
 class ToolResult(BaseModel):

agno/tools/google_drive.py

@@ -69,6 +69,7 @@ from pathlib import Path
 from typing import Any, List, Optional, Union
 
 from agno.tools import Toolkit
+from agno.utils.log import log_error
 
 try:
     from google.auth.transport.requests import Request
@@ -202,7 +203,7 @@ class GoogleDriveTools(Toolkit):
             items = results.get("files", [])
             return items
         except Exception as error:
-            print(f"Could not list files: {error}")
+            log_error(f"Could not list files: {error}")
             return []
 
     @authenticate
@@ -238,7 +239,7 @@ class GoogleDriveTools(Toolkit):
             )
             return uploaded_file
         except Exception as error:
-            print(f"Could not upload file '{file_path}': {error}")
+            log_error(f"Could not upload file '{file_path}': {error}")
             return None
 
     @authenticate
@@ -266,5 +267,5 @@ class GoogleDriveTools(Toolkit):
                     print(f"Download progress: {int(status.progress() * 100)}%.")
             return dest_path
         except Exception as error:
-            print(f"Could not download file '{file_id}': {error}")
+            log_error(f"Could not download file '{file_id}': {error}")
             return None

agno/tools/nano_banana.py

@@ -79,7 +79,7 @@ class NanoBananaTools(Toolkit):
 
             response = client.models.generate_content(
                 model=self.model,
-                contents=[prompt],
+                contents=[prompt],  # type: ignore
                 config=cfg,
             )