Add psalm

Author: Robin de Graaf

.github/workflows/tests.yml

@@ -32,3 +32,6 @@ jobs:
 
     - name: Run test suite
       run: vendor/bin/phpunit tests
+
+    - name: Run static analysis
+      run: vendor/bin/psalm

.gitignore

@@ -1,4 +1,5 @@
+.DS_Store
 .idea
 composer.lock
 vendor
-coverage
+coverage

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Parable PHP DI
 
+## 0.3.2
+
+- Add static analysis using psalm.
+
 ## 0.3.1
 
 _Fixes_

Makefile

@@ -4,6 +4,10 @@ dependencies:
 		--no-plugins \
 		--no-scripts
 
+psalm:
+	vendor/bin/psalm --clear-cache
+	vendor/bin/psalm
+
 tests: dependencies
 	vendor/bin/phpunit --verbose tests
 
@@ -13,3 +17,7 @@ coverage: dependencies
 
 tests-clean:
 	vendor/bin/phpunit --verbose tests
+
+coverage-clean:
+	rm -rf ./coverage
+	XDEBUG_MODE=coverage vendor/bin/phpunit --coverage-html ./coverage tests

composer.json

@@ -16,7 +16,8 @@
     "php": ">=8.0"
   },
   "require-dev": {
-    "phpunit/phpunit": "^9.0"
+    "phpunit/phpunit": "^9.0",
+    "vimeo/psalm": "^4.6"
   },
   "autoload": {
     "psr-4": {

psalm.xml

@@ -0,0 +1,19 @@
+<?xml version="1.0"?>
+<psalm
+    errorLevel="4"
+    resolveFromConfigFile="true"
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xmlns="https://getpsalm.org/schema/config"
+    xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
+>
+    <projectFiles>
+        <directory name="src" />
+        <ignoreFiles>
+            <directory name="vendor" />
+        </ignoreFiles>
+    </projectFiles>
+
+    <issueHandlers>
+        <UnresolvableInclude errorLevel="suppress" />
+    </issueHandlers>
+</psalm>

src/Container.php

@@ -15,7 +15,7 @@ class Container
 
     protected array $instances = [];
 
-    /** @var string[][] */
+    /** @var bool[][] */
     protected array $relationships = [];
 
     /** @var string[] */
@@ -98,7 +98,7 @@ protected function createInstance(string $name, int $useStoredDependencies): obj
         try {
             $dependencies = $this->getDependenciesFor($name, $useStoredDependencies);
         } catch (Throwable $t) {
-            throw new ContainerException($t->getMessage(), $t->getCode(), $t);
+            throw new ContainerException($t->getMessage(), (int)$t->getCode(), $t);
         }
 
         return new $name(...$dependencies);
@@ -146,14 +146,14 @@ public function getDependenciesFor(
                     'Could not create instance for class `%s`.',
                     $name
                 ),
-                $t->getCode(),
+                (int)$t->getCode(),
                 $t
             );
         }
 
         $constructor = $reflection->getConstructor();
 
-        if (!$constructor) {
+        if ($constructor === null) {
             return [];
         }
 
@@ -172,7 +172,9 @@ public function getDependenciesFor(
             $class = null;
 
             if ($parameter->getType() instanceof ReflectionNamedType && $builtIn === false) {
-                $class = new ReflectionClass($parameter->getType()->getName());
+                /** @var ReflectionNamedType $type */
+                $type = $parameter->getType();
+                $class = new ReflectionClass($type->getName());
             }
 
             if ($class === null) {