feat: use no timeout + add error handler

Author: mychidarko

src/Sprout/Process.php

@@ -12,21 +12,29 @@
 class Process
 {
     protected $process;
+    protected $errorHandler;
 
     public function __construct(string $command)
     {
         if (!\function_exists('proc_open')) {
             throw new \Exception('The Process class relies on proc_open, which is not available on your PHP installation.');
         }
 
-        $this->process = \Symfony\Component\Process\Process::fromShellCommandline($command);
+        $this->process = \Symfony\Component\Process\Process::fromShellCommandline(
+            $command,
+            null,
+            null,
+            null,
+            null
+        );
     }
 
     /**
      * Set an error action for current process
      */
     public function onError(callable $errorHandler): Process
     {
+        $this->errorHandler = $errorHandler;
         return $this;
     }
 
@@ -37,13 +45,30 @@ public function onError(callable $errorHandler): Process
      */
     public function run(?callable $callback = null): int
     {
-        return $this->process->run(function ($type, $buffer) use ($callback) {
-            if ($callback) {
-                $callback($type, $buffer);
-            } else {
-                echo $buffer;
+        try {
+            return $this->process->run(function ($type, $buffer) use ($callback) {
+                if ($callback) {
+                    $callback($type, $buffer);
+                } else {
+                    echo $buffer;
+                }
+            });
+        } catch (\Throwable $th) {
+            if ($this->errorHandler) {
+                \call_user_func($this->errorHandler, $th->getMessage(), $this->process);
             }
-        });
+
+            throw $th;
+        }
+    }
+
+    /**
+     * Check if the process is successful
+     * @return bool
+     */
+    public function isSuccessful(): bool
+    {
+        return $this->process->isSuccessful();
     }
 
     /**