Skip to content

[dotnet-port-fixes] Suppress duplicate hosted workflow responses - #808

Merged
George Adams (gdams) merged 1 commit into
mainfrom
copilot/dotnet-port-fixes-hosted-response-dedup-20260724-bcc912f62939d12b
Aug 7, 2026
Merged

[dotnet-port-fixes] Suppress duplicate hosted workflow responses#808
George Adams (gdams) merged 1 commit into
mainfrom
copilot/dotnet-port-fixes-hosted-response-dedup-20260724-bcc912f62939d12b

Conversation

@michelle-clayton-work

Copy link
Copy Markdown
Contributor

Summary

Align workflow/agentworkflow with the .NET hosted-workflow fix from microsoft/agent-framework#7217 (upstream commit bfc73a5b14b40d40a90bf949d601324894d1bc34).

This change:

  • synthesizes stable message IDs for contentful hosted-agent streaming updates when the provider does not supply one
  • suppresses completed *agent.Response messages that were already surfaced as streamed updates when IncludeOutputsInResponse is enabled
  • preserves the hosted response completion event as a raw-only observability update in streaming mode
  • adds end-to-end tests covering duplicate suppression with and without provider-supplied message IDs

Ported .NET PRs

Breaking Changes

No.

Tests and Examples

  • Added workflow tests for hosted-agent duplicate suppression and completion-event observability
  • go test ./workflow/agentworkflow
  • go test ./...
  • No examples changed

Notes

  • Upstream evidence: dotnet/src/Microsoft.Agents.AI.Workflows/WorkflowSession.cs, dotnet/src/Microsoft.Agents.AI.Workflows/Specialized/AIAgentHostExecutor.cs
  • The duplicate suppression is intentionally scoped to completed hosted-agent *agent.Response outputs; existing IncludeOutputsInResponse behavior for non-hosted generic workflow message outputs is unchanged
  • Commit ported from upstream head e90b6de5a72d922ea3b803cf4f194e2f1bae354b inspected during selection

Generated by .NET to Go Fixes and Test Porting Agent · 1.1K AIC · ⌖ 18.6 AIC · ⊞ 21.7K ·

Closes #735

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Note

This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Parity Review: Approved

This PR ports the hosted-workflow duplicate-suppression fix from upstream .NET PR microsoft/agent-framework#7217, aligning with WorkflowSession.cs and AIAgentHostExecutor.cs in dotnet/src/Microsoft.Agents.AI.Workflows/.

Exported API changes: None. All new symbols (generatedHostedMessageState, stampHostedUpdateMessageID, responseUpdateHasContent, streamedMessageKey, recordStreamedMessageID, messageWasStreamed) are unexported. The signatures of NewAgent and AgentConfig are unchanged.

Behavioral parity: The duplicate-suppression semantics match the upstream .NET fix — suppressing completed *agent.Response messages already surfaced as streamed updates when IncludeOutputsInResponse is enabled. Stable message-ID stamping for provider-ID-less updates mirrors the upstream behavior. No Python equivalent exists for this hosted-workflow pattern.

No cross-repo parity issues found. The public-api-change label is not warranted.

Generated by Go API Consistency Review Agent for #808 · sonnet46 · 18.3 AIC · ⌖ 5.56 AIC · ⊞ 5.7K ·

@github-actions github-actions Bot added the parity-approved Go API consistency review found no parity issues label Aug 7, 2026
@gdams
George Adams (gdams) added this pull request to the merge queue Aug 7, 2026
Merged via the queue into main with commit 0a5e0b9 Aug 7, 2026
22 checks passed
@gdams
George Adams (gdams) deleted the copilot/dotnet-port-fixes-hosted-response-dedup-20260724-bcc912f62939d12b branch August 7, 2026 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parity-approved Go API consistency review found no parity issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[dotnet-port-fixes] Suppress duplicate hosted workflow responses

3 participants