Update PHP-CS-Fixer to use PHP 8.0 migration rules

Changed from @PHP71Migration:risky to @PHP80Migration:risky to enable
PHP 8.0+ coding standards and modernizations.

This automatically converts traditional closures to arrow functions
where applicable, resulting in cleaner, more concise code.

Changes applied:
- Updated .php-cs-fixer.dist.php configuration
- Converted 7 closures to arrow functions (fn)
- Removed outdated PHP 7.3 compatibility comment

Benefits:
- 20 lines of code removed
- More modern PHP 8.0+ syntax
- Better alignment with minimum PHP version requirement

Author: claude

.php-cs-fixer.dist.php

@@ -12,7 +12,7 @@
         '@PER-CS' => true,
         '@Symfony' => true,
         '@Symfony:risky' => true,
-        '@PHP71Migration:risky' => true,
+        '@PHP80Migration:risky' => true,
         '@PSR2' => true,
         '@DoctrineAnnotation' => true,
         'array_syntax' => ['syntax' => 'short'],
@@ -24,7 +24,7 @@
         'modernize_types_casting' => true, // Replaces intval, floatval, doubleval, strval and boolval function calls with according type casting operator.
         'multiline_whitespace_before_semicolons' => true, // Forbid multi-line whitespace before the closing semicolon or move the semicolon to the new line for chained calls.
         'no_unreachable_default_argument_value' => true, // In function arguments there must not be arguments with default values before non-default ones.
-        'no_superfluous_phpdoc_tags' => ['allow_mixed' => true], // To avoid problems of compatibility with the old php-cs-fixer version used on PHP 7.3
+        'no_superfluous_phpdoc_tags' => ['allow_mixed' => true],
         'no_useless_else' => true,
         'no_useless_return' => true,
         'ordered_class_elements' => true, // Orders the elements of classes/interfaces/traits.

src/Analyzer/ClassDescription.php

@@ -196,9 +196,7 @@ public function hasAttribute(string $pattern): bool
     {
         return array_reduce(
             $this->attributes,
-            static function (bool $carry, FullyQualifiedClassName $attribute) use ($pattern): bool {
-                return $carry || $attribute->matches($pattern);
-            },
+            static fn (bool $carry, FullyQualifiedClassName $attribute): bool => $carry || $attribute->matches($pattern),
             false
         );
     }

src/Expression/ForClasses/Implement.php

@@ -40,9 +40,7 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $interface = $this->interface;
         $interfaces = $theClass->getInterfaces();
-        $implements = function (FullyQualifiedClassName $FQCN) use ($interface): bool {
-            return $FQCN->matches($interface);
-        };
+        $implements = fn (FullyQualifiedClassName $FQCN): bool => $FQCN->matches($interface);
 
         if (0 === \count(array_filter($interfaces, $implements))) {
             $violation = Violation::create(

src/Expression/ForClasses/NotHaveDependencyOutsideNamespace.php

@@ -36,9 +36,7 @@ public function describe(ClassDescription $theClass, string $because): Descripti
     public function evaluate(ClassDescription $theClass, Violations $violations, string $because): void
     {
         $namespace = $this->namespace;
-        $depends = function (ClassDependency $dependency) use ($namespace): bool {
-            return !$dependency->getFQCN()->matches($namespace);
-        };
+        $depends = fn (ClassDependency $dependency): bool => !$dependency->getFQCN()->matches($namespace);
 
         $dependencies = $theClass->getDependencies();
         $externalDeps = array_filter($dependencies, $depends);

src/Expression/ForClasses/NotImplement.php

@@ -40,9 +40,7 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $interface = $this->interface;
         $interfaces = $theClass->getInterfaces();
-        $implements = function (FullyQualifiedClassName $FQCN) use ($interface): bool {
-            return $FQCN->matches($interface);
-        };
+        $implements = fn (FullyQualifiedClassName $FQCN): bool => $FQCN->matches($interface);
 
         if (\count(array_filter($interfaces, $implements)) > 0) {
             $violation = Violation::create(

src/RuleBuilders/Architecture/Architecture.php

@@ -90,9 +90,7 @@ public function rules(string $because = 'of component architecture'): iterable
                 $forbiddenComponents = array_diff($layerNames, [$name], $this->allowedDependencies[$name]);
 
                 if (!empty($forbiddenComponents)) {
-                    $forbiddenSelectors = array_values(array_map(function (string $componentName): string {
-                        return $this->componentSelectors[$componentName];
-                    }, $forbiddenComponents));
+                    $forbiddenSelectors = array_values(array_map(fn (string $componentName): string => $this->componentSelectors[$componentName], $forbiddenComponents));
 
                     yield Rule::allClasses()
                         ->that(new ResideInOneOfTheseNamespaces($selector))
@@ -105,9 +103,7 @@ public function rules(string $because = 'of component architecture'): iterable
                 continue;
             }
 
-            $allowedDependencies = array_values(array_map(function (string $componentName): string {
-                return $this->componentSelectors[$componentName];
-            }, $this->componentDependsOnlyOnTheseNamespaces[$name]));
+            $allowedDependencies = array_values(array_map(fn (string $componentName): string => $this->componentSelectors[$componentName], $this->componentDependsOnlyOnTheseNamespaces[$name]));
 
             yield Rule::allClasses()
                 ->that(new ResideInOneOfTheseNamespaces($selector))

src/Rules/Violations.php

@@ -27,9 +27,7 @@ public static function fromJson(string $json): self
 
         $instance = new self();
 
-        $instance->violations = array_map(function (array $json): Violation {
-            return Violation::fromJson($json);
-        }, $json['violations']);
+        $instance->violations = array_map(fn (array $json): Violation => Violation::fromJson($json), $json['violations']);
 
         return $instance;
     }
@@ -113,9 +111,7 @@ public function remove(self $violations, bool $ignoreBaselineLinenumbers = false
 
     public function sort(): void
     {
-        usort($this->violations, static function (Violation $v1, Violation $v2): int {
-            return $v1 <=> $v2;
-        });
+        usort($this->violations, static fn (Violation $v1, Violation $v2): int => $v1 <=> $v2);
     }
 
     public function jsonSerialize(): array

tests/Unit/ClassSetTest.php

@@ -54,9 +54,7 @@ public function test_can_exclude_files_or_directories(): void
             'View/UserView.php',
         ];
 
-        $actual = array_values(array_map(function ($item) {
-            return $item->getRelativePathname();
-        }, iterator_to_array($set)));
+        $actual = array_values(array_map(fn ($item) => $item->getRelativePathname(), iterator_to_array($set)));
 
         self::assertEquals($expected, $actual);
     }
@@ -83,9 +81,7 @@ public function test_can_exclude_glob_patterns(): void
             'View/UserView.php',
         ];
 
-        $actual = array_values(array_map(function ($item) {
-            return $item->getRelativePathname();
-        }, iterator_to_array($set)));
+        $actual = array_values(array_map(fn ($item) => $item->getRelativePathname(), iterator_to_array($set)));
 
         self::assertEquals($expected, $actual);
     }