Windows Terminal scrolls to the top when pi-tui sends ESC[3J

[WodenJay]

What happened?

While using pi in Windows Terminal, the terminal repeatedly scrolls to the top during normal TUI redraws.

The problem is caused by this sequence in pi-tui/dist/tui.js:

buffer += "\x1b[2J\x1b[H\x1b[3J";

\x1b[3J clears the terminal scrollback buffer. Removing only that sequence completely stops the unwanted scrolling:

buffer += "\x1b[2J\x1b[H";

The same extensions remained enabled before and after the modification.

This appears related to #4506, which was closed during the project refactor, but the problem is still present on Windows Terminal in pi 0.80.6.

Steps to reproduce

1. Run pi in Windows Terminal.
2. Use it until a full TUI redraw occurs. (I ask pi to use find-skills skill to find some skills)
3. Observe that the terminal viewport unexpectedly scrolls to the top.
4. Remove \x1b[3J from the fullRender clear sequence.
5. Restart pi and observe that the scrolling no longer occurs.

Expected behavior

A full redraw should refresh the visible screen without clearing the terminal scrollback or changing the user's viewport position.

Version

0.80.6

[github-actions]

This issue was auto-closed. All issues from new contributors are auto-closed by default.

Maintainers review auto-closed issues daily and reopen worthwhile ones. Issues that do not meet the quality bar in CONTRIBUTING.md will not be reopened or receive a reply.

If a maintainer replies lgtmi on one of your issues, your future issues will stay open. If a maintainer replies lgtm, your future issues and PRs will stay open.

See CONTRIBUTING.md.

[badlogic]

maybe we can special case this for windows terminal, tho we need the scrollbuffer clrar.

[WodenJay]

maybe we can special case this for windows terminal, tho we need the scrollbuffer clrar.

One more thing: recently, I’ve been trying to keep the focus in the input box and avoid clicking anywhere else, and I don’t seem to run into this issue anymore. I think this is a complicated problem.

[petrroll]

The scroll up is known and has been reported to WT: microsoft/terminal#20370

It'll be fixed, no need to special case IMHO.