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

.NET: Forward A2A MessageSendParams.Configuration in the A2A adapter by SergeyMenshykh · Pull Request #7365 · microsoft/agent-framework · GitHub

  1. microsoft
  2. agent-framework

Repository navigation

.NET: Forward A2A MessageSendParams.Configuration in the A2A adapter - #7365

Merged
SergeyMenshykh merged 1 commit into
microsoft:mainfrom
SergeyMenshykh:sergeymenshykh-a2a-config-forwarding
Jul 28, 2026
ConversationCommits (1)ChecksFiles changed
Merged

.NET: Forward A2A MessageSendParams.Configuration in the A2A adapter#7365
SergeyMenshykh merged 1 commit into
microsoft:mainfrom
SergeyMenshykh:sergeymenshykh-a2a-config-forwarding

Conversation

SergeyMenshykh commented Jul 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Motivation & Context

RequestContext.Configuration (SendMessageConfiguration) is never read by the A2A hosting layer, so agent authors cannot see the caller's acceptedOutputModes, historyLength, etc. through AgentRunOptions.

Description & Review Guide

  • What are the major changes? — A2AAgentHandler now stores the whole SendMessageConfiguration in AdditionalProperties["a2a.configuration"] via a shared CreateRunOptions helper used by all three handler paths.
  • What is the impact of these changes? — Agents can read the caller's configuration; AgentRunMode stays authoritative for AllowBackgroundResponses.
  • What do you want reviewers to focus on? — Whether one key for the whole object is the right granularity.

Related Issue

Closes #5869

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.

Forward A2A MessageSendParams.Configuration in the A2A adapter
117fc52 
The A2A hosting layer now forwards the caller-supplied
SendMessageConfiguration from RequestContext.Configuration into
AgentRunOptions.AdditionalProperties under the key
'a2a.configuration'. This covers all three handler paths:
non-streaming, streaming, and task continuation.

The server-configured AgentRunMode remains authoritative for
AllowBackgroundResponses — the caller's ReturnImmediately is
forwarded but does not override the server decision.

Closes microsoft#5869

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 5ee820de-3e34-493b-a19c-1db6bc04871d
Copilot AI review requested due to automatic review settings July 28, 2026 13:29
agent-framework-automation Bot added the .NET Usage: [Issues, PRs], Target: .Net label Jul 28, 2026
Copilot AI reviewed Jul 28, 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

This PR updates the .NET A2A hosting adapter to forward the caller-provided MessageSendParams.Configuration (SendMessageConfiguration) into AgentRunOptions.AdditionalProperties so hosted agents can observe request configuration through the normal RunAsync/RunStreamingAsync options path.

Changes:

  • Refactors A2A option construction into a single CreateRunOptions helper used by non-streaming, streaming, and continuation paths.
  • Forwards the full SendMessageConfiguration object under the "a2a.configuration" key in AgentRunOptions.AdditionalProperties.
  • Adds unit tests verifying configuration forwarding (including in streaming mode) and ensuring server run mode remains authoritative for background execution.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
dotnet/src/Microsoft.Agents.AI.Hosting.A2A/A2AAgentHandler.cs Centralizes AgentRunOptions creation and forwards SendMessageConfiguration via AdditionalProperties.
dotnet/tests/Microsoft.Agents.AI.Hosting.A2A.UnitTests/A2AAgentHandlerTests.cs Adds coverage for configuration forwarding and run-mode authority behavior.

Comment thread dotnet/src/Microsoft.Agents.AI.Hosting.A2A/A2AAgentHandler.cs
github-actions Bot reviewed Jul 28, 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

Automated Code Review

Reviewers: 5 | Confidence: 89% | Result: All clear

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

Automated review by SergeyMenshykh's agents

SergeyMenshykh self-assigned this Jul 28, 2026
SergeyMenshykh marked this pull request as ready for review July 28, 2026 14:23
SergeyMenshykh moved this to In Review in Agent Framework Jul 28, 2026
github-actions Bot reviewed Jul 28, 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

Automated Code Review

Reviewers: 5 | Confidence: 88% | Result: All clear

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

Automated review by SergeyMenshykh's agents

SergeyMenshykh enabled auto-merge July 28, 2026 14:38
SergeyMenshykh added this pull request to the merge queue Jul 28, 2026
Merged via the queue into microsoft:main with commit 2694120 Jul 28, 2026
35 checks passed
github-project-automation Bot moved this from In Review to Done in Agent Framework Jul 28, 2026
github-actions Bot mentioned this pull request Jul 29, 2026
Closed
This was referenced Jul 30, 2026
Closed
Merged
Closed
Closed
This was referenced Sep 17, 2026
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Merged
Closed
Closed
This was referenced Sep 25, 2026
Open
Closed
Open
Open
Closed
Closed
Closed
Closed
Open
Open
This was referenced Oct 2, 2026
Closed
Closed
Open
Merged
Closed
Open
Open
Open

This branch was previously deployed

2 inactive deployments
github-app-auth — 117fc52a Deployed Jul 28, 2026 by SergeyMenshykh via team_check #1535
integration — 117fc52a Deployed Jul 28, 2026 by SergeyMenshykh via dotnet-test (net472, windows-latest, Release, true, integration) #24129
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
Peter Ibekwe (peibekwe) peibekwe approved these changes
westey (westey-m) westey-m approved these changes

Assignees

SergeyMenshykh

Labels

.NET Usage: [Issues, PRs], Target: .Net

Projects

Agent Framework
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feature]: Forward A2A MessageSendParams.Configuration in the A2A adapter

5 participants

Back | FazBrowse Home | New Git URL