rollback

Signed-off-by: myan <myan@redhat.com>

fix: resolve lint issues

- Fix import sorting in multiple files
- Remove duplicate ResponseComputerToolCall import
- Organize imports according to ruff rules

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

fix: resolve lint issues

- Fix import sorting in multiple files
- Remove duplicate ResponseComputerToolCall import
- Organize imports according to ruff rules

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

fix: restore correct context parameter for function tool hooks

- Function tools should use tool_context (ToolContext)
- Computer/shell tools use context_wrapper (RunContextWrapper)
- This maintains consistency with original codebase

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

chore: remove VSCode configuration files from PR

- Remove .vscode/ directory and all IDE-specific configuration
- These files should not be included in the repository

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: yanmxa

.vscode/launch.json

@@ -4,7 +4,7 @@
 
 from pydantic import BaseModel
 
-from agents import Agent, AgentHooks, RunContextWrapper, Runner, Tool, function_tool
+from agents import Action, Agent, AgentHooks, RunContextWrapper, Runner, Tool, function_tool
 
 
 class CustomAgentHooks(AgentHooks):
@@ -28,10 +28,10 @@ async def on_handoff(self, context: RunContextWrapper, agent: Agent, source: Age
             f"### ({self.display_name}) {self.event_counter}: Agent {source.name} handed off to {agent.name}"
         )
 
