Fixed psalm issues

Author: Nyholm

src/Issues/GitHub/CachedLabelsApi.php

@@ -16,7 +16,7 @@ class CachedLabelsApi
     private $labelsApi;
 
     /**
-     * @var string[][]
+     * @var array<array-key, array<array-key, bool>>
      */
     private $labelCache = [];
 

src/Issues/GitHubRequestHandler.php

@@ -7,6 +7,7 @@
 use Psr\Log\LoggerInterface;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
+use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
 use Symfony\Component\HttpKernel\Exception\PreconditionFailedHttpException;
@@ -39,7 +40,7 @@ public function __construct(EventDispatcherInterface $dispatcher, RepositoryProv
      */
     public function handle(Request $request)
     {
-        $data = json_decode($request->getContent(), true);
+        $data = json_decode((string) $request->getContent(), true);
         if (null === $data) {
             throw new BadRequestHttpException('Invalid JSON body!');
         }
@@ -67,7 +68,12 @@ public function handle(Request $request)
         }
 
         foreach ($repository->getSubscribers() as $subscriberId) {
-            $this->dispatcher->addSubscriber($this->container->get($subscriberId));
+            $subscriber = $this->container->get($subscriberId);
+            if (!$subscriber instanceof EventSubscriberInterface) {
+                throw new \LogicException(sprintf('Service "%s" is not an instance of "%s"', $subscriberId, EventSubscriberInterface::class));
+            }
+
+            $this->dispatcher->addSubscriber($subscriber);
         }
 
         $event = new GitHubEvent($data, $repository);
@@ -76,7 +82,7 @@ public function handle(Request $request)
         try {
             $this->dispatcher->dispatch($event, 'github.'.$eventName);
         } catch (\Exception $e) {
-            throw new \RuntimeException(sprintf('Failed dispatching "%s" event for "%s" repository.', $eventName, $repository->getFullName()), 0, $e);
+            throw new \RuntimeException(sprintf('Failed dispatching "%s" event for "%s" repository.', (string) $eventName, $repository->getFullName()), 0, $e);
         }
 
         $responseData = $event->getResponseData();

src/Repository/Provider/RepositoryProviderInterface.php

@@ -12,7 +12,7 @@ interface RepositoryProviderInterface
     /**
      * @param string $repositoryName e.g. symfony/symfony-docs
      *
-     * @return Repository
+     * @return Repository|null
      */
     public function getRepository($repositoryName);