shell-pool / shpool

Think tmux, then aim... lower

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Improving session restore functionality

Aetf opened this issue · comments

commented

Overview

Shpool currently has a session restore feature that uses an in-memory terminal emulator implementation to record shell output, and then generate appropriate format codes and data to restore the last known screen state when a user reconnects. There are some issues with the current functionality that we would like to address.

Problem

The current in-memory terminal crate we use, shpool_vt100, has some issues. In particular, it does not use a very memory efficient representation of the terminal, and a lack of correct text wrapping supports means we need to make each line 10k characters long. Worse than the excessive resource consumption though is the fact that not all control sequences are correctly handled, resulting in output being occasionally mangled.

List of control sequences we may consider support:

  • #44
  • any other?

Potential Solution

Fixing the shpool_vt100 crate and replacing it with a new crate, which could either be written from the ground up or extracted from one of several existing rust terminals.

Currently wezterm-term looks promising, but it is not published to crates.io, and we need more investigation to see if it actually fits shpool's use case.

Other terminals considered:

commented

I'll try to integrate wezterm-term, and @jalberss will give alacritty a try. We will see which one works better, or we need to write one from scratch after all.

In the end of the day, if we end up with both. We can have a selection of backends for the user to choose.