feat(clipboard): implement GFN paste protocol

[zortos293]

This PR implements official-style clipboard paste for OpenNOW using the GFN PASTE custom-message protocol over the data channel, with client-side availability advertisement and server request/response handling that mirrors the official JavaScript client's behavior.

• Added src/renderer/src/gfn/clipboardProtocol.ts implementing the GFN PASTE custom-message envelope, CLIENT_ADDED_DATA/CLIENT_REMOVED_DATA/CLIENT_DATA_RESPONSE and SERVER_DATA_REQUEST message types, trace ID generation, and UTF-8 byte size clamping.
• Added src/renderer/src/gfn/clipboardProtocol.test.ts with 3 test cases validating message construction and parsing.
• Integrated clipboard availability advertisement in src/renderer/src/gfn/webrtcClient.ts via setClipboardPasteEnabled and refreshClipboardAvailability, and server data request handling in onControlChannelMessage, with control channel open/close lifecycle hooks.
• Updated src/renderer/src/App.tsx to call client.pasteClipboardText() before falling back to sendText/sendPasteShortcut, and pass clipboardPaste/readClipboardText options to the client, with runtime setting changes propagated via setClipboardPasteEnabled.

Verification

• npm test -- clipboardProtocol (3 tests pass, 148 total)
• npm run typecheck (no errors)
• npm run lint (0 errors, 35 pre-existing warnings unrelated to this change)

---

Note

Medium Risk
Changes streaming paste behavior and sends clipboard content over the control channel when enabled, though it remains behind the clipboardPaste setting with legacy fallbacks.

Overview
Adds official GFN clipboard paste over the WebRTC control_channel using the PASTE custom-message protocol, so paste can follow the same request/response flow as the official client instead of only injecting text or simulating shortcuts.

A new clipboardProtocol module builds and parses the PASTE envelope (CLIENT_ADDED_DATA / CLIENT_REMOVED_DATA / CLIENT_DATA_RESPONSE / SERVER_DATA_REQUEST), generates trace IDs, and clamps clipboard text by UTF-8 byte size (with unit tests).

GfnWebRtcClient advertises clipboard availability when the control channel opens and on window focus, answers server data requests with host clipboard text, and exposes pasteClipboardText() (refresh availability then trigger paste). The clipboardPaste setting is passed in at connect and can be toggled live via setClipboardPasteEnabled.

App routes paste through client.pasteClipboardText() first, then falls back to sendText / sendPasteShortcut. Paste shortcut detection now ignores Shift and treats Ctrl/Cmd consistently; fallback paste shortcut always uses Ctrl.

^{Reviewed by Cursor Bugbot for commit eba0fd1. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: f845c444-c983-49e5-ba73-b960c8ba9665

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch capy/feat-gfn-clipboard-paste

---

_{Comment @coderabbitai help to get the list of available commands.}

[cursor]

Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit eba0fd1. Configure here.}

[cursor]

Oversized clipboard skips sendText

Medium Severity

When the control channel is open, pasteClipboardText returns true after sending the paste shortcut even if clampClipboardText rejected the clipboard (for example over clipboardMaxBytes). sendStreamClipboardPaste then exits early and never uses sendText, which previously could still inject the full local clipboard via chunked text input.

Additional Locations (2)

• opennow-stable/src/renderer/src/gfn/webrtcClient.ts#L1068-L1076
• opennow-stable/src/renderer/src/gfn/clipboardProtocol.ts#L51-L56

 

^{Reviewed by Cursor Bugbot for commit eba0fd1. Configure here.}

[cursor]

Empty clipboard sends paste shortcut

Medium Severity

With an open control channel, pasteClipboardText always calls sendPasteShortcut and reports success, even when local clipboard text is empty after clampClipboardText. The old sendStreamClipboardPaste path returned after a successful read without injecting Ctrl+V when there was no text.

Additional Locations (1)

• opennow-stable/src/renderer/src/App.tsx#L349-L353

 

^{Reviewed by Cursor Bugbot for commit eba0fd1. Configure here.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: eba0fd1fc0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Honor unavailable clipboard before sending paste

When the official paste path is available but refreshClipboardAvailability() returns false (for example the local clipboard is empty, unreadable, or exceeds clipboardMaxBytes), this still sends Ctrl+V and reports success to sendStreamClipboardPaste(). That suppresses the existing text-input fallback and can cause the remote session to paste stale remote clipboard contents even though the client just advertised CLIENT_REMOVED_DATA; return false or skip the shortcut when no clipboard data is available.

Useful? React with 👍 / 👎.