Skip to content

Codex web search uses the exec-compatible config override - #192

Merged
renmengye merged 2 commits into
mainfrom
fix/codex-web-search-flag
Aug 29, 2026
Merged

renmengye merged 2 commits into
mainfrom
fix/codex-web-search-flag

Conversation

@renmengye

Copy link
Copy Markdown
Member

Summary

#189 shipped --search in the codex extra args — but that flag exists only on the interactive command; codex exec rejects it, so every new fleet attempt aborted at session start (first casualty: agent-02's 09:30 attempt). The exec-compatible form is -c tools.web_search=true.

Verified live on the deployed codex-cli 0.130.0 before this PR: an authenticated codex exec --json -c tools.web_search=true session made two web_search calls and returned the fetched page title.

Test plan

  • ruff, format, mypy, pytest (881 passed)
  • Role-runner and harness tests now pin the config override and reject --search
  • Review round; merge restores the fleet (next tick deploys)

🤖 Generated with Claude Code

#189 passed --search, which only the interactive codex command accepts:
codex exec rejected it and every new fleet attempt aborted at session
start. The exec form is -c tools.web_search=true, verified live on the
deployed 0.130.0 — the probe session made real web_search calls.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Round 1 — reviewed head 05db53e8 — reviewer summarizer:hermes/gpt-5.6-terra over coverage+credentials+deployment+general+lifecycle+prose.

terra
Advisory findings from autoresearch — the code owner decides. Reply to disagree; the autoresearch:no-review label opts this PR out.

Verdict: nothing blocking — 1 advisory note.

1 finding attached to the lines below.

One advisory prose suggestion. No blocking findings. Rejected: none; the other lenses reported no findings. The prose lens notes that focused tests, full suite, Ruff, and mypy passed.

Comment thread src/autoresearch/role_runner.py Resolved
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Round 1 — reviewed head 669c742e — reviewer hermes/gpt-5.6-terra.

terra
Advisory findings from autoresearch — the code owner decides. Reply to disagree; the autoresearch:no-review label opts this PR out.

Verdict: no defects found.

@renmengye
renmengye merged commit c8f09f1 into main Aug 29, 2026
5 checks passed
@renmengye
renmengye deleted the fix/codex-web-search-flag branch August 29, 2026 14:21
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