Skip to content

update onboarding#410

Merged
graycyrus merged 3 commits into
tinyhumansai:mainfrom
graycyrus:feat/move-tools-to-settings
Apr 7, 2026
Merged

update onboarding#410
graycyrus merged 3 commits into
tinyhumansai:mainfrom
graycyrus:feat/move-tools-to-settings

Conversation

@graycyrus
Copy link
Copy Markdown
Contributor

@graycyrus graycyrus commented Apr 7, 2026

Summary by CodeRabbit

Release Notes

  • Refactor
    • Simplified the onboarding flow by reducing it from 4 steps to 3
    • Removed the local model configuration step and associated error handling
    • Reorganized the sequence of remaining onboarding steps
    • Updated onboarding task persistence with revised defaults

Remove the Tools toggle step from onboarding (5 → 4 steps) and add it
as a dedicated Tools panel under Settings > AI & Skills. This reduces
onboarding friction while keeping tool configuration accessible.
Remove the Local AI download/consent step, reducing onboarding from
4 to 3 steps (Welcome → Screen Permissions → Skills). Local AI setup
is already accessible from Settings and auto-bootstraps from Home.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 7, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 34944808-967f-497b-8cfb-4f7a9250104e

📥 Commits

Reviewing files that changed from the base of the PR and between db7eeee and af34703.

📒 Files selected for processing (1)
  • app/src/pages/onboarding/Onboarding.tsx

📝 Walkthrough

Walkthrough

The Local AI onboarding step and its associated client-side state management have been removed from the Onboarding component. The onboarding flow is condensed from 4 to 3 steps, with remaining steps shifted to new indices. Local model consent and download state now default to false in task persistence.

Changes

Cohort / File(s) Summary
Local AI Step Removal
app/src/pages/onboarding/Onboarding.tsx
Deleted LocalAIStep imports, rendering case, and related state fields (localModelConsentGiven, localModelDownloadStarted). Removed error handling UI (portal banner, auto-dismiss timer, retry handler, refs). Updated step indices: ScreenPermissionsStep shifted to index 1, SkillsStep to index 2. Modified setOnboardingTasks to write hardcoded false values for local model fields instead of deriving from step state.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 One step, two step, three steps now,
Local AI takes a bow,
Simpler flows make spirits bright,
Onboarding's lean and oh so right! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@graycyrus graycyrus merged commit c3dd137 into tinyhumansai:main Apr 7, 2026
7 of 8 checks passed
AusAgentSmith pushed a commit to AusAgentSmith/openhuman that referenced this pull request May 23, 2026
* feat(onboarding): move Tools step from onboarding to Settings

Remove the Tools toggle step from onboarding (5 → 4 steps) and add it
as a dedicated Tools panel under Settings > AI & Skills. This reduces
onboarding friction while keeping tool configuration accessible.

* feat(onboarding): remove Local AI step from onboarding

Remove the Local AI download/consent step, reducing onboarding from
4 to 3 steps (Welcome → Screen Permissions → Skills). Local AI setup
is already accessible from Settings and auto-bootstraps from Home.
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