fix: resolved #2833

src/Kernel/Config.php

@@ -86,25 +86,25 @@ public function all(): array
     }
 
     #[Pure]
-    public function offsetExists(mixed $key): bool
+    public function offsetExists(mixed $offset): bool
     {
-        return $this->has(strval($key));
+        return $this->has(strval($offset));
     }
 
     #[Pure]
-    public function offsetGet(mixed $key): mixed
+    public function offsetGet(mixed $offset): mixed
     {
-        return $this->get(strval($key));
+        return $this->get(strval($offset));
     }
 
-    public function offsetSet(mixed $key, mixed $value): void
+    public function offsetSet(mixed $offset, mixed $value): void
     {
-        $this->set(strval($key), $value);
+        $this->set(strval($offset), $value);
     }
 
-    public function offsetUnset(mixed $key): void
+    public function offsetUnset(mixed $offset): void
     {
-        $this->set(strval($key), null);
+        $this->set(strval($offset), null);
     }
 
     /**

src/Kernel/Exceptions/BadMethodCallException.php

@@ -2,6 +2,4 @@
 
 namespace EasyWeChat\Kernel\Exceptions;
 
-class BadMethodCallException extends Exception
-{
-}
+class BadMethodCallException extends Exception {}

src/Kernel/Exceptions/BadRequestException.php

@@ -4,6 +4,4 @@
 
 namespace EasyWeChat\Kernel\Exceptions;
 
-class BadRequestException extends Exception
-{
-}
+class BadRequestException extends Exception {}

src/Kernel/Exceptions/BadResponseException.php

@@ -4,6 +4,4 @@
 
 namespace EasyWeChat\Kernel\Exceptions;
 
-class BadResponseException extends Exception
-{
-}
+class BadResponseException extends Exception {}

src/Kernel/Exceptions/DecryptException.php

@@ -4,6 +4,4 @@
 
 namespace EasyWeChat\Kernel\Exceptions;
 
-class DecryptException extends Exception
-{
-}
+class DecryptException extends Exception {}

src/Kernel/Exceptions/Exception.php

@@ -6,6 +6,4 @@
 
 use Exception as BaseException;
 
-class Exception extends BaseException
-{
-}
+class Exception extends BaseException {}

src/Kernel/Exceptions/InvalidArgumentException.php

@@ -4,6 +4,4 @@
 
 namespace EasyWeChat\Kernel\Exceptions;
 
-class InvalidArgumentException extends Exception
-{
-}
+class InvalidArgumentException extends Exception {}

src/Kernel/Exceptions/InvalidConfigException.php

@@ -4,6 +4,4 @@
 
 namespace EasyWeChat\Kernel\Exceptions;
 
-class InvalidConfigException extends Exception
-{
-}
+class InvalidConfigException extends Exception {}

src/Kernel/Exceptions/RuntimeException.php

@@ -4,6 +4,4 @@
 
 namespace EasyWeChat\Kernel\Exceptions;
 
-class RuntimeException extends Exception
-{
-}
+class RuntimeException extends Exception {}

src/Kernel/Exceptions/ServiceNotFoundException.php

@@ -4,6 +4,4 @@
 
 namespace EasyWeChat\Kernel\Exceptions;
 
-class ServiceNotFoundException extends Exception
-{
-}
+class ServiceNotFoundException extends Exception {}

src/Kernel/Form/File.php

@@ -43,7 +43,7 @@ public static function fromContents(
         ?string $encoding = null
     ): DataPart {
         if ($contentType === null) {
-            $mimeTypes = new MimeTypes();
+            $mimeTypes = new MimeTypes;
 
             if ($filename) {
                 $ext = strtolower(pathinfo($filename, PATHINFO_EXTENSION));

src/Kernel/Form/Form.php

@@ -11,9 +11,7 @@ class Form
     /**
      * @param  array<string|array|DataPart>  $fields
      */
-    public function __construct(protected array $fields)
-    {
-    }
+    public function __construct(protected array $fields) {}
 
     /**
      * @param  array<string|array|DataPart>  $fields

src/Kernel/HttpClient/AccessTokenExpiredRetryStrategy.php

@@ -34,7 +34,7 @@ public function shouldRetry(
         ?string $responseContent,
         ?TransportExceptionInterface $exception
     ): ?bool {
-        if ((bool) $responseContent && $this->decider && ($this->decider)($context, $responseContent, $exception)) {
+        if ($responseContent && $this->decider && ($this->decider)($context, $responseContent, $exception)) {
             if ($this->accessToken instanceof RefreshableAccessTokenInterface) {
                 return (bool) $this->accessToken->refresh();
             }

src/Kernel/HttpClient/RequestUtil.php

@@ -104,8 +104,8 @@ public static function formatOptions(array $options, string $method): array
     }
 
     /**
-     * @param  array<string, array<string,mixed>|mixed>  $options
-     * @return array<string, array|mixed>
+     * @param  array{headers?:array<string, string>, xml?:array|string, body?:array|string, json?:array|string}  $options
+     * @return array{headers?:array<string, string|array<string, string>|array<string>>, xml?:array|string, body?:array|string}
      */
     public static function formatBody(array $options): array
     {
@@ -121,8 +121,7 @@ public static function formatBody(array $options): array
             }
 
             if (! $contentType) {
-                /** @phpstan-ignore-next-line */
-                $options['headers']['Content-Type'] = [$options['headers'][] = 'Content-Type: text/xml'];
+                $options['headers']['Content-Type'] = 'text/xml';
             }
 
             $options['body'] = $options['xml'];
@@ -143,8 +142,7 @@ public static function formatBody(array $options): array
             }
 
             if (! $contentType) {
-                /** @phpstan-ignore-next-line */
-                $options['headers']['Content-Type'] = [$options['headers'][] = 'Content-Type: application/json'];
+                $options['headers']['Content-Type'] = 'application/json';
             }
 
             $options['body'] = $options['json'];
@@ -156,7 +154,7 @@ public static function formatBody(array $options): array
 
     public static function createDefaultServerRequest(): ServerRequestInterface
     {
-        $psr17Factory = new Psr17Factory();
+        $psr17Factory = new Psr17Factory;
 
         $creator = new ServerRequestCreator(
             serverRequestFactory: $psr17Factory,

src/Kernel/HttpClient/RequestWithPresets.php

@@ -86,17 +86,16 @@ public function with(string|array $key, mixed $value = null): static
 
     /**
      * @throws RuntimeException
-     * @throws InvalidArgumentException
      */
     public function withFile(string $pathOrContents, string $formName = 'file', ?string $filename = null): static
     {
         $file = is_file($pathOrContents) ? File::fromPath(
             $pathOrContents,
             $filename
-        ) : File::withContents($pathOrContents, $filename);
+        ) : File::fromContents($pathOrContents, $filename);
 
         /**
-         * @var array{headers: array<string, string>, body: string}
+         * @var array{headers: array<string, string>, body: string} $options
          */
         $options = Form::create([$formName => $file])->toOptions();
 
@@ -109,7 +108,6 @@ public function withFile(string $pathOrContents, string $formName = 'file', ?str
 
     /**
      * @throws RuntimeException
-     * @throws InvalidArgumentException
      */
     public function withFileContents(string $contents, string $formName = 'file', ?string $filename = null): static
     {
@@ -118,7 +116,6 @@ public function withFileContents(string $contents, string $formName = 'file', ?s
 
     /**
      * @throws RuntimeException
-     * @throws InvalidArgumentException
      */
     public function withFiles(array $files): static
     {

src/Kernel/HttpClient/Response.php

@@ -47,8 +47,7 @@ public function __construct(
         protected ResponseInterface $response,
         protected ?Closure $failureJudge = null,
         protected bool $throw = true
-    ) {
-    }
+    ) {}
 
     public function throw(bool $throw = true): static
     {
@@ -161,7 +160,7 @@ public function toStream(?bool $throw = null)
             throw new BadMethodCallException(sprintf('%s does\'t implements %s', \get_class($this->response), StreamableInterface::class));
         }
 
-        return StreamWrapper::createResource(new MockResponse());
+        return StreamWrapper::createResource(new MockResponse);
     }
 
     /**
@@ -185,7 +184,7 @@ public function toPsrResponse(?ResponseFactoryInterface $responseFactory = null,
             }
 
             try {
-                $psr17Factory = class_exists(Psr17Factory::class, false) ? new Psr17Factory() : null;
+                $psr17Factory = class_exists(Psr17Factory::class, false) ? new Psr17Factory : null;
                 $responseFactory ??= $psr17Factory ?? Psr17FactoryDiscovery::findResponseFactory(); /** @phpstan-ignore-line */
                 $streamFactory ??= $psr17Factory ?? Psr17FactoryDiscovery::findStreamFactory(); /** @phpstan-ignore-line */
 

src/Kernel/ServerResponse.php

@@ -6,7 +6,6 @@
 use const PHP_OUTPUT_HANDLER_FLUSHABLE;
 use const PHP_OUTPUT_HANDLER_REMOVABLE;
 
-use JetBrains\PhpStorm\Pure;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\StreamInterface;
 
@@ -25,7 +24,6 @@ public function __construct(protected ResponseInterface $response)
         $this->response->getBody()->rewind();
     }
 
-    #[Pure]
     public static function make(ResponseInterface $response): ServerResponse
     {
         if ($response instanceof ServerResponse) {

src/Kernel/Traits/InteractWithHandlers.php

@@ -97,7 +97,7 @@ protected function makeClosure(callable|string $handler): callable
              *
              * @phpstan-ignore-next-line https://github.com/phpstan/phpstan/issues/5867
              */
-            return fn (): mixed => (new $handler())(...func_get_args());
+            return fn (): mixed => (new $handler)(...func_get_args());
         }
 
         throw new InvalidArgumentException(sprintf('Invalid handler: %s.', $handler));

src/Kernel/Traits/InteractWithServerRequest.php

@@ -32,7 +32,7 @@ public function setRequest(ServerRequestInterface $request): static
 
     public function setRequestFromSymfonyRequest(Request $symfonyRequest): static
     {
-        $psr17Factory = new Psr17Factory();
+        $psr17Factory = new Psr17Factory;
         $psrHttpFactory = new PsrHttpFactory($psr17Factory, $psr17Factory, $psr17Factory, $psr17Factory);
 
         $this->request = $psrHttpFactory->createRequest($symfonyRequest);

src/MiniApp/Application.php

@@ -156,7 +156,7 @@ public function createClient(): AccessTokenAwareClient
     {
         $httpClient = $this->getHttpClient();
 
-        if ((bool) $this->config->get('http.retry', false)) {
+        if ($this->config->get('http.retry', false)) {
             $httpClient = new RetryableHttpClient(
                 $httpClient,
                 $this->getRetryStrategy(),
@@ -169,7 +169,7 @@ public function createClient(): AccessTokenAwareClient
             accessToken: $this->getAccessToken(),
             failureJudge: fn (
                 Response $response
-            ) => (bool) ($response->toArray()['errcode'] ?? 0) || ! is_null($response->toArray()['error'] ?? null),
+            ) => ($response->toArray()['errcode'] ?? 0) || ! is_null($response->toArray()['error'] ?? null),
             throw: (bool) $this->config->get('http.throw', true),
         ))->setPresets($this->config->all());
     }

src/MiniApp/Server.php

@@ -4,6 +4,4 @@
 
 namespace EasyWeChat\MiniApp;
 
-class Server extends \EasyWeChat\OfficialAccount\Server
-{
-}
+class Server extends \EasyWeChat\OfficialAccount\Server {}

src/MiniApp/Utils.php

@@ -12,9 +12,7 @@
 
 class Utils
 {
-    public function __construct(protected Application $app)
-    {
-    }
+    public function __construct(protected Application $app) {}
 
     /**
      * @throws HttpException

src/OfficialAccount/AccessToken.php

@@ -69,7 +69,7 @@ public function getToken(): string
     {
         $token = $this->cache->get($this->getKey());
 
-        if ((bool) $token && is_string($token)) {
+        if ($token && is_string($token)) {
             return $token;
         }
 

src/OfficialAccount/Account.php

@@ -14,8 +14,7 @@ public function __construct(
         protected ?string $secret,
         protected ?string $token = null,
         protected ?string $aesKey = null
-    ) {
-    }
+    ) {}
 
     public function getAppId(): string
     {

src/OfficialAccount/JsApiTicket.php

@@ -26,7 +26,7 @@ public function getTicket(): string
         $key = $this->getKey();
         $ticket = $this->cache->get($key);
 
-        if ((bool) $ticket && \is_string($ticket)) {
+        if ($ticket && \is_string($ticket)) {
             return $ticket;
         }
 

src/OfficialAccount/Server.php

@@ -45,7 +45,7 @@ public function __construct(
      */
     public function serve(): ResponseInterface
     {
-        if ((bool) ($str = $this->request->getQueryParams()['echostr'] ?? '')) {
+        if ($str = $this->request->getQueryParams()['echostr'] ?? '') {
             return new Response(200, [], $str);
         }
 

src/OfficialAccount/Utils.php

@@ -15,9 +15,7 @@
 
 class Utils
 {
-    public function __construct(protected Application $app)
-    {
-    }
+    public function __construct(protected Application $app) {}
 
     /**
      * @param  array<string>  $jsApiList

src/OpenPlatform/Account.php

@@ -13,8 +13,7 @@ public function __construct(
         protected string $secret,
         protected string $token,
         protected string $aesKey
-    ) {
-    }
+    ) {}
 
     public function getAppId(): string
     {

src/OpenPlatform/AuthorizerAccessToken.php

@@ -11,9 +11,7 @@
 
 class AuthorizerAccessToken implements AccessToken, Stringable
 {
-    public function __construct(protected string $appId, protected string $accessToken)
-    {
-    }
+    public function __construct(protected string $appId, protected string $accessToken) {}
 
     public function getAppId(): string
     {

src/OpenPlatform/ComponentAccessToken.php

@@ -61,7 +61,7 @@ public function getToken(): string
     {
         $token = $this->cache->get($this->getKey());
 
-        if ((bool) $token && \is_string($token)) {
+        if ($token && \is_string($token)) {
             return $token;
         }
 

src/OpenPlatform/Server.php

@@ -48,7 +48,7 @@ public function __construct(
      */
     public function serve(): ResponseInterface
     {
-        if ((bool) ($str = $this->request->getQueryParams()['echostr'] ?? '')) {
+        if ($str = $this->request->getQueryParams()['echostr'] ?? '') {
             return new Response(200, [], $str);
         }