Fixed Psalm errors (#38)

* Added support of RR 2025

* fix composer.json

* ignore Psalm override errors

* fixed psalm errors

* added Symfony polyfill

* moved Symfony polyfill

Author: okorneliuk

composer.json

@@ -45,7 +45,8 @@
         "google/protobuf": "^3.7 || ^4.0",
         "spiral/roadrunner-worker": "^3.0",
         "spiral/goridge": "^4.0",
-        "spiral/roadrunner": "^2024.3 || ^2025.1"
+        "spiral/roadrunner": "^2024.3 || ^2025.1",
+        "symfony/polyfill-php83": "*"
     },
     "require-dev": {
         "jetbrains/phpstorm-attributes": "^1.0",

src/Context.php

@@ -18,6 +18,7 @@ public function __construct(
         private array $values,
     ) {}
 
+    #[\Override]
     public function withValue(string $key, mixed $value): ContextInterface
     {
         $ctx = clone $this;
@@ -26,16 +27,19 @@ public function withValue(string $key, mixed $value): ContextInterface
         return $ctx;
     }
 
+    #[\Override]
     public function getValue(string $key, mixed $default = null): mixed
     {
         return $this->values[$key] ?? $default;
     }
 
+    #[\Override]
     public function getValues(): array
     {
         return $this->values;
     }
 
+    #[\Override]
     public function offsetExists(mixed $offset): bool
     {
         \assert(\is_string($offset), 'Offset argument must be a type of string');
@@ -49,32 +53,37 @@ public function offsetExists(mixed $offset): bool
         return isset($this->values[$offset]) || \array_key_exists($offset, $this->values);
     }
 
+    #[\Override]
     public function offsetGet(mixed $offset): mixed
     {
         \assert(\is_string($offset), 'Offset argument must be a type of string');
 
         return $this->values[$offset] ?? null;
     }
 
+    #[\Override]
     public function offsetSet(mixed $offset, mixed $value): void
     {
         \assert(\is_string($offset), 'Offset argument must be a type of string');
 
         $this->values[$offset] = $value;
     }
 
+    #[\Override]
     public function offsetUnset(mixed $offset): void
     {
         \assert(\is_string($offset), 'Offset argument must be a type of string');
 
         unset($this->values[$offset]);
     }
 
+    #[\Override]
     public function getIterator(): \Traversable
     {
         return new \ArrayIterator($this->values);
     }
 
+    #[\Override]
     public function count(): int
     {
         return \count($this->values);

src/Exception/GRPCException.php

@@ -53,6 +53,7 @@ public static function create(
     /**
      * @return Message[]
      */
+    #[\Override]
     public function getDetails(): array
     {
         return $this->details;
@@ -61,11 +62,13 @@ public function getDetails(): array
     /**
      * @param Message[] $details
      */
+    #[\Override]
     public function setDetails(array $details): void
     {
         $this->details = $details;
     }
 
+    #[\Override]
     public function addDetails(Message $message): void
     {
         $this->details[] = $message;

src/Exception/GRPCExceptionInterface.php

@@ -18,7 +18,7 @@ interface GRPCExceptionInterface extends \Throwable
      * @return StatusCodeType
      * @psalm-mutation-free
      */
-    #[\ReturnTypeWillChange]
+    #[\ReturnTypeWillChange, \Override]
     public function getCode();
 
     /**

src/Exception/NotFoundException.php

@@ -6,6 +6,9 @@
 
 use Spiral\RoadRunner\GRPC\StatusCode;
 
+/**
+ * @final
+ */
 class NotFoundException extends InvokeException
 {
     protected const CODE = StatusCode::NOT_FOUND;

src/Exception/ServiceException.php

@@ -6,6 +6,9 @@
 
 use Spiral\RoadRunner\GRPC\StatusCode;
 
+/**
+ * @final
+ */
 class ServiceException extends GRPCException
 {
     /**

src/Exception/UnauthenticatedException.php

@@ -6,6 +6,9 @@
 
 use Spiral\RoadRunner\GRPC\StatusCode;
 
+/**
+ * @final
+ */
 class UnauthenticatedException extends InvokeException
 {
     protected const CODE = StatusCode::UNAUTHENTICATED;

src/Exception/UnimplementedException.php

@@ -6,6 +6,9 @@
 
 use Spiral\RoadRunner\GRPC\StatusCode;
 
+/**
+ * @final
+ */
 class UnimplementedException extends GRPCException
 {
     protected const CODE = StatusCode::UNIMPLEMENTED;

src/Invoker.php

@@ -16,6 +16,7 @@ final class Invoker implements InvokerInterface
         'Method %s input type must be an instance of %s, ' .
         'but the input is type of %s';
 
+    #[\Override]
     public function invoke(
         ServiceInterface $service,
         Method $method,

src/ResponseHeaders.php

@@ -46,11 +46,13 @@ public function get(string $key, ?string $default = null): ?string
         return $this->headers[$key] ?? $default;
     }
 
+    #[\Override]
     public function getIterator(): \Traversable
     {
         return new \ArrayIterator($this->headers);
     }
 
+    #[\Override]
     public function count(): int
     {
         return \count($this->headers);