Fix PHPStan type annotations and code style

- Add proper type annotation for $requestData parameter
- Fix code style to match project standards

Author: artem-schander

phpstan-baseline.neon

@@ -54,24 +54,6 @@ parameters:
 			count: 1
 			path: src/Support/AliasArguments/AliasArguments.php
 
-		-
-			rawMessage: 'Method Rebing\GraphQL\Support\AliasArguments\AliasArguments::getAliasesInFields() has parameter $fields with no value type specified in iterable type array.'
-			identifier: missingType.iterableValue
-			count: 1
-			path: src/Support/AliasArguments/AliasArguments.php
-
-		-
-			rawMessage: 'Method Rebing\GraphQL\Support\AliasArguments\AliasArguments::getAliasesInFields() has parameter $prefix with no type specified.'
-			identifier: missingType.parameter
-			count: 1
-			path: src/Support/AliasArguments/AliasArguments.php
-
-		-
-			rawMessage: 'Method Rebing\GraphQL\Support\AliasArguments\AliasArguments::getAliasesInFields() return type has no value type specified in iterable type array.'
-			identifier: missingType.iterableValue
-			count: 1
-			path: src/Support/AliasArguments/AliasArguments.php
-
 		-
 			rawMessage: 'Method Rebing\GraphQL\Support\AliasArguments\ArrayKeyChange::changeKey() has parameter $segments with no value type specified in iterable type array.'
 			identifier: missingType.iterableValue

src/Support/AliasArguments/AliasArguments.php

@@ -61,14 +61,15 @@ protected function getArrayDepth(array $array): int
 
     /**
      * Get aliases from fields, only traversing fields present in request data.
-     * 
+     *
      * This prevents exponential time complexity with circular type references by only
      * exploring the actual data structure sent by the client, not all possible fields
      * in the type schema.
      *
-     * @param array $fields Type field definitions
-     * @param array|null $requestData Actual request data at this level (null for initial call)
+     * @param array<string,mixed> $fields Type field definitions
+     * @param array<string,mixed>|null $requestData Actual request data at this level (null for initial call)
      * @param string $prefix Path prefix for nested fields
+     *
      * @return array<string,string> Map of field paths to their aliases
      */
     protected function getAliasesInFields(array $fields, ?array $requestData = null, string $prefix = ''): array
@@ -83,7 +84,7 @@ protected function getAliasesInFields(array $fields, ?array $requestData = null,
         foreach ($fields as $name => $arg) {
             // KEY FIX: Skip fields not present in actual request data
             // This prevents exponential explosion with circular type references
-            if ($requestData !== null && !array_key_exists($name, $requestData)) {
+            if (null !== $requestData && !\array_key_exists($name, $requestData)) {
                 continue;
             }
 
@@ -110,6 +111,7 @@ protected function getAliasesInFields(array $fields, ?array $requestData = null,
             }
 
             $isWrappedInList = $this->isWrappedInList($type);
+
             if ($isWrappedInList) {
                 $newPrefix .= '.*';
             }
@@ -121,20 +123,20 @@ protected function getAliasesInFields(array $fields, ?array $requestData = null,
             }
 
             // Get the actual data at this field (if requestData provided)
-            $fieldData = $requestData !== null ? ($requestData[$name] ?? null) : null;
+            $fieldData = null !== $requestData ? ($requestData[$name] ?? null) : null;
 
             // If it's a list, process each item
-            if ($isWrappedInList && is_array($fieldData)) {
+            if ($isWrappedInList && \is_array($fieldData)) {
                 foreach ($fieldData as $item) {
-                    if (is_array($item)) {
+                    if (\is_array($item)) {
                         $pathAndAlias = $pathAndAlias + $this->getAliasesInFields(
                             $type->getFields(),
                             $item,
                             $newPrefix
                         );
                     }
                 }
-            } elseif ($fieldData !== null && is_array($fieldData)) {
+            } elseif (null !== $fieldData && \is_array($fieldData)) {
                 // Single object
                 $pathAndAlias = $pathAndAlias + $this->getAliasesInFields(
                     $type->getFields(),