Skip to content

Conversation

@VincentLanglet
Copy link
Contributor

@VincentLanglet VincentLanglet commented Nov 22, 2025

Q A
Bug fix? yes
New feature? no
Docs? no
Issues Fix #937
License MIT

@carsonbot carsonbot added Bug Something isn't working Status: Needs Review labels Nov 22, 2025
@carsonbot carsonbot changed the title Use custom serializer for PlatformSubscriber Use custom serializer for PlatformSubscriber Nov 22, 2025
@chr-hertel
Copy link
Member

That's what I had in mind as well - even better! 👍

The only part missing is maybe the PropertyNormalizer - have a look at the recipe bot in the demo, see #908

@VincentLanglet
Copy link
Contributor Author

VincentLanglet commented Nov 22, 2025

The only part missing is maybe the PropertyNormalizer - have a look at the recipe bot in the demo, see #908

I'm not sure what I need to solve here with the PropertyNormalizer. Seems like the ObjectNormalizer is already covering it...

I tried to add a test with

final class MathReasoningWithProperties
{
    /**
     * @var Step[] $steps
     */
    public array $steps;

    public string $finalAnswer;

    public float $result;
}

and it works fine without PropertyNormalizer.

@chr-hertel
Copy link
Member

oh, true, with this branch it works as well for that demo without explicitly registering that PropertyNormalizer as service ... or maybe i was the one hallucinating the other night 🤷‍♂️

Thanks @VincentLanglet - looks good!

@chr-hertel
Copy link
Member

Thank you @VincentLanglet.

@chr-hertel chr-hertel merged commit cc76e10 into symfony:main Nov 22, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working Status: Reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Platform] StructuredOutput use Serializer to hydrate values but rely on ReflectionProperty for the schema

3 participants