fix: enable sequential tool execution in OpenAI client

Removes premature loop termination after tool execution to allow
sequential tool calls. The model can now see tool results and make
additional tool calls as needed.

Fixes #845

Co-authored-by: Mervin Praison <MervinPraison@users.noreply.github.com>

Author: claude[bot]

src/praisonai-agents/praisonaiagents/llm/openai_client.py

@@ -906,29 +906,7 @@ def chat_completion_with_tools(
                         should_continue = True
                         break
 
-                if not should_continue:
-                    # Get final response after tool calls
-                    if stream:
-                        final_response = self.process_stream_response(
-                            messages=messages,
-                            model=model,
-                            temperature=temperature,
-                            tools=formatted_tools,
-                            start_time=start_time,
-                            console=console,
-                            display_fn=display_fn,
-                            reasoning_steps=reasoning_steps,
-                            **kwargs
-                        )
-                    else:
-                        final_response = self.create_completion(
-                            messages=messages,
-                            model=model,
-                            temperature=temperature,
-                            stream=False,
-                            **kwargs
-                        )
-                    break
+                # Continue the loop to allow more tool calls
 
                 iteration_count += 1
             else:
@@ -1083,29 +1061,7 @@ async def achat_completion_with_tools(
                         should_continue = True
                         break
 
-                if not should_continue:
-                    # Get final response after tool calls
-                    if stream:
-                        final_response = await self.process_stream_response_async(
-                            messages=messages,
-                            model=model,
-                            temperature=temperature,
-                            tools=formatted_tools,
-                            start_time=start_time,
-                            console=console,
-                            display_fn=display_fn,
-                            reasoning_steps=reasoning_steps,
-                            **kwargs
-                        )
-                    else:
-                        final_response = await self.acreate_completion(
-                            messages=messages,
-                            model=model,
-                            temperature=temperature,
-                            stream=False,
-                            **kwargs
-                        )
-                    break
+                # Continue the loop to allow more tool calls
 
                 iteration_count += 1
             else:

test_sequential_fix.py

@@ -0,0 +1,48 @@
+#!/usr/bin/env python3
+"""
+Test script to verify sequential tool execution works correctly with OpenAI client.
+This demonstrates the fix for issue #845.
+"""
+
+from praisonaiagents import Agent
+
+def get_stock_price(company_name: str) -> str:
+    """
+    Get the stock price of a company
+
+    Args:
+        company_name (str): The name of the company
+        
+    Returns:
+        str: The stock price of the company
+    """
+    print(f"[Tool Called] get_stock_price with company_name='{company_name}'")
+    return f"The stock price of {company_name} is 100"
+
+def multiply(a: int, b: int) -> int:
+    """
+    Multiply two numbers
+    """
+    print(f"[Tool Called] multiply with a={a}, b={b}")
+    return a * b
+
+# Test with OpenAI client
+print("Testing sequential tool execution with OpenAI client...")
+print("-" * 60)
+
+agent = Agent(
+    instructions="You are a helpful assistant. You can use the tools provided to you to help the user.",
+    llm="gpt-4o",
+    tools=[get_stock_price, multiply]
+)
+
+result = agent.start("what is the stock price of Google? multiply the Google stock price with 2")
+print("\n" + "=" * 60)
+print("FINAL RESULT:")
+print(result)
+print("=" * 60)
+
+# Expected behavior:
+# 1. The agent should call get_stock_price("Google") and get "100"
+# 2. The agent should then call multiply(100, 2) and get "200"
+# 3. The final response should mention both the stock price and the result of multiplication