Skip to content

.NET Workflows - Minor updates to samples - #2707

Merged
Chris (crickman) merged 2 commits into
mainfrom
crickman/workfow-samples-minor-fixes
Dec 8, 2025
Merged

.NET Workflows - Minor updates to samples#2707
Chris (crickman) merged 2 commits into
mainfrom
crickman/workfow-samples-minor-fixes

Conversation

@crickman

@crickmanChris (crickman) commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

Motivation and Context

Noticed a mix of minor clean-up and bug-fix issues as part of exploring a feature update.

Description

  • Removed extraneous project references
  • Fixed typo in DeepResearch.yaml workflow
  • Removed duplicate output
  • Removed solution reference to missing workflow
  • Add missing example workflow to solution

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.

@crickmanChris (crickman) self-assigned this Dec 8, 2025
CopilotAI review requested due to automatic review settings December 8, 2025 20:02
@crickmanChris (crickman) added bug Usage: [Issues], Target: all issues (Legacy, prefer issue type: bug) workflows Usage: [Issues, PRs], Target: Workflows samples labels Dec 8, 2025
@markwallace-microsoftMark Wallace (markwallace-microsoft) added the .NET Usage: [Issues, PRs], Target: .Net label Dec 8, 2025
@crickmanChris (crickman) moved this to In Review in Agent FrameworkDec 8, 2025

CopilotAI 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.

Pull request overview

This PR performs minor clean-up and bug fixes across the .NET workflow samples, improving code quality and consistency without changing functionality.

  • Fixed a typo in the DeepResearch workflow YAML configuration
  • Added missing console color resets in WorkflowRunner for better terminal output handling
  • Removed extraneous project references from multiple sample projects
  • Removed duplicate workflow completion notification

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated no comments.

Show a summary per file
FileDescription
workflow-samples/DeepResearch.yamlFixed typo: changed "inputs:" to "input:" to match the correct InvokeAzureAgent schema
dotnet/src/Shared/Workflows/Execution/WorkflowRunner.csAdded Console.ResetColor() calls after setting console colors to properly restore terminal state
dotnet/samples/GettingStarted/Workflows/_Foundational/08_WriterCriticWorkflow/08_WriterCriticWorkflow.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/_Foundational/07_MixedWorkflowAgentsAndExecutors/07_MixedWorkflowAgentsAndExecutors.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/_Foundational/06_SubWorkflows/06_SubWorkflows.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/_Foundational/05_MultiModelService/05_MultiModelService.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/_Foundational/04_AgentWorkflowPatterns/04_AgentWorkflowPatterns.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/_Foundational/03_AgentsInWorkflows/03_AgentsInWorkflows.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/Observability/WorkflowAsAnAgent/WorkflowAsAnAgentObservability.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/Declarative/ExecuteWorkflow/Program.csRemoved duplicate "Done!" notification (already output by WorkflowRunner)
dotnet/samples/GettingStarted/Workflows/ConditionalEdges/03_MultiSelection/03_MultiSelection.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/ConditionalEdges/02_SwitchCase/02_SwitchCase.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/ConditionalEdges/01_EdgeCondition/01_EdgeCondition.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/Concurrent/Concurrent/Concurrent.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/Agents/WorkflowAsAnAgent/WorkflowAsAnAgent.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/samples/GettingStarted/Workflows/Agents/CustomAgentExecutors/CustomAgentExecutors.csprojRemoved unused Microsoft.Agents.AI.AzureAI.Persistent project reference
dotnet/agent-framework-dotnet.slnxFixed incorrect workflow sample file reference (ConfirmInput.yaml → CustomerSupport.yaml) and reordered Client/Server project pairs for consistency

Comment threaddotnet/agent-framework-dotnet.slnx
@crickman
Chris (crickman) added this pull request to the merge queueDec 8, 2025
github-merge-queueBot pushed a commit that referenced this pull request Dec 8, 2025
@github-merge-queue
github-merge-queueBot removed this pull request from the merge queue due to failed status checks Dec 8, 2025
@crickman
Chris (crickman) added this pull request to the merge queueDec 8, 2025
Merged via the queue into main with commit 1e6aedbDec 8, 2025
14 checks passed
@github-project-automationgithub-project-automationBot moved this from In Review to Done in Agent FrameworkDec 8, 2025
Aris Nguyen (arisng) pushed a commit to arisng/agent-framework that referenced this pull request Feb 2, 2026
@moonbox3Evan Mattson (moonbox3) added declarative Usage: [Issues, PRs], Target: declarative agents and workflows and removed declarative-workflow labels Jun 17, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugUsage: [Issues], Target: all issues (Legacy, prefer issue type: bug)declarativeUsage: [Issues, PRs], Target: declarative agents and workflows.NETUsage: [Issues, PRs], Target: .NetworkflowsUsage: [Issues, PRs], Target: Workflows

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants

@crickman@lokitoth@peibekwe@moonbox3@markwallace-microsoft