[Feature]: Hash key generator

config/settings.php

@@ -145,6 +145,7 @@
     | Supported:
     | - \Rawilk\Settings\Support\KeyGenerators\ReadableKeyGenerator
     | - \Rawilk\Settings\Support\KeyGenerators\Md5KeyGenerator (default)
+    | - \Rawilk\Settings\Support\KeyGenerators\HashKeyGenerator
     |
     */
     'key_generator' => \Rawilk\Settings\Support\KeyGenerators\Md5KeyGenerator::class,
@@ -194,4 +195,14 @@
         \Carbon\CarbonImmutable::class,
         \Illuminate\Support\Carbon::class,
     ],
+
+    /*
+    |--------------------------------------------------------------------------
+    | Hash Algorithm
+    |--------------------------------------------------------------------------
+    |
+    | The hashing algorithm to use for the HashKeyGenerator.
+    |
+    */
+    'hash_algorithm' => 'xxh128',
 ];

src/Contracts/ContextSerializer.php

@@ -8,5 +8,5 @@
 
 interface ContextSerializer
 {
-    public function serialize(Context $context = null): string;
+    public function serialize(?Context $context = null): string;
 }

src/Contracts/KeyGenerator.php

@@ -8,7 +8,7 @@
 
 interface KeyGenerator
 {
-    public function generate(string $key, Context $context = null): string;
+    public function generate(string $key, ?Context $context = null): string;
 
     public function removeContextFromKey(string $key): string;
 

src/Drivers/DatabaseDriver.php

@@ -17,8 +17,7 @@ public function __construct(
         protected Connection $connection,
         protected string $table,
         protected ?string $teamForeignKey = null,
-    ) {
-    }
+    ) {}
 
     public function forget($key, $teamId = null): void
     {

src/Drivers/EloquentDriver.php

@@ -10,9 +10,7 @@
 
 class EloquentDriver implements Driver
 {
-    public function __construct(protected Setting $model)
-    {
-    }
+    public function __construct(protected Setting $model) {}
 
     public function forget($key, $teamId = null): void
     {

src/Drivers/Factory.php

@@ -17,11 +17,9 @@ class Factory
 
     protected array $customCreators = [];
 
-    public function __construct(protected Application $app)
-    {
-    }
+    public function __construct(protected Application $app) {}
 
-    public function driver(string $driver = null): Driver
+    public function driver(?string $driver = null): Driver
     {
         return $this->resolveDriver($driver);
     }
@@ -64,7 +62,7 @@ protected function getDriverConfig(string $driver): ?array
         return $this->app['config']["settings.drivers.{$driver}"];
     }
 
-    protected function resolveDriver(string $driver = null): Driver
+    protected function resolveDriver(?string $driver = null): Driver
     {
         $driver = $driver ?: $this->getDefaultDriver();
 

src/Events/SettingWasDeleted.php

@@ -19,6 +19,5 @@ public function __construct(
         public string $cacheKey,
         public mixed $teamId,
         public bool|Context|null $context,
-    ) {
-    }
+    ) {}
 }

src/Events/SettingWasStored.php

@@ -20,6 +20,5 @@ public function __construct(
         public mixed $value,
         public mixed $teamId,
         public bool|Context|null $context,
-    ) {
-    }
+    ) {}
 }

src/Events/SettingsFlushed.php

@@ -18,6 +18,5 @@ public function __construct(
         public bool|Collection|string $keys,
         public mixed $teamId,
         public bool|Context|null $context,
-    ) {
-    }
+    ) {}
 }

src/Settings.php

@@ -66,8 +66,7 @@ public function __construct(
         protected Driver $driver,
         protected KeyGenerator $keyGenerator,
         protected ValueSerializer $valueSerializer,
-    ) {
-    }
+    ) {}
 
     // mainly for testing purposes
     public function getDriver(): Driver
@@ -79,7 +78,7 @@ public function getDriver(): Driver
      * Pass in `false` for context when calling `all()` to only return results
      * that do not have context.
      */
-    public function context(Context|bool $context = null): self
+    public function context(Context|bool|null $context = null): self
     {
         $this->context = $context;
 

src/Support/ContextSerializers/ContextSerializer.php

@@ -9,7 +9,7 @@
 
 class ContextSerializer implements ContextSerializerContract
 {
-    public function serialize(Context $context = null): string
+    public function serialize(?Context $context = null): string
     {
         return serialize($context);
     }

src/Support/ContextSerializers/DotNotationContextSerializer.php

@@ -9,7 +9,7 @@
 
 class DotNotationContextSerializer implements ContextSerializerContract
 {
-    public function serialize(Context $context = null): string
+    public function serialize(?Context $context = null): string
     {
         if (is_null($context)) {
             return '';

src/Support/KeyGenerators/HashKeyGenerator.php

@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rawilk\Settings\Support\KeyGenerators;
+
+use Rawilk\Settings\Contracts\ContextSerializer;
+use Rawilk\Settings\Contracts\KeyGenerator as KeyGeneratorContract;
+use Rawilk\Settings\Support\Context;
+use RuntimeException;
+
+class HashKeyGenerator implements KeyGeneratorContract
+{
+    protected ContextSerializer $serializer;
+
+    public function generate(string $key, ?Context $context = null): string
+    {
+        return hash(
+            config('settings.hash_algorithm', 'xxh128'),
+            $key . $this->serializer->serialize($context),
+        );
+    }
+
+    public function removeContextFromKey(string $key): string
+    {
+        throw new RuntimeException('HashKeyGenerator does not support removing the context from the key.');
+    }
+
+    public function setContextSerializer(ContextSerializer $serializer): static
+    {
+        $this->serializer = $serializer;
+
+        return $this;
+    }
+
+    public function contextPrefix(): string
+    {
+        throw new RuntimeException('HashKeyGenerator does not support a context prefix.');
+    }
+}

src/Support/KeyGenerators/Md5KeyGenerator.php

@@ -13,7 +13,7 @@ class Md5KeyGenerator implements KeyGeneratorContract
 {
     protected ContextSerializer $serializer;
 
-    public function generate(string $key, Context $context = null): string
+    public function generate(string $key, ?Context $context = null): string
     {
         return md5($key . $this->serializer->serialize($context));
     }

src/Support/KeyGenerators/ReadableKeyGenerator.php

@@ -13,7 +13,7 @@ class ReadableKeyGenerator implements KeyGeneratorContract
 {
     protected ContextSerializer $serializer;
 
-    public function generate(string $key, Context $context = null): string
+    public function generate(string $key, ?Context $context = null): string
     {
         $key = $this->normalizeKey($key);
 

tests/Support/Drivers/CustomDriver.php

@@ -34,7 +34,5 @@ public function all($teamId = null, $keys = null): array|Arrayable
         return [];
     }
 
-    public function flush($teamId = null, $keys = null): void
-    {
-    }
+    public function flush($teamId = null, $keys = null): void {}
 }

tests/Unit/KeyGenerators/HashKeyGeneratorTest.php

@@ -0,0 +1,43 @@
+<?php
+
+declare(strict_types=1);
+
+use Rawilk\Settings\Support\Context;
+use Rawilk\Settings\Support\ContextSerializers\ContextSerializer;
+use Rawilk\Settings\Support\ContextSerializers\DotNotationContextSerializer;
+use Rawilk\Settings\Support\KeyGenerators\HashKeyGenerator;
+
+beforeEach(function () {
+    $this->keyGenerator = (new HashKeyGenerator)
+        ->setContextSerializer(new ContextSerializer);
+
+    config([
+        'settings.hash_algorithm' => 'xxh128',
+    ]);
+});
+
+it('generates a hash of a key', function () {
+    // N; is for a serialized null context object
+    expect($this->keyGenerator->generate('my-key'))->toBe(hash('xxh128', 'my-keyN;'));
+});
+
+it('generates a hash of a key and context object', function () {
+    $context = new Context([
+        'id' => 123,
+    ]);
+
+    expect($this->keyGenerator->generate('my-key', $context))
+        ->toBe(hash('xxh128', 'my-key' . serialize($context)));
+});
+
+it('works with other context serializers', function () {
+    $this->keyGenerator->setContextSerializer(new DotNotationContextSerializer);
+
+    $context = new Context([
+        'id' => 123,
+        'bool-value' => false,
+    ]);
+
+    expect($this->keyGenerator->generate('my-key', $context))
+        ->toBe(hash('xxh128', 'my-keyid:123::bool-value:0'));
+});