[Kirill (llirik0)]

Summary

Prevent response-only metadata and provider lifecycle updates from creating or modifying messages when collected into a Response.

• Response.Update treats raw-only provider payloads as response metadata, not as evidence of a message. Genuine message updates retain their raw payloads.
• Agent.setAuthor decorates only message-bearing updates. OpenAI, AG-UI, and A2A lifecycle paths avoid introducing message discriminators for response-only events.
• OpenAI lifecycle updates clear their message ID and inherited role without resetting content-stream state. Regressions cover incomplete and completed-without-usage events arriving after a real message, including timestamp and raw-payload isolation.
• ToUpdates preserves response metadata when no messages exist. Its assistant-role fallback is limited to otherwise unidentifiable metadata-only messages; it is justified by the collector's existing assistant default, not a claim of identical .NET serialization behavior.

Rebase and #1133

Rebased onto main at 1ac53ce01a1c3a715f9fa318c2b9564fbfedfbf2, resolving conflicts while retaining upstream ConversationID propagation and MessageID-based AdditionalProperties scoping. #1133 remains open; this update preserves the scoping already present on main rather than introducing a competing rule.

Validation

On the updated candidate:

• go test ./...
• go test -race ./...
• go vet ./...
• golangci-lint run
• Catalog: golangci-lint run --config ../.golangci.yml
• git diff --check

All passed. The new post-message lifecycle regressions failed before the role-clearing fix and passed afterward.

[Kirill (llirik0)]

Kirill (Kirill (@llirik0)) please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

