Async Clipboard

[meowgorithm]

I've been seeing reports that clipboard operations can take a long time in WSL. While moving copy/paste to Cmds won't solve this issue, it's probably worth doing it regardless, both because clipboard stuff is I/O, and to provide a unified copy/paste API to all Bubble Tea applications.

Note that we also discussed this in @aymanbagabas's upcoming Kitty Keyboard work.

[ajayd-san]

Hey can I take this up ?

[meowgorithm]

Sounds good to me. @aymanbagabas, whatcha think?

[aymanbagabas]

I think that would be wonderful to add to Bubble Tea!

Looking at implementation details, there are a couple of golang clipboard libraries such as https://github.com/atotto/clipboard and https://github.com/golang-design/clipboard, but each has its own shortcomings. The former always assumes programs running locally (no remote sessions support) and the latter uses CGO.

Given this, I'm proposing porting atotto/clipboard to bubbletea with a few changes to recognize the program environment variables and use OSC52 when working with remote sessions like SSH. Something along the lines of this API

type ClipboardWriteMsg bool

func ClipboardWrite(text string) tea.Cmd {
  return func() tea.Msg {
    err := copyClipboard()
    return ClipboardWriteMsg(err == nil)
  }
}

type ClipboardReadMsg string

func ClipboardRead() tea.Cmd {
  return func() tea.Msg {
    text := readClipboard()
    return ClipboardReadMsg(text)
  }
}
  

[ajayd-san]

Hey, So tell me if I'm going in the right direction.

I'm thinking of adding ClipboardRead and ClipboardWrite functions in commands.go and I'll be using atotto/clipboard for normal sessions and x/clipboard for ssh session.

How do i know if I'm in a ssh session? Check if SSH_CLIENT or SSH_TTY is set in p.environ.

Does this look good?

[aymanbagabas]

I'll be using atotto/clipboard for normal sessions

That wouldn't work because atotto/clipboard uses os.Getenv() instead of p.environ to determine environment variables. We need to port atotto/clipboard to bubbletea and make it use the program's environment variables.

How do i know if I'm in a ssh session? Check if SSH_CLIENT or SSH_TTY is set in p.environ.

Correct, check for SSH_TTY to determine if the remote session is a terminal and use ansi.SetClipboard to use OSC52.

[ajayd-san]

Aight I'll fork atotto/clipboard make the required changes, and get back to you when I'm done.

[ajayd-san]

So, I've ported atotto/clipboard to use user defined env vars (p.environ) instead of os.GetEnv() :
ajayd-san/clipboard@b53899f

Have a look and let me know if it looks good.

[ajayd-san]

Hey, Can anyone tell me if I'm going in the right direction? Any input would be appreciated. Thanks