Repository navigation
.NET: fix: azure agent renamed to foundry agent - #10098
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
The solution filter includes solution-folder names instead of project-file paths, invalidating loading and building until the src, UnitTests, and Tests entries are removed.
1 open issue
What changed in this PR
This PR renames the .NET declarative workflow provider from AzureAgentProvider to FoundryAgentProvider, aligning its naming with the Foundry integration.
Changes:
- Renames the provider and updates callers, tests, and related comments.
- Removes the direct
HttpClientproperty and transport override, retaining SDK client-options customization. - Reorganizes the source-and-test solution filter and adds declarative integration tests.
| File | Description |
|---|---|
dotnet/tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/InvokeAgentFailureTest.cs |
Updates provider names in comments only. |
dotnet/tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/FoundryAgentUpdateTestHelpers.cs |
Updates the provider call and XML documentation. |
dotnet/tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/FoundryAgentProviderFailureTest.cs |
Renames the test class and updates documentation. |
dotnet/tests/Microsoft.Agents.AI.Workflows.Declarative.IntegrationTests/Framework/IntegrationTest.cs |
Uses the renamed provider in shared setup. |
dotnet/tests/Microsoft.Agents.AI.Workflows.Declarative.IntegrationTests/FoundryAgentProviderTest.cs |
Renames the test class and provider usage. |
dotnet/src/Shared/Workflows/Execution/WorkflowFactory.cs |
Constructs the renamed provider. |
dotnet/src/Microsoft.Agents.AI.Workflows.Declarative.Foundry/FoundryAgentProvider.cs |
Renames the provider and removes direct HTTP-client customization. |
dotnet/samples/03-workflows/Declarative/ExecuteWorkflow/Program.cs |
Updates the sample to the renamed provider. |
dotnet/samples/03-workflows/Declarative/AotCheckpointing/Program.cs |
Updates the checkpointing sample’s provider usage. |
dotnet/agent-framework-src-and-tests.slnf |
Reorders entries and adds folders and integration-test selection. |
Review effort: Balanced
There was a problem hiding this comment.
MAF Automated Review — Iteration 1
Result: Findings reported
Scope: full PR (2 commit(s)): a6acd398a16b, b9664a97e45f
Model: gpt-5.6-sol
Overview
The provider rename is applied consistently across the implementation, samples, and tests, with focused coverage preserving existing failure-handling behavior. The package is still awaiting its first release under this name, which constrains public API compatibility risk. However, the solution-filter update introduces non-project entries that make the repository's configured default .NET solution unloadable and unbuildable.
Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
1 verified finding remained after source verification (1 high) across 1 file. Details are attached to the affected lines below.
Affected areas: dotnet/agent-framework-src-and-tests.slnf
Code coverage overviewLanguages: C# C# / code-coverage/dotnet
View coverage of the 2 most impacted files.
ℹ️ Calculations represent line coverage. Results may change as calculations are updated. Updated |
b9664a9 to
a6acd39
Compare

context #9992 (comment)
fixes #9992