Add PHPStan static analysis setup

Add PHPStan as a Composer dev dependency, configure phpstan.neon at level 6, add the Composer analyse script, and fix reported iterable type issues across the SDK and tests. Integration tests requiring valid tokens remain skipped.

Author: Seraphim2000

composer.json

@@ -51,7 +51,8 @@
     },
     "require-dev": {
         "symfony/dotenv": "^5.3",
-        "phpunit/phpunit": "^9.5"
+        "phpunit/phpunit": "^9.5",
+        "phpstan/phpstan": "^2.1"
     },
     "autoload": {
         "psr-4": {
@@ -67,6 +68,7 @@
         }
     },
     "scripts": {
+        "analyse": "vendor/bin/phpstan analyse -c phpstan.neon",
         "test": "phpunit tests/",
         "test:unit": "phpunit tests/ --testsuite=unit",
         "example:token": "php examples/token_generation.php",

phpstan.neon

@@ -0,0 +1,5 @@
+parameters:
+	level: 6
+	paths:
+		- src
+		- tests

src/Cache/ArrayCache.php

@@ -4,7 +4,13 @@
 
 class ArrayCache implements CacheInterface
 {
+     /**
+     * @var array<string, mixed>
+     */
     private array $cache = [];
+    /**
+     * @var array<string, int>
+     */
     private array $expiry = [];
 
     public function get(string $key): mixed

src/Client.php

@@ -21,6 +21,15 @@ public function __construct(?string $token = null, HttpTransportInterface|PsrCli
         $this->transport = $transport ?? new CurlTransport($this->token);
     }
 
+    /**
+     * Execute HTTP request
+     *
+     * @param string $method HTTP method (GET, POST, PUT, DELETE, PATCH)
+     * @param string $url Target URL
+     * @param mixed $payload Request body (for POST/PUT/PATCH)
+     * @param array<string, scalar|null>|null $params Query parameters (for GET) or form data (for other methods)
+     * @return string Response body
+     */
     public function request(
         string $method,
         string $url,
@@ -34,6 +43,11 @@ public function request(
         return $this->transport->request($method, $url, $payload, $params);
     }
 
+    /**
+     * Execute GET request
+     *
+     * @param array<string, scalar|null>|null $params Query parameters
+     */
     public function get(string $url, ?array $params = null): string
     {
         return $this->request('GET', $url, null, $params);

src/Interfaces/HttpTransportInterface.php

@@ -4,10 +4,13 @@
 
 interface HttpTransportInterface
 {
+     /**
+     * @param array<string, mixed>|string|null $params
+     */
     public function request(
         string $method,
         string $url,
         mixed $payload = null,
-        ?array $params = null
+        array|string|null $params = null
     ): string;
 }

src/OauthClient.php

@@ -28,6 +28,9 @@ public function getScopes(bool $limit = false): string
         return $this->request('GET', $url);
     }
 
+    /**
+     * @param list<string> $scopes
+     */
     public function createToken(array $scopes, int $ttl = 3600): string
     {
         $body = [
@@ -56,6 +59,10 @@ public function getCounters(string $period, string $date): string
         return $this->request('GET', $this->url . '/counters/' . $period . '/' . $date);
     }
 
+    
+    /**
+     * @param array<string, mixed>|null $body
+     */
     private function request(string $method, string $url, ?array $body = null): string
     {
         $ch = curl_init();

src/Transports/CurlTransport.php

@@ -14,7 +14,7 @@ public function request(
         string $method,
         string $url,
         mixed $payload = null,
-        ?array $params = null
+        array|string|null $params = null
     ): string {
         if ($params && $method === 'GET') {
             $url .= '?' . http_build_query($params);

tests/Transports/FakeTransport.php

@@ -9,14 +9,17 @@ final class FakeTransport implements HttpTransportInterface
     public ?string $lastMethod = null;
     public ?string $lastUrl = null;
     public mixed $lastPayload = null;
-    public ?array $lastParams = null;
+     /**
+     * @var array<string, mixed>|string|null
+     */
+    public array|string|null $lastParams = null;
     public int $callCount = 0;
 
     public function request(
         string $method,
         string $url,
         mixed $payload = null,
-        ?array $params = null
+        array|string|null $params = null
     ): string {
         $this->callCount++;
         $this->lastMethod = $method;