misc: remove unneeded internal checks and exceptions

src/Definition/Methods.php

@@ -5,7 +5,6 @@
 namespace CuyZ\Valinor\Definition;
 
 use Countable;
-use CuyZ\Valinor\Definition\Exception\MethodNotFound;
 use IteratorAggregate;
 use Traversable;
 
@@ -33,10 +32,6 @@ public function has(string $name): bool
 
     public function get(string $name): MethodDefinition
     {
-        if (! $this->has($name)) {
-            throw new MethodNotFound($name);
-        }
-
         return $this->methods[$name];
     }
 

src/Definition/Parameters.php

@@ -5,8 +5,6 @@
 namespace CuyZ\Valinor\Definition;
 
 use Countable;
-use CuyZ\Valinor\Definition\Exception\InvalidParameterIndex;
-use CuyZ\Valinor\Definition\Exception\ParameterNotFound;
 use IteratorAggregate;
 use Traversable;
 
@@ -36,10 +34,6 @@ public function has(string $name): bool
 
     public function get(string $name): ParameterDefinition
     {
-        if (! $this->has($name)) {
-            throw new ParameterNotFound($name);
-        }
-
         return $this->parameters[$name];
     }
 
@@ -48,10 +42,6 @@ public function get(string $name): ParameterDefinition
      */
     public function at(int $index): ParameterDefinition
     {
-        if ($index >= $this->count()) {
-            throw new InvalidParameterIndex($index, $this);
-        }
-
         return array_values($this->parameters)[$index];
     }
 

src/Definition/Properties.php

@@ -5,7 +5,6 @@
 namespace CuyZ\Valinor\Definition;
 
 use Countable;
-use CuyZ\Valinor\Definition\Exception\PropertyNotFound;
 use IteratorAggregate;
 use Traversable;
 
@@ -33,10 +32,6 @@ public function has(string $name): bool
 
     public function get(string $name): PropertyDefinition
     {
-        if (! $this->has($name)) {
-            throw new PropertyNotFound($name);
-        }
-
         return $this->properties[$name];
     }
 

src/Mapper/Object/MethodArguments.php

@@ -5,7 +5,6 @@
 namespace CuyZ\Valinor\Mapper\Object;
 
 use CuyZ\Valinor\Definition\Parameters;
-use CuyZ\Valinor\Mapper\Object\Exception\MissingMethodArgument;
 use IteratorAggregate;
 use Traversable;
 
@@ -29,10 +28,6 @@ public function __construct(Parameters $parameters, array $arguments)
         foreach ($parameters as $parameter) {
             $name = $parameter->name();
 
-            if (! array_key_exists($parameter->name(), $arguments) && ! $parameter->isOptional()) {
-                throw new MissingMethodArgument($parameter);
-            }
-
             if ($parameter->isVariadic()) {
                 $this->arguments = [...$this->arguments, ...array_values($arguments[$name])]; // @phpstan-ignore-line we know that the argument is iterable
             } else {

src/Mapper/Object/ReflectionObjectBuilder.php

@@ -5,9 +5,7 @@
 namespace CuyZ\Valinor\Mapper\Object;
 
 use CuyZ\Valinor\Definition\ClassDefinition;
-use CuyZ\Valinor\Mapper\Object\Exception\MissingPropertyArgument;
 
-use function array_key_exists;
 use function count;
 
 /** @internal */
@@ -26,12 +24,6 @@ public function describeArguments(): Arguments
 
     public function build(array $arguments): object
     {
-        foreach ($this->class->properties() as $property) {
-            if (! array_key_exists($property->name(), $arguments) && ! $property->hasDefaultValue()) {
-                throw new MissingPropertyArgument($property);
-            }
-        }
-
         $object = new ($this->class->name())();
 
         if (count($arguments) > 0) {

src/Mapper/Tree/Shell.php

@@ -6,14 +6,12 @@
 
 use CuyZ\Valinor\Definition\Attributes;
 use CuyZ\Valinor\Definition\AttributesContainer;
-use CuyZ\Valinor\Mapper\Tree\Exception\CannotGetParentOfRootShell;
 use CuyZ\Valinor\Mapper\Tree\Exception\NewShellTypeDoesNotMatch;
-use CuyZ\Valinor\Mapper\Tree\Exception\ShellHasNoValue;
-use CuyZ\Valinor\Mapper\Tree\Exception\UnresolvableShellType;
 use CuyZ\Valinor\Type\Type;
 use CuyZ\Valinor\Type\Types\UnresolvableType;
 
 use function array_unshift;
+use function assert;
 use function implode;
 
 /** @internal */
@@ -31,9 +29,7 @@ final class Shell
 
     private function __construct(private Type $type)
     {
-        if ($type instanceof UnresolvableType) {
-            throw new UnresolvableShellType($type);
-        }
+        assert(! $type instanceof UnresolvableType);
     }
 
     public static function root(Type $type, mixed $value): self
@@ -64,22 +60,13 @@ public function isRoot(): bool
         return ! isset($this->parent);
     }
 
-    public function parent(): self
-    {
-        if (! isset($this->parent)) {
-            throw new CannotGetParentOfRootShell();
-        }
-
-        return $this->parent;
-    }
-
     public function withType(Type $newType): self
     {
         $clone = clone $this;
         $clone->type = $newType;
 
         if (! $newType->matches($this->type)) {
-            throw new NewShellTypeDoesNotMatch($this, $newType);
+            throw new NewShellTypeDoesNotMatch($this, $newType); // @todo remove?
         }
 
         return $clone;
@@ -106,9 +93,7 @@ public function hasValue(): bool
 
     public function value(): mixed
     {
-        if (! $this->hasValue) {
-            throw new ShellHasNoValue();
-        }
+        assert($this->hasValue);
 
         return $this->value;
     }