Update app.py

app.py

@@ -408,16 +408,8 @@ def reasoning_node(state: AgentState) -> AgentState:
 
 
 def tool_node(state: AgentState) -> AgentState:
-    """
-    Node for executing the chosen tool and returning its output.
-    """
-    print(f"DEBUG: Entering tool_node. Iteration: {state['iterations']}")
-
-    # Ensure history exists
-    if "history" not in state or not isinstance(state["history"], list):
-        raise ValueError("Invalid or missing history in state")
+    # ... (previous code)
 
-    # Find the most recent action request in history
     tool_call_dict = None
     for msg in reversed(state["history"]):
         if isinstance(msg, dict) and msg.get("type") == "action_request":
@@ -425,16 +417,22 @@ def tool_node(state: AgentState) -> AgentState:
             break
 
     if not tool_call_dict:
-        # This shouldn't happen if should_continue logic is correct, but for robustness
         print("WARNING: No action_request found in history, skipping tool execution.")
-        return state
+        return state # Or raise a more specific error if this truly shouldn't happen
 
     tool_name = tool_call_dict.get("tool")
     tool_input = tool_call_dict.get("input")
 
-    # Defensive check for missing tool or input
-    if not tool_name or tool_input is None:
-        raise ValueError(f"Tool name '{tool_name}' or input '{tool_input}' missing from action request: {tool_call_dict}")
+    # --- ADD THIS DEBUG PRINT ---
+    print(f"DEBUG: tool_node received action_request: tool='{tool_name}', input='{tool_input[:100]}...'")
+    # --- END DEBUG PRINT ---
+
+    if not tool_name or tool_input is None: # tool_input can be empty string for some tools, but not None
+        print(f"ERROR: Invalid tool call in action_request. Tool name: '{tool_name}', Input: '{tool_input}'")
+        # Instead of raising directly, you might want to send this back to reasoning
+        # Or provide a specific error message as tool output
+        state["history"].append(AIMessage(content=f"[Tool Error] Invalid tool call: Tool name '{tool_name}' or input was empty. LLM needs to provide valid action."))
+        return state
 
     # Look up and invoke the tool from the state's tool list
     available_tools = state.get("tools", [])