style: use cdn77/coding-standard

Author: simPod

composer.json

@@ -22,7 +22,7 @@
         "webonyx/graphql-php": "^15.4"
     },
     "require-dev": {
-        "doctrine/coding-standard": "^12.0",
+        "cdn77/coding-standard": "^7.4",
         "infection/infection": "^0.30.0",
         "phpstan/extension-installer": "^1.1",
         "phpstan/phpstan": "^2.0",

phpcs.xml.dist

@@ -14,9 +14,5 @@
     <file>src</file>
     <file>tests</file>
 
-    <rule ref="Doctrine">
-        <exclude name="SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingTraversableTypeHintSpecification" />
-        <exclude name="SlevomatCodingStandard.TypeHints.PropertyTypeHint.MissingTraversableTypeHintSpecification" />
-        <exclude name="SlevomatCodingStandard.TypeHints.ReturnTypeHint.MissingTraversableTypeHintSpecification" />
-    </rule>
+    <rule ref="Cdn77" />
 </ruleset>

src/Builder/ObjectBuilder.php

@@ -68,13 +68,13 @@ public function addField(FieldDefinition|array $field): self
     {
         if (is_callable($this->fields)) {
             $originalFields = $this->fields;
-            $closure        = static function () use ($field, $originalFields): array {
-                $originalFields   = $originalFields();
+            $closure = static function () use ($field, $originalFields): array {
+                $originalFields = $originalFields();
                 $originalFields[] = $field;
 
                 return $originalFields;
             };
-            $this->fields   = $closure;
+            $this->fields = $closure;
         } else {
             $this->fields[] = $field;
         }

src/Builder/TypeBuilder.php

@@ -6,7 +6,8 @@
 
 abstract class TypeBuilder
 {
-    public const VALID_NAME_PATTERN = '~^[_a-zA-Z][_a-zA-Z0-9]*$~';
+    // phpcs:ignore Cdn77.NamingConventions.ValidConstantName.ClassConstantNotUpperCase
+    public const string VALID_NAME_PATTERN = '~^[_a-zA-Z][_a-zA-Z0-9]*$~';
 
     protected string|null $description = null;
 

src/Error/FormattedError.php

@@ -10,8 +10,11 @@
 /** @deprecated Use {@see ProvidesExtensions} */
 class FormattedError extends \GraphQL\Error\FormattedError
 {
-    public static function createFromException(Throwable $exception, int $debugFlag = DebugFlag::NONE, string|null $internalErrorMessage = null): array
-    {
+    public static function createFromException(
+        Throwable $exception,
+        int $debugFlag = DebugFlag::NONE,
+        string|null $internalErrorMessage = null,
+    ): array {
         $arrayError = parent::createFromException($exception, $debugFlag, $internalErrorMessage);
 
         if ($exception instanceof \GraphQL\Error\Error && $exception->getPrevious() instanceof Error) {

src/Exception/InvalidArgument.php

@@ -21,7 +21,11 @@ private function __construct(string $message = '', int $code = 0, Throwable|null
 
     public static function invalidNameFormat(string $invalidName): self
     {
-        return new self(sprintf('Name "%s" does not match pattern "%s"', $invalidName, TypeBuilder::VALID_NAME_PATTERN));
+        return new self(sprintf(
+            'Name "%s" does not match pattern "%s"',
+            $invalidName,
+            TypeBuilder::VALID_NAME_PATTERN,
+        ));
     }
 
     public static function valueNotIso8601Compliant(mixed $invalidValue): self

tests/Builder/EnumBuilderTest.php

@@ -18,7 +18,7 @@ public function testCreate(): void
         $name = 'SomeEnum';
 
         $builder = EnumBuilder::create($name);
-        $object  = $builder
+        $object = $builder
             ->addValue('Value1', 'EnumName')
             ->addValue('Value2', null, 'Value 2 Description')
             ->addValue(0, 'Numeric', 'Value 2 Description')

tests/Builder/FieldBuilderTest.php

@@ -32,7 +32,7 @@ public function testCreate(): void
         self::assertIsCallable($field['resolve']);
 
         $resolveInfoReflection = new ReflectionClass(ResolveInfo::class);
-        $resolveInfo           = $resolveInfoReflection->newInstanceWithoutConstructor();
+        $resolveInfo = $resolveInfoReflection->newInstanceWithoutConstructor();
 
         self::assertSame('Resolver result', $field['resolve'](null, [], null, $resolveInfo));
 

tests/Builder/InputObjectBuilderTest.php

@@ -17,8 +17,8 @@ final class InputObjectBuilderTest extends TestCase
     public function testCreate(): void
     {
         $description = 'To the sichuan-style nachos add ghee, noodles, buttermilk and heated herring.';
-        $name        = 'SomeType';
-        $interface   = new class () extends InterfaceType {
+        $name = 'SomeType';
+        $interface = new class () extends InterfaceType {
             public function __construct()
             {
                 $builder = InterfaceBuilder::create('InterfaceA');
@@ -31,7 +31,7 @@ public function __construct()
         };
 
         $builder = InputObjectBuilder::create($name);
-        $object  = $builder
+        $object = $builder
             ->setDescription($description)
             ->setFields(
                 [

tests/Builder/InterfaceBuilderTest.php

@@ -18,7 +18,7 @@ final class InterfaceBuilderTest extends TestCase
 {
     public function testCreate(): void
     {
-        $name        = 'InterfaceA';
+        $name = 'InterfaceA';
         $description = 'Description';
 
         $interfaceA = new class () extends InterfaceType {