refactor(fake): improve Faker resolver instantiation using MakeExtension

- Use MakeExtension trait in Faker for consistent resolver creation
- Replace manual resolver instantiation with make() method
- Refactor FromDefaultValue to check ignoreFromDefaultValue flag
- Remove unused MakeExtension trait from FromTypeAttributes
- Rename constructor parameter 'case' to 'notation' for clarity
- Change visibility of ignoreFromDefaultValue to protected

Author: wilcorrea

src/Core/Fake/Faker.php

@@ -19,6 +19,7 @@
 use Constructo\Support\Reflective\Notation;
 use Constructo\Support\Set;
 use Constructo\Support\Value;
+use Constructo\Testing\MakeExtension;
 use DateTime;
 use Faker\Factory;
 use Faker\Generator;
@@ -190,19 +191,21 @@
  */
 class Faker extends Engine implements Contract
 {
+    use MakeExtension;
+
     protected readonly Generator $generator;
 
     /**
      * @param array<callable|Formatter> $formatters
      * @SuppressWarnings(StaticAccess)
      */
     public function __construct(
-        Notation $case = Notation::SNAKE,
+        Notation $notation = Notation::SNAKE,
         array $formatters = [],
         private readonly ?string $locale = null,
-        private readonly bool $ignoreFromDefaultValue = false,
+        protected readonly bool $ignoreFromDefaultValue = false,
     ) {
-        parent::__construct($case, $formatters);
+        parent::__construct($notation, $formatters);
 
         $this->generator = Factory::create($this->locale($locale));
     }
@@ -245,12 +248,9 @@ public function generator(): Generator
     private function resolveParameters(array $parameters, Set $presets): Set
     {
         $values = [];
-        $fromDefaultValue = $this->ignoreFromDefaultValue
-            ? null
-            : new FromDefaultValue($this->notation, $this->formatters, $this->locale);
         foreach ($parameters as $parameter) {
             $field = $this->casedField($parameter);
-            $generated = $this->generateValue($parameter, $presets, $fromDefaultValue);
+            $generated = $this->generateValue($parameter, $presets);
 
             if ($generated === null) {
                 continue;
@@ -274,31 +274,22 @@ private function locale(?string $locale): string
     /**
      * @throws ReflectionException
      */
-    private function generateValue(
-        ReflectionParameter $parameter,
-        Set $presets,
-        ?FromDefaultValue $fromDefaultValue = null,
-    ): ?Value {
-        return (new FromDependency(
-            $this->notation,
-            $this->formatters,
-            $this->locale,
-            $this->ignoreFromDefaultValue
-        ))
-            ->then(new FromTypeDate($this->notation, $this->formatters, $this->locale))
-            ->then(new FromCollection($this->notation, $this->formatters, $this->locale))
-            ->then(new FromTypeBuiltin($this->notation, $this->formatters, $this->locale))
-            ->then(
-                new FromTypeAttributes(
-                    $this->notation,
-                    $this->formatters,
-                    $this->locale,
-                    $this->ignoreFromDefaultValue
-                )
-            )
-            ->then(new FromEnum($this->notation, $this->formatters, $this->locale))
-            ->then($fromDefaultValue)
-            ->then(new FromPreset($this->notation, $this->formatters, $this->locale))
+    private function generateValue(ReflectionParameter $parameter, Set $presets): ?Value
+    {
+        $args = [
+            'notation' => $this->notation,
+            'formatters' => $this->formatters,
+            'locale' => $this->locale,
+            'ignoreFromDefaultValue' => $this->ignoreFromDefaultValue,
+        ];
+        return ($this->make(FromDependency::class, $args))
+            ->then($this->make(FromTypeDate::class, $args))
+            ->then($this->make(FromCollection::class, $args))
+            ->then($this->make(FromTypeBuiltin::class, $args))
+            ->then($this->make(FromTypeAttributes::class, $args))
+            ->then($this->make(FromEnum::class, $args))
+            ->then($this->make(FromDefaultValue::class, $args))
+            ->then($this->make(FromPreset::class, $args))
             ->resolve($parameter, $presets);
     }
 }

src/Core/Fake/Resolver/FromCollection.php

@@ -16,7 +16,6 @@
 
 final class FromCollection extends Resolver
 {
-    use MakeExtension;
     use ManagedExtension;
 
     /**

src/Core/Fake/Resolver/FromDefaultValue.php

@@ -12,6 +12,14 @@
 final class FromDefaultValue extends Resolver
 {
     public function resolve(ReflectionParameter $parameter, Set $presets): ?Value
+    {
+        if ($this->ignoreFromDefaultValue) {
+            return parent::resolve($parameter, $presets);
+        }
+        return $this->resolveFromDefaultValue($parameter, $presets);
+    }
+
+    private function resolveFromDefaultValue(ReflectionParameter $parameter, Set $presets): ?Value
     {
         if ($parameter->isDefaultValueAvailable()) {
             return new Value($parameter->getDefaultValue());

src/Core/Fake/Resolver/FromTypeAttributes.php

@@ -13,7 +13,6 @@
 use Constructo\Support\Reflective\Definition\TypeExtended;
 use Constructo\Support\Set;
 use Constructo\Support\Value;
-use Constructo\Testing\MakeExtension;
 use Constructo\Testing\ManagedExtension;
 use ReflectionNamedType;
 use ReflectionParameter;
@@ -22,7 +21,6 @@
 
 final class FromTypeAttributes extends Resolver
 {
-    use MakeExtension;
     use ManagedExtension;
 
     /**
@@ -69,8 +67,14 @@ private function resolveByAttributes(ReflectionParameter $parameter): ?Value
     private function resolveManaged(Managed $instance): ?Value
     {
         return match ($instance->management) {
-            'id' => new Value($this->managed()->id()),
-            'timestamp' => new Value($this->managed()->now()),
+            'id' => new Value(
+                $this->managed()
+                    ->id()
+            ),
+            'timestamp' => new Value(
+                $this->managed()
+                    ->now()
+            ),
             default => null,
         };
     }

src/Core/Fake/Resolver/FromTypeBuiltin.php

@@ -7,13 +7,11 @@
 use Constructo\Core\Fake\Resolver;
 use Constructo\Support\Set;
 use Constructo\Support\Value;
-use Constructo\Testing\MakeExtension;
 use Constructo\Testing\ManagedExtension;
 use ReflectionParameter;
 
 final class FromTypeBuiltin extends Resolver
 {
-    use MakeExtension;
     use ManagedExtension;
 
     public function resolve(ReflectionParameter $parameter, Set $presets): ?Value

src/Core/Fake/Resolver/FromTypeDate.php

@@ -18,7 +18,6 @@
 
 final class FromTypeDate extends Resolver
 {
-    use MakeExtension;
     use ManagedExtension;
 
     /**