Skip to content

DI-4253: support OPENAI_CODEX_REASONING_EFFORT env var#149

Merged
wiggzz merged 3 commits into
mainfrom
wtj/di-4253-codex-reasoning-effort
May 28, 2026
Merged

DI-4253: support OPENAI_CODEX_REASONING_EFFORT env var#149
wiggzz merged 3 commits into
mainfrom
wtj/di-4253-codex-reasoning-effort

Conversation

@wiggzz

@wiggzz wiggzz commented May 27, 2026

Copy link
Copy Markdown
Contributor

Why

When comparing the codex baseline against another agent, it's useful to be able to dial codex's reasoning effort to match the other agent's configuration, so the comparison isn't confounded by effort defaults. There's no way to do that today from outside the agent.

What

OpenAICodexAgent reads an optional OPENAI_CODEX_REASONING_EFFORT env var and, if set, passes it through as -c model_reasoning_effort=<value> on the codex command line. When unset, the command is unchanged.

Going through an env var rather than a constructor kwarg avoids touching the Harness / AgentFactory plumbing, since callers (run_harness.py, downstream wrappers) don't currently thread arbitrary agent kwargs through.

Drafted by Opus 4.7 under the direction of @wiggzz.

wiggzz added 2 commits May 27, 2026 11:26
Lets the ai-codegen-api ade-bench harness match the studio dev agent's
configured reasoning effort (low) when running the codex baseline, so
the comparison isn't confounded by effort defaults. When the env var is
unset, behavior is unchanged — codex picks its own default.
@wiggzz wiggzz marked this pull request as ready for review May 27, 2026 18:53
@wiggzz wiggzz requested a review from joellabes May 27, 2026 18:53
@wiggzz wiggzz merged commit df1e9e2 into main May 28, 2026
5 checks passed
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.

2 participants