Skip to content

Restore first-run welcome before provider setup - #717

Closed
TheGreatAxios wants to merge 1 commit into
mainfrom
hotfix-first-run-welcome
Closed

Restore first-run welcome before provider setup#717
TheGreatAxios wants to merge 1 commit into
mainfrom
hotfix-first-run-welcome

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Fresh unconfigured launches show the animated orange mountain and Corbits Code, your local software factory before provider/model setup.
  • Already-onboarded users still open setup directly; markOnboarded stamps only after the welcome gate completes.
  • Settings loaded for the provider write happen after welcome so a same-path stamp is not wiped by setup.

Verification

  • bun run check passes
  • Targeted: bun test ./src/tui/welcome.test.ts ./src/tui/onboarding.test.ts ./tests/unit/tui/onboarded-persistence.test.ts ./src/tui/provider-setup.test.ts ./src/tui/landing.test.ts ./src/tui/mark-anim.test.ts

Unconfigured launches jumped straight into model setup and skipped the
orange mountain plus product line. Gate setup on settings.onboarded,
show the welcome surface first, and stamp onboarded only after it
completes so returning users still open setup directly.
@TheGreatAxios

Copy link
Copy Markdown
Collaborator Author

Closing as a duplicate of #718, which already landed on main (same welcome/onboarding tree; this branch only still carries a stale 0.3.8 version bump and Unreleased changelog wording).

First-run welcome polish that is still worth doing (hold the filled mountain instead of looping into a fade, then auto-advance) will land on a new PR from current main rather than rebasing this conflicting branch.

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