Updated Rector to commit aea1570424613c9a0acbf80c3abeb41d7dd33dbe

rectorphp/rector-src@aea1570 [skip] improve unused skip resolver methods (#8072)

Author: TomasVotruba

src/Application/VersionResolver.php

@@ -19,12 +19,12 @@ final class VersionResolver
      * @api
      * @var string
      */
-    public const PACKAGE_VERSION = '2.5.1';
+    public const PACKAGE_VERSION = 'aea1570424613c9a0acbf80c3abeb41d7dd33dbe';
     /**
      * @api
      * @var string
      */
-    public const RELEASE_DATE = '2026-06-21 10:54:43';
+    public const RELEASE_DATE = '2026-06-21 12:54:50';
     /**
      * @var int
      */

src/Reporting/UnusedSkipResolver.php

@@ -42,23 +42,36 @@ public function __construct(SkippedClassResolver $skippedClassResolver, SkippedP
      */
     public function resolve(ProcessResult $processResult): array
     {
-        if (!SimpleParameterProvider::provideBoolParameter(Option::REPORT_UNUSED_SKIPS, \false)) {
+        if ($this->shouldSkipReporting()) {
             return [];
         }
+        $usedSkips = $processResult->getUsedSkips();
+        return array_merge($this->resolveUnusedRuleScopedSkips($usedSkips), $this->resolveUnusedGlobalSkips($usedSkips));
+    }
+    private function shouldSkipReporting(): bool
+    {
+        if (!SimpleParameterProvider::provideBoolParameter(Option::REPORT_UNUSED_SKIPS, \false)) {
+            return \true;
+        }
         // a narrowed run (cli paths, "--only" or "--only-suffix") only touches part of the codebase,
         // so skips outside that scope look falsely unused - reporting them would be noise
         if (SimpleParameterProvider::provideBoolParameter(Option::IS_RUN_NARROWED, \false)) {
-            return [];
+            return \true;
         }
         // a cached run only re-processes changed files, so skips on cached files never get a chance
         // to match and would all look falsely unused - reporting them would be noise
-        if (SimpleParameterProvider::provideBoolParameter(Option::IS_CACHED_RUN, \false)) {
-            return [];
-        }
-        // map of rule => (trackable skip path => relative display path); skips are tracked at
-        // runtime by their path, but rule-scoped ones are printed grouped under their rule so the
-        // user knows exactly what to remove. Skip-everywhere rule skips (null path) are forgotten
-        // from the container at boot, so they never reach the skipper and cannot be tracked.
+        return SimpleParameterProvider::provideBoolParameter(Option::IS_CACHED_RUN, \false);
+    }
+    /**
+     * Map of rule => (trackable skip path => relative display path); skips are tracked at
+     * runtime by their path, but rule-scoped ones are printed grouped under their rule so the
+     * user knows exactly what to remove. Skip-everywhere rule skips (null path) are forgotten
+     * from the container at boot, so they never reach the skipper and cannot be tracked.
+     *
+     * @return array<string, array<string, string>>
+     */
+    private function resolveRelativePathsByClass(): array
+    {
         $relativePathsByClass = [];
         foreach ($this->skippedClassResolver->resolve() as $rectorClass => $paths) {
             if ($paths === null) {
@@ -69,18 +82,32 @@ public function resolve(ProcessResult $processResult): array
                 $relativePathsByClass[$rectorClass][$path] = $this->filePathHelper->relativePath($path);
             }
         }
-        // global mask paths like "*/some/*" are hard to spot and report false positives, skip them
+        return $relativePathsByClass;
+    }
+    /**
+     * @return array<string, string>
+     */
+    private function resolveGlobalRelativePaths(): array
+    {
         $globalRelativePaths = [];
         foreach ($this->skippedPathsResolver->resolve() as $globalPath) {
+            // global mask paths like "*/some/*" are hard to spot and report false positives, skip them
             if (strpos($globalPath, '*') !== \false) {
                 continue;
             }
             $globalRelativePaths[$globalPath] = $this->filePathHelper->relativePath($globalPath);
         }
-        $usedSkips = $processResult->getUsedSkips();
+        return $globalRelativePaths;
+    }
+    /**
+     * @param string[] $usedSkips
+     * @return string[]
+     */
+    private function resolveUnusedRuleScopedSkips(array $usedSkips): array
+    {
         $unusedSkips = [];
         // group unused rule-scoped paths under their rule, matching the "->withSkip()" config shape
-        foreach ($relativePathsByClass as $rectorClass => $relativePaths) {
+        foreach ($this->resolveRelativePathsByClass() as $rectorClass => $relativePaths) {
             $unusedRelativePaths = [];
             foreach ($relativePaths as $path => $relativePath) {
                 if (!in_array($path, $usedSkips, \true)) {
@@ -93,7 +120,16 @@ public function resolve(ProcessResult $processResult): array
             // rule on its own line, with each unused path nested below it as a "->listing()" sub-item
             $unusedSkips[] = $rectorClass . ':' . "\n     * " . implode("\n     * ", $unusedRelativePaths);
         }
-        foreach ($globalRelativePaths as $path => $relativePath) {
+        return $unusedSkips;
+    }
+    /**
+     * @param string[] $usedSkips
+     * @return string[]
+     */
+    private function resolveUnusedGlobalSkips(array $usedSkips): array
+    {
+        $unusedSkips = [];
+        foreach ($this->resolveGlobalRelativePaths() as $path => $relativePath) {
             if (!in_array($path, $usedSkips, \true)) {
                 $unusedSkips[] = $relativePath;
             }