fix: allow PresetChild to accept BlockInterface classes as $type

Author: eldomagan

packages/core/src/Data/PresetChild.php

@@ -2,6 +2,7 @@
 
 namespace Craftile\Core\Data;
 
+use Craftile\Core\Contracts\BlockInterface;
 use JsonSerializable;
 
 /**
@@ -32,18 +33,26 @@ class PresetChild implements JsonSerializable
 
     /**
      * Create a new preset child instance.
+     *
+     * @param  string|class-string<BlockInterface>|null  $type  Block type string or BlockInterface class
      */
     public function __construct(?string $type = null)
     {
         // Call build() first to allow subclass configuration
         $this->build();
 
+        if ($type !== null && class_exists($type) && is_subclass_of($type, BlockInterface::class)) {
+            $type = $type::type();
+        }
+
         // Set type with priority: constructor param > build() set value > getType()
         $this->type = $type ?? $this->type ?? $this->getType();
     }
 
     /**
      * Create a new preset child.
+     *
+     * @param  string|class-string<BlockInterface>|null  $type  Block type string or BlockInterface class
      */
     public static function make(?string $type = null): static
     {
@@ -73,9 +82,15 @@ protected function build(): void {}
 
     /**
      * Set the block type.
+     *
+     * @param  string|class-string<BlockInterface>  $type  Block type string or BlockInterface class
      */
     public function type(string $type): static
     {
+        if (class_exists($type) && is_subclass_of($type, BlockInterface::class)) {
+            $type = $type::type();
+        }
+
         $this->type = $type;
 
         return $this;

tests/core/Data/PresetChildTest.php

@@ -3,6 +3,7 @@
 declare(strict_types=1);
 
 use Craftile\Core\Data\PresetChild;
+use Tests\Laravel\Stubs\Discovery\StubTestBlock;
 
 describe('PresetChild', function () {
     it('can be created with type', function () {
@@ -11,6 +12,24 @@
         expect($block->toArray())->toHaveKey('type', 'text');
     });
 
+    it('can be created with BlockInterface class', function () {
+        $block = PresetChild::make(StubTestBlock::class);
+
+        expect($block->toArray())->toHaveKey('type', 'stub-test-block');
+    });
+
+    it('can set type using BlockInterface class via type() method', function () {
+        $block = PresetChild::make('text')->type(StubTestBlock::class);
+
+        expect($block->toArray())->toHaveKey('type', 'stub-test-block');
+    });
+
+    it('can use BlockInterface class in constructor', function () {
+        $block = new PresetChild(StubTestBlock::class);
+
+        expect($block->toArray())->toHaveKey('type', 'stub-test-block');
+    });
+
     it('can set id', function () {
         $block = PresetChild::make('text')->id('heading');