Skip to content

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

Merged
noanflaherty merged 4 commits into
noanflaherty/managed-openai-native-web-searchfrom
managed-openai-web/pr-4-macos-web-search-provider-native-interop
Apr 17, 2026
Merged

[macOS] Allow provider-native web search when managed inference provider supports it#26230
noanflaherty merged 4 commits into
noanflaherty/managed-openai-native-web-searchfrom
managed-openai-web/pr-4-macos-web-search-provider-native-interop

Conversation

@noanflaherty
Copy link
Copy Markdown
Contributor

@noanflaherty noanflaherty commented Apr 17, 2026

Summary

  • Update InferenceServiceCard.wouldInvalidateWebSearch for capability-based checks
  • Allow inference-provider-native in WebSearchServiceCard when provider supports it
  • Remove unconditional managed-mode rewrite to perplexity
  • Extend tests for capability gating with managed OpenAI + provider-native

Part of plan: managed-openai-native-web-search.md (PR 4 of 4)


Open with Devin

devin-ai-integration[bot]

This comment was marked as resolved.

…event false-positive alerts

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@noanflaherty
Copy link
Copy Markdown
Contributor Author

@codex review this PR again — the previous issues have been fixed in commit 9a9ff87

@noanflaherty
Copy link
Copy Markdown
Contributor Author

@devin review this PR again — the previous issues have been fixed in commit 9a9ff87

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

devin-ai-integration[bot]

This comment was marked as resolved.

@noanflaherty
Copy link
Copy Markdown
Contributor Author

@codex review this PR again — the previous issues have been fixed in commit 171b834

@noanflaherty
Copy link
Copy Markdown
Contributor Author

@devin review this PR again — the previous issues have been fixed in commit 171b834

devin-ai-integration[bot]

This comment was marked as resolved.

chatgpt-codex-connector[bot]

This comment was marked as resolved.

@noanflaherty noanflaherty merged commit 6bad3b2 into noanflaherty/managed-openai-native-web-search Apr 17, 2026
6 checks passed
@noanflaherty noanflaherty deleted the managed-openai-web/pr-4-macos-web-search-provider-native-interop branch April 17, 2026 22:06
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