Skip to content

[Mate][Symfony] Add AI and MCP profiler formatters#1951

Draft
wachterjohannes wants to merge 2 commits intosymfony:mainfrom
wachterjohannes:mate-symfony-profiler-formatters
Draft

[Mate][Symfony] Add AI and MCP profiler formatters#1951
wachterjohannes wants to merge 2 commits intosymfony:mainfrom
wachterjohannes:mate-symfony-profiler-formatters

Conversation

@wachterjohannes
Copy link
Copy Markdown
Contributor

Q A
Bug fix? no
New feature? yes
Docs? yes
Issues None
License MIT

Summary

This adds Symfony bridge formatter support for the repo-owned ai and mcp profiler collectors.

The bridge now returns normalized collector payloads instead of falling back to raw dumped data when these collectors are present.

Changes

  • add AiCollectorFormatter for the symfony/ai-bundle profiler collector
  • add McpCollectorFormatter for the symfony/mcp-bundle profiler collector
  • add shared raw collector payload extraction logic for array-based collectors
  • register both formatters in the Symfony Mate bridge profiler formatter registry
  • add formatter tests for both collectors
  • add the Symfony bridge changelog entry under 0.8

Testing

  • php -l on the changed Symfony bridge PHP files
  • PHPUnit could not be run in this workspace because vendor/bin/phpunit is not installed here
  • PHP CS Fixer could not be run successfully because the installed fixer/config combination errors on the void_return rule customisation

@carsonbot carsonbot added Feature New feature Mate Issues & PRs about the AI Mate component Status: Needs Review labels Apr 17, 2026
@carsonbot carsonbot changed the title [Mate] [Symfony] Add AI and MCP profiler formatters [Mate][Symfony] Add AI and MCP profiler formatters Apr 17, 2026
@wachterjohannes wachterjohannes marked this pull request as draft April 17, 2026 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature Mate Issues & PRs about the AI Mate component Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants