[BUG] New keyboard shortcut conflicts with Redo standard shortcut

[swythan]

Problem (one or two sentences)

PR #7908 added a mapping from Ctrl+Y (on Windows, similar on other platforms) to be "Add to Context".

Ctrl+Y is a standard shortcut for the Redo operation. See the VS Code Default Keyboard Shortcuts docs (where it is 12th on the list).

Context (who is affected and when)

Anyone with Roo Code installed who is used to using Ctrl+Y as a shortcut for Redo in the editor.

Reproduction steps

1. Install Roo Code
2. Open a text file in a workspace
3. Make some edits in the file
4. Use Ctrl+Z to undo the edits
5. Use Ctrl+Y to (attempt to) redo the edits

Expected result

Edits removed via Undo are restored (via Redo)

Actual result

A reference to the current cursor position is added to the Roo Code input window

Variations tried (optional)

No response

App Version

3.28.16

API Provider (optional)

None

Model Used (optional)

No response

Roo Code Task Links (optional)

No response

Relevant logs or errors (optional)

I'm exploring a potential fix for this keyboard shortcut conflict where Ctrl+Y interferes with the standard Redo command. I'll investigate the keybinding configuration and work on a solution that preserves both functionalities.

[swythan]

The keyboard shortcut can be removed via "Preferences: Open Keyboard Shortcuts":

Or via "Preferences: Open Keyboard Shortcuts (JSON)":

    {
        "key": "ctrl+y",
        "command": "-roo-cline.addToContext",
        "when": "editorHasSelection && editorTextFocus"
    }

I've opened PR #8653 to address this keyboard shortcut conflict: #8653

The fix changes the "Add to Context" shortcut from Ctrl+Y/Cmd+Y to Ctrl+K Ctrl+A (Cmd+K Cmd+A on Mac), which follows VS Code's chord pattern and won't interfere with the standard Redo operation.

All CI checks have passed successfully. Feedback welcome!

[swythan]

Thanks.

That binding looks OK to me (at least with the out-of-the-box key bindings). The only binding I have for that chord currently looks like it is only active in the keybinding editor itself.

{ "key": "ctrl+k ctrl+a",         "command": "keybindings.editor.addKeybinding",
                                     "when": "inKeybindings && keybindingFocus" },

[mrubens]

Thanks for this @swythan, that chord looks good to me.