Automatically reset static caches in Type, Introspection and Directive when overriding standard types

src/Executor/ReferenceExecutor.php

@@ -69,6 +69,12 @@ class ReferenceExecutor implements ExecutorImplementation
      */
     protected \SplObjectStorage $fieldArgsCache;
 
+    protected FieldDefinition $schemaMetaFieldDef;
+
+    protected FieldDefinition $typeMetaFieldDef;
+
+    protected FieldDefinition $typeNameMetaFieldDef;
+
     protected function __construct(ExecutionContext $context)
     {
         if (! isset(static::$UNDEFINED)) {
@@ -701,23 +707,22 @@ protected function resolveField(
      */
     protected function getFieldDef(Schema $schema, ObjectType $parentType, string $fieldName): ?FieldDefinition
     {
-        static $schemaMetaFieldDef, $typeMetaFieldDef, $typeNameMetaFieldDef;
-        $schemaMetaFieldDef ??= Introspection::schemaMetaFieldDef();
-        $typeMetaFieldDef ??= Introspection::typeMetaFieldDef();
-        $typeNameMetaFieldDef ??= Introspection::typeNameMetaFieldDef();
+        $this->schemaMetaFieldDef ??= Introspection::schemaMetaFieldDef();
+        $this->typeMetaFieldDef ??= Introspection::typeMetaFieldDef();
+        $this->typeNameMetaFieldDef ??= Introspection::typeNameMetaFieldDef();
 
         $queryType = $schema->getQueryType();
 
-        if ($fieldName === $schemaMetaFieldDef->name && $queryType === $parentType) {
-            return $schemaMetaFieldDef;
+        if ($fieldName === $this->schemaMetaFieldDef->name && $queryType === $parentType) {
+            return $this->schemaMetaFieldDef;
         }
 
-        if ($fieldName === $typeMetaFieldDef->name && $queryType === $parentType) {
-            return $typeMetaFieldDef;
+        if ($fieldName === $this->typeMetaFieldDef->name && $queryType === $parentType) {
+            return $this->typeMetaFieldDef;
         }
 
-        if ($fieldName === $typeNameMetaFieldDef->name) {
-            return $typeNameMetaFieldDef;
+        if ($fieldName === $this->typeNameMetaFieldDef->name) {
+            return $this->typeNameMetaFieldDef;
         }
 
         return $parentType->findField($fieldName);

src/Type/Definition/Directive.php

@@ -31,9 +31,9 @@ class Directive
     /**
      * Lazily initialized.
      *
-     * @var array<string, Directive>
+     * @var array<string, Directive>|null
      */
-    protected static array $internalDirectives;
+    protected static ?array $internalDirectives = null;
 
     public string $name;
 
@@ -162,4 +162,9 @@ public static function isSpecifiedDirective(Directive $directive): bool
     {
         return \array_key_exists($directive->name, self::getInternalDirectives());
     }
+
+    public static function reset(): void
+    {
+        self::$internalDirectives = null;
+    }
 }

src/Type/Definition/Type.php

@@ -31,7 +31,10 @@ abstract class Type implements \JsonSerializable
     ];
 
     /** @var array<string, ScalarType> */
-    protected static array $standardTypes;
+    protected static ?array $standardTypes = [];
+
+    /** @var array<string, Type&NamedType>|null */
+    protected static ?array $builtInTypes = null;
 
     /**
      * @api
@@ -116,9 +119,7 @@ public static function nonNull($type): NonNull
      */
     public static function builtInTypes(): array
     {
-        static $builtInTypes;
-
-        return $builtInTypes ??= \array_merge(
+        return self::$builtInTypes ??= \array_merge(
             Introspection::getTypes(),
             self::getStandardTypes()
         );
@@ -261,4 +262,10 @@ public function jsonSerialize(): string
     {
         return $this->toString();
     }
+
+    public static function reset(): void
+    {
+        static::$standardTypes = [];
+        static::$builtInTypes = null;
+    }
 }

src/Type/Introspection.php

@@ -795,4 +795,9 @@ public static function typeNameMetaFieldDef(): FieldDefinition
             'resolve' => static fn ($source, array $args, $context, ResolveInfo $info): string => $info->parentType->name,
         ]);
     }
+
+    public static function reset(): void
+    {
+        self::$map = [];
+    }
 }

tests/ResetTest.php

@@ -0,0 +1,26 @@
+<?php declare(strict_types=1);
+
+namespace GraphQL\Tests;
+
+use GraphQL\Type\Definition\Directive;
+use GraphQL\Type\Definition\Type;
+use GraphQL\Type\Introspection;
+use PHPUnit\Framework\TestCase;
+
+final class ResetTest extends TestCase
+{
+    public function testReset(): void
+    {
+        $string = Type::string();
+        $schema = Introspection::_schema();
+        $directives = Directive::getInternalDirectives();
+
+        Type::reset();
+        Introspection::reset();
+        Directive::reset();
+
+        self::assertNotSame($string, Type::string());
+        self::assertNotSame($schema, Introspection::_schema());
+        self::assertNotSame($directives, Directive::getInternalDirectives());
+    }
+}