feat: enhance assistant mode with session list view and legacy config migration

- Add session list view for assistant mode with full workspace management
- Migrate legacy opencode.json permission format to granular permissions
- Enable assistant navigation from mobile tab bar and repo switcher
- Preserve assistant context in session routes via query params
- Auto-restart OpenCode server after assistant mode initialization
- Add comprehensive tests for legacy config detection and navigation

Author: chriswritescode-dev

backend/src/routes/repos.ts

@@ -410,6 +410,10 @@ app.get('/', async (c) => {
       const options = AssistantModeInitRequestSchema.parse(body)
 
       const status = await ensureAssistantMode(repo, options)
+      if (status.files.opencodeJson.created) {
+        opencodeServerManager.clearStartupError()
+        await opencodeServerManager.restart()
+      }
       return c.json(status)
     } catch (error: unknown) {
       logger.error('Failed to initialize assistant mode:', error)

backend/src/services/assistant-mode.ts

@@ -6,6 +6,7 @@ import type {
   OpenCodeConfigInput,
 } from '@opencode-manager/shared/types'
 import {
+  readFileContent,
   writeFileContent,
   fileExists,
   ensureDirectoryExists,
@@ -71,12 +72,13 @@ export function buildAssistantOpenCodeConfig(): OpenCodeConfigInput {
       'AGENTS.md',
     ],
     permission: {
-      allow: [
-        '**/*',
-      ],
-      ask: [
-        '../**/*',
-      ],
+      read: 'allow',
+      edit: 'allow',
+      glob: 'allow',
+      grep: 'allow',
+      list: 'allow',
+      bash: 'allow',
+      external_directory: 'ask',
     },
   }
 
@@ -110,7 +112,9 @@ export async function ensureAssistantMode(
     await writeFileContent(agentsMdPath, content)
   }
 
-  if (!opencodeJsonExists || overwriteOpenCodeConfig) {
+  const hasLegacyOpenCodeConfig = opencodeJsonExists && await isLegacyAssistantOpenCodeConfig(opencodeJsonPath)
+
+  if (!opencodeJsonExists || overwriteOpenCodeConfig || hasLegacyOpenCodeConfig) {
     const config = buildAssistantOpenCodeConfig()
     await writeFileContent(opencodeJsonPath, JSON.stringify(config, null, 2))
   }
@@ -128,12 +132,22 @@ export async function ensureAssistantMode(
       opencodeJson: {
         path: opencodeJsonPath,
         exists: true,
-        created: !opencodeJsonExists || overwriteOpenCodeConfig,
+        created: !opencodeJsonExists || overwriteOpenCodeConfig || hasLegacyOpenCodeConfig,
       },
     },
   }
 }
 
+async function isLegacyAssistantOpenCodeConfig(opencodeJsonPath: string): Promise<boolean> {
+  try {
+    const content = await readFileContent(opencodeJsonPath)
+    const config = JSON.parse(content) as { permission?: { allow?: unknown; ask?: unknown } }
+    return Array.isArray(config.permission?.allow) || Array.isArray(config.permission?.ask)
+  } catch {
+    return false
+  }
+}
+
 export async function getAssistantModeStatus(repo: Repo): Promise<AssistantModeStatus> {
   const assistantDir = getAssistantModeDirectory()
 

backend/test/routes/repos.test.ts

@@ -28,6 +28,13 @@ vi.mock('../../src/services/assistant-mode', () => ({
   buildAssistantOpenCodeConfig: vi.fn(),
 }))
 
+vi.mock('../../src/services/opencode-single-server', () => ({
+  opencodeServerManager: {
+    clearStartupError: vi.fn(),
+    restart: vi.fn().mockResolvedValue(undefined),
+  },
+}))
+
 import * as db from '../../src/db/queries'
 import { createRepoRoutes } from '../../src/routes/repos'
 import type { GitAuthService } from '../../src/services/git-auth'

backend/test/services/assistant-mode.test.ts

@@ -27,17 +27,33 @@ const mockRepo: Repo = {
   isLocal: false,
 }
 
-const writeFile = vi.fn()
-const mkdir = vi.fn()
-const access = vi.fn()
-const stat = vi.fn()
+const fsMocks = vi.hoisted(() => ({
+  writeFile: vi.fn(),
+  mkdir: vi.fn(),
+  access: vi.fn(),
+  stat: vi.fn(),
+  readFile: vi.fn(),
+}))
+
+const { writeFile, mkdir, access, readFile } = fsMocks
 
 vi.mock('fs/promises', () => ({
   default: {
-    writeFile,
-    mkdir,
-    access,
-    stat,
+    writeFile: fsMocks.writeFile,
+    readFile: fsMocks.readFile,
+    mkdir: fsMocks.mkdir,
+    access: fsMocks.access,
+    stat: fsMocks.stat,
+  },
+}))
+
+vi.mock('fs', () => ({
+  promises: {
+    writeFile: fsMocks.writeFile,
+    readFile: fsMocks.readFile,
+    mkdir: fsMocks.mkdir,
+    access: fsMocks.access,
+    stat: fsMocks.stat,
   },
 }))
 
@@ -67,13 +83,22 @@ describe('buildAssistantOpenCodeConfig', () => {
 
   it('has permission rules for the assistant workspace', () => {
     const config = buildAssistantOpenCodeConfig()
-    expect(config.permission).toBeDefined()
+    expect(config.permission).toEqual({
+      read: 'allow',
+      edit: 'allow',
+      glob: 'allow',
+      grep: 'allow',
+      list: 'allow',
+      bash: 'allow',
+      external_directory: 'ask',
+    })
   })
 })
 
 describe('ensureAssistantMode', () => {
   beforeEach(() => {
     vi.clearAllMocks()
+    readFile.mockResolvedValue(JSON.stringify(buildAssistantOpenCodeConfig()))
   })
 
   it('creates the shared assistant workspace and files when missing', async () => {
@@ -114,6 +139,24 @@ describe('ensureAssistantMode', () => {
     expect(result.files.opencodeJson.created).toBe(true)
   })
 
+  it('overwrites legacy invalid assistant opencode config', async () => {
+    access.mockResolvedValue(undefined)
+    readFile.mockResolvedValue(JSON.stringify({
+      instructions: ['AGENTS.md'],
+      permission: {
+        allow: ['**/*'],
+        ask: ['../**/*'],
+      },
+    }))
+
+    const result = await ensureAssistantMode(mockRepo)
+
+    expect(result.files.opencodeJson.created).toBe(true)
+    const content = writeFile.mock.calls[0]?.[1]
+    expect(Buffer.isBuffer(content)).toBe(true)
+    expect((content as Buffer).toString('utf8')).toContain('external_directory')
+  })
+
   it('returns a directory under the repos root', async () => {
     access.mockRejectedValue(new Error('File not found'))
     mkdir.mockResolvedValue(undefined)

frontend/src/App.tsx

@@ -189,6 +189,11 @@ const router = createBrowserRouter([
         element: <Repos />,
         loader: protectedLoader,
       },
+      {
+        path: '/assistant',
+        element: <AssistantRedirect />,
+        loader: protectedLoader,
+      },
       {
         path: '/repos/:id',
         element: <RepoDetail />,

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

@@ -5,12 +5,18 @@ vi.mock('@/hooks/useMobile', () => ({
 }))
 
 import { render, screen } from '@testing-library/react'
+import userEvent from '@testing-library/user-event'
 import { describe, it, expect, beforeEach } from 'vitest'
-import { MemoryRouter } from 'react-router-dom'
+import { MemoryRouter, Route, Routes, useLocation } from 'react-router-dom'
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
 import { MobileTabBar } from './MobileTabBar'
 import { useMobile } from '@/hooks/useMobile'
 
+function LocationSpy() {
+  const { pathname, search } = useLocation()
+  return <div data-testid="location">{`${pathname}${search}`}</div>
+}
+
 describe('MobileTabBar', () => {
   beforeEach(() => {
     vi.clearAllMocks()
@@ -56,6 +62,65 @@ describe('MobileTabBar', () => {
     expect(screen.getByText('Schedules')).toBeInTheDocument()
   })
 
+  it('renders global tabs on assistant session list path', () => {
+    vi.mocked(useMobile).mockReturnValue(true)
+    const queryClient = new QueryClient()
+    render(
+      <QueryClientProvider client={queryClient}>
+        <MemoryRouter initialEntries={['/repos/123/assistant?view=sessions']}>
+          <MobileTabBar />
+        </MemoryRouter>
+      </QueryClientProvider>,
+    )
+    expect(screen.getByText('Repos')).toBeInTheDocument()
+    expect(screen.getByText('Assistant')).toBeInTheDocument()
+    expect(screen.getByText('Schedules')).toBeInTheDocument()
+  })
+
+  it('navigates to assistant route when repo id is present', async () => {
+    vi.mocked(useMobile).mockReturnValue(true)
+    const queryClient = new QueryClient()
+    const user = userEvent.setup()
+
+    render(
+      <QueryClientProvider client={queryClient}>
+        <MemoryRouter initialEntries={['/repos/123']}>
+          <Routes>
+            <Route path="*" element={<>
+              <MobileTabBar />
+              <LocationSpy />
+            </>} />
+          </Routes>
+        </MemoryRouter>
+      </QueryClientProvider>,
+    )
+
+    await user.click(screen.getByRole('button', { name: 'Assistant' }))
+    expect(screen.getByTestId('location')).toHaveTextContent('/repos/123/assistant')
+  })
+
+  it('navigates to assistant route when assistant is clicked without repo id', async () => {
+    vi.mocked(useMobile).mockReturnValue(true)
+    const queryClient = new QueryClient()
+    const user = userEvent.setup()
+
+    render(
+      <QueryClientProvider client={queryClient}>
+        <MemoryRouter initialEntries={['/schedules']}>
+          <Routes>
+            <Route path="*" element={<>
+              <MobileTabBar />
+              <LocationSpy />
+            </>} />
+          </Routes>
+        </MemoryRouter>
+      </QueryClientProvider>,
+    )
+
+    await user.click(screen.getByRole('button', { name: 'Assistant' }))
+    expect(screen.getByTestId('location')).toHaveTextContent('/assistant')
+  })
+
   it('renders schedule tabs on /repos/:id/schedules path', () => {
     vi.mocked(useMobile).mockReturnValue(true)
     const queryClient = new QueryClient()