Cover the loader shapes the reverted autoloader attempts broke

#6292 reverted five commits because the attempts fixed one use case and broke another, and only some of those breakages had an e2e project. These two do not test PHPStan's current behaviour so much as pin the shapes a future attempt at phpstan/phpstan#14976 and phpstan/phpstan#14988 must not regress. class-alias-loader uses the real typo3/class-alias-loader: it replaces Composer's loader with its own wrapper, resolves a legacy name through a class alias map, and an IDE-only stub in the analysed paths declares that name as a plain class. The alias has to win. 2.2.9 returns the stub and reports a missing method. robot-loader registers a Nette RobotLoader over a directory Composer does not map, so only that loader can find the class. RobotLoader locks and writes a cache file while resolving, which fails inside the file-read trap: on 598faaa the analysis dies in RobotLoader::acquireLock() with "Unable to create file '<tmp>/....php.lock'". Both pass on this commit. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ondrejmirtes · Aug 27, 2026 · Aug 27, 2026 · Aug 27, 2026 · 7132109ddcf384733194bf37778890a402bad52d
commit 7132109ddcf384733194bf37778890a402bad52d
diff --git a/.github/workflows/e2e-tests.yml b/.github/workflows/e2e-tests.yml
@@ -141,6 +141,14 @@ jobs:
               cd e2e/bug-12972c
               composer install
               ../../bin/phpstan analyze
+          - script: |
+              cd e2e/class-alias-loader
+              composer install
+              ../../bin/phpstan analyze
+          - script: |
+              cd e2e/robot-loader
+              composer install
+              ../../bin/phpstan analyze
           - script: |
               cd e2e/bug-15102b
               ../../bin/phpstan analyze

diff --git a/e2e/class-alias-loader/.gitignore b/e2e/class-alias-loader/.gitignore
@@ -0,0 +1,2 @@
+/vendor/
+composer.lock
diff --git a/e2e/class-alias-loader/alias-map.php b/e2e/class-alias-loader/alias-map.php
@@ -0,0 +1,7 @@
+<?php declare(strict_types = 1);
+
+// typo3/class-alias-loader replaces Composer's class loader with its own wrapper and
+// resolves these names through class_alias() at runtime.
+return [
+	'AliasLoaderE2e\\LegacyName' => 'AliasLoaderE2e\\ModernName',
+];
diff --git a/e2e/class-alias-loader/composer.json b/e2e/class-alias-loader/composer.json
@@ -0,0 +1,23 @@
+{
+	"require": {
+		"typo3/class-alias-loader": "^1.2"
+	},
+	"autoload": {
+		"psr-4": {
+			"AliasLoaderE2e\\": "src/"
+		}
+	},
+	"extra": {
+		"typo3/class-alias-loader": {
+			"class-alias-maps": [
+				"alias-map.php"
+			],
+			"always-add-alias-loader": true
+		}
+	},
+	"config": {
+		"allow-plugins": {
+			"typo3/class-alias-loader": true
+		}
+	}
+}
diff --git a/e2e/class-alias-loader/phpstan.dist.neon b/e2e/class-alias-loader/phpstan.dist.neon
@@ -0,0 +1,7 @@
+parameters:
+	level: 8
+	bootstrapFiles:
+		- vendor/autoload.php
+	paths:
+		- test.php
+		- stub
diff --git a/e2e/class-alias-loader/src/ModernName.php b/e2e/class-alias-loader/src/ModernName.php
@@ -0,0 +1,13 @@
+<?php declare(strict_types = 1);
+
+namespace AliasLoaderE2e;
+
+class ModernName
+{
+
+	public function doFoo(): int
+	{
+		return 1;
+	}
+
+}
diff --git a/e2e/class-alias-loader/stub/LegacyName.php b/e2e/class-alias-loader/stub/LegacyName.php
@@ -0,0 +1,13 @@
+<?php declare(strict_types = 1);
+
+namespace AliasLoaderE2e;
+
+// An IDE-only stub, never loaded at runtime: it declares the legacy name as a plain class so
+// editors resolve it. At runtime typo3/class-alias-loader makes the name a class_alias to
+// ModernName, which is the declaration PHPStan has to use - it has doFoo(), this does not.
+die('never loaded at runtime');
+
+class LegacyName
+{
+
+}
diff --git a/e2e/class-alias-loader/test.php b/e2e/class-alias-loader/test.php
@@ -0,0 +1,7 @@
+<?php declare(strict_types = 1);
+
+use AliasLoaderE2e\LegacyName;
+
+function (LegacyName $legacy): int {
+	return $legacy->doFoo();
+};
diff --git a/e2e/robot-loader/.gitignore b/e2e/robot-loader/.gitignore
@@ -0,0 +1,2 @@
+/vendor/
+composer.lock
diff --git a/e2e/robot-loader/bootstrap.php b/e2e/robot-loader/bootstrap.php
@@ -0,0 +1,8 @@
+<?php declare(strict_types = 1);
+
+require __DIR__ . '/vendor/autoload.php';
+
+$loader = new Nette\Loaders\RobotLoader();
+$loader->addDirectory(__DIR__ . '/classes');
+$loader->setTempDirectory(sys_get_temp_dir() . '/phpstan-e2e-robot-loader');
+$loader->register();
diff --git a/e2e/robot-loader/classes/Discovered.php b/e2e/robot-loader/classes/Discovered.php
@@ -0,0 +1,15 @@
+<?php declare(strict_types = 1);
+
+namespace RobotLoaderE2e;
+
+// Deliberately outside any Composer autoload mapping: only RobotLoader can find this class,
+// by scanning the directory. PHPStan has to consult the bootstrap-registered loader for it.
+class Discovered
+{
+
+	public function doFoo(): int
+	{
+		return 1;
+	}
+
+}
diff --git a/e2e/robot-loader/composer.json b/e2e/robot-loader/composer.json
@@ -0,0 +1,5 @@
+{
+	"require": {
+		"nette/robot-loader": "^4.0"
+	}
+}
diff --git a/e2e/robot-loader/phpstan.dist.neon b/e2e/robot-loader/phpstan.dist.neon
@@ -0,0 +1,6 @@
+parameters:
+	level: 8
+	bootstrapFiles:
+		- bootstrap.php
+	paths:
+		- test.php
diff --git a/e2e/robot-loader/test.php b/e2e/robot-loader/test.php
@@ -0,0 +1,7 @@
+<?php declare(strict_types = 1);
+
+use RobotLoaderE2e\Discovered;
+
+function (Discovered $discovered): int {
+	return $discovered->doFoo();
+};