Skip to content

show bash mode in the TUI#18271

Merged
abhinav-oai merged 8 commits into
mainfrom
abhinav/tui-bash-mode
Apr 21, 2026
Merged

show bash mode in the TUI#18271
abhinav-oai merged 8 commits into
mainfrom
abhinav/tui-bash-mode

Conversation

@abhinav-oai

Copy link
Copy Markdown
Collaborator

What

  • Explicitly show our "bash mode" by changing the color and adding a callout similar to how we do for Plan mode (shift + tab to cycle)
  • Also replace our composer prefix with a bang !

Why

  • It was unclear that we had a Bash mode
  • This feels more responsive
  • It looks cool!

abhinav-oai added a commit that referenced this pull request Apr 17, 2026
Add the missing argument comment on the regression test cursor position literal.

Co-authored-by: Codex <noreply@openai.com>
fn shell_mode_footer_line(&self) -> Option<Line<'static>> {
self.is_bang_shell_command()
.then_some(())
.map(|_| Line::from(vec![Span::from("Bash mode").light_red()]))

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

btw the copy is just this now, no more (! to toggle) since it's not really a toggle but actually a prefix

@fcoury-oai fcoury-oai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found one bug: when you open an external editor and leaves without saving it returns the command with double !.

abhinav-oai and others added 6 commits April 20, 2026 10:24
Move shell-prefix handling into ChatComposer and drop the generic hidden-prefix textarea APIs.

Co-authored-by: Codex <noreply@openai.com>
Prevent absorbed bang commands from entering slash-command flows, preserve history navigation from the visible start of recalled shell drafts, and satisfy the argument comment lint.

Co-authored-by: Codex <noreply@openai.com>
Add the missing argument comment on the regression test cursor position literal.

Co-authored-by: Codex <noreply@openai.com>
@abhinav-oai abhinav-oai force-pushed the abhinav/tui-bash-mode branch from 515924f to 99c73c8 Compare April 20, 2026 17:27

@fcoury-oai fcoury-oai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! 👍

@abhinav-oai abhinav-oai merged commit ef071cf into main Apr 21, 2026
35 of 36 checks passed
@abhinav-oai abhinav-oai deleted the abhinav/tui-bash-mode branch April 21, 2026 07:15
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants