[CodeQuality] Skip native function on MoveInnerFunctionToTopLevelRector

rules-tests/CodeQuality/Rector/StmtsAwareInterface/MoveInnerFunctionToTopLevelRector/Fixture/skip_redefine_native_function.php.inc

@@ -0,0 +1,20 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\StmtsAwareInterface\MoveInnerFunctionToTopLevelRector\Fixture;
+
+function SkipRedefineNativeFunction(): void
+{
+    function strlen(): int
+    {
+        return 1;
+    }
+
+    echo strlen();
+}
+
+function bar()
+{
+    echo strlen('abc');
+}
+
+bar();

rules/CodeQuality/Rector/StmtsAwareInterface/MoveInnerFunctionToTopLevelRector.php

@@ -5,7 +5,10 @@
 namespace Rector\CodeQuality\Rector\StmtsAwareInterface;
 
 use PhpParser\Node;
+use PhpParser\Node\Name;
 use PhpParser\Node\Stmt\Function_;
+use PHPStan\Reflection\Native\NativeFunctionReflection;
+use PHPStan\Reflection\ReflectionProvider;
 use Rector\PhpParser\Enum\NodeGroup;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
@@ -16,6 +19,12 @@
  */
 final class MoveInnerFunctionToTopLevelRector extends AbstractRector
 {
+    public function __construct(
+        private readonly ReflectionProvider $reflectionProvider
+    ) {
+
+    }
+
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition(
@@ -126,6 +135,7 @@ private function extractInnerFunctions(Function_ $outerFunction, array $siblingS
      */
     private function hasSiblingFunctionOfSameName(Function_ $innerFunction, array $siblingStmts): bool
     {
+        $innerFunctionName = new Name($innerFunction->name->toString());
         foreach ($siblingStmts as $siblingStmt) {
             if (! $siblingStmt instanceof Function_) {
                 continue;
@@ -134,6 +144,13 @@ private function hasSiblingFunctionOfSameName(Function_ $innerFunction, array $s
             if ($this->nodeNameResolver->areNamesEqual($siblingStmt, $innerFunction)) {
                 return true;
             }
+
+            if ($this->reflectionProvider->hasFunction($innerFunctionName, null)) {
+                $functionReflection = $this->reflectionProvider->getFunction($innerFunctionName, null);
+                if ($functionReflection instanceof NativeFunctionReflection) {
+                    return true;
+                }
+            }
         }
 
         return false;