feat(mcp): introduce empty mcp server#1338
feat(mcp): introduce empty mcp server#1338VincentMolinie merged 7 commits intofeat/forest-mcp/mainfrom
Conversation
|
CU-86c6ur50w |
1 new issue
This is from Qlty Cloud, the successor to Code Climate Quality. Learn more. |
|
Diff Coverage: Not applicable. There was no coverage data reported for the files in this diff. Total Coverage: This PR will not change total coverage. 🛟 Help
This is from Qlty Cloud, the successor to Code Climate Quality. Learn more. |
packages/mcp-server/test/integration/server-instance.integration.test.ts
Outdated
Show resolved
Hide resolved
| it('should handle CORS', async () => { | ||
| const response = await request(httpServer) | ||
| .post('/mcp') | ||
| .set('Origin', 'https://example.com') | ||
| .send({ jsonrpc: '2.0', method: 'initialize', id: 1 }); | ||
|
|
||
| expect(response.headers['access-control-allow-origin']).toBe('*'); | ||
| }); | ||
|
|
||
| it('should parse JSON bodies', async () => { | ||
| const requestBody = { jsonrpc: '2.0', method: 'tools/list', id: 1 }; | ||
|
|
||
| const response = await request(httpServer) | ||
| .post('/mcp') | ||
| .set('Content-Type', 'application/json') | ||
| .send(requestBody); | ||
|
|
||
| expect(response.status).not.toBe(400); | ||
| }); |
There was a problem hiding this comment.
I don't know what to think about this kind of tests that claudes generates. If it wasn't generated, we would never have included them... should we keep them ?
There was a problem hiding this comment.
The cors part is interesting. Let me get rid of the JSON bodies one
packages/mcp-server/test/integration/server-instance.integration.test.ts
Outdated
Show resolved
Hide resolved
# @forestadmin/agent-client 1.0.0 (2025-12-18) ### Bug Fixes * **release:** remove unused dependency ([#1375](#1375)) ([4fe06d7](4fe06d7)) ### Features * **agent client:** add a packages to call an agent ([#1368](#1368)) ([891427d](891427d)) * **mcp:** introduce Forest mcp server ([#1338](#1338)) ([#1340](#1340)) ([093ab0a](093ab0a))
# @forestadmin/agent [1.68.0](https://github.com/ForestAdmin/agent-nodejs/compare/@forestadmin/agent@1.67.0...@forestadmin/agent@1.68.0) (2025-12-19) ### Bug Fixes * **mcp server:** fix build of the mcp server ([#1378](#1378)) ([e746e99](e746e99)) ### Features * **mcp-server:** add enableCount option to list tool ([#1374](#1374)) ([b43de39](b43de39)) * **mcp:** introduce Forest mcp server ([#1338](#1338)) ([#1340](#1340)) ([093ab0a](093ab0a)) ### Dependencies * **@forestadmin/mcp-server:** upgraded to 1.0.1
2 participants
Definition of Done
General
Security