uses psalm as phar

Author: micheleorselli

.gitignore

@@ -6,6 +6,7 @@
 /web
 /bin/
 !/bin/box.phar
+!/bin/psalm.phar
 phparkitect.phar
 composer.lock
 .php-version

Makefile

@@ -36,7 +36,7 @@ csfix: ## it launches cs fix
 	bin/php-cs-fixer fix -v
 
 psalm: ## it launches psalm
-	bin/psalm
+	bin/psalm.phar
 
 build: ## it launches all the build
 	composer install

composer.json

@@ -37,7 +37,6 @@
     "require-dev": {
         "roave/security-advisories": "dev-master",
         "symfony/var-dumper": "^3.0|^4.0|^5.0|^6.0|^7.0",
-        "vimeo/psalm": "^4.6",
         "phpunit/phpunit": "^7.5|^9.0|^10.0",
         "mikey179/vfsstream": "^1.6",
         "phpspec/prophecy": "^1.10",
@@ -55,7 +54,8 @@
         }
     },
     "config": {
-        "bin-dir": "bin"
+        "bin-dir": "bin",
+        "sort-packages": true
     },
     "bin": [
         "bin-stub/phparkitect"

psalm.xml

@@ -4,7 +4,8 @@
     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"
-
+    findUnusedCode="false"
+    findUnusedBaselineEntry="false"
     allowStringToStandInForClass="true"
 >
     <projectFiles>

src/Analyzer/ClassDescription.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Arkitect\Analyzer;
@@ -46,6 +47,7 @@ class ClassDescription
      * @param list<FullyQualifiedClassName> $interfaces
      * @param ?FullyQualifiedClassName      $extends
      * @param list<FullyQualifiedClassName> $attributes
+     * @param list<string>                  $docBlock
      */
     public function __construct(
         FullyQualifiedClassName $FQCN,

src/Analyzer/FileVisitor.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Arkitect\Analyzer;
@@ -30,7 +31,7 @@ public function enterNode(Node $node): void
 
             foreach ($node->implements as $interface) {
                 $this->classDescriptionBuilder
-                     ->addInterface($interface->toString(), $interface->getLine());
+                    ->addInterface($interface->toString(), $interface->getLine());
             }
 
             if (!$node->isAnonymous() && null !== $node->extends) {
@@ -76,7 +77,8 @@ public function enterNode(Node $node): void
          *
          * @see FileVisitorTest::test_it_should_return_errors_for_const_outside_namespace
          */
-        if ($node instanceof Node\Expr\ClassConstFetch
+        if (
+            $node instanceof Node\Expr\ClassConstFetch
             && method_exists($node->class, 'toString')
         ) {
             if ($this->isSelfOrStaticOrParent($node->class->toString())) {
@@ -93,7 +95,8 @@ public function enterNode(Node $node): void
          *
          * @see FileVisitorTest::test_should_returns_all_dependencies
          */
-        if ($node instanceof Node\Expr\StaticCall
+        if (
+            $node instanceof Node\Expr\StaticCall
             && method_exists($node->class, 'toString')
         ) {
             if ($this->isSelfOrStaticOrParent($node->class->toString())) {
@@ -104,7 +107,8 @@ public function enterNode(Node $node): void
                 ->addDependency(new ClassDependency($node->class->toString(), $node->getLine()));
         }
 
-        if ($node instanceof Node\Expr\Instanceof_
+        if (
+            $node instanceof Node\Expr\Instanceof_
             && method_exists($node->class, 'toString')
         ) {
             if ($this->isSelfOrStaticOrParent($node->class->toString())) {
@@ -114,11 +118,13 @@ public function enterNode(Node $node): void
                 ->addDependency(new ClassDependency($node->class->toString(), $node->getLine()));
         }
 
-        if ($node instanceof Node\Expr\New_
+        if (
+            $node instanceof Node\Expr\New_
             && !($node->class instanceof Node\Expr\Variable)
         ) {
-            if ((method_exists($node->class, 'isAnonymous') && $node->class->isAnonymous())
-                || !method_exists($node->class, 'toString')) {
+            if ((method_exists($node->class, 'isAnonymous') && true === $node->class->isAnonymous())
+                || !method_exists($node->class, 'toString')
+            ) {
                 return;
             }
 
@@ -215,7 +221,7 @@ public function enterNode(Node $node): void
             $returnType = $node->returnType;
             if ($returnType instanceof Node\Name\FullyQualified) {
                 $this->classDescriptionBuilder
-                  ->addDependency(new ClassDependency($returnType->toString(), $returnType->getLine()));
+                    ->addDependency(new ClassDependency($returnType->toString(), $returnType->getLine()));
             }
         }
     }
@@ -271,7 +277,7 @@ private function addParamDependency(Node\Param $node): void
             $type = $nullableType->type;
         }
 
-        if (method_exists($type, 'isSpecialClassName') && $type->isSpecialClassName()) {
+        if (method_exists($type, 'isSpecialClassName') && true === $type->isSpecialClassName()) {
             return;
         }
 

src/CLI/Command/Check.php

@@ -97,11 +97,11 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $startTime = microtime(true);
 
         try {
-            $verbose = $input->getOption('verbose');
-            $stopOnFailure = $input->getOption(self::STOP_ON_FAILURE_PARAM);
-            $useBaseline = $input->getOption(self::USE_BASELINE_PARAM);
-            $skipBaseline = $input->getOption(self::SKIP_BASELINE_PARAM);
-            $ignoreBaselineLinenumbers = $input->getOption(self::IGNORE_BASELINE_LINENUMBERS_PARAM);
+            $verbose = (bool) $input->getOption('verbose');
+            $stopOnFailure = (bool) $input->getOption(self::STOP_ON_FAILURE_PARAM);
+            $useBaseline = (string) $input->getOption(self::USE_BASELINE_PARAM);
+            $skipBaseline = (bool) $input->getOption(self::SKIP_BASELINE_PARAM);
+            $ignoreBaselineLinenumbers = (bool) $input->getOption(self::IGNORE_BASELINE_LINENUMBERS_PARAM);
 
             if (true !== $skipBaseline && !$useBaseline && file_exists(self::DEFAULT_BASELINE_FILENAME)) {
                 $useBaseline = self::DEFAULT_BASELINE_FILENAME;

src/ClassSet.php

@@ -1,10 +1,14 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Arkitect;
 
 use Symfony\Component\Finder\Finder;
 
+/**
+ * @template-implements \IteratorAggregate<string, \SplFileInfo>
+ */
 class ClassSet implements \IteratorAggregate
 {
     /** @var string */

src/Rules/ParsingErrors.php

@@ -1,10 +1,14 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Arkitect\Rules;
 
 use Arkitect\Exceptions\IndexNotFoundException;
 
+/**
+ * @template-implements \IteratorAggregate<ParsingError>
+ */
 class ParsingErrors implements \IteratorAggregate, \Countable
 {
     /**

src/Rules/Violations.php

@@ -7,12 +7,16 @@
 use Arkitect\Exceptions\FailOnFirstViolationException;
 use Arkitect\Exceptions\IndexNotFoundException;
 
+/**
+ * @template-implements \IteratorAggregate<Violation>
+ */
 class Violations implements \IteratorAggregate, \Countable, \JsonSerializable
 {
     /**
      * @var Violation[]
      */
     private $violations;
+
     /**
      * @var bool
      */