Skip to content

Conversation

alexx-ftw
Copy link

@alexx-ftw alexx-ftw commented Oct 8, 2025

What changed

Why

How to try locally

  1. cd codex-rs
  2. just fmt
  3. just fix -p codex-tui
  4. cargo test -p codex-tui -- burst_paste_large

Flags added or removed

  • none

Safety limits

  • unchanged

Fallback behavior

  • existing paste-burst placeholder expansion still applies

Tests run: just fmt, just fix -p codex-tui, cargo test -p codex-tui

Copy link

github-actions bot commented Oct 8, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@alexx-ftw
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Oct 8, 2025
@alexx-ftw alexx-ftw force-pushed the fix/paste-burst-newline branch from 9fed247 to b0e4fc2 Compare October 8, 2025 22:45
@alexx-ftw
Copy link
Author

Heads-up: there's a regression in the backtrack flow. Double-tapping Esc in the main chat (composer empty) now crashes with a stack overflow because we re-enter open_backtrack_preview without advancing the selection when the transcript has no user messages left. Logged as issue #4977 for follow-up.

@alexx-ftw
Copy link
Author

Updated PR with a fix for #4977: Esc backtrack now reacts only to key presses (ignoring repeats) and closes the overlay when no user messages remain, so double Esc no longer overflows the stack.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant