Skip to content

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

Open
SergeyMenshykh wants to merge 2 commits into
microsoft:mainfrom
SergeyMenshykh:sergeymenshykh-fix-a2a-default-session-store
Open

.NET: Change A2A default session store to NoopAgentSessionStore#6635
SergeyMenshykh wants to merge 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 Issues related to the .NET codebase 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NET Issues related to the .NET codebase

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

3 participants