-
Notifications
You must be signed in to change notification settings - Fork 935
.NET: Update to latest Azure.AI.*, OpenAI, and M.E.AI* #2850
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
.NET: Update to latest Azure.AI.*, OpenAI, and M.E.AI* #2850
Conversation
Absorb breaking changes in Responses surface area
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates dependencies to the latest versions of Azure.AI., OpenAI, and Microsoft.Extensions.AI. packages and absorbs breaking changes from the OpenAI Response API surface area. The primary changes involve renaming types and methods to align with the updated SDK naming conventions.
- Renamed
OpenAIResponseClient→ResponsesClient - Renamed
OpenAIResponse→ResponseResult - Renamed
ResponseCreationOptions→CreateResponseOptions - Renamed
GetOpenAIResponseClient()→GetResponsesClient() - Updated namespaces from
OpenAIResponse.IntegrationTeststoResponseResult.IntegrationTests - Removed Azure.AI.Inference package and related code
- Updated package versions for Azure.AI.Projects, Azure.AI.OpenAI, OpenAI, and Microsoft.Extensions.AI packages
Reviewed changes
Copilot reviewed 32 out of 32 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| dotnet/Directory.Packages.props | Updated package versions for Azure.AI., OpenAI, and M.E.AI. dependencies; removed Azure.AI.Inference |
| dotnet/.editorconfig | Added diagnostic suppression for CA2252 (preview features) |
| dotnet/tests/OpenAIResponse.IntegrationTests/*.cs | Updated namespace to ResponseResult.IntegrationTests and string constants |
| dotnet/tests/Microsoft.Agents.AI..UnitTests/.cs | Updated type and method names to match new SDK naming |
| dotnet/tests/AzureAI.IntegrationTests/AIProjectClientFixture.cs | Updated method call to AsResponseResultItem() |
| dotnet/src/Microsoft.Agents.AI.*.cs | Updated type names, method calls, and API usage throughout |
| dotnet/src/Microsoft.Agents.AI.Hosting.OpenAI/Responses/Models/ConversationReference.cs | Changed serialization to always use object format rather than conditional string/object |
| dotnet/samples//.cs | Updated client creation and type references across all samples |
| dotnet/samples/AgentWebChat/AgentWebChat.AgentHost/*.cs | Removed Azure.AI.Inference integration code |
| .github/upgrades/prompts/SemanticKernelToAgentFramework.md | Updated example code with new type names |
...GettingStarted/AgentWithOpenAI/Agent_OpenAI_Step04_CreateFromOpenAIResponseClient/Program.cs
Outdated
Show resolved
Hide resolved
...GettingStarted/AgentWithOpenAI/Agent_OpenAI_Step04_CreateFromOpenAIResponseClient/Program.cs
Show resolved
Hide resolved
dotnet/samples/GettingStarted/AgentWithOpenAI/Agent_OpenAI_Step05_Conversation/Program.cs
Show resolved
Hide resolved
dotnet/src/Microsoft.Agents.AI.Workflows.Declarative.AzureAI/AzureAgentProvider.cs
Show resolved
Hide resolved
dotnet/src/Microsoft.Agents.AI.Workflows.Declarative.AzureAI/AzureAgentProvider.cs
Show resolved
Hide resolved
dotnet/src/Microsoft.Agents.AI.OpenAI/Extensions/AgentRunResponseExtensions.cs
Show resolved
Hide resolved
dotnet/samples/AgentWebChat/AgentWebChat.AgentHost/Utilities/ChatClientExtensions.cs
Show resolved
Hide resolved
…hatClientExtensions.cs
dotnet/samples/AgentWebChat/AgentWebChat.AgentHost/Utilities/ChatClientExtensions.cs
Show resolved
Hide resolved
…hatClientExtensions.cs
dotnet/samples/AgentWebChat/AgentWebChat.AgentHost/Utilities/ChatClientExtensions.cs
Outdated
Show resolved
Hide resolved
…hatClientExtensions.cs
…p04_CreateFromOpenAIResponseClient/Program.cs Co-authored-by: Copilot <[email protected]>
|
Integration tests are failing. This integration test has issues and needs to be skipped. |
Absorb breaking changes in Responses surface area PR.