[PropertyInfo] Adds static cache to PhpStanExtractor

Author: mvhirsch

src/Symfony/Component/PropertyInfo/Extractor/PhpStanExtractor.php

@@ -21,6 +21,7 @@
 use PHPStan\PhpDocParser\Parser\PhpDocParser;
 use PHPStan\PhpDocParser\Parser\TokenIterator;
 use PHPStan\PhpDocParser\Parser\TypeParser;
+use Symfony\Component\PropertyInfo\PhpStan\NameScope;
 use Symfony\Component\PropertyInfo\PhpStan\NameScopeFactory;
 use Symfony\Component\PropertyInfo\PropertyTypeExtractorInterface;
 use Symfony\Component\PropertyInfo\Type as LegacyType;
@@ -55,6 +56,9 @@ final class PhpStanExtractor implements PropertyTypeExtractorInterface, Construc
     private array $accessorPrefixes;
     private array $arrayMutatorPrefixes;
 
+    /** @var array<string, NameScope> */
+    private array $contexts = [];
+
     /**
      * @param list<string>|null $mutatorPrefixes
      * @param list<string>|null $accessorPrefixes
@@ -86,7 +90,6 @@ public function getTypes(string $class, string $property, array $context = []):
     {
         /** @var PhpDocNode|null $docNode */
         [$docNode, $source, $prefix, $declaringClass] = $this->getDocBlock($class, $property);
-        $nameScope = $this->nameScopeFactory->create($class, $declaringClass);
         if (null === $docNode) {
             return null;
         }
@@ -120,6 +123,7 @@ public function getTypes(string $class, string $property, array $context = []):
                 continue;
             }
 
+            $nameScope ??= $this->contexts[$class.'/'.$declaringClass] ??= $this->nameScopeFactory->create($class, $declaringClass);
             foreach ($this->phpStanTypeHelper->getTypes($tagDocNode->value, $nameScope) as $type) {
                 switch ($type->getClassName()) {
                     case 'self':

src/Symfony/Component/PropertyInfo/PhpStan/NameScopeFactory.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Component\PropertyInfo\PhpStan;
 
+use phpDocumentor\Reflection\Types\Context;
 use phpDocumentor\Reflection\Types\ContextFactory;
 
 /**
@@ -20,6 +21,9 @@
  */
 final class NameScopeFactory
 {
+    /** @var array<string, Context> */
+    private array $contexts = [];
+
     public function create(string $calledClassName, ?string $declaringClassName = null): NameScope
     {
         $declaringClassName ??= $calledClassName;
@@ -60,7 +64,7 @@ private function extractFromFullClassName(\ReflectionClass $reflection): array
             }
 
             $factory = new ContextFactory();
-            $context = $factory->createForNamespace($namespace, $contents);
+            $context = $this->contexts[$namespace.$fileName] ??= $factory->createForNamespace($namespace, $contents);
 
             return [$namespace, $context->getNamespaceAliases()];
         }