Fix send_chat_history to send request with empty chat history

[Copilot]

send_chat_history was short-circuiting on empty chat_history_messages, returning success without making the HTTP request. The MCP platform requires the request even with empty chat history.

Changes

• Core service: Removed early return that bypassed HTTP call when list is empty
• Documentation: Updated docstring to reflect that empty lists are valid and will send requests
• Tests: Updated test to verify HTTP request is made with empty chatHistory payload

Before

if len(chat_history_messages) == 0:
    self._logger.warning("Empty message list provided to send_chat_history")
    return OperationResult.success()  # No HTTP request made

After

# Note: Empty chat_history_messages is allowed - we still send the request to MCP platform
# The platform needs to receive the request even with empty chat history

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• il-1234567812345678123456781234567.8.tenant.api.powerplatform.com
  • Triggering command: /home/REDACTED/work/Agent365-python/Agent365-python/.venv/bin/pytest pytest tests/ -v --tb=short -m not integration (dns block)
• il-testtenan.t.tenant.api.powerplatform.com
  • Triggering command: /home/REDACTED/work/Agent365-python/Agent365-python/.venv/bin/pytest pytest tests/ -v --tb=short -m not integration (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

On mcp_tool_server_configuration_service.py, In send_chat_history, if chat_history_messages is empty we immediately short-circuit the call and return success. This is incorrect - we need to allow the call to pass and we need to send a request with empty chat history to MCP platform. Please update this. Make sure to update any tests and if there is any documentation related to this behavior, that will also need to be updated.

---

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

[Copilot]

Pull request overview

Fixes send_chat_history so it no longer short-circuits on an empty chat_history_messages list and instead sends the required MCP platform request (with an empty chatHistory payload).

Changes:

• Removed the early return that previously skipped the HTTP request when chat_history_messages was empty.
• Updated the send_chat_history docstring to document that empty lists are valid and still result in a request.
• Updated unit tests to assert that an HTTP POST is made and that the serialized payload contains chatHistory: [].

Reviewed changes

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

File	Description
tests/tooling/services/test_send_chat_history.py	Updates the empty-list test to verify an HTTP request is made and includes an empty chatHistory array.
libraries/microsoft-agents-a365-tooling/microsoft_agents_a365/tooling/services/mcp_tool_server_configuration_service.py	Removes empty-list short-circuit and updates documentation to reflect the new behavior.