-
-
Notifications
You must be signed in to change notification settings - Fork 248
[DRAFT] OPENAI Multiple tool call optimization with structured output support #655
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.
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
base: main
Are you sure you want to change the base?
[DRAFT] OPENAI Multiple tool call optimization with structured output support #655
Conversation
…configurable parallel tool calls and optional toolChoice finalization
…ng handling in ToolCallMap to resolve the latest reasoning item - Renamed `$reasonings` to `$reasoningItems` for clarity. - Added `resolveReasoningItem` method to determine the most recent reasoning item (ID and summary). - Updated `map` logic to utilize resolved reasoning instead of indexing directly.
…Step logic and support for dynamic parallel tool calls configuration - Introduced `finalRequestStep` method to calculate the final step in requests. - Updated `tool_choice` mapping to consider final request step logic. - Enabled `parallel_tool_calls` configuration to support dynamic values from the request.
… mapping in OpenAI Text handler - Simplified `ToolCallMap` usage by directly inlining the mapping logic during `AssistantMessage` creation. - Removed redundant debug logging for tool calls.
…equest # Conflicts: # src/Providers/OpenAI/Handlers/Text.php
…ToolCalls` property and related method from `Text\Request` - Eliminated unused `parallelToolCalls` property and its associated getter method. - Simplified constructor by removing the redundant property declaration.
|
@SolStis86 Anything i can do to help get this completed? also running into this issue Edit: I tried this locally and can confirm it fixed the error for me |
Hi @RhysLees - potentially yeah. I think we need to start fresh from the latest version of Prism as this is forked from an older version. Can we get in touch privately? |
Sure thing, what's the best way to get in touch? |
Description
OPENAI Handler Optimization For Tool Calling
This is just a drafting conceptual PR that enables tool calls with structured output via the Text handler.
Like I say - this is just drafting and requires tests writing etc but am using it locally and it works great