style: apply changes from ECS

src/Analyser/AffectedInspections.php

@@ -39,6 +39,7 @@ public function addInspection(Inspection $inspection): self
         }
 
         $this->inspections->add($inspection);
+
         return $this;
     }
 }

src/Commands/AnalyseCommand.php

@@ -28,8 +28,11 @@
 class AnalyseCommand extends Command
 {
     private const ORIGINAL_SCHEMA = 'sdl';
+
     private const COMPILED_SCHEMA = 'compiled_schema';
+
     private const INPUT = 'input';
+
     private const FORMAT = 'format';
 
     protected static $defaultName = 'analyse';

src/Configuration/Visitor.php

@@ -7,5 +7,6 @@
 class Visitor
 {
     public const ORIGINAL = 'graphlint.original_visitor';
+
     public const COMPILED = 'graphlint.compiled_visitor';
 }

src/Fixes/AddFieldFixer.php

@@ -41,12 +41,14 @@ public function fix(ProblemDescriptor $problemDescriptor): void
     public function withFieldDefinitionNode(FieldDefinitionNode $fieldDefinitionNode): self
     {
         $this->fieldDefinitionNode = $fieldDefinitionNode;
+
         return $this;
     }
 
     public function atTop(bool $top = true): self
     {
         $this->fieldAtTop = $top;
+
         return $this;
     }
 }

src/Fixes/RenameFixer.php

@@ -25,6 +25,7 @@ public function fix(ProblemDescriptor $problemDescriptor): void
     public function withName(string $name): self
     {
         $this->newName = $name;
+
         return $this;
     }
 }

src/Fixes/SuffixNameFixer.php

@@ -31,6 +31,7 @@ public function fix(ProblemDescriptor $problemDescriptor): void
     public function withSuffix(string $suffix): self
     {
         $this->suffix = $suffix;
+
         return $this;
     }
 }

src/Graphlint.php

@@ -74,6 +74,7 @@ private function dispatchEvent(EventInterface $event): void
     public function addListener(GraphlintListener $listener): self
     {
         $this->listeners[] = $listener;
+
         return $this;
     }
 }

src/Inspections/CamelCaseFieldDefinitionInspection.php

@@ -49,7 +49,7 @@ public function visitFieldDefinition(
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Fields should always be written in CamelCase.",
+            'Fields should always be written in CamelCase.',
         );
     }
 }

src/Inspections/DescriptionRequiredInspection.php

@@ -135,7 +135,7 @@ private function hasDescription(StringValueNode|null $descriptionNode): bool
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "All definition nodes must have a description",
+            'All definition nodes must have a description',
         );
     }
 }

src/Inspections/DisallowEnumInspection.php

@@ -23,7 +23,7 @@ public function visitEnumTypeDefinition(
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Enums are not allowed.",
+            'Enums are not allowed.',
         );
     }
 }

src/Inspections/IgnoreNextLineSuppressorInspection.php

@@ -26,6 +26,6 @@ public function shouldSuppress(Node $node, array $parents, Inspection $inspectio
             return false;
         }
 
-        return Str::of($comment)->trim()->is("@graphlint-ignore-next-line");
+        return Str::of($comment)->trim()->is('@graphlint-ignore-next-line');
     }
 }

src/Inspections/InputSuffixInputObjectTypeDefinitionInspection.php

@@ -43,7 +43,7 @@ public function visitInputObjectTypeDefinition(
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Input objects should always be suffixed with input.",
+            'Input objects should always be suffixed with input.',
         );
     }
 }

src/Inspections/ModelDirectiveRequiresIdFieldInspection.php

@@ -25,7 +25,7 @@ public function visitObjectTypeDefinition(
     ): void {
         $hasModelDirective = $this->listFinder->contains(
             $objectTypeDefinitionNode->directives,
-            "model",
+            'model',
         );
 
         if (! $hasModelDirective) {
@@ -34,7 +34,7 @@ public function visitObjectTypeDefinition(
 
         $hasIdField = $this->listFinder->contains(
             $objectTypeDefinitionNode->fields,
-            "id",
+            'id',
         );
 
         if ($hasIdField) {
@@ -45,15 +45,15 @@ public function visitObjectTypeDefinition(
             $objectTypeDefinitionNode,
             $this->definition()->getTitle(),
             $this->addFieldFixer->withFieldDefinitionNode(
-                Parser::fieldDefinition("id: ID!")
+                Parser::fieldDefinition('id: ID!')
             )->atTop()
         );
     }
 
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Object types with @model directive on it, must have an id field.",
+            'Object types with @model directive on it, must have an id field.',
         );
     }
 }

src/Inspections/MutationFieldArgumentNamedInputInspection.php

@@ -37,11 +37,11 @@ public function visitObjectTypeDefinition(
         // Take all the fields
         Collection::make($fields)
             // Get all arguments of the fields
-            ->flatMap(fn(FieldDefinitionNode $node) => iterator_to_array($node->arguments))
+            ->flatMap(fn (FieldDefinitionNode $node) => iterator_to_array($node->arguments))
             // Reject arguments which are named `input`
-            ->reject(fn(InputValueDefinitionNode $node) => $this->nameResolver->getName($node) === 'input')
+            ->reject(fn (InputValueDefinitionNode $node) => $this->nameResolver->getName($node) === 'input')
             // Register a problem on each of the arguments
-            ->each(fn(InputValueDefinitionNode $node) => $problemsHolder->registerProblemWithDescription(
+            ->each(fn (InputValueDefinitionNode $node) => $problemsHolder->registerProblemWithDescription(
                 $node->name,
                 $this->definition()->getTitle(),
                 $this->renameFixer->withName('input'),
@@ -51,7 +51,7 @@ public function visitObjectTypeDefinition(
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Mutation field may have one argument named input.",
+            'Mutation field may have one argument named input.',
         );
     }
 }

src/Inspections/NonNullableIdInspection.php

@@ -45,7 +45,7 @@ public function visitFieldDefinition(
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Fields named ID must be non nullable.",
+            'Fields named ID must be non nullable.',
         );
     }
 }

src/Inspections/NonNullableInsideListInspection.php

@@ -47,7 +47,7 @@ public function visitListType(
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Type inside of a list must be non nullable.",
+            'Type inside of a list must be non nullable.',
         );
     }
 }

src/Inspections/NonNullableListInspection.php

@@ -63,7 +63,7 @@ public function visitListType(
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Lists must be non nullable.",
+            'Lists must be non nullable.',
         );
     }
 

src/Inspections/PascalCaseObjectTypeDefinitionInspection.php

@@ -51,7 +51,7 @@ public function visitObjectTypeDefinition(
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Object types must be PascalCase.",
+            'Object types must be PascalCase.',
         );
     }
 }

src/Inspections/UpperSnakeCaseEnumCaseDefinitionInspection.php

@@ -51,7 +51,7 @@ public function visitEnumValueDefinition(
     public function definition(): InspectionDescription
     {
         return new InspectionDescription(
-            "Enum cases must be UPPER_CASE.",
+            'Enum cases must be UPPER_CASE.',
         );
     }
 }

src/Listeners/CheckstyleListener.php

@@ -57,7 +57,7 @@ public function afterAnalyse(AfterAnalyseEvent $event): void
                 $location = $this->compiledPath ?? $problem->getNode()->loc?->source?->name;
 
                 if ($location === null) {
-                    throw new ShouldNotHappenException("No location on node.");
+                    throw new ShouldNotHappenException('No location on node.');
                 }
 
                 /** @var DOMElement $file */

src/Listeners/ConsolePrinterListener.php

@@ -29,7 +29,7 @@ public function __construct(
 
     public function beforeAnalyse(BeforeAnalyseEvent $event): void
     {
-        $this->style->info("Analysing schema...");
+        $this->style->info('Analysing schema...');
     }
 
     public function afterAnalyse(AfterAnalyseEvent $event): void
@@ -45,14 +45,15 @@ public function afterAnalyse(AfterAnalyseEvent $event): void
 
             if ($problemCount === 0) {
                 $this->style->success("No problems found in $type schema!");
+
                 continue;
             }
 
             $this->hasErrors = true;
             $this->style->error("Found $problemCount problems in $type schema");
 
             $unfixableErrors = Collection::make($problems)
-                ->filter(fn(ProblemDescriptor $descriptor) => $descriptor->getFix() === null)
+                ->filter(fn (ProblemDescriptor $descriptor) => $descriptor->getFix() === null)
                 ->count();
             $this->style->warning("$unfixableErrors cannot be automatically fixed in $type schema");
 

src/PostFixes/NodeReplacerPostFixer.php

@@ -14,7 +14,7 @@ public function __construct(
     ) {
     }
 
-    public function visitNode(Node $node): ?Node
+    public function visitNode(Node $node): Node|null
     {
         $nodes = $this->nodeReplacerCollector->getNodesToReplace();
 

src/PostFixes/PostFixer.php

@@ -8,5 +8,5 @@
 
 abstract class PostFixer
 {
-    abstract public function visitNode(Node $node): ?Node;
+    abstract public function visitNode(Node $node): Node|null;
 }

src/ProblemDescriptor.php

@@ -12,7 +12,7 @@ class ProblemDescriptor
     public function __construct(
         private readonly Node $node,
         private readonly string|null $description,
-        private readonly ?Fixer $fix,
+        private readonly Fixer|null $fix,
     ) {
     }
 
@@ -21,7 +21,7 @@ public function getNode(): Node
         return $this->node;
     }
 
-    public function getFix(): ?Fixer
+    public function getFix(): Fixer|null
     {
         return $this->fix;
     }

src/ProblemsHolder.php

@@ -15,8 +15,7 @@ class ProblemsHolder
     private array $problems = [];
 
     public function registerProblem(
-        Node $node,
-        ?Fixer $fix = null,
+        Node $node, Fixer|null $fix = null,
     ): void {
         $this->problems[] = new ProblemDescriptor(
             $node,
@@ -27,8 +26,7 @@ public function registerProblem(
 
     public function registerProblemWithDescription(
         Node $node,
-        string $description,
-        ?Fixer $fix = null,
+        string $description, Fixer|null $fix = null,
     ): void {
         $this->problems[] = new ProblemDescriptor(
             $node,

src/Utils/NodeNameResolver.php

@@ -13,7 +13,7 @@ class NodeNameResolver
     /**
      * @param Node|NodeList<Node>|null $node
      */
-    public function getName(Node|NodeList|null $node): ?string
+    public function getName(Node|NodeList|null $node): string|null
     {
         if (! $node instanceof Node) {
             return null;

src/Utils/ProblemOutputGenerator.php

@@ -28,7 +28,7 @@ public function generate(): array
     private function title(): string
     {
         return sprintf(
-            "<bg=red;options=bold>%s  %d: %s  %s</>",
+            '<bg=red;options=bold>%s  %d: %s  %s</>',
             PHP_EOL . PHP_EOL,
             $this->index + 1,
             $this->descriptor->getDescription(),
@@ -59,7 +59,7 @@ private function startToken(): Token
         $startToken = $this->descriptor->getNode()->loc?->startToken;
 
         if ($startToken === null) {
-            throw new ShouldNotHappenException("No location on node.");
+            throw new ShouldNotHappenException('No location on node.');
         }
 
         return $startToken;
@@ -70,7 +70,7 @@ private function endToken(): Token
         $endToken = $this->descriptor->getNode()->loc?->endToken;
 
         if ($endToken === null) {
-            throw new ShouldNotHappenException("No location on node.");
+            throw new ShouldNotHappenException('No location on node.');
         }
 
         return $endToken;
@@ -81,7 +81,7 @@ private function schemaBody(): string
         $body = $this->descriptor->getNode()->loc?->source?->body;
 
         if ($body === null) {
-            throw new ShouldNotHappenException("No source on node.");
+            throw new ShouldNotHappenException('No source on node.');
         }
 
         return $body;

src/Utils/RootTypeName.php

@@ -7,5 +7,5 @@
 enum RootTypeName: string
 {
     case Query = 'Query';
-    case Mutation = "Mutation";
+    case Mutation = 'Mutation';
 }