diff --git a/ui/desktop/src/App.test.tsx b/ui/desktop/src/App.test.tsx index 769b9a9a..fcbb69c6 100644 --- a/ui/desktop/src/App.test.tsx +++ b/ui/desktop/src/App.test.tsx @@ -200,8 +200,8 @@ describe('App Component - Brand New State', () => { window.location.hash = ''; window.location.search = ''; window.location.pathname = '/'; - window.sessionStorage.clear(); - window.localStorage.clear(); + window.sessionStorage?.clear?.(); + window.localStorage?.clear?.(); }); afterEach(() => { @@ -246,10 +246,10 @@ describe('App Component - Brand New State', () => { expect(mockElectron.reactReady).toHaveBeenCalled(); }); - expect(screen.getByText(/^Select an AI model provider/)).toBeInTheDocument(); + expect(screen.getByText(/^Welcome to goose/)).toBeInTheDocument(); }); - it('should not redirect to /welcome when provider is configured', async () => { + it('should not redirect when provider is configured', async () => { // Mock provider configured mockElectron.getConfig.mockReturnValue({ GOOSE_DEFAULT_PROVIDER: 'openai', diff --git a/ui/desktop/src/App.tsx b/ui/desktop/src/App.tsx index 75367dc1..cb238473 100644 --- a/ui/desktop/src/App.tsx +++ b/ui/desktop/src/App.tsx @@ -52,7 +52,7 @@ import { useNavigation } from './hooks/useNavigation'; import { errorMessage } from './utils/conversionUtils'; import { getInitialWorkingDir } from './utils/workingDir'; import { usePageViewTracking } from './hooks/useAnalytics'; -import { trackOnboardingCompleted, trackErrorWithContext } from './utils/analytics'; +import { trackErrorWithContext } from './utils/analytics'; import { AppEvents } from './constants/events'; import { registerPlatformEventHandlers } from './utils/platform_events'; @@ -247,25 +247,6 @@ const ConfigureProvidersRoute = () => { ); }; -const WelcomeRoute = () => { - const navigate = useNavigate(); - - return ( -
- { - navigate('/', { replace: true }); - }} - isOnboarding={true} - onProviderLaunched={(model?: string) => { - trackOnboardingCompleted('other', model); - navigate('/', { replace: true }); - }} - /> -
- ); -}; - // Wrapper component for SharedSessionRoute to access parent state const SharedSessionRouteWrapper = ({ isLoadingSharedSession, @@ -627,7 +608,6 @@ export function AppInner() {
} /> - } /> } /> } /> { permission: '/permission', ConfigureProviders: '/configure-providers', sharedSession: '/shared-session', - welcome: '/welcome', }; if (viewType) { diff --git a/ui/desktop/src/utils/navigationUtils.ts b/ui/desktop/src/utils/navigationUtils.ts index 33f29f06..aa4f41b4 100644 --- a/ui/desktop/src/utils/navigationUtils.ts +++ b/ui/desktop/src/utils/navigationUtils.ts @@ -3,7 +3,6 @@ import { Recipe } from '../api'; import { UserInput } from '../types/message'; export type View = - | 'welcome' | 'chat' | 'pair' | 'settings' @@ -77,9 +76,6 @@ export const createNavigationHandler = (navigate: NavigateFunction) => { navigate('/shared-session', { state: options }); break; - case 'welcome': - navigate('/welcome', { state: options }); - break; case 'extensions': navigate('/extensions', { state: options }); break;