.NET: Simplify A2A client-server sample - #7891
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9bde6d5d-8515-4ce3-abff-1e30a0b25f45
There was a problem hiding this comment.
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.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9bde6d5d-8515-4ce3-abff-1e30a0b25f45
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9bde6d5d-8515-4ce3-abff-1e30a0b25f45
8b9e7ff to
579e619
Compare
There was a problem hiding this comment.
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
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9bde6d5d-8515-4ce3-abff-1e30a0b25f45
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
Related Issue
N/A - no matching open issue was found.
Contribution Checklist
breaking changelabel (or add "[BREAKING]" to the title prefix, before or after any language prefix) - a workflow keeps the label and title prefix in sync automatically.