perf(doc): Improve type coverage and doc comments

- Add type coverage for parameters and return types in phpstan configuration.
- Update __call method in HasOptions trait to declare return type.
- Modify setOption, getOption, and getValidatedOption methods to use mixed types.
- Add suppression comments for multiple class declarations in doc comment rector files.

Author: guanguans

phpstan.neon

@@ -16,6 +16,12 @@ parameters:
         lineBefore: 3
         lineAfter: 3
         editorUrl: 'phpstorm://open?file=%%file%%&line=%%line%%'
+    type_coverage:
+        declare: 100
+        return_type: 100
+        param_type: 100
+        property_type: 100
+        constant_type: 100
 #    checkOctaneCompatibility: true
 #    checkModelProperties: true
 #    checkMissingIterableValueType: true

src/Foundation/Client.php

@@ -13,17 +13,14 @@
 
 namespace Guanguans\Notify\Foundation;
 
-use Guanguans\Notify\Foundation\Authenticators\AggregateAuthenticator;
 use Guanguans\Notify\Foundation\Authenticators\NullAuthenticator;
-use Guanguans\Notify\Foundation\Authenticators\OptionsAuthenticator;
 use Guanguans\Notify\Foundation\Concerns\Dumpable;
 use Guanguans\Notify\Foundation\Concerns\HasHttpClient;
 use Guanguans\Notify\Foundation\Contracts\Authenticator;
 use Guanguans\Notify\Foundation\Contracts\Message;
 use Guanguans\Notify\Foundation\Support\Utils;
 use GuzzleHttp\Exception\GuzzleException;
 use GuzzleHttp\Promise\PromiseInterface;
-use GuzzleHttp\RequestOptions;
 use Psr\Http\Message\ResponseInterface;
 
 class Client implements Contracts\Client

src/Foundation/Concerns/HasOptions.php

@@ -1,8 +1,5 @@
 <?php
 
-/** @noinspection MissingReturnTypeInspection */
-/** @noinspection MissingParameterTypeDeclarationInspection */
-
 declare(strict_types=1);
 
 /**
@@ -44,14 +41,16 @@ trait HasOptions
     /**
      * @throws \ReflectionException
      */
-    public function __call(string $name, array $arguments)
+    public function __call(string $name, array $arguments): self
     {
-        $defined = Utils::definedFor($this);
-
         foreach ([null, 'snake', 'pascal', 'kebab'] as $case) {
-            $casedName = $case ? Str::{$case}($name) : $name;
-
-            if (\in_array($casedName, $defined, true)) {
+            if (
+                \in_array(
+                    $casedName = $case ? Str::{$case}($name) : $name,
+                    $defined ??= Utils::definedFor($this),
+                    true
+                )
+            ) {
                 if ([] === $arguments) {
                     throw new InvalidArgumentException(
                         \sprintf('The method [%s::%s] require an argument.', static::class, $name),
@@ -65,7 +64,7 @@ public function __call(string $name, array $arguments)
         throw new BadMethodCallException(\sprintf('The method [%s::%s] does not exist.', static::class, $name));
     }
 
-    public function setOption(string $option, $value): self
+    public function setOption(string $option, mixed $value): self
     {
         $this->options[$option] = $value;
 
@@ -79,7 +78,7 @@ public function setOptions(array $options): self
         return $this;
     }
 
-    public function getOption(string $option, $default = null)
+    public function getOption(string $option, mixed $default = null): mixed
     {
         return $this->options[$option] ?? $default;
     }
@@ -89,7 +88,7 @@ public function getOptions(): array
         return $this->options;
     }
 
-    public function getValidatedOption(string $option, $default = null)
+    public function getValidatedOption(string $option, mixed $default = null): mixed
     {
         return $this->getValidatedOptions()[$option] ?? $default;
     }

src/Foundation/Rectors/HasHttpClientDocCommentRector.php

@@ -1,5 +1,7 @@
 <?php
 
+/** @noinspection PhpMultipleClassDeclarationsInspection */
+
 declare(strict_types=1);
 
 /**

src/Foundation/Rectors/HasOptionsDocCommentRector.php

@@ -1,5 +1,7 @@
 <?php
 
+/** @noinspection PhpMultipleClassDeclarationsInspection */
+
 declare(strict_types=1);
 
 /**

src/Foundation/Response.php

@@ -98,7 +98,7 @@ public function __toString(): string
     }
 
     /**
-     * Create a instance from a PSR response.
+     * Create an instance from a PSR response.
      */
     public static function fromPsrResponse(ResponseInterface $response): self
     {

src/Foundation/Support/Arr.php

@@ -54,7 +54,7 @@ public static function set(array &$array, mixed $key, mixed $value): array
             return $array = $value;
         }
 
-        $keys = (array) explode('.', $key);
+        $keys = explode('.', $key);
 
         /** @noinspection SuspiciousLoopInspection */
         foreach ($keys as $i => $key) {
@@ -82,11 +82,10 @@ public static function set(array &$array, mixed $key, mixed $value): array
     /**
      * Get an item from an array using "dot" notation.
      *
-     * @noinspection MultipleReturnStatementsInspection
-     *
+     * @param array|\ArrayAccess|mixed $array
      * @param null|array-key $key
      */
-    public static function get(array|\ArrayAccess $array, mixed $key, mixed $default = null): mixed
+    public static function get(mixed $array, mixed $key, mixed $default = null): mixed
     {
         if (!static::accessible($array)) {
             return value($default);

src/Ntfy/Authenticator.php

@@ -28,7 +28,7 @@ public function __construct(
         #[\SensitiveParameter]
         ?string $password = null
     ) {
-        $authenticators = match (\func_num_args()) {
+        parent::__construct(...match (\func_num_args()) {
             0 => [
                 new NullAuthenticator,
             ],
@@ -42,8 +42,6 @@ public function __construct(
                 new BasicAuthenticator($usernameOrToken, $password),
             ],
             default => throw new InvalidArgumentException('The number of arguments must be 0, 1 or 2.'),
-        };
-
-        parent::__construct(...$authenticators);
+        });
     }
 }