Improve Log parsing, fix ProcessHelper bug, and update .scrutinizer.yml

Author: igorsgm

.scrutinizer.yml

@@ -2,6 +2,7 @@ build:
   environment:
     php:
       version: 8.1
+    node: "v16.20.0"
   nodes:
     analysis:
       project_setup:
@@ -10,6 +11,8 @@ build:
         override: [ php-scrutinizer-run ]
     coverage:
       tests:
+        before:
+          - command: git config --global user.email "[email protected]" && git config --global user.name "GitHub Actions"
         override:
           - command: XDEBUG_MODE=coverage ./vendor/bin/pest --coverage --coverage-clover=coverage.clover
             coverage:

src/Git/Log.php

@@ -3,6 +3,7 @@
 namespace Igorsgm\GitHooks\Git;
 
 use Carbon\Carbon;
+use Illuminate\Support\Str;
 
 class Log
 {
@@ -52,17 +53,30 @@ public function __construct(string $log)
      */
     private function parse(array $lines): void
     {
-        foreach ($lines as $line) {
-            if (strpos($line, 'commit') === 0) {
+        $handlers = collect([
+            'commit' => function ($line) {
                 preg_match('/(?<hash>[a-z0-9]{40})/', $line, $matches);
                 $this->hash = $matches['hash'] ?? null;
-            } elseif (strpos($line, 'Author') === 0) {
+            },
+            'Author' => function ($line) {
                 $this->author = substr($line, strlen('Author:') + 1);
-            } elseif (strpos($line, 'Date') === 0) {
+            },
+            'Date' => function ($line) {
                 $this->date = Carbon::parse(substr($line, strlen('Date:') + 3));
-            } elseif (strpos($line, 'Merge') === 0) {
+            },
+            'Merge' => function ($line) {
                 $merge = substr($line, strlen('Merge:') + 1);
                 $this->merge = explode(' ', $merge);
+            },
+        ]);
+
+        foreach ($lines as $line) {
+            $handler = $handlers->first(function ($handler, $prefix) use ($line) {
+                return Str::startsWith($line, $prefix);
+            });
+
+            if ($handler !== null) {
+                $handler($line);
             } elseif (! empty($line)) {
                 $this->message .= substr($line, 4)."\n";
             }

src/Traits/ProcessHelper.php

@@ -31,7 +31,9 @@ public function runCommands($commands, $params = [])
             });
         }
 
-        if (! empty($input->definition) && $input->definition->hasOption('quiet') && $input->getOption('quiet')) {
+        if (! empty($input->definition) && $input->definition->hasOption('quiet') &&
+            ! empty($input) && $input->getOption('quiet')
+        ) {
             $commands = $this->transformCommands($commands, function ($value) {
                 return $value.' --quiet';
             });
@@ -68,7 +70,7 @@ public function runCommands($commands, $params = [])
     }
 
     /**
-     * @param  array  $commands
+     * @param  array|string  $commands
      * @param  callable  $callback
      * @return array
      */