| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364 |
- // @vitest-environment jsdom
- // ChatView behavior: flow derivation, streaming isolation (Profiler counts),
- // toolview dispatch and selection handoff — driven through a scripted
- // ObservableSnapshot fake, no wire.
- import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
- import { Profiler } from 'react'
- import { act, cleanup, fireEvent, render } from '@testing-library/react'
- import type {
- AssistantMessageNode, ConversationNode, ConversationSnapshot, RunningToolCall, SessionId, SessionListState, ToolResultNode, UserMessageNode, WorkspaceListState,
- } from '@deepseek-ai/dsh-client-runtime/client'
- import { bindSnapshotSelector } from '@deepseek-ai/dsh-client-web-react'
- import { createSnapshotStore, PendingWait } from '@deepseek-ai/dsh-client-runtime/client'
- import { RpcId } from '@deepseek-ai/dsh-client-connection/client'
- import type { ChatViewSlotProps, SelectionTarget } from '@deepseek-ai/dsh-client-ui-conversation/client'
- import { createChatStore } from '../src/client/stores.ts'
- import { ChatView } from '../src/client/chat/ChatView.tsx'
- import { deriveChatFlow, flowKeys } from '../src/client/chat/chat-flow.ts'
- afterEach(cleanup)
- // Keyless create() persists under the bare declared key; clear between cases
- // so one harness's selection cannot rehydrate into the next.
- beforeEach(() => {
- localStorage.clear()
- })
- const SID = 's1' as SessionId
- function snapshotBase(): ConversationSnapshot {
- return {
- sessionId: SID, nodes: [], foldDegraded: false, partial: null, runningCalls: [], codeDispatches: new Map(),
- pending: [], queue: [], todos: [], running: false, composerPhase: 'active', removed: false, openState: 'open', openError: null,
- hasMore: false, loadingOlder: false, promptError: null, blank: false, lastAgentError: null,
- }
- }
- /** Scripted snapshot source: set() swaps the top-level object like the real Session. */
- function makeSource(init?: Partial<ConversationSnapshot>) {
- let snap: ConversationSnapshot = { ...snapshotBase(), ...init }
- const subs = new Set<() => void>()
- return {
- set(next: Partial<ConversationSnapshot>) {
- snap = { ...snap, ...next }
- for (const fn of [...subs]) fn()
- },
- source: {
- getSnapshot: () => snap,
- subscribe: (fn: () => void) => {
- subs.add(fn)
- return () => subs.delete(fn)
- },
- },
- }
- }
- const user = (seq: number, text: string): UserMessageNode => ({
- kind: 'user', seq, time: seq * 1_000, content: [{ type: 'text', text }] as never, source: null,
- })
- const assistant = (seq: number, text: string): AssistantMessageNode => ({
- kind: 'assistant', seq, time: seq * 1_000, turn: 1, step: 1, blocks: [{ kind: 'text', text }],
- })
- const toolResult = (seq: number, callId: string, name = 'bash'): ToolResultNode => ({
- kind: 'tool-result', seq, time: seq * 1_000, callId,
- call: { name, argsRaw: `{"command":"cmd-${callId}","description":"run ${callId}"}` },
- callTime: seq * 1_000 - 500,
- content: [], isError: false, callView: null, resultView: null,
- })
- const runningCall = (callId: string, name = 'bash'): RunningToolCall => ({
- callId, name, argsRaw: `{"command":"cmd-${callId}"}`, turn: 2, step: 1, time: 1_000, callView: null,
- })
- /** Empty sessions-list hook for the global standard-kit seat. */
- function emptySessions() {
- const store = createSnapshotStore<SessionListState>(
- { ids: [], byId: {}, current: undefined, phase: 'ready' })
- return bindSnapshotSelector(store)
- }
- function emptyWorkspaces() {
- const store = createSnapshotStore<WorkspaceListState>({
- items: [], state: 'idle', phase: 'ready', error: null,
- baselinesReady: true, recentWorkspaceId: undefined,
- })
- return bindSnapshotSelector(store)
- }
- function makeHarness(init?: Partial<ConversationSnapshot>) {
- const { set, source } = makeSource(init)
- const openDetails = vi.fn<(t: SelectionTarget) => void>()
- const loadOlder = vi.fn()
- // Selection rides the REAL chat store (same construction path as
- // production; the view reads it through the PropsStore useStore share).
- // renderSlot stub renders the render-site fallback (an empty keyed ledger:
- // every tool lands on GenericToolCard); keyed dispatch to registered rows
- // is the slot machinery's behavior, covered by its own specs.
- const chat = createChatStore().create()
- const renderSlot = ((_key: string, _owner: object, opts?: { fallback?: React.ReactNode }) =>
- opts?.fallback ?? null) as unknown as ChatViewSlotProps['renderSlot']
- // SessionProvider seat arrives with the session-scope child declaration;
- // ChatView never invokes it (render-prop pass-through stub).
- const SessionProviderStub: ChatViewSlotProps['SessionProvider'] = ({ children }) => <>{children(SID)}</>
- const props: ChatViewSlotProps = {
- sessionId: SID,
- useSession: bindSnapshotSelector(source),
- useSessions: emptySessions(),
- useWorkspaces: emptyWorkspaces(),
- useInput: (() => { throw new Error('unused') }) as never,
- inputActions: { setDraft: () => {}, submit: () => {} } as never,
- useStore: bindSnapshotSelector(chat),
- actions: chat.actions,
- renderSlot,
- SessionProvider: SessionProviderStub,
- openDetails,
- loadOlder,
- }
- const setSelection = (next: SelectionTarget | null): void => { chat.actions.select(next) }
- return { set, ChatView, props, openDetails, loadOlder, setSelection }
- }
- describe('chat-flow derivation', () => {
- it('groups consecutive tool results and keeps stable keys', () => {
- const nodes: ConversationNode[] = [
- user(1, 'hi'), assistant(2, 'let me look'), toolResult(3, 'a'), toolResult(4, 'b'),
- assistant(5, 'found'), toolResult(6, 'c'),
- ]
- const items = deriveChatFlow(nodes)
- expect(items.map((i) => i.kind)).toEqual(['node', 'node', 'tool-group', 'node', 'tool-group'])
- const group = items[2]!
- expect(group.kind === 'tool-group' && group.results.map((r) => r.callId)).toEqual(['a', 'b'])
- expect(flowKeys(items)).toBe('n1|n2|g3|n5|g6')
- expect(flowKeys(deriveChatFlow([...nodes, toolResult(7, 'd')]))).toBe('n1|n2|g3|n5|g6')
- })
- })
- describe('ChatView', () => {
- it('a windowless tool result (call head truncated) renders with an empty tool name', () => {
- const h = makeHarness({
- nodes: [{ ...toolResult(3, 'w1'), call: null }],
- })
- const view = render(<h.ChatView {...h.props} />)
- // classifyTool('') → others; the summary slot falls back to the callId.
- expect(view.container.querySelector('[data-variant="others"]')).not.toBeNull()
- expect(view.getByText('w1')).toBeTruthy()
- })
- it('prepend keeps the viewport anchored when the reader is NOT at the bottom (no lastKey force)', () => {
- // Covers the prepend early-return arm where lastItem exists but the key
- // path is not taken (anchor branch wins before the appended-user check).
- const h = makeHarness({ nodes: [user(9, 'late')], hasMore: true })
- const view = render(<h.ChatView {...h.props} />)
- const scroller = view.container.querySelector('[class*="scroll"]') as HTMLDivElement
- Object.defineProperty(scroller, 'scrollHeight', { value: 800, writable: true })
- Object.defineProperty(scroller, 'clientHeight', { value: 200, writable: true })
- scroller.scrollTop = 50
- fireEvent.scroll(scroller)
- fireEvent.click(view.getByText('加载更早'))
- Object.defineProperty(scroller, 'scrollHeight', { value: 1300, writable: true })
- act(() => h.set({ nodes: [assistant(2, 'older'), user(9, 'late')] }))
- expect(scroller.scrollTop).toBe(550) // 50 + (1300 - 800)
- })
- it('renders the fixture main line: bubble, narration, grouped tool rows', () => {
- const h = makeHarness({
- nodes: [user(1, 'do the thing'), assistant(2, 'running tools'), toolResult(3, 'a'), toolResult(4, 'b')],
- })
- const view = render(<h.ChatView {...h.props} />)
- expect(view.getByText('do the thing')).toBeTruthy()
- expect(view.getByText('running tools')).toBeTruthy()
- expect(view.getAllByText('Bash')).toHaveLength(2)
- expect(view.getByText('run a')).toBeTruthy()
- })
- it('renders assistant Markdown across history, streaming, final, and interrupted states while user text stays literal', () => {
- const markdown = '# Rendered\n\n- **one**\n- `two`'
- const h = makeHarness({ nodes: [user(1, markdown), assistant(2, markdown)] })
- const view = render(<h.ChatView {...h.props} />)
- expect(view.container.querySelectorAll('h1')).toHaveLength(1)
- const literal = view.getByText((_content, element) => (
- element?.tagName === 'DIV' && element.childElementCount === 0 && element.textContent === markdown
- ))
- expect(literal.querySelector('h1')).toBeNull()
- act(() => {
- h.set({ partial: { turn: 2, step: 1, blocks: [{ kind: 'text', text: markdown }] } })
- })
- expect(view.container.querySelectorAll('h1')).toHaveLength(2)
- expect(view.container.querySelector('[data-streaming="true"] h1')?.textContent).toBe('Rendered')
- act(() => {
- h.set({
- nodes: [user(1, markdown), assistant(2, markdown), assistant(3, markdown)],
- partial: null,
- })
- })
- expect(view.container.querySelectorAll('h1')).toHaveLength(2)
- expect(view.container.querySelector('[data-streaming="true"]')).toBeNull()
- act(() => {
- h.set({
- nodes: [
- user(1, markdown),
- assistant(2, markdown),
- { ...assistant(3, markdown), interrupted: true },
- ],
- })
- })
- expect(view.getByText('已停止')).toBeTruthy()
- expect(view.container.querySelectorAll('h1')).toHaveLength(2)
- })
- it('streaming partial frames re-render only the tail (Profiler count)', () => {
- const h = makeHarness({
- nodes: [user(1, 'q'), assistant(2, 'old answer'), toolResult(3, 'a')],
- })
- let renders = 0
- const counting = (
- <Profiler id="chat" onRender={() => { renders += 1 }}>
- <h.ChatView {...h.props} />
- </Profiler>
- )
- const view = render(counting)
- const before = renders
- const beforeHtml = view.container.querySelector('[class*="toolGroup"]')!.innerHTML
- act(() => {
- h.set({ partial: { turn: 2, step: 1, blocks: [{ kind: 'text', text: 'streaming…' }] } })
- })
- act(() => {
- h.set({ partial: { turn: 2, step: 1, blocks: [{ kind: 'text', text: 'streaming… more' }] } })
- })
- expect(view.getByText('streaming… more')).toBeTruthy()
- // Each chunk commits exactly one profiler pass (the tail), never a full-tree storm.
- expect(renders - before).toBe(2)
- expect(view.container.querySelector('[class*="toolGroup"]')!.innerHTML).toBe(beforeHtml)
- })
- it('streaming leaves neighbor tool rows and history items at zero re-renders', () => {
- const h = makeHarness({
- nodes: [user(1, 'q'), assistant(2, 'old'), toolResult(3, 'a')],
- })
- // Count renderSlot invocations: the memo boundary holds when CallRow does
- // not re-render, so the row's renderSlot call count freezes during chunks.
- let rowRenders = 0
- h.props.renderSlot = (((_key: string, _owner: object) => {
- rowRenders += 1
- return <div data-testid="counting-row" />
- }) as unknown as ChatViewSlotProps['renderSlot'])
- const view = render(<h.ChatView {...h.props} />)
- expect(view.getByTestId('counting-row')).toBeTruthy()
- const afterMount = rowRenders
- act(() => {
- h.set({ partial: { turn: 2, step: 1, blocks: [{ kind: 'text', text: 'chunk1' }] } })
- })
- act(() => {
- h.set({ partial: { turn: 2, step: 1, blocks: [{ kind: 'text', text: 'chunk1 chunk2' }] } })
- })
- expect(rowRenders).toBe(afterMount)
- })
- it('tool row expands to the args body via the leading slot toggle', () => {
- const h = makeHarness({ nodes: [toolResult(3, 'a')] })
- const view = render(<h.ChatView {...h.props} />)
- expect(view.queryByText(/"command": "cmd-a"/)).toBeNull()
- fireEvent.click(view.container.querySelector('button[aria-expanded]')!)
- expect(view.getByText(/"command": "cmd-a"/)).toBeTruthy()
- })
- it('clicking a tool row opens details with callId and toolName; selection marks data-selected', () => {
- const h = makeHarness({ nodes: [toolResult(3, 'a')] })
- const view = render(<h.ChatView {...h.props} />)
- fireEvent.click(view.getByText('run a'))
- expect(h.openDetails).toHaveBeenCalledWith({ turnSeq: 3, callId: 'a', toolName: 'bash' })
- expect(view.container.querySelector('[data-selected]')).toBeNull()
- act(() => h.setSelection({ turnSeq: 3, callId: 'a', toolName: 'bash' }))
- expect(view.container.querySelector('[data-selected]')).not.toBeNull()
- })
- it('running calls render as a live tool group with the running state', () => {
- const h = makeHarness({ runningCalls: [runningCall('r1')], running: true })
- const view = render(<h.ChatView {...h.props} />)
- expect(view.container.querySelector('[data-state="running"]')).not.toBeNull()
- expect(view.getByText('cmd-r1')).toBeTruthy()
- })
- it('dispatches each tool row through the keyed slot with the tool name as entryKey', () => {
- const h = makeHarness({ nodes: [toolResult(3, 'a')] })
- const calls: { key: string; entryKey?: string }[] = []
- h.props.renderSlot = (((key: string, _owner: object, opts?: { entryKey?: string; fallback?: React.ReactNode }) => {
- calls.push({ key, ...(opts?.entryKey !== undefined ? { entryKey: opts.entryKey } : {}) })
- return opts?.fallback ?? null
- }) as unknown as ChatViewSlotProps['renderSlot'])
- render(<h.ChatView {...h.props} />)
- // Keyed dispatch: slot name is the declared hole, entryKey the wire tool
- // name, and the fallback (GenericToolCard) renders on an empty ledger.
- // (Registered-row takeover and live unload are slot machinery behavior,
- // owned by the slot system's own specs.)
- expect(calls).toEqual([{ key: 'conversation.chat.toolview', entryKey: 'bash' }])
- })
- it('prepend compensates scrollTop by the height delta; a trailing user node force-scrolls', () => {
- const h = makeHarness({ nodes: [user(5, 'later'), assistant(6, 'a')], hasMore: true })
- const view = render(<h.ChatView {...h.props} />)
- const scroller = view.container.querySelector('[class*="scroll"]') as HTMLDivElement
- // jsdom has no layout: fake the metrics the anchor math reads.
- Object.defineProperty(scroller, 'scrollHeight', { value: 1000, writable: true })
- Object.defineProperty(scroller, 'clientHeight', { value: 400, writable: true })
- // Arm the paging anchor, then deliver an older page (head seq decreases).
- fireEvent.click(view.getByText('加载更早'))
- Object.defineProperty(scroller, 'scrollHeight', { value: 1600, writable: true })
- act(() => h.set({ nodes: [user(1, 'old'), assistant(2, 'b'), user(5, 'later'), assistant(6, 'a')] }))
- expect(scroller.scrollTop).toBe(600) // 0 + (1600 - 1000)
- // A new trailing user bubble (own words) force-scrolls to the bottom.
- act(() => h.set({ nodes: [user(1, 'old'), assistant(2, 'b'), user(5, 'later'), assistant(6, 'a'), user(9, 'mine')] }))
- expect(scroller.scrollTop).toBe(1600)
- })
- it('scrolling away disables follow and shows the back-to-bottom button; clicking returns', () => {
- const h = makeHarness({ nodes: [user(1, 'q'), assistant(2, 'a')] })
- const view = render(<h.ChatView {...h.props} />)
- const scroller = view.container.querySelector('[class*="scroll"]') as HTMLDivElement
- Object.defineProperty(scroller, 'scrollHeight', { value: 1000, writable: true })
- Object.defineProperty(scroller, 'clientHeight', { value: 300, writable: true })
- scroller.scrollTop = 100 // far from bottom
- fireEvent.scroll(scroller)
- const backButton = view.getByLabelText('回到底部')
- expect(backButton).toBeTruthy()
- // Streaming growth must NOT drag a scrolled-away reader down.
- act(() => h.set({ partial: { turn: 1, step: 1, blocks: [{ kind: 'text', text: 'grow' }] } }))
- expect(scroller.scrollTop).toBe(100)
- fireEvent.click(backButton)
- expect(scroller.scrollTop).toBe(1000)
- // At the bottom again: follow re-arms and the button unmounts.
- expect(view.queryByLabelText('回到底部')).toBeNull()
- })
- it('paging button loads older and shows its busy label', () => {
- const h = makeHarness({ nodes: [user(5, 'later')], hasMore: true })
- const view = render(<h.ChatView {...h.props} />)
- fireEvent.click(view.getByText('加载更早'))
- expect(h.loadOlder).toHaveBeenCalledTimes(1)
- act(() => h.set({ loadingOlder: true }))
- expect(view.getByText('加载中…')).toBeTruthy()
- })
- it('shows open error and loading states', () => {
- const h = makeHarness({
- openState: 'error',
- openError: { code: 'internal', message: 'boom' } as never,
- })
- const view = render(<h.ChatView {...h.props} />)
- expect(view.getByText(/历史加载失败:boom/)).toBeTruthy()
- const loading = makeHarness({ openState: 'loading' })
- const lv = render(<loading.ChatView {...loading.props} />)
- expect(lv.getByText('载入历史…')).toBeTruthy()
- })
- it('pending interactions render placeholder cards', () => {
- const h = makeHarness({
- pending: [new PendingWait('approval', RpcId('r1'), SID,
- { approvalId: 'ap1', toolName: 'bash' } as PendingWait<'approval'>['payload'], vi.fn())],
- })
- const view = render(<h.ChatView {...h.props} />)
- expect(view.getByText(/等待审批/)).toBeTruthy()
- })
- })
|