Increase phpstan level to 9

lib/EmitterTrait.php

@@ -172,7 +172,7 @@ public function removeAllListeners(string $eventName = null): void
     /**
      * The list of listeners.
      *
-     * @var array<string, array{0:boolean, 1:array<int, mixed>, 2:array<int, mixed>}>
+     * @var array<string, array{0:boolean, 1:array<int, mixed>, 2:array<int, callable>}>
      */
     protected array $listeners = [];
 }

lib/Loop/Loop.php

@@ -302,7 +302,7 @@ protected function runStreams(?float $timeout): void
     /**
      * A list of timers, added by setTimeout.
      *
-     * @var array<int, mixed>
+     * @var array<int, array{0:float, 1:callable}>
      */
     protected array $timers = [];
 

lib/Promise.php

@@ -188,16 +188,21 @@ public function wait()
             return $this->value;
         } else {
             // If we got here, it means that the asynchronous operation
-            // errored. Therefore we need to throw an exception.
-            throw $this->value;
+            // errored. Therefore, we need to throw an exception.
+            if ($this->value instanceof \Throwable) {
+                throw $this->value;
+            }
+            // The state should have been REJECTED, with "value" a Throwable
+            // But "value" was not a Throwable. So throw a more general exception.
+            throw new \LogicException('The Promise was not fulfilled but no exception was specified');
         }
     }
 
     /**
      * A list of subscribers. Subscribers are the callbacks that want us to let
      * them know if the callback was fulfilled or rejected.
      *
-     * @var array<int, mixed>
+     * @var array<int, array{0:Promise<TReturn>, 1:callable|null, 2:callable|null}>
      */
     protected array $subscribers = [];
 
@@ -251,7 +256,11 @@ private function invokeCallback(Promise $subPromise, callable $callBack = null):
                 if (self::FULFILLED === $this->state) {
                     $subPromise->fulfill($this->value);
                 } else {
-                    $subPromise->reject($this->value);
+                    if ($this->value instanceof \Throwable) {
+                        $subPromise->reject($this->value);
+                    } else {
+                        throw new \LogicException('The subPromise was not fulfilled but no exception was specified');
+                    }
                 }
             }
         });

lib/WildcardEmitterTrait.php

@@ -216,14 +216,14 @@ public function removeAllListeners(string $eventName = null): void
     /**
      * The list of listeners.
      *
-     * @var array<string, mixed>
+     * @var array<string, array<int, array{0:int, 1:callable}>>
      */
     protected array $listeners = [];
 
     /**
      * The list of "wildcard listeners".
      *
-     * @var array<string, mixed>
+     * @var array<string, array<int, array{0:int, 1:callable}>>
      */
     protected array $wildcardListeners = [];
 
@@ -233,7 +233,7 @@ public function removeAllListeners(string $eventName = null): void
      *
      * If the list of listeners changes though, the index clears.
      *
-     * @var array<string, mixed>
+     * @var array<string, array<int, callable>>
      */
     protected array $listenerIndex = [];
 }

phpstan.neon

@@ -1,5 +1,5 @@
 parameters:
-  level: 8
+  level: 9
   phpVersion: 70430 # PHP 7.4.30
   ignoreErrors:
   -