From 3ea35ba5d23b6262c72f0f71f116a218bbbdaa90 Mon Sep 17 00:00:00 2001 From: codex Date: Tue, 23 Jun 2026 04:06:21 +0000 Subject: [PATCH] fix: update codex supervise mcp registration --- bot_bottle/contrib/codex/agent_provider.py | 6 +++--- tests/unit/test_contrib_codex_provider.py | 8 ++++---- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/bot_bottle/contrib/codex/agent_provider.py b/bot_bottle/contrib/codex/agent_provider.py index 386c838..57f7e82 100644 --- a/bot_bottle/contrib/codex/agent_provider.py +++ b/bot_bottle/contrib/codex/agent_provider.py @@ -261,8 +261,8 @@ class CodexAgentProvider(AgentProvider): return info(f"registering supervise MCP server in agent codex config → {supervise_url}") r = bottle.exec( - f"codex mcp add --transport http " - f"{_SUPERVISE_MCP_NAME} {supervise_url}", + f"codex mcp add {_SUPERVISE_MCP_NAME} --url " + f"{shlex.quote(supervise_url)}", user="node", ) if r.returncode != 0: @@ -270,7 +270,7 @@ class CodexAgentProvider(AgentProvider): f"`codex mcp add supervise` failed (exit {r.returncode}): " f"{(r.stderr or r.stdout or '').strip()}. Inside the bottle, " f"register manually with: " - f"codex mcp add --transport http supervise {supervise_url}" + f"codex mcp add supervise --url {shlex.quote(supervise_url)}" ) diff --git a/tests/unit/test_contrib_codex_provider.py b/tests/unit/test_contrib_codex_provider.py index 0d678c2..54e2b2c 100644 --- a/tests/unit/test_contrib_codex_provider.py +++ b/tests/unit/test_contrib_codex_provider.py @@ -292,10 +292,10 @@ class TestCodexSuperviseMcp(unittest.TestCase): bottle.exec.assert_called_once() script = bottle.exec.call_args.args[0] self.assertEqual("node", bottle.exec.call_args.kwargs.get("user")) - self.assertIn("codex mcp add", script) - self.assertIn("--transport http", script) - self.assertIn("supervise", script) - self.assertIn(_URL, script) + self.assertEqual( + f"codex mcp add supervise --url {_URL}", + script, + ) def test_logs_warning_on_failure_but_does_not_raise(self): bottle = _make_bottle(