@yusukebe I have added a simple test to check if the error is getting thrown or not, and the original tests are also passing. Is it okay, or should I add more test?

@MathurAditya724

This may not be the Ramen API thing, but a @hono/mcp matter. The response for a GET request without proper headers will be 500. The following test will fail:

it('Should return 200 for GET request without headers', async () => {
  const res = await app.request('/mcp')
  expect(res.status).toBe(200)
})

Is this expected behavior?

Yes, it is. This is happens in the official SDK also https://github.com/honojs/middleware/blob/main/packages/mcp/src/index.ts#L89-L102

This is in the official one - https://github.com/modelcontextprotocol/typescript-sdk/blob/main/src/server/streamableHttp.ts#L155-L167

I think we won't be able to get rid of onError handler, so I am adding it back

Hi @MathurAditya724 !

Thank you for the explanation. I got it. Looks good. Thank you for the contribution to Ramen API 🍜