🎨 Phpstan

Author: 何平

cloud-admin/Annotation/EnumMessage.php

@@ -18,7 +18,6 @@
 final class EnumMessage extends AbstractAnnotation
 {
     /**
-     * @param string $message
      * @phpstan-param string $message
      */
     public function __construct(public readonly string $message = '') {}

cloud-admin/Annotation/HttpCode.php

@@ -19,7 +19,6 @@
 final class HttpCode extends AbstractAnnotation
 {
     /**
-     * @param int $code
      * @phpstan-param int $code
      */
     public function __construct(public readonly int $code = Status::OK) {}

cloud-admin/Context/Coroutine.php

@@ -25,12 +25,11 @@
 final class Coroutine
 {
     /**
-     * @var LoggerInterface|StdoutLoggerInterface|mixed $logger
+     * @var LoggerInterface|mixed|StdoutLoggerInterface
      */
     private readonly LoggerInterface $logger;
 
     /**
-     * @param ContainerInterface $container
      * @throws ContainerExceptionInterface
      * @throws NotFoundExceptionInterface
      */

cloud-admin/Http2/Config/Ssl.php

@@ -41,13 +41,7 @@ public function __construct(/**
      * @var array{certificate:string,certificate_key:string,verify_peer:bool,verify_peer_name:bool,allow_self_signed:bool,alpn_protocols:string}
      */
     protected array $config = [])
-    {
-//        $this
-//            ->setCertificate($this->config['certificate'] ?? '')
-//            ->setCertificateKey($config['certificate_key'] ?? '')
-//            ->setVerifyPeer($this->config['verify_peer'] ?? false)
-//            ->setVerifyPeerName($this->config['verify_peer_name'] ?? false)
-//            ->setAllowSelfSigned($this->config['allow_self_signed'] ?? false);
+    {;
     }
 
     /**

cloud-admin/Interfaces/LockInterface.php

@@ -14,14 +14,12 @@
 interface LockInterface
 {
     /**
-     * @return mixed
      * @phpstan-return mixed
      */
-    public function lock():mixed;
+    public function lock(): mixed;
 
     /**
-     * @return bool
      * @phpstan-return bool
      */
-    public function unlock():bool;
+    public function unlock(): bool;
 }

cloud-admin/Log/SwowSocketHandler.php

@@ -45,8 +45,6 @@ public function __construct(
     }
 
     /**
-     * @param LogRecord $record
-     * @return void
      * @phpstan-param LogRecord $record
      */
     protected function write(LogRecord $record): void
@@ -57,8 +55,6 @@ protected function write(LogRecord $record): void
     }
 
     /**
-     * @param string $level
-     * @return Color
      * @phpstan-return Color
      */
     protected function getColorFromLevel(string $level = LogLevel::DEBUG): Color
@@ -74,8 +70,6 @@ protected function getColorFromLevel(string $level = LogLevel::DEBUG): Color
     }
 
     /**
-     * @param LogRecord $record
-     * @return string
      * @phpstan-param LogRecord $record
      * @phpstan-return string
      */

cloud-admin/Lru/LRUMemoizer.php

@@ -35,6 +35,8 @@ public function __construct(
      * @template T
      * @param callable(): T $factory
      * @return T
+     * @phpstan-return T
+     * @phpstan-param callable(): T $factory
      */
     public function get(string $key, callable $factory): mixed
     {

cloud-admin/Middleware/ProfilerMiddleware.php

@@ -18,7 +18,9 @@
 use Hyperf\Context\Context;
 use Hyperf\Contract\ConfigInterface;
 use Hyperf\HttpServer\Request;
+use Psr\Container\ContainerExceptionInterface;
 use Psr\Container\ContainerInterface;
+use Psr\Container\NotFoundExceptionInterface;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;
 use Psr\Http\Server\MiddlewareInterface;
@@ -55,6 +57,10 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
         return $response;
     }
 
+    /**
+     * @throws ContainerExceptionInterface
+     * @throws NotFoundExceptionInterface
+     */
     private function logAndSave(string $startTime, ?ServerRequestInterface $request, ?ResponseInterface $response): bool
     {
         $times = explode(' ', $startTime);

cloud-admin/RateLimit/TokenBucket.php

@@ -41,6 +41,9 @@ public function __construct(private readonly int $capacity, private int $tokenRa
         $this->runTicker();
     }
 
+    /**
+     * @phpstan-return void
+     */
     public function acquireToken(): void
     {
         if ($this->tokens < 1) {

cloud-admin/SDB/Business/Route.php

@@ -35,11 +35,18 @@
  */
 final class Route
 {
+    /**
+     * @phpstan-param ContainerInterface $container
+     * @phpstan-param ConfigInterface $config
+     */
     public function __construct(
         private readonly ContainerInterface $container,
         private readonly ConfigInterface $config,
     ) {}
 
+    /**
+     * @phpstan-return array<array-key, mixed>
+     */
     public function getRoute(): array
     {
         $factory = $this->container->get(DispatcherFactory::class);
@@ -56,6 +63,12 @@ public function getRoute(): array
         return array_slice($rows, 0, count($rows) - 1);
     }
 
+    /**
+     * @phpstan-param string $server
+     * @phpstan-param RouteCollector $router
+     * @phpstan-param string|null $path
+     * @phpstan-return array<array-key, mixed>
+     */
     private function analyzeRouter(
         string $server,
         RouteCollector $router,
@@ -89,6 +102,12 @@ private function analyzeRouter(
         return $data;
     }
 
+    /**
+     * @phpstan-param string $server
+     * @phpstan-param RouteCollector $router
+     * @phpstan-param string|null $path
+     * @phpstan-return array<array-key, mixed>
+     */
     private function analyzeHandler(
         array &$data,
         string $serverName,

cloud-admin/Server/SslConfig.php

@@ -37,14 +37,7 @@ public function __construct(/**
      * @var array{certificate:string,certificate_key:string,verify_peer:bool,verify_peer_name:bool,allow_self_signed:bool}
      */
         private array $config = []
-    ) {
-        //        $this
-        //            ->setCertificate($this->config['certificate'] ?? '')
-        //            ->setCertificateKey($config['certificate_key'] ?? '')
-        //            ->setVerifyPeer($this->config['verify_peer'] ?? false)
-        //            ->setVerifyPeerName($this->config['verify_peer_name'] ?? false)
-        //            ->setAllowSelfSigned($this->config['allow_self_signed'] ?? false);
-    }
+    ) {}
 
     /**
      * @return bool

composer.json

@@ -102,7 +102,7 @@
     "post-root-package-install": [
       "@php -r \"file_exists('.env') || copy('.env.example', '.env');\""
     ],
-    "analyse": "@php vendor/bin/phpstan analyse --memory-limit 512M -l 0 -c phpstan.neon",
+    "analyse": "@php vendor/bin/phpstan analyse --memory-limit 1024M -l 0 -c phpstan.neon",
     "cs-fix": "@php vendor/bin/php-cs-fixer fix $1",
     "rector": "rector process --clear-cache",
     "route": "@php bin/hyperf.php describe:routes",