fix for mautic/mautic@bf1249b

Author: TomasVotruba

config/set/rector-preset.php

@@ -2,12 +2,12 @@
 
 declare(strict_types=1);
 
-use Rector\Utils\Rector\RemoveRefactorDuplicatedNodeInstanceCheckRector;
-use Rector\PHPUnit\CodeQuality\Rector\Class_\AddSeeTestAnnotationRector;
 use Rector\CodingStyle\Rector\PostInc\PostIncDecToPreIncDecRector;
 use Rector\Config\RectorConfig;
+use Rector\PHPUnit\CodeQuality\Rector\Class_\AddSeeTestAnnotationRector;
 use Rector\Privatization\Rector\Class_\FinalizeTestCaseClassRector;
 use Rector\TypeDeclaration\Rector\StmtsAwareInterface\DeclareStrictTypesRector;
+use Rector\Utils\Rector\RemoveRefactorDuplicatedNodeInstanceCheckRector;
 
 return static function (RectorConfig $rectorConfig): void {
     $rectorConfig->rules([

rules/DeadCode/NodeAnalyzer/CallCollectionAnalyzer.php

@@ -11,6 +11,7 @@
 use PhpParser\Node\Identifier;
 use PhpParser\Node\Name;
 use PHPStan\Type\MixedType;
+use PHPStan\Type\NeverType;
 use Rector\Enum\ObjectReference;
 use Rector\NodeNameResolver\NodeNameResolver;
 use Rector\NodeTypeResolver\NodeTypeResolver;
@@ -35,6 +36,12 @@ public function isExists(array $calls, string $classMethodName, string $classNam
 
             $callerTypeClassName = ClassNameFromObjectTypeResolver::resolve($callerType);
             if ($callerTypeClassName === null) {
+                // the caller scope is unreachable, e.g. behind mutual recursion, so the type
+                // resolves to never; the call still exists in code, keep the method to be safe
+                if ($callerType instanceof NeverType && $this->shouldSkip($call, $classMethodName)) {
+                    return true;
+                }
+
                 // handle fluent by $this->bar()->baz()->qux()
                 // that methods don't have return type
                 if ($callerType instanceof MixedType && ! $callerType->isExplicitMixed()) {