[rnambaale]

Description

Adds the frontend for the backend's new GET /search endpoint (keyword mode): a search field in the top navigation that searches coaching sessions, goals, actions, agreements and topics and links each hit to where it lives in the app. Results render in a dropdown under the field with keyboard navigation, load-more pagination, and inline loading, empty and error states.

GitHub Issue: [Resolves] refactor-group/refactor-platform-rs#404

Changes

• src/types/search.ts - hit tagged union with Option-wrapped nullable fields, typed error codes, a parser that drops unknown hit types, and splitSnippet so <mark> markers render as text nodes, never HTML.
• src/lib/api/search.ts - SearchApi.search returning a neverthrow ResultAsync<SearchResponse, SearchError>; useSearch hook with 250 ms debounce, AbortController cancellation, reset to page one on query or filter change, and cursor-based loadMore.
• src/lib/utils/search-hit-href.ts - maps hits to routes: sessions to their page, actions to the actions panel with highlight, goals to their creating session's goals panel or the dashboard, agreements and topics to their session.
• src/components/ui/search/ - GlobalSearch (cmdk-based field, ⌘K / / focus shortcut, skeleton loading rows, empty and error states) and SearchSnippet.
• src/components/ui/site-header.tsx - renders the field between the sidebar toggle and the right-hand nav.
• src/components/ui/command.tsx - fixes the shared CommandItem disabled selector. cmdk 1.x always emits data-disabled="false", so data-[disabled] matched every item and made results unclickable; now data-[disabled=true].
• Tests for the parser, URL helper, hook (debounce, abort, error mapping, load more, filter reset) and component.

Screenshots / Videos Showing UI Changes

---

Testing Strategy

• npm run test — new suites under __tests__/types/search.test.ts, __tests__/lib/utils/search-hit-href.test.ts, __tests__/lib/api/search.test.tsx, __tests__/components/ui/search/.
• Manually against a backend with the search PR: type two or more characters in the header field, confirm results appear after a short pause, click and Enter both navigate, Escape clears, ⌘K focuses the field from outside an input.
• Verify the trailing-space opt-out of prefix matching still works (plann vs plann).
• Verify agreement and topic hits are clickable; they are rendered disabled if the response lacks coaching_session_id.

Concerns

describe any concerns that might be worth mentioning or discussing