Skip to content

stop blocking unified_exec on Windows#19435

Merged
iceweasel-oai merged 4 commits intomainfrom
codex/windows-unified-exec-opt-in
Apr 29, 2026
Merged

stop blocking unified_exec on Windows#19435
iceweasel-oai merged 4 commits intomainfrom
codex/windows-unified-exec-opt-in

Conversation

@iceweasel-oai
Copy link
Copy Markdown
Collaborator

@iceweasel-oai iceweasel-oai commented Apr 24, 2026

Summary

  • remove the Windows-specific unified-exec environment block from tool selection
  • keep unified_exec default-off on Windows unless the feature is explicitly enabled
  • normalize model-provided shell_type = unified_exec to shell_command when the feature is disabled
  • drop obsolete tests tied to the removed environment gate and keep the feature-flag regression coverage

Why

Now that the session/long-lived process backend is implemented for the Windows sandbox, we don't need to hard disable it anymore. We will be rolling out slowly using a feature gate.

Impact

This allows manual Windows opt-in in CLI and app-backed flows while preserving the existing default-off behavior for Windows users.

@iceweasel-oai iceweasel-oai changed the title [codex] Allow manual unified_exec opt-in on Windows stop blocking unified_exec on Windows Apr 24, 2026
@iceweasel-oai
Copy link
Copy Markdown
Collaborator Author

@codex review

@iceweasel-oai iceweasel-oai marked this pull request as ready for review April 24, 2026 21:01
@iceweasel-oai iceweasel-oai requested a review from a team as a code owner April 24, 2026 21:01
@iceweasel-oai iceweasel-oai requested a review from jif-oai April 27, 2026 23:36
Copy link
Copy Markdown
Collaborator

@jif-oai jif-oai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok after my commebnts

Comment thread codex-rs/tools/src/tool_config.rs
Comment thread codex-rs/core/src/tools/spec_tests.rs
@iceweasel-oai iceweasel-oai force-pushed the codex/windows-unified-exec-opt-in branch from a2609b9 to e1f9bda Compare April 28, 2026 17:26
iceweasel-oai and others added 3 commits April 28, 2026 10:50
Updates the plan mode nudge test to pass the new `duration_ms` argument
to task completion.

Co-authored-by: Codex <noreply@openai.com>
…-exec-opt-in

# Conflicts:
#	codex-rs/tui/src/chatwidget/tests/plan_mode.rs
@iceweasel-oai iceweasel-oai merged commit 13dbcda into main Apr 29, 2026
25 checks passed
@iceweasel-oai iceweasel-oai deleted the codex/windows-unified-exec-opt-in branch April 29, 2026 23:06
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants