Skip to content

Remove ToolSearch feature toggle#23389

Merged
sayan-oai merged 1 commit into
mainfrom
toolsearch-flag-removed
May 19, 2026
Merged

Remove ToolSearch feature toggle#23389
sayan-oai merged 1 commit into
mainfrom
toolsearch-flag-removed

Conversation

@sayan-oai

Copy link
Copy Markdown
Collaborator

Summary

  • mark ToolSearch as removed and ignore stale config writes for its legacy key
  • make search tool exposure depend only on model capability, not a feature toggle
  • remove app-server enablement support and prune now-obsolete test coverage/setup

Verification

  • cargo test -p codex-features
  • cargo test -p codex-tools
  • cargo test -p codex-core search_tool_requires_model_capability
  • cargo test -p codex-app-server experimental_feature_enablement_set_

Notes

  • This keeps the legacy config key as a no-op for compatibility while removing the ability to toggle the behavior off cleanly.
  • No developer-facing docs update outside the touched app-server README was needed.

@sayan-oai sayan-oai requested a review from a team as a code owner May 19, 2026 01:03
@sayan-oai sayan-oai enabled auto-merge (squash) May 19, 2026 01:05
@sayan-oai sayan-oai merged commit daa1182 into main May 19, 2026
31 checks passed
@sayan-oai sayan-oai deleted the toolsearch-flag-removed branch May 19, 2026 01:24
@github-actions github-actions Bot locked and limited conversation to collaborators May 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants