[PratikDhanave (PratikDhanave)]

The Responses path built a TextContent from an output_text part (non-streaming responsesProcessResponse, and the streaming output_item.done handler) but never read output_text.logprobs. So a caller that requested logprobs via raw params (Include: ["message.output_text.logprobs"], TopLogprobs) never received them.

The Python client preserves them on the text content's additional_properties["logprobs"] on both parse paths.

Change

• Non-streaming: populate the TextContent AdditionalProperties with the output_text logprobs when present.
• Streaming: the output_item.done handler previously emitted a text content only when annotations were present (the text itself already arrived via deltas); it now also emits when logprobs are present and carries them, mirroring the annotations handling. Key uses the PascalCase convention already used elsewhere in the provider.

Test

• TestResponsesOutputTextLogprobsSurfaced_NonStreaming: a canned response whose output_text carries a logprobs array now surfaces Logprobs on the TextContent AdditionalProperties. Fails before the change (nil), passes after.

[Copilot]

🟡 Changes recommended

Streaming logprobs can be lost during response collection, and the new streaming path lacks regression coverage.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Updates Responses API handling to expose output_text logprobs for non-streaming and streaming responses.

Changes:

• Propagates logprobs to TextContent.AdditionalProperties.
• Emits streaming metadata for logprobs.
• Adds non-streaming regression coverage.

File summaries

File	Summary
provider/openaiprovider/responses.go	Propagates output text logprobs.
provider/openaiprovider/responses_test.go	Tests non-streaming logprobs exposure.

Review details

Suppressed comments (1)

provider/openaiprovider/responses.go:1602

• When logprobs are present without annotations, this creates an empty TextContent immediately after the delta text. ResponseStream.Collect() then coalesces adjacent text contents and copies AdditionalProperties only from the first item (message/content.go:1175-1179), so the returned collected response loses these logprobs. Preserve the metadata through coalescing (for example, merge the content metadata) or otherwise make the logprobs update survive Collect(); add a streaming regression test for the logprobs-only case.

						populateAnnotations(outputText.Annotations, annotatedContent)
						if len(outputText.Logprobs) > 0 {
							annotatedContent.AdditionalProperties = map[string]any{"Logprobs": outputText.Logprobs}

• Files reviewed: 2/2 changed files
• Comments generated: 1
• Review effort level: Lite

---

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

[github-actions]

Warning

Firewall blocked 4 domains

The following domains were blocked by the firewall during workflow execution:

• proxy.golang.org
• raw.githubusercontent.com
• storage.googleapis.com
• sum.golang.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "proxy.golang.org"
    - "raw.githubusercontent.com"
    - "storage.googleapis.com"
    - "sum.golang.org"

See Network Configuration for more information.

Generated by Go API Consistency Review Agent for #1105 · copilot · auto · 273.7 AIC · ⌖ 6.27 AIC · ⊞ 13.9K

[github-actions]

API consistency review

Result	Scope
unable to determine	user-visible behavior

Changed Go contract

provider/openaiprovider/responses.go: Responses output_text logprobs are now surfaced on message.TextContent.AdditionalProperties["Logprobs"], in both non-streaming (responsesProcessResponse) and streaming (output_item.done, which now also emits a text content when logprobs are present). No exported identifiers changed.

Upstream evidence reviewed

No upstream source could be reviewed. GitHub code search was rate-limited (429, retry after about 12 minutes). python/packages/openai/agent_framework_openai/_chat_client.py is about 244 KB, which is too large to fetch in full here. The .NET OpenAI Responses adapter (OpenAIResponsesChatClient.cs) was not checked either.

Assessment

The PR description says Python stores logprobs under additional_properties["logprobs"] on both parse paths. I could not confirm this. I also could not check the key naming (Logprobs against logprobs), whether .NET MEAI surfaces logprobs, or how upstream streaming emits them. Maintainers should check these points manually. No parity finding is reported. The public-api-change label was removed because no exported Go APIs changed.

Generated by Go API Consistency Review Agent · copilot · auto · 20.9 AIC · ⌖ 0.618 AIC · ⊞ 5.1K · ◷