Fix symbol discovery after class name tokens

Author: LubuSeb

src/Reflection/BetterReflection/SourceLocator/PhpFileCleaner.php

@@ -146,7 +146,10 @@ public function clean(string $contents, int $maxMatches): string
 				if (isset($this->typeConfig[$char])) {
 					$type = $this->typeConfig[$char];
 
-					if (substr($this->contents, $this->index, $type['length']) === $type['name']) {
+					if (
+						substr($this->contents, $this->index, $type['length']) === $type['name']
+						&& $this->match('~.\\b(?<![\\$:>])' . $type['name'] . '\\b~Ais', offset: $this->index - 1)
+					) {
 						if ($maxMatches === 1 && $this->match($type['pattern'], $match, $this->index - 1)) {
 							return $clean . $match[0];
 						}

tests/PHPStan/Reflection/BetterReflection/SourceLocator/SymbolFinderInFilesTest.php

@@ -13,6 +13,15 @@ final class SymbolFinderInFilesTest extends PHPStanTestCase
 	 */
 	public static function dataFindSymbols(): iterable
 	{
+		yield 'class name expressions do not hide later global constants' => [
+			__DIR__ . '/data/symbol-finder/class-name-before-global-constant.php',
+			[
+				[],
+				['symbolfinderclassname', 'classname', 'symbolfinderanonymousclass'],
+				['AFTER_CLASS_NAME', 'AFTER_CLASS_PREFIX', 'AFTER_ANONYMOUS_CLASS'],
+			],
+		];
+
 		yield 'namespaced constants do not leak a class entry' => [
 			__DIR__ . '/data/symbol-finder/namespaced-constants.php',
 			[

tests/PHPStan/Reflection/BetterReflection/SourceLocator/data/symbol-finder/class-name-before-global-constant.php

@@ -0,0 +1,24 @@
+<?php declare(strict_types = 1);
+
+function symbolFinderClassName(): string
+{
+	return \stdClass::class;
+}
+
+const AFTER_CLASS_NAME = 'found';
+
+function className(): string
+{
+	return 'not a class declaration';
+}
+
+const AFTER_CLASS_PREFIX = 'still found';
+
+function symbolFinderAnonymousClass(): object
+{
+	return new class {
+		public const MEMBER = 'not global';
+	};
+}
+
+const AFTER_ANONYMOUS_CLASS = 'also found';

turbo-ext/src/SymbolScan.h

@@ -448,8 +448,12 @@ inline void PhpFileCleaner::clean(zend_long maxMatches, std::string &out)
 			for (const TypeConfig &type : types) {
 				if (type.firstByte != c) continue;
 
-				if (index + type.length <= len && memcmp(contents + index, type.name, type.length) == 0) {
-					if (maxMatches == 1 && prevByteOpensKeyword(index)) {
+				if (index + type.length <= len
+					&& memcmp(contents + index, type.name, type.length) == 0
+					&& prevByteOpensKeyword(index)
+					&& (index + type.length == len || !isWordByte((unsigned char) contents[index + type.length]))
+				) {
+					if (maxMatches == 1) {
 						size_t end;
 						if (matchSpacesAndName(index + type.length, &end)) {
 							clean.append(contents + index - 1, end - (index - 1));

turbo-ext/tests/php-file-cleaner-corpus.php

@@ -88,6 +88,7 @@
 	'short echo tag' => "<?php\nclass A {}\n?>\n<?= 'class NotAClass' ?>\n<?php class B {}\n",
 	'anon class' => "<?php\n\$a = new class extends Foo implements Bar {};\nclass Real {}\n",
 	'property named class' => "<?php\nclass A { public \$class = 1; }\n\$x = Foo::class;\n\$y = \$obj->class;\n",
+	'class name before global constant' => "<?php\nfunction f() { return Foo::class; }\nconst AFTER_CLASS_NAME = 1;\nfunction className() {}\nconst AFTER_CLASS_PREFIX = 2;\n",
 	'function by reference' => "<?php\nfunction &refFn() {}\nfunction plain(): void {}\n",
 	'namespace braces' => "<?php\nnamespace A\\B { class C {} }\nnamespace D { const E = 1; }\n",
 	'no php tag' => "just text, no php at all\n",

turbo-ext/tests/symbol-finder-corpus.php

@@ -64,6 +64,7 @@
 	'interface const' => "<?php\ninterface I { const X = 1; }\n",
 	'enum const' => "<?php\nenum E: string { case A = 'a'; const X = 1; }\n",
 	'global constants' => "<?php\nconst A = 1;\nconst B = 2;\n",
+	'class name before global constant' => "<?php\nfunction f() { return Foo::class; }\nconst AFTER_CLASS_NAME = 1;\nfunction className() {}\nconst AFTER_CLASS_PREFIX = 2;\n",
 	'define' => "<?php\ndefine('FOO', 1);\ndefine(\"N\\\\BAR\", 2);\ndefine('N\\\\\\\\BAZ', 3);\n",
 	'define leading slash' => "<?php\n\\define('SLASHED', 1);\n",
 	'define with variable' => "<?php\ndefine(\$name, 1);\n\$x = 'function evil(';\n",