Skip to content

[chore] Retire legacy Agenta 1.0 onboarding - #5087

Merged
mmabrouk merged 1 commit into
big-agentsfrom
chore/retire-legacy-onboarding
Jul 6, 2026
Merged

[chore] Retire legacy Agenta 1.0 onboarding#5087
mmabrouk merged 1 commit into
big-agentsfrom
chore/retire-legacy-onboarding

Conversation

@mmabrouk

@mmabrouk mmabrouk commented Jul 6, 2026

Copy link
Copy Markdown
Member

Context

The playground-native agent onboarding merged in #5076 replaces the pre-pivot onboarding. Several old surfaces were still live or mounted: the /get-started three-option page, the home WelcomeCards banner, a floating checklist widget, a guided-tour engine, and a sidebar "Get Started Guide" item. New users were still routed through /get-started.

Changes

Removed outright (the pivot makes them dead):

  • The /get-started three-option page (OSS and EE) and its views.
  • The home WelcomeCards banner and the now-orphaned old app-management home page (nothing imports it; /apps renders agent-home).
  • The EE demo-applications section.

Parked behind kill switches, code kept for possible reuse:

  • Sidebar "Get Started Guide" item hidden behind a constant.
  • The onboarding checklist widget is no longer mounted, and its new-user auto-open is disabled.
  • The nextstepjs guided-tour engine is gated behind ONBOARDING_TOURS_ENABLED (false), with the provider loaded via next/dynamic so the disabled path never runs tour logic.

New users now route straight to the resolved post-login path (/apps) instead of /get-started.

Scope / risk

Frontend only. The parked widget and tour code stay in the tree. Flip the constants to revive them.

Merge order: merge the survey PR #5085 (feat/onboarding-survey-v2) BEFORE this one. This PR deletes the /get-started route, and #5085 is what stops the post-signup flow from redirecting there. Merging this first would briefly point post-signup redirects at a deleted route on big-agents.

Follow-up (not blocking): because the tour hook is in the _app import chain, nextstepjs is still bundled (never executed) while tours are off. Fully excluding it from the bundle is a small later refactor.

Tests / notes

  • pnpm lint-fix green; types:check shows no new errors.
  • Verified the dev web compiles and serves. Playground still works with the tour glue turned into a no-op.

How to QA

Prerequisites: EE dev stack.

Steps:

  1. Sign up or sign in as a new user.
  2. Observe the landing page and the sidebar.

Expected result: You land on /apps (agent-home), not /get-started. No welcome banner, no floating checklist, no "Get Started Guide" sidebar item. The playground opens normally.

Edge cases: Existing users with onboarding localStorage keys should see no errors. The tour provider must render its children unchanged when disabled.

Automated tests: None specific. Verified via lint, types, and a manual dev-stack check.

https://claude.ai/code/session_01AuS2c7XQhyECUGg3afeZQ9

Remove the pre-pivot onboarding surfaces the new playground-native flow
replaces: the /get-started three-option page (OSS + EE), the home
WelcomeCards banner and the now-orphaned old app-management home page,
and the EE demo-applications section.

Park the reusable pieces behind kill switches instead of deleting them:
hide the sidebar Get Started Guide item, stop mounting the onboarding
checklist widget (and disable its new-user auto-open), and gate the
nextstepjs guided-tour engine behind ONBOARDING_TOURS_ENABLED so it never
runs. Route new users straight to the resolved post-login path instead of
/get-started.

Claude-Session: https://claude.ai/code/session_01AuS2c7XQhyECUGg3afeZQ9
@dosubot dosubot Bot added size:XXL This PR changes 1000+ lines, ignoring generated files. frontend labels Jul 6, 2026
@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e4fb8c3c-1222-4591-adc5-ac41baa7180d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • ✅ Review completed - (🔄 Check again to review again)
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/retire-legacy-onboarding

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

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

Labels

frontend size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant