Skip to content

Conversation

@brunobergher
Copy link
Collaborator

@brunobergher brunobergher commented Dec 23, 2025

Considering adoption and how much easier it is to get started with the Roo provider, this makes it the default path, while still leaving other providers as a secondary option. Plus other UX niceties.

Screenshot 2025-12-23 at 17 47 03
Screenshot 2025-12-23 at 17 47 08
Screenshot 2025-12-23 at 17 47 23

Important

Enhances signup UX by defaulting to Roo provider, adds UI improvements, and updates translations.

  • Behavior:
    • Default signup path set to Roo provider, with other providers as secondary options.
    • Added flag roo-auth-skip-model in WebAuthService.ts to handle cases where no model is selected during signup.
  • UI Enhancements:
    • Updated WelcomeViewProvider.tsx to include a landing screen and provider selection screen.
    • Added animations and UI elements in RooHero.tsx.
    • Adjusted ApiOptions.tsx to filter out Roo provider on the welcome screen.
  • Testing:
    • Added tests in WelcomeViewProvider.spec.tsx to cover new signup flow and UI changes.
  • Translations:
    • Updated translation files for multiple languages to reflect new UI text and options.
  • Styles:
    • Added new animations ground-slide and sun in index.css.

This description was created by Ellipsis for 99f2e72. You can customize this summary. It will automatically update as commits are pushed.

@roomote
Copy link
Contributor

roomote bot commented Dec 23, 2025

Rooviewer Clock   See task on Roo Cloud

All issues have been resolved. No new issues found.

  • Missing TriangleAlert mock in lucide-react test mock
  • Tests reference removed translation key welcome:providerSignup.rooCloudProvider (should be rooCloudDescription) - all instances updated to welcome:greeting
  • Missing Brain icon mock in lucide-react test mock - component imports and uses Brain but mock doesn't include it
  • Incorrect translation key test IDs in provider selection test - uses trans-welcome:introduction and trans-welcome:chooseProvider but should use trans-welcome:providerSignup.chooseProvider
Previous reviews

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

brunobergher and others added 2 commits December 23, 2025 12:30
…r.spec.tsx

Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
…r.spec.tsx

Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Dec 23, 2025
- Add missing TriangleAlert icon to lucide-react mock
- Update test assertions to match new component structure
- Replace references to 'rooCloudProvider' heading with 'greeting'
- Update 'Use Roo Code Provider' to 'backToRoo' translation key
- All 9 tests now passing
brunobergher and others added 4 commits December 23, 2025 18:05
…r.spec.tsx

Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
…r.spec.tsx

Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
@brunobergher brunobergher marked this pull request as ready for review December 23, 2025 18:42
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. UI/UX UI/UX related or focused labels Dec 23, 2025
@roomote
Copy link
Contributor

roomote bot commented Dec 23, 2025

Rooviewer Clock   See task on Roo Cloud

All issues have been resolved. Tests pass.

  • Missing TriangleAlert mock in lucide-react test mock
  • Tests reference removed translation key welcome:providerSignup.rooCloudProvider (should be rooCloudDescription) - all instances updated to welcome:greeting
  • Missing Brain icon mock in lucide-react test mock - component imports and uses Brain but mock doesn't include it
  • Incorrect translation key test IDs in provider selection test - uses trans-welcome:introduction and trans-welcome:chooseProvider but should use trans-welcome:providerSignup.chooseProvider
Previous reviews

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 23, 2025
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
…hout model selection

When a user signs up via Roo Code Cloud but doesn't select a provider model during the signup flow, they should be taken to the provider selection screen with '3rd-party Provider' pre-selected instead of automatically being configured with the Roo provider.

Changes:
- WebAuthService: Store roo-auth-skip-model flag when providerModel is not provided
- ExtensionState: Add cloudAuthSkipModel field to pass flag to webview
- ClineProvider: Read and pass the flag to webview state
- WebviewMessage: Add clearCloudAuthSkipModel message type
- webviewMessageHandler: Add handler to clear the flag
- WelcomeViewProvider: Check flag on auth completion and navigate to custom provider selection
- Tests: Add test for skip model flag behavior
@mrubens mrubens merged commit 57cafa5 into main Dec 23, 2025
10 checks passed
@mrubens mrubens deleted the bb/cloudy-signup branch December 23, 2025 22:59
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Dec 23, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Dec 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files. UI/UX UI/UX related or focused

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants