fix: open assistant navigation to session list

Author: chriswritescode-dev

frontend/src/components/navigation/MobileTabBar.test.tsx

@@ -77,7 +77,7 @@ describe('MobileTabBar', () => {
     expect(screen.getByText('Schedules')).toBeInTheDocument()
   })
 
-  it('navigates to /assistant when assistant is clicked from repo context', async () => {
+  it('navigates to assistant session list when assistant is clicked from repo context', async () => {
     vi.mocked(useMobile).mockReturnValue(true)
     const queryClient = new QueryClient()
     const user = userEvent.setup()
@@ -96,10 +96,10 @@ describe('MobileTabBar', () => {
     )
 
     await user.click(screen.getByRole('button', { name: 'Assistant' }))
-    expect(screen.getByTestId('location')).toHaveTextContent('/assistant')
+    expect(screen.getByTestId('location')).toHaveTextContent('/assistant?view=sessions')
   })
 
-  it('navigates to assistant route when assistant is clicked without repo id', async () => {
+  it('navigates to assistant session list when assistant is clicked without repo id', async () => {
     vi.mocked(useMobile).mockReturnValue(true)
     const queryClient = new QueryClient()
     const user = userEvent.setup()
@@ -118,7 +118,7 @@ describe('MobileTabBar', () => {
     )
 
     await user.click(screen.getByRole('button', { name: 'Assistant' }))
-    expect(screen.getByTestId('location')).toHaveTextContent('/assistant')
+    expect(screen.getByTestId('location')).toHaveTextContent('/assistant?view=sessions')
   })
 
   it('renders schedule tabs on /repos/:id/schedules path', () => {

frontend/src/components/navigation/MobileTabBar.tsx

@@ -6,7 +6,7 @@ import { useMobile } from '@/hooks/useMobile'
 import { useMobileTabBar } from '@/hooks/useMobileTabBar'
 import { useUrlParams } from '@/hooks/useUrlParams'
 import { useScheduleUrlState, type ScheduleTab } from '@/hooks/useScheduleUrlState'
-import { getAssistantPath, isAssistantPath } from '@/lib/navigation'
+import { getAssistantPath, getAssistantSessionListPath, isAssistantPath } from '@/lib/navigation'
 
 interface TabDef {
   key: string
@@ -74,7 +74,7 @@ function buildGlobalTabs({ pathname, openSheet, open, close, navigate, isInsideR
 
   const handleAssistantClick = () => {
     close()
-    navigate(getAssistantPath())
+    navigate(getAssistantSessionListPath())
   }
 
   return [

frontend/src/components/navigation/moreDrawerItems.test.ts

@@ -90,7 +90,7 @@ describe('buildNavModel', () => {
     expect(model.primary[0].key).toBe('new-repo')
     expect(model.primary[0].onSelect).toBe('new-repo')
     expect(model.primary[1].key).toBe('assistant')
-    expect(model.primary[1].to).toBe('/assistant')
+    expect(model.primary[1].to).toBe('/assistant?view=sessions')
   })
 
   it('returns new-session and assistant primary CTAs for repo detail', () => {
@@ -99,7 +99,7 @@ describe('buildNavModel', () => {
     expect(model.primary[0].key).toBe('new-session')
     expect(model.primary[0].onSelect).toBe('new-session')
     expect(model.primary[1].key).toBe('assistant')
-    expect(model.primary[1].to).toBe('/assistant')
+    expect(model.primary[1].to).toBe('/assistant?view=sessions')
   })
 
   it('returns new-session and assistant primary CTAs for session detail', () => {
@@ -109,7 +109,7 @@ describe('buildNavModel', () => {
     expect(model.primary[0].onSelect).toBe('new-session')
     expect(model.primary[0].variant).toBe('primary')
     expect(model.primary[1].key).toBe('assistant')
-    expect(model.primary[1].to).toBe('/assistant')
+    expect(model.primary[1].to).toBe('/assistant?view=sessions')
     expect(model.primary[1].variant).toBe('secondary')
   })
 
@@ -120,7 +120,7 @@ describe('buildNavModel', () => {
     expect(model.primary[0].onSelect).toBe('new-session')
     expect(model.primary[0].variant).toBe('primary')
     expect(model.primary[1].key).toBe('assistant')
-    expect(model.primary[1].to).toBe('/assistant')
+    expect(model.primary[1].to).toBe('/assistant?view=sessions')
     expect(model.primary[1].variant).toBe('secondary')
   })
 
@@ -131,7 +131,7 @@ describe('buildNavModel', () => {
     expect(model.primary[0].onSelect).toBe('new-session')
     expect(model.primary[0].variant).toBe('primary')
     expect(model.primary[1].key).toBe('assistant')
-    expect(model.primary[1].to).toBe('/assistant')
+    expect(model.primary[1].to).toBe('/assistant?view=sessions')
     expect(model.primary[1].variant).toBe('secondary')
   })
 
@@ -141,21 +141,21 @@ describe('buildNavModel', () => {
     expect(model1.primary[0].key).toBe('new-schedule')
     expect(model1.primary[0].onSelect).toBe('new-schedule')
     expect(model1.primary[1].key).toBe('assistant')
-    expect(model1.primary[1].to).toBe('/assistant')
+    expect(model1.primary[1].to).toBe('/assistant?view=sessions')
 
     const model2 = buildNavModel('/repos/5/schedules')
     expect(model2.primary).toHaveLength(2)
     expect(model2.primary[0].key).toBe('new-schedule')
     expect(model2.primary[0].onSelect).toBe('new-schedule')
     expect(model2.primary[1].key).toBe('assistant')
-    expect(model2.primary[1].to).toBe('/assistant')
+    expect(model2.primary[1].to).toBe('/assistant?view=sessions')
   })
 
   it('returns assistant primary for unknown routes', () => {
     const model = buildNavModel('/unknown/path')
     expect(model.primary).toHaveLength(1)
     expect(model.primary[0].key).toBe('assistant')
-    expect(model.primary[0].to).toBe('/assistant')
+    expect(model.primary[0].to).toBe('/assistant?view=sessions')
   })
 
   it('preserves backwards compatibility with buildMoreItems', () => {

frontend/src/components/navigation/moreDrawerItems.ts

@@ -1,6 +1,6 @@
 import type { LucideIcon } from 'lucide-react'
 import { Plug, Sparkles, ShieldOff, CalendarClock, GitCommitHorizontal, Code2, Settings, LogOut, Plus, Bot, Folder, Clock, SquarePlus } from 'lucide-react'
-import { getAssistantPath, isAssistantPath } from '@/lib/navigation'
+import { getAssistantSessionListPath, isAssistantPath } from '@/lib/navigation'
 
 export interface MoreDrawerItem {
   key: string
@@ -30,7 +30,7 @@ function getAssistantNavItem(_pathname: string, variant: NavPrimaryCta['variant'
     key: 'assistant',
     label: 'Assistant',
     icon: Bot,
-    to: getAssistantPath(),
+    to: getAssistantSessionListPath(),
     variant,
   }
 }