fix: psalm 5

psalm.xml

@@ -2,9 +2,10 @@
 <psalm 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"
-       errorLevel="6"
-       findUnusedBaselineEntry="false"
-       findUnusedCode="false"
+       errorLevel="5"
+       findUnusedBaselineEntry="true"
+       findUnusedCode="true"
+       findUnusedPsalmSuppress="true"
 >
     <projectFiles>
         <directory name="src" />

src/Commands/PromiseClean.php

@@ -3,7 +3,7 @@
 namespace Tochka\Promises\Commands;
 
 use Illuminate\Console\Command;
-use Tochka\Promises\Facades\GarbageCollector;
+use Tochka\Promises\Core\GarbageCollectorInterface;
 
 class PromiseClean extends Command
 {
@@ -12,10 +12,13 @@ class PromiseClean extends Command
     protected $signature = 'promise:clean';
     protected $description = 'Собрать мусор и удалить';
 
-    public function handle(): void
+    /**
+     * @psalm-suppress PossiblyUnusedMethod
+     */
+    public function handle(GarbageCollectorInterface $garbageCollector): void
     {
         $this->subscribeSignals();
 
-        GarbageCollector::clean($this->shouldQuit(...), $this->paused(...));
+        $garbageCollector->clean($this->shouldQuit(...), $this->paused(...));
     }
 }

src/Commands/PromiseGc.php

@@ -5,7 +5,7 @@
 namespace Tochka\Promises\Commands;
 
 use Illuminate\Console\Command;
-use Tochka\Promises\Facades\GarbageCollector;
+use Tochka\Promises\Core\GarbageCollectorInterface;
 
 /**
  * @codeCoverageIgnore
@@ -17,10 +17,13 @@ class PromiseGc extends Command
     protected $signature = 'promise:gc';
     protected $description = 'Сборщик мусора';
 
-    public function handle(): void
+    /**
+     * @psalm-suppress PossiblyUnusedMethod
+     */
+    public function handle(GarbageCollectorInterface $garbageCollector): void
     {
         $this->subscribeSignals();
 
-        GarbageCollector::handle($this->shouldQuit(...), $this->paused(...));
+        $garbageCollector->handle($this->shouldQuit(...), $this->paused(...));
     }
 }

src/Commands/PromiseMakeMigration.php

@@ -28,6 +28,9 @@ class PromiseMakeMigration extends Command
     /** @var array<MigrationContract> */
     private array $migrations;
 
