feat: add sandbox agents with clearer PTY Python validation#1221
feat: add sandbox agents with clearer PTY Python validation#1221seratch merged 2 commits intofeat/sandbox-agentsfrom
Conversation
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e2b72e4511
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
1 participant
This pull request adds the sandbox agents runtime and provider integrations, and improves the local Unix/Docker PTY path so
tty: truefails early with a configuration error when the Python PTY bridge is unavailable.The PTY bridge currently requires Python 3 because Node.js does not provide a standard-library API for allocating and driving a POSIX pseudo-terminal, while Python's standard library provides the needed
pty,select,signal,os.execvpe, andwaitpidprimitives without adding a native addon or shipping a platform-specific helper binary. Alternatives such asnode-ptyor a small compiled helper remain possible future directions, but they add distribution, ABI, or prebuilt-binary complexity; for now the SDK keeps the Python bridge and makes the requirement explicit and diagnosable. The implementation checks the configured Python executable using the same PATH/environment used for the PTY bridge, caches successful checks, and reports that users can installpython3or setOPENAI_AGENTS_PYTHON.