Skip to content

.Net: Add conversation state integration for ChatCompletionAgent with tests - #11690

Merged
westey (westey-m) merged 2 commits into
microsoft:feature-conversation-state-extensionsfrom
westey-m:chat-completion-conv-state
Apr 24, 2025
Merged

.Net: Add conversation state integration for ChatCompletionAgent with tests#11690
westey (westey-m) merged 2 commits into
microsoft:feature-conversation-state-extensionsfrom
westey-m:chat-completion-conv-state

Conversation

@westey-m

@westey-m westey (westey-m) commented Apr 23, 2025

Copy link
Copy Markdown
Contributor

Motivation and Context

#10100
#10712

Other agents and actual state components to follow in future PRs.

Description

  • Add code to integrate conversation state with ChatCompletionAgent
  • Add integration tests for the integration

Contribution Checklist

@westey-m
westey (westey-m) requested a review from a team as a code owner April 23, 2025 13:36
@markwallace-microsoft Mark Wallace (markwallace-microsoft) added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel labels Apr 23, 2025
@github-actions github-actions Bot changed the title Add conversation state integration for ChatCompletionAgent with tests .Net: Add conversation state integration for ChatCompletionAgent with tests Apr 23, 2025
rogerbarreto
Roger Barreto (rogerbarreto) approved these changes Apr 23, 2025
View reviewed changes
Comment thread dotnet/src/Agents/Core/ChatCompletionAgent.cs Resolved
Comment thread dotnet/src/IntegrationTests/IntegrationTests.csproj Resolved
Comment thread dotnet/src/Agents/Core/ChatCompletionAgent.cs Resolved
SergeyMenshykh
SergeyMenshykh approved these changes Apr 23, 2025
View reviewed changes
Comment thread dotnet/src/Agents/Core/ChatCompletionAgent.cs Resolved
@westey-m
westey (westey-m) merged commit 50881e7 into microsoft:feature-conversation-state-extensions Apr 24, 2025

This branch was previously deployed

1 inactive deployment
integration — 080c23cd Deployed Apr 23, 2025 by westey-m
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rogerbarreto Roger Barreto (rogerbarreto) rogerbarreto approved these changes
@SergeyMenshykh SergeyMenshykh SergeyMenshykh approved these changes
+1 more reviewer
@shethaadit Adit Sheth (shethaadit) shethaadit approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

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