chore(core): discovery cache improvements (#768)

Author: brendt

src/Tempest/Cache/src/CacheConfig.php

@@ -5,7 +5,7 @@
 namespace Tempest\Cache;
 
 use Psr\Cache\CacheItemPoolInterface;
-use Tempest\Core\Commands\DiscoveryGenerateCommand;
+use Tempest\Core\DiscoveryCache;
 use function Tempest\env;
 
 final class CacheConfig
@@ -62,7 +62,7 @@ private function resolveDiscoveryCacheStrategy(): DiscoveryCacheStrategy
             return $current;
         }
 
-        $original = DiscoveryCacheStrategy::make(@file_get_contents(DiscoveryGenerateCommand::CURRENT_DISCOVERY_STRATEGY));
+        $original = DiscoveryCacheStrategy::make(@file_get_contents(DiscoveryCache::CURRENT_DISCOVERY_STRATEGY));
 
         if ($current !== $original) {
             return DiscoveryCacheStrategy::INVALID;

src/Tempest/Core/src/Commands/DiscoveryGenerateCommand.php

@@ -18,8 +18,6 @@
 {
     use HasConsole;
 
-    public const string CURRENT_DISCOVERY_STRATEGY = __DIR__ . '/../.cache/current_discovery_strategy';
-
     public function __construct(
         private Kernel $kernel,
         private DiscoveryCache $discoveryCache,
@@ -45,7 +43,7 @@ public function __invoke(): void
 
         $this->generateDiscoveryCache($strategy);
 
-        $this->storeDiscoveryCacheStrategy($strategy);
+        $this->discoveryCache->storeStrategy($strategy);
     }
 
     public function clearDiscoveryCache(): void
@@ -100,17 +98,6 @@ private function resolveDiscoveryCacheStrategy(): DiscoveryCacheStrategy
         return DiscoveryCacheStrategy::make(env('DISCOVERY_CACHE'));
     }
 
-    private function storeDiscoveryCacheStrategy(DiscoveryCacheStrategy $strategy): void
-    {
-        $dir = dirname(self::CURRENT_DISCOVERY_STRATEGY);
-
-        if (! is_dir($dir)) {
-            mkdir($dir, recursive: true);
-        }
-
-        file_put_contents(self::CURRENT_DISCOVERY_STRATEGY, $strategy->value);
-    }
-
     public function resolveKernel(): Kernel
     {
         $container = new GenericContainer();

src/Tempest/Core/src/DiscoveryCache.php

@@ -14,7 +14,11 @@
 
 final class DiscoveryCache implements Cache
 {
-    use IsCache;
+    use IsCache {
+        clear as parentClear;
+    }
+
+    public const string CURRENT_DISCOVERY_STRATEGY = __DIR__ . '/.cache/current_discovery_strategy';
 
     private CacheItemPoolInterface $pool;
 
@@ -68,8 +72,26 @@ public function isValid(): bool
         return $this->cacheConfig->discoveryCache->isValid();
     }
 
+    public function clear(): void
+    {
+        $this->parentClear();
+
+        $this->storeStrategy(DiscoveryCacheStrategy::INVALID);
+    }
+
     public function getStrategy(): DiscoveryCacheStrategy
     {
         return $this->cacheConfig->discoveryCache;
     }
+
+    public function storeStrategy(DiscoveryCacheStrategy $strategy): void
+    {
+        $dir = dirname(self::CURRENT_DISCOVERY_STRATEGY);
+
+        if (! is_dir($dir)) {
+            mkdir($dir, recursive: true);
+        }
+
+        file_put_contents(self::CURRENT_DISCOVERY_STRATEGY, $strategy->value);
+    }
 }