Introduction of phpstan

.github/workflows/php-qa.yml

@@ -13,7 +13,7 @@ on:
             - '**workflows/php-qa.yml'
             - '**.php'
             - '**phpcs.xml.dist'
-            - '**psalm.xml'
+            - '**phpstan.neon.dist'
             - '**composer.json'
     workflow_dispatch:
         inputs:
@@ -25,7 +25,7 @@ on:
                 options:
                     - 'Run all'
                     - 'Run PHPCS only'
-                    - 'Run Psalm only'
+                    - 'Run PHPStan only'
                     - 'Run lint only'
 
 concurrency:
@@ -50,11 +50,10 @@ jobs:
             PHP_VERSION: '8.3'
 
     static-code-analysis:
-        if: ${{ (github.event_name != 'workflow_dispatch') || ((github.event.inputs.jobs == 'Run all') || (github.event.inputs.jobs == 'Run Psalm only')) }}
+        if: ${{ (github.event_name != 'workflow_dispatch') || ((github.event.inputs.jobs == 'Run all') || (github.event.inputs.jobs == 'Run PHPStan only')) }}
         uses: inpsyde/reusable-workflows/.github/workflows/static-analysis-php.yml@main
         strategy:
             matrix:
                 php: [ '7.4', '8.0', '8.1', '8.2', '8.3' ]
         with:
             PHP_VERSION: ${{ matrix.php }}
-            PSALM_ARGS: --output-format=github --no-suggestions --no-cache --no-diff --find-unused-psalm-suppress

composer.json

