feat: added @hono/mcp package#31
Conversation
|
@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? |
|
This may not be the Ramen API thing, but a 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 |
|
Hi @MathurAditya724 ! Thank you for the explanation. I got it. Looks good. Thank you for the contribution to Ramen API 🍜 |
Resolves #30