Skip to content

.NET: fix: azure agent renamed to foundry agent - #10098

Merged
Vincent Biret (baywet) merged 1 commit into
mainfrom
fix/rename-azure-agent
Nov 4, 2026
Merged

Vincent Biret (baywet) merged 1 commit into
mainfrom
fix/rename-azure-agent

Conversation

@baywet

@baywet Vincent Biret (baywet) commented Nov 4, 2026 •

Copy link
Copy Markdown
Member

context #9992 (comment)

fixes #9992

@baywet Vincent Biret (baywet) self-assigned this Nov 4, 2026
Copilot AI balanced review requested due to automatic review settings November 4, 2026 15:02
@agent-framework-automation agent-framework-automation Bot added .NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows labels Nov 4, 2026
@github-actions github-actions Bot changed the title fix: azure agent renamed to foundry agent .NET: fix: azure agent renamed to foundry agent Nov 4, 2026

Copilot AI 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.

🟡 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 HttpClient property 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

@github-actions github-actions Bot 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.

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

@github-code-quality

github-code-quality Bot commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Code coverage overview

Languages: C#

C# / code-coverage/dotnet

main 85d73a9 fix/rename-azure-age... a6acd39 +/-
86% 86% 0%
View coverage of the 2 most impacted files.
File main 85d73a9 fix/rename-azure-age... a6acd39 +/-
/home/runner/wo...SessionStore.cs 100% 56% -44%
/home/runner/wo...gentProvider.cs 0% 60% +60%

ℹ️ Calculations represent line coverage. Results may change as calculations are updated.


Updated November 04, 2026 15:31 UTC

Copilot AI 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.

🟢 Approval recommended

No active review issues are reported for the Foundry provider rename and HTTP customization simplification.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

@baywet
Vincent Biret (baywet) added this pull request to the merge queue Nov 4, 2026
Merged via the queue into main with commit 579e4cf Nov 4, 2026
39 checks passed
This was referenced Nov 9, 2026

This branch was previously deployed

2 inactive deployments
github-app-auth — a6acd398 Deployed Nov 4, 2026 by baywet via add_label #27109
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

.NET: Align HTTP client infrastructure and developer experience

3 participants