[LiveComponent] Optimize LiveComponentStack::isLiveComponent

Kocal · Kocal · commit f6ea6cc18036 · 2025-06-04T23:30:44.000+02:00
Using Reflection during runtime is costly.
 Here I suggest using static caching to prevent unnecessary usages of Reflection.
diff --git a/src/LiveComponent/src/Util/LiveComponentStack.php b/src/LiveComponent/src/Util/LiveComponentStack.php
@@ -41,7 +41,9 @@ public function getCurrentLiveComponent(): ?MountedComponent
 
     private function isLiveComponent(string $classname): bool
     {
-        return [] !== (new \ReflectionClass($classname))->getAttributes(AsLiveComponent::class);
+        static $cache = [];
+
+        return $cache[$classname] ??= [] !== (new \ReflectionClass($classname))->getAttributes(AsLiveComponent::class);
     }
 
     public function getCurrentComponent(): ?MountedComponent

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,9 @@ public function getCurrentLiveComponent(): ?MountedComponent`
`41`	`41`
`42`	`42`	`private function isLiveComponent(string $classname): bool`
`43`	`43`	`{`
`44`		`- return [] !== (new \ReflectionClass($classname))->getAttributes(AsLiveComponent::class);`
	`44`	`+ static $cache = [];`
	`45`	`+`
	`46`	`+ return $cache[$classname] ??= [] !== (new \ReflectionClass($classname))->getAttributes(AsLiveComponent::class);`
`45`	`47`	`}`
`46`	`48`
`47`	`49`	`public function getCurrentComponent(): ?MountedComponent`