feat: implement metadata support

wilcorrea · wilcorrea · commit fa1fa8d3a17e · 2025-08-04T01:03:30.000-03:00
diff --git a/.junie/guidelines.md b/.junie/guidelines.md
@@ -61,10 +61,10 @@ make lint-psalm     # Static analysis
 make test
 
 # Run specific test file (within Docker container)
-docker compose exec app vendor/bin/phpunit tests/Path/To/TestFile.php
+docker compose exec constructo vendor/bin/phpunit tests/Path/To/TestFile.php
 
 # Run tests by filter/pattern (correct approach)
-docker compose exec app vendor/bin/phpunit --filter=TestClassName
+docker compose exec constructo vendor/bin/phpunit --filter=TestClassName
 # Note: make test FILTER=TestName is NOT the correct command for filtering
 ```
 
diff --git a/src/Core/Reflect/Chain.php b/src/Core/Reflect/Chain.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Field;
+namespace Constructo\Core\Reflect;
 
 use Constructo\Support\Metadata\Schema\Field;
 use Constructo\Support\Metadata\Schema\Registry\Types;
diff --git a/src/Core/Reflect/Reflector.php b/src/Core/Reflect/Reflector.php
@@ -4,9 +4,9 @@
 
 namespace Constructo\Core\Reflect;
 
-use Constructo\Core\Reflect\Resolver\Field\ManagedChain;
-use Constructo\Core\Reflect\Resolver\Field\RequirementChain;
-use Constructo\Core\Reflect\Resolver\Field\TypeChain;
+use Constructo\Core\Reflect\Resolve\ManagedChain;
+use Constructo\Core\Reflect\Resolve\RequirementChain;
+use Constructo\Core\Reflect\Resolve\TypeChain;
 use Constructo\Factory\SchemaFactory;
 use Constructo\Support\Cache;
 use Constructo\Support\Metadata\Schema;
diff --git a/src/Core/Reflect/Resolve/ManagedChain.php b/src/Core/Reflect/Resolve/ManagedChain.php
@@ -2,8 +2,9 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Field;
+namespace Constructo\Core\Reflect\Resolve;
 
+use Constructo\Core\Reflect\Chain;
 use Constructo\Support\Metadata\Schema\Field;
 use Constructo\Support\Reflective\Attribute\Managed;
 use ReflectionParameter;
diff --git a/src/Core/Reflect/Resolve/RequirementChain.php b/src/Core/Reflect/Resolve/RequirementChain.php
@@ -2,8 +2,9 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Field;
+namespace Constructo\Core\Reflect\Resolve;
 
+use Constructo\Core\Reflect\Chain;
 use Constructo\Support\Metadata\Schema\Field;
 use Constructo\Support\Metadata\Schema\Registry\Specs;
 use ReflectionParameter;
@@ -14,7 +15,7 @@ public function __construct(
         private readonly ?Field $parent = null,
         ?Specs $specs = null,
     ) {
-        parent::__construct($specs);
+        parent::__construct();
     }
 
     public function resolve(ReflectionParameter $parameter, Field $field, array $path): void
diff --git a/src/Core/Reflect/Resolve/Type/BuiltinNamedTypeHandler.php b/src/Core/Reflect/Resolve/Type/BuiltinNamedTypeHandler.php
@@ -2,10 +2,10 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Type;
+namespace Constructo\Core\Reflect\Resolve\Type;
 
-use Constructo\Core\Reflect\Resolver\Type\Contract\NamedTypeHandler;
-use Constructo\Core\Reflect\Resolver\Type\Contract\NamedTypeResolution;
+use Constructo\Core\Reflect\Resolve\Type\Contract\NamedTypeHandler;
+use Constructo\Core\Reflect\Resolve\Type\Contract\NamedTypeResolution;
 use Constructo\Support\Metadata\Schema\Field;
 use ReflectionNamedType;
 
diff --git a/src/Core/Reflect/Resolve/Type/Contract/NamedTypeHandler.php b/src/Core/Reflect/Resolve/Type/Contract/NamedTypeHandler.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Type\Contract;
+namespace Constructo\Core\Reflect\Resolve\Type\Contract;
 
 use Constructo\Support\Metadata\Schema\Field;
 use ReflectionNamedType;
diff --git a/src/Core/Reflect/Resolve/Type/Contract/NamedTypeResolution.php b/src/Core/Reflect/Resolve/Type/Contract/NamedTypeResolution.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Type\Contract;
+namespace Constructo\Core\Reflect\Resolve\Type\Contract;
 
 enum NamedTypeResolution
 {
diff --git a/src/Core/Reflect/Resolve/Type/Contract/TypeHandler.php b/src/Core/Reflect/Resolve/Type/Contract/TypeHandler.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Type\Contract;
+namespace Constructo\Core\Reflect\Resolve\Type\Contract;
 
 use Constructo\Support\Metadata\Schema\Field;
 use Constructo\Support\Metadata\Schema\Registry\Types;
diff --git a/src/Core/Reflect/Resolve/Type/DefineAttributeTypeHandler.php b/src/Core/Reflect/Resolve/Type/DefineAttributeTypeHandler.php
@@ -2,9 +2,9 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Type;
+namespace Constructo\Core\Reflect\Resolve\Type;
 
-use Constructo\Core\Reflect\Resolver\Type\Contract\TypeHandler;
+use Constructo\Core\Reflect\Resolve\Type\Contract\TypeHandler;
 use Constructo\Support\Metadata\Schema\Field;
 use Constructo\Support\Metadata\Schema\Registry\Specs;
 use Constructo\Support\Reflective\Attribute\Define;
diff --git a/src/Core/Reflect/Resolve/Type/DependencyTypeHandler.php b/src/Core/Reflect/Resolve/Type/DependencyTypeHandler.php
@@ -2,10 +2,10 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Type;
+namespace Constructo\Core\Reflect\Resolve\Type;
 
-use Constructo\Core\Reflect\Resolver\Type\Contract\NamedTypeHandler;
-use Constructo\Core\Reflect\Resolver\Type\Contract\NamedTypeResolution;
+use Constructo\Core\Reflect\Resolve\Type\Contract\NamedTypeHandler;
+use Constructo\Core\Reflect\Resolve\Type\Contract\NamedTypeResolution;
 use Constructo\Support\Metadata\Schema\Field;
 use ReflectionNamedType;
 
diff --git a/src/Core/Reflect/Resolve/Type/EnumNamedTypeHandler.php b/src/Core/Reflect/Resolve/Type/EnumNamedTypeHandler.php
@@ -2,11 +2,11 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Type;
+namespace Constructo\Core\Reflect\Resolve\Type;
 
 use BackedEnum;
-use Constructo\Core\Reflect\Resolver\Type\Contract\NamedTypeHandler;
-use Constructo\Core\Reflect\Resolver\Type\Contract\NamedTypeResolution;
+use Constructo\Core\Reflect\Resolve\Type\Contract\NamedTypeHandler;
+use Constructo\Core\Reflect\Resolve\Type\Contract\NamedTypeResolution;
 use Constructo\Support\Metadata\Schema\Field;
 use ReflectionEnum;
 use ReflectionEnumBackedCase;
diff --git a/src/Core/Reflect/Resolve/Type/PatternAttributeTypeHandler.php b/src/Core/Reflect/Resolve/Type/PatternAttributeTypeHandler.php
@@ -2,9 +2,9 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Type;
+namespace Constructo\Core\Reflect\Resolve\Type;
 
-use Constructo\Core\Reflect\Resolver\Type\Contract\TypeHandler;
+use Constructo\Core\Reflect\Resolve\Type\Contract\TypeHandler;
 use Constructo\Support\Metadata\Schema\Field;
 use Constructo\Support\Reflective\Attribute\Pattern;
 use ReflectionAttribute;
diff --git a/src/Core/Reflect/Resolve/TypeChain.php b/src/Core/Reflect/Resolve/TypeChain.php
@@ -2,13 +2,14 @@
 
 declare(strict_types=1);
 
-namespace Constructo\Core\Reflect\Resolver\Field;
+namespace Constructo\Core\Reflect\Resolve;
 
-use Constructo\Core\Reflect\Resolver\Type\BuiltinNamedTypeHandler;
-use Constructo\Core\Reflect\Resolver\Type\DefineAttributeTypeHandler;
-use Constructo\Core\Reflect\Resolver\Type\DependencyTypeHandler;
-use Constructo\Core\Reflect\Resolver\Type\EnumNamedTypeHandler;
-use Constructo\Core\Reflect\Resolver\Type\PatternAttributeTypeHandler;
+use Constructo\Core\Reflect\Chain;
+use Constructo\Core\Reflect\Resolve\Type\BuiltinNamedTypeHandler;
+use Constructo\Core\Reflect\Resolve\Type\DefineAttributeTypeHandler;
+use Constructo\Core\Reflect\Resolve\Type\DependencyTypeHandler;
+use Constructo\Core\Reflect\Resolve\Type\EnumNamedTypeHandler;
+use Constructo\Core\Reflect\Resolve\Type\PatternAttributeTypeHandler;
 use Constructo\Support\Metadata\Schema\Field;
 use ReflectionParameter;
 
diff --git a/tests/Core/Reflect/Resolve/ManagedChainTest.php b/tests/Core/Reflect/Resolve/ManagedChainTest.php
@@ -0,0 +1,97 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Constructo\Test\Core\Reflect\Resolve;
+
+use Constructo\Core\Reflect\Resolve\ManagedChain;
+use Constructo\Factory\DefaultSpecsFactory;
+use Constructo\Support\Metadata\Schema\Field;
+use Constructo\Support\Metadata\Schema\Field\Rules;
+use Constructo\Support\Metadata\Schema\Registry\Specs;
+use Constructo\Support\Reflective\Attribute\Managed;
+use PHPUnit\Framework\TestCase;
+use ReflectionAttribute;
+use ReflectionParameter;
+
+final class ManagedChainTest extends TestCase
+{
+    private ManagedChain $chain;
+    private Specs $specs;
+
+    protected function setUp(): void
+    {
+        $specsData = [];
+
+        $specsFactory = new DefaultSpecsFactory($specsData);
+        $this->specs = $specsFactory->make();
+        $this->chain = new ManagedChain();
+    }
+
+    public function testResolveWithManagedAttribute(): void
+    {
+        $parameter = $this->createParameterWithManagedAttribute();
+        $field = new Field($this->specs, new Rules(), 'test');
+        $path = ['test'];
+
+        $this->chain->resolve($parameter, $field, $path);
+
+        $this->assertFalse($field->isAvailable());
+    }
+
+    public function testResolveWithoutManagedAttribute(): void
+    {
+        $parameter = $this->createParameterWithoutManagedAttribute();
+        $field = new Field($this->specs, new Rules(), 'test');
+        $path = ['test'];
+
+        $this->chain->resolve($parameter, $field, $path);
+
+        $this->assertTrue($field->isAvailable());
+    }
+
+    public function testResolveCallsParentWhenNoManagedAttribute(): void
+    {
+        $parameter = $this->createParameterWithoutManagedAttribute();
+        $field = new Field($this->specs, new Rules(), 'test');
+        $path = ['test'];
+
+        // Create a mock previous chain to verify it gets called
+        $mockPreviousChain = $this->createMock(ManagedChain::class);
+        $mockPreviousChain->expects($this->once())
+            ->method('resolve')
+            ->with($parameter, $field, $path);
+
+        // Set up the chain with a previous chain
+        $chain = new ManagedChain();
+        $mockPreviousChain->then($chain);
+
+        $chain->resolve($parameter, $field, $path);
+    }
+
+    private function createParameterWithManagedAttribute(): ReflectionParameter
+    {
+        $parameter = $this->createMock(ReflectionParameter::class);
+        $reflectionAttribute = $this->createMock(ReflectionAttribute::class);
+        $managedAttribute = new Managed('test-management');
+
+        $reflectionAttribute->method('newInstance')->willReturn($managedAttribute);
+
+        $parameter->method('getAttributes')
+            ->with(Managed::class)
+            ->willReturn([$reflectionAttribute]);
+
+        return $parameter;
+    }
+
+    private function createParameterWithoutManagedAttribute(): ReflectionParameter
+    {
+        $parameter = $this->createMock(ReflectionParameter::class);
+
+        $parameter->method('getAttributes')
+            ->with(Managed::class)
+            ->willReturn([]);
+
+        return $parameter;
+    }
+}
diff --git a/tests/Core/Reflect/Resolve/RequirementChainTest.php b/tests/Core/Reflect/Resolve/RequirementChainTest.php
diff --git a/tests/Core/Reflect/Resolve/Type/BuiltinNamedTypeHandlerTest.php b/tests/Core/Reflect/Resolve/Type/BuiltinNamedTypeHandlerTest.php
diff --git a/tests/Core/Reflect/Resolve/Type/DefineAttributeTypeHandlerTest.php b/tests/Core/Reflect/Resolve/Type/DefineAttributeTypeHandlerTest.php
diff --git a/tests/Core/Reflect/Resolve/Type/DependencyTypeHandlerTest.php b/tests/Core/Reflect/Resolve/Type/DependencyTypeHandlerTest.php
diff --git a/tests/Core/Reflect/Resolve/Type/EnumNamedTypeHandlerTest.php b/tests/Core/Reflect/Resolve/Type/EnumNamedTypeHandlerTest.php
diff --git a/tests/Core/Reflect/Resolve/Type/PatternAttributeTypeHandlerTest.php b/tests/Core/Reflect/Resolve/Type/PatternAttributeTypeHandlerTest.php
diff --git a/tests/Core/Reflect/Resolve/TypeChainTest.php b/tests/Core/Reflect/Resolve/TypeChainTest.php

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`
`3`	`3`	`declare(strict_types=1);`
`4`	`4`
`5`		`-namespace Constructo\Core\Reflect\Resolver\Type\Contract;`
	`5`	`+namespace Constructo\Core\Reflect\Resolve\Type\Contract;`
`6`	`6`
`7`	`7`	`enum NamedTypeResolution`
`8`	`8`	`{`