valknar/llmx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tui: fix extra blank lines in streamed agent messages (#3065)

Browse Source 
Fixes excessive blank lines appearing during agent message streaming.

- Only insert a separator blank line for new, non-streaming history
cells.
- Streaming continuations now append without adding a spacer,
eliminating extra gaps between chunks.

Affected area: TUI display of agent messages (tui/src/app.rs).
This commit is contained in:
Jeremy Rose
2025-09-02 13:45:51 -07:00
committed by GitHub
parent 7bcdc5cc7c
commit 46e35a2345
1 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
codex-rs/tui/src/app.rs
View File

@@ -191,10 +191,15 @@ impl App {
self.transcript_lines.extend(cell_transcript.clone()); self.transcript_lines.extend(cell_transcript.clone());
let mut display = cell.display_lines(tui.terminal.last_known_screen_size.width); let mut display = cell.display_lines(tui.terminal.last_known_screen_size.width);
if !display.is_empty() { if !display.is_empty() {
if self.has_emitted_history_lines { // Only insert a separating blank line for new cells that are not
display.insert(0, Line::from("")); // part of an ongoing stream. Streaming continuations should not
} else { // accrue extra blank lines between chunks.
self.has_emitted_history_lines = true; if !cell.is_stream_continuation() {
if self.has_emitted_history_lines {
display.insert(0, Line::from(""));
} else {
self.has_emitted_history_lines = true;
}
} }
if self.overlay.is_some() { if self.overlay.is_some() {
self.deferred_history_lines.extend(display); self.deferred_history_lines.extend(display);