fix(config): remove web search option from codex command#257
Merged
folke merged 3 commits intofolke:mainfrom Mar 16, 2026
Merged
fix(config): remove web search option from codex command#257folke merged 3 commits intofolke:mainfrom
folke merged 3 commits intofolke:mainfrom
Conversation
khaneliman
added a commit
to khaneliman/khanelivim
that referenced
this pull request
Feb 22, 2026
Contributor
Author
WorkaroundYou can work around this until this PR is merged by overriding return {
{
"folke/sidekick.nvim",
config = function()
require("sidekick").setup({
cli = {
tools = {
codex = {
cmd = { "codex" },
},
},
},
})
end,
},
} |
Owner
|
ty |
folke
pushed a commit
that referenced
this pull request
Mar 20, 2026
🤖 I have created a release *beep* *boop* --- ## [2.2.0](v2.1.0...v2.2.0) (2026-03-20) ### Features * **agents:** add pi agent ([#247](#247)) ([7e9da9b](7e9da9b)) * **nes:** set `nes.diff.show = "cursor"` to only show nes diffs at the cursor. Closes [#266](#266) ([b1568d3](b1568d3)) * **terminal:** add files/buffers inline with spaces from picker. Closes [#133](#133) ([e743ac7](e743ac7)) * **terminal:** make re-entering terminal/normal mode more sensible ([9eb6530](9eb6530)) ### Bug Fixes * **codex:** update deprecated search option ([#166](#166)) ([88cb6dd](88cb6dd)) * **config:** remove web search option from codex command ([#257](#257)) ([f8b4f58](f8b4f58)) * **config:** update codex search option ([#167](#167)) ([c302dba](c302dba)) * **nes:** correct inline diff artifact when appending to end of line ([#250](#250)) ([f95ba54](f95ba54)) * **nes:** never process cancelled (or out of order) requests ([f8eac10](f8eac10)) * **opencode:** Use alt+p instead of ctrl+p. Fixes [#175](#175) ([a6fe80f](a6fe80f)) * **terminal:** ready check should not fail if cli window is closed. Fixes [#252](#252) ([6b69c42](6b69c42)) * **treesitter:** dont use treesitter stringbuffers ([317ada1](317ada1)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
msurovcak
pushed a commit
to msurovcak/sidekick.nvim
that referenced
this pull request
Apr 14, 2026
## Description <!-- Describe the big picture of your changes to communicate to the maintainers why we should accept this pull request. --> When using Codex, the following error is displayed: <img width="525" height="60" alt="Screenshot 2026-02-16 12 26 55" src="https://github.com/user-attachments/assets/bc5f63a0-b00a-4f42-9271-8e43dcab7d4a" /> The Codex command was originally defined as `codex --search`, and was later changed to `codex --enable web_search_request` in folke#167 to keep up with Codex updates. However, web search is now enabled by default in the current version of Codex, making these CLI options unnecessary. This PR removes the option. Reference: [Web search is now enabled by default - Codex changelog](https://developers.openai.com/codex/changelog/#codex-2026-01-28-mdx) ## Related Issue(s) <!-- If this PR fixes any issues, please link to the issue here. - Fixes #<issue_number> --> ## Screenshots <!-- Add screenshots of the changes if applicable. -->
msurovcak
pushed a commit
to msurovcak/sidekick.nvim
that referenced
this pull request
Apr 14, 2026
🤖 I have created a release *beep* *boop* --- ## [2.2.0](folke/sidekick.nvim@v2.1.0...v2.2.0) (2026-03-20) ### Features * **agents:** add pi agent ([folke#247](folke#247)) ([7e9da9b](folke@7e9da9b)) * **nes:** set `nes.diff.show = "cursor"` to only show nes diffs at the cursor. Closes [folke#266](folke#266) ([b1568d3](folke@b1568d3)) * **terminal:** add files/buffers inline with spaces from picker. Closes [folke#133](folke#133) ([e743ac7](folke@e743ac7)) * **terminal:** make re-entering terminal/normal mode more sensible ([9eb6530](folke@9eb6530)) ### Bug Fixes * **codex:** update deprecated search option ([folke#166](folke#166)) ([88cb6dd](folke@88cb6dd)) * **config:** remove web search option from codex command ([folke#257](folke#257)) ([f8b4f58](folke@f8b4f58)) * **config:** update codex search option ([folke#167](folke#167)) ([c302dba](folke@c302dba)) * **nes:** correct inline diff artifact when appending to end of line ([folke#250](folke#250)) ([f95ba54](folke@f95ba54)) * **nes:** never process cancelled (or out of order) requests ([f8eac10](folke@f8eac10)) * **opencode:** Use alt+p instead of ctrl+p. Fixes [folke#175](folke#175) ([a6fe80f](folke@a6fe80f)) * **terminal:** ready check should not fail if cli window is closed. Fixes [folke#252](folke#252) ([6b69c42](folke@6b69c42)) * **treesitter:** dont use treesitter stringbuffers ([317ada1](folke@317ada1)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
When using Codex, the following error is displayed:
The Codex command was originally defined as
codex --search, and was later changed tocodex --enable web_search_requestin #167 to keep up with Codex updates.However, web search is now enabled by default in the current version of Codex, making these CLI options unnecessary. This PR removes the option.
Reference: Web search is now enabled by default - Codex changelog
Related Issue(s)
Screenshots