fix(tmux): use native OSC 52 for SSH clipboard copy#71
Merged
Conversation
…64 pipe The manual base64 | xargs printf pipeline broke clipboard copy on remote tmux sessions because Linux base64 wraps at 76 chars, causing xargs to generate multiple incomplete OSC 52 sequences. Additionally, /dev/tty may not be available in tmux server's daemonized copy-pipe subprocess. Replace with copy-selection-and-cancel and rely on tmux's native OSC 52 mechanism (set-clipboard external + terminal-features clipboard) which handles encoding and delivery correctly.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
base64 | xargs printfOSC 52 pipeline withcopy-selection-and-cancelfor SSH sessionsset-clipboard external+terminal-features clipboard) which correctly handles encoding and deliveryRoot cause
Linux
base64wraps output at 76 chars by default.xargs -I {}processed each line separately, generating multiple incomplete OSC 52 sequences. Additionally,/dev/ttymay be unavailable in tmux server's daemonized copy-pipe subprocess.Test plan
y— verify text lands in local clipboardMouseDragEnd1Pane) also works