Skip to content

fix: render web_search_call output items in LLM context diagnostics#26243

Merged
noanflaherty merged 1 commit into
noanflaherty/managed-openai-native-web-searchfrom
run-plan/managed-openai-web/fix-r2-2
Apr 17, 2026
Merged

fix: render web_search_call output items in LLM context diagnostics#26243
noanflaherty merged 1 commit into
noanflaherty/managed-openai-native-web-searchfrom
run-plan/managed-openai-web/fix-r2-2

Conversation

@noanflaherty
Copy link
Copy Markdown
Contributor

@noanflaherty noanflaherty commented Apr 17, 2026

Summary

Fixes gap identified during plan review for managed-openai-native-web-search.md.

Gap: normalizeOpenAiResponsesResponsePayload silently skips web_search_call items in response output, making native web search invisible in diagnostics.


Open with Devin

@noanflaherty noanflaherty merged commit 3b08296 into noanflaherty/managed-openai-native-web-search Apr 17, 2026
10 checks passed
@noanflaherty noanflaherty deleted the run-plan/managed-openai-web/fix-r2-2 branch April 17, 2026 22:37
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

noanflaherty added a commit that referenced this pull request Apr 17, 2026
* [managed] Enable OpenAI managed-proxy fallback routing in provider bootstrap (#26211)

* [openai] Route inference-provider-native web search to Responses native web_search tool (#26212)

* [macOS] Allow managed inference provider selection beyond Anthropic (#26218)

* [macOS] Allow managed inference provider selection beyond Anthropic

* fix: capture draftProvider before async Task to prevent race condition

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* [macOS] Allow provider-native web search when managed inference provider supports it (#26230)

* [macOS] Allow provider-native web search when managed inference provider supports it

* fix: gate mode-specific web search invalidation on modeChanging to prevent false-positive alerts

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: generalize alert message to cover both mode and provider changes

* fix: scope provider-native invalidation to your-own web search mode

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: emit server_tool_start/complete events for OpenAI native web search (#26240)

* fix: render web_search_call output items in LLM context diagnostics (#26243)

* fix: add server_tool_use content blocks and tests for OpenAI native web search (#26246)

* fix: diagnostics polish — web_search_call tests, tool count fix, loop consistency (#26250)

* fix: emit paired web_search_tool_result blocks to prevent repairHistory corruption

The OpenAI Responses provider emitted server_tool_use content blocks for
web_search_call items but did not emit matching web_search_tool_result blocks.
repairHistory() treats any unpaired server_tool_use as an interrupted search
and injects a synthetic web_search_tool_result_error, which corrupts
conversation history by making successful searches appear as failures.

After each server_tool_use block, also emit a paired web_search_tool_result
with empty content (since OpenAI weaves search results into the text output).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant