Declare missing types

jeromegamez · jeromegamez · commit 006fcbefbceb · 2024-08-19T18:47:24.000+02:00
diff --git a/rector.php b/rector.php
@@ -14,6 +14,7 @@
     ->withPhpSets()
     ->withPreparedSets(
         deadCode: true,
+        typeDeclarations: true,
         privatization: true,
         earlyReturn: true,
         phpunitCodeQuality: true,
diff --git a/src/Firebase/Auth.php b/src/Firebase/Auth.php
@@ -88,7 +88,7 @@ public function getUser(Stringable|string $uid): UserRecord
 
     public function getUsers(array $uids): array
     {
-        $uids = array_map(static fn($uid) => Uid::fromString($uid)->value, $uids);
+        $uids = array_map(static fn($uid): string => Uid::fromString($uid)->value, $uids);
 
         $users = array_fill_keys($uids, null);
 
@@ -490,7 +490,7 @@ public function unlinkProvider($uid, $provider): UserRecord
         $provider = array_values(
             array_filter(
                 array_map('strval', (array) $provider),
-                static fn(string $value) => $value !== '',
+                static fn(string $value): bool => $value !== '',
             ),
         );
 
diff --git a/src/Firebase/Auth/ActionCodeSettings/ValidatedActionCodeSettings.php b/src/Firebase/Auth/ActionCodeSettings/ValidatedActionCodeSettings.php
@@ -52,7 +52,7 @@ public static function fromArray(array $settings): self
     {
         $instance = new self();
 
-        $settings = array_filter($settings, static fn($value) => $value !== null);
+        $settings = array_filter($settings, static fn($value): bool => $value !== null);
 
         foreach ($settings as $key => $value) {
             switch (mb_strtolower($key)) {
@@ -116,6 +116,6 @@ public function toArray(): array
             'androidMinimumVersion' => $this->androidMinimumVersion,
             'androidInstallApp' => $this->androidInstallApp,
             'iOSBundleId' => $this->iOSBundleId,
-        ], static fn($value) => is_bool($value) || (is_string($value) && $value !== ''));
+        ], static fn($value): bool => is_bool($value) || (is_string($value) && $value !== ''));
     }
 }
diff --git a/src/Firebase/Auth/UserRecord.php b/src/Firebase/Auth/UserRecord.php
@@ -144,7 +144,7 @@ private static function mfaInfoFromResponseData(array $data): ?MfaInfo
     private static function userInfoFromResponseData(array $data): array
     {
         return array_map(
-            static fn(array $userInfoData) => UserInfo::fromResponseData($userInfoData),
+            static fn(array $userInfoData): UserInfo => UserInfo::fromResponseData($userInfoData),
             $data['providerUserInfo'],
         );
     }
diff --git a/src/Firebase/Database/Query/Sorter/OrderByChild.php b/src/Firebase/Database/Query/Sorter/OrderByChild.php
@@ -38,7 +38,7 @@ public function modifyValue(mixed $value): mixed
 
         $expression = str_replace('/', '.', $this->childKey);
 
-        uasort($value, static fn($a, $b) => search($expression, $a) <=> search($expression, $b));
+        uasort($value, static fn($a, $b): int => search($expression, $a) <=> search($expression, $b));
 
         return $value;
     }
diff --git a/src/Firebase/DynamicLink/EventStatistics.php b/src/Firebase/DynamicLink/EventStatistics.php
@@ -103,12 +103,12 @@ public function appReOpens(): self
 
     public function filterByType(string $type): self
     {
-        return $this->filter(static fn(array $event) => ($event['event'] ?? null) === $type);
+        return $this->filter(static fn(array $event): bool => ($event['event'] ?? null) === $type);
     }
 
     public function filterByPlatform(string $platform): self
     {
-        return $this->filter(static fn(array $event) => ($event['platform'] ?? null) === $platform);
+        return $this->filter(static fn(array $event): bool => ($event['platform'] ?? null) === $platform);
     }
 
     public function filter(callable $filter): self
diff --git a/src/Firebase/Http/Middleware.php b/src/Firebase/Http/Middleware.php
@@ -5,11 +5,13 @@
 namespace Kreait\Firebase\Http;
 
 use Beste\Json;
+use Closure;
 use Exception;
 use Fig\Http\Message\StatusCodeInterface as StatusCode;
 use GuzzleHttp\Exception\RequestException;
 use GuzzleHttp\MessageFormatter;
 use GuzzleHttp\Promise\Create;
