-
Notifications
You must be signed in to change notification settings - Fork 432
Fix WebSocket utilities and Add OSC 52 clipboard support #2245
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
WalkthroughThe changes introduce a new function, Estimated code review effort🎯 3 (Moderate) | ⏱️ ~15 minutes Note 🔌 MCP (Model Context Protocol) integration is now available in Early Access!Pro users can now connect to remote MCP servers under the Integrations page to get reviews and chat conversations that understand additional development context. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
392488b
to
eebc2fc
Compare
- Fix WebSocket utilities timing issue: * Replace async import() with synchronous require() for Node.js 'ws' module * Prevents race condition where newWebSocket was called before import resolved * Fixes fallback to browser WebSocket in Node.js environment - Add handleOsc52Command() for set/query/clear clipboard operations - Register OSC 52 handler in TermWrap constructor - Support base64 encoding/decoding for clipboard data - Add error handling for clipboard access failures
eebc2fc
to
d769694
Compare
Fix WebSocket utilities timing issue:
Add handleOsc52Command() for set/query/clear clipboard operations
Register OSC 52 handler in TermWrap constructor
Support base64 encoding/decoding for clipboard data
Add error handling for clipboard access failures