Visitar URL original
Add AsChatClient for OpenAIResponseClient by stephentoub · Pull Request #6103 · dotnet/extensions · GitHub
Skip to content

Add AsChatClient for OpenAIResponseClient - #6103

Merged
stephentoub merged 2 commits into
dotnet:mainfrom
stephentoub:responsesclient
Mar 15, 2025
Merged

Add AsChatClient for OpenAIResponseClient#6103
stephentoub merged 2 commits into
dotnet:mainfrom
stephentoub:responsesclient

Conversation

@stephentoub

@stephentoub stephentoub commented Mar 13, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

Closes #6090
Closes #6091

cc: @SteveSandersonMS, @eiriktsarpalis, @trrwilson

Microsoft Reviewers: Open in CodeFlow

@stephentoub
stephentoub requested review from a team as code owners March 13, 2025 22:08
@github-actions github-actions Bot added the area-ai Microsoft.Extensions.AI libraries label Mar 13, 2025
RussKie
RussKie approved these changes Mar 13, 2025
View reviewed changes

@RussKie RussKie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✔️ from infra side

Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIModelMapper.ChatCompletion.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.Abstractions/HostedCodeInterpreterTool.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIResponseChatClient.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIResponseChatClient.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIResponseChatClient.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIResponseChatClient.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIResponseChatClient.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIResponseChatClient.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIResponseChatClient.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIResponseChatClient.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread src/Libraries/Microsoft.Extensions.AI.OpenAI/OpenAIResponseChatClient.cs Resolved
SteveSandersonMS
SteveSandersonMS reviewed Mar 14, 2025
View reviewed changes
Comment thread test/Libraries/Microsoft.Extensions.AI.OpenAI.Tests/OpenAIResponseClientTests.cs Resolved
@stephentoub
stephentoub enabled auto-merge (squash) March 15, 2025 01:43
@stephentoub
stephentoub merged commit 3d61789 into dotnet:main Mar 15, 2025
@stephentoub
stephentoub deleted the responsesclient branch March 21, 2025 14:26
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@SteveSandersonMS SteveSandersonMS SteveSandersonMS left review comments
+1 more reviewer
@RussKie RussKie RussKie approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@stephentoub stephentoub

Labels

area-ai Microsoft.Extensions.AI libraries

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add a HostedWebSearchTool Add AsChatClient for OpenAIResponsesClient

3 participants

@stephentoub @SteveSandersonMS @RussKie