-    async def on_tool_start(self, context: RunContextWrapper, agent: Agent, tool: Tool) -> None:
+    async def on_tool_start(self, context: RunContextWrapper, agent: Agent, action: Action) -> None:
         self.event_counter += 1
         print(
-            f"### ({self.display_name}) {self.event_counter}: Agent {agent.name} started tool {tool.name}"
+            f"### ({self.display_name}) {self.event_counter}: Agent {agent.name} started tool {action.function_tool.name} with arguments {action.tool_call.arguments}"
         )
 
     async def on_tool_end(

.vscode/settings.json

@@ -4,10 +4,9 @@
 
 from pydantic import BaseModel
 
-from agents import Agent, RunContextWrapper, RunHooks, Runner, Tool, Usage, function_tool
+from agents import Action, Agent, RunContextWrapper, RunHooks, Runner, Tool, Usage, function_tool
 from agents.items import ModelResponse, TResponseInputItem
 
-
 class ExampleHooks(RunHooks):
     def __init__(self):
         self.event_counter = 0
@@ -43,10 +42,10 @@ async def on_agent_end(self, context: RunContextWrapper, agent: Agent, output: A
             f"### {self.event_counter}: Agent {agent.name} ended with output {output}. Usage: {self._usage_to_str(context.usage)}"
         )
 
-    async def on_tool_start(self, context: RunContextWrapper, agent: Agent, tool: Tool) -> None:
+    async def on_tool_start(self, context: RunContextWrapper, agent: Agent, action: Action) -> None:
         self.event_counter += 1
         print(
-            f"### {self.event_counter}: Tool {tool.name} started. Usage: {self._usage_to_str(context.usage)}"
+            f"### {self.event_counter}: Tool {action.function_tool.tool.name} started. Usage: {self._usage_to_str(context.usage)}"
         )
 
     async def on_tool_end(

examples/basic/agent_lifecycle_example.py

@@ -65,6 +65,7 @@
     StreamEvent,
 )
 from .tool import (
+    Action,
     CodeInterpreterTool,
     ComputerTool,
     FileSearchTool,
@@ -250,6 +251,7 @@ def enable_verbose_stdout_logging():
     "MCPToolApprovalRequest",
     "MCPToolApprovalFunctionResult",
     "function_tool",
+    "Action",
     "Usage",
     "add_trace_processor",
     "agent_span",

examples/basic/lifecycle_example.py

@@ -78,6 +78,8 @@
     LocalShellTool,
     MCPToolApprovalRequest,
     Tool,
+    ToolRunComputerAction,
+    ToolRunFunction,
 )
 from .tool_context import ToolContext
 from .tracing import (
@@ -126,19 +128,6 @@ class ToolRunHandoff:
     handoff: Handoff
     tool_call: ResponseFunctionToolCall
 
-
-@dataclass
-class ToolRunFunction:
-    tool_call: ResponseFunctionToolCall
-    function_tool: FunctionTool
-
-
-@dataclass
-class ToolRunComputerAction:
-    tool_call: ResponseComputerToolCall
-    computer_tool: ComputerTool
-
-
 @dataclass
 class ToolRunMCPApprovalRequest:
     request_item: McpApprovalRequest
@@ -557,9 +546,9 @@ async def execute_function_tool_calls(
         context_wrapper: RunContextWrapper[TContext],
         config: RunConfig,
     ) -> list[FunctionToolResult]:
-        async def run_single_tool(
-            func_tool: FunctionTool, tool_call: ResponseFunctionToolCall
-        ) -> Any:
+        async def run_single_tool(action: ToolRunFunction) -> Any:
+            func_tool = action.function_tool
+            tool_call = action.tool_call
             with function_span(func_tool.name) as span_fn:
                 tool_context = ToolContext.from_agent_context(
                     context_wrapper,
@@ -570,9 +559,9 @@ async def run_single_tool(
                     span_fn.span_data.input = tool_call.arguments
                 try:
                     _, _, result = await asyncio.gather(
-                        hooks.on_tool_start(tool_context, agent, func_tool),
+                        hooks.on_tool_start(tool_context, agent, action),
                         (
-                            agent.hooks.on_tool_start(tool_context, agent, func_tool)
+                            agent.hooks.on_tool_start(tool_context, agent, action)
                             if agent.hooks
                             else _coro.noop_coroutine()
                         ),
@@ -604,8 +593,7 @@ async def run_single_tool(
 
         tasks = []
         for tool_run in tool_runs:
-            function_tool = tool_run.function_tool
-            tasks.append(run_single_tool(function_tool, tool_run.tool_call))
+            tasks.append(run_single_tool(tool_run))
 
         results = await asyncio.gather(*tasks)
 
@@ -1066,9 +1054,9 @@ async def execute(
         )
 
         _, _, output = await asyncio.gather(
-            hooks.on_tool_start(context_wrapper, agent, action.computer_tool),
+            hooks.on_tool_start(context_wrapper, agent, action),
             (
-                agent.hooks.on_tool_start(context_wrapper, agent, action.computer_tool)
+                agent.hooks.on_tool_start(context_wrapper, agent, action)
                 if agent.hooks
                 else _coro.noop_coroutine()
             ),

src/agents/__init__.py

@@ -5,7 +5,7 @@
 from .agent import Agent, AgentBase
 from .items import ModelResponse, TResponseInputItem
 from .run_context import RunContextWrapper, TContext
-from .tool import Tool
+from .tool import Action, Tool
 
 TAgent = TypeVar("TAgent", bound=AgentBase, default=AgentBase)
 
@@ -59,8 +59,8 @@ async def on_handoff(
     async def on_tool_start(
         self,
         context: RunContextWrapper[TContext],
-        agent: TAgent,
-        tool: Tool,
+        agent: Agent[TContext],
+        action: Action,
     ) -> None:
         """Called concurrently with tool invocation."""
         pass
@@ -110,8 +110,8 @@ async def on_handoff(
     async def on_tool_start(
         self,
         context: RunContextWrapper[TContext],
-        agent: TAgent,
-        tool: Tool,
+        agent: Agent[TContext],
+        action: Action,
     ) -> None:
         """Called concurrently with tool invocation."""
         pass

src/agents/_run_impl.py

@@ -6,6 +6,7 @@
 from dataclasses import dataclass
 from typing import TYPE_CHECKING, Any, Callable, Literal, Union, overload
 
+from openai.types.responses import ResponseFunctionToolCall
 from openai.types.responses.file_search_tool_param import Filters, RankingOptions
 from openai.types.responses.response_computer_tool_call import (
     PendingSafetyCheck,
@@ -294,6 +295,19 @@ def name(self):
 ]
 """A tool that can be used in an agent."""
 
+@dataclass
+class ToolRunFunction:
+    tool_call: ResponseFunctionToolCall
+    function_tool: FunctionTool
+
+
+@dataclass
+class ToolRunComputerAction:
+    tool_call: ResponseComputerToolCall
+    computer_tool: ComputerTool
+
+Action = Union[ToolRunFunction, ToolRunComputerAction]
+"""An action that can be performed by an agent. It contains the tool call and the tool"""
 
 def default_tool_error_function(ctx: RunContextWrapper[Any], error: Exception) -> str:
     """The default tool error function, which just returns a generic error message."""

src/agents/lifecycle.py

@@ -7,7 +7,7 @@
 import pytest
 from typing_extensions import TypedDict
 
-from agents.agent import Agent
+from agents.agent import Action, Agent
 from agents.lifecycle import AgentHooks
 from agents.run import Runner
 from agents.run_context import RunContextWrapper, TContext
@@ -53,7 +53,7 @@ async def on_tool_start(
         self,
         context: RunContextWrapper[TContext],
         agent: Agent[TContext],
-        tool: Tool,
+        action: Action,
     ) -> None:
         self.events["on_tool_start"] += 1
 

src/agents/tool.py

@@ -23,6 +23,7 @@
 )
 
 from agents import (
+    Action,
     Agent,
     AgentHooks,
     AsyncComputer,
@@ -32,9 +33,9 @@
     RunContextWrapper,
     RunHooks,
 )
-from agents._run_impl import ComputerAction, RunImpl, ToolRunComputerAction
+from agents._run_impl import ComputerAction, RunImpl
 from agents.items import ToolCallOutputItem
-from agents.tool import ComputerToolSafetyCheckData
+from agents.tool import ComputerToolSafetyCheckData, ToolRunComputerAction
 
 
 class LoggingComputer(Computer):
@@ -224,9 +225,9 @@ def __init__(self) -> None:
         self.ended: list[tuple[Agent[Any], Any, str]] = []
 
     async def on_tool_start(
-        self, context: RunContextWrapper[Any], agent: Agent[Any], tool: Any
+        self, context: RunContextWrapper[Any], agent: Agent[Any], action: Action,
     ) -> None:
-        self.started.append((agent, tool))
+        self.started.append((agent, action.computer_tool))
 
     async def on_tool_end(
         self, context: RunContextWrapper[Any], agent: Agent[Any], tool: Any, result: str
@@ -243,9 +244,9 @@ def __init__(self) -> None:
         self.ended: list[tuple[Agent[Any], Any, str]] = []
 
     async def on_tool_start(
-        self, context: RunContextWrapper[Any], agent: Agent[Any], tool: Any
+        self, context: RunContextWrapper[Any], agent: Agent[Any], action: Action,
     ) -> None:
-        self.started.append((agent, tool))
+        self.started.append((agent, action.computer_tool))
 
     async def on_tool_end(
         self, context: RunContextWrapper[Any], agent: Agent[Any], tool: Any, result: str