[feat] Agent-first sidebar for new signups, Playground as app default (v0.106.1) - #5536
Merged
Conversation
2 phase: 1. is hide the page 2. add a switch toggle on settings profile tab to get back the llm apps view - this is for people who is older user and invitated there team member to work on
… durable per-user nav default to control the simplified, -focused sidebar for new signups without impacting existing users. - export navSimplDefaultAtom from onboarding index- read and set navSimplifiedDefaultAtom in post-auth redirect hook so signup flow seeds the simplified-nav default when creating new users- add advancedNavHiddenAtom selector that derives from the new durable navSimplifiedAtom (separating signup-era default from transient isOnboarding/isUser state) - update design docs to explain the new atom, the rationale (do not reuse sticky isNewUserAtom) and the1/Phase2 migration seamThis prevents existing users from losing advanced navigation due to thesticky isNewUser flag while allowing newups to get a simplifiedsidebar by default.
…ents Starting a new agent puts an existing user into an onboarding session, and the sidebar greyed out Agents, Evaluation runs and Annotation Queues for the whole session. Those users watched their own agents sit listed under a disabled parent, with no way to click through. deadEndNavDisabledAtom asked "is onboarding running" when it meant "would this page be empty". It now also requires the project to have no agents. The agent list is read inside the guard, never above it, so the query stays untracked and unfetched whenever onboarding is off, matching what gatedSidebarSource does for the lazy submenu. Claude-Session: https://claude.ai/code/session_01Tjg6u7XP8huKzZqq4kGBmA
…ags tab Settings tabs now declare which scope they belong to (project, organization or personal) and the sidebar renders them under those headings, with a new Organization General tab for renaming, transferring and deleting an org. Adds a Feature Flags tab holding two per-user switches, both stored per user in local storage: one turns the playground session inspector on, the other overrides the simplified navigation chosen at signup. The nav override sits in front of the durable signup default (override ?? default), so an existing user who wants the simplified nav, or a new user who wants the full one, can say so without touching the signup-era value. The session inspector is now off unless its flag is on, in both the chat slice and the panel button. Also updates the simplify-nav design workspace to match. Claude-Session: https://claude.ai/code/session_01Tjg6u7XP8huKzZqq4kGBmA
… rejection antd passes onOk straight to the OK button's onClick, so the awaited mutateAsync in handleDelete rejected into nothing when the backend refused the delete. The mutation's onError already shows the message, so catch and return instead of letting the rejection escape. Claude-Session: https://claude.ai/code/session_01Tjg6u7XP8huKzZqq4kGBmA
…rst-signup-users [feat] Agent-first sidebar for new signups, Playground as app default
dosubot
Bot
added
the
size:XXL
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
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 |
Contributor
Railway Preview Environment
|
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
2 participants
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 v0.106.1 work collects on
temp-106.1, a branch cut fromrelease/v0.106.0while 106.0 was still open. This pull request brings that branch intorelease/v0.106.1.It carries one feature today. It grows as more 106.1 work merges into
temp-106.1, so expect the diff to get bigger while 106.1 is being built.Changes
Everything here comes from #5478, "Agent-first sidebar for new signups, Playground as app default", by @ashrafchowdury. That pull request was reviewed on its own and merged into
temp-106.1on 28 July, so this is an integration step rather than new code.What it does, in short. A new signup used to land on a platform that showed every page at once, which buries the thing they actually came for. The sidebar now starts in a simplified shape for fresh signups, and the Playground becomes the default page when you open an app. A durable per-user setting named
navSimplifiedDefaultrecords that choice, so it survives across sessions rather than resetting. The settings page groups its entries by scope and gains a personal feature flags tab.The change touches 26 files, all of them under
web/oss/src. No API code, no services code, and nothing underweb/eemoves.Tests
What to QA
Note for reviewers
Do not merge this while
temp-106.1is still collecting work. It stays open until 106.1 is ready to close.