Skip to content

Remove the finish-setup popup from the composer - #5291

Merged
SawyerHood merged 1 commit into
mainfrom
bb/disable-onboarding-popup-thr_j44waq3caq
Oct 9, 2026
Merged

SawyerHood merged 1 commit into
mainfrom
bb/disable-onboarding-popup-thr_j44waq3caq

Conversation

@SawyerHood

Copy link
Copy Markdown
Collaborator

Human comments

What was wrong

Skipping first-run setup could leave a “Finish setting up bb” reminder above the home composer. The reminder persisted through its own checklist setting and reopened unfinished onboarding steps.

What changed

Deleted the checklist component, its composer integration, and the setting from the app, server, CLI/SDK contract, and documentation. Removed its stories, tests, and unused onboarding reopen state. The first-run setup guide and Settings → General → Setup guide replay action remain available.

How you verified

  • Turbo typechecks for app, server, CLI, domain, and database passed; app lint passed with no errors.
  • 182 focused tests passed across onboarding/composer (93), database/settings migration (62), CLI settings (18), and server general settings (9).
  • In a fresh source dev app, clicked Skip setup, confirmed the prompt was present without the reminder, and reloaded to confirm it stayed absent. The server config also omitted the removed setting.

AGENT GENERATED

@SawyerHood
SawyerHood merged commit 9221061 into main Oct 9, 2026
48 checks passed
@SawyerHood
SawyerHood deleted the bb/disable-onboarding-popup-thr_j44waq3caq branch October 9, 2026 22:10
brsbl added a commit that referenced this pull request Oct 9, 2026
#5222 removed the setup checklist and banner after #5291 and added
useSetupComplete: the setup guide is finished or skipped and at least
one thread exists. Home sections now get experimental_setupComplete
from that hook. Plugin SDK moves to 0.6.39, one above the base's 0.6.38.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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