Revert "chore(deps): update dependency friendsofphp/php-cs-fixer to v…

…3.64.0 (#1605)"

This reverts commit 4cf4870.

composer.json

@@ -18,7 +18,7 @@
     "amphp/http-server": "^2.1",
     "dms/phpunit-arraysubset-asserts": "dev-master",
     "ergebnis/composer-normalize": "^2.28",
-    "friendsofphp/php-cs-fixer": "3.64.0",
+    "friendsofphp/php-cs-fixer": "3.63.2",
     "mll-lab/php-cs-fixer-config": "^5",
     "nyholm/psr7": "^1.5",
     "phpbench/phpbench": "^1.2",

src/Error/CoercionError.php

@@ -25,7 +25,7 @@ public static function make(
         string $message,
         ?array $inputPath,
         $invalidValue,
-        ?\Throwable $previous = null,
+        ?\Throwable $previous = null
     ): self {
         $instance = new static($message, null, null, [], null, $previous);
         $instance->inputPath = $inputPath;

src/Error/Error.php

@@ -87,7 +87,7 @@ public function __construct(
         ?array $path = null,
         ?\Throwable $previous = null,
         ?array $extensions = null,
-        ?array $unaliasedPath = null,
+        ?array $unaliasedPath = null
     ) {
         parent::__construct($message, 0, $previous);
 

src/Executor/ExecutionContext.php

@@ -64,7 +64,7 @@ public function __construct(
         array $variableValues,
         array $errors,
         callable $fieldResolver,
-        PromiseAdapter $promiseAdapter,
+        PromiseAdapter $promiseAdapter
     ) {
         $this->schema = $schema;
         $this->fragments = $fragments;

src/Executor/Executor.php

@@ -103,7 +103,7 @@ public static function execute(
         $contextValue = null,
         ?array $variableValues = null,
         ?string $operationName = null,
-        ?callable $fieldResolver = null,
+        ?callable $fieldResolver = null
     ): ExecutionResult {
         $promiseAdapter = new SyncPromiseAdapter();
 
@@ -143,7 +143,7 @@ public static function promiseToExecute(
         $contextValue = null,
         ?array $variableValues = null,
         ?string $operationName = null,
-        ?callable $fieldResolver = null,
+        ?callable $fieldResolver = null
     ): Promise {
         $executor = (self::$implementationFactory)(
             $promiseAdapter,

src/Executor/ReferenceExecutor.php

@@ -87,7 +87,7 @@ public static function create(
         $contextValue,
         array $variableValues,
         ?string $operationName,
-        callable $fieldResolver,
+        callable $fieldResolver
     ): ExecutorImplementation {
         $exeContext = static::buildExecutionContext(
             $schema,
@@ -141,7 +141,7 @@ protected static function buildExecutionContext(
         array $rawVariableValues,
         ?string $operationName,
         callable $fieldResolver,
-        PromiseAdapter $promiseAdapter,
+        PromiseAdapter $promiseAdapter
     ) {
         /** @var array<int, Error> $errors */
         $errors = [];
@@ -394,7 +394,7 @@ protected function collectFields(
         ObjectType $runtimeType,
         SelectionSetNode $selectionSet,
         \ArrayObject $fields,
-        \ArrayObject $visitedFragmentNames,
+        \ArrayObject $visitedFragmentNames
     ): \ArrayObject {
         $exeContext = $this->exeContext;
         foreach ($selectionSet->selections as $selection) {
@@ -606,7 +606,7 @@ protected function resolveField(
         string $responseName,
         array $path,
         array $unaliasedPath,
-        $contextValue,
+        $contextValue
     ) {
         $exeContext = $this->exeContext;
 
@@ -723,7 +723,7 @@ protected function resolveFieldValueOrError(
         callable $resolveFn,
         $rootValue,
         ResolveInfo $info,
-        $contextValue,
+        $contextValue
     ) {
         try {
             // Build a map of arguments from the field.arguments AST, using the
@@ -765,7 +765,7 @@ protected function completeValueCatchingError(
         array $path,
         array $unaliasedPath,
         $result,
-        $contextValue,
+        $contextValue
     ) {
         // Otherwise, error protection is applied, logging the error and resolving
         // a null value for this field if one is encountered.
@@ -859,7 +859,7 @@ protected function completeValue(
         array $path,
         array $unaliasedPath,
         &$result,
-        $contextValue,
+        $contextValue
     ) {
         // If result is an Error, throw a located error.
         if ($result instanceof \Throwable) {
@@ -1002,7 +1002,7 @@ protected function completeListValue(
         array $path,
         array $unaliasedPath,
         iterable &$results,
-        $contextValue,
+        $contextValue
     ) {
         $itemType = $returnType->getWrappedType();
 
@@ -1078,7 +1078,7 @@ protected function completeAbstractValue(
         array $path,
         array $unaliasedPath,
         &$result,
-        $contextValue,
+        $contextValue
     ) {
         $typeCandidate = $returnType->resolveType($result, $contextValue, $info);
 
@@ -1211,7 +1211,7 @@ protected function completeObjectValue(
         array $path,
         array $unaliasedPath,
         &$result,
-        $contextValue,
+        $contextValue
     ) {
         // If there is an isTypeOf predicate function, call it with the
         // current result. If isTypeOf returns false, then raise an error rather
@@ -1266,7 +1266,7 @@ protected function completeObjectValue(
     protected function invalidReturnTypeError(
         ObjectType $returnType,
         $result,
-        \ArrayObject $fieldNodes,
+        \ArrayObject $fieldNodes
     ): Error {
         $safeResult = Utils::printSafe($result);
 
@@ -1294,7 +1294,7 @@ protected function collectAndExecuteSubfields(
         array $path,
         array $unaliasedPath,
         &$result,
-        $contextValue,
+        $contextValue
     ) {
         $subFieldNodes = $this->collectSubFields($returnType, $fieldNodes);
 
@@ -1440,7 +1440,7 @@ protected function ensureValidRuntimeType(
         $runtimeTypeOrName,
         AbstractType $returnType,
         ResolveInfo $info,
-        &$result,
+        &$result
     ): ObjectType {
         $runtimeType = \is_string($runtimeTypeOrName)
             ? $this->exeContext->schema->getType($runtimeTypeOrName)

src/GraphQL.php

@@ -89,7 +89,7 @@ public static function executeQuery(
         ?array $variableValues = null,
         ?string $operationName = null,
         ?callable $fieldResolver = null,
-        ?array $validationRules = null,
+        ?array $validationRules = null
     ): ExecutionResult {
         $promiseAdapter = new SyncPromiseAdapter();
 
@@ -131,7 +131,7 @@ public static function promiseToExecute(
         ?array $variableValues = null,
         ?string $operationName = null,
         ?callable $fieldResolver = null,
-        ?array $validationRules = null,
+        ?array $validationRules = null
     ): Promise {
         try {
             $documentNode = $source instanceof DocumentNode

src/Server/Helper.php

@@ -247,7 +247,7 @@ protected function promiseToExecuteOperation(
         PromiseAdapter $promiseAdapter,
         ServerConfig $config,
         OperationParams $op,
-        bool $isBatch = false,
+        bool $isBatch = false
     ): Promise {
         try {
             if ($config->getSchema() === null) {
@@ -357,7 +357,7 @@ protected function resolveValidationRules(
         ServerConfig $config,
         OperationParams $params,
         DocumentNode $doc,
-        string $operationType,
+        string $operationType
     ): ?array {
         $validationRules = $config->getValidationRules();
 
@@ -379,7 +379,7 @@ protected function resolveRootValue(
         ServerConfig $config,
         OperationParams $params,
         DocumentNode $doc,
-        string $operationType,
+        string $operationType
     ) {
         $rootValue = $config->getRootValue();
 
@@ -395,7 +395,7 @@ protected function resolveContextValue(
         ServerConfig $config,
         OperationParams $params,
         DocumentNode $doc,
-        string $operationType,
+        string $operationType
     ) {
         $context = $config->getContext();
 

src/Server/StandardServer.php

@@ -139,7 +139,7 @@ public function executeRequest($parsedBody = null)
     public function processPsrRequest(
         RequestInterface $request,
         ResponseInterface $response,
-        StreamInterface $writableBodyStream,
+        StreamInterface $writableBodyStream
     ) {
         $result = $this->executePsrRequest($request);
 

src/Type/Definition/ResolveInfo.php

@@ -145,7 +145,7 @@ public function __construct(
         $rootValue,
         OperationDefinitionNode $operation,
         array $variableValues,
-        array $unaliasedPath = [],
+        array $unaliasedPath = []
     ) {
         $this->fieldDefinition = $fieldDefinition;
         $this->fieldName = $fieldDefinition->name;

src/Utils/ASTDefinitionBuilder.php

@@ -91,7 +91,7 @@ public function __construct(
         array $typeDefinitionsMap,
         array $typeExtensionsMap,
         callable $resolveType,
-        ?callable $typeConfigDecorator = null,
+        ?callable $typeConfigDecorator = null
     ) {
         $this->typeDefinitionsMap = $typeDefinitionsMap;
         $this->typeExtensionsMap = $typeExtensionsMap;

src/Utils/BreakingChangesFinder.php

@@ -90,7 +90,7 @@ public static function findBreakingChanges(Schema $oldSchema, Schema $newSchema)
      */
     public static function findRemovedTypes(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();
@@ -119,7 +119,7 @@ public static function findRemovedTypes(
      */
     public static function findTypesThatChangedKind(
         Schema $schemaA,
-        Schema $schemaB,
+        Schema $schemaB
     ): array {
         $schemaATypeMap = $schemaA->getTypeMap();
         $schemaBTypeMap = $schemaB->getTypeMap();
@@ -191,7 +191,7 @@ private static function typeKindName(NamedType $type): string
      */
     public static function findFieldsThatChangedTypeOnObjectOrInterfaceTypes(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();
@@ -238,7 +238,7 @@ public static function findFieldsThatChangedTypeOnObjectOrInterfaceTypes(
 
     private static function isChangeSafeForObjectOrInterfaceField(
         Type $oldType,
-        Type $newType,
+        Type $newType
     ): bool {
         if ($oldType instanceof NamedType) {
             return // if they're both named types, see if their names are equivalent
@@ -275,7 +275,7 @@ private static function isChangeSafeForObjectOrInterfaceField(
      */
     public static function findFieldsThatChangedTypeOnInputObjectTypes(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();
@@ -350,7 +350,7 @@ public static function findFieldsThatChangedTypeOnInputObjectTypes(
     /** @throws InvariantViolation */
     private static function isChangeSafeForInputObjectFieldOrFieldArg(
         Type $oldType,
-        Type $newType,
+        Type $newType
     ): bool {
         if ($oldType instanceof NamedType) {
             if (! $newType instanceof NamedType) {
@@ -394,7 +394,7 @@ private static function isChangeSafeForInputObjectFieldOrFieldArg(
      */
     public static function findTypesRemovedFromUnions(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();
@@ -434,7 +434,7 @@ public static function findTypesRemovedFromUnions(
      */
     public static function findValuesRemovedFromEnums(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();
@@ -476,7 +476,7 @@ public static function findValuesRemovedFromEnums(
      */
     public static function findArgChanges(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();
@@ -580,7 +580,7 @@ public static function findArgChanges(
      */
     public static function findInterfacesRemovedFromObjectTypes(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();
@@ -828,7 +828,7 @@ public static function findDangerousChanges(Schema $oldSchema, Schema $newSchema
      */
     public static function findValuesAddedToEnums(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();
@@ -865,7 +865,7 @@ public static function findValuesAddedToEnums(
      */
     public static function findInterfacesAddedToObjectTypes(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();
@@ -912,7 +912,7 @@ public static function findInterfacesAddedToObjectTypes(
      */
     public static function findTypesAddedToUnions(
         Schema $oldSchema,
-        Schema $newSchema,
+        Schema $newSchema
     ): array {
         $oldTypeMap = $oldSchema->getTypeMap();
         $newTypeMap = $newSchema->getTypeMap();

src/Utils/BuildSchema.php

@@ -72,7 +72,7 @@ class BuildSchema
     public function __construct(
         DocumentNode $ast,
         ?callable $typeConfigDecorator = null,
-        array $options = [],
+        array $options = []
     ) {
         $this->ast = $ast;
         $this->typeConfigDecorator = $typeConfigDecorator;
@@ -102,7 +102,7 @@ public function __construct(
     public static function build(
         $source,
         ?callable $typeConfigDecorator = null,
-        array $options = [],
+        array $options = []
     ): Schema {
         $doc = $source instanceof DocumentNode
             ? $source
@@ -135,7 +135,7 @@ public static function build(
     public static function buildAST(
         DocumentNode $ast,
         ?callable $typeConfigDecorator = null,
-        array $options = [],
+        array $options = []
     ): Schema {
         return (new self($ast, $typeConfigDecorator, $options))->buildSchema();
     }