Skip to content

.NET: feat: Remove [Experimental] tag from .NET Orchestrations#6164

Merged
lokitoth merged 1 commit into
mainfrom
dev/dotnet_workflow/remove_orchestrations_experimental_tag
May 29, 2026
Merged

.NET: feat: Remove [Experimental] tag from .NET Orchestrations#6164
lokitoth merged 1 commit into
mainfrom
dev/dotnet_workflow/remove_orchestrations_experimental_tag

Conversation

@lokitoth
Copy link
Copy Markdown
Contributor

Description

  • Remove [Experimental] tag on .NET Orchestration Patterns

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • [ ] Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

Copilot AI review requested due to automatic review settings May 28, 2026 20:46
@lokitoth lokitoth self-assigned this May 28, 2026
@lokitoth lokitoth added .NET workflows Related to Workflows in agent-framework labels May 28, 2026
@lokitoth lokitoth changed the title feat: Remove [Experimental] tag from .NET Orchestrations .NET: feat: Remove [Experimental] tag from .NET Orchestrations May 28, 2026
@lokitoth lokitoth moved this to In Review in Agent Framework May 28, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the [Experimental(MAAIW001)] opt-in diagnostic from the .NET workflow orchestration public surface (Handoff and Magentic) and deletes the now-unreferenced internal DiagnosticConstants helper. This effectively promotes these orchestration patterns from "evaluation only" to a stable public API for consumers of Microsoft.Agents.AI.Workflows.

Changes:

  • Strip [Experimental(DiagnosticConstants.ExperimentalFeatureDiagnostic)] from Handoff entry points/internals (AgentWorkflowBuilder.CreateHandoffBuilderWith, HandoffWorkflowBuilder, HandoffWorkflowBuilderCore, HandoffAgentExecutor, HandoffMessagesFilter).
  • Strip the same attribute from Magentic public API/events (MagenticWorkflowBuilder, MagenticProgressLedger, MagenticPlanReviewRequest/Response, MagenticOrchestratorEvent and subclasses).
  • Delete the internal DiagnosticConstants class from HandoffWorkflowBuilder.cs and tidy a couple of trailing-whitespace comment lines.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
dotnet/src/Microsoft.Agents.AI.Workflows/AgentWorkflowBuilder.cs Removes [Experimental] from CreateHandoffBuilderWith.
dotnet/src/Microsoft.Agents.AI.Workflows/HandoffWorkflowBuilder.cs Removes [Experimental] from HandoffWorkflowBuilder/HandoffWorkflowBuilderCore and deletes internal DiagnosticConstants.
dotnet/src/Microsoft.Agents.AI.Workflows/Specialized/HandoffAgentExecutor.cs Removes [Experimental] from internal HandoffAgentExecutor.
dotnet/src/Microsoft.Agents.AI.Workflows/Specialized/HandoffMessagesFilter.cs Removes [Experimental] from class and IsHandoffFunctionName.
dotnet/src/Microsoft.Agents.AI.Workflows/MagenticWorkflowBuilder.cs Removes [Experimental] from MagenticWorkflowBuilder.
dotnet/src/Microsoft.Agents.AI.Workflows/MagenticProgressLedger.cs Removes [Experimental] and trims trailing whitespace in a TODO comment.
dotnet/src/Microsoft.Agents.AI.Workflows/MagenticPlanReviewRequest.cs Removes [Experimental] from the record.
dotnet/src/Microsoft.Agents.AI.Workflows/MagenticPlanReviewResponse.cs Removes [Experimental] from the record.
dotnet/src/Microsoft.Agents.AI.Workflows/Specialized/Magentic/MagenticOrchestrator.cs Removes [Experimental] from MagenticOrchestratorEvent base and three event subclasses.

Comment thread dotnet/src/Microsoft.Agents.AI.Workflows/AgentWorkflowBuilder.cs
Comment thread dotnet/src/Microsoft.Agents.AI.Workflows/MagenticWorkflowBuilder.cs Outdated
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Automated Code Review

Reviewers: 4 | Confidence: 83% | Result: All clear

Reviewed: Correctness, Security Reliability, Test Coverage, Design Approach


Automated review by lokitoth's agents

@lokitoth lokitoth force-pushed the dev/dotnet_workflow/remove_orchestrations_experimental_tag branch from af2010c to d11234c Compare May 28, 2026 20:56
@lokitoth lokitoth force-pushed the dev/dotnet_workflow/remove_orchestrations_experimental_tag branch from d11234c to 7bc76c1 Compare May 28, 2026 21:55
@lokitoth lokitoth force-pushed the dev/dotnet_workflow/remove_orchestrations_experimental_tag branch from 7bc76c1 to 44bc4e6 Compare May 28, 2026 21:56
@lokitoth lokitoth force-pushed the dev/dotnet_workflow/remove_orchestrations_experimental_tag branch from 44bc4e6 to 01fbc82 Compare May 28, 2026 22:11
@lokitoth lokitoth added this pull request to the merge queue May 28, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 28, 2026
@lokitoth lokitoth added this pull request to the merge queue May 28, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 28, 2026
@lokitoth lokitoth added this pull request to the merge queue May 29, 2026
Merged via the queue into main with commit 1fccf16 May 29, 2026
26 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in Agent Framework May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NET workflows Related to Workflows in agent-framework

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants