perf: minor render and file-collection optimizations (#156)

* perf: short-circuit exclude pattern matching

* perf: cache resolved working directory in result renderer

Author: konradmichalik

src/FileDetector/Collector.php

@@ -80,10 +80,7 @@ public function collectFiles(
                 if ($excludePatterns) {
                     $files = array_filter(
                         $files,
-                        static fn ($file) => !array_filter(
-                            $excludePatterns,
-                            static fn ($pattern) => fnmatch($pattern, basename((string) $file)),
-                        ),
+                        fn ($file) => !$this->matchesAnyPattern(basename((string) $file), $excludePatterns),
                     );
                 }
 
@@ -227,6 +224,23 @@ private function findFiles(string $path, array $supportedExtensions, bool $recur
         return $files;
     }
 
+    /**
+     * Returns true as soon as the basename matches any exclude pattern,
+     * short-circuiting instead of evaluating every pattern.
+     *
+     * @param string[] $patterns
+     */
+    private function matchesAnyPattern(string $basename, array $patterns): bool
+    {
+        foreach ($patterns as $pattern) {
+            if (fnmatch($pattern, $basename)) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
     /**
      * Rejects files larger than the configured limit to avoid loading huge
      * (potentially malicious) files entirely into memory.

src/Result/AbstractValidationResultRenderer.php

@@ -26,6 +26,13 @@
  */
 abstract class AbstractValidationResultRenderer implements ValidationResultRendererInterface
 {
+    /**
+     * Resolved current working directory, cached per renderer instance so it is
+     * not recomputed for every path during grouping. `false` means resolution
+     * failed; null means not yet resolved.
+     */
+    private string|false|null $resolvedCwd = null;
+
     public function __construct(
         protected readonly OutputInterface $output,
         protected readonly bool $dryRun = false,
@@ -112,12 +119,8 @@ protected function normalizePath(string $path): string
 
         $normalizedPath = rtrim($realPath, \DIRECTORY_SEPARATOR);
 
-        $cwd = getcwd();
+        $realCwd = $this->resolvedCwd();
         // @codeCoverageIgnoreStart
-        if (false === $cwd) {
-            return $normalizedPath;
-        }
-        $realCwd = realpath($cwd);
         if (false === $realCwd) {
             return $normalizedPath;
         }
@@ -156,4 +159,18 @@ protected function calculateExitCode(ValidationResult $validationResult): int
         return $validationResult->getOverallResult()
             ->resolveErrorToCommandExitCode($this->dryRun, $this->strict);
     }
+
+    /**
+     * Resolves and caches the current working directory for this renderer.
+     */
+    private function resolvedCwd(): string|false
+    {
+        if (null !== $this->resolvedCwd) {
+            return $this->resolvedCwd;
+        }
+
+        $cwd = getcwd();
+
+        return $this->resolvedCwd = false === $cwd ? false : realpath($cwd);
+    }
 }