@linxin666/dsh-remote-web-ui 0.3.6 → 0.3.9

package/src/mobile/views/SessionListView.test.tsx

@@ -1,207 +0,0 @@
-// @vitest-environment jsdom
-/** SessionListView: owned-row filtering, incremental pages, session creation. */
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import { cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react'
-import type { WorkspaceView as WorkspaceRow } from '@deepseek-ai/dsh-host-apiproxy/api/workspace'
-import { SessionListView, type SessionListViewProps } from './SessionListView.tsx'
-import { type SessionView } from './App.tsx'
-
-// The api module is fully mocked; the view's App.tsx helpers stay real.
-vi.mock('../api.ts', () => ({
-  listSessions: vi.fn(),
-  listWorkspaces: vi.fn(),
-  listAgentPresets: vi.fn(),
-  createSession: vi.fn(),
-}))
-import { createSession, listAgentPresets, listSessions, listWorkspaces } from '../api.ts'
-
-const listSessionsMock = vi.mocked(listSessions)
-const listWorkspacesMock = vi.mocked(listWorkspaces)
-const listAgentPresetsMock = vi.mocked(listAgentPresets)
-const createSessionMock = vi.mocked(createSession)
-
-const workspace: WorkspaceRow = {
-  workspaceId: 'w-1' as never,
-  path: '/tmp/demo',
-  title: '演示项目',
-  sessionIds: ['s-1'] as never,
-  createdAt: '2026-01-01T00:00:00.000Z',
-  updatedAt: '2026-01-01T00:00:00.000Z',
-}
-
-/** A session.list summary row. */
-function summary(sessionId: string, updatedAt: number, extra: Record<string, unknown> = {}): never {
-  return { sessionId, updatedAt, running: false, blank: false, ...extra } as never
-}
-
-let picked: SessionView | undefined
-
-function renderList(props: Partial<SessionListViewProps> = {}): void {
-  picked = undefined
-  render(
-    <SessionListView
-      workspace={workspace}
-      onBack={() => {}}
-      onPick={(session) => { picked = session }}
-      {...props}
-    />,
-  )
-}
-
-beforeEach(() => {
-  listSessionsMock.mockResolvedValue({ items: [], hasMore: false })
-  listWorkspacesMock.mockResolvedValue([workspace])
-  listAgentPresetsMock.mockResolvedValue({ presets: [], authorable: false, hasDocument: false })
-  createSessionMock.mockResolvedValue({ sessionId: 's-new' })
-})
-
-afterEach(() => {
-  cleanup()
-  vi.clearAllMocks()
-})
-
-describe('SessionListView roster', () => {
-  it('renders the sessions owned by the workspace', async () => {
-    listSessionsMock.mockResolvedValue({
-      items: [
-        summary('s-1', 1_700_000_000_000, { projections: { values: { title: '改造移动端' } } }),
-        summary('s-other', 1_600_000_000_000, { cwd: '/tmp/foreign' }),
-      ],
-      hasMore: false,
-    })
-    renderList()
-
-    expect(await screen.findByText('改造移动端')).toBeTruthy()
-    // The foreign session (not in workspace.sessionIds) stays hidden.
-    expect(screen.queryByText(/foreign/)).toBeNull()
-  })
-
-  it('shows the empty state when the workspace owns no sessions', async () => {
-    renderList()
-    expect(await screen.findByText(/还没有会话/)).toBeTruthy()
-  })
-
-  it('refreshes the owned-id set from workspace.list on mount', async () => {
-    listSessionsMock.mockResolvedValue({
-      items: [summary('s-2', 1_700_000_000_000, { cwd: '/tmp/other' })],
-      hasMore: false,
-    })
-    listWorkspacesMock.mockResolvedValue([
-      { ...workspace, sessionIds: ['s-2'] as never },
-    ])
-    renderList()
-    // s-2 is absent from the stale row but present in the fresh roster.
-    expect(await screen.findByText(/other/)).toBeTruthy()
-  })
-})
-
-describe('SessionListView creation', () => {
-  it('creates a blank session in the workspace and opens it immediately', async () => {
-    renderList()
-    await screen.findByText(/还没有会话/)
-
-    fireEvent.click(screen.getByRole('button', { name: '+ 新建会话' }))
-
-    await waitFor(() => {
-      expect(createSessionMock).toHaveBeenCalledWith({ workspaceId: 'w-1' })
-    })
-    await waitFor(() => {
-      expect(picked).toBeDefined()
-    })
-    expect(picked?.sessionId).toBe('s-new')
-    expect(picked?.blank).toBe(true)
-    // The fresh row is prepended without waiting for a refetch.
-    expect(await screen.findByText('新会话')).toBeTruthy()
-  })
-
-  it('creates the session with the selected agent preset', async () => {
-    listAgentPresetsMock.mockResolvedValue({
-      presets: [
-        { id: 'router-standard', name: 'Router Standard', trust: 'system', isDefault: true },
-        { id: 'router-spec', name: 'Router Spec', description: '规格驱动模式', trust: 'system', isDefault: false },
-      ],
-      authorable: false,
-      hasDocument: false,
-    })
-    renderList()
-
-    const picker = await screen.findByRole('combobox', { name: 'Agent 模式' })
-    expect((picker as HTMLSelectElement).value).toBe('router-standard')
-    fireEvent.change(picker, { target: { value: 'router-spec' } })
-    fireEvent.click(screen.getByRole('button', { name: '+ 新建会话' }))
-
-    await waitFor(() => {
-      expect(createSessionMock).toHaveBeenCalledWith({ workspaceId: 'w-1', agentPreset: 'router-spec' })
-    })
-    expect(await screen.findByText('规格驱动模式')).toBeTruthy()
-  })
-
-  it('skips a broken default preset and disables it in the picker', async () => {
-    listAgentPresetsMock.mockResolvedValue({
-      presets: [
-        { id: 'broken-default', trust: 'system', isDefault: true, broken: 'missing plugin' },
-        { id: 'local-router', name: 'Local Router', trust: 'user', isDefault: false },
-      ],
-      authorable: false,
-      hasDocument: false,
-    })
-    renderList()
-
-    const picker = await screen.findByRole('combobox', { name: 'Agent 模式' }) as HTMLSelectElement
-    expect(picker.value).toBe('local-router')
-    expect(screen.getByRole('option', { name: 'broken-default（默认）（不可用）' }).hasAttribute('disabled')).toBe(true)
-    expect(screen.getByRole('option', { name: 'Local Router（本地）' })).toBeTruthy()
-  })
-
-  it('keeps default session creation available when the preset roster fails', async () => {
-    listAgentPresetsMock.mockRejectedValue(new Error('preset roster unavailable'))
-    renderList()
-    await screen.findByText(/还没有会话/)
-
-    const button = await screen.findByRole('button', { name: '+ 新建会话' })
-    await waitFor(() => { expect(button.hasAttribute('disabled')).toBe(false) })
-    fireEvent.click(button)
-
-    await waitFor(() => {
-      expect(createSessionMock).toHaveBeenCalledWith({ workspaceId: 'w-1' })
-    })
-  })
-
-  it('waits for the preset roster before enabling session creation', async () => {
-    let resolvePresets: (value: Awaited<ReturnType<typeof listAgentPresets>>) => void = () => {}
-    listAgentPresetsMock.mockReturnValue(new Promise(resolve => { resolvePresets = resolve }))
-    renderList()
-
-    const button = screen.getByRole('button', { name: '+ 新建会话' })
-    expect(button.hasAttribute('disabled')).toBe(true)
-
-    resolvePresets({ presets: [], authorable: false, hasDocument: false })
-    await waitFor(() => { expect(button.hasAttribute('disabled')).toBe(false) })
-  })
-
-  it('keeps the button disabled while a creation is in flight', async () => {
-    let resolveCreate: (value: { sessionId: string }) => void = () => {}
-    createSessionMock.mockReturnValue(new Promise(resolve => { resolveCreate = resolve }))
-    renderList()
-    await screen.findByText(/还没有会话/)
-
-    fireEvent.click(screen.getByRole('button', { name: '+ 新建会话' }))
-    const button = await screen.findByRole('button', { name: '创建中…' })
-    expect(button.hasAttribute('disabled')).toBe(true)
-
-    resolveCreate({ sessionId: 's-new' })
-    await waitFor(() => { expect(picked).toBeDefined() })
-  })
-
-  it('shows the stale-host hint when creation is refused with HTTP 403', async () => {
-    createSessionMock.mockRejectedValue(new Error('HTTP 403'))
-    renderList()
-    await screen.findByText(/还没有会话/)
-
-    fireEvent.click(screen.getByRole('button', { name: '+ 新建会话' }))
-
-    expect(await screen.findByText(/HTTP 403/)).toBeTruthy()
-    expect(await screen.findByText(/重启 dsh web/)).toBeTruthy()
-    expect(picked).toBeUndefined()
-  })
-})

package/src/mobile/views/SessionListView.tsx

@@ -1,244 +0,0 @@
-/**
- * Sessions level: one workspace's sessions, loaded incrementally — the
- * first page fetches session.list with a cursor, scrolling appends further
- * pages (never the whole list at once). Rows are filtered to the opened
- * workspace by its owned session ids; extra pages are pulled on demand so
- * a workspace with many sessions converges without a full transfer.
- *
- * Creating a session is this level's other action: the workspace's id is
- * sent to session.create (the host attaches the new session to it), the
- * fresh row is prepended optimistically, and the user lands straight in
- * the new chat — the same "new session opens" flow as the desktop UI.
- */
-
-import { useCallback, useEffect, useRef, useState } from 'react'
-import type { WorkspaceView as WorkspaceRow } from '@deepseek-ai/dsh-host-apiproxy/api/workspace'
-import type { AgentPresetEntry } from '@deepseek-ai/dsh-host-apiproxy/api/agent-presets'
-import type { SessionSummary } from '@deepseek-ai/dsh-host-apiproxy/api/sessions'
-import { createSession, listAgentPresets, listSessions, listWorkspaces } from '../api.ts'
-import { errorText, formatTime, staleHostHint, toSessionView, type SessionView } from './App.tsx'
-import { ThemeToggle } from '../theme-toggle.tsx'
-
-/** Props for the session list. */
-export interface SessionListViewProps {
-  workspace: WorkspaceRow
-  onBack(): void
-  onPick(session: SessionView): void
-}
-
-/** Rows that belong to the opened workspace (its owned session id set). */
-function ownedItems(page: SessionSummary[], workspace: WorkspaceRow): SessionView[] {
-  const owned = new Set(workspace.sessionIds)
-  return page
-    .filter(item => owned.has(item.sessionId as never))
-    .map(item => toSessionView(item))
-}
-
-/**
- * Render one workspace's paged session list.
- * @param props - the workspace, back action, and pick action.
- * @returns the session list.
- */
-export function SessionListView({ workspace, onBack, onPick }: SessionListViewProps) {
-  const [rows, setRows] = useState<SessionView[]>([])
-  const [hasMore, setHasMore] = useState(false)
-  const [loading, setLoading] = useState(true)
-  const [error, setError] = useState<string | undefined>(undefined)
-  const [creating, setCreating] = useState(false)
-  const [createError, setCreateError] = useState<string | undefined>(undefined)
-  const [presets, setPresets] = useState<readonly AgentPresetEntry[]>([])
-  const [selectedPreset, setSelectedPreset] = useState<string | undefined>(undefined)
-  const [presetsLoading, setPresetsLoading] = useState(true)
-  const cursorRef = useRef<string | undefined>(undefined)
-  const busyRef = useRef(false)
-  // The freshest owned-id set: the mount effect re-reads the workspace roster
-  // so a session created since this row was captured still shows; loadMore
-  // filters later pages through this ref instead of the stale prop.
-  const workspaceRef = useRef(workspace)
-
-  // First page on mount. The workspace roster is re-read alongside so the
-  // owned-id set is fresh: a session created (or attached) since this
-  // workspace row was captured must not vanish from the filter.
-  useEffect(() => {
-    let cancelled = false
-    setLoading(true)
-    setError(undefined)
-    void Promise.all([listSessions(), listWorkspaces()]).then(
-      ([page, workspaces]) => {
-        if (cancelled) return
-        const fresh = workspaces.find(candidate => candidate.workspaceId === workspace.workspaceId)
-        const current = fresh ?? workspace
-        workspaceRef.current = current
-        setRows(ownedItems(page.items, current))
-        cursorRef.current = page.nextCursor
-        setHasMore(page.hasMore)
-        setLoading(false)
-      },
-      (reason: unknown) => {
-        if (cancelled) return
-        setError(errorText(reason))
-        setLoading(false)
-      },
-    )
-    return () => { cancelled = true }
-  }, [workspace])
-
-  useEffect(() => {
-    let cancelled = false
-    setPresets([])
-    setSelectedPreset(undefined)
-    setPresetsLoading(true)
-    void listAgentPresets().then(
-      (roster) => {
-        if (cancelled) return
-        const usable = roster.presets.filter(preset => preset.broken === undefined)
-        setPresets(roster.presets)
-        setSelectedPreset((usable.find(preset => preset.isDefault) ?? usable[0])?.id)
-        setPresetsLoading(false)
-      },
-      () => {
-        if (cancelled) return
-        setPresetsLoading(false)
-      },
-    )
-    return () => { cancelled = true }
-  }, [workspace])
-
-  /** Pull the next page and append the rows that belong to this workspace. */
-  const loadMore = useCallback(() => {
-    if (busyRef.current) return
-    const cursor = cursorRef.current
-    if (cursor === undefined) return
-    busyRef.current = true
-    setLoading(true)
-    void listSessions(cursor).then(
-      (page) => {
-        busyRef.current = false
-        setLoading(false)
-        cursorRef.current = page.nextCursor
-        setHasMore(page.hasMore)
-        setRows(previous => [...previous, ...ownedItems(page.items, workspaceRef.current)])
-      },
-      (reason: unknown) => {
-        busyRef.current = false
-        setLoading(false)
-        setError(errorText(reason))
-      },
-    )
-  }, [workspace])
-
-  /** Create a blank session in this workspace and open it immediately. */
-  const handleCreate = useCallback(() => {
-    if (creating) return
-    setCreating(true)
-    setCreateError(undefined)
-    void createSession({
-      workspaceId: workspace.workspaceId,
-      ...(selectedPreset !== undefined ? { agentPreset: selectedPreset } : {}),
-    }).then(
-      (created) => {
-        setCreating(false)
-        const view: SessionView = {
-          sessionId: created.sessionId,
-          title: '新会话',
-          updatedAt: Date.now(),
-          running: false,
-          blank: true,
-        }
-        setRows(previous => [view, ...previous])
-        onPick(view)
-      },
-      (reason: unknown) => {
-        setCreating(false)
-        setCreateError(errorText(reason))
-      },
-    )
-  }, [creating, workspace, onPick, selectedPreset])
-
-  const createHint = createError !== undefined ? staleHostHint(createError) : undefined
-  const selectedPresetEntry = presets.find(preset => preset.id === selectedPreset)
-
-  return (
-    <div className="mobile">
-      <header className="mobile-header">
-        <button type="button" className="mobile-back" aria-label="返回" onClick={onBack}>‹</button>
-        <h1 className="mobile-title mobile-titleInline">{workspace.title}</h1>
-        <ThemeToggle />
-      </header>
-      {error !== undefined && <p className="mobile-error mobile-pad">{error}</p>}
-      <div className="mobile-create mobile-pad">
-        {presets.length > 0 && (
-          <label className="mobile-preset">
-            <span className="mobile-presetLabel">Agent 模式</span>
-            <select
-              className="mobile-presetSelect"
-              value={selectedPreset ?? ''}
-              disabled={selectedPreset === undefined}
-              onChange={(event) => { setSelectedPreset(event.target.value) }}
-            >
-              {selectedPreset === undefined && <option value="">无可用 Agent 模式</option>}
-              {presets.map(preset => (
-                <option key={preset.id} value={preset.id} disabled={preset.broken !== undefined}>
-                  {preset.name ?? preset.id}
-                  {preset.isDefault ? '（默认）' : ''}
-                  {preset.trust === 'user' ? '（本地）' : ''}
-                  {preset.broken !== undefined ? '（不可用）' : ''}
-                </option>
-              ))}
-            </select>
-          </label>
-        )}
-        {selectedPresetEntry?.description !== undefined && (
-          <p className="mobile-presetDescription">{selectedPresetEntry.description}</p>
-        )}
-        <button
-          type="button"
-          className="mobile-new"
-          disabled={creating || presetsLoading}
-          onClick={() => { void handleCreate() }}
-        >
-          {creating ? '创建中…' : '+ 新建会话'}
-        </button>
-      </div>
-      {createError !== undefined && (
-        <p className="mobile-error mobile-pad">
-          {createError}
-          {createHint !== undefined && <span className="mobile-hint">{createHint}</span>}
-        </p>
-      )}
-      <ul className="mobile-list">
-        {rows.map(row => (
-          <li key={row.sessionId}>
-            <button type="button" className="mobile-row" onClick={() => { onPick(row) }}>
-              <span className="mobile-rowMain">
-                <span className="mobile-rowTitle">
-                  {row.blank ? '新会话' : row.title}
-                  {row.running ? <span className="mobile-live">●</span> : null}
-                </span>
-                <span className="mobile-rowMeta">{formatTime(row.updatedAt)}</span>
-              </span>
-              <span className="mobile-chevron">›</span>
-            </button>
-          </li>
-        ))}
-      </ul>
-      {hasMore && (
-        <div className="mobile-pad">
-          <button
-            type="button"
-            className="mobile-button mobile-block"
-            disabled={loading}
-            onClick={() => { void loadMore() }}
-          >
-            {loading ? '加载中…' : '加载更多会话'}
-          </button>
-        </div>
-      )}
-      {!hasMore && rows.length === 0 && !loading && (
-        <div className="mobile-empty">
-          <p className="mobile-muted">该工作区还没有会话，点上方按钮新建一个</p>
-        </div>
-      )}
-    </div>
-  )
-}

package/src/mobile/views/WorkspaceView.test.tsx

@@ -1,227 +0,0 @@
-// @vitest-environment jsdom
-/** Mobile workspace landing: roster rendering and QR deep-link selection. */
-import { afterEach, describe, expect, it, vi } from 'vitest'
-import { cleanup, render, screen, waitFor, fireEvent } from '@testing-library/react'
-import type { WorkspaceView as WorkspaceRow } from '@deepseek-ai/dsh-host-apiproxy/api/workspace'
-import { mobileWorkspaceTarget } from './App.tsx'
-import { WorkspaceView } from './WorkspaceView.tsx'
-
-vi.mock('../api.ts', () => ({
-  listWorkspaces: vi.fn(),
-  listDirectory: vi.fn(),
-  createWorkspace: vi.fn(),
-}))
-import { listWorkspaces, listDirectory, createWorkspace } from '../api.ts'
-
-const listWorkspacesMock = vi.mocked(listWorkspaces)
-const listDirectoryMock = vi.mocked(listDirectory)
-const createWorkspaceMock = vi.mocked(createWorkspace)
-
-const workspaces: WorkspaceRow[] = [
-  {
-    workspaceId: 'ws-1' as never,
-    path: '/tmp/first',
-    title: 'First',
-    sessionIds: [] as never,
-    createdAt: '2026-01-01T00:00:00.000Z',
-    updatedAt: '2026-01-01T00:00:00.000Z',
-  },
-  {
-    workspaceId: 'ws-2' as never,
-    path: '/tmp/second',
-    title: 'Second',
-    sessionIds: [] as never,
-    createdAt: '2026-01-01T00:00:00.000Z',
-    updatedAt: '2026-01-01T00:00:00.000Z',
-  },
-]
-
-afterEach(() => {
-  cleanup()
-  vi.clearAllMocks()
-})
-
-describe('mobile workspace deep link', () => {
-  it('reads a non-empty workspace target from the query', () => {
-    expect(mobileWorkspaceTarget('?workspace=ws-2')).toBe('ws-2')
-    expect(mobileWorkspaceTarget('?workspace=')).toBeUndefined()
-    expect(mobileWorkspaceTarget('')).toBeUndefined()
-  })
-
-  it('opens the targeted workspace as soon as the roster loads', async () => {
-    listWorkspacesMock.mockResolvedValue(workspaces)
-    const onPick = vi.fn()
-
-    render(<WorkspaceView initialWorkspaceId="ws-2" onPick={onPick} />)
-
-    await waitFor(() => expect(onPick).toHaveBeenCalledWith(workspaces[1]))
-    expect(onPick).toHaveBeenCalledTimes(1)
-  })
-
-  it('falls back to the roster when the target no longer exists', async () => {
-    listWorkspacesMock.mockResolvedValue(workspaces)
-    const onPick = vi.fn()
-
-    render(<WorkspaceView initialWorkspaceId="missing" onPick={onPick} />)
-
-    expect(await screen.findByText('First')).toBeTruthy()
-    expect(await screen.findByText('Second')).toBeTruthy()
-    expect(onPick).not.toHaveBeenCalled()
-  })
-})
-
-describe('mobile workspace creation', () => {
-  it('shows new workspace button and navigates to directory browser', async () => {
-    listWorkspacesMock.mockResolvedValue(workspaces)
-    listDirectoryMock.mockResolvedValue({
-      path: '/home/user',
-      home: '/home/user',
-      crumbs: [{ name: '', path: '/', hidden: false }, { name: 'home', path: '/home', hidden: false }, { name: 'user', path: '/home/user', hidden: false }],
-      entries: [
-        { name: 'projects', path: '/home/user/projects', hidden: false },
-        { name: '.config', path: '/home/user/.config', hidden: true }
-      ],
-      truncated: false
-    })
-
-    const onPick = vi.fn()
-    render(<WorkspaceView onPick={onPick} />)
-
-    // Wait for workspaces to load
-    const createBtn = await screen.findByText('+ 新建工作区')
-    
-    // Click create button
-    fireEvent.click(createBtn)
-
-    // Verify directory browser renders
-    expect(await screen.findByText('选择目录')).toBeTruthy()
-    
-    // Verify crumbs
-    expect(await screen.findByText('user')).toBeTruthy()
-    
-    // Verify entries
-    expect(await screen.findByText('projects')).toBeTruthy()
-    expect(await screen.findByText('.config')).toBeTruthy()
-  })
-
-  it('navigates into directory and allows breadcrumb navigation', async () => {
-    listWorkspacesMock.mockResolvedValue(workspaces)
-    
-    // Initial dir
-    listDirectoryMock.mockResolvedValueOnce({
-      path: '/home/user',
-      home: '/home/user',
-      crumbs: [{ name: '', path: '/', hidden: false }, { name: 'home', path: '/home', hidden: false }, { name: 'user', path: '/home/user', hidden: false }],
-      entries: [{ name: 'projects', path: '/home/user/projects', hidden: false }],
-      truncated: false
-    })
-
-    const onPick = vi.fn()
-    render(<WorkspaceView onPick={onPick} />)
-
-    // Open dir browser
-    fireEvent.click(await screen.findByText('+ 新建工作区'))
-    
-    // Wait for first dir listing
-    const projBtn = await screen.findByText('projects')
-
-    // Prepare next listing
-    listDirectoryMock.mockResolvedValueOnce({
-      path: '/home/user/projects',
-      home: '/home/user',
-      crumbs: [
-        { name: '', path: '/', hidden: false },
-        { name: 'home', path: '/home', hidden: false },
-        { name: 'user', path: '/home/user', hidden: false },
-        { name: 'projects', path: '/home/user/projects', hidden: false }
-      ],
-      entries: [{ name: 'foo', path: '/home/user/projects/foo', hidden: false }],
-      truncated: false
-    })
-
-    // Click into folder
-    fireEvent.click(projBtn)
-
-    // Verify new contents load
-    expect(await screen.findByText('foo')).toBeTruthy()
-    
-    // Check breadcrumb
-    const homeCrumb = screen.getByText('home')
-    expect(homeCrumb).toBeTruthy()
-    
-    // Click breadcrumb
-    listDirectoryMock.mockResolvedValueOnce({
-      path: '/home',
-      home: '/home/user',
-      crumbs: [{ name: '', path: '/', hidden: false }, { name: 'home', path: '/home', hidden: false }],
-      entries: [{ name: 'user', path: '/home/user', hidden: false }],
-      truncated: false
-    })
-    
-    fireEvent.click(homeCrumb)
-    expect(await screen.findByText('user')).toBeTruthy()
-  })
-
-  it('creates workspace successfully', async () => {
-    listWorkspacesMock.mockResolvedValue(workspaces)
-    listDirectoryMock.mockResolvedValue({
-      path: '/home/user/projects/foo',
-      home: '/home/user',
-      crumbs: [{ name: '', path: '/', hidden: false }, { name: 'foo', path: '/home/user/projects/foo', hidden: false }],
-      entries: [],
-      truncated: false
-    })
-
-    const newWorkspace = { ...workspaces[0], workspaceId: 'ws-new' as never, title: 'foo' }
-    createWorkspaceMock.mockResolvedValue({
-      workspace: newWorkspace,
-      created: true
-    })
-
-    const onPick = vi.fn()
-    render(<WorkspaceView onPick={onPick} />)
-
-    fireEvent.click(await screen.findByText('+ 新建工作区'))
-    
-    const selectBtn = await screen.findByText('选择此目录')
-    fireEvent.click(selectBtn)
-
-    expect(await screen.findByText('创建中…')).toBeTruthy()
-    
-    await waitFor(() => {
-      expect(createWorkspaceMock).toHaveBeenCalledWith('/home/user/projects/foo')
-      expect(onPick).toHaveBeenCalledWith(newWorkspace)
-    })
-  })
-
-  it('handles errors from directory listing and workspace creation', async () => {
-    listWorkspacesMock.mockResolvedValue(workspaces)
-    listDirectoryMock.mockRejectedValue(new Error('Permission denied'))
-
-    const onPick = vi.fn()
-    render(<WorkspaceView onPick={onPick} />)
-
-    fireEvent.click(await screen.findByText('+ 新建工作区'))
-
-    // Verify error from listing
-    expect(await screen.findByText('Permission denied')).toBeTruthy()
-    
-    // Retry with success
-    listDirectoryMock.mockResolvedValue({
-      path: '/home',
-      home: '/home/user',
-      crumbs: [],
-      entries: [],
-      truncated: false
-    })
-    fireEvent.click(screen.getByText('重试'))
-    
-    const selectBtn = await screen.findByText('选择此目录')
-    
-    // Fail creation
-    createWorkspaceMock.mockRejectedValue(new Error('Already exists'))
-    fireEvent.click(selectBtn)
-
-    expect(await screen.findByText('Already exists')).toBeTruthy()
-  })
-})