What's the proper way to implement safe wakeup after deserializing immutable object?

[andrew-demb]

https://psalm.dev/r/0b9d93cacf

<?php

/**
 * @psalm-immutable
 */
class Some
{
    // it was a `?string $locale` in past
    private string $locale;
    
    public function __construct(
        string $locale,
    ) {
        $this->locale = $locale;
    }

    public function __wakeup(): void
    {
    	// it was a `?string $locale` in past
        if (false === isset($this->locale)) {
            $this->locale = '';
        }
    }
}

Psalm output (using commit [37cc4fd](https://github.com/vimeo/psalm/commit/37cc4fd)): 

ERROR: [RedundantPropertyInitializationCheck](https://psalm.dev/261) - 20:13 - Property $this->locale with type string should already be set in the constructor

ERROR: [InaccessibleProperty](https://psalm.dev/054) - 21:13 - Some::$locale is marked readonly

What's the proper way to handle it?
Suppress both issues or psalm may allow such a checks and updates for magic method like a __wakeup?

[weirdan]

I think Psalm should treat __wakeup() and __unserialize() as constructors (also unserialize() when the class implements Serializable, with appropriate deprecation warnings).

[andrew-demb]

Created an issue #10182. Thanks.