From aeb924dcbbd5d39494adc4f730787635e360a49f Mon Sep 17 00:00:00 2001 From: Space-Banane Date: Sat, 23 May 2026 00:08:03 +0200 Subject: [PATCH] [fix]. Wire runner reasoning effort --- src/gitea_codex_bot/workers/container_runner.py | 9 ++++++--- tests/test_container_runner.py | 11 +++++++++++ 2 files changed, 17 insertions(+), 3 deletions(-) diff --git a/src/gitea_codex_bot/workers/container_runner.py b/src/gitea_codex_bot/workers/container_runner.py index 08895c0..281ecf5 100644 --- a/src/gitea_codex_bot/workers/container_runner.py +++ b/src/gitea_codex_bot/workers/container_runner.py @@ -76,10 +76,13 @@ def _build_install_and_run_command(settings: Settings) -> str: ] ) model = settings.openai_review_model.strip() + reasoning_effort = settings.openai_reasoning_effort.strip() + codex_exec_parts = ["codex exec --skip-git-repo-check --json"] if model: - steps.append(f"codex exec --skip-git-repo-check --json -m {shlex.quote(model)}") - else: - steps.append("codex exec --skip-git-repo-check --json") + codex_exec_parts.append(f"-m {shlex.quote(model)}") + if reasoning_effort: + codex_exec_parts.append(f"--reasoning-effort {shlex.quote(reasoning_effort)}") + steps.append(" ".join(codex_exec_parts)) return "; ".join(steps) diff --git a/tests/test_container_runner.py b/tests/test_container_runner.py index fd41790..0d41def 100644 --- a/tests/test_container_runner.py +++ b/tests/test_container_runner.py @@ -59,6 +59,17 @@ def test_build_install_command_chatgpt_mode_copies_auth_json(monkeypatch: pytest assert 'printf "%s" "$CODEX_AUTH_JSON_B64" | base64 -d > /root/.codex/auth.json' in command assert "codex exec --skip-git-repo-check --json -m gpt-5.3-codex" in command + assert f"--reasoning-effort {settings.openai_reasoning_effort}" in command + + +def test_build_install_command_includes_configured_reasoning_effort(monkeypatch: pytest.MonkeyPatch) -> None: + monkeypatch.setenv("OPENAI_REASONING_EFFORT", "medium") + get_settings.cache_clear() + settings = get_settings() + + command = _build_install_and_run_command(settings) + + assert "--reasoning-effort medium" in command def test_chatgpt_mode_requires_existing_auth_json(monkeypatch: pytest.MonkeyPatch, tmp_path: Path) -> None: