Fix MCP installation docs: correct packages for Symfony and Laravel#2266
Fix MCP installation docs: correct packages for Symfony and Laravel#2266
Conversation
…plicit symfony/mcp-bundle Co-authored-by: soyuka <1321971+soyuka@users.noreply.github.com>
core/mcp.md
Outdated
| ### Laravel | ||
|
|
||
| MCP support is optional in Laravel. Install the required packages: | ||
| MCP support is optional in Laravel. Install the required package: |
There was a problem hiding this comment.
Its always optional (also for symfony) either move this at the start of document or remove
There was a problem hiding this comment.
Removed the "MCP support is optional in Laravel." text (commit 4bd3ae9).
|
|
||
| ### Symfony | ||
|
|
||
| Install the [MCP Bundle](https://github.com/symfony-tools/mcp-bundle): |
There was a problem hiding this comment.
Why remove the link? Can you check on api-platform/core if mcp bundle if also requires by api-platform/laravel (look for ApiPlatformProvider.php) and routes/api.php
There was a problem hiding this comment.
Restored the link. After checking src/Laravel/composer.json in api-platform/core: api-platform/mcp is in require (already bundled with api-platform/laravel) and symfony/mcp-bundle is in require-dev only. The route in routes/api.php is gated on class_exists(McpController::class) from symfony/mcp-bundle, so Laravel users need to install symfony/mcp-bundle explicitly. Updated the Laravel command to composer require symfony/mcp-bundle (commit 4bd3ae9).
There was a problem hiding this comment.
It's probably a mistake it should be in require dev for laravel also I'll fix later but for now assume it is inside dev reqs
…, remove 'optional' text Co-authored-by: soyuka <1321971+soyuka@users.noreply.github.com>
soyuka
left a comment
There was a problem hiding this comment.
Assume mcp is only req dependency I'll make sure it is also, both symfony and laravel need to install both mcp bundle and our package
…ymfony/mcp-bundle Co-authored-by: soyuka <1321971+soyuka@users.noreply.github.com>
Done — both Symfony and Laravel sections now show |
The MCP installation instructions had two errors: Symfony was missing
api-platform/mcp, and the link to the MCP Bundle had been removed. Both Symfony and Laravel require the same two packages.Changes (
core/mcp.md)api-platform/mcpto the install command alongsidesymfony/mcp-bundle; restore the link to the MCP Bundleapi-platform/mcpandsymfony/mcp-bundle; remove the misleading "MCP support is optional in Laravel" note (MCP is optional for both frameworks)Original prompt
This pull request was created from Copilot chat.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.