contour-terminal / contour

Modern C++ Terminal Emulator
http://contour-terminal.org/
Apache License 2.0
2.39k stars 101 forks source link

Dynamically manage the scrollback buffer and remove (prompt, output) blocks atomically #836

Open whisperity opened 1 year ago

whisperity commented 1 year ago

More complex alternative of #835.

Abstract

Currently the scrollback buffer size is an explicit hard limit, which means that in case multiple long commands exceed the limit, the top of the buffer is cleared, potentially resulting in one command's output being cleared partially. Instead, if the shell integration is enabled and Contour has tracked where the prompts are in the scrollback, the clearing should be done for one atomic unit of executed command-line, not in a way that it "cuts" the history in the "middle".

Motivation

C++ SFINAE outputs.

Specification

Expected outcome: as long as the terminal does not exceed the hard limit, the oldest line in the scrollback buffer is a proper prompt, with the full output of said command.

christianparpart commented 1 year ago

After some conversations... this actually sounds like we want something similar towards the Blocks feature that also comes with Warp. Except that this ticket's request only cares about proper eviction of prompt + output rather than cutting it off in the middle.

The question I am having is how to implement it without severely sacrificing performance. The grid buffer (scrollback + main page area) is currently actually a ring buffer of lines.

We could design a double-layer principle where we still have that ring buffer for the current block to be constructed (if integration provides that meta data) and whenever a block is completed, it's then promoted into another data structure that keeps the historic blocks. for convenient rendering, viewport jumping actions through the scrollback and atomic eviction.

EDIT: The cool thing about this solution is (or should be) that users could still have commands with huge output (like cat 4gb.txt which would all go into a single block, not evicting older blocks, but actually truncating this block's history instead. That sounds like useful to me (not necessarily to everyone).