Initial implementation of API to send chat history to MCP platform. by pontemonti · Pull Request #148 · microsoft/Agent365-dotnet

pontemonti · 2025-12-27T00:42:48Z

No description provided.

Copilot

Pull request overview

This PR implements an API for sending chat history to an MCP (Model Context Protocol) platform endpoint for real-time threat protection analysis. The implementation adds new data models, service methods, and utility functions to support this feature.

Key changes:

Adds SendChatHistoryAsync methods to IMcpToolServerConfigurationService for sending chat history to the MCP platform
Introduces new models (ChatMessageRequest, ChatHistoryMessage) to represent chat history data
Creates utility method GetChatHistoryEndpoint for constructing the chat history endpoint URL
Adds singleton Instance property to Agent365SdkUserAgentConfiguration and uses new keyword in derived classes to provide type-specific instances
Implements GetDefaultHttpClient utility method in Runtime to centralize HttpClient creation with standard configuration

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
src/Tooling/Extensions/SemanticKernel/Agent365SemanticKernelSdkUserAgentConfiguration.cs	Adds `new` keyword to Instance property to hide base class member and return derived type
src/Tooling/Extensions/AzureAIFoundry/Agent365AzureAIFoundrySdkUserAgentConfiguration.cs	Adds `new` keyword to Instance property to hide base class member and return derived type
src/Tooling/Extensions/AgentFramework/Agent365AgentFrameworkSdkUserAgentConfiguration.cs	Adds `new` keyword to Instance property to hide base class member and return derived type
src/Tooling/Core/Utils/Utility.cs	Adds GetChatHistoryEndpoint method to construct endpoint URL for chat history API
src/Tooling/Core/Services/McpToolServerConfigurationService.cs	Implements SendChatHistoryAsync methods to POST chat history to MCP platform; refactors GetMCPServerFromToolingGatewayAsync to use new GetDefaultHttpClient utility
src/Tooling/Core/Services/IMcpToolServerConfigurationService.cs	Adds interface definitions for SendChatHistoryAsync methods (with and without ToolOptions)
src/Tooling/Core/Models/ChatMessageRequest.cs	New model representing the request payload for sending chat messages to threat protection endpoint
src/Tooling/Core/Models/ChatHistoryMessage.cs	New model representing individual messages in chat history with id, role, content, and timestamp
src/Runtime/Core/Utility.cs	Adds GetDefaultHttpClient utility method for creating HttpClient instances with standard timeout and user agent configuration
src/Runtime/Core/Agent365SdkUserAgentConfiguration.cs	Adds static Instance property to provide singleton access to default configuration
src/Tests/Runtime.Tests/Agent365SdkUserAgentConfigurationTests.cs	Adds tests for Instance singleton property behavior and validation
src/Tests/Microsoft.Agents.A365.Tooling.Tests/Utils/UtilityTests.cs	New test file with comprehensive tests for GetChatHistoryEndpoint and other utility methods
src/Tests/Microsoft.Agents.A365.Tooling.Tests/Services/McpToolServerConfigurationServiceTests.cs	New test file with tests for SendChatHistoryAsync parameter validation
src/Tests/Microsoft.Agents.A365.Tooling.Tests/Models/ChatMessageRequestTests.cs	New test file with tests for ChatMessageRequest serialization and constructor behavior
src/Tests/Microsoft.Agents.A365.Tooling.Tests/Models/ChatHistoryMessageTests.cs	New test file with tests for ChatHistoryMessage serialization and property handling
src/Tests/Microsoft.Agents.A365.Tooling.Tests/Microsoft.Agents.A365.Tooling.Tests.csproj	New test project file for Tooling component tests
src/Microsoft.Agents.A365.Sdk.sln	Adds new test project to solution file

rahuldevikar761

If it's not urgent, I want to discuss this offline.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 12 comments.

pontemonti · 2025-12-31T05:07:23Z

If it's not urgent, I want to discuss this offline.

We'll want to get this in ASAP so that we can proceed with the other implementations - there are teams waiting for this.

Copilot · 2025-12-31T05:10:46Z

@pontemonti I've opened a new pull request, #150, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…ryMessageTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated no new comments.

Trying to get this in ASAP, we can discuss any follow-ups later.

Copilot · 2025-12-31T05:16:43Z

@pontemonti I've opened a new pull request, #151, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot

Pull request overview

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

* Initial plan * Wrap HttpResponseMessage in using statement for proper disposal Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

…164) * Initial plan * Fix HttpResponseMessage disposal issues by using lambda factory pattern Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 18 out of 18 changed files in this pull request and generated no new comments.

…ServerConfigurationServiceTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

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

Copilot · 2025-12-31T17:30:29Z

@pontemonti I've opened a new pull request, #165, to work on those changes. Once the pull request is ready, I'll request review from you.

) * Initial plan * Fix HttpResponseMessage disposal race condition in test Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot

Pull request overview

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

Initial implementation of API to send chat history to MCP platform.

b6aa26b

pontemonti marked this pull request as ready for review December 27, 2025 04:53

pontemonti requested a review from a team as a code owner December 27, 2025 04:53

Copilot AI review requested due to automatic review settings December 27, 2025 04:53

Copilot started reviewing on behalf of pontemonti December 27, 2025 04:54 View session

