feat(ai): Librarian Agent web UI — Ask the librarian (AI-Agent-3)#399
Open
mrviduus wants to merge 1 commit into
Open
feat(ai): Librarian Agent web UI — Ask the librarian (AI-Agent-3)#399mrviduus wants to merge 1 commit into
mrviduus wants to merge 1 commit into
Conversation
Frontend for the shipped Librarian agent: NL book-discovery at /:lang/discover. Query -> POST /me/librarian -> the agent's reasoning + ranked recs. Library recs link to the book page; external (open_library) recs are clearly-marked suggestions with NO in-app nav (recommend-only — never masquerade as a catalog book). usedExternal note when it reached beyond the catalog. New DiscoverPage, useLibrarian state machine, LibrarianResults/Card, librarian.ts client. Reuses Tutor-UI hardening (mounted-guard/abort, same-query retry, clamped LLM strings). Auth-gated, noindex. QA: 0 blockers (null-slug -> no broken link; external -> never a link; both test-covered). tsc clean; 595 web tests green (11 new); vite build green; browser-checked (reasoning + library link + marked external + usedExternal + edge states, 0 console errors). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The web UI for the Librarian agent — surfaces a shipped agent that had no UI (architect → frontend → adversarial QA cycle).
What
/:lang/discover(entry from the Discover menu). NL query →POST /me/librarian→ the agent'sreasoning+ ranked recommendation cards (title, authors, per-itemwhy)./:lang/books/{slug}. External (open_library) recs → clearly-marked "Suggestion — not in your library yet", no in-app navigation (backend is recommend-only; an external suggestion must never look like a real catalog book).usedExternalnote when it reached beyond the catalog.useLibrarianstate machine (idle→asking→results|empty|error), client query guard mirroring the backend (≥2 / ≤500).AbortController, same-queryretry(), line-clamped + React-escaped LLM strings. Auth-gated (signed-out → auth modal), SEO-noindexed.QA (adversarial) — 0 blockers / 0 should-fixes
The two real risks both guarded + test-covered: a
libraryrec with a nullslug→ falls back to a non-link card (no/books/undefined); anopen_libraryrec → never renders a link. Only minor/info (pages pluralization matches existing patterns).Verify
tscclean · 595 web tests green (11 new) ·vite buildgreen · browser-checked (query → reasoning + library link + marked external suggestion + usedExternal + empty / error / min-query-blocked / signed-out, 0 console errors).Deferred
Mobile Librarian UI, SSE streaming, "add external book to my library"/ingest, pagination.
🤖 Generated with Claude Code