feat(05): Onboarding wizard and profile switcher - #47
Open
tpaulshippy wants to merge 7 commits into
Open
Conversation
- UserAccount.onboarding_completed_at + onboarding_completed() heuristic fallback (hasPin && profiles>=1) for pre-wizard accounts - GET /api/user now returns onboardingCompleted - POST /api/user/onboarding/complete: idempotent, 403 for teen delegated sessions - POST /api/onboarding/bootstrap: renames/recreates default profile, updates/creates first bot, sets PIN via existing account path (until feature 02 hashing), marks complete; retry-safe matching by first profile/bot - get_delegated_tokens embeds is_teen_delegated claim so the API can recognize delegated sessions - seed_e2e_onboarding command: fresh-state 'e2e-test-user' plus second profile for switcher testing - Tests: bootstrap rename/create/pin/flag, GET reflects flag, teen delegated 403, seed idempotency
Wizard: - Gate in index.tsx: token && !onboardingCompleted && !teen-delegated routes to /onboarding; account fetch failures fall back to chat - 4 steps: welcome, kid name (pre-fills signal-created profile), first bot (Blank/Character template + appearance, defaults Penelope/teal/ sparkles), protect (PIN + confirm required, notifications optional) - Finish calls POST /onboarding/bootstrap, pre-selects the renamed profile and first bot so chat works immediately, then marks complete - Skip-resistant: no swipe/header-back on critical steps; forward only with valid input Profile switcher: - New ProfileSwitcher chip (avatar initial + name) on the Chats header and drawer; action sheet lists profiles with checkmark, switching updates selectedProfile storage without opening Settings - Manage profiles… is PIN-gated via PinWrapper; teen delegated sessions get a display-only chip Empty states & polish: - selectBot: 'Create a tutor' CTA when no bots exist - botChat: auto-selects first profile instead of dead-end message; forces onboarding/profile when none exist - flashcards: 'Go to chats' link on empty state - profilesList: router.dismiss() instead of double router.back() - settings: debug updateId gated behind __DEV__ Tests: wizard step transitions, gate routing incl. teen-delegated, switcher storage change + PIN gate + read-only mode
front/e2e/05-onboarding.e2e.js modeled on chatImageUpload.e2e.js: - fresh-state injection (tokens only, no profile/bot) gates app to /onboarding - welcome → kid name (pre-filled) → bot defaults → PIN + notifications prompt → finish lands in a chat that sends via the real API - verifies server state: profile renamed, single bot, pin + flag set - relaunch skips the wizard; switcher flow: chip → option tap → chip updates; Manage profiles… through the PIN gate into Profiles Adds menu-button/drawer-chats testIDs for navigation automation.
- POST /api/onboarding/bootstrap now echoes the exact profile_id and bot_id it configured, so the protect step can select them even though profile listings are name-ordered (the renamed default is not always results[0]) - protect.tsx matches fetched lists by returned id with first-item fallback - seed_e2e_onboarding resets the default/second profile names to Jordan/Maya each run for deterministic e2e pre-fill - tests: response id fields; wizard selects non-first listed profile
- Defer refreshSelected() and isTeenDelegatedSession() via microtask - Escape apostrophe with ' - Add jest globals to eslint config
tpaulshippy
pushed a commit
that referenced
this pull request
Aug 26, 2026
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.
Roadmap 05: Onboarding & Profile Switcher
Implements #57.
What changed
onboarding_completed_atfield,POST /api/user/onboarding/complete,POST /api/onboarding/bootstrap(atomic setup), teen-delegated claim in JWTsEvidence
https://github.com/tpaulshippy/bots/raw/pr-47-onboarding/evidence/pr47-onboarding.mp4