build: add PHPStan static analysis at level 5

Add phpstan/phpstan ^2 at level 5 with a baseline of 13 errors (all
interface method gaps on ClientInterface and ConnectorInterface).
Globally ignore new.static (deliberate polymorphism pattern).

Add PHPStan step to GitHub Actions CI and a Makefile with lint/test
targets.

Fixes found by PHPStan:
- Project::systemInformation() return type missing false case
- RestoreOptions: redundant @return PHPDocs conflicting with native
  static return type
- LogItem: static:: to self:: for private method
- Tree::getBlob(): always-true instanceof and unreachable code
- Resolver: always-true explode() assignment in condition
- ResourceWithReferences: always-true getResponse() check
- Result::getEntity(): redundant isset() on initialized property
- Subscription::create(): new self() vs new static() return mismatch
- EnvironmentTest: incorrect @var annotation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: pjcdawkins

.github/workflows/ci.yml

@@ -26,5 +26,8 @@ jobs:
             - name: Install dependencies
               run: composer install --no-progress --no-suggest --no-interaction
 
+            - name: Run PHPStan
+              run: ./vendor/bin/phpstan analyse
+
             - name: Run PHPUnit tests
               run: ./vendor/bin/phpunit

Makefile

@@ -0,0 +1,9 @@
+.PHONY: lint lint-phpstan test
+
+lint: lint-phpstan
+
+lint-phpstan:
+	./vendor/bin/phpstan analyse
+
+test:
+	./vendor/bin/phpunit

composer.json

@@ -12,7 +12,8 @@
     },
     "require-dev": {
         "phpunit/phpunit": "^11",
-        "symplify/easy-coding-standard": "^12.3"
+        "symplify/easy-coding-standard": "^12.3",
+        "phpstan/phpstan": "^2"
     },
     "autoload": {
         "psr-4": {

composer.lock

@@ -0,0 +1,79 @@
+parameters:
+	ignoreErrors:
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:get\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/Model/Activity.php
+
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:patch\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/Model/Organization/Address.php
+
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:get\(\)\.$#'
+			identifier: method.notFound
+			count: 2
+			path: src/Model/Organization/Organization.php
+
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:patch\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/Model/Organization/Organization.php
+
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:patch\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/Model/Organization/Profile.php
+
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:get\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/Model/Ref/Resolver.php
+
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:get\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/Model/SetupOptions.php
+
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:patch\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/Model/Team/Team.php
+
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:patch\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/Model/UserAccess/ProjectUserAccess.php
+
+		-
+			message: '#^Call to an undefined method GuzzleHttp\\ClientInterface\:\:post\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/PlatformClient.php
+
+		-
+			message: '#^Call to an undefined method Platformsh\\Client\\Connection\\ConnectorInterface\:\:getAccessToken\(\)\.$#'
+			identifier: method.notFound
+			count: 1
+			path: src/PlatformClient.php
+
+		-
+			message: '#^Call to an undefined method Platformsh\\Client\\Connection\\ConnectorInterface\:\:getAccountsEndpoint\(\)\.$#'
+			identifier: method.notFound
+			count: 2
+			path: src/PlatformClient.php
+
+		-
+			message: '#^Call to an undefined method Platformsh\\Client\\Connection\\ConnectorInterface\:\:getConfig\(\)\.$#'
+			identifier: method.notFound
+			count: 3
+			path: src/PlatformClient.php

phpstan-baseline.neon

@@ -0,0 +1,12 @@
+includes:
+	- phpstan-baseline.neon
+
+parameters:
+	level: 5
+	paths:
+		- src
+		- tests
+	ignoreErrors:
+		# Deliberate pattern: resource classes use new static() for polymorphism.
+		-
+			identifier: new.static

phpstan.neon

@@ -34,7 +34,7 @@ public function __toString()
      */
     public static function singleFromJson(string $str): self|false
     {
-        $data = static::decode($str);
+        $data = self::decode($str);
         if (isset($data['data']['timestamp'], $data['data']['message'])) {
             $id = isset($data['_id']) ? (string) $data['_id'] : '';
             return new static($data['data']['timestamp'], $data['data']['message'], $id);
@@ -43,7 +43,7 @@ public static function singleFromJson(string $str): self|false
     }
 
     /**
-     * @return static[]
+     * @return self[]
      *@deprecated use LogItem::multipleFromJsonStreamWithSeal() instead
      */
     public static function multipleFromJsonStream(string $str): array
@@ -77,7 +77,7 @@ public static function multipleFromJsonStreamWithSeal(string $str): array
             if ($line === '') {
                 continue;
             }
-            $data = static::decode($line);
+            $data = self::decode($line);
             if (is_array($data)) {
                 if (! empty($data['seal'])) {
                     $seal = true;

src/Model/ActivityLog/LogItem.php

@@ -16,45 +16,30 @@ class RestoreOptions
 
     private ?string $resourcesInit;
 
-    /**
-     * @return RestoreOptions
-     */
     public function setEnvironmentName(?string $environmentName): static
     {
         $this->environmentName = $environmentName;
         return $this;
     }
 
-    /**
-     * @return RestoreOptions
-     */
     public function setBranchFrom(?string $branchFrom): static
     {
         $this->branchFrom = $branchFrom;
         return $this;
     }
 
-    /**
-     * @return RestoreOptions
-     */
     public function setRestoreCode(?bool $restoreCode): static
     {
         $this->restoreCode = $restoreCode;
         return $this;
     }
 
-    /**
-     * @return RestoreOptions
-     */
     public function setRestoreResources(?bool $restoreResources): static
     {
         $this->restoreResources = $restoreResources;
         return $this;
     }
 
-    /**
-     * @return RestoreOptions
-     */
     public function setResourcesInit(?string $init): static
     {
         $this->resourcesInit = $init;

src/Model/Backups/RestoreOptions.php

@@ -68,14 +68,11 @@ public function getBlob(string $path): false|Blob
         if ($object === false) {
             return false;
         }
-        if ($object instanceof Blob) {
-            return $object;
-        }
         if ($object instanceof self) {
             throw new GitObjectTypeException('The requested file is a directory', $path);
         }
 
-        return false;
+        return $object;
     }
 
     /**

src/Model/Git/Tree.php

@@ -508,7 +508,7 @@ public function clearBuildCache(): Result
     /**
      * Returns system information about the project, e.g. the API version.
      */
-    public function systemInformation(): System
+    public function systemInformation(): System|false
     {
         return System::get($this->getLink('#system'), '', $this->client);
     }