Add API to send chat history to MCP platform for threat protection by Copilot · Pull Request #105 · microsoft/Agent365-python

Copilot · 2026-01-08T00:28:24Z

Implementation Plan: Chat History API for MCP Platform

Based on the .NET SDK implementation in PR #148.

Changes Required:

Create ChatHistoryMessage model class
Create ChatMessageRequest model class
Create OperationResult and OperationError classes
Add get_chat_history_endpoint() utility function
Add send_chat_history() method to McpToolServerConfigurationService
Update model exports in __init__.py files
Create unit tests for new models and functionality
Add comprehensive unit tests for send_chat_history method
Address PR feedback:
- Updated copyright headers to # Copyright (c) Microsoft Corporation. and # Licensed under the MIT License.
- Removed examples/send_chat_history_example.py
- Updated send_chat_history to use TurnContext parameter instead of conversation_id, message_id, user_message
- Removed auth_token parameter (no authentication required for this endpoint)
- Added type annotation for turn_context parameter as TurnContext
- Removed TYPE_CHECKING conditional import and always import TurnContext
- Added microsoft-agents-hosting-core as a dependency in pyproject.toml
- Ran uv run --frozen ruff format . to format code correctly
- Updated datetime import style for consistency across test files

Summary:

✅ Successfully implemented generic API to send chat history to MCP platform
✅ All tests passing (39 unit tests total: 27 for models + 12 for send_chat_history method)
✅ Code follows repository conventions and passes linting
✅ All PR feedback addressed
✅ TurnContext properly imported with dependency added
✅ Comprehensive test coverage including validation, error handling, and success scenarios
✅ Code properly formatted with ruff
✅ Consistent datetime import style across all test files

Original prompt

This section details on the original issue you should resolve

<issue_title>Implementation of API to send chat history to MCP platform.</issue_title>
<issue_description>Implement the generic API to send chat history to MCP platform. Follow the changes that were made for the .NET version of the SDK: microsoft/Agent365-dotnet#148</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes Implementation of API to send chat history to MCP platform. #104

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

pontemonti · 2026-01-08T01:09:33Z

@copilot -

Update the copyright headers in the added and updated files:

# Copyright (c) Microsoft Corporation.
# Licensed under the MIT License.

Remove examples/send_chat_history_example.py
In send_chat_history, do not include an auth_token parameter. This request does not require authentication.
In send_chat_history, include TurnContext (from Agents SDK) as parameter instead of conversation_id, message_id and user_message. Those values can be retrieved from TurnContext.

…se TurnContext, remove auth Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot · 2026-01-08T01:16:04Z

@copilot -
Update the copyright headers in the added and updated files:

...

All feedback addressed in commit 6c49d71:

✅ Updated copyright headers to # Copyright (c) Microsoft Corporation. and # Licensed under the MIT License. in all added and updated files
✅ Removed examples/send_chat_history_example.py
✅ Removed auth_token parameter from send_chat_history - no authentication headers are now sent
✅ Updated send_chat_history to accept turn_context parameter - extracts conversation_id, message_id, and user_message from turn_context.activity

...a365-tooling/microsoft_agents_a365/tooling/services/mcp_tool_server_configuration_service.py

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

...a365-tooling/microsoft_agents_a365/tooling/services/mcp_tool_server_configuration_service.py

…nts-hosting-core dependency Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

pontemonti · 2026-01-08T03:59:57Z

@copilot - add tests for send_chat_history

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot · 2026-01-08T04:07:26Z

@copilot - add tests for send_chat_history

Added comprehensive unit tests in commit 4443b7d. Created 12 tests covering:

Successful requests
HTTP error responses
Custom options
Parameter validation (turn_context, chat_history_messages, activity, conversation_id, message_id, user_message)
Error handling (ClientError, TimeoutError)
Correct payload generation

All 39 tests (27 existing + 12 new) passing.

Copilot · 2026-01-08T04:35:49Z

@copilot - run uv run --frozen ruff format . to format the code correctly

Code formatted in commit 1b6f196. Ran uv run --frozen ruff format . which reformatted 2 files (service and test file) with proper whitespace and line length adjustments.

Copilot

Pull request overview

