Visitar URL original
.NET: [BREAKING] Rename ChatMessageStore to ChatHistoryProvider by westey-m · Pull Request #3375 · microsoft/agent-framework · GitHub
Skip to content

.NET: [BREAKING] Rename ChatMessageStore to ChatHistoryProvider - #3375

Merged
westey (westey-m) merged 10 commits into
microsoft:mainfrom
westey-m:chathistoryprovider-rename
Jan 23, 2026
Merged

.NET: [BREAKING] Rename ChatMessageStore to ChatHistoryProvider#3375
westey (westey-m) merged 10 commits into
microsoft:mainfrom
westey-m:chathistoryprovider-rename

Conversation

@westey-m

@westey-m westey (westey-m) commented Jan 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Motivation and Context

Currently there is confusion about ChatMessageStore. Many people assume based on the name that it is a full CRUD abstraction for storing chat history. The intention though is to just provide ChatHistory to an Agent per run and store the results of the agent.

A ChatHistoryProvider implementation may of course use a full CRUD abstraction for loading and saving messages, and we may considering providing something like that in future.

#1712

Description

  • Rename ChatMessageStore to ChatHistoryProvider

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

Copilot AI review requested due to automatic review settings January 22, 2026 14:36
@markwallace-microsoft Mark Wallace (markwallace-microsoft) added .NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows labels Jan 22, 2026
@github-actions github-actions Bot changed the title Rename ChatMessageStore to ChatHistoryProvider .NET: Rename ChatMessageStore to ChatHistoryProvider Jan 22, 2026
Copilot AI reviewed Jan 22, 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.

Pull request overview

This pull request renames ChatMessageStore to ChatHistoryProvider throughout the codebase to better reflect its purpose. The name ChatMessageStore implied it was a full CRUD abstraction for storing chat history, when the actual intention is to provide chat history to an agent per run and store the results of the agent's execution.

Changes:

  • Renamed the core abstraction class from ChatMessageStore to ChatHistoryProvider
  • Renamed all implementations: InMemoryChatMessageStore → InMemoryChatHistoryProvider, CosmosChatMessageStore → CosmosChatHistoryProvider, WorkflowMessageStore → WorkflowChatHistoryProvider
  • Updated all properties, parameters, variables, and factory methods to use the new naming convention
  • Updated all unit tests, integration tests, and samples to reflect the new naming
  • Updated XML documentation and comments throughout

Reviewed changes

Copilot reviewed 43 out of 43 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
ChatHistoryProvider.cs Core abstraction class renamed with updated documentation
InMemoryChatHistoryProvider.cs In-memory implementation renamed with updated documentation
CosmosChatHistoryProvider.cs Cosmos DB implementation renamed with updated documentation
WorkflowChatHistoryProvider.cs Workflow implementation renamed
ChatClientAgent.cs Updated to use ChatHistoryProvider throughout
ChatClientAgentThread.cs Updated property names and documentation
ChatClientAgentOptions.cs Updated factory property name
Extension files Renamed extension classes and methods
Test files Updated all test classes and methods
Sample files Updated all sample code

Comment thread dotnet/src/Microsoft.Agents.AI.Abstractions/InMemoryAgentThread.cs Resolved
Comment thread dotnet/src/Microsoft.Agents.AI.Abstractions/InMemoryAgentThread.cs Resolved
Comment thread dotnet/tests/Microsoft.Agents.AI.Abstractions.UnitTests/InMemoryAgentThreadTests.cs Resolved
Comment thread dotnet/src/Microsoft.Agents.AI.CosmosNoSql/CosmosDBChatExtensions.cs Resolved
Comment thread dotnet/src/Microsoft.Agents.AI.Abstractions/InMemoryAgentThread.cs Resolved
Comment thread dotnet/src/Microsoft.Agents.AI.Abstractions/InMemoryAgentThread.cs Resolved
Comment thread dotnet/src/Microsoft.Agents.AI.Abstractions/InMemoryAgentThread.cs Resolved
Comment thread dotnet/src/Microsoft.Agents.AI.Abstractions/InMemoryAgentThread.cs Resolved
Comment thread dotnet/samples/GettingStarted/Agents/Agent_Step07_3rdPartyThreadStorage/Program.cs Resolved
rogerbarreto
Roger Barreto (rogerbarreto) reviewed Jan 23, 2026
View reviewed changes
Comment thread dotnet/src/Microsoft.Agents.AI.Abstractions/ChatHistoryProviderExtensions.cs Resolved
This was referenced Mar 9, 2026
Closed
Closed
Open
This was referenced Mar 16, 2026
Closed
Merged
Closed
Open
Closed
Open
Open
Open
Closed
Closed
Closed
This was referenced Apr 6, 2026
Closed
Open
Open
This was referenced May 2, 2026
Closed
Closed
Merged
This was referenced May 9, 2026
Closed
Closed
Closed
Closed
Closed
This was referenced May 18, 2026
Closed
Closed
@dependabot dependabot Bot mentioned this pull request Jun 10, 2026
Closed
@dependabot dependabot Bot mentioned this pull request Jun 24, 2026
Closed
@dependabot dependabot Bot mentioned this pull request Jul 15, 2026
Open

This branch was previously deployed

1 inactive deployment
integration — 5867dc2b Deployed Jan 23, 2026 by westey-m via dotnet-build-and-test (net472, windows-latest, Release, true, integration) #10033
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@rogerbarreto Roger Barreto (rogerbarreto) rogerbarreto approved these changes
@SergeyMenshykh SergeyMenshykh SergeyMenshykh approved these changes
+1 more reviewer
@markwallace-microsoft Mark Wallace (markwallace-microsoft) markwallace-microsoft approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@westey-m @rogerbarreto @SergeyMenshykh @markwallace-microsoft