hermes - 💡(How to fix) Fix cli.py crashes on startup in WSL terminals that don't support c-S-c keybinding [2 comments, 3 participants]

TL;DR

The issue can be resolved by wrapping the @kb.add('c-S-c') block in a try/except to catch and silently ignore exceptions when the terminal doesn't support the Ctrl+Shift+C keyboard shortcut.

Guidance

• Verify that the terminal emulator is handling the Ctrl+Shift+C shortcut for copy, which might be the cause of the conflict.
• Implement the suggested fix by wrapping the @kb.add('c-S-c') block in a try/except to prevent the CLI from crashing.
• Consider logging the exception for debugging purposes, rather than completely ignoring it, to help identify any other potential issues.
• Test the fix on different WSL terminal emulators to ensure the solution is robust.

Example

try:
    @kb.add('c-S-c')  # Ctrl+Shift+C
    def handle_ctrl_shift_c(event):
        """Copy text to clipboard (terminal-native)."""
        return
except Exception as e:
    print(f"Warning: Failed to register Ctrl+Shift+C shortcut: {e}")

Notes

This fix assumes that silently skipping the unsupported keyboard shortcut is the desired behavior. If the shortcut is essential, an alternative solution might be needed.

Recommendation

Apply the workaround by implementing the try/except block, as it provides a straightforward and non-intrusive fix for the issue.