Remove unnecessary assertions and suppress warnings

This commit eliminates redundant assert statements to streamline the code and improve readability. Additionally, it uses `@phpstan-ignore-line` annotations to suppress specific warnings, enhancing compatibility with static analysis tools.

Author: koriym

src/di/Argument.php

@@ -91,7 +91,7 @@ public function getMeta(): string
     /**
      * {@inheritDoc}
      */
-    public function serialize(): ?string
+    public function serialize(): ?string // @phpstan-ignore-line
     {
         return serialize($this->__serialize());
     }

src/di/Di/Inject.php

@@ -25,7 +25,7 @@ final class Inject implements InjectInterface
     public $optional = false;
 
     /**
-     * @SuppressWarnings(PHPMD.BooleanArgumentFlag)
+     * @SuppressWarnings(PHPMD.BooleanArgumentFlag) // @phpstan-ignore-line
      */
     public function __construct(bool $optional = false)
     {

src/di/InjectionPoint.php

@@ -107,7 +107,7 @@ public function __unserialize(array $array): void
         [$this->pClass, $this->pFunction, $this->pName] = $array;
     }
 
-    public function serialize(): ?string
+    public function serialize(): ?string // @phpstan-ignore-line
     {
         return serialize($this->__serialize());
     }

src/di/MultiBinding/LazyProvider.php

@@ -31,7 +31,6 @@ public function __construct(string $class)
     public function __invoke(InjectorInterface $injector)
     {
         $provider = $injector->getInstance($this->class);
-        assert($provider instanceof ProviderInterface);
 
         return $provider->get();
     }

src/di/Name.php

@@ -150,7 +150,6 @@ private function parseName(string $name): array
             $exploded = explode('=', $keyValue);
             if (isset($exploded[1])) {
                 [$key, $value] = $exploded;
-                assert(is_string($key));
                 if (isset($key[0]) && $key[0] === '$') {
                     $key = substr($key, 1);
                 }

tests/di/ArgumentsTest.php

@@ -29,7 +29,6 @@ public function testInject(): void
         $this->assertInstanceOf(FakeTyre::class, $parameters[0]);
         $this->assertInstanceOf(FakeTyre::class, $parameters[1]);
         $param0 = $parameters[0];
-        assert(is_object($param0));
         $this->assertNotSame(spl_object_hash($param0), $parameters[1]);
     }
 

tests/di/Fake/FakePhp8Car.php

@@ -63,7 +63,7 @@ public function setQualiferMirrors(#[FakeRight] FakeMirrorInterface $rightMirror
     }
 
     #[Inject]
-    public function notQualifer(#[FakeNotQualifer] FakeMirrorInterface $rightMirror =  null): void
+    public function notQualifer(#[FakeNotQualifer] ?FakeMirrorInterface $rightMirror =  null): void
     {
     }
 

tests/di/GrapherTest.php

@@ -28,7 +28,6 @@ public function testGetInstanceWithArgs(): void
         $grapher = new Grapher(new FakeUntargetModule(), __DIR__ . '/tmp');
         $instance = $grapher->newInstanceArgs(FakeUntargetChild::class, ['1']);
         $this->assertInstanceOf(FakeUntargetChild::class, $instance);
-        assert(is_object($instance));
         assert(property_exists($instance, 'val'));
         $this->assertSame('1', $instance->val);
     }