fix: restore ZapierActionTool import and enhance logging in Zapier adapter

- Reintroduced the import of ZapierActionTool in __init__.py for proper accessibility.
- Added logging for error handling in ZapierActionsAdapter to improve debugging.
- Updated ZapierActionTools factory function to include logging for missing API key.

Author: lorenzejay

crewai_tools/__init__.py

@@ -6,7 +6,6 @@
     S3ReaderTool,
     S3WriterTool,
 )
-from .adapters.zapier_adapter import ZapierActionTool
 from .tools import (
     AIMindTool,
     ApifyActorsTool,
@@ -72,3 +71,4 @@
     YoutubeVideoSearchTool,
     ZapierActionTools,
 )
+from .adapters.zapier_adapter import ZapierActionTool

crewai_tools/adapters/zapier_adapter.py

@@ -1,4 +1,5 @@
 import os
+import logging
 from typing import List
 
 import requests
@@ -7,6 +8,8 @@
 
 ACTIONS_URL = "https://actions.zapier.com/api/v2/ai-actions"
 
+logger = logging.getLogger(__name__)
+
 
 class ZapierActionTool(BaseTool):
     """
@@ -37,8 +40,6 @@ def _run(self, **kwargs) -> str:
                 }
             action_params = {"instructions": instructions, "params": formatted_params}
 
-        print(f"Executing action {self.action_id} with payload: {action_params}")
-
         execute_url = f"{ACTIONS_URL}/{self.action_id}/execute/"
         response = requests.request(
             "POST", execute_url, headers=headers, json=action_params
@@ -59,6 +60,7 @@ class ZapierActionsAdapter:
     def __init__(self, api_key: str = None):
         self.api_key = api_key or os.getenv("ZAPIER_API_KEY")
         if not self.api_key:
+            logger.error("Zapier Actions API key is required")
             raise ValueError("Zapier Actions API key is required")
 
     def get_zapier_actions(self):
@@ -74,7 +76,6 @@ def get_zapier_actions(self):
     def tools(self) -> List[BaseTool]:
         """Convert Zapier actions to BaseTool instances"""
         actions_response = self.get_zapier_actions()
-        print("actions_response", actions_response)
         tools = []
 
         for action in actions_response.get("results", []):

crewai_tools/tools/zapier_action_tool/zapier_action_tool.py

@@ -1,15 +1,28 @@
 import os
+import logging
 from typing import List, Optional
 from crewai.tools import BaseTool
 from crewai_tools.adapters.zapier_adapter import ZapierActionsAdapter
 
+logger = logging.getLogger(__name__)
+
 
 def ZapierActionTools(
     zapier_api_key: Optional[str] = None, action_list: Optional[List[str]] = None
 ) -> List[BaseTool]:
+    """Factory function that returns Zapier action tools.
+
+    Args:
+        zapier_api_key: The API key for Zapier.
+        action_list: Optional list of specific tool names to include.
+
+    Returns:
+        A list of Zapier action tools.
+    """
     if zapier_api_key is None:
         zapier_api_key = os.getenv("ZAPIER_API_KEY")
         if zapier_api_key is None:
+            logger.error("ZAPIER_API_KEY is not set")
             raise ValueError("ZAPIER_API_KEY is not set")
     adapter = ZapierActionsAdapter(zapier_api_key)
     all_tools = adapter.tools()