• (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

• (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

[Copilot]

Copilot review overview

🟡 Changes recommended

Message-only metadata can be lost during response round trips.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1

Open (1)

• Preserve metadata-only messages during response update conversion · New

What changed in this PR

Prevents response-only metadata updates from creating empty assistant messages during response round trips.

Changes:

• Gates message mutation on message-related update fields.
• Adds metadata round-trip tests.

File	Description
agent/​response.go	Updates response aggregation behavior.
agent/​response_test.go	Tests metadata-only round trips.

---

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

[github-actions]

Scope: user-visible behavior

Changed Go contract: agent.Response.Update (skips creating/mutating a message when a ResponseUpdate carries no message-related data — no MessageID, AuthorName, Role, Contents, or RawRepresentation) and agent.Response.ToUpdates (defaults an empty per-message Role to message.RoleAssistant when converting to updates). No exported types/signatures were added, removed, or changed — same public methods, same shapes.

Upstream evidence reviewed:

• Python python/packages/core/agent_framework/_types.py, _process_update (~line 2038): creates a new Message("assistant", []) whenever response.messages is empty or the update signals a new message boundary, without checking whether the update actually carries message data.
• Python _finalize_response (~line 2278) → _coalesce_function_call_occurrences (~line 2287): after all updates are applied, filters response.messages to [m for m in response.messages if m.contents], discarding any message left empty — this is how Python avoids surfacing an empty message from metadata-only updates (AgentResponse.from_updates / from_update_generator always call _finalize_response before returning).
• .NET dotnet/src/Microsoft.Agents.AI.Abstractions/AgentResponse.cs, ToAgentResponseUpdates() (~line 265): emits one update per message, using Role = message.Role (no default-to-assistant substitution), plus a single "extra" metadata update only when AdditionalProperties or Usage is set. AgentResponseExtensions.ToAgentResponse delegates message reconstruction to Microsoft.Extensions.AI's ToChatResponse(), whose empty-message handling isn't vendored in this repo, so its exact behavior couldn't be directly inspected here.

Result: aligned. Go's new guard in Update achieves the same end state as Python's post-hoc empty-message filter in _finalize_response — a response built purely from response-level metadata (continuation token, additional properties) round-trips through ToUpdates/Update without producing a spurious empty assistant message. The mechanism differs (guard-on-write vs. filter-on-finalize) but that's an implementation detail, not an observable divergence, and no new exported API surface was introduced. The Role default-to-assistant addition in ToUpdates is a minor Go-specific normalization (guards against emitting an update with an empty Role for an existing message) and doesn't conflict with either upstream implementation. No parity concerns found; public-api-change label not applicable since no exported API surface changed.

Generated by Go API Consistency Review Agent · copilot · auto · 82.2 AIC · ⌖ 7.41 AIC · ⊞ 9.2K · ◷

[Copilot]

Copilot review overview

🟡 Changes recommended

Metadata-only updates can still produce empty assistant messages during collection; the agent-level path needs correction and regression coverage.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1

Open (1)

• Metadata-only updates incorrectly collected as empty assistant messages · New

Resolved since last review (1)

• Preserve metadata-only messages during response update conversion

[PratikDhanave (PratikDhanave)]

Reviewed this closely since it touches the same Response.Update/ToUpdates seam I was in for #1133. The core change is right and matches M.E.AI ChatResponseExtensions.ProcessUpdate: a response-metadata-only update should not materialize an empty message. Tests are a good round-trip shape. A few notes:

Composition with #1133 (worth the maintainers knowing before merge order is decided). #1133 scopes AdditionalProperties in Response.Update to the message when the update carries a MessageID and to the response otherwise. These two PRs are complementary and both needed:

• Metadata-only update (no MessageID): this PR already skips message creation, so AdditionalProperties only reaches the response — which happens to give the same result Scope Response.Update AdditionalProperties by MessageID #1133 does for that case.
• Update with a MessageID + AdditionalProperties: this PR still enters the message block, where today the map is copied to both the message and the response (the double-copy Scope Response.Update AdditionalProperties by MessageID #1133 fixes). This PR does not address that case.

So they dont conflict, but whichever merges second will want a trivial rebase in the same block. No action needed here beyond awareness.

Question — RawRepresentation != nil in the guard. Including it means a provider update that carries only a raw representation (no id/role/contents) will still create an empty-content message. Within ToUpdates this cant happen (the extra response-level update has RawRepresentation == nil), so the round-trip tests pass — but for a live streaming provider that emits a final raw-only event, is materializing a message the intended behavior, or should raw-only also be treated as response-level? Might be worth a comment on the guard either way.

Question — the ToUpdates role default (msg.Role == "" -> RoleAssistant). This is a separate behavior tweak bundled into the same PR. It looks reasonable, but does it mirror what .NET does for an unroled message on ToChatResponseUpdates, and is it needed for the metadata-message case specifically? A one-line rationale in the code or PR body would help future readers see why it rides along.

Nothing blocking from my perspective — the direction is correct and well-tested. 👍

[Kirill (llirik0)]

Rebased onto current main and resolved the conflicts. Thanks PratikDhanave (@PratikDhanave) for the detailed review. Raw-only updates are now explicitly response-level; regressions also cover lifecycle events arriving after a real OpenAI message, so inherited roles cannot leak lifecycle timestamps or raw payloads into that message. The ToUpdates assistant fallback is limited to otherwise unidentifiable metadata-only messages, with the collector-default rationale documented in code. The rebase preserves the MessageID-based AdditionalProperties scoping now present on main; #1133 is still open. Full tests, race tests, vet, and root/catalog lint pass.

[Copilot]

🟡 Changes recommended

Content-bearing updates can corrupt tool-message grouping, while metadata round trips and empty artifact handling can lose data.

4 open issues

• Preserve response-only raw events in mixed responses · New
• Retain streamed empty artifact messages · New
• Preserve assistant role for consecutive tool calls · New
• Default content-bearing streamed updates to assistant role · New

1 resolved since last review

• Metadata-only updates incorrectly collected as empty assistant messages

_{Review effort: Max}

[Copilot]

🔵 Needs a closer look

Previously reported message-preservation issues in A2A task snapshots and OpenAI tool rounds remain unresolved.

4 open issues

• Default content-bearing streamed updates to assistant role
• Preserve assistant role for consecutive tool calls
• Retain streamed empty artifact messages
• Preserve response-only raw events in mixed responses

_{Review effort: Balanced}