Remove broken result provider

composer.json

@@ -1,8 +1,8 @@
 {
     "name": "phpro/grumphp",
-    "type": "composer-plugin",
     "description": "A composer plugin that enables source code quality checks.",
     "license": "MIT",
+    "type": "composer-plugin",
     "authors": [
         {
             "name": "Toon Verwerft",
@@ -81,12 +81,6 @@
         "symplify/easy-coding-standard": "Lets GrumPHP check coding standard.",
         "vimeo/psalm": "Lets GrumPHP discover errors in your code without running it."
     },
-    "config": {
-        "sort-packages": true
-    },
-    "extra": {
-        "class": "GrumPHP\\Composer\\GrumPHPPlugin"
-    },
     "autoload": {
         "psr-4": {
             "GrumPHP\\": "src"
@@ -100,6 +94,12 @@
     "bin": [
         "bin/grumphp"
     ],
+    "config": {
+        "sort-packages": true
+    },
+    "extra": {
+        "class": "GrumPHP\\Composer\\GrumPHPPlugin"
+    },
     "scripts": {
         "post-install-cmd": [
             "GrumPHP\\Composer\\DevelopmentIntegrator::integrate"

phive.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phive xmlns="https://phar.io/phive">
-  <phar name="psalm" version="^4.13.1" installed="4.13.1" location="./tools/psalm" copy="true"/>
-  <phar name="phpcs" version="^3.6.1" installed="3.6.1" location="./tools/phpcs" copy="true"/>
-  <phar name="phpcbf" version="^3.6.1" installed="3.6.1" location="./tools/phpcbf" copy="true"/>
-  <phar name="composer-normalize" version="^2.16.0" installed="2.16.0" location="./tools/composer-normalize" copy="true"/>
+  <phar name="psalm" version="^4.13.1" installed="4.20.0" location="./tools/psalm" copy="true"/>
+  <phar name="phpcs" version="^3.6.1" installed="3.6.2" location="./tools/phpcs" copy="true"/>
+  <phar name="phpcbf" version="^3.6.1" installed="3.6.2" location="./tools/phpcbf" copy="true"/>
+  <phar name="composer-normalize" version="^2.16.0" installed="2.23.1" location="./tools/composer-normalize" copy="true"/>
 </phive>

src/Configuration/Environment/DotEnvRegistrar.php

@@ -14,6 +14,7 @@ public static function register(EnvConfig $config): void
         $env = new Dotenv();
 
         if ($config->hasFiles()) {
+            /** @psalm-suppress InvalidArgument - Psalm types in Dotenv class are not valid currently  */
             $env->overload(...$config->getFiles());
         }
 

src/Fixer/Provider/FixableProcessProvider.php

@@ -12,7 +12,6 @@
 class FixableProcessProvider
 {
     /**
-     * @param Process $process
      * @param int[] $successExitCodes
      *
      * @return callable(): FixResult

src/Runner/MiddlewareStack.php

@@ -7,9 +7,6 @@
 use GrumPHP\Collection\TaskResultCollection;
 use GrumPHP\Runner\Middleware\RunnerMiddlewareInterface;
 
-/**
- * @psalm-immutable
- */
 class MiddlewareStack
 {
     /**

src/Runner/TaskHandler/Middleware/ErrorHandlingTaskHandlerMiddleware.php

@@ -4,6 +4,7 @@
 
 namespace GrumPHP\Runner\TaskHandler\Middleware;
 
+use Amp\Parallel\Worker\TaskFailureException;
 use function Amp\call;
 use Amp\Promise;
 use GrumPHP\Exception\PlatformException;

src/Runner/TaskHandler/Middleware/ParallelProcessingMiddleware.php

@@ -16,7 +16,6 @@
 use GrumPHP\Runner\TaskResultInterface;
 use GrumPHP\Runner\TaskRunnerContext;
 use GrumPHP\Task\TaskInterface;
-use Laravel\SerializableClosure\SerializableClosure;
 
 class ParallelProcessingMiddleware implements TaskHandlerMiddlewareInterface
 {
@@ -62,19 +61,12 @@ public function handle(TaskInterface $task, TaskRunnerContext $runnerContext, ca
          */
         $enqueueParallelTask = function () use ($task, $runnerContext, $next, $currentEnv): Promise {
             return parallel(
-                static function (array $parentEnv) use ($task, $runnerContext, $next): SerializableClosure {
+                static function (array $parentEnv) use ($task, $runnerContext, $next): TaskResultInterface {
                     $_ENV = array_merge($parentEnv, $_ENV);
                     /** @var TaskResultInterface $result */
                     $result = wait($next($task, $runnerContext));
 
-                    return new SerializableClosure(
-                        /**
-                         * @return TaskResultInterface
-                         */
-                        static function () use ($result) {
-                            return $result;
-                        }
-                    );
+                    return $result;
                 },
                 $this->poolFactory->create()
             )($currentEnv);
@@ -86,9 +78,7 @@ static function () use ($result) {
              */
             function () use ($enqueueParallelTask, $task, $runnerContext): \Generator {
                 try {
-                    /** @var callable(): TaskResultInterface $resultProvider */
-                    $resultProvider = yield $enqueueParallelTask();
-                    $result = $resultProvider();
+                    $result = yield $enqueueParallelTask();
                 } catch (\Throwable $error) {
                     return TaskResult::createFailed(
                         $task,

src/Runner/TaskHandler/TaskHandler.php

@@ -12,9 +12,6 @@
 use GrumPHP\Runner\TaskRunnerContext;
 use GrumPHP\Task\TaskInterface;
 
-/**
- * @psalm-immutable
- */
 class TaskHandler
 {
     /**