diff --git a/apps/gateway/src/lib/durable-mcp-client.ts b/apps/gateway/src/lib/durable-mcp-client.ts index 75b6bacb..6c5d3678 100644 --- a/apps/gateway/src/lib/durable-mcp-client.ts +++ b/apps/gateway/src/lib/durable-mcp-client.ts @@ -12,7 +12,7 @@ export type DurableMcpClientInfo = { } // TODO: not sure if there's a better way to handle re-using client connections -// across requests. +// across requests. Maybe we use one DurableObject per customer<>originUrl connection? export class DurableMcpClient extends DurableObject { protected client?: McpClient @@ -75,9 +75,9 @@ export class DurableMcpClient extends DurableObject { arguments: args }) - // TODO: The `McpToolCallResponse` type is too complex for the CF + // TODO: The `McpToolCallResponse` type is seemingly too complex for the CF // serialization type inference to handle, so bypass it by serializing to - // a string and parse on the other end. + // a string and parsing it on the other end. return JSON.stringify(toolCallResponse) } } diff --git a/apps/gateway/src/lib/resolve-origin-request.ts b/apps/gateway/src/lib/resolve-origin-request.ts index ef45dbb5..d0a3ee2b 100644 --- a/apps/gateway/src/lib/resolve-origin-request.ts +++ b/apps/gateway/src/lib/resolve-origin-request.ts @@ -228,6 +228,8 @@ export async function resolveOriginRequest( updateOriginRequest(originRequest, { consumer, deployment }) } + assert(ctx.get('sessionId'), 500, 'Internal error: sessionId should be set') + return { deployment, consumer,