fix(mcp): return resource read results for resources - #8436
fix(mcp): return resource read results for resources#8436ousamabenyounes wants to merge 3 commits into
Conversation
Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com>
|
Pushed The CI logs were failing before tests because Validation: reproduced the Composer conflict locally (RED), then reran |
Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com>
|
Pushed an additional CI fix in 6c18dc3.\n\nWhat changed:\n- Laravel provider now passes the new Symfony MCP bundle MiddlewareFactory argument required by the current McpController constructor.\n- Removed obsolete SerializerPropertyMetadataFactory is_array() guards that PHPStan reported as always true after the typed helper return.\n\nLocal validation:\n- Laravel package build + tests: 220 passed / 673 assertions.\n- PHPStan targeted check on src/Metadata/Property/Factory/SerializerPropertyMetadataFactory.php: no errors.\n- PHP CS Fixer dry-run on both touched files: 0 fixable files. |
|
Follow-up on the remaining red checks after 6c18dc3:
I tried to rerun the failed workflow, but GitHub returned |
Fixes #8428.
What changed
ReadResourceResultforMcpResourceoperations handled byStructuredContentProcessor.CallToolResultpath for MCP tools.contentsviaTextResourceContents.mcp/sdkconstraints withsymfony/mcp-bundle dev-main, which now requiresmcp/sdk ^0.7.RED
Original behavior RED:
GREEN