Copilot AI reviewed Dec 27, 2025

View reviewed changes

rahuldevikar761 previously requested changes Dec 27, 2025

View reviewed changes

tmlsousa previously approved these changes Dec 29, 2025

View reviewed changes

Johan Broberg added 5 commits December 30, 2025 20:15

Address PR feedback to clarify documentation.

1db31a8

Address PR feedback about documentation

c2625aa

Address PR feedback related to references to instance fields

6974dd2

Address PR feedback related to documentation

0501b22

Address PR feedback about required vs optional properties

89a4f9c

pontemonti dismissed tmlsousa’s stale review via 89a4f9c December 31, 2025 04:57

Merge branch 'main' into users/johanb/RealTimeThreatProtection

60f28f3

Copilot AI review requested due to automatic review settings December 31, 2025 04:59

Copilot started reviewing on behalf of pontemonti December 31, 2025 04:59 View session

Update src/Tooling/Core/Services/McpToolServerConfigurationService.cs

9ee8b64

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI reviewed Dec 31, 2025

View reviewed changes

Copilot AI mentioned this pull request Dec 31, 2025

Refactor HttpClient to use IHttpClientFactory to prevent socket exhaustion #150

Merged

Update src/Tooling/Core/Utils/Utility.cs

ab38ed1

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings December 31, 2025 05:11

Copilot started reviewing on behalf of pontemonti December 31, 2025 05:12 View session

Update src/Tests/Microsoft.Agents.A365.Tooling.Tests/Models/ChatHisto…

5aca00f

…ryMessageTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI reviewed Dec 31, 2025

View reviewed changes

Copilot AI mentioned this pull request Dec 31, 2025

Add comprehensive test coverage for SendChatHistoryAsync parameter validation #151

Merged

Copilot started reviewing on behalf of pontemonti December 31, 2025 16:40 View session

Copilot AI reviewed Dec 31, 2025

View reviewed changes

Comment thread ...Tests/Microsoft.Agents.A365.Tooling.Tests/Services/McpToolServerConfigurationServiceTests.cs

Copilot AI and others added 2 commits December 31, 2025 08:54

Copilot AI review requested due to automatic review settings December 31, 2025 16:59

Copilot started reviewing on behalf of pontemonti December 31, 2025 16:59 View session

Copilot AI reviewed Dec 31, 2025

View reviewed changes

Update src/Tests/Microsoft.Agents.A365.Tooling.Tests/Services/McpTool…

6e26de6

…ServerConfigurationServiceTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings December 31, 2025 17:05

Copilot started reviewing on behalf of pontemonti December 31, 2025 17:05 View session

Copilot AI reviewed Dec 31, 2025

View reviewed changes

Comment thread ...Tests/Microsoft.Agents.A365.Tooling.Tests/Services/McpToolServerConfigurationServiceTests.cs Outdated

Copilot AI mentioned this pull request Dec 31, 2025

Fix HttpResponseMessage disposal race condition in async mock test #165

Merged

Copilot AI and others added 2 commits December 31, 2025 09:53

Redesign SendChatHistoryAsync to return a result object.

c8b23ba

Copilot AI review requested due to automatic review settings December 31, 2025 22:39

Copilot started reviewing on behalf of pontemonti December 31, 2025 22:39 View session

Copilot AI reviewed Dec 31, 2025

View reviewed changes

Comment thread src/Tooling/Core/Services/McpToolServerConfigurationService.cs

Comment thread src/Tooling/Core/Services/McpToolServerConfigurationService.cs

Comment thread ...Tests/Microsoft.Agents.A365.Tooling.Tests/Services/McpToolServerConfigurationServiceTests.cs

Add CancellationToken argument

98a57a2

joratz approved these changes Jan 5, 2026

View reviewed changes

pontemonti enabled auto-merge (squash) January 6, 2026 19:02

mengyimicro approved these changes Jan 6, 2026

View reviewed changes

pontemonti merged commit 3dfea1a into main Jan 6, 2026
5 checks passed

pontemonti deleted the users/johanb/RealTimeThreatProtection branch January 6, 2026 20:04

pontemonti mentioned this pull request Jan 8, 2026

Implementation of API to send chat history to MCP platform. microsoft/Agent365-python#104

Closed

Copilot AI mentioned this pull request Jan 8, 2026

Add API to send chat history to MCP platform for threat protection microsoft/Agent365-python#105

Merged

9 tasks

pontemonti mentioned this pull request Jan 8, 2026

Implementation of API to send chat history to MCP platform. microsoft/Agent365-nodejs#138

Closed

Copilot AI mentioned this pull request Jan 8, 2026

Add chat history API for MCP platform real-time threat protection microsoft/Agent365-nodejs#139

Merged

25 tasks

This was referenced Jan 14, 2026

Implementation of API to send chat history to MCP platform. teamplannerhackathon/Agent365-python#1

Open

Implementation of API to send chat history to MCP platform. teamplannerhackathon/Agent365-nodejs#2

Open

Conversation

pontemonti commented Dec 27, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rahuldevikar761 left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

pontemonti commented Dec 31, 2025

Uh oh!

Copilot AI commented Dec 31, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI commented Dec 31, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI commented Dec 31, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants