Working on next version

Author: GrahamCampbell

src/Auth/Authenticator/AbstractAuthenticator.php

@@ -23,11 +23,6 @@
  */
 abstract class AbstractAuthenticator implements AuthenticatorInterface
 {
-    /**
-     * The client to perform the authentication on.
-     *
-     * @var \Gitlab\Client|null
-     */
     private ?Client $client = null;
 
     /**

src/Auth/AuthenticatorFactory.php

@@ -36,9 +36,9 @@ public function make(string $method): AuthenticatorInterface
     {
         return match ($method) {
             'job_token' => new Authenticator\JobTokenAuthenticator(),
-            'oauth' => new Authenticator\OauthAuthenticator(),
-            'token' => new Authenticator\TokenAuthenticator(),
-            default => throw new InvalidArgumentException("Unsupported authentication method [$method]."),
+            'oauth'     => new Authenticator\OauthAuthenticator(),
+            'token'     => new Authenticator\TokenAuthenticator(),
+            default     => throw new InvalidArgumentException("Unsupported authentication method [$method]."),
         };
     }
 }

src/Cache/ConnectionFactory.php

@@ -25,11 +25,6 @@
  */
 class ConnectionFactory
 {
-    /**
-     * The cache factory instance.
-     *
-     * @var \Illuminate\Contracts\Cache\Factory|null
-     */
     private ?Factory $cache;
 
     /**
@@ -39,9 +34,9 @@ class ConnectionFactory
      *
      * @return void
      */
-    public function __construct(Factory $cache = null)
-    {
-        $this->cache = $cache;
+    public function __construct(
+        private readonly ?Factory $cache = null,
+    ) {
     }
 
     /**

src/Cache/Connector/IlluminateConnector.php

@@ -28,25 +28,9 @@
  */
 final class IlluminateConnector implements ConnectorInterface
 {
-    /**
-     * The minimum cache lifetime of 12 hours.
-     *
-     * @var int
-     */
     private const MIN_CACHE_LIFETIME = 43200;
-
-    /**
-     * The maximum cache lifetime of 48 hours.
-     *
-     * @var int
-     */
     private const MAX_CACHE_LIFETIME = 172800;
 
-    /**
-     * The cache factory instance.
-     *
-     * @var \Illuminate\Contracts\Cache\Factory|null
-     */
     private ?Factory $cache;
 
     /**
@@ -56,9 +40,9 @@ final class IlluminateConnector implements ConnectorInterface
      *
      * @return void
      */
-    public function __construct(Factory $cache = null)
-    {
-        $this->cache = $cache;
+    public function __construct(
+        private readonly ?Factory $cache = null,
+    ) {
     }
 
     /**

src/GitLabFactory.php

@@ -31,27 +31,6 @@
  */
 class GitLabFactory
 {
-    /**
-     * The http client builder factory instance.
-     *
-     * @var \GrahamCampbell\GitLab\HttpClient\BuilderFactory
-     */
-    private BuilderFactory $builder;
-
-    /**
-     * The authenticator factory instance.
-     *
-     * @var \GrahamCampbell\GitLab\Auth\AuthenticatorFactory
-     */
-    private AuthenticatorFactory $auth;
-
-    /**
-     * The cache factory instance.
-     *
-     * @var \GrahamCampbell\GitLab\Cache\ConnectionFactory
-     */
-    private ConnectionFactory $cache;
-
     /**
      * Create a new gitlab factory instance.
      *
@@ -61,11 +40,11 @@ class GitLabFactory
      *
      * @return void
      */
-    public function __construct(BuilderFactory $builder, AuthenticatorFactory $auth, ConnectionFactory $cache)
-    {
-        $this->builder = $builder;
-        $this->auth = $auth;
-        $this->cache = $cache;
+    public function __construct(
+        private readonly BuilderFactory $builder,
+        private readonly AuthenticatorFactory $auth,
+        private readonly ConnectionFactory $cache,
+    ) {
     }
 
     /**

src/GitLabManager.php

@@ -65,12 +65,7 @@
  */
 class GitLabManager extends AbstractManager
 {
-    /**
-     * The factory instance.
-     *
-     * @var \GrahamCampbell\GitLab\GitLabFactory
-     */
-    protected GitLabFactory $factory;
+    protected readonly GitLabFactory $factory;
 
     /**
      * Create a new gitlab manager instance.

src/HttpClient/BuilderFactory.php

@@ -26,34 +26,6 @@
  */
 class BuilderFactory
 {
-    /**
-     * The http client instance.
-     *
-     * @var \Psr\Http\Client\ClientInterface
-     */
-    private ClientInterface $httpClient;
-
-    /**
-     * The request factory instance.
-     *
-     * @var \Psr\Http\Message\RequestFactoryInterface
-     */
-    private RequestFactoryInterface $requestFactory;
-
-    /**
-     * The stream factory instance.
-     *
-     * @var \Psr\Http\Message\StreamFactoryInterface
-     */
-    private StreamFactoryInterface $streamFactory;
-
-    /**
-     * The uri factory instance.
-     *
-     * @var \Psr\Http\Message\UriFactoryInterface
-     */
-    private UriFactoryInterface $uriFactory;
-
     /**
      * Create a new connection factory instance.
      *
@@ -64,12 +36,12 @@ class BuilderFactory
      *
      * @return void
      */
-    public function __construct(ClientInterface $httpClient, RequestFactoryInterface $requestFactory, StreamFactoryInterface $streamFactory, UriFactoryInterface $uriFactory)
-    {
-        $this->httpClient = $httpClient;
-        $this->requestFactory = $requestFactory;
-        $this->streamFactory = $streamFactory;
-        $this->uriFactory = $uriFactory;
+    public function __construct(
+        private readonly ClientInterface $httpClient,
+        private readonly RequestFactoryInterface $requestFactory,
+        private readonly StreamFactoryInterface $streamFactory,
+        private readonly UriFactoryInterface $uriFactory,
+    ) {
     }
 
     /**