Skip to content

refactor: adapt exec-server to shell-escalation executor API#12615

Closed
bolinfest wants to merge 1 commit intopr12614from
pr12615
Closed

refactor: adapt exec-server to shell-escalation executor API#12615
bolinfest wants to merge 1 commit intopr12614from
pr12615

Conversation

@bolinfest
Copy link
Copy Markdown
Collaborator

@bolinfest bolinfest commented Feb 23, 2026

Why

This is the exec-server follow-up to the shell-escalation API prep in #12614, split out of PR #12584 to keep review scope smaller.

Separating the exec-server adoption from the core zsh-fork migration makes it easier to review the adapter wiring and confirm this remains behavior-preserving.

What Changed

  • Update exec-server imports to the new codex_shell_escalation::unix::* module layout.
  • Introduce McpShellCommandExecutor in exec-server/src/unix/mcp.rs to implement shell-escalation's new ShellCommandExecutor trait by delegating to codex_core::exec::process_exec_tool_call.
  • Convert codex_core::SandboxState into shell-escalation's crate-local SandboxState before invoking run_escalate_server.
  • Update the Unix exec-wrapper entrypoint import in exec-server/src/unix.rs.
  • Add tokio-util dependency/lockfile wiring for CancellationToken use in the adapter.

Verification


Stack created with Sapling. Best reviewed with ReviewStack.

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