Update to Rector 2.7 (#199)

Co-authored-by: Kevin Pfeifer <kevin@KP-MBP16M4-2025.local>
Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

Author: 3 people

composer.json

@@ -66,7 +66,7 @@
         "phpstan": "tools/phpstan analyse",
         "stan-baseline": "tools/phpstan --generate-baseline",
         "stan-setup": "phive install",
-        "rector-setup": "cp composer.json composer.backup && composer require --dev rector/rector:\"^2.2\" && mv composer.backup composer.json",
+        "rector-setup": "cp composer.json composer.backup && composer require --dev rector/rector:\"^2.7\" && mv composer.backup composer.json",
         "rector-check": "vendor/bin/rector process --dry-run",
         "rector-fix": "vendor/bin/rector process",
         "test": "phpunit",

rector.php

@@ -1,12 +1,12 @@
 <?php
 declare(strict_types=1);
 
-use Rector\CodeQuality\Rector\If_\SimplifyIfElseToTernaryRector;
 use Rector\CodingStyle\Rector\ClassMethod\MakeInheritedMethodVisibilitySameAsParentRector;
 use Rector\Config\RectorConfig;
+use Rector\DeadCode\Rector\ClassMethod\RemoveMixedDocblockOverruledByNativeTypeRector;
 use Rector\DeadCode\Rector\ClassMethod\RemoveNullTagValueNodeRector;
 use Rector\DeadCode\Rector\ClassMethod\RemoveUselessReturnTagRector;
-use Rector\Strict\Rector\Empty_\DisallowedEmptyRuleFixerRector;
+use Rector\DeadCode\Rector\ClassMethod\RemoveUselessUnionReturnDocblockRector;
 use Rector\TypeDeclarationDocblocks\Rector\ClassMethod\DocblockReturnArrayFromDirectArrayInstanceRector;
 use Rector\ValueObject\PhpVersion;
 
@@ -17,11 +17,11 @@
         __DIR__ . '/tests',
     ])
     ->withSkip([
-        DisallowedEmptyRuleFixerRector::class,
-        SimplifyIfElseToTernaryRector::class,
         MakeInheritedMethodVisibilitySameAsParentRector::class,
         RemoveNullTagValueNodeRector::class,
         RemoveUselessReturnTagRector::class,
+        RemoveUselessUnionReturnDocblockRector::class,
+        RemoveMixedDocblockOverruledByNativeTypeRector::class,
         DocblockReturnArrayFromDirectArrayInstanceRector::class => [
             __DIR__ . '/src/Mailer/Transport/QueueTransport.php',
         ],

src/Queue/Processor.php

@@ -122,7 +122,7 @@ public function processMessage(Message $message): string|object
         $callable = $message->getCallable();
         $response = $callable($message);
         if ($response === null) {
-            $response = InteropProcessor::ACK;
+            return InteropProcessor::ACK;
         }
 
         return $response;

src/QueueManager.php

@@ -91,7 +91,9 @@ public static function setConfig(string|array $key, ?array $config = null): void
             }
 
             return;
-        } elseif (is_array($key)) {
+        }
+
+        if (is_array($key)) {
             throw new LogicException('If config is not null, key must be a string.');
         }
 

tests/test_app/src/Queue/TestCustomProcessor.php

@@ -110,7 +110,7 @@ public function processMessage(Message $message): string|object
         $callable = $message->getCallable();
         $response = $callable($message);
         if ($response === null) {
-            $response = InteropProcessor::ACK;
+            return InteropProcessor::ACK;
         }
 
         return $response;