Fix psalm issues

spiral · May 18, 2024 · 6319726 · 6319726
1 parent 6c148a8
commit 6319726
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 2 deletions.
diff --git a/psalm.xml b/psalm.xml
@@ -14,6 +14,12 @@
         </ignoreFiles>
     </projectFiles>
     <issueHandlers>
+        <DeprecatedInterface>
+            <errorLevel type="suppress">
+                <file name="src/Commands/InfoCommand.php"/>
+                <file name="src/DeclarationLocator.php"/>
+            </errorLevel>
+        </DeprecatedInterface>
         <PropertyNotSetInConstructor>
             <errorLevel type="suppress">
                 <file name="src/Commands/Scaffolder/ActivityCommand.php"/>

diff --git a/src/Config/TemporalConfig.php b/src/Config/TemporalConfig.php
@@ -32,6 +32,7 @@
  */
 final class TemporalConfig extends InjectableConfig
 {
+    /** @var non-empty-string */
     public const CONFIG = 'temporal';
 
     protected array $config = [

diff --git a/src/Dispatcher.php b/src/Dispatcher.php
@@ -32,7 +32,7 @@ public function serve(): void
         /**
          * @var array<class-string<WorkflowInterface>|class-string<ActivityInterface>, ReflectionClass> $declarations
          */
-        $declarations = $this->container->get(DeclarationLocatorInterface::class)->getDeclarations();
+        $declarations = $this->container->get(DeclarationRegistryInterface::class)->getDeclarations();
 
         // factory initiates and runs task queue specific activity and workflow workers
         /** @var WorkerFactoryInterface $factory */

diff --git a/src/Scaffolder/Declaration/ActivityDeclaration.php b/src/Scaffolder/Declaration/ActivityDeclaration.php
@@ -13,14 +13,15 @@
 
 final class ActivityDeclaration extends AbstractDeclaration
 {
+    /** @var non-empty-string */
     public const TYPE = 'activity';
 
     public function __construct(
         ScaffolderConfig $config,
         string $name,
         ?string $comment = null,
         ?string $namespace = null,
-        private ?string $activityName = null,
+        private readonly ?string $activityName = null,
     ) {
         parent::__construct($config, $name, $comment, $namespace);
     }

diff --git a/src/Scaffolder/Declaration/WorkflowDeclaration.php b/src/Scaffolder/Declaration/WorkflowDeclaration.php
@@ -14,6 +14,7 @@
 
 final class WorkflowDeclaration extends AbstractDeclaration
 {
+    /** @var non-empty-string */
     public const TYPE = 'workflow';
 
     public function __construct(