feat(session-export): add command and Header action

packages/client/ui-trajectory/tests/export-log.client.spec.ts

@@ -1,51 +0,0 @@
-// @vitest-environment jsdom
-/**
- * Session-log export browser delivery: safe filename derivation and a native
- * download handoff that leaves the streamed response outside JavaScript.
- */
-
-import { afterEach, describe, expect, it, vi } from 'vitest'
-import { downloadSessionLog, sessionLogZipFilename } from '../src/client/export-log.ts'
-
-afterEach(() => {
-  vi.restoreAllMocks()
-})
-
-describe('sessionLogZipFilename', () => {
-  it('keeps safe session ids verbatim', () => {
-    expect(sessionLogZipFilename('session-abc_1-2')).toBe('dsh-session-session-abc_1-2.zip')
-  })
-
-  it('neutralizes unsafe id characters that could shape the filename', () => {
-    expect(sessionLogZipFilename('../evil')).toBe('dsh-session-___evil.zip')
-    expect(sessionLogZipFilename('a/b')).toBe('dsh-session-a_b.zip')
-  })
-
-  it('strips dots so a dot-only id cannot shape a dot segment', () => {
-    expect(sessionLogZipFilename('..')).toBe('dsh-session-__.zip')
-  })
-})
-
-describe('downloadSessionLog', () => {
-  it('hands the descendant-inclusive endpoint directly to the browser', async () => {
-    const click = vi.spyOn(HTMLAnchorElement.prototype, 'click').mockImplementation(() => {})
-
-    await downloadSessionLog('session/with spaces')
-
-    expect(click).toHaveBeenCalledOnce()
-    const anchor = click.mock.contexts[0] as HTMLAnchorElement
-    const url = new URL(anchor.href)
-    expect(url.pathname).toBe('/api/session.export')
-    expect(url.searchParams.get('sessionId')).toBe('session/with spaces')
-    expect(url.searchParams.get('includeDescendants')).toBe('true')
-    expect(anchor.download).toBe('dsh-session-session_with_spaces.zip')
-  })
-
-  it('rejects when the browser download handoff fails', async () => {
-    vi.spyOn(HTMLAnchorElement.prototype, 'click').mockImplementation(() => {
-      throw new Error('download denied')
-    })
-
-    await expect(downloadSessionLog('session-root')).rejects.toThrow('download denied')
-  })
-})

packages/client/ui-trajectory/tests/toolbar.client.spec.tsx

@@ -1,61 +0,0 @@
-// @vitest-environment jsdom
-/** Trajectory toolbar export button: click dispatch, in-flight disable, and error surfacing. */
-
-import { afterEach, describe, expect, it, vi } from 'vitest'
-import { cleanup, fireEvent, render, screen } from '@testing-library/react'
-import type { LocaleKeysOf } from '@deepseek-ai/dsh-client-ui-slots'
-import { TrajectoryToolbar, type TrajectoryToolbarProps } from '../src/client/TrajectoryToolbar.tsx'
-import { zh, type TrajectoryKey } from '../src/client/locales.ts'
-
-/** Test translator pinned to the Simplified Chinese dictionary. */
-const zhT = (key: LocaleKeysOf<'trajectory'>): string => zh[key as TrajectoryKey] ?? key
-
-afterEach(() => {
-  cleanup()
-  vi.restoreAllMocks()
-})
-
-function baseProps(overrides: Partial<TrajectoryToolbarProps> = {}): TrajectoryToolbarProps {
-  return {
-    actualDuration: false,
-    onActualDurationChange: vi.fn(),
-    actualTime: false,
-    onActualTimeChange: vi.fn(),
-    allTurnsCollapsed: false,
-    onToggleAllTurns: vi.fn(),
-    allAssistantsCollapsed: false,
-    onToggleAllAssistants: vi.fn(),
-    searchQuery: '',
-    onSearchQueryChange: vi.fn(),
-    exporting: false,
-    onExport: vi.fn(),
-    exportError: null,
-    t: zhT,
-    ...overrides,
-  }
-}
-
-describe('TrajectoryToolbar export', () => {
-  it('renders the export button and dispatches the export callback on click', () => {
-    const onExport = vi.fn()
-    render(<TrajectoryToolbar {...baseProps({ onExport })} />)
-    const button = screen.getByRole('button', { name: 'Export session log' })
-    fireEvent.click(button)
-    expect(onExport).toHaveBeenCalledTimes(1)
-  })
-
-  it('disables the button while an export is in flight and blocks dispatch', () => {
-    const onExport = vi.fn()
-    render(<TrajectoryToolbar {...baseProps({ exporting: true, onExport })} />)
-    const button = screen.getByRole('button', { name: 'Export session log' }) as HTMLButtonElement
-    expect(button.disabled).toBe(true)
-    fireEvent.click(button)
-    expect(onExport).not.toHaveBeenCalled()
-  })
-
-  it('surfaces an export failure as the button title', () => {
-    render(<TrajectoryToolbar {...baseProps({ exportError: 'Export failed: internal boom' })} />)
-    const button = screen.getByRole('button', { name: 'Export session log' })
-    expect(button.title).toBe('Export failed: internal boom')
-  })
-})

