feat(web-ui): global command palette (Cmd/Ctrl+K)

[dimaldim]

Related issue

Closes #1385

Summary

Add a global command palette opened with Cmd+K (Ctrl+K on Windows/Linux), available across the web UI and the desktop (Electron) app. It has two groups:

• Actions: New chat, Go to Inbox, Go to Settings, Toggle conversations sidebar, Toggle workspace sidebar, Open keyboard shortcuts. Filtered client-side as you type.
• Sessions: fuzzy session switching from the same server-search source the sidebar uses (useConversations -> GET /v1/sessions?search_query=, debounced), so it finds sessions beyond the first page rather than client-filtering a single page. Archived sessions excluded, matching the sidebar default.

The hotkey is bound once in AppShell. It does not fire when focus is inside an xterm terminal or the Monaco editor (both own Cmd/Ctrl+K), and is disabled in embedded mode where Cmd/Ctrl+K belongs to the host page. The desktop app loads the same SPA and binds only Cmd/Ctrl+N and Cmd/Ctrl+F natively, so Cmd/Ctrl+K reaches the renderer unchanged. Adds an "Open command palette" row to the keyboard-shortcuts dialog.

Complements PR #1064 (Cmd/Ctrl+Shift+F sidebar search): that PR reserved Cmd/Ctrl+K for exactly this palette. The two are additive and the palette coexists with the existing sidebar search.

Type of change

• Bug fix
• Feature
• Refactor / chore
• Docs
• Test / CI
• Breaking change

Test coverage

• Unit tests added / updated
• Integration tests added / updated
• E2E tests added / updated
• Manual verification completed
• Existing tests cover this change
• Not applicable

Coverage rationale

Vitest unit tests:

• useCommandPaletteHotkey.test.tsx: the chord predicate (Cmd/Ctrl+K, rejecting Alt/Shift/AltGraph/other keys), auto-repeat ignore, disabled mode, bail when focus is in an xterm/Monaco surface, and listener cleanup on unmount.
• CommandPalette.test.tsx: action commands render, run, and close the palette (navigation, sidebar-toggle callbacks, keyboard-shortcuts); client-side action filtering; session list mapping with the "New session" fallback; debounced server search (archived excluded); page dedupe; and the empty state.
• KeyboardShortcutsDialog.test.tsx: the new "Open command palette" row.

Playwright e2e (tests/e2e_ui/sessions/test_command_palette.py): Cmd/Ctrl+K opens the palette from a focused composer, selecting a session navigates to it, and the palette closes. It selects from the default session list rather than typing-to-filter, because the server search reindex is asynchronous and would make a type-then-filter assertion timing-dependent (search filtering is covered by the unit tests).

Also adds a no-op ResizeObserver polyfill to test-setup.ts (cmdk constructs one on mount; jsdom does not implement it), guarded so it never clobbers a real implementation or the per-test stubs other suites set.

Validated locally (from ap-web/):

• npm run type-check
• npm test (3158 passed, 3 expected fail, 2 skipped)
• npm run build
• npx oxlint on the touched files (clean; full npm run lint reports pre-existing errors outside this change)

E2E (repo root, venv active): python -m pytest tests/e2e_ui/sessions/test_command_palette.py --ui-skip-build (passes). Manually verified in a real browser and in the desktop app.

[daniellok-db]

lg, thanks for working on this!

[github-actions]

🏷️ Doc impact: needs-doc-update

Adds a new user-facing command palette (⌘/Ctrl+K) with actions and session switching, plus a new documented keyboard shortcut, affecting the web UI usage/keyboard-shortcuts docs.

Drafting a docs PR to omnigent-ai/omnigent-site (staged on 0.5-docs until release)…

_{Auto-classified on merge. Set the label manually before merging to override. · run}

[omnigent-ci]

Polly AI Review

Missing visual demonstration

This PR adds a user-visible feature — a global command palette (⌘/Ctrl+K) with an Actions group, a Sessions group, and a new keyboard-shortcuts row. The "Attached images/videos" scan reports (none found), and the PR checks "UI / frontend" work implicitly (it's a feature touching the web UI shell). Please attach a screenshot or short video of the palette open (showing both groups and the empty/searching state) so reviewers can confirm the appearance and behavior without checking out the branch.

Blocking issues

None. I verified the key contracts against the codebase:

• useConversations(searchQuery = "", includeArchived = false, …) matches the palette's useConversations(debouncedQuery, false) call and the useConversations("", false) cache-sharing claim.
• conversationDisplayLabel / getConversationAgentType exist in sidebarNav.ts and behave as the tests assert (null title → UNTITLED_CONVERSATION_LABEL).
• CommandInput in components/ui/command.tsx spreads ...props, so data-testid, value, and onValueChange reach the underlying cmdk input.
• useIsEmbedded() correctly gates both the hotkey (!isEmbedded) and the mounted <CommandPalette>.
• CommandItem values are unique and namespaced (action:<id> vs raw session id), avoiding cmdk selection collisions; page dedupe is handled explicitly.

Security vulnerabilities

None. This is client-side navigation and hotkey handling only; no new network surface, no injection, no auth/permission changes. Session ids are used only to build client routes.

Non-blocking notes

• Ctrl+K hijacks the macOS "kill-to-end-of-line" editing shortcut. isCommandPaletteHotkey accepts metaKey || ctrlKey, and focusOwnsHotkey() only bails for .xterm / .monaco-editor. On macOS, Ctrl+K in a normal text field (e.g. the composer) is the standard emacs-style "delete to end of line"; this hook will preventDefault it and open the palette instead. Consider restricting the Ctrl variant to non-mac, or letting Ctrl+K pass through when focus is in a plain editable field on macOS. Minor, but a real regression for mac keyboard users.
• Path/description drift. The e2e docstring and the PR description reference ap-web/src/..., while the actual tree is web/src/.... Harmless, but worth aligning to avoid confusing future readers.
• Actions filter uses query while sessions use debouncedQuery. Intentional and reasonable (actions are static/local), but it does mean the two groups update on slightly different cadences as the user types; acceptable.

Summary

A clean, well-scoped feature with strong test coverage (chord predicate, disabled/embedded/focus-bail cases, listener cleanup, action filtering, debounced server search, page dedupe, empty state, plus an e2e open→select→navigate flow) and a correctly-guarded ResizeObserver polyfill. I found no blocking correctness or security issues; the code matches the contracts it depends on. The only real item worth addressing before merge is the macOS Ctrl+K editing-shortcut conflict, and the PR should include a screenshot/video of the palette since it's a user-visible UI addition.

---

_{Automated review by Polly · workflow run}