[fix] Cacher/EnumDefaultRequiredValueException

Author: ReallifeKip

CHANGELOG.md

@@ -1,5 +1,14 @@
 # CHANGELOG
 
+## [v4.5.1] - 2026-05-29
+
+### Fixed
+
+- **`ib-cacher` discarded Enum case default values, causing `RequiredValueException` at runtime.**
+  `Cacher::containsNonSerializable()` treated any object as non-serializable via a bare `is_object()` check, including Enum cases. Defaults declared in `defaultValues()` as Enum cases were silently replaced with `null` in the generated cache. At runtime there is no fallback to re-invoke `defaultValues()`, so any property with `allowsNull = false` and an Enum default would throw `RequiredValueException` whenever a cache file was present — even though the same call succeeded without a cache.
+
+  Fixed by checking `instanceof \UnitEnum` before `is_object()`. PHP 8.1+ `var_export()` serializes all Enum cases (backed and pure) as `\ClassName::CaseName`, which is a valid PHP expression that reconstructs the correct case on `require`.
+
 ## [v4.5.0] - 2026-05-28
 
 ### Added

src/CLI/Cacher.php

@@ -217,6 +217,9 @@ private static function defaultValueValidate(string $classname, array $defaults,
      */
     private static function containsNonSerializable(mixed $value): bool
     {
+        if ($value instanceof \UnitEnum) {
+            return false;
+        }
         if (\is_object($value)) {
             return true;
         }

tests/Bugs/MinimalDTO.php

@@ -0,0 +1,17 @@
+<?php
+
+namespace Tests\Bugs;
+
+use ReallifeKip\ImmutableBase\Objects\DataTransferObject;
+
+readonly class MinimalDTO extends DataTransferObject
+{
+    public StatusEnum $status;
+
+    public static function defaultValues(): array
+    {
+        return [
+            'status' => StatusEnum::Active,
+        ];
+    }
+}

tests/Bugs/StatusEnum.php

@@ -0,0 +1,8 @@
+<?php
+
+namespace Tests\Bugs;
+
+enum StatusEnum: string
+{
+    case Active = 'active';
+}

tests/CLI/CacherTest.php

@@ -236,6 +236,52 @@ public function testLoadCacheSkipsWhenFileNotExists(): void
         $this->assertEmpty(ImmutableBase::state()['cachedMeta']);
     }
 
+    public function testEnumCaseDefaultIsPreservedInCache(): void
+    {
+        $initialLevel = ob_get_level();
+        try {
+            ob_start();
+            (new Cacher())->scan($this->scanDir);
+        } finally {
+            while (ob_get_level() > $initialLevel) {
+                ob_end_clean();
+            }
+        }
+
+        $cache         = require $this->cachePath;
+        $statusDefault = $cache[\Tests\Bugs\MinimalDTO::class]['types']['status']['defaults'] ?? 'missing';
+
+        $this->assertInstanceOf(
+            \Tests\Bugs\StatusEnum::class,
+            $statusDefault,
+            'Enum case default must be preserved in cache, not replaced with null'
+        );
+        $this->assertSame(\Tests\Bugs\StatusEnum::Active, $statusDefault);
+    }
+
+    public function testEnumCaseDefaultWorksAfterLoadingCache(): void
+    {
+        $initialLevel = ob_get_level();
+        try {
+            ob_start();
+            (new Cacher())->scan($this->scanDir);
+        } finally {
+            while (ob_get_level() > $initialLevel) {
+                ob_end_clean();
+            }
+        }
+
+        $s               = &ImmutableBase::state();
+        $s['cachedMeta'] = [];
+        $s['cachePath']  = null;
+        $s['properties'] = [];
+        $s['refs']       = [];
+        ImmutableBase::loadCache();
+
+        $dto = \Tests\Bugs\MinimalDTO::fromArray([]);
+        $this->assertSame(\Tests\Bugs\StatusEnum::Active, $dto->status);
+    }
+
     public function testCachedMetaRestoredDuringBuild(): void
     {
         $initialLevel = ob_get_level();