Skip to content

Allow selecting a specific shell#11851

Merged
jr merged 2 commits intomainfrom
jr/add-cli-shell-param
Mar 4, 2026
Merged

Allow selecting a specific shell#11851
jr merged 2 commits intomainfrom
jr/add-cli-shell-param

Conversation

@jr
Copy link
Collaborator

@jr jr commented Mar 4, 2026

Add --terminal-shell CLI flag to specify which shell ExecaTerminalProcess uses for inline command execution. The shell path is validated at the CLI layer and passed through the standard settings mechanism (BaseTerminal static getter/setter), matching how all other CLI terminal settings flow through the system.

Interactively review PR in Roo Code Cloud

Add --terminal-shell CLI flag to specify which shell ExecaTerminalProcess
uses for inline command execution. The shell path is validated at the CLI
layer and passed through the standard settings mechanism (BaseTerminal
static getter/setter), matching how all other CLI terminal settings flow
through the system.
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. Enhancement New feature or request labels Mar 4, 2026
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 4, 2026
@jr jr merged commit 7dc83a5 into main Mar 4, 2026
10 checks passed
@jr jr deleted the jr/add-cli-shell-param branch March 4, 2026 07:31
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Mar 4, 2026
@github-actions github-actions bot mentioned this pull request Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants