d25d482dc2
CI / Migrate Dev DB (push) Has been skipped
CI / Detect Version (push) Has been cancelled
CI / Migrate DB (push) Has been cancelled
CI / Build Dev ECR (./docker/app.Dockerfile, ECR_APP) (push) Has been cancelled
CI / Build Dev ECR (./docker/db.Dockerfile, ECR_MIGRATIONS) (push) Has been cancelled
CI / Build Dev ECR (./docker/pii.Dockerfile, ECR_PII) (push) Has been cancelled
CI / Build Dev ECR (./docker/realtime.Dockerfile, ECR_REALTIME) (push) Has been cancelled
CI / Deploy Trigger.dev (Dev) (push) Has been cancelled
CI / Build AMD64 (./docker/app.Dockerfile, ECR_APP, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Build AMD64 (./docker/db.Dockerfile, ECR_MIGRATIONS, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build AMD64 (./docker/pii.Dockerfile, ECR_PII, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build AMD64 (./docker/realtime.Dockerfile, ECR_REALTIME, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (./docker/app.Dockerfile, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (./docker/db.Dockerfile, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (./docker/pii.Dockerfile, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (./docker/realtime.Dockerfile, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Check Docs Changes (push) Has been cancelled
CI / Process Docs (push) Has been cancelled
CI / Create GitHub Release (push) Has been cancelled
CI / Test and Build (push) Has been cancelled
Publish CLI Package / publish-npm (push) Has been cancelled
Publish Python SDK / publish-pypi (push) Has been cancelled
Publish TypeScript SDK / publish-npm (push) Has been cancelled
247 lines
7.9 KiB
TypeScript
247 lines
7.9 KiB
TypeScript
/**
|
|
* @vitest-environment node
|
|
*/
|
|
import { copilotHttpMock, copilotHttpMockFns } from '@sim/testing'
|
|
import { NextRequest } from 'next/server'
|
|
import { beforeEach, describe, expect, it, vi } from 'vitest'
|
|
|
|
const {
|
|
mockGetAccessibleCopilotChat,
|
|
mockReconcileChatStreamMarkers,
|
|
mockReadEvents,
|
|
mockReadFilePreviewSessions,
|
|
mockGetLatestRunForStream,
|
|
} = vi.hoisted(() => ({
|
|
mockGetAccessibleCopilotChat: vi.fn(),
|
|
mockReconcileChatStreamMarkers: vi.fn(),
|
|
mockReadEvents: vi.fn(),
|
|
mockReadFilePreviewSessions: vi.fn(),
|
|
mockGetLatestRunForStream: vi.fn(),
|
|
}))
|
|
|
|
vi.mock('@sim/db', () => ({ db: {} }))
|
|
|
|
vi.mock('@sim/db/schema', () => ({
|
|
copilotChats: {
|
|
id: 'copilotChats.id',
|
|
userId: 'copilotChats.userId',
|
|
type: 'copilotChats.type',
|
|
updatedAt: 'copilotChats.updatedAt',
|
|
lastSeenAt: 'copilotChats.lastSeenAt',
|
|
},
|
|
}))
|
|
|
|
vi.mock('drizzle-orm', () => ({
|
|
and: vi.fn((...conditions: unknown[]) => ({ type: 'and', conditions })),
|
|
eq: vi.fn((field: unknown, value: unknown) => ({ type: 'eq', field, value })),
|
|
sql: Object.assign(
|
|
vi.fn((strings: TemplateStringsArray, ...values: unknown[]) => ({
|
|
type: 'sql',
|
|
strings,
|
|
values,
|
|
})),
|
|
{ raw: vi.fn() }
|
|
),
|
|
}))
|
|
|
|
vi.mock('@/lib/copilot/request/http', () => copilotHttpMock)
|
|
|
|
vi.mock('@/lib/copilot/chat/lifecycle', () => ({
|
|
getAccessibleCopilotChatAuth: mockGetAccessibleCopilotChat,
|
|
getAccessibleCopilotChatWithMessages: mockGetAccessibleCopilotChat,
|
|
}))
|
|
|
|
vi.mock('@/lib/copilot/chat/stream-liveness', () => ({
|
|
reconcileChatStreamMarkers: mockReconcileChatStreamMarkers,
|
|
}))
|
|
|
|
vi.mock('@/lib/copilot/request/session/buffer', () => ({
|
|
readEvents: mockReadEvents,
|
|
}))
|
|
|
|
vi.mock('@/lib/copilot/request/session/file-preview-session', () => ({
|
|
readFilePreviewSessions: mockReadFilePreviewSessions,
|
|
}))
|
|
|
|
vi.mock('@/lib/copilot/async-runs/repository', () => ({
|
|
getLatestRunForStream: mockGetLatestRunForStream,
|
|
}))
|
|
|
|
vi.mock('@/lib/copilot/request/session/types', () => ({
|
|
toStreamBatchEvent: (e: unknown) => e,
|
|
}))
|
|
|
|
vi.mock('@/lib/copilot/chat/effective-transcript', () => ({
|
|
buildEffectiveChatTranscript: ({ messages }: { messages: unknown[] }) => messages,
|
|
}))
|
|
|
|
vi.mock('@/lib/copilot/chat/persisted-message', () => ({
|
|
normalizeMessage: (m: unknown) => m,
|
|
}))
|
|
|
|
vi.mock('@/lib/copilot/chat-status', () => ({
|
|
chatPubSub: { publishStatusChanged: vi.fn() },
|
|
}))
|
|
|
|
vi.mock('@/lib/posthog/server', () => ({
|
|
captureServerEvent: vi.fn(),
|
|
}))
|
|
|
|
import { GET } from '@/app/api/mothership/chats/[chatId]/route'
|
|
|
|
function makeContext(chatId: string) {
|
|
return { params: Promise.resolve({ chatId }) }
|
|
}
|
|
|
|
function createRequest(chatId: string) {
|
|
return new NextRequest(`http://localhost:3000/api/mothership/chats/${chatId}`, {
|
|
method: 'GET',
|
|
})
|
|
}
|
|
|
|
describe('GET /api/mothership/chats/[chatId]', () => {
|
|
beforeEach(() => {
|
|
vi.clearAllMocks()
|
|
copilotHttpMockFns.mockAuthenticateCopilotRequestSessionOnly.mockResolvedValue({
|
|
userId: 'user-1',
|
|
isAuthenticated: true,
|
|
})
|
|
mockReconcileChatStreamMarkers.mockImplementation(
|
|
async (candidates: Array<{ chatId: string; streamId: string | null }>) =>
|
|
new Map(
|
|
candidates.map((candidate) => [
|
|
candidate.chatId,
|
|
{
|
|
chatId: candidate.chatId,
|
|
streamId: candidate.streamId,
|
|
status: candidate.streamId ? 'active' : 'inactive',
|
|
},
|
|
])
|
|
)
|
|
)
|
|
mockReadEvents.mockResolvedValue([])
|
|
mockReadFilePreviewSessions.mockResolvedValue([])
|
|
mockGetLatestRunForStream.mockResolvedValue(null)
|
|
})
|
|
|
|
it('clears activeStreamId when the redis lock has expired (stuck-yellow bug)', async () => {
|
|
mockGetAccessibleCopilotChat.mockResolvedValueOnce({
|
|
id: 'chat-stuck',
|
|
type: 'mothership',
|
|
title: 'Stuck',
|
|
messages: [],
|
|
resources: [],
|
|
conversationId: 'stream-orphaned',
|
|
createdAt: new Date('2026-05-11T12:00:00Z'),
|
|
updatedAt: new Date('2026-05-11T12:00:00Z'),
|
|
})
|
|
mockReconcileChatStreamMarkers.mockResolvedValueOnce(
|
|
new Map([['chat-stuck', { chatId: 'chat-stuck', streamId: null, status: 'inactive' }]])
|
|
)
|
|
|
|
const response = await GET(createRequest('chat-stuck'), makeContext('chat-stuck'))
|
|
expect(response.status).toBe(200)
|
|
const body = await response.json()
|
|
|
|
expect(mockReconcileChatStreamMarkers).toHaveBeenCalledWith(
|
|
[{ chatId: 'chat-stuck', streamId: 'stream-orphaned' }],
|
|
{ repairVerifiedStaleMarkers: true }
|
|
)
|
|
expect(body.success).toBe(true)
|
|
expect(body.chat.activeStreamId).toBeNull()
|
|
expect(body.chat.streamSnapshot).toBeUndefined()
|
|
expect(mockReadEvents).not.toHaveBeenCalled()
|
|
})
|
|
|
|
it('returns the live activeStreamId when redis confirms the lock', async () => {
|
|
mockGetAccessibleCopilotChat.mockResolvedValueOnce({
|
|
id: 'chat-live',
|
|
type: 'mothership',
|
|
title: 'Live',
|
|
messages: [],
|
|
resources: [],
|
|
conversationId: 'stream-live',
|
|
createdAt: new Date('2026-05-11T12:00:00Z'),
|
|
updatedAt: new Date('2026-05-11T12:00:00Z'),
|
|
})
|
|
mockGetLatestRunForStream.mockResolvedValueOnce({ status: 'active' })
|
|
|
|
const response = await GET(createRequest('chat-live'), makeContext('chat-live'))
|
|
expect(response.status).toBe(200)
|
|
const body = await response.json()
|
|
|
|
expect(body.chat.activeStreamId).toBe('stream-live')
|
|
expect(mockReadEvents).toHaveBeenCalledWith('stream-live', '0')
|
|
expect(body.chat.streamSnapshot).toBeDefined()
|
|
expect(body.chat.streamSnapshot.status).toBe('active')
|
|
})
|
|
|
|
it('uses the Redis lock owner when it differs from a stale persisted streamId', async () => {
|
|
mockGetAccessibleCopilotChat.mockResolvedValueOnce({
|
|
id: 'chat-mismatch',
|
|
type: 'mothership',
|
|
title: 'Mismatch',
|
|
messages: [],
|
|
resources: [],
|
|
conversationId: 'stream-stale',
|
|
createdAt: new Date('2026-05-11T12:00:00Z'),
|
|
updatedAt: new Date('2026-05-11T12:00:00Z'),
|
|
})
|
|
mockReconcileChatStreamMarkers.mockResolvedValueOnce(
|
|
new Map([
|
|
['chat-mismatch', { chatId: 'chat-mismatch', streamId: 'stream-live', status: 'active' }],
|
|
])
|
|
)
|
|
|
|
const response = await GET(createRequest('chat-mismatch'), makeContext('chat-mismatch'))
|
|
expect(response.status).toBe(200)
|
|
const body = await response.json()
|
|
|
|
expect(body.chat.activeStreamId).toBe('stream-live')
|
|
expect(mockReadEvents).toHaveBeenCalledWith('stream-live', '0')
|
|
})
|
|
|
|
it('returns null when the persisted stream marker is already null', async () => {
|
|
mockGetAccessibleCopilotChat.mockResolvedValueOnce({
|
|
id: 'chat-idle',
|
|
type: 'mothership',
|
|
title: 'Idle',
|
|
messages: [],
|
|
resources: [],
|
|
conversationId: null,
|
|
createdAt: new Date('2026-05-11T12:00:00Z'),
|
|
updatedAt: new Date('2026-05-11T12:00:00Z'),
|
|
})
|
|
|
|
const response = await GET(createRequest('chat-idle'), makeContext('chat-idle'))
|
|
expect(response.status).toBe(200)
|
|
|
|
expect(mockReconcileChatStreamMarkers).toHaveBeenCalledWith(
|
|
[{ chatId: 'chat-idle', streamId: null }],
|
|
{ repairVerifiedStaleMarkers: true }
|
|
)
|
|
const body = await response.json()
|
|
expect(body.chat.activeStreamId).toBeNull()
|
|
})
|
|
|
|
it('returns 404 when the chat does not exist', async () => {
|
|
mockGetAccessibleCopilotChat.mockResolvedValueOnce(null)
|
|
|
|
const response = await GET(createRequest('chat-missing'), makeContext('chat-missing'))
|
|
expect(response.status).toBe(404)
|
|
expect(mockReconcileChatStreamMarkers).not.toHaveBeenCalled()
|
|
})
|
|
|
|
it('returns 401 when unauthenticated', async () => {
|
|
copilotHttpMockFns.mockAuthenticateCopilotRequestSessionOnly.mockResolvedValueOnce({
|
|
userId: null,
|
|
isAuthenticated: false,
|
|
})
|
|
|
|
const response = await GET(createRequest('chat-x'), makeContext('chat-x'))
|
|
expect(response.status).toBe(401)
|
|
expect(mockGetAccessibleCopilotChat).not.toHaveBeenCalled()
|
|
expect(mockReconcileChatStreamMarkers).not.toHaveBeenCalled()
|
|
})
|
|
})
|