+    /**
+     * @psalm-suppress PossiblyUnusedMethod
+     */
     public function __construct(Filesystem $files, Composer $composer)
     {
         parent::__construct();
@@ -44,9 +47,7 @@ public function __construct(Filesystem $files, Composer $composer)
     }
 
     /**
-     * Execute the console command.
-     *
-     * @throws \Exception
+     * @psalm-suppress PossiblyUnusedMethod
      */
     public function handle(): void
     {
@@ -94,7 +95,7 @@ public function handle(): void
     protected function createTableMigration(string $migrationName): string
     {
         /** @var MigrationCreator $migrationCreator */
-        $migrationCreator = $this->laravel['migration.creator'];
+        $migrationCreator = $this->laravel->make('migration.creator');
 
         return $migrationCreator->create(
             $migrationName,

src/Commands/PromiseWatch.php

@@ -5,7 +5,7 @@
 namespace Tochka\Promises\Commands;
 
 use Illuminate\Console\Command;
-use Tochka\Promises\Facades\PromiseWatcher;
+use Tochka\Promises\Core\PromiseWatcherInterface;
 
 /**
  * @codeCoverageIgnore
@@ -18,10 +18,13 @@ class PromiseWatch extends Command
 
     protected $description = 'Смотритель промисов';
 
-    public function handle(): void
+    /**
+     * @psalm-suppress PossiblyUnusedMethod
+     */
+    public function handle(PromiseWatcherInterface $promiseWatcher): void
     {
         $this->subscribeSignals();
 
-        PromiseWatcher::watch($this->shouldQuit(...), $this->paused(...));
+        $promiseWatcher->watch($this->shouldQuit(...), $this->paused(...));
     }
 }

src/Conditions/AllJobsInStates.php

@@ -7,6 +7,9 @@
 use Tochka\Promises\Enums\StateEnum;
 use Tochka\Promises\Models\PromiseJob;
 
+/**
+ * @api
+ */
 final class AllJobsInStates implements ConditionContract
 {
     /** @var array<StateEnum> */

src/Conditions/AndConditions.php

@@ -5,6 +5,9 @@
 use Tochka\Promises\Contracts\ConditionContract;
 use Tochka\Promises\Core\BasePromise;
 
+/**
+ * @api
+ */
 final class AndConditions implements ConditionContract
 {
     /** @var array<ConditionContract> */

src/Conditions/EmptyJobs.php

@@ -6,6 +6,9 @@
 use Tochka\Promises\Core\BasePromise;
 use Tochka\Promises\Models\PromiseJob;
 
+/**
+ * @api
+ */
 final class EmptyJobs implements ConditionContract
 {
     public function condition(BasePromise $basePromise): bool

src/Conditions/JobInState.php

@@ -8,6 +8,9 @@
 use Tochka\Promises\Enums\StateEnum;
 use Tochka\Promises\Models\PromiseJob;
 
+/**
+ * @api
+ */
 final class JobInState implements ConditionContract
 {
     private ?int $job_id;

src/Conditions/OneJobInState.php

@@ -6,6 +6,9 @@
 use Tochka\Promises\Core\BasePromise;
 use Tochka\Promises\Enums\StateEnum;
 
+/**
+ * @api
+ */
 final class OneJobInState implements ConditionContract
 {
     /** @var array<StateEnum> */

src/Conditions/OrConditions.php

@@ -5,6 +5,9 @@
 use Tochka\Promises\Contracts\ConditionContract;
 use Tochka\Promises\Core\BasePromise;
 
+/**
+ * @api
+ */
 final class OrConditions implements ConditionContract
 {
     /** @var array<ConditionContract> */

src/Conditions/Positive.php

@@ -5,6 +5,9 @@
 use Tochka\Promises\Contracts\ConditionContract;
 use Tochka\Promises\Core\BasePromise;
 
+/**
+ * @api
+ */
 final class Positive implements ConditionContract
 {
     public function condition(BasePromise $basePromise): bool

src/Conditions/PromiseInState.php

@@ -6,6 +6,9 @@
 use Tochka\Promises\Core\BasePromise;
 use Tochka\Promises\Enums\StateEnum;
 
+/**
+ * @api
+ */
 final class PromiseInState implements ConditionContract
 {
     /** @var array<StateEnum> */

src/Conditions/Timeout.php

@@ -6,6 +6,9 @@
 use Tochka\Promises\Contracts\ConditionContract;
 use Tochka\Promises\Core\BasePromise;
 
+/**
+ * @api
+ */
 final class Timeout implements ConditionContract
 {
     private Carbon $expired_at;

src/Contracts/ConditionContract.php

@@ -5,7 +5,7 @@
 use Tochka\Promises\Core\BasePromise;
 
 /**
- * @codeCoverageIgnore
+ * @api
  */
 interface ConditionContract
 {

src/Contracts/ConditionTransitionsContract.php

@@ -5,7 +5,7 @@
 use Tochka\Promises\Core\Support\ConditionTransition;
 
 /**
- * @codeCoverageIgnore
+ * @api
  */
 interface ConditionTransitionsContract
 {

src/Contracts/CustomConnection.php

@@ -2,6 +2,9 @@
 
 namespace Tochka\Promises\Contracts;
 
+/**
+ * @api
+ */
 interface CustomConnection
 {
     public function getConnection(): ?string;

src/Contracts/CustomQueue.php

@@ -2,6 +2,9 @@
 
 namespace Tochka\Promises\Contracts;
 
+/**
+ * @api
+ */
 interface CustomQueue
 {
     public function getQueue(): ?string;

src/Contracts/DispatcherContract.php

@@ -3,7 +3,7 @@
 namespace Tochka\Promises\Contracts;
 
 /**
- * @codeCoverageIgnore
+ * @api
  */
 interface DispatcherContract
 {

src/Contracts/JobFacadeContract.php

@@ -5,7 +5,7 @@
 /**
  * Указывает, что BaseJob на самом деле скрывает под собой другой обработчик
  *
- * @codeCoverageIgnore
+ * @api
  */
 interface JobFacadeContract
 {

src/Contracts/JobStateContract.php

@@ -5,7 +5,7 @@
 use Tochka\Promises\Enums\StateEnum;
 
 /**
- * @codeCoverageIgnore
+ * @api
  */
 interface JobStateContract
 {

src/Contracts/MayPromised.php

@@ -3,7 +3,7 @@
 namespace Tochka\Promises\Contracts;
 
 /**
- * @codeCoverageIgnore
+ * @api
  */
 interface MayPromised
 {

src/Contracts/NestedEventContract.php

@@ -2,6 +2,9 @@
 
 namespace Tochka\Promises\Contracts;
 
+/**
+ * @api
+ */
 interface NestedEventContract
 {
     public function isNested(): bool;

src/Contracts/PromiseHandler.php

@@ -3,7 +3,7 @@
 namespace Tochka\Promises\Contracts;
 
 /**
- * @codeCoverageIgnore
+ * @api
  */
 interface PromiseHandler extends MayPromised
 {

src/Contracts/PromisedEvent.php

@@ -3,7 +3,7 @@
 namespace Tochka\Promises\Contracts;
 
 /**
- * @codeCoverageIgnore
+ * @api
  */
 interface PromisedEvent
 {

src/Contracts/StateChangedContract.php

@@ -5,7 +5,7 @@
 use Tochka\Promises\Enums\StateEnum;
 
 /**
- * @codeCoverageIgnore
+ * @api
  */
 interface StateChangedContract
 {

src/Contracts/StatesContract.php

@@ -5,7 +5,7 @@
 use Tochka\Promises\Enums\StateEnum;
 
 /**
- * @codeCoverageIgnore
+ * @api
  */
 interface StatesContract
 {

src/Core/BaseJob.php

@@ -12,6 +12,9 @@
 use Tochka\Promises\Enums\StateEnum;
 use Tochka\Promises\Models\PromiseJob;
 
+/**
+ * @api
+ */
 class BaseJob implements StatesContract, ConditionTransitionsContract
 {
     use ConditionTransitions;
@@ -20,7 +23,7 @@ class BaseJob implements StatesContract, ConditionTransitionsContract
     use Time;
 
     private ?int $id = null;
-    private ?int $promise_id;
+    private int $promise_id;
     private MayPromised $initial_job;
     private MayPromised $result_job;
     private ?\Throwable $exception = null;

src/Core/BasePromise.php

@@ -14,6 +14,9 @@
 use Tochka\Promises\Enums\StateEnum;
 use Tochka\Promises\Models\Promise;
 
+/**
+ * @api
+ */
 class BasePromise implements StatesContract, ConditionTransitionsContract
 {
     use ConditionTransitions;

src/Core/FakePromiseRunner.php

@@ -7,9 +7,10 @@
 use Tochka\Promises\Contracts\PromiseHandler;
 
 /**
+ * @api
  * @codeCoverageIgnore
  */
-class FakePromiseRunner
+class FakePromiseRunner implements PromiseRunnerInterface
 {
     /** @var array<string, PromiseHandler> */
     private array $promises = [];