feat: add support for PHP 8.4

.github/workflows/tests.yml

@@ -15,6 +15,7 @@ jobs:
           - "8.1"
           - "8.2"
           - "8.3"
+          - "8.4"
 
     env:
       php-extensions: ds,yaml

composer.json

@@ -15,7 +15,7 @@
         }
     ],
     "require": {
-        "php": "~8.1.0 || ~8.2.0 || ~8.3.0",
+        "php": "~8.1.0 || ~8.2.0 || ~8.3.0 || ~8.4.0",
         "composer-runtime-api": "^2.0",
         "psr/simple-cache": "^1.0 || ^2.0 || ^3.0"
     },

src/Definition/Repository/Reflection/ReflectionFunctionDefinitionRepository.php

@@ -19,6 +19,7 @@
 
 use function array_map;
 use function str_ends_with;
+use function str_starts_with;
 
 /** @internal */
 final class ReflectionFunctionDefinitionRepository implements FunctionDefinitionRepository
@@ -57,7 +58,8 @@ public function for(callable $function): FunctionDefinition
         $class = $reflection->getClosureScopeClass();
         $returnType = $returnTypeResolver->resolveReturnTypeFor($reflection);
         $nativeReturnType = $returnTypeResolver->resolveNativeReturnTypeFor($reflection);
-        $isClosure = $name === '{closure}' || str_ends_with($name, '\\{closure}');
+        // PHP8.4 Only `str_starts_with($name, '{closure:')` is needed
+        $isClosure = $name === '{closure}' || str_ends_with($name, '\\{closure}') || str_starts_with($name, '{closure:');
 
         if ($returnType instanceof UnresolvableType) {
             $returnType = $returnType->forFunctionReturnType($signature);
@@ -83,7 +85,8 @@ public function for(callable $function): FunctionDefinition
      */
     private function signature(ReflectionFunction $reflection): string
     {
-        if (str_contains($reflection->name, '{closure}')) {
+        // PHP8.4 Only `str_starts_with($name, '{closure:')` is needed
+        if ($reflection->name === '{closure}' || str_ends_with($reflection->name, '\\{closure}') || str_starts_with($reflection->name, '{closure:')) {
             $startLine = $reflection->getStartLine();
             $endLine = $reflection->getEndLine();
 

tests/Functional/Definition/Repository/Reflection/ReflectionFunctionDefinitionRepositoryTest.php

@@ -39,7 +39,7 @@ public function test_function_data_can_be_retrieved(): void
         $function = $this->repository->for($callback);
         $parameters = $function->parameters;
 
-        self::assertSame(__NAMESPACE__ . '\{closure}', $function->name);
+        self::assertSame('{closure:' . __CLASS__ . '::' . __FUNCTION__ . '():37}', $function->name);
         self::assertInstanceOf(NativeStringType::class, $function->returnType);
 
         self::assertTrue($parameters->has('foo'));

tests/Functional/Definition/Repository/Reflection/TypeResolver/ReflectionTypeResolverTest.php

@@ -98,7 +98,7 @@ public static function native_type_is_resolved_properly_data_provider(): iterabl
     }
 
     // PHP8.2 move to data provider
-    #[RequiresPhp('8.2')]
+    #[RequiresPhp('>=8.2')]
     public function test_disjunctive_normal_form_type_is_resolved_properly(): void
     {
         $reflectionType = (new ReflectionProperty(ObjectWithPropertyWithNativeDisjunctiveNormalFormType::class, 'someProperty'))->getType();
@@ -109,7 +109,7 @@ public function test_disjunctive_normal_form_type_is_resolved_properly(): void
     }
 
     // PHP8.2 move to data provider
-    #[RequiresPhp('8.2')]
+    #[RequiresPhp('>=8.2')]
     public function test_native_null_type_is_resolved_properly(): void
     {
         $reflectionType = (new ReflectionProperty(ObjectWithPropertyWithNativePhp82StandaloneTypes::class, 'nativeNull'))->getType();
@@ -120,7 +120,7 @@ public function test_native_null_type_is_resolved_properly(): void
     }
 
     // PHP8.2 move to data provider
-    #[RequiresPhp('8.2')]
+    #[RequiresPhp('>=8.2')]
     public function test_native_true_type_is_resolved_properly(): void
     {
         $reflectionType = (new ReflectionProperty(ObjectWithPropertyWithNativePhp82StandaloneTypes::class, 'nativeTrue'))->getType();
@@ -131,7 +131,7 @@ public function test_native_true_type_is_resolved_properly(): void
     }
 
     // PHP8.2 move to data provider
-    #[RequiresPhp('8.2')]
+    #[RequiresPhp('>=8.2')]
     public function test_native_false_type_is_resolved_properly(): void
     {
         $reflectionType = (new ReflectionProperty(ObjectWithPropertyWithNativePhp82StandaloneTypes::class, 'nativeFalse'))->getType();