From e7b9f3a5d7e06c690c82b9fd13a93310bcf388ed Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E8=B0=A2=E6=A0=8B=E6=A2=81?= Date: Wed, 3 Dec 2025 14:32:37 +0800 Subject: [PATCH] feat: re-enable web search for clink codex using correct --enable flag The previous fix (aceddb6) removed --search entirely, disabling web search. This restores web search functionality using the correct --enable flag that works with the codex exec subcommand. Related to #338 --- conf/cli_clients/codex.json | 4 +++- tests/test_clink_tool.py | 8 +++++++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/conf/cli_clients/codex.json b/conf/cli_clients/codex.json index 9e2a6e3..4f210ed 100644 --- a/conf/cli_clients/codex.json +++ b/conf/cli_clients/codex.json @@ -3,7 +3,9 @@ "command": "codex", "additional_args": [ "--json", - "--dangerously-bypass-approvals-and-sandbox" + "--dangerously-bypass-approvals-and-sandbox", + "--enable", + "web_search_request" ], "env": {}, "roles": { diff --git a/tests/test_clink_tool.py b/tests/test_clink_tool.py index 3781484..6bbf522 100644 --- a/tests/test_clink_tool.py +++ b/tests/test_clink_tool.py @@ -60,7 +60,13 @@ def test_registry_lists_roles(): assert "default" in roles assert "default" in registry.list_roles("codex") codex_client = registry.get_client("codex") - assert codex_client.config_args == ["--json", "--dangerously-bypass-approvals-and-sandbox"] + # Verify codex uses --enable web_search_request (not --search which is unsupported by exec) + assert codex_client.config_args == [ + "--json", + "--dangerously-bypass-approvals-and-sandbox", + "--enable", + "web_search_request", + ] @pytest.mark.asyncio