Skip to content

Conversation

@crickman
Copy link
Contributor

@crickman crickman commented Nov 11, 2025

Motivation and Context

"Customer Support" is one of the "hero scenarios" ported from FDL, but it needed certain features from the V2 API exposed.

Description

image

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.

@crickman crickman self-assigned this Nov 11, 2025
Copilot AI review requested due to automatic review settings November 11, 2025 22:54
@crickman crickman added the samples Issue relates to the samples label Nov 11, 2025
@markwallace-microsoft markwallace-microsoft added documentation Improvements or additions to documentation .NET workflows Related to Workflows in agent-framework labels Nov 11, 2025
@crickman crickman changed the base branch from main to feature-foundry-agents November 11, 2025 22:55
Copy link
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

This PR adds a new "CustomerSupport" declarative workflow sample demonstrating structured inputs/outputs and human-in-the-loop patterns for a customer support case scenario. The workflow includes multiple agents that handle service requests, ticketing, routing, support, resolution, and escalation.

Key changes:

  • New CustomerSupport workflow sample with YAML definition and C# implementation
  • Updates to InputArguments sample to align property names with CustomerSupport patterns
  • New ConfirmInput workflow demonstrating user input confirmation
  • Solution file updated to include new CustomerSupport project

Reviewed Changes

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

Show a summary per file
File Description
workflow-samples/CustomerSupport.yaml Defines the workflow structure with agent invocations, conditional logic, and loop handling for customer support scenarios
dotnet/samples/.../CustomerSupport/Program.cs Implements the C# sample that creates agents and executes the CustomerSupport workflow
dotnet/samples/.../CustomerSupport/CustomerSupport.csproj Project configuration for the CustomerSupport sample
dotnet/samples/.../InputArguments/Program.cs Updates structured output schema to match CustomerSupport patterns (IsResolved, NeedsTicket)
dotnet/samples/.../ConfirmInput/ConfirmInput.yaml New workflow demonstrating user input confirmation without agents
dotnet/samples/.../ConfirmInput/ConfirmInput.csproj Project file changes for ConfirmInput sample
dotnet/agent-framework-dotnet.slnx Adds CustomerSupport project to the solution

Copilot AI review requested due to automatic review settings November 11, 2025 23:04
Copilot finished reviewing on behalf of crickman November 13, 2025 16:57
Copy link
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

Copilot reviewed 23 out of 24 changed files in this pull request and generated 4 comments.

Copilot AI review requested due to automatic review settings November 13, 2025 18:17
Copilot finished reviewing on behalf of crickman November 13, 2025 18:19
Copy link
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

Copilot reviewed 24 out of 24 changed files in this pull request and generated 3 comments.

Copilot AI review requested due to automatic review settings November 13, 2025 18:27
Copilot finished reviewing on behalf of crickman November 13, 2025 18:30
Copy link
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

Copilot reviewed 24 out of 24 changed files in this pull request and generated 3 comments.

id: log_ticket
activity: "Created ticket #{Local.TicketParameters.TicketId}"

# Determine which team for which route the ticket.
Copy link

Copilot AI Nov 13, 2025

Choose a reason for hiding this comment

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

Corrected grammar from "for which route" to "to which to route".

Suggested change
# Determine which team for which route the ticket.
# Determine to which team to route the ticket.

Copilot uses AI. Check for mistakes.
/// <param name="projectCredentials">The credentials used to authenticate with the Foundry project. This must be a valid instance of <see cref="TokenCredential"/>.</param>
/// <param name="httpClient">An optional <see cref="HttpClient"/> instance to be used for making HTTP requests. If not provided, a default client will be used.</param>
public sealed class AzureAgentProvider(Uri projectEndpoint, TokenCredential projectCredentials, HttpClient? httpClient = null) : WorkflowAgentProvider
public sealed class AzureAgentProvider(Uri projectEndpoint, TokenCredential projectCredentials) : WorkflowAgentProvider
Copy link

Copilot AI Nov 13, 2025

Choose a reason for hiding this comment

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

Removing the httpClient parameter from the constructor is a breaking change. Consider keeping the parameter with a default value of null to maintain backward compatibility while also supporting the new property-based approach. The current change would break existing code that passes HttpClient through the constructor.

Copilot uses AI. Check for mistakes.
Instructions =
"""
Use your knowledge to work with the user to provide the best possible troubleshooting steps
for issues related to Windows operating system. Utilize the "Attempted Resolutions Steps" as
Copy link

Copilot AI Nov 13, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'Resolutions' to 'Resolution'.

Suggested change
for issues related to Windows operating system. Utilize the "Attempted Resolutions Steps" as
for issues related to Windows operating system. Utilize the "Attempted Resolution Steps" as

Copilot uses AI. Check for mistakes.
Copilot AI review requested due to automatic review settings November 13, 2025 19:07
Copilot finished reviewing on behalf of crickman November 13, 2025 19:09
Copy link
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

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

@crickman crickman merged commit 1d6f53b into feature-foundry-agents Nov 13, 2025
15 checks passed
@crickman crickman deleted the crickman/workflows-declarative-sample-customersupport branch November 13, 2025 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation .NET samples Issue relates to the samples workflows Related to Workflows in agent-framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants