Skip to content

Stream built-in tool calls from OpenAI, Google, Anthropic and return them on next request (required for OpenAI reasoning) #2877

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Stream built-in tool calls from OpenAI, Google, Anthropic and return them on next request (required for OpenAI reasoning) #2877

wants to merge 18 commits into from
+19,918 −3,481

Conversation

DouweM
Copy link
Collaborator

@DouweM DouweM commented Sep 11, 2025 •
edited
Loading

To do:

  • Store tool name on BuiltinTool classes
  • Consistent BuiltinToolCall tool name, args, metadata; BuiltinToolResult content
  • Google
  • Anthropic
  • Groq
  • AG-UI
  • Stream more events, e.g. OpenAI response.code_interpreter_call_code.delta
  • Add tests for backward compatibility
  • Re-record all relevant VCRs

@DouweM DouweM self-assigned this Sep 11, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 11, 2025 •
edited
Loading

Docs Preview

commit: d50db6e
Preview URL: https://60073ce3-pydantic-ai-previews.pydantic.workers.dev

@DouweM
Update snapshots
71314f2
@DouweM
some coverage
5c5729f
@DouweM
Merge branch 'main' into openai-responses-return-built-in-tools
64bf391 
# Conflicts:
#	pydantic_ai_slim/pydantic_ai/messages.py
#	pydantic_ai_slim/pydantic_ai/models/openai.py
#	tests/models/cassettes/test_model_names/test_known_model_names.yaml
#	tests/models/test_openai_responses.py
@DouweM
Reset test_known_model_names cassette
1a77db8
@DouweM
Remove code execution tests that weren't actually using code execution
b26fe42
@DouweM
Add openai_include_code_execution_results and openai_include_web_sear…
87a86f7 
…ch_results OpenAI Responses model settings
@DouweM
Handle web search builtin tool calls
d141a4d
This was referenced Sep 15, 2025
Open
Open
@DouweM DouweM changed the title Return OpenAI Responses built-in tool calls to API so they work with reasoning Parse and return built-in tool calls from OpenAI, Google, Anthropic Sep 16, 2025
@DouweM DouweM changed the title Parse and return built-in tool calls from OpenAI, Google, Anthropic Stream built-in tool calls from OpenAI, Google, Anthropic and return them on next request Sep 16, 2025
@DouweM DouweM changed the title Stream built-in tool calls from OpenAI, Google, Anthropic and return them on next request Stream built-in tool calls from OpenAI, Google, Anthropic and return them on next request (required for OpenAI reasoning) Sep 17, 2025
@DouweM DouweM mentioned this pull request Sep 17, 2025
Open
@DouweM
Don't join consecutive text parts with double newline, and don't cons…
5defea4 
…ider text parts before built-in tool calls as output
@DouweM DouweM mentioned this pull request Sep 17, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@DouweM DouweM

Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@DouweM