Skip to content

Conversation

stephentoub
Copy link
Member

@stephentoub stephentoub commented Jun 11, 2025

In OpenAIAssistantChatClient, as part of providing ChatOptions.Tools to the service, the only API available to do so is ToolsOverride, which ends up replacing rather than augmenting any tools defined at the assistant level. To work around that, this changes the implementation to retrieve the assistant's tools and send them as part of the override, effectively implementing a merge rather than override.

Microsoft Reviewers: Open in CodeFlow

In OpenAIAssistantChatClient, as part of providing ChatOptions.Tools to the service, the only API available to do so is ToolsOverride, which ends up replacing rather than augmenting any tools defined at the assistant level. To work around that, this changes the implementation to retrieve the assistant's tools and send them as part of the override, effectively implementing a merge rather than override.
@stephentoub stephentoub requested a review from a team as a code owner June 11, 2025 02:38
@github-actions github-actions bot added the area-ai Microsoft.Extensions.AI libraries label Jun 11, 2025
@stephentoub stephentoub merged commit 4bea223 into dotnet:main Jun 12, 2025
7 checks passed
@stephentoub stephentoub deleted the assistanttoolsmerge branch June 12, 2025 15:27
@github-actions github-actions bot locked and limited conversation to collaborators Jul 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-ai Microsoft.Extensions.AI libraries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants