Merge pull request #5 from KaririCode-Framework/develop

Develop

Author: walmir-silva

devkit.php

@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+/**
+ * KaririCode Devkit project-level overrides.
+ *
+ * Exclude test files from cs-fixer to prevent @PHP84Migration from transforming
+ * `(new Foo())->method()` → `new Foo()->method()` — that PHP 8.4 syntax prevents
+ * pcov from correctly attributing code-coverage to the tested class.
+ */
+return [
+    'cs_fixer_finder_exclude_dirs' => ['tests'],
+];

src/Attribute/Transform.php

@@ -17,6 +17,9 @@
     /** @var list<string|array{0: string, 1: array<string, mixed>}> */
     public array $rules;
 
+    /**
+     * @param string|array{0: string, 1: array<string, mixed>} ...$rules
+     */
     public function __construct(string|array ...$rules)
     {
         $this->rules = array_values($rules);

src/Configuration/TransformerConfiguration.php

@@ -16,5 +16,6 @@
     public function __construct(
         public bool $trackTransformations = true,
         public bool $preserveOriginal = true,
-    ) {}
+    ) {
+    }
 }

src/Contract/RuleRegistry.php

@@ -14,8 +14,11 @@
 interface RuleRegistry
 {
     public function register(string $alias, TransformationRule $rule): void;
+
     public function resolve(string $alias): TransformationRule;
+
     public function has(string $alias): bool;
+
     /** @return list<string> */
     public function aliases(): array;
 }

src/Core/AttributeTransformer.php

@@ -37,9 +37,12 @@ public function transform(object $object): TransformationResult
         /** @var TransformAttributeHandler $handler */
         $handler = $this->inspector->inspect($object, $handler);
 
+        /** @var array<string, list<string|array{0: string, 1: array<string, mixed>}>> $fieldRules */
+        $fieldRules = $handler->getFieldRules();
+
         $result = $this->engine->transform(
             $handler->getProcessedPropertyValues(),
-            $handler->getFieldRules(),
+            $fieldRules,
         );
 
         $handler->setProcessedValues($result->getTransformedData());

src/Core/InMemoryRuleRegistry.php

@@ -20,6 +20,7 @@ final class InMemoryRuleRegistry implements RuleRegistry
     /** @var array<string, TransformationRule> */
     private array $rules = [];
 
+    #[\Override]
     public function register(string $alias, TransformationRule $rule): void
     {
         if (isset($this->rules[$alias])) {
@@ -28,11 +29,21 @@ public function register(string $alias, TransformationRule $rule): void
         $this->rules[$alias] = $rule;
     }
 
+    #[\Override]
     public function resolve(string $alias): TransformationRule
     {
         return $this->rules[$alias] ?? throw InvalidRuleException::unknownAlias($alias);
     }
 
-    public function has(string $alias): bool { return isset($this->rules[$alias]); }
-    public function aliases(): array { return array_keys($this->rules); }
+    #[\Override]
+    public function has(string $alias): bool
+    {
+        return isset($this->rules[$alias]);
+    }
+
+    #[\Override]
+    public function aliases(): array
+    {
+        return array_keys($this->rules);
+    }
 }

src/Core/TransformAttributeHandler.php

