| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143 |
- /** ui-subagent browser half: catalog actions and read-only composer routing. */
- import { Context } from '@deepseek-ai/cordis'
- import { stubSettingsScope } from '@deepseek-ai/dsh-client-test-runtime'
- import { describe, expect, it } from 'vitest'
- import type {
- SessionListState, SessionSnapshot, SessionSummary,
- } from '@deepseek-ai/dsh-api-session-controller/client'
- import type { SubagentAddress } from '@deepseek-ai/dsh-subagent/client'
- import { SlotRegistry } from '@deepseek-ai/dsh-client-ui-renderer/client'
- import type { SessionId } from '@deepseek-ai/dsh-session/types'
- import type { ComposerChainProps } from '@deepseek-ai/dsh-client-ui-conversation/client'
- import { apply as applyLocale, inject as localeInject } from '@deepseek-ai/dsh-client-locale/client'
- import {
- SubagentHeaderLineage, type SubagentCatalogInjected,
- } from '../src/client/SubagentHeaderLineage.tsx'
- import {
- SubagentReadOnlyComposer, type SubagentReadOnlyMatch,
- } from '../src/client/SubagentReadOnlyComposer.tsx'
- import { apply, inject } from '../src/client/index.ts'
- function summary(partial: Partial<SessionSummary> & { id: SessionId }): SessionSummary {
- return {
- displayTitle: partial.id,
- running: false,
- updatedAt: 0,
- ...partial,
- } as SessionSummary
- }
- const sid = (id: string) => id as SessionId
- /** Fake root sessions face for catalog actions. */
- function sessionsWith(sessions: SessionSummary[]) {
- const byId: Record<string, SessionSummary> = {}
- for (const s of sessions) byId[s.id] = s
- const snapshot = { ids: sessions.map(s => s.id), byId, current: undefined } as unknown as SessionListState
- const actionCalls: { method: string; args: unknown[] }[] = []
- return {
- list: {
- getSnapshot: () => snapshot,
- subscribe: () => () => {},
- },
- actionCalls,
- openSubagent: (address: SubagentAddress) => {
- actionCalls.push({ method: 'openSubagent', args: [address] })
- },
- refreshSubagents: (parentSessionId: SessionId) => {
- actionCalls.push({ method: 'refreshSubagents', args: [parentSessionId] })
- return Promise.resolve()
- },
- setSubagentCatalogOpen: (parentSessionId: SessionId, open: boolean) => {
- actionCalls.push({ method: 'setSubagentCatalogOpen', args: [parentSessionId, open] })
- },
- }
- }
- async function provideSlotFaces(ctx: Context): Promise<void> {
- await ctx.plugin(SlotRegistry).await()
- ctx.slots.register({
- name: 'root',
- children: {
- 'conversation.session.header.lineage': { kind: 'single', scope: 'session' },
- 'conversation.composer': { kind: 'chain', scope: 'session' },
- },
- } as never, () => null)
- }
- /** Boot the plugin over fake sessions and slot faces. */
- async function fullBench(sessions: SessionSummary[]) {
- const ctx = new Context()
- const face = sessionsWith(sessions)
- ctx.provide('sessions', face)
- ctx.provide('remote', { $on: () => () => {} } as never)
- ctx.provide('settingsScope', { bind: () => stubSettingsScope().scope } as never)
- await provideSlotFaces(ctx)
- await ctx.plugin({ inject: localeInject, apply: applyLocale }).await()
- await ctx.plugin({ inject: [...inject], apply }).await()
- return { face, ctx }
- }
- const FAMILY: SessionSummary[] = [
- summary({ id: sid('parent'), displayTitle: 'parent', running: true }),
- summary({ id: sid('c1'), parentId: sid('parent'), displayTitle: 'worker-1', running: true }),
- summary({ id: sid('c2'), parentId: sid('parent'), displayTitle: 'worker-2', running: true }),
- // Filtered out: not running / other parent / label miss.
- summary({ id: sid('c3'), parentId: sid('parent'), displayTitle: 'worker-3', running: false }),
- summary({ id: sid('c4'), parentId: sid('other'), displayTitle: 'worker-4', running: true }),
- summary({ id: sid('c5'), parentId: sid('parent'), displayTitle: 'scout', running: true }),
- ]
- describe('apply', () => {
- it('declares the services it binds', () => {
- expect(inject).toEqual(['sessions', 'slots', 'locale'])
- })
- it('registers catalog actions and selects read-only subagent composers from session facts', async () => {
- const { ctx, face } = await fullBench(FAMILY)
- const catalogEntry = ctx.slots.entries('conversation.session.header.lineage')
- .find(entry => entry.component === SubagentHeaderLineage)!
- const actions = (catalogEntry.inject as unknown as (id: SessionId) => SubagentCatalogInjected)(sid('parent'))
- const address: SubagentAddress = {
- parentSessionId: sid('parent'),
- childSessionId: sid('c1'),
- mode: 'continuable',
- }
- actions.openChild(address)
- actions.refresh(sid('parent'))
- actions.setCatalogOpen(sid('parent'), true)
- expect(face.actionCalls).toEqual([
- { method: 'openSubagent', args: [address] },
- { method: 'refreshSubagents', args: [sid('parent')] },
- { method: 'setSubagentCatalogOpen', args: [sid('parent'), true] },
- ])
- const composerEntry = ctx.slots.entries('conversation.composer')
- .find(entry => entry.component === SubagentReadOnlyComposer)!
- const select = composerEntry.select as (owner: ComposerChainProps) => SubagentReadOnlyMatch | null
- const owner = (
- subagent: SessionSnapshot['subagent'] | undefined,
- running = false,
- ): ComposerChainProps => ({
- sessionId: subagent?.address.childSessionId,
- session: subagent === undefined
- ? undefined
- : ({ subagent, running } as SessionSnapshot),
- pendingInteraction: undefined,
- })
- expect(select(owner(undefined))).toBeNull()
- expect(select(owner(null))).toBeNull()
- expect(select(owner({ address: { ...address, mode: 'one-shot' }, parentAvailable: true })))
- .toEqual({ reason: 'one-shot' })
- // One-shot stays read-only even while running: it has no stop action.
- expect(select(owner({ address: { ...address, mode: 'one-shot' }, parentAvailable: true }, true)))
- .toEqual({ reason: 'one-shot' })
- expect(select(owner({ address }))).toBeNull()
- expect(select(owner({ address, parentAvailable: true }))).toBeNull()
- expect(select(owner({ address, parentAvailable: false })))
- .toEqual({ reason: 'parent-unavailable' })
- // A RUNNING parent-offline continuable yields the default composer, whose
- // disabled input still carries the primary Stop; stopped, it takes back over.
- expect(select(owner({ address, parentAvailable: false }, true))).toBeNull()
- })
- })
|