Ugrade Psalm version

Author: viktorprogger

composer.json

@@ -26,7 +26,7 @@
         "roave/infection-static-analysis-plugin": "^1.16",
         "symfony/console": "^6.2",
         "symfony/http-client": "^7.0",
-        "vimeo/psalm": "^5.4",
+        "vimeo/psalm": "^6.8",
         "yiisoft/definitions": "^3.3",
         "yiisoft/test-support": "^3.0"
     },

psalm.xml

@@ -14,4 +14,7 @@
             <directory name="vendor"/>
         </ignoreFiles>
     </projectFiles>
+    <issueHandlers>
+        <MissingOverrideAttribute errorLevel="suppress" />
+    </issueHandlers>
 </psalm>

src/Console/GetUpdatesCommand.php

@@ -7,6 +7,7 @@
 use Botasis\Client\Telegram\Client\ClientInterface;
 use Botasis\Client\Telegram\Request\TelegramRequest;
 use Botasis\Runtime\Application;
+use Botasis\Runtime\Update\Update;
 use Botasis\Runtime\Update\UpdateFactory;
 use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
@@ -48,6 +49,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $data = ['allowed_updates' => $input->getOption('allowed-updates')];
         $request = new TelegramRequest('getUpdates', $data);
 
+        /** @var array $update */
         foreach ($this->client->send($request)['result'] ?? [] as $update) {
             try {
                 $update = $this->updateFactory->create($update);
@@ -58,6 +60,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             }
         }
 
+        /** @var Update|null $update */
         if ($update !== null) {
             $data['offset'] = $update->id->value + 1;
             $this->client->send(new TelegramRequest('getUpdates', $data));

src/Console/SetTelegramWebhookCommand.php

@@ -73,9 +73,11 @@ protected function configure(): void
 
     public function execute(InputInterface $input, OutputInterface $output): int
     {
+        /** @var string $url */
         $url = $input->getOption('url');
 
         if ($url === '') {
+            /** @var bool $answer */
             $answer = $this->questionHelper->ask(
                 $input,
                 $output,
@@ -108,16 +110,19 @@ public function execute(InputInterface $input, OutputInterface $output): int
             'drop_pending_updates' => $input->getOption('drop_pending_updates'),
         ];
 
+        /** @var string|null $ip */
         $ip = $input->getOption('ip_address');
         if ($ip !== null && $ip !== '') {
             $fields['ip_address'] = $ip;
         }
 
+        /** @var int|null $connections */
         $connections = $input->getOption('max_connections');
         if ($connections !== null && $connections > 0) {
             $fields['max_connections'] = $connections;
         }
 
+        /** @var string|null $token */
         $token = $input->getOption('secret_token');
         if ($token !== null && $token !== '') {
             $fields['secret_token'] = $token;

src/Entity/User/UserFactory.php

@@ -4,14 +4,23 @@
 
 namespace Botasis\Runtime\Entity\User;
 
+/**
+ * Factory for creating User objects from Telegram API data.
+ */
 final readonly class UserFactory
 {
     public function __construct()
     {
     }
 
+    /**
+     * Converts array, received from Telegram API, into User object
+     */
     public function create(array $userData): User
     {
+        /**
+         * @psalm-suppress MixedArgument
+         */
         return new User(
             (string)$userData['id'],
             (bool) $userData['is_bot'],

src/Event/IgnoredErrorHandler.php

@@ -25,7 +25,10 @@ public function __construct(string ...$ignoredErrors)
 
     public function handle(RequestErrorEvent $event): RequestErrorEvent
     {
-        if (isset($this->ignoredErrors[$event->exception->responseDecoded['description'] ?? ''])) {
+        /** @var string $description */
+        $description = $event->exception->responseDecoded['description'] ?? '';
+
+        if (isset($this->ignoredErrors[$description])) {
             $event->suppressException = true;
         }
 

src/Event/RequestTagsHandler.php

@@ -20,28 +20,29 @@ final class RequestTagsHandler
     private array $tagsError;
 
     /**
-     * @param array<string, list<mixed>> $tagsSuccess
-     * @param array<string, list<mixed>> $tagsError
+     * @param array<string, list<string|array>> $tagsSuccess
+     * @param array<string, list<string|array>> $tagsError
      */
     public function __construct(
         CallableFactory $factory,
         private Injector $injector,
         array $tagsSuccess = [],
         array $tagsError = [],
     ) {
+        $listenersSuccess = $listenersError = [];
         foreach ($tagsSuccess as $tag => $definitions) {
             foreach ($definitions as $index => $definition) {
-                $tagsSuccess[$tag][$index] = $factory->create($definition);
+                $listenersSuccess[$tag][$index] = $factory->create($definition);
             }
         }
         foreach ($tagsError as $tag => $definitions) {
             foreach ($definitions as $index => $definition) {
-                $tagsError[$tag][$index] = $factory->create($definition);
+                $listenersError[$tag][$index] = $factory->create($definition);
             }
         }
 
-        $this->tagsSuccess = $tagsSuccess;
-        $this->tagsError = $tagsError;
+        $this->tagsSuccess = $listenersSuccess;
+        $this->tagsError = $listenersError;
     }
 
     public function handleSuccess(RequestSuccessEvent $event): void

src/InvalidCallableConfigurationException.php

@@ -7,11 +7,11 @@
 use InvalidArgumentException;
 use Yiisoft\FriendlyException\FriendlyExceptionInterface;
 
-class InvalidCallableConfigurationException extends InvalidArgumentException implements FriendlyExceptionInterface
+final class InvalidCallableConfigurationException extends InvalidArgumentException implements FriendlyExceptionInterface
 {
     public function getName(): string
     {
-        return 'Invalid event listener configuration.';
+        return 'Invalid callable configuration.';
     }
 
     public function getSolution(): ?string