packages/client/ui-trajectory/tests/views.client.spec.tsx

@@ -136,12 +136,6 @@ function standaloneDuration(): Pick<
   }
 }
 
-function standaloneExport(
-  onExport: () => Promise<void> = vi.fn(() => Promise.resolve()),
-): Pick<ComponentProps<typeof TrajectoryView>, 'exportLog'> {
-  return { exportLog: onExport }
-}
-
 function fakeSession(nodes: ConversationSnapshot['nodes']) {
   const store = createSnapshotStore(historySnapshot(nodes))
   return { store, useSession: bindSnapshotSelector(store) }
@@ -253,7 +247,6 @@ function mount(slots: SlotsService, nodes: ConversationSnapshot['nodes'] = NODES
         return {
           loadOlder: trajectory.loadOlder,
           setActualDuration: trajectory.setActualDuration,
-          exportLog: trajectory.exportLog,
           useDuration: bindSnapshotSelector(trajectory.hooks.duration),
           t: (key: TrajectoryKey) => zh[key],
         }
@@ -1134,7 +1127,6 @@ describe('timeline projection', () => {
         ...standaloneProps([]),
         ...standaloneHistory(historySnapshot([])),
         ...standaloneDuration(),
-        ...standaloneExport(),
       },
     ))
     expect(screen.getByRole('toolbar', { name: '轨迹工具栏' })).toBeTruthy()
@@ -1142,41 +1134,6 @@ describe('timeline projection', () => {
   })
 })
 
-describe('session log export', () => {
-  afterEach(() => {
-    vi.unstubAllGlobals()
-    Reflect.deleteProperty(HTMLAnchorElement.prototype, 'click')
-  })
-
-  it('downloads the host-streamed ZIP with descendants on click', async () => {
-    const clickAnchor = vi.fn()
-    HTMLAnchorElement.prototype.click = clickAnchor
-    const b = await bench(historySnapshot(NODES))
-    mount(b.slots)
-    fireEvent.click(screen.getByRole('tab', { name: 'Trajectory' }))
-    fireEvent.click(screen.getByRole('button', { name: 'Export session log' }))
-    await vi.waitFor(() => { expect(clickAnchor).toHaveBeenCalledOnce() })
-    const anchor = clickAnchor.mock.contexts[0] as HTMLAnchorElement
-    const url = new URL(anchor.href)
-    expect(url.pathname).toBe('/api/session.export')
-    expect(url.searchParams.get('sessionId')).toBe(SID)
-    expect(url.searchParams.get('includeDescendants')).toBe('true')
-  })
-
-  it('surfaces a browser handoff failure in the visible alert bar', async () => {
-    HTMLAnchorElement.prototype.click = vi.fn(() => { throw new Error('download denied') })
-    const b = await bench(historySnapshot(NODES))
-    mount(b.slots)
-    fireEvent.click(screen.getByRole('tab', { name: 'Trajectory' }))
-    fireEvent.click(screen.getByRole('button', { name: 'Export session log' }))
-    await vi.waitFor(() => {
-      const alert = screen.queryByRole('alert')
-      expect(alert).not.toBeNull()
-      expect(alert!.textContent).toContain('download denied')
-    })
-  })
-})
-
 describe('TrajectoryView state', () => {
   it('persists the duration preference through the runtime snapshot-store seam', () => {
     const firstDuration = createTrajectoryDurationStore()
@@ -1187,7 +1144,6 @@ describe('TrajectoryView state', () => {
     const first = render(
       <TrajectoryView
         {...commonProps}
-        {...standaloneExport()}
         useDuration={bindSnapshotSelector(firstDuration)}
         setActualDuration={(value) => { firstDuration.set(value) }}
       />,
@@ -1203,7 +1159,6 @@ describe('TrajectoryView state', () => {
     render(
       <TrajectoryView
         {...commonProps}
-        {...standaloneExport()}
         useDuration={bindSnapshotSelector(restoredDuration)}
         setActualDuration={(value) => { restoredDuration.set(value) }}
       />,
@@ -1228,7 +1183,6 @@ describe('TrajectoryView state', () => {
       <TrajectoryView
         {...standaloneProps([])}
         {...standaloneDuration()}
-        {...standaloneExport()}
         useSession={bindSnapshotSelector(store)}
         loadOlder={vi.fn(() => Promise.resolve(false))}
       />,