fix(openai-agents): accept sync tool handlers

[donei003]

Summary

create_openai_agent_handler() runs await handler(args) for every tool call. When a tool handler is a sync function, the result is a plain value. The await then raises TypeError: object dict can't be used in 'await' expression, and the agent run fails.

This change calls the tool handler and awaits the result only when it is awaitable. The OpenAI messages, Claude messages, and Claude agents handlers already accept sync tool handlers. The OpenAI agent handler now does the same.

Why this appears in offline evaluations only

• Online path: wrap_tool_handlers in tracking.py wraps each tool in an async wrapper. The handler always gets a coroutine function, so the unconditional await works.
• Offline path: EvaluationsModule.run gives the handler the tools from TrajectoryRecorder.wrap. That wrapper keeps a sync tool sync on purpose (see TrajectoryRecorder._record). The unconditional await then fails.

The handler is the correct place for the fix. The tool type is Callable[..., Any], which allows sync tools, and a handler that someone calls directly with sync tools fails in the same way. inspect.isawaitable on the result also covers a sync wrapper that returns a coroutine, which the offline recorder can return.

Testing

• New unit tests in packages/openai-agents/tests/test_handler.py cover a sync tool, an async tool, and a sync wrapper that returns an awaitable. Without the fix, the sync-tool test fails with the TypeError.
• uv run pytest packages/openai-agents: 130 passed.
• ruff check, ruff format --check, and mypy packages/openai-agents/src pass.
• Live check: I ran an offline evaluation with only create_openai_agent_handler(), a sync lookup_order tool, and a {{message_history}} trajectory judge. Before the fix, the row errored with generation did not complete. After the fix, the row passed.

🤖 Generated with Claude Code

---

Note

Overview
Fixes tool dispatch in create_openai_agent_handler() so _build_agent_tools no longer always awaits the registered handler. It now invokes handler(args) and awaits only when inspect.isawaitable is true, matching other provider handlers and fixing TypeError when offline evaluations pass sync tools (e.g. from trajectory recording) instead of async-wrapped online tools.

Adds unit tests covering plain sync handlers, async handlers, and sync functions that return a coroutine.

^{Reviewed by Cursor Bugbot for commit 0ac9231. Bugbot is set up for automated code reviews on this repo. Configure here.}

[devin-ai-integration]

🔍 Devin Review: 1 flag

Not posted on this PR by your GitHub settings — view it in Devin Review. (Configure)