Missing DateTimeZone is formatted as ?

[dejagersh]

Hi there, not sure if this is intended, but consider this minimal example:

<?php

declare(strict_types=1);

use CuyZ\Valinor\Mapper\MappingError;
use CuyZ\Valinor\Mapper\Tree\Message\Messages;
use CuyZ\Valinor\MapperBuilder;

require_once __DIR__ . '/vendor/autoload.php';

class City
{
    public function __construct(public readonly DateTimeZone $timeZone)
    {
    }
}

try {
    $city = (new MapperBuilder())->mapper()->map(
        City::class,
        [] // Missing `timeZone`
    );
} catch (MappingError $e) {
    $messages = Messages::flattenFromNode($e->node())->toArray();

    var_dump($messages[0]->toString());
}

Output:

Cannot be empty and must be filled with a value matching type `?`

This seems to be the responsible method, and the red block is where it falls into.

I'm not so familiar of course with this codebase, but can't it somehow use getType() here? When I replace that ? with $type->toString() it shows up nicely as DateTimeZone.

Cannot be empty and must be filled with a value matching type `DateTimeZone`.

I am trying to send these errors to our error reporting system, and looking to get something more meaningful than ? :)