| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196 |
- import { Context } from '@deepseek-ai/cordis'
- import AgentRegistry from '@deepseek-ai/dsh-agent'
- import type { Agent } from '@deepseek-ai/dsh-agent'
- import { createUserMessage } from '@deepseek-ai/dsh-llm'
- import SessionStore, { SessionId } from '@deepseek-ai/dsh-session'
- import type { SessionEvent, SessionHeader } from '@deepseek-ai/dsh-session'
- import { describe, expect, it, vi } from 'vitest'
- import SessionController from '../src/index.ts'
- import type { ApiSessionAgentController } from '../src/agent.ts'
- import { createSessionTestController, testSessionPersistence } from './test-remote.ts'
- const defaults = {
- defaultModelSelection: () => ({ provider: 'fixture', model: 'fixture-model' }),
- cwd: '/tmp',
- }
- describe('SessionController facade', () => {
- it('does not require the Tools service', () => {
- expect(SessionController.inject).not.toContain('tools')
- })
- it('owns Host service methods and publishes Agent lifecycle projections', async () => {
- const ctx = new Context()
- await ctx.plugin(SessionStore)
- await ctx.plugin(AgentRegistry)
- const sessionId = SessionId('controller-session')
- const header: SessionHeader = {
- version: 0,
- id: sessionId,
- createdAt: 1,
- cwd: '/workspace',
- }
- const events: SessionEvent[] = []
- const inspect = vi.fn(() => Promise.resolve({ meta: header, events }))
- ctx.provide('sessionPersistence', testSessionPersistence(ctx, {
- list: () => Promise.resolve([header]),
- inspect,
- }) as never)
- const controller = createSessionTestController(ctx, defaults)
- const status = vi.fn()
- const failure = vi.fn()
- const activity = vi.fn()
- ctx.on('api-session/status', status)
- ctx.on('api-session/error', failure)
- ctx.on('api-session/activity', activity)
- await expect(controller.inspect(sessionId)).resolves.toEqual({ meta: header, events })
- expect(inspect).toHaveBeenCalledOnce()
- const session = ctx.sessions.create(sessionId, { meta: header })
- const agent = {
- id: sessionId,
- session,
- status: 'idle',
- ctx,
- } as Agent
- ctx.agents.register(agent)
- const consumeSelection = vi.spyOn(
- (controller as unknown as { agents: ApiSessionAgentController }).agents,
- 'consumeSelection',
- )
- await expect(controller.resolveAgent(sessionId)).resolves.toEqual({ agent })
- await expect(controller.inspect(sessionId)).resolves.toEqual({ meta: header, events })
- expect(inspect).toHaveBeenCalledOnce()
- ctx.emit('agent/status', { agent, status: 'running' })
- ctx.emit('agent/error', { agent, turn: 1, step: 0, error: new Error('fixture failure') })
- session.append('user/message', createUserMessage({
- content: [{ type: 'text', text: 'hello' }],
- source: { kind: 'user' },
- }), { surfaceOp: 'append' })
- expect(status).toHaveBeenCalledWith(sessionId, true)
- expect(failure).toHaveBeenCalledWith(sessionId, expect.stringContaining('fixture failure'))
- expect(activity).toHaveBeenCalledWith(sessionId, expect.any(Number))
- session.append('request/header', {
- header: { config: { provider: 'fixture', model: 'fixture-model' } },
- reason: 'initial',
- })
- expect(consumeSelection).toHaveBeenCalledWith(
- agent, 'fixture', 'fixture-model', undefined,
- )
- const unowned = ctx.sessions.create(SessionId('controller-unowned'), {
- meta: { cwd: '/workspace' },
- })
- unowned.append('request/header', {
- header: { config: { provider: 'fixture', model: 'other-model' } },
- reason: 'initial',
- })
- expect(consumeSelection).toHaveBeenCalledTimes(1)
- const abort = new AbortController()
- const iterator = controller.follow({
- address: { kind: 'session', sessionId },
- }, abort.signal)[Symbol.asyncIterator]()
- await expect(iterator.next()).resolves.toMatchObject({
- done: false,
- value: { type: 'snapshot', cursor: 1 },
- })
- abort.abort()
- await expect(iterator.next()).resolves.toEqual({ done: true, value: undefined })
- })
- it.each(['success', 'domain-error', 'throw'] as const)(
- 'promotes a prepared follow observation in the background: %s',
- async (outcome) => {
- const ctx = new Context()
- await ctx.plugin(SessionStore)
- await ctx.plugin(AgentRegistry)
- const sessionId = SessionId(`background-${outcome}`)
- const header: SessionHeader = {
- version: 0, id: sessionId, createdAt: 1, cwd: '/workspace',
- }
- ctx.provide('sessionPersistence', testSessionPersistence(ctx, {
- list: () => Promise.resolve([header]),
- inspect: () => Promise.resolve({ meta: header, events: [] }),
- }) as never)
- const controller = createSessionTestController(ctx, defaults)
- const agents = (controller as unknown as { agents: ApiSessionAgentController }).agents
- const apiError = vi.fn()
- ctx.on('api-session/error', apiError)
- const logError = vi.spyOn(ctx.logger, 'error').mockImplementation(() => {})
- const live = { id: sessionId, session: { id: sessionId }, ctx, status: 'idle' } as unknown as Agent
- const resolve = vi.spyOn(agents, 'resolveObservedAgent')
- if (outcome === 'success') resolve.mockResolvedValue({ agent: live })
- else if (outcome === 'domain-error') {
- resolve.mockResolvedValue({
- error: { code: 'internal', message: 'activation unavailable', details: {} },
- })
- } else {
- resolve.mockRejectedValue(new Error('activation crashed'))
- }
- const abort = new AbortController()
- const iterator = controller.follow({
- address: { kind: 'session', sessionId },
- }, abort.signal)[Symbol.asyncIterator]()
- await expect(iterator.next()).resolves.toMatchObject({ value: { type: 'snapshot' } })
- const waiting = iterator.next()
- await vi.waitFor(() => { expect(resolve).toHaveBeenCalledOnce() })
- if (outcome === 'domain-error') {
- await vi.waitFor(() => {
- expect(apiError).toHaveBeenCalledWith(sessionId, 'activation unavailable')
- })
- } else if (outcome === 'throw') {
- await vi.waitFor(() => {
- expect(logError).toHaveBeenCalledWith(expect.stringContaining('activation crashed'))
- })
- } else {
- expect(apiError).not.toHaveBeenCalled()
- }
- abort.abort()
- await expect(waiting).resolves.toMatchObject({ done: true })
- await ctx.fiber.dispose()
- },
- )
- it('waits for an admitted background promotion during teardown', async () => {
- const ctx = new Context()
- await ctx.plugin(SessionStore)
- await ctx.plugin(AgentRegistry)
- const sessionId = SessionId('background-disposal')
- const header: SessionHeader = {
- version: 0, id: sessionId, createdAt: 1, cwd: '/workspace',
- }
- ctx.provide('sessionPersistence', testSessionPersistence(ctx, {
- list: () => Promise.resolve([header]),
- inspect: () => Promise.resolve({ meta: header, events: [] }),
- }) as never)
- const controller = createSessionTestController(ctx, defaults)
- const agents = (controller as unknown as { agents: ApiSessionAgentController }).agents
- const started = Promise.withResolvers<undefined>()
- const release = Promise.withResolvers<undefined>()
- vi.spyOn(agents, 'resolveObservedAgent').mockImplementation(async () => {
- started.resolve(undefined)
- await release.promise
- return {
- agent: { id: sessionId, session: { id: sessionId }, ctx, status: 'idle' } as unknown as Agent,
- }
- })
- const iterator = controller.follow({
- address: { kind: 'session', sessionId },
- }, new AbortController().signal)[Symbol.asyncIterator]()
- await expect(iterator.next()).resolves.toMatchObject({ value: { type: 'snapshot' } })
- const waiting = iterator.next()
- await started.promise
- let disposed = false
- const disposal = ctx.fiber.dispose().then(() => { disposed = true })
- await Promise.resolve()
- expect(disposed).toBe(false)
- release.resolve(undefined)
- await disposal
- await expect(waiting).resolves.toMatchObject({ done: true })
- })
- })
|