[Php80] Validate ArrayItem on CurlyListNodeAnnotationToAttributeMapper (

#2814)

* [Php80] Validate ArrayItemNode and ArrayItem on CurlyListNodeAnnotationToAttributeMapper

* phpstan

* [ci-review] Rector Rectify

* final touch: no need double cast

Co-authored-by: GitHub Action <action@github.com>

packages/PhpAttribute/AnnotationToAttributeMapper/CurlyListNodeAnnotationToAttributeMapper.php

@@ -12,6 +12,7 @@
 use Rector\PhpAttribute\Contract\AnnotationToAttributeMapperInterface;
 use Rector\PhpAttribute\Enum\DocTagNodeState;
 use Symfony\Contracts\Service\Attribute\Required;
+use Webmozart\Assert\Assert;
 
 /**
  * @implements AnnotationToAttributeMapperInterface<CurlyListNode>
@@ -51,18 +52,23 @@ public function map($value): Array_
                 continue;
             }
 
-            ++$loop;
+            Assert::isInstanceOf($valueExpr, ArrayItem::class);
+
+            if (! is_numeric($arrayItemNode->key)) {
+                $arrayItems[] = $valueExpr;
+                continue;
+            }
 
-            $keyExpr = $loop !== $arrayItemNode->key && is_numeric($arrayItemNode->key)
-                    ? new LNumber((int) $arrayItemNode->key)
-                    : null;
+            ++$loop;
 
-            if ($valueExpr instanceof ArrayItem && $keyExpr instanceof LNumber) {
-                $valueExpr->key = $keyExpr;
+            $arrayItemNodeKey = (int) $arrayItemNode->key;
+            if ($loop === $arrayItemNodeKey) {
                 $arrayItems[] = $valueExpr;
-            } else {
-                $arrayItems[] = new ArrayItem($valueExpr, $keyExpr);
+                continue;
             }
+
+            $valueExpr->key = new LNumber($arrayItemNodeKey);
+            $arrayItems[] = $valueExpr;
         }
 
         return new Array_($arrayItems);