[chore] Retire legacy Agenta 1.0 onboarding - #5087
Merged
Merged
Conversation
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
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
mmabrouk
added a commit
that referenced
this pull request
Jul 10, 2026
mmabrouk
added a commit
that referenced
this pull request
Jul 10, 2026
mmabrouk
added a commit
that referenced
this pull request
Jul 10, 2026
This was referenced Jul 18, 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
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.
Context
The playground-native agent onboarding merged in #5076 replaces the pre-pivot onboarding. Several old surfaces were still live or mounted: the
/get-startedthree-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):
/get-startedthree-option page (OSS and EE) and its views./appsrenders agent-home).Parked behind kill switches, code kept for possible reuse:
ONBOARDING_TOURS_ENABLED(false), with the provider loaded vianext/dynamicso 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-startedroute, 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
_appimport 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-fixgreen;types:checkshows no new errors.How to QA
Prerequisites: EE dev stack.
Steps:
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