[dotnet-code] Simplify group chat dispatch internals

[Michelle Clayton (michelle-clayton-work)]

Summary

Extracted the GroupChat host's next-agent selection and turn-token dispatch into an unexported dispatchNextAgent helper. This keeps the existing behavior intact while making the Go host structure closer to the .NET GroupChatHost flow, where history update, broadcast, selection, dispatch, and completion are separated into focused steps.

.NET Reference

• dotnet/src/Microsoft.Agents.AI.Workflows/Specialized/GroupChatHost.cs - separates the group chat host turn flow into broadcast and completion helpers around the manager's next-agent selection and dispatch path.

Public API and Behavior

No public Go API changed. No intentional behavior change was made.

Tests

• go test ./workflow/agentworkflow

Notes

Rejected sampled candidates:

• dotnet/src/Microsoft.Agents.AI.Workflows/Specialized/HandoffStartExecutor.cs - no corresponding Go handoff workflow implementation was found in this checkout.
• dotnet/src/Microsoft.Agents.AI.Workflows/Checkpointing/EdgeIdConverter.cs - the EdgeId dictionary-key converter pattern is .NET-specific; the Go workflow JSON paths use different marshal/unmarshal shapes.
• dotnet/src/Microsoft.Agents.AI.Workflows.Declarative/ObjectModel/AddConversationMessageExecutor.cs - no narrow corresponding Go declarative executor surface was found for a safe production-code cleanup.

Open [dotnet-code] PRs checked before editing; none targeted GroupChat.

Generated by .NET-to-Go Code Portability Refactoring Agent · gpt55 · 80.9 AIC · ⌖ 17.6 AIC · ⊞ 23.2K · ◷

Closes #841

[Copilot]

Pull request overview

Refactors the Go groupChatHostExecutor turn flow by extracting next-agent selection and turn-token dispatch into a focused unexported helper, aligning the internal structure more closely with the referenced .NET GroupChatHost flow while preserving behavior.

Changes:

• Extracted next-agent selection + dispatch logic into dispatchNextAgent.
• Simplified handleTurn by delegating dispatch/complete branching to the helper’s result.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[github-actions]

Parity Review — Approved ✅

This PR only extracts the unexported dispatchNextAgent helper inside workflow/agentworkflow/groupchat.go. No exported Go types, functions, methods, options, or observable runtime behavior changed.

The refactor mirrors the structural separation already present in the upstream .NET GroupChatHost.cs (broadcast → select → dispatch → complete), preserving semantic parity rather than introducing any divergence.

Scope verdict: internal implementation change — no public API surface affected.
Parity verdict: aligned with .NET reference; no Python-side equivalent divergence identified.

Generated by Go API Consistency Review Agent for #860 · sonnet46 · 14.3 AIC · ⌖ 7.49 AIC · ⊞ 5.7K · ◷