Skip to content

POC, not for merge: LA personalization settings tab - #4019

Draft
yodem wants to merge 8 commits into
masterfrom
poc/la-personalization-settings
Draft

yodem wants to merge 8 commits into
masterfrom
poc/la-personalization-settings

Conversation

@yodem

@yodem yodem commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator

POC, not for merge. Draft only.

Adds a tab bar to Settings and a Library Assistant tab at /settings/library-assistant. The Library Assistant on/off setting moves there from Account Settings. A panel reads and writes Instructions and Profile, which are stored in the chatbot service, using the user's signed chatbot token.

  • Only the tab styling is taken from the API-keys settings work; nothing else from it
  • English UI text only on this branch; no translation files touched
  • Keyboard-accessible On/Off controls

Pairs with the ai-chatbot draft PR for LA personalization phase 1. Plan, design and build log are in the Sefaria wiki under wiki/projects/la-personalization/.

Tests on the branch: Jest 227 passed; reader/tests/library_assistant_setting_test.py 26 passed.

🤖 Generated with Claude Code

https://claude.ai/code/session_01McshJKHz87vJURqXt4nyJS

yodem and others added 8 commits November 12, 2026 16:34
…, not for merge)

Adds a tab bar to Settings and a Library Assistant tab at /settings/library-assistant.
The Library Assistant on/off switch moves there from Account Settings. The panel reads
and writes Instructions and Profile in the chatbot service with the user's signed token.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01McshJKHz87vJURqXt4nyJS
…onses, no updates after unmount

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01McshJKHz87vJURqXt4nyJS
… page switch while saving

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01McshJKHz87vJURqXt4nyJS
…me for the Library Assistant settings tab

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01McshJKHz87vJURqXt4nyJS
…ackground clear of text

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01McshJKHz87vJURqXt4nyJS
…rries it

The view built its props before the context processor applied the query
parameter to the session, so the first request of a session had no version
and the settings panel called the default chatbot. Apply the parameter in
one shared helper from base_props too, and let chatbotApiBaseUrl fall back
to the page URL (integers only; clear means none).

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01McshJKHz87vJURqXt4nyJS

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant