Skip to content

test(agentic_common): update Codex model-forwarding regression to gpt-5.6 (#1986)#1990

Open
prompt-driven-github[bot] wants to merge 1 commit into
mainfrom
fix/codex-gpt-5-6-model-forwarding-test
Open

test(agentic_common): update Codex model-forwarding regression to gpt-5.6 (#1986)#1990
prompt-driven-github[bot] wants to merge 1 commit into
mainfrom
fix/codex-gpt-5-6-model-forwarding-test

Conversation

@prompt-driven-github

Copy link
Copy Markdown
Contributor

Summary

  • Updates test_codex_model_env_var_passed_to_cli to use gpt-5.6 instead of the legacy o3-pro example, matching the cloud default introduced in pdd_cloud#3233.
  • Adds an explicit assertion that --model gpt-5.6 appears before the exec subcommand, locking in the top-level flag ordering required by Codex's CLI.
  • No changes to production code; model selection pass-through remains generic.

Test plan

  • pytest -vv tests/test_agentic_common.py -k codex_model — 1 passed
  • pytest -vv tests/test_agentic_common.py -k codex — 45 passed, 0 failures
  • Existing no-override test (test_codex_no_model_env_var_omits_model_flag) unchanged and passing

Closes #1986

🤖 Generated with Claude Code

…-5.6

Replaces the legacy o3-pro example in test_codex_model_env_var_passed_to_cli
with gpt-5.6, matching the cloud default set in pdd_cloud#3233. Adds an
explicit assertion that --model precedes the exec subcommand, locking in the
top-level flag contract required by Codex's CLI.

Closes #1986

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

Make GPT-5.6 the default Codex model across the pdd CLI

1 participant