ChannelServiceRoutes improved treatment of handler results

[Rodrigo Brandão (rodrigobr-msft)]

This pull request introduces several improvements and type-safety enhancements to the channel service HTTP route handling, especially around serialization, deserialization, and response typing. The changes ensure better type consistency, clearer error handling, and improved compatibility with different model types.

Type and Serialization Improvements:

• Introduced a generic type variable AgentsModelT to the ChannelServiceRoutes class and updated the deserialize_from_body and serialize_model methods to use this generic type, allowing for more precise type checking and flexibility when working with different agent models. Overloads were added to serialize_model for single models and lists of models. [1] [2] [3] [4]

Response Typing and Consistency:

• Updated the return types of get_activity_members and get_conversation_members methods to return list[dict] instead of dict, reflecting the actual data returned and improving type safety. [1] [2]
• Modified the json_response helper in the aiohttp integration to accept both dict and list[dict], ensuring compatibility with updated route handler return types.

Error Handling and Robustness:

• Improved error handling in delete_conversation_member to return an empty dictionary if no result is found, preventing serialization errors and ensuring consistent response shapes.
• Enhanced the initialization logic of HttpAdapterBase to require either a channel_service_client_factory or a connection_manager, raising a clear error if neither is provided. This ensures proper configuration and prevents runtime issues.

Minor Response Fix:

• Set content_type=None in the accepted() static method of HttpResponse to clarify the response type for 202 responses.

[Copilot]

Pull request overview

This PR refines the hosting HTTP channel service pipeline by making handler result serialization more type-safe and aligning route handler return shapes with what the HTTP layers can reliably serialize, while also tightening adapter initialization requirements.

Changes:

• Added generic/overloaded serialization + generic deserialization to improve type safety in ChannelServiceRoutes.
• Adjusted member endpoints to return list[dict] and delete endpoints to return empty payloads where appropriate.
• Hardened HttpAdapterBase initialization and tweaked HttpResponseFactory.accepted() to avoid content-type issues on empty responses.

Reviewed changes

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

File	Description
libraries/microsoft-agents-hosting-core/microsoft_agents/hosting/core/http/_http_response.py	Adjusts 202 Accepted response metadata for empty responses.
libraries/microsoft-agents-hosting-core/microsoft_agents/hosting/core/http/_http_adapter_base.py	Makes adapter initialization stricter by requiring a client factory or connection manager.
libraries/microsoft-agents-hosting-core/microsoft_agents/hosting/core/http/_channel_service_routes.py	Improves typing/serialization and updates endpoint handler return shapes.
libraries/microsoft-agents-hosting-aiohttp/microsoft_agents/hosting/aiohttp/channel_service_route_table.py	Updates aiohttp JSON response helper typing to support list payloads.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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