Skip to content

Conversation

@gameofby
Copy link
Contributor

@gameofby gameofby commented Dec 7, 2025

Issue #7661 revealed that users are confused by deprecation warnings like:

tools.web_search is deprecated. Use web_search_request instead.

This message misleadingly suggests renaming the config key from web_search to web_search_request, when the actual required change is to move and rename the configuration from the [tools] section to the [features] section.

This PR clarifies the warning messages and documentation to make it clear that deprecated [tools] configurations should be moved to [features]. Changes made:

  • Updated deprecation warning format in codex-rs/core/src/codex.rs:520 to include [features]. prefix
  • Updated corresponding test expectations in codex-rs/core/tests/suite/deprecation_notice.rs:39
  • Improved documentation in docs/config.md to clarify upfront that [tools] options are deprecated in favor of [features]

@github-actions
Copy link

github-actions bot commented Dec 7, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@gameofby
Copy link
Contributor Author

gameofby commented Dec 7, 2025

I have read the CLA Document and I hereby sign the CLA

@gameofby
Copy link
Contributor Author

gameofby commented Dec 7, 2025

recheck

@etraut-openai
Copy link
Collaborator

Thanks for the contribution!

@etraut-openai etraut-openai merged commit 9892365 into openai:main Dec 8, 2025
26 of 27 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2025
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