chore: clarify default shell for unified_exec (#8997)

sayan-oai · web-flow · commit 5675af519013 · 2026-01-13T21:37:41.000-08:00
The description of the `shell` arg for `exec_command` states the default is `/bin/bash`, but AFAICT it's the user's default shell. Default logic [here](https://github.com/openai/codex/blob/2a06d64bc996e4d848b95285700b195c2852a42f/codex-rs/core/src/tools/handlers/unified_exec.rs#L123). EDIT: #9004 has an alternative where we inform the model of the default shell itself.
diff --git a/codex-rs/core/src/tools/spec.rs b/codex-rs/core/src/tools/spec.rs
@@ -157,7 +157,7 @@ fn create_exec_command_tool() -> ToolSpec {
         (
             "shell".to_string(),
             JsonSchema::String {
-                description: Some("Shell binary to launch. Defaults to /bin/bash.".to_string()),
+                description: Some("Shell binary to launch. Defaults to the user's default shell.".to_string()),
             },
         ),
         (

Original file line number	Diff line number	Diff line change
`@@ -157,7 +157,7 @@ fn create_exec_command_tool() -> ToolSpec {`
`157`	`157`	`(`
`158`	`158`	`"shell".to_string(),`
`159`	`159`	`JsonSchema::String {`
`160`		`- description: Some("Shell binary to launch. Defaults to /bin/bash.".to_string()),`
	`160`	`+ description: Some("Shell binary to launch. Defaults to the user's default shell.".to_string()),`
`161`	`161`	`},`
`162`	`162`	`),`
`163`	`163`	`(`