fix(shortcuts): match single-letter hotkeys by layout-aware key

[haacked]

Problem

On non-QWERTY keyboard layouts (Dvorak, AZERTY, …), single-letter shortcuts fire based on the physical key position rather than the letter the user actually typed. The reported case: selecting text in the chat composer and pressing Cmd+C navigates to the Inbox instead of copying. On Dvorak the key labelled "C" sits at the physical QWERTY-"I" slot, and the Inbox shortcut is mod+i.

Root cause: react-hotkeys-hook@4.6.2 matches a hotkey's key against both the layout-aware event.key and the physical event.code (mapKey(code)). For mod+i it fires whenever event.code === "KeyI", regardless of the logical letter. Every single-letter mod+<letter> shortcut (mod+k, mod+n, mod+b, mod+w, mod+o, mod+r, mod+s, mod+f, mod+j, …) has the same problem. The library version in use has no option to disable code-based matching (useKey only exists in v5).

Closes #1797

Changes

Patch the react-hotkeys-hook matcher (patches/react-hotkeys-hook.patch, wired via pnpm patchedDependencies) to derive the pressed key from the layout-aware event.key. It falls back to the physical event.code only when a modifier (e.g. macOS Option) turns event.key into a non-alphanumeric glyph or dead key — so special keys (arrows, escape, space) and Option shortcuts keep working, while single-letter shortcuts now match the printed letter on every layout.

No call sites change; the fix applies to all current and future useHotkeys usages.

How did you test this?

Added packages/ui/src/features/command/hotkeysLayout.test.ts, a Vitest regression guard that renders useHotkeys("mod+i", …) and dispatches keydown events:

• QWERTY Cmd+I (key: "i", code: "KeyI") → fires.
• Dvorak Cmd+C (key: "c", code: "KeyI") → does not fire (the bug that hijacked copy).

Manually tested by switching my system keyboard layout to Dvorak and confirming Cmd+C now copies selected text instead of navigating to the Inbox. Also re-checked other mod+<letter> shortcuts on QWERTY to confirm no regression.

Automatic notifications

• Publish to changelog?
• Alert Sales and Marketing teams?

---

Created with PostHog from a Slack thread

[trunk-io]

✨ Submitted to Merge by @haacked. It will be added to the merge queue once all branch protection rules pass, there are no merge conflicts with the target branch, and impacted targets for the current PR head commit have been uploaded. See more details here.

[github-actions]

React Doctor found no issues in the changed files. 🎉

_{Reviewed by React Doctor for commit d0b6475.}

[greptile-apps]

_{Reviews (1): Last reviewed commit: "fix(shortcuts): match single-letter hotk..." | Re-trigger Greptile}

[haacked]

/trunk merge