acf4e225e6
Complete rewrite of the Hermes SQLite database schema synchronization mechanism with comprehensive error handling, automatic recovery, and full test coverage. ## Database Schema Synchronization - **Unified sync mechanism**: Single `syncTable()` function handles all schema changes - **Automatic column sync**: Adds missing columns and removes extra columns - **Table rebuilding**: Automatically rebuilds tables when primary keys or types change - **Data preservation**: Preserves data during schema changes when compatible - **Index management**: Creates and removes indexes as needed ## Error Recovery & Reliability - **Automatic backup**: Backs up corrupted database before recovery - **Retry limiting**: Prevents infinite loops with retry limit - **Duplicate prevention**: Avoids multiple backup files - **Safe file operations**: Uses copy+delete instead of rename for safety ## Composite Primary Keys - Fixed GC_ROOM_AGENTS and GC_ROOM_MEMBERS with proper composite primary keys - Prevents duplicate entries while allowing same roomId with different agentId/userId ## Test Coverage - **10 new integration tests** for schema synchronization (tests/server/schema-sync.test.ts) - **3 updated tests** for Hermes schemas (tests/server/hermes-schemas.test.ts) - All 327 tests passing (47 test files, 325 passed, 2 skipped) ## Bug Fixes - Fixed module import issues (unified ES6 imports, removed mixed require()) - Fixed mock issues in sessions routes tests - Fixed i18n coverage test to handle newly added keys - Fixed profiles store test to match current implementation Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
112 lines
5.3 KiB
TypeScript
112 lines
5.3 KiB
TypeScript
import { beforeEach, describe, expect, it, vi } from 'vitest'
|
|
|
|
const listConversationsMock = vi.fn(async (ctx: any) => { ctx.body = { sessions: [{ id: 'conversation-1' }] } })
|
|
const getConversationMessagesMock = vi.fn(async (ctx: any) => { ctx.body = { session_id: ctx.params.id, messages: [] } })
|
|
const getConversationMessagesPaginatedMock = vi.fn(async (ctx: any) => { ctx.body = { session_id: ctx.params.id, messages: [], pagination: {} } })
|
|
const listMock = vi.fn(async (ctx: any) => { ctx.body = { sessions: [{ id: 's1' }] } })
|
|
const listHermesSessionsMock = vi.fn(async (ctx: any) => { ctx.body = { sessions: [{ id: 'hermes-1' }] } })
|
|
const getHermesSessionMock = vi.fn(async (ctx: any) => { ctx.body = { session: { id: ctx.params.id } } })
|
|
const searchMock = vi.fn(async (ctx: any) => { ctx.body = { results: [{ id: 'search-1' }] } })
|
|
const getMock = vi.fn(async (ctx: any) => { ctx.body = { session: { id: ctx.params.id } } })
|
|
const removeMock = vi.fn(async (ctx: any) => { ctx.body = { ok: true } })
|
|
const renameMock = vi.fn(async (ctx: any) => { ctx.body = { ok: true } })
|
|
const setWorkspaceMock = vi.fn(async (ctx: any) => { ctx.body = { ok: true } })
|
|
const listWorkspaceFoldersMock = vi.fn(async (ctx: any) => { ctx.body = { folders: [] } })
|
|
const usageBatchMock = vi.fn(async (ctx: any) => { ctx.body = {} })
|
|
const usageSingleMock = vi.fn(async (ctx: any) => { ctx.body = { input_tokens: 0, output_tokens: 0 } })
|
|
const usageStatsMock = vi.fn(async (ctx: any) => { ctx.body = { total_input_tokens: 0, total_output_tokens: 0 } })
|
|
const contextLengthMock = vi.fn(async (ctx: any) => { ctx.body = { context_length: 200000 } })
|
|
|
|
vi.mock('../../packages/server/src/controllers/hermes/sessions', () => ({
|
|
listConversations: listConversationsMock,
|
|
getConversationMessages: getConversationMessagesMock,
|
|
getConversationMessagesPaginated: getConversationMessagesPaginatedMock,
|
|
list: listMock,
|
|
listHermesSessions: listHermesSessionsMock,
|
|
getHermesSession: getHermesSessionMock,
|
|
search: searchMock,
|
|
get: getMock,
|
|
remove: removeMock,
|
|
rename: renameMock,
|
|
setWorkspace: setWorkspaceMock,
|
|
listWorkspaceFolders: listWorkspaceFoldersMock,
|
|
usageBatch: usageBatchMock,
|
|
usageSingle: usageSingleMock,
|
|
usageStats: usageStatsMock,
|
|
contextLength: contextLengthMock,
|
|
}))
|
|
|
|
describe('session routes', () => {
|
|
beforeEach(() => {
|
|
vi.resetModules()
|
|
listConversationsMock.mockClear()
|
|
getConversationMessagesMock.mockClear()
|
|
getConversationMessagesPaginatedMock.mockClear()
|
|
listMock.mockClear()
|
|
searchMock.mockClear()
|
|
getMock.mockClear()
|
|
removeMock.mockClear()
|
|
renameMock.mockClear()
|
|
})
|
|
|
|
it('registers conversations, session list, and search routes', async () => {
|
|
const { sessionRoutes } = await import('../../packages/server/src/routes/hermes/sessions')
|
|
const paths = sessionRoutes.stack.map((entry: any) => entry.path)
|
|
|
|
expect(paths).toEqual(expect.arrayContaining([
|
|
'/api/hermes/sessions/conversations',
|
|
'/api/hermes/sessions/conversations/:id/messages',
|
|
'/api/hermes/sessions/conversations/:id/messages/paginated',
|
|
'/api/hermes/sessions',
|
|
'/api/hermes/search/sessions',
|
|
'/api/hermes/sessions/search',
|
|
'/api/hermes/sessions/usage',
|
|
'/api/hermes/usage/stats',
|
|
'/api/hermes/sessions/context-length',
|
|
'/api/hermes/sessions/:id',
|
|
'/api/hermes/sessions/:id/usage',
|
|
'/api/hermes/sessions/:id/rename',
|
|
]))
|
|
})
|
|
|
|
it('delegates session search to the controller', async () => {
|
|
const { sessionRoutes } = await import('../../packages/server/src/routes/hermes/sessions')
|
|
const layer = sessionRoutes.stack.find((entry: any) => entry.path === '/api/hermes/search/sessions')
|
|
const handler = layer.stack[0]
|
|
const ctx: any = { query: { q: 'docker', limit: '8' }, body: null, params: {} }
|
|
|
|
await handler(ctx)
|
|
|
|
expect(searchMock).toHaveBeenCalledWith(ctx)
|
|
expect(ctx.body).toEqual({ results: [{ id: 'search-1' }] })
|
|
})
|
|
|
|
it('keeps the legacy search path wired to the same controller', async () => {
|
|
const { sessionRoutes } = await import('../../packages/server/src/routes/hermes/sessions')
|
|
const layer = sessionRoutes.stack.find((entry: any) => entry.path === '/api/hermes/sessions/search')
|
|
const handler = layer.stack[0]
|
|
const ctx: any = { query: { q: 'docker' }, body: null, params: {} }
|
|
|
|
await handler(ctx)
|
|
|
|
expect(searchMock).toHaveBeenCalledWith(ctx)
|
|
expect(ctx.body).toEqual({ results: [{ id: 'search-1' }] })
|
|
})
|
|
|
|
it('delegates conversations list and detail routes', async () => {
|
|
const { sessionRoutes } = await import('../../packages/server/src/routes/hermes/sessions')
|
|
const listLayer = sessionRoutes.stack.find((entry: any) => entry.path === '/api/hermes/sessions/conversations')
|
|
const detailLayer = sessionRoutes.stack.find((entry: any) => entry.path === '/api/hermes/sessions/conversations/:id/messages')
|
|
|
|
const listCtx: any = { query: {}, body: null, params: {} }
|
|
await listLayer.stack[0](listCtx)
|
|
expect(listConversationsMock).toHaveBeenCalledWith(listCtx)
|
|
expect(listCtx.body).toEqual({ sessions: [{ id: 'conversation-1' }] })
|
|
|
|
const detailCtx: any = { params: { id: 'child-session' }, query: {}, body: null }
|
|
await detailLayer.stack[0](detailCtx)
|
|
expect(getConversationMessagesMock).toHaveBeenCalledWith(detailCtx)
|
|
expect(detailCtx.body).toEqual({ session_id: 'child-session', messages: [] })
|
|
})
|
|
})
|