Ability to remove trailing whitespace in neovim captures

[CaptaiNiveau]

Hey!

I love this plugin in my Kitty life, a significant improvement over the previous default binds from Kitty :)
Recently, I've been using the scrollback quite often to effectively bridge my copy paste buffer from remote VMs over SSH.
That means I open files in neovim over ssh and can't access my local clipboard. Then, to copy text, I use the scrollback plugin.
However, neovim prints all its lines with trailing whitespaces. Those also make it into the scrollback captured by this plugin.
I could potentially fix this with some workaround for myself locally, but maybe other people run into this issue, too.

What do you think would be a good way to deal with this?
Sometimes someone might need the trailing whitespace, maybe removing the trailing whitespaces in the output can be a command flag?
Probably not too hard to implement - if you think that this is a good idea, I could also try to create a PR.

Thanks!

[mikesmithgh]

Hey! So I actually had a similar issue in the past mostly around copying large auth tokens. I have an old draft PR #308 which is out of date but is probably what you are looking for. It joins all the new lines so the result is a single string. I'll revisit this to see if I can bring it up to date with main.

[CaptaiNiveau]

That also sounds useful and now that you mention it, I had that issue before, too.

My use case is copying for example docker-compose.yml. They do need their newlines, but having all the trailing white space is annoying. Even worse, copying multi-line commands with trailing \ break when there's whitespace afterwards. So maybe both options would be good? :D

[mikesmithgh]

Oh I see now, I could probably add a hook to let users apply a function to the yank results to do things like this. That way it would be more flexible.

[CaptaiNiveau]

Yes, that sounds good ^^
I still need to wrap my head around the yank window, I'll probably read the docs now.