Swap keyboard shortcut behavior in Launcher menu

[jbecke]

Change default launcher shortcut behavior to open items in a new split. Holding option now opens in the current split instead.

[github-actions]

Preview: https://claude-swap-launcher-shortcuts-eszg0x.preview.macro.com/app (1e90320)

[claude]

Code review

Missing tests for behavioral changes

This PR changes the keyboard shortcut behavior in the Launcher component by inverting the shouldInsert logic, which affects when items open in a new split vs. current split. According to CLAUDE.md Rule 4, tests should be written for changes like this.

Issue: No tests were added for this behavioral change affecting 7 different launcher actions:

• Note creation
• Email compose
• Channel compose
• Chat creation
• Document creation
• Folder creation
• Spreadsheet creation

Why this matters: The codebase already has patterns for testing similar keyboard modifier behavior (see openInNewSplitForMention.test.ts). This behavioral change affects user workflows and could regress without test coverage.

Suggested fix: Add unit tests that verify the shouldInsert parameter is correctly set based on the pressedKeys().has('opt') state for these launcher actions.