Merge pull request modelcontextprotocol#409 from mhart/patch-1

ihrpr · web-flow · commit 465ba67194c8 · 2025-04-28T18:26:57.000+01:00
Ensure JSONRPCError messages are sent for streamable http
diff --git a/src/server/streamableHttp.ts b/src/server/streamableHttp.ts
@@ -587,7 +587,7 @@ export class StreamableHTTPServerTransport implements Transport {
       }
     }
 
-    if (isJSONRPCResponse(message)) {
+    if (isJSONRPCResponse(message) || isJSONRPCError(message)) {
       this._requestResponseMap.set(requestId, message);
       const relatedIds = Array.from(this._requestToStreamMapping.entries())
         .filter(([_, streamId]) => this._streamMapping.get(streamId) === response)

Original file line number	Diff line number	Diff line change
`@@ -587,7 +587,7 @@ export class StreamableHTTPServerTransport implements Transport {`
`587`	`587`	`}`
`588`	`588`	`}`
`589`	`589`
`590`		`- if (isJSONRPCResponse(message)) {`
	`590`	`+ if (isJSONRPCResponse(message) \|\| isJSONRPCError(message)) {`
`591`	`591`	`this._requestResponseMap.set(requestId, message);`
`592`	`592`	`const relatedIds = Array.from(this._requestToStreamMapping.entries())`
`593`	`593`	`.filter(([_, streamId]) => this._streamMapping.get(streamId) === response)`