+use GuzzleHttp\Promise\PromiseInterface;
 use GuzzleHttp\Psr7\Query;
 use Psr\Http\Message\RequestInterface;
 use Psr\Http\Message\ResponseInterface;
@@ -29,7 +31,7 @@ final class Middleware
      */
     public static function ensureJsonSuffix(): callable
     {
-        return static fn(callable $handler) => static function (RequestInterface $request, ?array $options = null) use ($handler) {
+        return static fn(callable $handler): Closure => static function (RequestInterface $request, ?array $options = null) use ($handler) {
             $uri = $request->getUri();
             $path = '/'.ltrim($uri->getPath(), '/');
 
@@ -47,7 +49,7 @@ public static function ensureJsonSuffix(): callable
      */
     public static function addDatabaseAuthVariableOverride(?array $override): callable
     {
-        return static fn(callable $handler) => static function (RequestInterface $request, ?array $options = null) use ($handler, $override) {
+        return static fn(callable $handler): Closure => static function (RequestInterface $request, ?array $options = null) use ($handler, $override) {
             $uri = $request->getUri();
 
             $uri = $uri->withQuery(Query::build(
@@ -60,16 +62,16 @@ public static function addDatabaseAuthVariableOverride(?array $override): callab
 
     public static function log(LoggerInterface $logger, MessageFormatter $formatter, string $logLevel, string $errorLogLevel): callable
     {
-        return static fn(callable $handler) => static fn($request, array $options) => $handler($request, $options)->then(
-            static function (ResponseInterface $response) use ($logger, $request, $formatter, $logLevel, $errorLogLevel) {
+        return static fn(callable $handler): Closure => static fn($request, array $options) => $handler($request, $options)->then(
+            static function (ResponseInterface $response) use ($logger, $request, $formatter, $logLevel, $errorLogLevel): ResponseInterface {
                 $message = $formatter->format($request, $response);
                 $messageLogLevel = $response->getStatusCode() >= StatusCode::STATUS_BAD_REQUEST ? $errorLogLevel : $logLevel;
 
                 $logger->log($messageLogLevel, $message);
 
                 return $response;
             },
-            static function (Exception $reason) use ($logger, $request, $formatter, $errorLogLevel) {
+            static function (Exception $reason) use ($logger, $request, $formatter, $errorLogLevel): PromiseInterface {
                 $response = $reason instanceof RequestException ? $reason->getResponse() : null;
                 $message = $formatter->format($request, $response, $reason);
 
diff --git a/src/Firebase/Messaging.php b/src/Firebase/Messaging.php
@@ -89,14 +89,14 @@ public function sendAll($messages, bool $validateOnly = false): MulticastSendRep
         $sendReports = array_fill(0, count($messages), null);
 
         $config = [
-            'fulfilled' => function (ResponseInterface $response, int $index) use ($messages, &$sendReports) {
+            'fulfilled' => function (ResponseInterface $response, int $index) use ($messages, &$sendReports): void {
                 $message = $messages[$index];
 
                 $json = Json::decode((string) $response->getBody(), true);
 
                 $sendReports[$index] = SendReport::success($message->target(), $json, $message);
             },
-            'rejected' => function (Throwable $reason, int $index) use ($messages, &$sendReports) {
+            'rejected' => function (Throwable $reason, int $index) use ($messages, &$sendReports): void {
                 $message = $messages[$index];
 
                 $error = $this->exceptionConverter->convertException($reason);
@@ -159,7 +159,7 @@ public function unsubscribeFromTopic(string|Topic $topic, RegistrationTokens|Reg
     public function unsubscribeFromTopics(array $topics, RegistrationTokens|RegistrationToken|array|string $registrationTokenOrTokens): array
     {
         $topics = array_map(
-            static fn($topic) => $topic instanceof Topic ? $topic : Topic::fromValue($topic),
+            static fn($topic): Topic => $topic instanceof Topic ? $topic : Topic::fromValue($topic),
             $topics,
         );
 
@@ -177,7 +177,7 @@ public function unsubscribeFromAllTopics($registrationTokenOrTokens): array
         foreach ($tokens as $token) {
             $promises[$token->value()] = $this->appInstanceApi
                 ->getAppInstanceAsync($token)
-                ->then(function (AppInstance $appInstance) use ($token) {
+                ->then(function (AppInstance $appInstance) use ($token): array {
                     $topics = [];
 
                     foreach ($appInstance->topicSubscriptions() as $subscription) {
@@ -186,7 +186,7 @@ public function unsubscribeFromAllTopics($registrationTokenOrTokens): array
 
                     return array_keys($this->unsubscribeFromTopics($topics, $token));
                 })
-                ->otherwise(static fn(Throwable $e) => $e->getMessage())
+                ->otherwise(static fn(Throwable $e): string => $e->getMessage())
             ;
         }
 
diff --git a/src/Firebase/Messaging/AndroidConfig.php b/src/Firebase/Messaging/AndroidConfig.php
@@ -234,7 +234,7 @@ public function withNotificationVisibility(string $notificationVisibility): self
 
     public function jsonSerialize(): array
     {
-        return array_filter($this->config, static fn($value) => $value !== null && $value !== []);
+        return array_filter($this->config, static fn($value): bool => $value !== null && $value !== []);
     }
 
     /**
diff --git a/src/Firebase/Messaging/AppInstance.php b/src/Firebase/Messaging/AppInstance.php
@@ -64,7 +64,7 @@ public function isSubscribedToTopic(Topic|string $topic): bool
         $topic = $topic instanceof Topic ? $topic : Topic::fromValue($topic);
 
         return $this->topicSubscriptions
-            ->filter(static fn(TopicSubscription $subscription) => $topic->value() === $subscription->topic()->value())
+            ->filter(static fn(TopicSubscription $subscription): bool => $topic->value() === $subscription->topic()->value())
             ->count() > 0
         ;
     }
diff --git a/src/Firebase/Messaging/AppInstanceApiClient.php b/src/Firebase/Messaging/AppInstanceApiClient.php
@@ -47,7 +47,7 @@ public function subscribeToTopics(array $topics, RegistrationTokens $tokens): ar
                         'registration_tokens' => $tokenStrings,
                     ],
                 ])
-                ->then(static fn(ResponseInterface $response) => Json::decode((string) $response->getBody(), true))
+                ->then(static fn(ResponseInterface $response): mixed => Json::decode((string) $response->getBody(), true))
             ;
         }
 
@@ -114,7 +114,7 @@ public function unsubscribeFromTopics(array $topics, RegistrationTokens $tokens)
                         'registration_tokens' => $tokenStrings,
                     ],
                 ])
-                ->then(static fn(ResponseInterface $response) => Json::decode((string) $response->getBody(), true))
+                ->then(static fn(ResponseInterface $response): mixed => Json::decode((string) $response->getBody(), true))
             ;
         }
 
@@ -165,12 +165,12 @@ public function getAppInstanceAsync(RegistrationToken $registrationToken): Promi
     {
         return $this->client
             ->requestAsync('GET', '/iid/'.$registrationToken->value().'?details=true')
-            ->then(static function (ResponseInterface $response) use ($registrationToken) {
+            ->then(static function (ResponseInterface $response) use ($registrationToken): AppInstance {
                 $data = Json::decode((string) $response->getBody(), true);
 
                 return AppInstance::fromRawData($registrationToken, $data);
             })
-            ->otherwise(fn(Throwable $e) => Create::rejectionFor($this->errorHandler->convertException($e)))
+            ->otherwise(fn(Throwable $e): PromiseInterface => Create::rejectionFor($this->errorHandler->convertException($e)))
         ;
     }
 
diff --git a/src/Firebase/Messaging/CloudMessage.php b/src/Firebase/Messaging/CloudMessage.php
@@ -250,7 +250,7 @@ public function jsonSerialize(): array
 
         return array_filter(
             $data,
-            static fn($value) => $value !== null && $value !== [],
+            static fn($value): bool => $value !== null && $value !== [],
         );
     }
 
diff --git a/src/Firebase/Messaging/MulticastSendReport.php b/src/Firebase/Messaging/MulticastSendReport.php
@@ -43,12 +43,12 @@ public function getItems(): array
 
     public function successes(): self
     {
-        return $this->filter(static fn(SendReport $item) => $item->isSuccess());
+        return $this->filter(static fn(SendReport $item): bool => $item->isSuccess());
     }
 
     public function failures(): self
     {
-        return $this->filter(static fn(SendReport $item) => $item->isFailure());
+        return $this->filter(static fn(SendReport $item): bool => $item->isFailure());
     }
 
     public function hasFailures(): bool
@@ -77,8 +77,8 @@ public function map(callable $callback): array
     public function validTokens(): array
     {
         return $this->successes()
-            ->filter(static fn(SendReport $report) => $report->target()->type() === MessageTarget::TOKEN)
-            ->map(static fn(SendReport $report) => $report->target()->value())
+            ->filter(static fn(SendReport $report): bool => $report->target()->type() === MessageTarget::TOKEN)
+            ->map(static fn(SendReport $report): string => $report->target()->value())
         ;
     }
 
@@ -90,8 +90,8 @@ public function validTokens(): array
     public function unknownTokens(): array
     {
         return $this
-            ->filter(static fn(SendReport $report) => $report->messageWasSentToUnknownToken())
-            ->map(static fn(SendReport $report) => $report->target()->value())
+            ->filter(static fn(SendReport $report): bool => $report->messageWasSentToUnknownToken())
+            ->map(static fn(SendReport $report): string => $report->target()->value())
         ;
     }
 
@@ -103,8 +103,8 @@ public function unknownTokens(): array
     public function invalidTokens(): array
     {
         return $this
-            ->filter(static fn(SendReport $report) => $report->messageTargetWasInvalid())
-            ->map(static fn(SendReport $report) => $report->target()->value())
+            ->filter(static fn(SendReport $report): bool => $report->messageTargetWasInvalid())
+            ->map(static fn(SendReport $report): string => $report->target()->value())
         ;
     }
 
diff --git a/src/Firebase/Messaging/Notification.php b/src/Firebase/Messaging/Notification.php
@@ -99,6 +99,6 @@ public function jsonSerialize(): array
             'title' => $this->title,
             'body' => $this->body,
             'image' => $this->imageUrl,
-        ], static fn($value) => $value !== null);
+        ], static fn($value): bool => $value !== null);
     }
 }
diff --git a/src/Firebase/Messaging/WebPushConfig.php b/src/Firebase/Messaging/WebPushConfig.php
@@ -146,7 +146,7 @@ public function withUrgency(string $urgency): self
     public function jsonSerialize(): array
     {
         // @phpstan-ignore notIdentical.alwaysTrue
-        return array_filter($this->config, static fn($value) => $value !== null);
+        return array_filter($this->config, static fn($value): bool => $value !== null);
     }
 
     /**
diff --git a/src/Firebase/RemoteConfig/ApiClient.php b/src/Firebase/RemoteConfig/ApiClient.php
@@ -9,7 +9,6 @@
 use Kreait\Firebase\Exception\RemoteConfigApiExceptionConverter;
 use Kreait\Firebase\Exception\RemoteConfigException;
 use Psr\Http\Message\ResponseInterface;
-use Psr\Http\Message\UriInterface;
 use Throwable;
 
 use function array_filter;
@@ -118,12 +117,13 @@ public function rollbackToVersion(VersionNumber $versionNumber): ResponseInterfa
     }
 
     /**
-     * @param string|UriInterface $uri
+     * @param non-empty-string $method
+     * @param non-empty-string $uri
      * @param array<string, mixed>|null $options
      *
      * @throws RemoteConfigException
      */
-    private function requestApi(string $method, $uri, ?array $options = null): ResponseInterface
+    private function requestApi(string $method, string $uri, ?array $options = null): ResponseInterface
     {
         $options ??= [];
         $options['decode_content'] = 'gzip';
diff --git a/src/Firebase/RemoteConfig/ConditionalValue.php b/src/Firebase/RemoteConfig/ConditionalValue.php
@@ -44,7 +44,7 @@ public static function basedOn($condition): self
     /**
      * @return RemoteConfigParameterValueShape|non-empty-string
      */
-    public function value()
+    public function value(): string|array
     {
         $data = $this->value->toArray();
 
diff --git a/src/Firebase/RemoteConfig/Template.php b/src/Firebase/RemoteConfig/Template.php
@@ -181,7 +181,7 @@ public function withCondition(Condition $condition): self
     public function conditionNames(): array
     {
         return array_values(array_unique(
-            array_map(static fn(Condition $c) => $c->name(), $this->conditions),
+            array_map(static fn(Condition $c): string => $c->name(), $this->conditions),
         ));
     }
 
@@ -192,7 +192,7 @@ public function withRemovedCondition(string $name): self
     {
         $template = clone $this;
         $template->conditions = array_values(
-            array_filter($this->conditions, static fn(Condition $c) => $c->name() !== $name),
+            array_filter($this->conditions, static fn(Condition $c): bool => $c->name() !== $name),
         );
 
         return $template;
@@ -207,9 +207,9 @@ public function withRemovedCondition(string $name): self
      */
     public function jsonSerialize(): array
     {
-        $conditions = array_map(fn(Condition $c) => $c->jsonSerialize(), $this->conditions);
-        $parameters = array_map(fn(Parameter $p) => $p->jsonSerialize(), $this->parameters);
-        $parameterGroups = array_map(fn(ParameterGroup $p) => $p->jsonSerialize(), $this->parameterGroups);
+        $conditions = array_map(fn(Condition $c): array => $c->jsonSerialize(), $this->conditions);
+        $parameters = array_map(fn(Parameter $p): array => $p->jsonSerialize(), $this->parameters);
+        $parameterGroups = array_map(fn(ParameterGroup $p): array => $p->jsonSerialize(), $this->parameterGroups);
 
         return [
             'conditions' => $conditions !== [] ? $conditions : null,
@@ -273,7 +273,7 @@ private static function buildParameterGroup(string $name, array $parameterGroupD
 
     private function assertThatAllConditionalValuesAreValid(Parameter $parameter): void
     {
-        $conditionNames = array_map(static fn(Condition $c) => $c->name(), $this->conditions);
+        $conditionNames = array_map(static fn(Condition $c): string => $c->name(), $this->conditions);
 
         foreach ($parameter->conditionalValues() as $conditionalValue) {
             if (!in_array($conditionalValue->conditionName(), $conditionNames, true)) {
diff --git a/src/Firebase/Request/EditUserTrait.php b/src/Firebase/Request/EditUserTrait.php
@@ -160,7 +160,7 @@ public function prepareJsonSerialize(): array
             'phoneNumber' => $this->phoneNumber,
             'photoUrl' => $this->photoUrl,
             'password' => $this->clearTextPassword,
-        ], static fn($value) => $value !== null);
+        ], static fn($value): bool => $value !== null);
     }
 
     public function hasUid(): bool
diff --git a/src/Firebase/Request/UpdateUser.php b/src/Firebase/Request/UpdateUser.php
@@ -145,7 +145,7 @@ public static function withProperties(array $properties): self
                 case 'removeproviders':
                     $request = array_reduce(
                         (array) $value,
-                        static fn(self $request, $provider) => $request->withRemovedProvider($provider),
+                        static fn(self $request, $provider): \Kreait\Firebase\Request\UpdateUser => $request->withRemovedProvider($provider),
                         $request,
                     );
 
diff --git a/tests/Integration/FactoryTest.php b/tests/Integration/FactoryTest.php
diff --git a/tests/IntegrationTestCase.php b/tests/IntegrationTestCase.php
diff --git a/tests/Unit/Http/HttpClientOptionsTest.php b/tests/Unit/Http/HttpClientOptionsTest.php
diff --git a/tests/Unit/Http/MiddlewareTest.php b/tests/Unit/Http/MiddlewareTest.php
diff --git a/tests/Unit/Messaging/MessageDataTest.php b/tests/Unit/Messaging/MessageDataTest.php
diff --git a/tests/Unit/RemoteConfig/TemplateTest.php b/tests/Unit/RemoteConfig/TemplateTest.php

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public static function fromArray(array $settings): self`
`52`	`52`	`{`
`53`	`53`	`$instance = new self();`
`54`	`54`
`55`		`- $settings = array_filter($settings, static fn($value) => $value !== null);`
	`55`	`+ $settings = array_filter($settings, static fn($value): bool => $value !== null);`
`56`	`56`
`57`	`57`	`foreach ($settings as $key => $value) {`
`58`	`58`	`switch (mb_strtolower($key)) {`
`@@ -116,6 +116,6 @@ public function toArray(): array`
`116`	`116`	`'androidMinimumVersion' => $this->androidMinimumVersion,`
`117`	`117`	`'androidInstallApp' => $this->androidInstallApp,`
`118`	`118`	`'iOSBundleId' => $this->iOSBundleId,`
`119`		`- ], static fn($value) => is_bool($value) \|\| (is_string($value) && $value !== ''));`
	`119`	`+ ], static fn($value): bool => is_bool($value) \|\| (is_string($value) && $value !== ''));`
`120`	`120`	`}`
`121`	`121`	`}`
Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ private static function mfaInfoFromResponseData(array $data): ?MfaInfo`
`144`	`144`	`private static function userInfoFromResponseData(array $data): array`
`145`	`145`	`{`
`146`	`146`	`return array_map(`
`147`		`- static fn(array $userInfoData) => UserInfo::fromResponseData($userInfoData),`
	`147`	`+ static fn(array $userInfoData): UserInfo => UserInfo::fromResponseData($userInfoData),`
`148`	`148`	`$data['providerUserInfo'],`
`149`	`149`	`);`
`150`	`150`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public function modifyValue(mixed $value): mixed`
`38`	`38`
`39`	`39`	`$expression = str_replace('/', '.', $this->childKey);`
`40`	`40`
`41`		`- uasort($value, static fn($a, $b) => search($expression, $a) <=> search($expression, $b));`
	`41`	`+ uasort($value, static fn($a, $b): int => search($expression, $a) <=> search($expression, $b));`
`42`	`42`
`43`	`43`	`return $value;`
`44`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -103,12 +103,12 @@ public function appReOpens(): self`
`103`	`103`
`104`	`104`	`public function filterByType(string $type): self`
`105`	`105`	`{`
`106`		`- return $this->filter(static fn(array $event) => ($event['event'] ?? null) === $type);`
	`106`	`+ return $this->filter(static fn(array $event): bool => ($event['event'] ?? null) === $type);`
`107`	`107`	`}`
`108`	`108`
`109`	`109`	`public function filterByPlatform(string $platform): self`
`110`	`110`	`{`
`111`		`- return $this->filter(static fn(array $event) => ($event['platform'] ?? null) === $platform);`
	`111`	`+ return $this->filter(static fn(array $event): bool => ($event['platform'] ?? null) === $platform);`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`public function filter(callable $filter): self`
Original file line number	Diff line number	Diff line change
`@@ -234,7 +234,7 @@ public function withNotificationVisibility(string $notificationVisibility): self`
`234`	`234`
`235`	`235`	`public function jsonSerialize(): array`
`236`	`236`	`{`
`237`		`- return array_filter($this->config, static fn($value) => $value !== null && $value !== []);`
	`237`	`+ return array_filter($this->config, static fn($value): bool => $value !== null && $value !== []);`
`238`	`238`	`}`
`239`	`239`
`240`	`240`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ public function isSubscribedToTopic(Topic\|string $topic): bool`
`64`	`64`	`$topic = $topic instanceof Topic ? $topic : Topic::fromValue($topic);`
`65`	`65`
`66`	`66`	`return $this->topicSubscriptions`
`67`		`- ->filter(static fn(TopicSubscription $subscription) => $topic->value() === $subscription->topic()->value())`
	`67`	`+ ->filter(static fn(TopicSubscription $subscription): bool => $topic->value() === $subscription->topic()->value())`
`68`	`68`	`->count() > 0`
`69`	`69`	`;`
`70`	`70`	`}`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public function subscribeToTopics(array $topics, RegistrationTokens $tokens): ar`
`47`	`47`	`'registration_tokens' => $tokenStrings,`
`48`	`48`	`],`
`49`	`49`	`])`
`50`		`- ->then(static fn(ResponseInterface $response) => Json::decode((string) $response->getBody(), true))`
	`50`	`+ ->then(static fn(ResponseInterface $response): mixed => Json::decode((string) $response->getBody(), true))`
`51`	`51`	`;`
`52`	`52`	`}`
`53`	`53`
`@@ -114,7 +114,7 @@ public function unsubscribeFromTopics(array $topics, RegistrationTokens $tokens)`
`114`	`114`	`'registration_tokens' => $tokenStrings,`
`115`	`115`	`],`
`116`	`116`	`])`
`117`		`- ->then(static fn(ResponseInterface $response) => Json::decode((string) $response->getBody(), true))`
	`117`	`+ ->then(static fn(ResponseInterface $response): mixed => Json::decode((string) $response->getBody(), true))`
`118`	`118`	`;`
`119`	`119`	`}`
`120`	`120`
`@@ -165,12 +165,12 @@ public function getAppInstanceAsync(RegistrationToken $registrationToken): Promi`
`165`	`165`	`{`
`166`	`166`	`return $this->client`
`167`	`167`	`->requestAsync('GET', '/iid/'.$registrationToken->value().'?details=true')`
`168`		`- ->then(static function (ResponseInterface $response) use ($registrationToken) {`
	`168`	`+ ->then(static function (ResponseInterface $response) use ($registrationToken): AppInstance {`
`169`	`169`	`$data = Json::decode((string) $response->getBody(), true);`
`170`	`170`
`171`	`171`	`return AppInstance::fromRawData($registrationToken, $data);`
`172`	`172`	`})`
`173`		`- ->otherwise(fn(Throwable $e) => Create::rejectionFor($this->errorHandler->convertException($e)))`
	`173`	`+ ->otherwise(fn(Throwable $e): PromiseInterface => Create::rejectionFor($this->errorHandler->convertException($e)))`
`174`	`174`	`;`
`175`	`175`	`}`
`176`	`176`
Original file line number	Diff line number	Diff line change
`@@ -250,7 +250,7 @@ public function jsonSerialize(): array`
`250`	`250`
`251`	`251`	`return array_filter(`
`252`	`252`	`$data,`
`253`		`- static fn($value) => $value !== null && $value !== [],`
	`253`	`+ static fn($value): bool => $value !== null && $value !== [],`
`254`	`254`	`);`
`255`	`255`	`}`
`256`	`256`
Original file line number	Diff line number	Diff line change
`@@ -43,12 +43,12 @@ public function getItems(): array`
`43`	`43`
`44`	`44`	`public function successes(): self`
`45`	`45`	`{`
`46`		`- return $this->filter(static fn(SendReport $item) => $item->isSuccess());`
	`46`	`+ return $this->filter(static fn(SendReport $item): bool => $item->isSuccess());`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`public function failures(): self`
`50`	`50`	`{`
`51`		`- return $this->filter(static fn(SendReport $item) => $item->isFailure());`
	`51`	`+ return $this->filter(static fn(SendReport $item): bool => $item->isFailure());`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`public function hasFailures(): bool`
`@@ -77,8 +77,8 @@ public function map(callable $callback): array`
`77`	`77`	`public function validTokens(): array`
`78`	`78`	`{`
`79`	`79`	`return $this->successes()`
`80`		`- ->filter(static fn(SendReport $report) => $report->target()->type() === MessageTarget::TOKEN)`
`81`		`- ->map(static fn(SendReport $report) => $report->target()->value())`
	`80`	`+ ->filter(static fn(SendReport $report): bool => $report->target()->type() === MessageTarget::TOKEN)`
	`81`	`+ ->map(static fn(SendReport $report): string => $report->target()->value())`
`82`	`82`	`;`
`83`	`83`	`}`
`84`	`84`
`@@ -90,8 +90,8 @@ public function validTokens(): array`
`90`	`90`	`public function unknownTokens(): array`
`91`	`91`	`{`
`92`	`92`	`return $this`
`93`		`- ->filter(static fn(SendReport $report) => $report->messageWasSentToUnknownToken())`
`94`		`- ->map(static fn(SendReport $report) => $report->target()->value())`
	`93`	`+ ->filter(static fn(SendReport $report): bool => $report->messageWasSentToUnknownToken())`
	`94`	`+ ->map(static fn(SendReport $report): string => $report->target()->value())`
`95`	`95`	`;`
`96`	`96`	`}`
`97`	`97`
`@@ -103,8 +103,8 @@ public function unknownTokens(): array`
`103`	`103`	`public function invalidTokens(): array`
`104`	`104`	`{`
`105`	`105`	`return $this`
`106`		`- ->filter(static fn(SendReport $report) => $report->messageTargetWasInvalid())`
`107`		`- ->map(static fn(SendReport $report) => $report->target()->value())`
	`106`	`+ ->filter(static fn(SendReport $report): bool => $report->messageTargetWasInvalid())`
	`107`	`+ ->map(static fn(SendReport $report): string => $report->target()->value())`
`108`	`108`	`;`
`109`	`109`	`}`
`110`	`110`
Original file line number	Diff line number	Diff line change
`@@ -99,6 +99,6 @@ public function jsonSerialize(): array`
`99`	`99`	`'title' => $this->title,`
`100`	`100`	`'body' => $this->body,`
`101`	`101`	`'image' => $this->imageUrl,`
`102`		`- ], static fn($value) => $value !== null);`
	`102`	`+ ], static fn($value): bool => $value !== null);`
`103`	`103`	`}`
`104`	`104`	`}`