@@ -27,12 +27,14 @@
     },
     "require-dev": {
         "brain/monkey": "^2.6.1",
-        "inpsyde/php-coding-standards": "^2@dev",
-        "inpsyde/wp-stubs-versions": "dev-latest",
+        "inpsyde/php-coding-standards": "^2.0.0",
         "roots/wordpress-no-content": "@dev",
         "mikey179/vfsstream": "^v1.6.11",
         "phpunit/phpunit": "^9.6.19",
-        "vimeo/psalm": "^5.24.0"
+        "phpstan/phpstan": "^2.1.1",
+        "szepeviktor/phpstan-wordpress": "^2",
+        "swissspidy/phpstan-no-private": "^v1.0.0",
+        "phpstan/phpstan-deprecation-rules": "^2.0.1"
     },
     "extra": {
         "branch-alias": {
@@ -53,13 +55,13 @@
     "minimum-stability": "dev",
     "prefer-stable": true,
     "scripts": {
-        "cs": "@php ./vendor/squizlabs/php_codesniffer/bin/phpcs",
-        "psalm": "@php ./vendor/vimeo/psalm/psalm --no-suggestions --report-show-info=false --find-unused-psalm-suppress --no-diff --no-cache --no-file-cache --output-format=compact",
+        "phpcs": "@php ./vendor/squizlabs/php_codesniffer/bin/phpcs",
+        "phpstan": "@php ./vendor/bin/phpstan analyse --memory-limit=1G",
         "tests": "@php ./vendor/phpunit/phpunit/phpunit --no-coverage",
         "tests:coverage": "@php ./vendor/phpunit/phpunit/phpunit",
         "qa": [
-            "@cs",
-            "@psalm",
+            "@phpcs",
+            "@phpstan",
             "@tests"
         ]
     },

phpstan.neon.dist

@@ -0,0 +1,13 @@
+includes:
+    - vendor/szepeviktor/phpstan-wordpress/extension.neon
+    - vendor/phpstan/phpstan-deprecation-rules/rules.neon
+    - vendor/swissspidy/phpstan-no-private/rules.neon
+parameters:
+    level: 8
+    paths:
+        - src/
+    treatPhpDocTypesAsCertain: false
+    ignoreErrors:
+        - '#Trait Inpsyde\\Modularity\\Module\\ModuleClassNameIdTrait is used zero times and is not analysed.#'
+        # TODO: Check how we can point ABSPATH constant in phpstan to vendor/roots/wordpress-no-content/
+        - '#Path in require_once\(\) \".*\" is not a file or it does not exist.#'

src/Container/ReadOnlyContainer.php

@@ -26,7 +26,7 @@ class ReadOnlyContainer implements ContainerInterface
     /**
      * @param array<string, Service> $services
      * @param array<string, bool> $factoryIds
-     * @param ServiceExtensions|array $extensions
+     * @param ServiceExtensions|array<string, ExtendingService> $extensions
      * @param ContainerInterface[] $containers
      */
     public function __construct(

src/Container/ServiceExtensions.php

@@ -20,6 +20,7 @@
 
     /**
      * @param string $type
+     *
      * @return string
      */
     final public static function typeId(string $type): string
@@ -30,18 +31,22 @@
     /**
      * @param string $extensionId
      * @param ExtendingService $extender
+     *
      * @return static
      */
     public function add(string $extensionId, callable $extender): ServiceExtensions
     {
-        isset($this->extensions[$extensionId]) or $this->extensions[$extensionId] = [];
+        if (!isset($this->extensions[$extensionId])) {
+            $this->extensions[$extensionId] = [];
+        }
         $this->extensions[$extensionId][] = $extender;
 
         return $this;
     }
 
     /**
      * @param string $extensionId
+     *
      * @return bool
      */
     public function has(string $extensionId): bool
@@ -53,6 +58,7 @@
      * @param mixed $service
      * @param string $id
      * @param Container $container
+     *
      * @return mixed
      */
     final public function resolve($service, string $id, Container $container)
@@ -68,6 +74,7 @@
      * @param string $id
      * @param mixed $service
      * @param Container $container
+     *
      * @return mixed
      */
     protected function resolveById(string $id, $service, Container $container)
@@ -83,20 +90,19 @@
      * @param string $className
      * @param object $service
      * @param Container $container
-     * @param array $extendedClasses
+     * @param string[] $extendedClasses
+     *
      * @return mixed
      *
-     * phpcs:disable Generic.Metrics.CyclomaticComplexity
-     * phpcs:disable Inpsyde.CodeQuality.ReturnTypeDeclaration
+     * phpcs:disable Generic.Metrics.CyclomaticComplexity.TooHigh
+     * phpcs:disable Inpsyde.CodeQuality.ReturnTypeDeclaration.NoReturnType
      */
     protected function resolveByType(
         string $className,
         object $service,
         Container $container,
         array $extendedClasses = []
     ) {
-        // phpcs:enable Generic.Metrics.CyclomaticComplexity
-        // phpcs:enable Inpsyde.CodeQuality.ReturnTypeDeclaration
 
         $extendedClasses[] = $className;
 
@@ -111,7 +117,9 @@
 
         // 2nd group of extensions: targeting parent classes
         $parents = class_parents($service, false);
-        ($parents === false) and $parents = [];
+        if ($parents === false) {
+            $parents = [];
+        }
         foreach ($parents as $parentName) {
             $byParent = $this->extensions[self::typeId($parentName)] ?? null;
             if (($byParent !== null) && ($byParent !== [])) {
@@ -121,7 +129,9 @@
 
         // 3rd group of extensions: targeting implemented interfaces
         $interfaces = class_implements($service, false);
-        ($interfaces === false) and $interfaces = [];
+        if ($interfaces === false) {
+            $interfaces = [];
+        }
         foreach ($interfaces as $interfaceName) {
             $byInterface = $this->extensions[self::typeId($interfaceName)] ?? null;
             if (($byInterface !== null) && ($byInterface !== [])) {
@@ -163,6 +173,7 @@
      * @param object $service
      * @param Container $container
      * @param list<ExtendingService> $extenders
+     *
      * @return list{mixed, int}
      */
     private function extendByType(

src/Package.php

@@ -6,8 +6,8 @@
 
 use Inpsyde\Modularity\Container\ContainerConfigurator;
 use Inpsyde\Modularity\Container\PackageProxyContainer;
-use Inpsyde\Modularity\Module\ExtendingModule;
 use Inpsyde\Modularity\Module\ExecutableModule;
+use Inpsyde\Modularity\Module\ExtendingModule;
 use Inpsyde\Modularity\Module\FactoryModule;
 use Inpsyde\Modularity\Module\Module;
 use Inpsyde\Modularity\Module\ServiceModule;
@@ -17,6 +17,8 @@
 /**
  * @psalm-import-type Service from \Inpsyde\Modularity\Module\ServiceModule
  * @psalm-import-type ExtendingService from \Inpsyde\Modularity\Module\ExtendingModule
+ *
+ * phpcs:disable WordPress.PHP.DevelopmentFunctions.error_log_trigger_error
  */
 class Package
 {
@@ -205,12 +207,12 @@ class Package
      */
     public static function new(Properties $properties, ContainerInterface ...$containers): Package
     {
-        return new self($properties, ...array_values($containers));
+        return new self($properties, ...$containers);
     }
 
     /**
      * @param Properties $properties
-     * @param list<ContainerInterface> $containers
+     * @param ContainerInterface ...$containers
      */
     private function __construct(Properties $properties, ContainerInterface ...$containers)
     {
@@ -511,7 +513,7 @@ private function addModuleServices(Module $module, string $status): bool
     {
         /** @var null|array<string, Service|ExtendingService> $services */
         $services = null;
-        /** @var null|callable(string, Service|ExtendingService) $addCallback */
+        /** @var null|callable(string, Service|ExtendingService): void $addCallback */
         $addCallback = null;
         switch ($status) {
             case self::MODULE_REGISTERED:
@@ -552,7 +554,7 @@ private function doExecute(): void
             $success = $executable->run($this->container());
             $this->moduleProgress(
                 $executable->id(),
-                $success ? self::MODULE_EXECUTED : self::MODULE_EXECUTION_FAILED
+                $success ? self::MODULE_EXECUTED : self::MODULE_EXECUTION_FAILED,
             );
         }
     }
@@ -569,7 +571,9 @@ private function moduleProgress(
         ?array $serviceIds = null
     ): void {
 
-        isset($this->moduleStatus[$status]) or $this->moduleStatus[$status] = [];
+        if (!isset($this->moduleStatus[$status])) {
+            $this->moduleStatus[$status] = [];
+        }
         $this->moduleStatus[$status][] = $moduleId;
 
         if (($serviceIds === null) || ($serviceIds === []) || !$this->properties->isDebug()) {
@@ -740,7 +744,7 @@ private function progress(int $status): void
      * @param string $packageName
      * @param string $reason
      * @param bool $throw
-     * @return ($throw is true ? never: false)
+     * @return bool
      */
     private function handleConnectionFailure(string $packageName, string $reason, bool $throw): bool
     {

src/Properties/BaseProperties.php

@@ -4,6 +4,9 @@
 
 namespace Inpsyde\Modularity\Properties;
 
+/**
+ * phpcs:disable PHPCompatibility.FunctionDeclarations.RemovedImplicitlyNullableParam.Deprecated
+ */
 class BaseProperties implements Properties
 {
     protected ?bool $isDebug = null;
@@ -40,11 +43,14 @@ protected function __construct(
 
     /**
      * @param string $name
+     *
      * @return lowercase-string
      */
     protected function sanitizeBaseName(string $name): string
     {
-        substr_count($name, '/') and $name = dirname($name);
+        if (substr_count($name, '/')) {
+            $name = dirname($name);
+        }
 
         return strtolower(pathinfo($name, PATHINFO_FILENAME));
     }
@@ -144,7 +150,9 @@ public function requiresWp(): ?string
     {
         $value = $this->get(self::PROP_REQUIRES_WP);
 
-        return (($value !== '') && is_string($value)) ? $value : null;
+        return (($value !== '') && is_string($value))
+            ? $value
+            : null;
     }
 
     /**
@@ -154,11 +162,13 @@ public function requiresPhp(): ?string
     {
         $value = $this->get(self::PROP_REQUIRES_PHP);
 
-        return (($value !== '') && is_string($value)) ? $value : null;
+        return (($value !== '') && is_string($value))
+            ? $value
+            : null;
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public function tags(): array
     {
@@ -168,6 +178,7 @@ public function tags(): array
     /**
      * @param string $key
      * @param mixed $default
+     *
      * @return mixed
      */
     public function get(string $key, $default = null)
@@ -177,6 +188,7 @@ public function get(string $key, $default = null)
 
     /**
      * @param string $key
+     *
      * @return bool
      */
     public function has(string $key): bool