Daniil Gentili

Could you please add some tests?

Wait a second, did you deploy an MTProto server on your own server? If so, what software did you use?

Hi, thanks for your PR, pls add some unit tests in `tests/danog/MadelineProto/EntitiesTest.php`

The functionality was moved to https://github.com/danog/telegram-entities, could you please repeat the PR there? https://github.com/danog/telegram-entities Thanks!

I agree with all listed changes except for the ignoreInternalFunction* changes, which would actually decrease strictness. We already use all these options @ work (and I actually contributed a few...

Yeah that's quite on common on codebases of libraries, usable by users without a static analyzer to catch misuses of APIs. A good idea would be to enable the behavior...

Just using a direct check fixes: https://psalm.dev/r/fb503c39ce Now, a new issue is emitted due to the recently merged https://github.com/vimeo/psalm/pull/10502, and both in case of the `!= ''` comparison and in...

>loose comparisons are a pain and huge source of bugs +++, it along with empty and isset are banned in my codebase (and I'm very thankful for that) I consider...

@kkmuffme 1. Sure 2. Sure, that can be done. 3. This is an error like in 2, `@satisfies` is not `@var`.

Hmm the size of the go.sum is indicative of the number of dependencies more than project size itself, which is calculated in another way, but will try to add something...