Skip to content

🤖 fix: avoid OpenAI tool search name collision#3719

Merged
ibetitsmike merged 1 commit into
mainfrom
mike/fix-tool-search-result-validation
Jul 13, 2026
Merged

🤖 fix: avoid OpenAI tool search name collision#3719
ibetitsmike merged 1 commit into
mainfrom
mike/fix-tool-search-result-validation

Conversation

@ibetitsmike

Copy link
Copy Markdown
Contributor

Summary

Rename Mux's deferred MCP catalog search tool from tool_search to tool_catalog_search so OpenAI Responses treats it as a custom function tool rather than its native tool-search primitive.

Background

AI SDK 7 reserves tool_search for OpenAI's native tool search and validates results against { tools: [...] }. Mux returned { query, matches, totalDeferred }, so continuing after a catalog search failed with a type-validation error.

Implementation

  • Register and advertise the live Mux tool as tool_catalog_search.
  • Normalize completed historical Mux tool_search calls in provider-bound messages without changing persisted history.
  • Keep legacy transcript rendering and deferred-tool activation compatible with existing workspaces.
  • Avoid rewriting unrelated MCP tools that happen to use the legacy name.
  • Update tool definitions, UI mappings, generated documentation, and regression coverage.

Validation

  • MUX_ESLINT_CONCURRENCY=1 make static-check
  • OpenAI Responses wire regression for completed legacy history
  • Tool catalog, legacy activation, tool renderer, tool definition, and stream active-tool tests

Risks

Low. The live name changes only while the tool-search experiment is enabled. Compatibility handling is request-only and requires the historical result to match Mux's catalog-search shape.

Mux worked on behalf of Mike.


Generated with mux • Model: openai:gpt-5.6-sol • Thinking: xhigh • Cost: $27.56

Rename Mux's deferred tool catalog search and preserve compatibility with completed legacy history.

> Mux worked on behalf of Mike.

---

_Generated with `mux` • Model: `openai:gpt-5.6-sol` • Thinking: `xhigh` • Cost: `$27.56`_

<!-- mux-attribution: model=openai:gpt-5.6-sol thinking=xhigh costs=27.56 -->
@ibetitsmike

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. You're on a roll.

Reviewed commit: 0c370469a7

ℹ️ 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".

@ibetitsmike ibetitsmike added this pull request to the merge queue Jul 13, 2026
Merged via the queue into main with commit 7b10f5d Jul 13, 2026
23 checks passed
@ibetitsmike ibetitsmike deleted the mike/fix-tool-search-result-validation branch July 13, 2026 21:46
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