Feat: separate tool_call_item and tool_call_output_item in stream events

[gdisk]

Summary

During my analysis of the streaming API (run_streamed) results, I noticed that tool_call_item and tool_call_output_item events are currently being emitted concurrently upon tool call completion (as evidenced in #831). This implementation conflates what should logically be distinct events. The current PR addresses this by properly separating these event triggers to better reflect the actual workflow.

Test plan

The test file is created in tests named test_stream_events.py.Run the test script below to test.

pytest -s test_stream_events.py

The test result is:

======================================================================== test session starts ======================================================================== 
platform win32 -- Python 3.12.10, pytest-8.3.5, pluggy-1.5.0
rootdir: D:\moon\projects\openai-agents-python
configfile: pyproject.toml
plugins: anyio-4.9.0, inline-snapshot-0.22.3, asyncio-0.26.0, mock-3.14.0
asyncio: mode=Mode.AUTO, asyncio_default_fixture_loop_scope=session, asyncio_default_test_loop_scope=function
collected 1 item                                                                                                                                                      

tests\test_stream_events.py === Run starting ===
Agent updated: Joker
-- Message output:
 a_message
-- Tool was called at 1751271106971851300
-- Tool output: success! at 1751271109987313900
-- Message output:
 done
=== Run complete ===
.

========================================================================= 1 passed in 3.07s =========================================================================

Issue number

#831

Checks

• I've added new tests (if relevant)
• I've run make lint and make format
• I've made sure tests pass

[legeRise]

hi i was just trying to get an effect of calling a tool with its name displays with a spinner and then after sometime the output
and
noticed both events are emitting at the same time

so is it fixed Now? can i use it

[gdisk]

hi i was just trying to get an effect of calling a tool with its name displays with a spinner and then after sometime the output and noticed both events are emitting at the same time

so is it fixed Now? can i use it

Yeah, just check out this PR and run the tests to verify.

[legeRise]

hi i was just trying to get an effect of calling a tool with its name displays with a spinner and then after sometime the output and noticed both events are emitting at the same time
so is it fixed Now? can i use it

Yeah, just check out this PR and run the tests to verify.

thank you so much. it works. hope it gets merged soon

[Suneel5]

Waiting for it to get merged soon

[mattmorgis]

@gdisk thank you for this PR 🙏 hoping they merge soon

cc/ @rm-openai in case you haven't seen yet.

[seratch]

Checked this change and it looks good to me. The proposed flow is quite similar to the one we do in Agents SDK TS.

@rm-openai What do you think about this?

[rm-openai]

might be better to instead capture all the events and do a snapshot() to ensure they are in the right order + params

[gdisk]

might be better to instead capture all the events and do a snapshot() to ensure they are in the right order + params

The order of these events is correct, but tool_call_item is triggered immediately after tool_call_output_item (upon tool call completion), not at the start of the tool call.