[CodingStyle] Skip non-native array type on PHP 7.4 for ArraySpreadInsteadOfArrayMergeRector

Author: samsonasik

rules-tests/CodingStyle/Rector/FuncCall/ArraySpreadInsteadOfArrayMergeRector/FixturePhp74/skip_doblock_based_type_array_merge.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\Tests\CodingStyle\Rector\FuncCall\ArraySpreadInsteadOfArrayMergeRector\FixturePhp74;
+
+class SkipDocblockBasedTypeArrayMerge
+{
+    /**
+     * @param array<int, string> $iter1
+     * @param array<int, string> $iter2
+     */
+    public function run($iter1, $iter2)
+    {
+        $values = array_merge($iter1, $iter2);
+    }
+}

rules/CodingStyle/Rector/FuncCall/ArraySpreadInsteadOfArrayMergeRector.php

@@ -135,6 +135,11 @@ private function shouldSkipArrayForInvalidKeys(Expr $expr): bool
         $type = $this->getType($expr);
 
         if ($type->getIterableKeyType()->isInteger()->yes()) {
+            if (! $this->phpVersionProvider->isAtLeastPhpVersion(PhpVersionFeature::ARRAY_SPREAD_STRING_KEYS)) {
+                $nativeType = $this->nodeTypeResolver->getNativeType($expr);
+                return ! $nativeType->isArray()->yes();
+            }
+
             return false;
         }