Apply cwd to bash kernel contexts#269
Conversation
The change_current_directory handler in the template server only had branches for python, javascript/typescript, r, and java — bash fell through to the no-op else, so a context created with a non-default cwd still returned `/` from `pwd`. Added a bash branch that sends `cd <path>` and added bash cwd tests across sync/async python and js suites. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
PR SummaryLow Risk Overview Adds bash Reviewed by Cursor Bugbot for commit 7416325. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8f692797f4
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 8f69279. Configure here.
Matches the quoting used in the js, r, and java branches so paths containing spaces or shell metacharacters cd to the intended directory. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Summary
change_current_directoryintemplate/server/messaging.pyhandled python, javascript/typescript, r, and java but fell through to a no-op for bash, so a context created with a non-defaultcwdstill returned/frompwd.cd <path>to the kernel, plus bash cwd tests across sync/async Python and JS suites (the gap that let this regression slip in).@e2b/code-interpreter-template.Refs: e2b-dev/E2B#1318
🤖 Generated with Claude Code