This PR implements a generic API to send chat history to the MCP (Model Context Protocol) platform for real-time threat protection, following the pattern established in the .NET SDK implementation (PR #148).

Key Changes:

Added new model classes (ChatHistoryMessage, ChatMessageRequest) for structuring chat history data
Added operation result classes (OperationResult, OperationError) for standardized error handling
Implemented send_chat_history() method in McpToolServerConfigurationService that extracts conversation data from TurnContext and sends it to the MCP platform

Reviewed changes

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

Show a summary per file

File	Description
libraries/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/models/chat_history_message.py	New model class representing a single chat message with validation
libraries/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/models/chat_message_request.py	New model class representing the complete request payload for threat protection
libraries/microsoft-agents-a365-runtime/microsoft_agents_a365/runtime/operation_result.py	New class for representing operation success/failure states
libraries/microsoft-agents-a365-runtime/microsoft_agents_a365/runtime/operation_error.py	New class for wrapping exceptions in operation results
libraries/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/services/mcp_tool_server_configuration_service.py	Added `send_chat_history()` method with comprehensive validation and error handling
libraries/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/utils/utility.py	Added `get_chat_history_endpoint()` utility function for endpoint URL construction
libraries/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/models/init.py	Updated exports to include new model classes
libraries/microsoft-agents-a365-runtime/microsoft_agents_a365/runtime/init.py	Updated exports to include new runtime classes
libraries/microsoft-agents-a365-tooling/pyproject.toml	Added `microsoft-agents-hosting-core` dependency for TurnContext
tests/tooling/models/test_chat_history_message.py	Comprehensive unit tests for ChatHistoryMessage model (7 test cases)
tests/tooling/models/test_chat_message_request.py	Comprehensive unit tests for ChatMessageRequest model (6 test cases)
tests/runtime/test_operation_result.py	Comprehensive unit tests for OperationResult class (9 test cases)
tests/runtime/test_operation_error.py	Comprehensive unit tests for OperationError class (5 test cases)
tests/tooling/services/test_send_chat_history.py	Comprehensive unit tests for send_chat_history method (12 test cases covering success, error handling, and validation)
tests/tooling/init.py	New test package initialization with correct copyright header
tests/tooling/models/init.py	New test package initialization with correct copyright header
tests/tooling/services/init.py	New test package initialization with correct copyright header

tests/tooling/services/test_send_chat_history.py

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

* Fix inconsistent string representation for failed result with no errors Improves the __str__ method of OperationResult to return "Failed" instead of "Failed : " when there are no errors. Also removes the extra space before the colon for consistency. Addresses code review Comment 1. Co-Authored-By: Claude <noreply@anthropic.com> * Return defensive copy of errors list to protect singleton The errors property now returns a copy of the internal errors list to protect the singleton instance returned by success() from accidental modification. Updated docstring to document this behavior. Addresses code review Comment 2. Co-Authored-By: Claude <noreply@anthropic.com> * Use explicit None check for timestamp validation Changed timestamp validation from falsy check (if not self.timestamp) to explicit None check (if self.timestamp is None) for safer and more intentional validation behavior. Updated error message and test accordingly. Addresses code review Comment 3. Co-Authored-By: Claude <noreply@anthropic.com> * Move local imports to top of file Moved OperationError, OperationResult, ChatMessageRequest, and get_chat_history_endpoint imports from inside the send_chat_history method to the top of the file with the other imports. Removed the misleading comment about circular dependencies as there is no cycle in the import graph. Addresses code review Comment 4. Co-Authored-By: Claude <noreply@anthropic.com> * Change endpoint URL log level from INFO to DEBUG Detailed operational information like endpoint URLs should be logged at DEBUG level rather than INFO level. INFO level is reserved for higher-level operation status messages. Addresses code review Comment 5. Co-Authored-By: Claude <noreply@anthropic.com> * Use consistent async test pattern for validation tests Converted validation tests from synchronous methods using asyncio.run() to async methods with @pytest.mark.asyncio decorator for consistency with the other tests in the test suite. Addresses code review Comment 7. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Johan Broberg <johanb@microsoft.com> Co-authored-by: Claude <noreply@anthropic.com>

Add .strip() checks to string validation in ChatHistoryMessage and ChatMessageRequest to reject whitespace-only values (CRM-001, CRM-002). Add missing return type annotations on to_dict() methods (CRM-005, CRM-006). Improve __post_init__ docstrings with validation details (CRM-015). Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

- Add explicit None checks for turn_context and chat_history_messages (CRM-003) - Add return type annotation OperationResult to send_chat_history (CRM-004) - Use explicit None and .strip() checks for field validation (CRM-007) - Add 30 second HTTP timeout to prevent indefinite hangs (CRM-008) - Log only URL path to avoid potential PII exposure (CRM-009) - Use aiohttp.ClientResponseError for consistent error handling (CRM-010) - Enhance docstring with detailed validation requirements (CRM-013) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Extract hardcoded endpoint path to CHAT_HISTORY_ENDPOINT_PATH constant for better maintainability and discoverability (CRM-017). Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Add tests for whitespace-only string validation in ChatHistoryMessage and ChatMessageRequest (CRM-012). Add test for empty chat_history list validation in send_chat_history (CRM-011). Update HTTP error test to match new aiohttp.ClientResponseError format. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix(runtime): implement thread-safe singleton with eager initialization Convert OperationResult.success() singleton from lazy to eager initialization at module level. Python's import lock ensures thread-safe initialization, eliminating the race condition in the previous check-and-create pattern. Addresses CRM-001 from code review. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix(tooling): include error response body in HTTP error log message Add truncated error_text (max 500 chars) to the log message when HTTP errors occur in send_chat_history. This improves debugging by showing the actual error response from the MCP platform. Addresses CRM-002 from code review. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix(tooling): reorder exception handlers for proper timeout handling Move asyncio.TimeoutError handler before aiohttp.ClientError to ensure timeouts are caught correctly. Since aiohttp.ServerTimeoutError inherits from both exceptions, the previous order could misclassify timeouts. Addresses CRM-003 from code review. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor(tooling): add type hints for local variables in send_chat_history Add Optional[str] type annotations for conversation_id, message_id, and user_message variables to improve code clarity and IDE support. Addresses CRM-004 from code review. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * docs(tooling): improve validation error messages in ChatHistoryMessage Update error messages to say "cannot be empty or whitespace-only" instead of just "cannot be empty" for clearer feedback when whitespace validation fails. Addresses CRM-005 from code review. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor(tooling): extract timeout and HTTP status code to constants Add DEFAULT_REQUEST_TIMEOUT_SECONDS and HTTP_STATUS_OK module-level constants to replace magic values. Improves maintainability and makes configuration easier to modify. Addresses CRM-006 from code review. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * docs(runtime): enhance defensive copy docstring in OperationResult.errors Move the defensive copy rationale into a prominent Note section in the docstring to make it more visible to developers. This clarifies why the property returns a copy rather than the internal list. Addresses CRM-007 from code review. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * test(tooling): use Mock(spec=TurnContext) for stricter interface validation Import TurnContext and use it as a spec for the mock fixture. This ensures the mock matches the actual TurnContext interface, catching potential issues if the API changes. Addresses CRM-008 from code review. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * docs(tooling): add usage example to send_chat_history docstring Add an Example section showing how to create ChatHistoryMessage objects and call send_chat_history with proper error handling. This helps developers understand the intended usage pattern. Addresses CRM-009 from code review. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> --------- Co-authored-by: Johan Broberg <johanb@microsoft.com> Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

...s/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/models/chat_history_message.py

libraries/microsoft-agents-a365-runtime/microsoft_agents_a365/runtime/__init__.py

tirthdoshi009 · 2026-01-21T15:50:16Z

🧹 Deployment Cleanup Complete

The Azure Container App deployment for this PR has been cleaned up.

Container App: agent-framework-pr-105
Resource Group: agent365-samples-rg
Status: Deleted

Thank you for contributing! 🎉

Initial plan

30a75cb

Copilot AI assigned Copilot and pontemonti Jan 8, 2026

Copilot started work on behalf of pontemonti January 8, 2026 00:29 View session

Copilot AI and others added 4 commits January 8, 2026 00:35

Add chat history models and send_chat_history method

70b54d2

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Add unit tests for chat history models and operation result classes

f7bf975

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Fix type annotation for chat_history_messages parameter

34a8605

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Add usage example for send_chat_history API

6545d57

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Implement API to send chat history to MCP platform~~ Add API to send chat history to MCP platform for threat protection Jan 8, 2026

Copilot finished work on behalf of pontemonti January 8, 2026 00:46

Copilot AI requested a review from pontemonti January 8, 2026 00:46

Copilot started work on behalf of pontemonti January 8, 2026 01:10 View session

Address PR feedback: update copyright headers, remove example file, u…

6c49d71

…se TurnContext, remove auth Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot finished work on behalf of pontemonti January 8, 2026 01:17

pontemonti reviewed Jan 8, 2026

View reviewed changes

...a365-tooling/microsoft_agents_a365/tooling/services/mcp_tool_server_configuration_service.py Outdated Show resolved Hide resolved

Copilot started work on behalf of pontemonti January 8, 2026 02:16 View session

Add type annotation for turn_context parameter

4660d6c

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot AI requested a review from pontemonti January 8, 2026 02:20

Copilot finished work on behalf of pontemonti January 8, 2026 02:20

pontemonti reviewed Jan 8, 2026

View reviewed changes

...a365-tooling/microsoft_agents_a365/tooling/services/mcp_tool_server_configuration_service.py Outdated Show resolved Hide resolved

Copilot started work on behalf of pontemonti January 8, 2026 02:27 View session

Remove TYPE_CHECKING and always import TurnContext, add microsoft-age…

38206e1

…nts-hosting-core dependency Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot AI requested a review from pontemonti January 8, 2026 02:31

Copilot finished work on behalf of pontemonti January 8, 2026 02:31

Copilot started work on behalf of pontemonti January 8, 2026 04:00 View session

Add comprehensive unit tests for send_chat_history method

4443b7d

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot AI requested a review from pontemonti January 8, 2026 04:36

Copilot finished work on behalf of pontemonti January 8, 2026 04:36

pontemonti marked this pull request as ready for review January 8, 2026 04:41

pontemonti requested a review from a team as a code owner January 8, 2026 04:41

Copilot AI review requested due to automatic review settings January 8, 2026 04:41

Copilot started reviewing on behalf of pontemonti January 8, 2026 04:42 View session

Copilot AI reviewed Jan 8, 2026

View reviewed changes

tests/tooling/services/test_send_chat_history.py Outdated Show resolved Hide resolved

Copilot started work on behalf of pontemonti January 8, 2026 05:00 View session

Use consistent datetime import style in test_send_chat_history.py

a79f8d9

Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

Copilot finished work on behalf of pontemonti January 8, 2026 05:03

pontemonti approved these changes Jan 8, 2026

View reviewed changes

pontemonti enabled auto-merge (squash) January 8, 2026 05:10

pontemonti and others added 6 commits January 17, 2026 13:42

refactor(tooling): extract endpoint path to constant

6f22dde

Extract hardcoded endpoint path to CHAT_HISTORY_ENDPOINT_PATH constant for better maintainability and discoverability (CRM-017). Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

tmlsousa reviewed Jan 19, 2026

View reviewed changes

...s/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/models/chat_history_message.py Outdated Show resolved Hide resolved

tmlsousa previously approved these changes Jan 19, 2026

View reviewed changes

...s/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/models/chat_history_message.py Outdated Show resolved Hide resolved

libraries/microsoft-agents-a365-runtime/microsoft_agents_a365/runtime/__init__.py Show resolved Hide resolved

Johan Broberg added 2 commits January 19, 2026 21:53

Update model classes to use pydantic instead of @dataclass.

971ee1d

chore: add changelog for new chat history API features

1a7c827

pontemonti dismissed tmlsousa’s stale review via 1a7c827 January 20, 2026 06:05

style: format ChatHistoryMessage instantiation for improved readability

c3c77dc

tmlsousa approved these changes Jan 20, 2026

View reviewed changes

rahuldevikar761 approved these changes Jan 21, 2026

View reviewed changes

pontemonti merged commit 62f6f7c into main Jan 21, 2026
8 checks passed

pontemonti deleted the copilot/implement-chat-history-api branch January 21, 2026 05:02

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add API to send chat history to MCP platform for threat protection#105

Add API to send chat history to MCP platform for threat protection#105
pontemonti merged 21 commits intomainfrom
copilot/implement-chat-history-api

Copilot AI commented Jan 8, 2026 •

edited

Loading

Uh oh!

pontemonti commented Jan 8, 2026

Uh oh!

Copilot AI commented Jan 8, 2026

Uh oh!

Uh oh!

Uh oh!

pontemonti commented Jan 8, 2026

Uh oh!

Copilot AI commented Jan 8, 2026

Uh oh!

Copilot AI commented Jan 8, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tirthdoshi009 commented Jan 21, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

Copilot AI commented Jan 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Implementation Plan: Chat History API for MCP Platform

Changes Required:

Summary:

Comments on the Issue (you are @copilot in this section)

Uh oh!

pontemonti commented Jan 8, 2026

Uh oh!

Copilot AI commented Jan 8, 2026

Uh oh!

Uh oh!

Uh oh!

pontemonti commented Jan 8, 2026

Uh oh!

Copilot AI commented Jan 8, 2026

Uh oh!

Copilot AI commented Jan 8, 2026

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!

tirthdoshi009 commented Jan 21, 2026

🧹 Deployment Cleanup Complete

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Copilot AI commented Jan 8, 2026 •

edited

Loading