.Net Fix Agent Orchestration: Fix message type #12715

crickman · 2025-07-15T03:20:34Z

Replace inadvertant use of GroupChatOrchestration Result message with the expected Magentic message type.

MagenticOrchestration does not define a handler for GroupChatOrchestration Result message since it was not intended to mix message protocols.

The code builds clean without any errors or warnings
The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
All unit tests pass, and I have added new tests where possible
I didn't break anyone 😄

Fix message type

ee1be7f

crickman requested review from TaoChenOSU and alliscode July 15, 2025 03:20

crickman self-assigned this Jul 15, 2025

crickman requested a review from a team as a code owner July 15, 2025 03:20

crickman added this to Semantic Kernel Jul 15, 2025

crickman added bug Something isn't working agents labels Jul 15, 2025

crickman moved this to Sprint: In Review in Semantic Kernel Jul 15, 2025

moonbox3 added the .NET Issue or Pull requests regarding .NET code label Jul 15, 2025

crickman temporarily deployed to integration July 15, 2025 03:20 — with GitHub Actions Inactive

github-actions bot changed the title ~~.NET Fix Agent Orchestration: Fix message type~~ .Net Fix Agent Orchestration: Fix message type Jul 15, 2025

TaoChenOSU approved these changes Jul 15, 2025

View reviewed changes

crickman enabled auto-merge July 15, 2025 15:21

alliscode approved these changes Jul 15, 2025

View reviewed changes

crickman added this pull request to the merge queue Jul 15, 2025

Merged via the queue into main with commit d16fed4 Jul 15, 2025
25 checks passed

crickman deleted the fix_magentic_message_protocol branch July 15, 2025 16:59

github-project-automation bot moved this from Sprint: In Review to Sprint: Done in Semantic Kernel Jul 15, 2025

Provide feedback