Updated Rector to commit df98b3b4e5f024d2260edc233dc9cb4adfe6a3e0

rectorphp/rector-src@df98b3b [skip] display skips only on uncached run (#8071)

Author: TomasVotruba

src/Application/VersionResolver.php

@@ -19,12 +19,12 @@ final class VersionResolver
      * @api
      * @var string
      */
-    public const PACKAGE_VERSION = '84ab911ef53267aa1c4a9466064def614e486eea';
+    public const PACKAGE_VERSION = 'df98b3b4e5f024d2260edc233dc9cb4adfe6a3e0';
     /**
      * @api
      * @var string
      */
-    public const RELEASE_DATE = '2026-06-20 23:18:03';
+    public const RELEASE_DATE = '2026-06-21 10:54:43';
     /**
      * @var int
      */

src/Caching/UnchangedFilesFilter.php

@@ -4,6 +4,8 @@
 namespace Rector\Caching;
 
 use Rector\Caching\Detector\ChangedFilesDetector;
+use Rector\Configuration\Option;
+use Rector\Configuration\Parameter\SimpleParameterProvider;
 final class UnchangedFilesFilter
 {
     /**
@@ -29,6 +31,11 @@ public function filterFilePaths(array $filePaths): array
             $changedFileInfos[] = $filePath;
             $this->changedFilesDetector->invalidateFile($filePath);
         }
+        // some files were served from cache, so rules ran on a subset only - unused skip reporting
+        // would then flag every cached file's skip as falsely unused
+        if (count($changedFileInfos) < count($filePaths)) {
+            SimpleParameterProvider::setParameter(Option::IS_CACHED_RUN, \true);
+        }
         return $changedFileInfos;
     }
 }

src/Configuration/Option.php

@@ -132,6 +132,15 @@ final class Option
      * @var string
      */
     public const IS_RUN_NARROWED = 'is_run_narrowed';
+    /**
+     * True when the unchanged-files cache dropped at least one file from the run, so rules only ran
+     * on the changed subset. Unused skip reporting is then disabled, as skips on cached files never
+     * get a chance to match and would all look falsely unused.
+     *
+     * @internal
+     * @var string
+     */
+    public const IS_CACHED_RUN = 'is_cached_run';
     /**
      * @internal Use RectorConfig::fileExtensions() instead
      * @var string

src/Reporting/MissConfigurationReporter.php

@@ -43,7 +43,9 @@ public function reportUnusedSkips(ProcessResult $processResult): void
             return;
         }
         $this->symfonyStyle->warning(sprintf('%s never matched any element. You can remove %s from "->withSkip()"', count($unusedSkips) > 1 ? 'These skips are unused, they' : 'This skip is unused, it', count($unusedSkips) > 1 ? 'them' : 'it'));
-        $this->symfonyStyle->listing($unusedSkips);
+        // add a blank line between items, so grouped rule skips stay visually separated
+        $spacedUnusedSkips = array_map(static fn(string $unusedSkip): string => $unusedSkip . "\n", $unusedSkips);
+        $this->symfonyStyle->listing($spacedUnusedSkips);
     }
     public function reportSkippedNeverRegisteredRules(): void
     {

src/Reporting/UnusedSkipResolver.php

@@ -50,6 +50,11 @@ public function resolve(ProcessResult $processResult): array
         if (SimpleParameterProvider::provideBoolParameter(Option::IS_RUN_NARROWED, \false)) {
             return [];
         }
+        // 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