@@ -31,13 +31,13 @@ final class TransformAttributeHandler implements PropertyAttributeHandler, Prope
     #[\Override]
     public function handleAttribute(string $propertyName, object $attribute, mixed $value): mixed
     {
-        if (!$attribute instanceof Transform) {
+        if (! $attribute instanceof Transform) {
             return null;
         }
 
         $this->data[$propertyName] = $value;
 
-        if (!isset($this->fieldRules[$propertyName])) {
+        if (! isset($this->fieldRules[$propertyName])) {
             $this->fieldRules[$propertyName] = [];
         }
 
@@ -84,7 +84,7 @@ public function applyChanges(object $object): void
     {
         foreach ($this->processedValues as $property => $value) {
             try {
-                (new PropertyAccessor($object, $property))->setValue($value);
+                new PropertyAccessor($object, $property)->setValue($value);
             } catch (\ReflectionException) {
                 // Property doesn't exist — skip silently
             }

src/Core/TransformationContextImpl.php

@@ -15,28 +15,61 @@
  */
 final readonly class TransformationContextImpl implements TransformationContext
 {
-    /** @param array<string, mixed> $rootData @param array<string, mixed> $parameters */
+    /**
+     * @param array<string, mixed> $rootData
+     * @param array<string, mixed> $parameters
+     */
     private function __construct(
         private string $fieldName,
         private array $rootData,
         private array $parameters,
-    ) {}
+    ) {
+    }
 
+    /**
+     * @param array<string, mixed> $rootData
+     */
     public static function create(array $rootData): self
     {
         return new self('', $rootData, []);
     }
 
-    public function getFieldName(): string { return $this->fieldName; }
-    public function getRootData(): array { return $this->rootData; }
-    public function getParameter(string $key, mixed $default = null): mixed { return $this->parameters[$key] ?? $default; }
-    public function getParameters(): array { return $this->parameters; }
+    #[\Override]
+    public function getFieldName(): string
+    {
+        return $this->fieldName;
+    }
+
+    /** @return array<string, mixed> */
+    #[\Override]
+    public function getRootData(): array
+    {
+        return $this->rootData;
+    }
+
+    #[\Override]
+    public function getParameter(string $key, mixed $default = null): mixed
+    {
+        return $this->parameters[$key] ?? $default;
+    }
+
+    /** @return array<string, mixed> */
+    #[\Override]
+    public function getParameters(): array
+    {
+        return $this->parameters;
+    }
 
+    #[\Override]
     public function withField(string $field): static
     {
         return new self($field, $this->rootData, $this->parameters);
     }
 
+    /**
+     * @param array<string, mixed> $parameters
+     */
+    #[\Override]
     public function withParameters(array $parameters): static
     {
         return new self($this->fieldName, $this->rootData, [...$this->parameters, ...$parameters]);

src/Core/TransformerEngine.php

@@ -20,12 +20,13 @@
  * @author  Walmir Silva <walmir.silva@kariricode.org>
  * @since   3.1.0 ARFA 1.3
  */
-final class TransformerEngine
+final readonly class TransformerEngine
 {
     public function __construct(
-        private readonly RuleRegistry $registry,
-        private readonly ?TransformerConfiguration $configuration = null,
-    ) {}
+        private RuleRegistry $registry,
+        private ?TransformerConfiguration $configuration = null,
+    ) {
+    }
 
     /**
      * @param array<string, mixed> $data
@@ -59,34 +60,40 @@ public function transform(array $data, array $fieldRules): TransformationResult
         return $result;
     }
 
+    /** @param array<string, mixed> $data */
     private function resolveValue(array $data, string $field): mixed
     {
-        if (array_key_exists($field, $data)) {
+        if (\array_key_exists($field, $data)) {
             return $data[$field];
         }
         $segments = explode('.', $field);
         $current = $data;
         foreach ($segments as $segment) {
-            if (!is_array($current) || !array_key_exists($segment, $current)) {
+            if (! \is_array($current) || ! \array_key_exists($segment, $current)) {
                 return null;
             }
             $current = $current[$segment];
         }
+
         return $current;
     }
 
-    /** @return array{0: TransformationRule, 1: array<string, mixed>} */
+    /**
+     * @param string|array{0: string|TransformationRule, 1: array<string, mixed>}|TransformationRule $definition
+     * @return array{0: TransformationRule, 1: array<string, mixed>}
+     */
     private function resolveRule(string|array|TransformationRule $definition): array
     {
         if ($definition instanceof TransformationRule) {
             return [$definition, []];
         }
-        if (is_string($definition)) {
+        if (\is_string($definition)) {
             return [$this->registry->resolve($definition), []];
         }
         $ruleRef = $definition[0];
         $params = $definition[1] ?? [];
         $rule = $ruleRef instanceof TransformationRule ? $ruleRef : $this->registry->resolve($ruleRef);
+
         return [$rule, $params];
     }
 }

src/Event/TransformationCompletedEvent.php

@@ -15,5 +15,7 @@
  */
 final readonly class TransformationCompletedEvent
 {
-    public function __construct(public TransformationResult $result, public float $durationMs, public float $timestamp = 0) {}
+    public function __construct(public TransformationResult $result, public float $durationMs, public float $timestamp = 0)
+    {
+    }
 }