Skip to content

.NET: Change A2A default session store to NoopAgentSessionStore#6635

Merged
SergeyMenshykh merged 2 commits into
microsoft:mainfrom
SergeyMenshykh:sergeymenshykh-fix-a2a-default-session-store
Jun 23, 2026
Merged

.NET: Change A2A default session store to NoopAgentSessionStore#6635
SergeyMenshykh merged 2 commits into
microsoft:mainfrom
SergeyMenshykh:sergeymenshykh-fix-a2a-default-session-store

Conversation

@SergeyMenshykh

Copy link
Copy Markdown
Contributor

Align the A2A hosting layer default session store with the AG-UI sibling by using \NoopAgentSessionStore, making persistence an explicit opt-in choice.

Update samples to document how to register a persistent session store for multi-turn conversations.

Align the A2A hosting layer default session store with the AG-UI
sibling by using NoopAgentSessionStore, making persistence an explicit
opt-in choice.

Update samples to document how to register a persistent session store
for multi-turn conversations.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 19, 2026 19:17
@moonbox3 moonbox3 added the .NET Usage: [Issues, PRs], Target: .Net label Jun 19, 2026
@github-actions github-actions Bot changed the title Change A2A default session store to NoopAgentSessionStore .NET: Change A2A default session store to NoopAgentSessionStore Jun 19, 2026
@SergeyMenshykh SergeyMenshykh self-assigned this Jun 19, 2026
@SergeyMenshykh SergeyMenshykh moved this to In Review in Agent Framework Jun 19, 2026

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.

Pull request overview

Updates the .NET A2A hosting defaults to use NoopAgentSessionStore (making session persistence an explicit opt-in), and adjusts tests/samples to reflect the new default behavior and recommended production isolation setup.

Changes:

  • Switch A2A’s default AgentSessionStore from in-memory to NoopAgentSessionStore.
  • Update unit test naming/comments to reflect the new default session-store behavior.
  • Update end-to-end samples to document session isolation requirements and how to opt into persistent session storage.

Reviewed changes

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

File Description
dotnet/src/Microsoft.Agents.AI.Hosting.A2A/A2AServerServiceCollectionExtensions.cs Changes the fallback session store to NoopAgentSessionStore when none is registered.
dotnet/tests/Microsoft.Agents.AI.Hosting.A2A.UnitTests/A2AServerServiceCollectionExtensionsTests.cs Updates test naming/comments to describe the new noop session store default.
dotnet/samples/05-end-to-end/AgentWebChat/AgentWebChat.AgentHost/Program.cs Documents production session isolation and the new default non-persistent session behavior.
dotnet/samples/05-end-to-end/A2AClientServer/A2AServer/Program.cs Documents production session isolation and provides an example of explicitly registering a session store.

Comment thread dotnet/samples/05-end-to-end/A2AClientServer/A2AServer/Program.cs

@github-actions 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.

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

Rename test to FallsBackToNoopSessionStoreDefaultAsync to avoid
implying all stores default to noop (task store still uses InMemory).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@SergeyMenshykh SergeyMenshykh force-pushed the sergeymenshykh-fix-a2a-default-session-store branch from 2d7f0f0 to 651993c Compare June 19, 2026 19:33
@SergeyMenshykh SergeyMenshykh marked this pull request as ready for review June 19, 2026 19:51

@github-actions 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.

Automated Code Review

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

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


Automated review by SergeyMenshykh's agents

@SergeyMenshykh SergeyMenshykh enabled auto-merge June 19, 2026 20:14
@SergeyMenshykh SergeyMenshykh disabled auto-merge June 23, 2026 08:55
@SergeyMenshykh SergeyMenshykh added this pull request to the merge queue Jun 23, 2026
Merged via the queue into microsoft:main with commit 88f0b23 Jun 23, 2026
29 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in Agent Framework Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants