Skip to content

tui: clarify pending steer follow-ups#13841

Merged
charley-oai merged 6 commits intomainfrom
codex-worktree-too-pending-steer-ui
Mar 9, 2026
Merged

tui: clarify pending steer follow-ups#13841
charley-oai merged 6 commits intomainfrom
codex-worktree-too-pending-steer-ui

Conversation

@charley-oai
Copy link
Copy Markdown
Contributor

@charley-oai charley-oai commented Mar 7, 2026

Summary

  • split the pending input preview into labeled pending-steer and queued follow-up sections
  • explain that pending steers submit after the next tool call and that Esc can interrupt and send them immediately
  • treat Esc as an interrupt-plus-resubmit path when pending steers exist, with updated TUI snapshots and tests

Queues and steers:
Screenshot 2026-03-07 at 10 17 17 PM

After pressing Esc:
Screenshot 2026-03-07 at 10 17 21 PM

Codex author

codex resume 019cc6f4-2cca-7803-b717-8264526dbd97

@charley-oai
Copy link
Copy Markdown
Contributor Author

@codex review this

@chatgpt-codex-connector
Copy link
Copy Markdown
Contributor

Codex Review: Didn't find any major issues. What shall we delve into next?

ℹ️ 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".

charley-oai and others added 6 commits March 8, 2026 12:23
Label pending steers and queued follow-up messages separately in the bottom pane, and explain that Esc can interrupt the current turn and send pending steers immediately.

Also wire Esc to promote pending steers into the queued-input flow after an interrupt instead of restoring them to the composer.

Tests: just fmt; cargo test -p codex-tui

Co-authored-by: Codex <noreply@openai.com>
Use the same arrow glyph for pending steers and queued follow-ups now that the sections are labeled separately.

Tests: just fmt; cargo test -p codex-tui

Co-authored-by: Codex <noreply@openai.com>
Prefix the pending-input section headers with a bullet so the follow-up preview sits more naturally under the working status line.

Tests: just fmt; cargo test -p codex-tui

Co-authored-by: Codex <noreply@openai.com>
When Esc interrupts a running turn, merge every pending steer into one immediate follow-up submission instead of only resubmitting the first steer.

Tests: just fmt; cargo test -q -p codex-tui

Co-authored-by: Codex <noreply@openai.com>
Show an informational banner when Esc interrupts a turn specifically to submit pending steer instructions, and keep the generic error banner for ordinary interrupts.

Tests: just fmt; cargo test -q -p codex-tui

Co-authored-by: Codex <noreply@openai.com>
@charley-oai charley-oai force-pushed the codex-worktree-too-pending-steer-ui branch from 598814c to 67671bb Compare March 8, 2026 19:23
@charley-oai charley-oai merged commit 4ad3b59 into main Mar 9, 2026
29 of 31 checks passed
@charley-oai charley-oai deleted the codex-worktree-too-pending-steer-ui branch March 9, 2026 03:13
@github-actions github-actions bot locked and limited conversation to collaborators Mar 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants