-
-
Notifications
You must be signed in to change notification settings - Fork 56
Description
Hi.
I was testing out things with https://github.com/symfony/mcp-sdk and made small MCP server to test it out https://github.com/ineersa/mcp-sql-server
While developing I found few issues with current implementation and missing parts, one of them is that \Symfony\AI\McpSdk\Server\RequestHandler\InitializeHandler
has hardcoded values:
public function createResponse(Request $message): Response
{
return new Response($message->id, [
'protocolVersion' => '2025-03-26',
'capabilities' => [
'prompts' => ['listChanged' => false],
'tools' => ['listChanged' => false],
'resources' => ['listChanged' => false, 'subscribe' => false],
],
'serverInfo' => ['name' => $this->name, 'version' => $this->version],
]);
}
So to define your actual capabilities for example you need to just copy paste this file to your code (since it's final), which is not nice.
Same goes with version, I've tried to integrate it with PHPStorm AI Assistant and it's using older version, and just swapping version to older one made it work.
I can make PR with changes for that but i can't really understand where i should do it: this repository or https://github.com/symfony/mcp-sdk ?
I forked mcp-sdk to https://github.com/ineersa/mcp-sdk and made changes, pointed my composer there and tested it out and it's working, so I could make PR to that repository, but it doesn't have codestyles/cs-fixer so codestyles might be not there.
Could you please guide me where I should make PRs and what repositories used for what?
Thanks!