feat: mark all possible classes as readonly

src/Client/Http/RequestFactory.php

@@ -24,7 +24,7 @@
 
 use const PHP_QUERY_RFC3986;
 
-final class RequestFactory
+final readonly class RequestFactory
 {
     private UriInterface|null $uri;
 
@@ -36,7 +36,7 @@
         UriFactoryInterface|null $uriFactory = null,
         UriInterface|string $uri = '',
     ) {
         if ($uriFactory === null && $uri === '') {
             $uri = null;
         } elseif (is_string($uri)) {
             if ($uriFactory === null) {
@@ -62,7 +62,7 @@
         );
 
         if ($this->uri === null) {
             $uri = $query === '' ? '' : '?' . $query;
         } else {
             $uriQuery = $this->uri->getQuery();
             try {
@@ -95,7 +95,7 @@
 
         /** @var array<string, Type> $paramToType */
         $paramToType = array_reduce(
             array_keys($matches[1]),
             static function (array $acc, string|int $k) use ($matches) {
                 $acc[$matches[1][$k]] = Type::fromString($matches[2][$k]);
 
@@ -120,7 +120,7 @@
         try {
             $body    = new MultipartStream($streamElements);
             $request = $request->withBody($body)
                 ->withHeader('Content-Type', 'multipart/form-data; boundary=' . $body->getBoundary());
         } catch (InvalidArgumentException) {
             absurd();
         }

src/Client/Http/RequestOptions.php

@@ -4,7 +4,7 @@
 
 namespace SimPod\ClickHouseClient\Client\Http;
 
-final class RequestOptions
+final readonly class RequestOptions
 {
     /** @param array<string, mixed> $params */
     public function __construct(

src/Client/Http/RequestSettings.php

@@ -4,7 +4,7 @@
 
 namespace SimPod\ClickHouseClient\Client\Http;
 
-final class RequestSettings
+final readonly class RequestSettings
 {
     /** @var array<string, float|int|string> */
     public array $settings;

src/Client/PsrClickHouseAsyncClient.php

@@ -21,7 +21,7 @@
 
 use function uniqid;
 
-class PsrClickHouseAsyncClient implements ClickHouseAsyncClient
+readonly class PsrClickHouseAsyncClient implements ClickHouseAsyncClient
 {
     private SqlFactory $sqlFactory;
 
@@ -98,7 +98,7 @@
         );
 
         $id = uniqid('', true);
         $this->sqlLogger?->startQuery($id, $sql);
 
         return Create::promiseFor(
             $this->asyncClient->sendAsyncRequest($request),
@@ -117,7 +117,7 @@
 
                     return $processResponse($response);
                 },
                 fn () => $this->sqlLogger?->stopQuery($id),
             );
     }
 }

src/Client/PsrClickHouseClient.php

@@ -36,7 +36,7 @@
 use function sprintf;
 use function uniqid;
 
-class PsrClickHouseClient implements ClickHouseClient
+readonly class PsrClickHouseClient implements ClickHouseClient
 {
     private ValueFormatter $valueFormatter;
 
@@ -64,7 +64,7 @@
 
     public function executeQueryWithParams(string $query, array $params, array $settings = []): void
     {
         $this->executeRequest(
             $this->sqlFactory->createWithParameters($query, $params),
             params: $params,
             settings: $settings,

src/Format/Json.php

@@ -11,7 +11,7 @@
  * @template T
  * @implements Format<\SimPod\ClickHouseClient\Output\Json<T>>
  */
-final class Json implements Format
+final readonly class Json implements Format
 {
     /** @throws JsonException */
     public static function output(string $contents): Output

src/Format/JsonCompact.php

@@ -11,7 +11,7 @@
  * @template T
  * @implements Format<\SimPod\ClickHouseClient\Output\JsonCompact<T>>
  */
-final class JsonCompact implements Format
+final readonly class JsonCompact implements Format
 {
     /** @throws JsonException */
     public static function output(string $contents): Output

src/Format/JsonEachRow.php

@@ -11,7 +11,7 @@
  * @template T
  * @implements Format<\SimPod\ClickHouseClient\Output\JsonEachRow<T>>
  */
-final class JsonEachRow implements Format
+final readonly class JsonEachRow implements Format
 {
     /** @throws JsonException */
     public static function output(string $contents): Output

src/Format/Null_.php

@@ -12,7 +12,7 @@
  * @template T
  * @implements Format<\SimPod\ClickHouseClient\Output\Null_<T>>
  */
-final class Null_ implements Format
+final readonly class Null_ implements Format
 {
     public static function output(string $contents): Output
     {

src/Format/Pretty.php

@@ -11,7 +11,7 @@
  * @template T
  * @implements Format<Basic<T>>
  */
-final class Pretty implements Format
+final readonly class Pretty implements Format
 {
     public static function output(string $contents): Output
     {

src/Format/PrettySpace.php

@@ -11,7 +11,7 @@
  * @template T
  * @implements Format<Basic<T>>
  */
-final class PrettySpace implements Format
+final readonly class PrettySpace implements Format
 {
     public static function output(string $contents): Output
     {

src/Format/RowBinary.php

@@ -11,7 +11,7 @@
  * @template T
  * @implements Format<Basic<T>>
  */
-final class RowBinary implements Format
+final readonly class RowBinary implements Format
 {
     public static function output(string $contents): Output
     {

src/Format/TabSeparated.php

@@ -11,7 +11,7 @@
  * @template T
  * @implements Format<Basic<T>>
  */
-final class TabSeparated implements Format
+final readonly class TabSeparated implements Format
 {
     public static function output(string $contents): Output
     {

src/Logger/LoggerChain.php

@@ -4,7 +4,7 @@
 
 namespace SimPod\ClickHouseClient\Logger;
 
-final class LoggerChain implements SqlLogger
+final readonly class LoggerChain implements SqlLogger
 {
     /** @param SqlLogger[] $loggers */
     public function __construct(private array $loggers = [])

src/Logger/PsrLogger.php

@@ -6,7 +6,7 @@
 
 use Psr\Log\LoggerInterface;
 
-final class PsrLogger implements SqlLogger
+final readonly class PsrLogger implements SqlLogger
 {
     public function __construct(private LoggerInterface $logger)
     {

src/Output/Basic.php

@@ -9,7 +9,7 @@
  * @template T
  * @implements Output<T>
  */
-final class Basic implements Output
+final readonly class Basic implements Output
 {
     public function __construct(public string $contents)
     {

src/Output/Json.php

@@ -15,7 +15,7 @@
  * @template T
  * @implements Output<T>
  */
-final class Json implements Output
+final readonly class Json implements Output
 {
     /** @var list<T> */
     public array $data;

src/Output/JsonCompact.php

@@ -15,7 +15,7 @@
  * @template T
  * @implements Output<T>
  */
-final class JsonCompact implements Output
+final readonly class JsonCompact implements Output
 {
     /** @var list<T> */
     public array $data;

src/Output/JsonEachRow.php

@@ -17,7 +17,7 @@
  * @template T
  * @implements Output<T>
  */
-final class JsonEachRow implements Output
+final readonly class JsonEachRow implements Output
 {
     /** @var list<T> */
     public array $data;

src/Output/Null_.php

@@ -11,7 +11,7 @@
  * @template T
  * @implements Output<T>
  */
-final class Null_ implements Output
+final readonly class Null_ implements Output
 {
     public function __construct(string $_)
     {

src/Param/ParamValueConverterRegistry.php

@@ -32,9 +32,9 @@
  * @phpstan-type Converter Closure(mixed, Type|string|null, bool):(StreamInterface|string)
  * @phpstan-type ConverterRegistry array<string, Converter>
  */
-final class ParamValueConverterRegistry
+final readonly class ParamValueConverterRegistry
 {
-    private const CaseInsensitiveTypes = [
+    private const array CaseInsensitiveTypes = [
         'bool',
         'date',
         'date32',

src/Snippet/CurrentDatabase.php

@@ -9,7 +9,7 @@
 use SimPod\ClickHouseClient\Exception\ServerError;
 use SimPod\ClickHouseClient\Format\JsonEachRow;
 
-final class CurrentDatabase
+final readonly class CurrentDatabase
 {
     /**
      * @throws ClientExceptionInterface

src/Snippet/DatabaseSize.php

@@ -12,7 +12,7 @@
 use SimPod\ClickHouseClient\Format\JsonEachRow;
 use SimPod\ClickHouseClient\Sql\Expression;
 
-final class DatabaseSize
+final readonly class DatabaseSize
 {
     /**
      * @throws ClientExceptionInterface

src/Snippet/Parts.php

@@ -13,7 +13,7 @@
 
 use function sprintf;
 
-final class Parts
+final readonly class Parts
 {
     /**
      * @return array<array<string, mixed>>

src/Snippet/ShowCreateTable.php

@@ -9,7 +9,7 @@
 use SimPod\ClickHouseClient\Exception\ServerError;
 use SimPod\ClickHouseClient\Format\JsonEachRow;
 
-final class ShowCreateTable
+final readonly class ShowCreateTable
 {
     /**
      * @throws ClientExceptionInterface

src/Snippet/ShowDatabases.php

@@ -11,7 +11,7 @@
 
 use function array_map;
 
-final class ShowDatabases
+final readonly class ShowDatabases
 {
     /**
      * @return list<string>

src/Snippet/TableSizes.php

@@ -13,7 +13,7 @@
 use SimPod\ClickHouseClient\Sql\Expression;
 
 /** @phpstan-type Entry array{table: string, database: string, size: string, min_date: string, max_date: string} */
-final class TableSizes
+final readonly class TableSizes
 {
     /**
      * @return array<Entry>

src/Snippet/Version.php

@@ -9,7 +9,7 @@
 use SimPod\ClickHouseClient\Exception\ServerError;
 use SimPod\ClickHouseClient\Format\JsonEachRow;
 
-final class Version
+final readonly class Version
 {
     /**
      * @throws ClientExceptionInterface

src/Sql/Escaper.php

@@ -11,7 +11,7 @@
  *
  * @link https://github.com/ClickHouse/clickhouse-jdbc/blob/8481c1323f5de09bb9dbbf67085e5e1b2585756a/src/main/java/ru/yandex/clickhouse/ClickHouseUtil.java
  */
-final class Escaper
+final readonly class Escaper
 {
     public static function escape(string $s): string
     {

src/Sql/Expression.php

@@ -4,7 +4,7 @@
 
 namespace SimPod\ClickHouseClient\Sql;
 
-final class Expression
+final readonly class Expression
 {
     public string $innerExpression;
 

src/Sql/SqlFactory.php

@@ -13,7 +13,7 @@
 use function str_replace;
 
 /** @internal */
-final class SqlFactory
+final readonly class SqlFactory
 {
     public function __construct(private ValueFormatter $valueFormatter)
     {

tests/ClickHouseVersion.php

@@ -15,9 +15,9 @@
 
 use const STR_PAD_LEFT;
 
-final class ClickHouseVersion
+final readonly class ClickHouseVersion
 {
-    private const EnvName = 'CLICKHOUSE_VERSION';
+    private const string EnvName = 'CLICKHOUSE_VERSION';
 
     /** @throws RuntimeException */
     public static function get(): int