lsm / neokai / 25999287047 / 1

81%

dev: 81%

push

github

feat(web): add command palette (Cmd+K) with action registry (#1932)

* feat(web): add command palette (Cmd+K) with action registry

Wires a global command palette so every action is reachable from the
keyboard. The palette is mounted once in App.tsx and toggled via Cmd+K
(or Ctrl+K). Commands are looked up through a small registry that
supports fuzzy search and per-command keyboard shortcuts.

- `lib/command-registry.ts` — typed registry + subsequence fuzzy ranker
- `lib/default-commands.ts` — seeds session/navigation/space/settings
  actions on app mount
- `hooks/useGlobalShortcuts.ts` — global keydown listener; the palette
  toggle works inside text inputs, other shortcuts do not
- `islands/CommandPalette.tsx` — Dialog + Combobox composition from
  @neokai/ui, grouped by category with Escape-to-close
- `components/settings/ShortcutsSettings.tsx` — new Settings ▸ Shortcuts
  panel listing every command with a registered shortcut

* fix(web): address review feedback on command palette

- Pass target tab to navigateToSettings() in settings.* commands so
  the URL and signal stay aligned (it defaults to 'general' when called
  with no section argument).
- Ignore auto-repeat keydown events so a held shortcut fires once per
  physical press; protects non-idempotent commands like session.new.
- Restrict palette toggle to Cmd+K on macOS and Ctrl+K on non-mac so
  we don't hijack the native macOS Ctrl+K (kill-to-end-of-line) text
  editing shortcut.

* fix(web): address round-2 review feedback on command palette

- Register default commands at module-load time so the registry is
  populated before the palette can render.
- Add shortcut collision detection in CommandRegistry.register():
  warns when two commands share the same mod+key combo.
- Wrap command.run() in try/catch in both CommandPalette and
  useGlobalShortcuts so async rejections don't become unhandled.
- Add component tests for CommandPalette (open, search, empty state,
  async rejection boundary... (continued)

8336 of 11600 branches covered (71.86%)

Branch coverage included in aggregate %.

10704 of 13302 relevant lines covered (80.47%)

57.42 hits per line