mirror of
https://github.com/hanzoai/insights.git
synced 2026-11-03 00:52:32 +00:00
A key now opens with what it is allowed to touch. `pk-` is publishable and safe in a browser; `sk-` is secret and never leaves a server. The five marks it replaces all began `ph` -- the upstream's initials -- and they were the last of its brand a customer read back to us, in every SDK snippet. No word rule could reach them, because `ph` is not a word. The marks were also not telling the truth. The constant said `phc_` while every live project token said `hi_`, so the middleware that routes on the constant matched no real token at all. Two spellings existed and one was fiction. Minting now happens in exactly one function and reading in exactly one, so a second spelling has nowhere to live. `phx_` (personal) and `phs_` (project secret) both become `sk-`. The mark now says only that a value is secret; which store holds it is settled by asking. That was already how personal keys worked -- keys minted before the marks carry none, so the store always decided -- and both secret resolvers already returned None on a miss and fell through, so one shared extractor replaces the two that could drift apart. OAuth's access and refresh tokens stay distinguishable as `at-` and `rt-`, because the parser tells them apart before either is looked up, and the domain-verification token becomes `vt-` rather than keeping a sixth mint of its own. `bin/debrand --check` now refuses the marks as a shape, so a seventh cannot be coined. Two rules that were about to rename things that are not ours to spell are bounded: `Lemon` no longer eats `LemonSqueezy`, and the `hog_flow` rules no longer reach a Postgres table, a Datastore table or a route. Keys are stored hashed, so a mark cannot be rewritten in place: the three live project tokens have to be reminted.
396 lines
12 KiB
TypeScript
396 lines
12 KiB
TypeScript
import { afterAll, beforeAll, describe, expect, it, vi } from 'vitest'
|
|
|
|
vi.mock('@/lib/insights', () => ({
|
|
getInsightsClient: () => ({
|
|
capture: vi.fn(),
|
|
shutdown: () => Promise.resolve(),
|
|
}),
|
|
}))
|
|
|
|
vi.mock('@/lib/insights/flags', () => ({
|
|
isFeatureFlagEnabled: vi.fn().mockResolvedValue(false),
|
|
evaluateFeatureFlags: vi.fn().mockResolvedValue({}),
|
|
}))
|
|
|
|
vi.mock('@/resources/internals', () => ({
|
|
fetchContextMillResources: vi.fn().mockRejectedValue(new Error('mocked')),
|
|
filterValidEntries: vi.fn().mockReturnValue([]),
|
|
loadManifestFromArchive: vi.fn().mockReturnValue({ resources: [] }),
|
|
clearResourceCache: vi.fn(),
|
|
}))
|
|
|
|
vi.mock('@/resources', () => ({
|
|
getPromptsFromManifest: vi.fn().mockResolvedValue([]),
|
|
}))
|
|
|
|
import { McpDispatcher } from '@/hono/dispatcher'
|
|
import { ToolCatalog } from '@/hono/tool-catalog'
|
|
import type { RequestProperties } from '@/lib/request-properties'
|
|
|
|
class MockRedis {
|
|
private store = new Map<string, string>()
|
|
async get(key: string): Promise<string | null> {
|
|
return this.store.get(key) ?? null
|
|
}
|
|
async set(key: string, value: string): Promise<'OK'> {
|
|
this.store.set(key, value)
|
|
return 'OK'
|
|
}
|
|
async del(key: string): Promise<number> {
|
|
return this.store.delete(key) ? 1 : 0
|
|
}
|
|
async expire(): Promise<number> {
|
|
return 1
|
|
}
|
|
async ttl(): Promise<number> {
|
|
return 3600
|
|
}
|
|
}
|
|
|
|
const mockUserResponse = {
|
|
uuid: 'test-uuid',
|
|
distinct_id: 'test-distinct-id',
|
|
first_name: 'Test',
|
|
last_name: 'User',
|
|
email: '[email protected]',
|
|
}
|
|
|
|
const mockOrgsResponse = {
|
|
results: [{ id: 'org-1', name: 'Test Org', slug: 'test' }],
|
|
}
|
|
|
|
const mockProjectsResponse = {
|
|
results: [{ id: 1, uuid: 'proj-1', name: 'Test Project', organization: 'org-1' }],
|
|
}
|
|
|
|
let fetchCallCount = 0
|
|
|
|
function createMockFetch(): typeof fetch {
|
|
return async (input: string | URL | Request): Promise<Response> => {
|
|
fetchCallCount++
|
|
const url = typeof input === 'string' ? input : input instanceof URL ? input.toString() : input.url
|
|
const path = new URL(url).pathname
|
|
|
|
let body: unknown = {}
|
|
if (path.includes('/v1/users/@me')) {
|
|
body = mockUserResponse
|
|
} else if (path.includes('/v1/organizations')) {
|
|
body = mockOrgsResponse
|
|
} else if (path.includes('/v1/projects') || path.includes('/v1/environments')) {
|
|
body = mockProjectsResponse
|
|
} else if (path.includes('/decide')) {
|
|
body = { featureFlags: {} }
|
|
}
|
|
|
|
return new Response(JSON.stringify(body), {
|
|
status: 200,
|
|
headers: { 'Content-Type': 'application/json' },
|
|
})
|
|
}
|
|
}
|
|
|
|
const WARM_USER_HASH = 'warm-user-for-profiling'
|
|
|
|
function makeProps(overrides: Partial<RequestProperties> = {}): RequestProperties {
|
|
return {
|
|
userHash: WARM_USER_HASH,
|
|
apiToken: 'sk-test_token',
|
|
sessionId: 'sess-warm',
|
|
mcpClientName: 'test-profiler',
|
|
mcpClientVersion: '1.0.0',
|
|
mcpProtocolVersion: '2025-03-26',
|
|
transport: 'streamable-http',
|
|
requestStartTime: Date.now(),
|
|
...overrides,
|
|
}
|
|
}
|
|
|
|
function makeColdProps(): RequestProperties {
|
|
return makeProps({
|
|
userHash: `user-${Math.random().toString(36).slice(2, 10)}`,
|
|
sessionId: `sess-${Math.random().toString(36).slice(2, 10)}`,
|
|
})
|
|
}
|
|
|
|
function makeJsonRpcBody(method: string, params?: Record<string, unknown>): string {
|
|
return JSON.stringify({
|
|
jsonrpc: '2.0',
|
|
id: Math.floor(Math.random() * 1_000_000),
|
|
method,
|
|
...(params ? { params } : {}),
|
|
})
|
|
}
|
|
|
|
function makeRequest(body: string): Request {
|
|
return new Request('http://localhost/mcp', {
|
|
method: 'POST',
|
|
headers: { 'Content-Type': 'application/json' },
|
|
body,
|
|
})
|
|
}
|
|
|
|
function _fmt(bytes: number): string {
|
|
if (Math.abs(bytes) < 1024) {
|
|
return `${bytes} B`
|
|
}
|
|
const kb = bytes / 1024
|
|
if (Math.abs(kb) < 1024) {
|
|
return `${kb.toFixed(1)} KB`
|
|
}
|
|
return `${(kb / 1024).toFixed(2)} MB`
|
|
}
|
|
|
|
function gc(): void {
|
|
if (global.gc) {
|
|
global.gc()
|
|
global.gc()
|
|
}
|
|
}
|
|
|
|
function percentile(sorted: number[], p: number): number {
|
|
const idx = Math.ceil((p / 100) * sorted.length) - 1
|
|
return sorted[Math.max(0, idx)]!
|
|
}
|
|
|
|
function stats(timings: number[]): { min: number; p50: number; p95: number; p99: number; max: number; avg: number } {
|
|
const sorted = [...timings].sort((a, b) => a - b)
|
|
return {
|
|
min: sorted[0]!,
|
|
p50: percentile(sorted, 50),
|
|
p95: percentile(sorted, 95),
|
|
p99: percentile(sorted, 99),
|
|
max: sorted[sorted.length - 1]!,
|
|
avg: timings.reduce((a, b) => a + b, 0) / timings.length,
|
|
}
|
|
}
|
|
|
|
const INIT_PARAMS = {
|
|
protocolVersion: '2025-03-26',
|
|
capabilities: {},
|
|
clientInfo: { name: 'test', version: '1.0' },
|
|
}
|
|
|
|
describe('McpDispatcher profiling', () => {
|
|
let dispatcher: McpDispatcher
|
|
let catalog: ToolCatalog
|
|
let originalFetch: typeof fetch
|
|
|
|
beforeAll(async () => {
|
|
originalFetch = globalThis.fetch
|
|
globalThis.fetch = createMockFetch()
|
|
|
|
process.env.INSIGHTS_API_BASE_URL = 'http://localhost:19876'
|
|
process.env.INSIGHTS_ANALYTICS_API_KEY = 'pk-test'
|
|
process.env.INSIGHTS_ANALYTICS_HOST = 'http://localhost:19876'
|
|
|
|
const redis = new MockRedis()
|
|
catalog = new ToolCatalog()
|
|
dispatcher = new McpDispatcher(catalog, redis as any)
|
|
await dispatcher.warmup()
|
|
|
|
// Prime the warm cache
|
|
await dispatcher.handleRequest(makeRequest(makeJsonRpcBody('tools/list')), makeProps())
|
|
})
|
|
|
|
afterAll(() => {
|
|
globalThis.fetch = originalFetch
|
|
})
|
|
|
|
it('reports warmup stats', () => {
|
|
const entries = catalog.getPreBuiltEntries()
|
|
|
|
expect(entries.length).toBeGreaterThan(0)
|
|
})
|
|
|
|
it('measures cold cache init latency', async () => {
|
|
const coldTimings: number[] = []
|
|
const coldFetchCounts: number[] = []
|
|
|
|
for (let i = 0; i < 20; i++) {
|
|
const before = fetchCallCount
|
|
const start = performance.now()
|
|
const resp = await dispatcher.handleRequest(
|
|
makeRequest(makeJsonRpcBody('initialize', INIT_PARAMS)),
|
|
makeColdProps()
|
|
)
|
|
const ms = performance.now() - start
|
|
const fetches = fetchCallCount - before
|
|
|
|
coldTimings.push(ms)
|
|
coldFetchCounts.push(fetches)
|
|
expect(resp.status).toBe(200)
|
|
}
|
|
|
|
const _s = stats(coldTimings)
|
|
const _avgFetches = coldFetchCounts.reduce((a, b) => a + b, 0) / coldFetchCounts.length
|
|
}, 30_000)
|
|
|
|
it('measures warm cache request latency', async () => {
|
|
for (const [_label, method, params] of [
|
|
['initialize', 'initialize', INIT_PARAMS],
|
|
['tools/list', 'tools/list', undefined],
|
|
['tools/call', 'tools/call', { name: 'user-get', arguments: {} }],
|
|
['resources/list', 'resources/list', undefined],
|
|
['ping', 'ping', undefined],
|
|
] as const) {
|
|
// JIT warmup
|
|
for (let i = 0; i < 10; i++) {
|
|
await dispatcher.handleRequest(
|
|
makeRequest(makeJsonRpcBody(method, params as Record<string, unknown> | undefined)),
|
|
makeProps()
|
|
)
|
|
}
|
|
|
|
const timings: number[] = []
|
|
for (let i = 0; i < 100; i++) {
|
|
const start = performance.now()
|
|
await dispatcher.handleRequest(
|
|
makeRequest(makeJsonRpcBody(method, params as Record<string, unknown> | undefined)),
|
|
makeProps()
|
|
)
|
|
timings.push(performance.now() - start)
|
|
}
|
|
|
|
const _s = stats(timings)
|
|
}
|
|
}, 120_000)
|
|
|
|
it('measures cold vs warm fetch overhead', async () => {
|
|
// Cold: new userHash
|
|
const coldBefore = fetchCallCount
|
|
await dispatcher.handleRequest(
|
|
makeRequest(makeJsonRpcBody('tools/call', { name: 'user-get', arguments: {} })),
|
|
makeColdProps()
|
|
)
|
|
const _coldFetches = fetchCallCount - coldBefore
|
|
|
|
// Warm: reused userHash (cache hit)
|
|
const warmBefore = fetchCallCount
|
|
await dispatcher.handleRequest(
|
|
makeRequest(makeJsonRpcBody('tools/call', { name: 'user-get', arguments: {} })),
|
|
makeProps()
|
|
)
|
|
const _warmFetches = fetchCallCount - warmBefore
|
|
}, 15_000)
|
|
|
|
it('measures per-request memory (cold cache)', async () => {
|
|
for (const N of [1, 10, 50]) {
|
|
gc()
|
|
await new Promise((r) => setTimeout(r, 30))
|
|
gc()
|
|
|
|
const before = process.memoryUsage().heapUsed
|
|
const start = performance.now()
|
|
|
|
for (let i = 0; i < N; i++) {
|
|
await dispatcher.handleRequest(
|
|
makeRequest(makeJsonRpcBody('tools/call', { name: 'user-get', arguments: {} })),
|
|
makeColdProps()
|
|
)
|
|
}
|
|
|
|
const _elapsed = performance.now() - start
|
|
gc()
|
|
const after = process.memoryUsage().heapUsed
|
|
const _delta = after - before
|
|
}
|
|
}, 120_000)
|
|
|
|
it('measures concurrent burst (warm cache)', async () => {
|
|
for (const burst of [10, 50, 100, 200]) {
|
|
gc()
|
|
await new Promise((r) => setTimeout(r, 50))
|
|
gc()
|
|
const before = process.memoryUsage().heapUsed
|
|
const burstStart = performance.now()
|
|
|
|
const results = await Promise.all(
|
|
Array.from({ length: burst }, () =>
|
|
dispatcher.handleRequest(
|
|
makeRequest(makeJsonRpcBody('tools/call', { name: 'user-get', arguments: {} })),
|
|
makeProps()
|
|
)
|
|
)
|
|
)
|
|
|
|
const _burstMs = performance.now() - burstStart
|
|
gc()
|
|
const after = process.memoryUsage().heapUsed
|
|
const _delta = after - before
|
|
|
|
for (const r of results) {
|
|
expect(r.status).toBe(200)
|
|
}
|
|
}
|
|
}, 60_000)
|
|
|
|
it('measures concurrent burst (cold cache)', async () => {
|
|
for (const burst of [10, 50, 100]) {
|
|
gc()
|
|
await new Promise((r) => setTimeout(r, 50))
|
|
gc()
|
|
const before = process.memoryUsage().heapUsed
|
|
const burstStart = performance.now()
|
|
|
|
const results = await Promise.all(
|
|
Array.from({ length: burst }, () =>
|
|
dispatcher.handleRequest(
|
|
makeRequest(makeJsonRpcBody('tools/call', { name: 'user-get', arguments: {} })),
|
|
makeColdProps()
|
|
)
|
|
)
|
|
)
|
|
|
|
const _burstMs = performance.now() - burstStart
|
|
gc()
|
|
const after = process.memoryUsage().heapUsed
|
|
const _delta = after - before
|
|
|
|
for (const r of results) {
|
|
expect(r.status).toBe(200)
|
|
}
|
|
}
|
|
}, 60_000)
|
|
|
|
it('measures batch JSON-RPC', async () => {
|
|
for (const batchSize of [5, 20, 50]) {
|
|
gc()
|
|
await new Promise((r) => setTimeout(r, 50))
|
|
gc()
|
|
|
|
const before = process.memoryUsage().heapUsed
|
|
const batchStart = performance.now()
|
|
|
|
const batch = Array.from({ length: batchSize }, (_, i) => ({
|
|
jsonrpc: '2.0' as const,
|
|
id: i + 1,
|
|
method: i === 0 ? 'initialize' : i === 1 ? 'tools/list' : 'tools/call',
|
|
params: i === 0 ? INIT_PARAMS : i === 1 ? {} : { name: 'user-get', arguments: {} },
|
|
}))
|
|
|
|
const resp = await dispatcher.handleRequest(
|
|
new Request('http://localhost/mcp', {
|
|
method: 'POST',
|
|
headers: { 'Content-Type': 'application/json' },
|
|
body: JSON.stringify(batch),
|
|
}),
|
|
makeProps()
|
|
)
|
|
|
|
const results = (await resp.json()) as unknown[]
|
|
const _batchMs = performance.now() - batchStart
|
|
|
|
gc()
|
|
const after = process.memoryUsage().heapUsed
|
|
const _delta = after - before
|
|
|
|
expect(results.length).toBe(batchSize)
|
|
}
|
|
}, 60_000)
|
|
|
|
it('reports final memory', () => {
|
|
gc()
|
|
const _m = process.memoryUsage()
|
|
})
|
|
})
|