feat(ui): command palette uses server-side book search #4

Merged

zombor merged 1 commit from bd-grimmory-n55.6 into develop 2026-05-11 00:53:46 +00:00

Conversation 0 Commits 1 Files changed 2 +239 -97

zombor commented 2026-05-11 00:13:21 +00:00

Owner

Copy link

Summary

Bead: grimmory-n55.6 (part of epic grimmory-n55 — Scale to tens of thousands of books)

command-palette.service.ts filtered this.bookService.books() in memory, which only works while the full catalog is loaded client-side. The epic's goal is to retire that global signal, so the palette switches to server-side search via the existing AppBooksApiService.searchBooks endpoint.

What changed

• command-palette.service.ts: 150ms debounce, size=20 cap, keepPreviousData for stale-while-loading. Preserves keyboard navigation, grouped sections, and select-to-navigate. isSearching reflects both debounce-pending and in-flight state.
• command-palette.service.spec.ts: rewritten with HttpTestingController. Covers debounce coalescing (no per-keystroke fetch), empty/1-char no-network, graceful error handling, selection-navigates, audiobook thumbnails.

Test plan

• 12/12 feature specs pass.
• Pre-existing magic-shelf date failures reproduce on develop (unrelated to this PR).
• just ui check — not run by agent; CI will verify.
• Manually exercise the palette in dev against the prod server: type queries, observe single debounced request, navigate via keyboard.

## Summary Bead: **grimmory-n55.6** (part of epic **grimmory-n55** — Scale to tens of thousands of books) `command-palette.service.ts` filtered `this.bookService.books()` in memory, which only works while the full catalog is loaded client-side. The epic's goal is to retire that global signal, so the palette switches to server-side search via the existing `AppBooksApiService.searchBooks` endpoint. ## What changed - `command-palette.service.ts`: 150ms debounce, `size=20` cap, `keepPreviousData` for stale-while-loading. Preserves keyboard navigation, grouped sections, and select-to-navigate. `isSearching` reflects both debounce-pending and in-flight state. - `command-palette.service.spec.ts`: rewritten with `HttpTestingController`. Covers debounce coalescing (no per-keystroke fetch), empty/1-char no-network, graceful error handling, selection-navigates, audiobook thumbnails. ## Test plan - [x] 12/12 feature specs pass. - [ ] Pre-existing magic-shelf date failures reproduce on `develop` (unrelated to this PR). - [ ] `just ui check` — not run by agent; CI will verify. - [ ] Manually exercise the palette in dev against the prod server: type queries, observe single debounced request, navigate via keyboard.

zombor added 2 commits 2026-05-11 00:13:21 +00:00

bd init: initialize beads issue tracking d728fd0780

feat(ui): command palette uses server-side book search ... 8ba81e753f

Replace the in-memory filterBooksBySearchTerm against bookService.books()
with a debounced TanStack query against /api/v1/app/books/search. The
palette no longer depends on the global catalog signal, advancing the
n55 epic toward retiring bookService.books().

- 150ms debounce + distinctUntilChanged on the trimmed query
- 20-result cap per call (?size=20)
- keepPreviousData so stale results stay on screen while a new request
  is in flight; isSearching also reflects in-flight fetches so the
  empty-state message is suppressed during loading
- search errors are swallowed into an empty page so the palette stays
  usable; navigation, grouped sections, and keyboard nav are untouched
- spec rewritten with HttpTestingController: debounce coalescing, empty
  query bypass, error graceful fallback, selection-navigates, audiobook
  thumbnails

Bead: grimmory-n55.6

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

zombor referenced this pull request 2026-05-11 00:48:31 +00:00

chore: bd init + Claude Code project assets #5

zombor merged commit cfaa082c6b into develop 2026-05-11 00:53:46 +00:00

zombor deleted branch bd-grimmory-n55.6 2026-05-11 00:53:47 +00:00

zombor referenced this pull request from a commit 2026-05-11 00:53:47 +00:00

Merge pull request 'feat(ui): command palette uses server-side book search' (#4) from bd-grimmory-n55.6 into develop

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

zombor/grimmory!4

No description provided.