FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

.NET: Simplify A2A client-server sample by SergeyMenshykh · Pull Request #7891 · microsoft/agent-framework · GitHub

  1. microsoft
  2. agent-framework

Repository navigation

.NET: Simplify A2A client-server sample - #7891

Merged
SergeyMenshykh merged 4 commits into
microsoft:mainfrom
SergeyMenshykh:sergeymenshykh-simplify-a2a-policy-sample
Aug 27, 2026
ConversationCommits (4)ChecksFiles changed
Merged

.NET: Simplify A2A client-server sample#7891
SergeyMenshykh merged 4 commits into
microsoft:mainfrom
SergeyMenshykh:sergeymenshykh-simplify-a2a-policy-sample

Conversation

SergeyMenshykh commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Motivation & Context

The A2A client-server sample currently includes several agents and coordination layers, making the basic setup harder to follow. This streamlines it into a focused policy-agent example.

Description & Review Guide

  • What are the major changes? Keep one policy agent, use Foundry Responses, simplify the client and server, correct the HTTP request, and update setup guidance.
  • What is the impact of these changes? The sample is smaller and quicker to run while retaining agent discovery and both A2A bindings.
  • What do you want reviewers to focus on? The simplified end-to-end flow and instructions.

Related Issue

N/A - no matching open issue was found.

Contribution Checklist

  • The code builds clean without any errors or warnings
  • All unit tests pass, and I have added new tests where possible
  • The PR follows the Contribution Guidelines
  • This PR is linked to an issue and there is no other open PR for this issue (see Related Issue above).
  • This is not a breaking change. If it is a breaking change, add the breaking change label (or add "[BREAKING]" to the title prefix, before or after any language prefix) - a workflow keeps the label and title prefix in sync automatically.

.NET: Simplify A2A client-server sample
5c9e0b1 
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 9bde6d5d-8515-4ce3-abff-1e30a0b25f45
Copilot AI balanced review requested due to automatic review settings August 26, 2026 16:44
SergeyMenshykh self-assigned this Aug 26, 2026
SergeyMenshykh moved this to In Review in Agent Framework Aug 26, 2026
SergeyMenshykh marked this pull request as ready for review August 26, 2026 16:45
Copilot AI reviewed Aug 26, 2026
View reviewed changes

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Pull request overview

Simplifies the A2A sample into a focused Foundry-backed policy-agent flow.

Changes:

  • Replaces the multi-agent setup with one policy agent.
  • Simplifies client discovery, invocation, configuration, and documentation.
  • Updates the JSON-RPC request and retains both A2A bindings.

Reviewed changes

Copilot reviewed 12 out of 18 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
README.md Streamlines end-to-end instructions.
A2AServer/README.md Adds server-specific guidance.
A2AServer/Program.cs Hosts the Foundry policy agent.
A2AServer/PolicyAgentCard.cs Defines policy-agent discovery metadata.
A2AServer/Models/InvoiceQuery.cs Removes invoice mock tooling.
A2AServer/HostAgentFactory.cs Removes multi-provider agent factory.
A2AServer/A2AServer.http Updates policy-agent requests.
A2AServer/A2AServer.csproj Removes unused dependencies.
A2AClient/README.md Documents the simplified client.
A2AClient/Program.cs Directly discovers and invokes the agent.
A2AClient/HostClientAgent.cs Removes coordinating host agent.
A2AClient/A2AClient.csproj Removes obsolete dependencies.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment thread dotnet/samples/05-end-to-end/A2AClientServer/A2AServer/PolicyAgentCard.cs Outdated
Comment thread dotnet/samples/05-end-to-end/A2AClientServer/A2AClient/Program.cs Outdated
.NET: Use top-level statements in A2A client
2e51389 
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 9bde6d5d-8515-4ce3-abff-1e30a0b25f45
.NET: Advertise A2A text MIME types
579e619 
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 9bde6d5d-8515-4ce3-abff-1e30a0b25f45
SergeyMenshykh force-pushed the sergeymenshykh-simplify-a2a-policy-sample branch from 8b9e7ff to 579e619 Compare August 26, 2026 16:59
agent-framework-automation Bot added documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs .NET Usage: [Issues, PRs], Target: .Net labels Aug 26, 2026
SergeyMenshykh mentioned this pull request Aug 26, 2026
Open
3 of 11 tasks
github-actions Bot reviewed Aug 26, 2026
View reviewed changes

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

MAF Automated Review — Iteration 1

Result: Findings reported
Scope: full PR (3 commit(s)): 5c9e0b1b7193, 2e51389ecbcf, 579e619fb06e
Model: gpt-5.6-sol

Overview

The review found 1 verified inline finding(s).

Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
1 verified finding remained after source verification (1 medium) across 1 file. Details are attached to the affected lines below.

Affected areas: dotnet/samples/05-end-to-end/A2AClientServer/A2AClient/README.md

Comment thread dotnet/samples/05-end-to-end/A2AClientServer/A2AClient/README.md
.NET: Clarify A2A endpoint configuration
d87f38f 
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 9bde6d5d-8515-4ce3-abff-1e30a0b25f45
SergeyMenshykh added this pull request to the merge queue Aug 27, 2026
Merged via the queue into microsoft:main with commit 8c4e452 Aug 27, 2026
26 checks passed
This was referenced Sep 28, 2026
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Open
Closed
Closed
Open
Open
Open
Closed
Open
Open
Closed
Closed
Open
Open
Merged
This was referenced Oct 5, 2026
Closed
Open
Closed
Open
Closed
Open
Open
Open

This branch was previously deployed

2 inactive deployments
integration — d87f38fe Deployed Aug 26, 2026 by SergeyMenshykh via dotnet-test (net10.0, ubuntu-latest, Release, true, integration) #25725
github-app-auth — d87f38fe Deployed Aug 26, 2026 by SergeyMenshykh via add_label #21366
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

github-actions[bot] github-actions[bot] left review comments
Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
westey (westey-m) westey-m approved these changes
chetantoshniwal Awaiting requested review from chetantoshniwal
Peter Ibekwe (peibekwe) Awaiting requested review from peibekwe peibekwe is a code owner
Roger Barreto (rogerbarreto) Awaiting requested review from rogerbarreto rogerbarreto is a code owner

Assignees

SergeyMenshykh

Labels

documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs .NET Usage: [Issues, PRs], Target: .Net

Projects

Agent Framework
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL