Use shipmonk/coding-standard (#307)

Co-authored-by: Claude <noreply@anthropic.com>

Author: JanTvrdik

composer.json

@@ -20,10 +20,10 @@
         "phpstan/phpstan-phpunit": "^2.0.4",
         "phpstan/phpstan-strict-rules": "^2.0.3",
         "phpunit/phpunit": "^9.6.22",
+        "shipmonk/coding-standard": "^0.1.3",
         "shipmonk/composer-dependency-analyser": "^1.8.1",
         "shipmonk/dead-code-detector": "^0.9.0",
-        "shipmonk/name-collision-detector": "^2.1.1",
-        "slevomat/coding-standard": "^8.16.0"
+        "shipmonk/name-collision-detector": "^2.1.1"
     },
     "autoload": {
         "psr-4": {

composer.lock

@@ -38,7 +38,10 @@ public function getNodeType(): string
      * @param BinaryOp $node
      * @return list<IdentifierRuleError>
      */
-    public function processNode(Node $node, Scope $scope): array
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array
     {
         if (
             !$node instanceof Greater
@@ -99,7 +102,10 @@ private function isComparable(Type $type): bool
         return true;
     }
 
-    private function isComparableTogether(Type $leftType, Type $rightType): bool
+    private function isComparableTogether(
+        Type $leftType,
+        Type $rightType
+    ): bool
     {
         $intType = new IntegerType();
         $floatType = new FloatType();
@@ -149,7 +155,10 @@ private function isComparableTogether(Type $leftType, Type $rightType): bool
     /**
      * @param Type[] $allowedTypes
      */
-    private function containsOnlyTypes(Type $checkedType, array $allowedTypes): bool
+    private function containsOnlyTypes(
+        Type $checkedType,
+        array $allowedTypes
+    ): bool
     {
         $allowedType = TypeCombinator::union(...$allowedTypes);
         return $allowedType->isSuperTypeOf($checkedType)->yes();

phpcs.xml.dist

@@ -27,7 +27,10 @@ public function getNodeType(): string
      * @param InClassNode $node
      * @return list<IdentifierRuleError>
      */
-    public function processNode(Node $node, Scope $scope): array
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array
     {
         $classReflection = $node->getClassReflection();
         $backedEnumType = $classReflection->getBackedEnumType();
@@ -55,7 +58,10 @@ public function processNode(Node $node, Scope $scope): array
         return [$error];
     }
 
-    private function hasGenericsTag(ClassReflection $classReflection, string $expectedTag): bool
+    private function hasGenericsTag(
+        ClassReflection $classReflection,
+        string $expectedTag
+    ): bool
     {
         if ($classReflection->isBackedEnum()) {
             $tags = $classReflection->getImplementsTags();

src/Rule/AllowComparingOnlyComparableTypesRule.php

@@ -29,7 +29,10 @@ class ClassSuffixNamingRule implements Rule
     /**
      * @param array<class-string, string> $superclassToSuffixMapping
      */
-    public function __construct(ReflectionProvider $reflectionProvider, array $superclassToSuffixMapping = [])
+    public function __construct(
+        ReflectionProvider $reflectionProvider,
+        array $superclassToSuffixMapping = []
+    )
     {
         foreach ($superclassToSuffixMapping as $className => $suffix) {
             if (!$reflectionProvider->hasClass($className)) {

src/Rule/BackedEnumGenericsRule.php

@@ -32,7 +32,10 @@ public function getNodeType(): string
      * @param BinaryOp $node
      * @return list<IdentifierRuleError>
      */
-    public function processNode(Node $node, Scope $scope): array
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array
     {
         if (!$node instanceof Identical && !$node instanceof NotIdentical) {
             return [];

src/Rule/ClassSuffixNamingRule.php

@@ -27,7 +27,10 @@ public function getNodeType(): string
      * @param FuncCall $node
      * @return list<IdentifierRuleError>
      */
-    public function processNode(Node $node, Scope $scope): array
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array
     {
         if (!$node->name instanceof Name) {
             return [];

src/Rule/EnforceEnumMatchRule.php

@@ -28,7 +28,10 @@ public function getNodeType(): string
      * @param ReturnStatementsNode $node
      * @return list<IdentifierRuleError>
      */
-    public function processNode(Node $node, Scope $scope): array
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array
     {
         $methodReflection = $scope->getFunction();
 

src/Rule/EnforceIteratorToArrayPreserveKeysRule.php

@@ -67,7 +67,10 @@ public function getNodeType(): string
      * @param ReturnStatementsNode $node
      * @return list<IdentifierRuleError>
      */
-    public function processNode(Node $node, Scope $scope): array
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array
     {
         if ($this->treatPhpDocTypesAsCertain === false) {
             return [];
@@ -202,7 +205,10 @@ private function getTypeOfReturnStatements(ReturnStatementsNode $node): Type
         return TypeCombinator::union(...$types);
     }
 
-    private function getPhpDocReturnType(Node $node, Scope $scope): ?Type
+    private function getPhpDocReturnType(
+        Node $node,
+        Scope $scope
+    ): ?Type
     {
         $docComment = $node->getDocComment();