Skip to content

codex-local 11/11: local web search via DuckDuckGo MCP; disable server-side web_search - #15

Closed
ehrvs wants to merge 1 commit into
pr/10-preflightfrom
pr/11-websearch
Closed

ehrvs wants to merge 1 commit into
pr/10-preflightfrom
pr/11-websearch

Conversation

@ehrvs

@ehrvs ehrvs commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator

Problem

codex sends the built-in server-side `{"type":"web_search"}` tool; Ollama answers `something went wrong, please see the ollama server logs` (reproduced with curl against the PAIR Ollama on :11435; its stderr is not logged locally, so the cause, Ollama web search needing an ollama.com login/key, is inferred).

Change

  • Seeded config gets `web_search = "disabled"`, and every run also passes `-c web_search="disabled"` (like the retry overrides), so existing installs are covered.
  • Seeded `[mcp_servers.web-search]`: `uvx duckduckgo-mcp-server==0.7.0` (search, fetch_content; no API key; MIT; ~1.5k GitHub stars; releases through 2026-09-04), with `default_tools_approval_mode = "approve"` since the tools are read-only and `approval_policy=never` otherwise rejects calls.
  • `sync-config` keeps codex-local's own web-search entry (adds the default if an older home lacks it).
  • install.py / verify_install.py warn (not fail) if `uvx` is missing, with an install hint.
  • Tests: seed contents, sync preservation, override list, runner detection. No upstream edits; fork-surface unchanged.

Live result

Temp homes, `codex-local exec ... "Search the web for recommended open source window managers for macOS..."` on :11435: `web-search/search` completed, no built-in web_search, no Ollama error, answer named Rectangle, Amethyst, yabai.

🤖 Generated with Claude Code

https://claude.ai/code/session_01RPCzgwU247gMtjybR5v4iR

…r-side web_search

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RPCzgwU247gMtjybR5v4iR
@ehrvs

ehrvs commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded: merged into codex-local/main via the integration PR #21 (3908b3d).

@ehrvs ehrvs closed this Oct 16, 2026
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