Skip to content
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

feat(openai): support required tool calling #236

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mattmcdev
Copy link
Contributor

This PR just adds a valid mapping for ToolChoice::Any with OpenAI and updates the docs to remove it as a limitation.

@mattmcdev mattmcdev force-pushed the feat-open-ai-tool-choice-any branch from fe34d8d to d791efa Compare March 7, 2025 13:38
@sixlive
Copy link
Contributor

sixlive commented Mar 7, 2025

Can we add a test to validate that this works as expected?

@mattmcdev mattmcdev force-pushed the feat-open-ai-tool-choice-any branch 3 times, most recently from badadb2 to ecb4e98 Compare March 7, 2025 14:11
@mattmcdev
Copy link
Contributor Author

Done

Copy link

kinsta bot commented Mar 8, 2025

Preview deployments for prism ⚡️

Status Branch preview Commit preview
✅ Ready Visit preview Visit preview

Commit: 53b4268e72e6be35a41de2d6713fb9c457337f46

Deployment ID: eb57111b-afb8-42b9-a90f-a37968ad2355

Static site name: prism-97nz9

Copy link

kinsta bot commented Mar 8, 2025

Preview deployments for prism ⚡️

Status Branch preview Commit preview
❌ Failed to deploy N/A N/A

Commit: 04a0aff955203c9bfba5f6b15db4e1a242355f05

Deployment ID: fb34d344-5f3b-409f-85e3-056bd7c42845

Static site name: prism-97nz9

@mattmcdev mattmcdev force-pushed the feat-open-ai-tool-choice-any branch 2 times, most recently from 3200dd0 to 9c68564 Compare March 11, 2025 11:54
@mattmcdev mattmcdev force-pushed the feat-open-ai-tool-choice-any branch from 9c68564 to a3e912a Compare March 12, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants