fix: tranformation case issue

Author: holiq

README.md

@@ -550,6 +550,9 @@ readonly class CreateUserData extends DataTransferObject
 
     /**
      * Apply data transformations
+     *
+     * Note: Transform keys support both camelCase and snake_case.
+     * Use camelCase (firstName) or snake_case (first_name) - both work!
      */
     protected static function transforms(): array
     {

src/Actions/Filesystem/FilePresentAction.php

@@ -23,7 +23,7 @@ public function execute(FilePresentData $data, bool $withForce = false): bool
         }
 
         if ($filesystem->exists(path: $path)) {
-            return throw new FileAlreadyExistException($data->fileName);
+            throw new FileAlreadyExistException($data->fileName);
         }
 
         return false;

src/Foundation/DataTransferObject.php

@@ -116,7 +116,9 @@ public function get(string $property, mixed $default = null): mixed
     }
 
     /**
-     * Convert DTO to JSON
+     * Convert DTO to snake_case JSON
+     *
+     * @throws \JsonException
      */
     public function toJson(int $options = 0): string
     {
@@ -129,6 +131,22 @@ public function toJson(int $options = 0): string
         return $json;
     }
 
+    /**
+     * Convert DTO to camelCase JSON
+     *
+     * @throws \JsonException
+     */
+    public function toCamelJson(int $options = 0): string
+    {
+        $json = json_encode($this->toCamelCase(), $options);
+
+        if ($json === false) {
+            throw new \JsonException('Failed to encode DTO to JSON');
+        }
+
+        return $json;
+    }
+
     /**
      * Convert DTO to camelCase array
      *
@@ -244,8 +262,6 @@ private function getPropertyValue(\ReflectionProperty $property): mixed
             return null;
         }
 
-        $property->setAccessible(true);
-
         return $property->getValue($this);
     }
 

src/Foundation/DataTransferObject/HasResolvable.php

@@ -3,7 +3,7 @@
 namespace Holiq\ActionData\Foundation\DataTransferObject;
 
 use CuyZ\Valinor\Mapper\MappingError;
-use CuyZ\Valinor\MapperBuilder;
+use CuyZ\Valinor\Mapper\TreeMapper;
 use Holiq\ActionData\Exceptions\InvalidArgumentException;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Foundation\Http\FormRequest;
@@ -37,7 +37,7 @@ public static function resolveFrom(mixed $abstract): static
         }
 
         throw new InvalidArgumentException(
-            'Unsupported data type for DTO resolution. Expected FormRequest, Model, or array, got: ' . get_debug_type($abstract)
+            'Unsupported data type for DTO resolution. Expected FormRequest, Model, or array, got: ' . get_debug_type($abstract),
         );
     }
 
@@ -56,13 +56,20 @@ public static function resolve(array $data): static
         $data = static::applyTransforms($data);
 
         /** @var static $instance */
-        $instance = (new MapperBuilder())
-            ->mapper()
+        $instance = static::mapper()
             ->map(signature: static::class, source: static::resolveTheArrayKeyForm(data: $data));
 
         return $instance;
     }
 
+    /**
+     * Get the mapper instance
+     */
+    protected static function mapper(): TreeMapper
+    {
+        return MapperRegistry::getMapper();
+    }
+
     /**
      * Apply data transformations
      *
@@ -77,8 +84,25 @@ protected static function applyTransforms(array $data): array
         $transforms = static::transforms();
 
         foreach ($transforms as $key => $transform) {
+            // Try exact key match first
             if (array_key_exists($key, $data)) {
                 $data[$key] = $transform($data[$key]);
+
+                continue;
+            }
+
+            // Try snake_case version of the key (for camelCase transform keys)
+            $snakeKey = Str::snake($key);
+            if ($snakeKey !== $key && array_key_exists($snakeKey, $data)) {
+                $data[$snakeKey] = $transform($data[$snakeKey]);
+
+                continue;
+            }
+
+            // Try camelCase version of the key (for snake_case transform keys)
+            $camelKey = Str::camel($key);
+            if ($camelKey !== $key && array_key_exists($camelKey, $data)) {
+                $data[$camelKey] = $transform($data[$camelKey]);
             }
         }
 

src/Foundation/DataTransferObject/MapperRegistry.php

@@ -0,0 +1,27 @@
+<?php
+
+namespace Holiq\ActionData\Foundation\DataTransferObject;
+
+use CuyZ\Valinor\Mapper\TreeMapper;
+use CuyZ\Valinor\MapperBuilder;
+
+class MapperRegistry
+{
+    private static ?TreeMapper $mapperInstance = null;
+
+    /**
+     * Get the TreeMapper instance
+     */
+    public static function getMapper(): TreeMapper
+    {
+        return self::$mapperInstance ??= (new MapperBuilder())->mapper();
+    }
+
+    /**
+     * Reset the TreeMapper instance
+     */
+    public static function resetMapper(): void
+    {
+        self::$mapperInstance = null;
+    }
+}

tests/Unit/DataTransferObjectTest.php

@@ -59,7 +59,8 @@ public function __construct(
     $data = new TestUserData(
         firstName: 'John',
         lastName: 'Doe',
-        email: 'john@example.com'
+        email: 'john@example.com',
+        password: 'secret123'
     );
 
     $array = $data->toArray();
@@ -68,7 +69,7 @@ public function __construct(
         'first_name' => 'John',
         'last_name' => 'Doe',
         'email' => 'john@example.com',
-        'password' => null,
+        'password' => 'secret123',
     ]);
 });
 
@@ -136,14 +137,20 @@ public function __construct(
     );
 
     $json = $data->toJson();
-    $decoded = json_decode($json, true);
 
-    expect($decoded)->toBe([
-        'first_name' => 'John',
-        'last_name' => 'Doe',
-        'email' => 'john@example.com',
-        'password' => null,
-    ]);
+    expect($json)->toBeJson();
+});
+
+it('can convert to camelCase JSON', function () {
+    $data = new TestUserData(
+        firstName: 'John',
+        lastName: 'Doe',
+        email: 'john@example.com'
+    );
+
+    $json = $data->toCamelJson();
+
+    expect($json)->toBeJson();
 });
 
 it('can convert to camelCase array', function () {