Rework onboarding "Choose how to access AI": Subscription + Set up later#12993
Merged
Conversation
Contributor
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
Contributor
There was a problem hiding this comment.
Overview
This PR reworks the onboarding AI access slide to replace the BYOK option with Subscription / Set up later, moves the subscription checkout trigger to Next, and removes the now-unused BYOK onboarding modal and event plumbing.
Concerns
- For this user-facing onboarding change, please include screenshots or a screen recording demonstrating the updated Subscription and Set up later flows end to end. The PR description does not include visual evidence, and the manual testing checkbox is still unchecked, so the UI behavior and fallback bar cannot be reviewed visually.
Verdict
Found: 0 critical, 1 important, 0 suggestions
Request changes
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
tylerlam-warp
approved these changes
Jun 25, 2026
Replace the BYOK card with a skippable "Set up later" option, remove all BYOK plumbing (add-key/add-endpoint buttons, events, status line, set_byok_status, and the now-orphaned provider_keys_modal), and rename AiAccessChoice::Byok -> SetUpLater. Also remove this slide's "I don't want AI" button. Gated by OpenWarpNewSettingsModes. Co-Authored-By: Oz <oz-agent@warp.dev>
Remove the in-card "Choose plan" button. The bottom "Next" button now opens the upgrade page in the browser when Subscription is selected (or advances if the user already has a plan), and advances on the "Set up later" path. Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
cb48f16 to
20c7ee1
Compare
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.
Description
Reworks the "Choose how to access AI" onboarding slide (Warp Agent path), gated by
OpenWarpNewSettingsModes. The slide now offers two options:Primary action moved onto Next. The in-card "Choose plan" button is removed; the bottom Next button drives the flow based on the selected card:
All bring-your-own-key/endpoint (BYOK) code is removed: the add-key / add-custom-endpoint buttons, their actions/events, the "connected keys" status line, and the
set_byok_statusplumbing across the onboarding view androot_view. The now-orphanedprovider_keys_modal(BYOK "Add API key" dialog) is deleted. The settings page'sCustomEndpointModalis intentionally untouched (still used by settings).This slide's own "I don't want AI" escape button is also removed (the other slides' copies are handled in PR-B).
Why: part of the agent-onboarding rework — simplify AI access to a clear "subscribe now or explore first" choice, and let users proceed without subscribing.
Notes for the integrator
PR-C of a 4-PR split. Shared-file edits are localized:
model.rs: renamedAiAccessChoice::Byok→SetUpLater. Also removed the now-unusedNoAiConfirmationSource::AiAccessvariant + its singlecancel_no_aiarm (authorized bounded exception; removing this slide's no-AI trigger orphaned it under-D warnings). Expect a small localized conflict with PR-B here, resolved at integration (merge order B→C).agent_onboarding_view.rs/bin/main.rs: removed the BYOK event variants.LoginSlideView::newcall sites (owned by PR-D).Conversation: https://staging.warp.dev/conversation/23cf4d0f-2950-4c25-a229-37c9715cf2c3
Run: https://oz.staging.warp.dev/runs/019efa09-3960-738c-965a-d4ccfea2a0f0
Linked Issue
N/A — tracked by the onboarding rework plan.
Testing
./script/format— cleancargo clippy -p onboarding --all-targets --all-features --tests -- -D warnings— cleancargo nextest run -p onboarding— 13/13 passcargo clippy -p warp --all-targets --tests -- -D warnings— clean (default features;--all-featuresis blocked by an unrelatedcommand-signatures-v2yarn/corepack environment issue that WARP.md special-cases)cargo nextest run -p warp -E 'test(onboarding) | test(login)'— passI have manually tested my changes locally with
./script/runAgent Mode
CHANGELOG-IMPROVEMENT: The onboarding "Choose how to access AI" step now offers "Subscription" or "Set up later" and can be skipped without subscribing; choosing Subscription and pressing Next opens the upgrade page directly.