Add build rule declaring native types from PHPDoc types

build/phpstan.neon

@@ -240,6 +240,7 @@ rules:
 	- PHPStan\Build\MemoizationPropertyRule
 	- PHPStan\Build\OrChainIdenticalComparisonToInArrayRule
 	- PHPStan\Build\NoPhpVersionInjectionInScopeAwareExtensionRule
+	- PHPStan\Build\MissingNativeTypeRule
 
 services:
 	-

src/Analyser/ArgumentsHandler.php

@@ -126,7 +126,6 @@ public function __construct(
 	}
 
 	/**
-	 * @param MethodReflection|FunctionReflection|null $calleeReflection
 	 * @param ParametersAcceptor[] $parametersAcceptors
 	 * @param ParametersAcceptor[]|null $namedArgumentsVariants
 	 * @param callable(Node $node, Scope $scope): void $nodeCallback
@@ -135,7 +134,7 @@ public function __construct(
 	public function processArgs(
 		NodeScopeResolver $nodeScopeResolver,
 		Node\Stmt $stmt,
-		$calleeReflection,
+		MethodReflection|FunctionReflection|null $calleeReflection,
 		?ExtendedMethodReflection $nakedMethodReflection,
 		array $parametersAcceptors,
 		?array $namedArgumentsVariants,
@@ -1096,10 +1095,7 @@ public function processDroppedArgs(
 		}
 	}
 
-	/**
-	 * @param MethodReflection|FunctionReflection|null $calleeReflection
-	 */
-	private function callCallbackImmediately(?ParameterReflection $parameter, ?Type $parameterType, $calleeReflection): bool
+	private function callCallbackImmediately(?ParameterReflection $parameter, ?Type $parameterType, MethodReflection|FunctionReflection|null $calleeReflection): bool
 	{
 		$parameterCallableType = null;
 		if ($parameterType !== null && $calleeReflection instanceof FunctionReflection) {
@@ -1135,10 +1131,7 @@ private function shouldInvalidateCallbackExpressions(?ParameterReflection $param
 		return true;
 	}
 
-	/**
-	 * @param MethodReflection|FunctionReflection|null $calleeReflection
-	 */
-	private function getParameterTypeFromParameterClosureTypeExtension(CallLike $callLike, $calleeReflection, ParameterReflection $parameter, MutatingScope $scope): ?Type
+	private function getParameterTypeFromParameterClosureTypeExtension(CallLike $callLike, MethodReflection|FunctionReflection|null $calleeReflection, ParameterReflection $parameter, MutatingScope $scope): ?Type
 	{
 		if ($callLike instanceof FuncCall && $calleeReflection instanceof FunctionReflection) {
 			foreach ($this->functionParameterClosureTypeExtensions->getAll() as $functionParameterClosureTypeExtension) {
@@ -1176,10 +1169,7 @@ private function getParameterTypeFromParameterClosureTypeExtension(CallLike $cal
 		return null;
 	}
 
-	/**
-	 * @param MethodReflection|FunctionReflection|null $calleeReflection
-	 */
-	private function getParameterOutExtensionsType(CallLike $callLike, $calleeReflection, ParameterReflection $currentParameter, MutatingScope $scope): ?Type
+	private function getParameterOutExtensionsType(CallLike $callLike, MethodReflection|FunctionReflection|null $calleeReflection, ParameterReflection $currentParameter, MutatingScope $scope): ?Type
 	{
 		$paramOutTypes = [];
 		if ($callLike instanceof FuncCall && $calleeReflection instanceof FunctionReflection) {
@@ -1231,12 +1221,9 @@ private function getParameterOutExtensionsType(CallLike $callLike, $calleeReflec
 		return null;
 	}
 
-	/**
-	 * @param FunctionReflection|MethodReflection|null $calleeReflection
-	 */
 	private function resolveClosureThisType(
 		?CallLike $call,
-		$calleeReflection,
+		FunctionReflection|MethodReflection|null $calleeReflection,
 		ParameterReflection $parameter,
 		MutatingScope $scope,
 	): ?Type

src/Analyser/CalledMethodProcessor.php

@@ -130,7 +130,7 @@ public function clearCalledMethodResults(): void
 	 * @param Node[]|Node|scalar|null $node
 	 * @param callable(Node $node, Scope $scope): void $nodeCallback
 	 */
-	private function processNodesForCalledMethod(NodeScopeResolver $nodeScopeResolver, $node, ExpressionResultStorage $storage, string $fileName, MethodReflection $methodReflection, callable $nodeCallback): void
+	private function processNodesForCalledMethod(NodeScopeResolver $nodeScopeResolver, Node|int|float|string|bool|array|null $node, ExpressionResultStorage $storage, string $fileName, MethodReflection $methodReflection, callable $nodeCallback): void
 	{
 		if ($node instanceof Node) {
 			$declaringClass = $methodReflection->getDeclaringClass();

src/Analyser/Error.php

@@ -417,12 +417,9 @@ private function renameFixedFile(string $oldFile, string $newFile): array
 		);
 	}
 
-	/**
-	 * @return mixed
-	 */
 	#[ReturnTypeWillChange]
 	#[Override]
-	public function jsonSerialize()
+	public function jsonSerialize(): mixed
 	{
 		return [
 			'message' => $this->message,

src/Analyser/Ignore/IgnoredError.php

@@ -36,7 +36,7 @@ public static function getIgnoredErrorLabel(array|string $ignoredError): string
 	 * @param mixed[]|string $ignoredError
 	 * @return string Representation of the ignored error
 	 */
-	public static function stringifyPattern($ignoredError): string
+	public static function stringifyPattern(string|array $ignoredError): string
 	{
 		if (!is_array($ignoredError)) {
 			return $ignoredError;

src/Analyser/ImpurePoint.php

@@ -33,10 +33,7 @@ public function getScope(): Scope
 		return $this->scope;
 	}
 
-	/**
-	 * @return Node\Expr|Node\Stmt|VirtualNode
-	 */
-	public function getNode()
+	public function getNode(): Node\Expr|Node\Stmt|VirtualNode
 	{
 		return $this->node;
 	}

src/Analyser/InternalError.php

@@ -87,12 +87,9 @@ public static function decode(array $json): self
 		return new self($json['message'], $json['contextDescription'], $json['trace'], $json['traceAsString'], $json['shouldReportBug']);
 	}
 
-	/**
-	 * @return mixed
-	 */
 	#[ReturnTypeWillChange]
 	#[Override]
-	public function jsonSerialize()
+	public function jsonSerialize(): mixed
 	{
 		return [
 			'message' => $this->message,

src/Analyser/InternalThrowPoint.php

@@ -71,10 +71,7 @@ public function getType(): Type
 		return $this->type;
 	}
 
-	/**
-	 * @return Node\Expr|Node\Stmt
-	 */
-	public function getNode()
+	public function getNode(): Node\Expr|Node\Stmt
 	{
 		return $this->node;
 	}

src/Analyser/MutatingScope.php

@@ -1716,10 +1716,7 @@ public function getTrackedExpressionType(Expr $node): Type
 		return $this->expressionTypes[$this->getNodeKey($node)]->getType();
 	}
 
-	/**
-	 * @param MethodReflection|FunctionReflection|null $reflection
-	 */
-	public function pushInFunctionCall($reflection, ?ParameterReflection $parameter, bool $rememberTypes): self
+	public function pushInFunctionCall(MethodReflection|FunctionReflection|null $reflection, ?ParameterReflection $parameter, bool $rememberTypes): self
 	{
 		$stack = $this->inFunctionCallsStack;
 		$stack[] = [$reflection, $parameter];

src/Analyser/NodeCallbackScope.php

@@ -271,10 +271,7 @@ private function preprocessScope(MutatingScope $scope): Scope
 		return $scope;
 	}
 
-	/**
-	 * @param MethodReflection|FunctionReflection|null $reflection
-	 */
-	public function pushInFunctionCall($reflection, ?ParameterReflection $parameter, bool $rememberTypes): self
+	public function pushInFunctionCall(MethodReflection|FunctionReflection|null $reflection, ?ParameterReflection $parameter, bool $rememberTypes): self
 	{
 		/** @var self $scope */
 		$scope = parent::pushInFunctionCall($reflection, $parameter, $rememberTypes);

src/Analyser/ResultCache/ResultCacheManager.php

@@ -1853,10 +1853,9 @@ private function encodeValueDependencies(array $valueDependencies, ResultCachePa
 	}
 
 	/**
-	 * @param mixed $encoded
 	 * @return list<array{extensionClass: string, storedKey: string, value: string, analysis: list<string>, declarations: list<string>}>
 	 */
-	private function decodeValueDependencies($encoded, ResultCachePathTransformer $transformer): array
+	private function decodeValueDependencies(mixed $encoded, ResultCachePathTransformer $transformer): array
 	{
 		if (!is_array($encoded) || !is_array($encoded['paths'] ?? null) || !is_array($encoded['entries'] ?? null)) {
 			throw new RuntimeException('The value dependencies are malformed.');
@@ -2326,10 +2325,9 @@ private function encodeDependencyGraph(array $invertedDependencies, ResultCacheP
 	}
 
 	/**
-	 * @param mixed $dependencyGraph
 	 * @return array<string, array{fileHash: string, fileStat?: string, dependentFiles: list<string>, usedTraitDependentFiles?: list<string>}>
 	 */
-	private function decodeDependencyGraph($dependencyGraph, ResultCachePathTransformer $transformer): array
+	private function decodeDependencyGraph(mixed $dependencyGraph, ResultCachePathTransformer $transformer): array
 	{
 		if (!is_array($dependencyGraph) || !is_array($dependencyGraph['paths'] ?? null) || !is_array($dependencyGraph['entries'] ?? null)) {
 			throw new RuntimeException('The dependency graph is malformed.');

src/Analyser/StmtHandler/TraitUseHandler.php

@@ -170,7 +170,7 @@ private function processTraitUse(NodeScopeResolver $nodeScopeResolver, Node\Stmt
 	 * @param callable(Node $node, Scope $scope): void $nodeCallback
 	 * @param list<Dependencies|null> $dependencies what the walked trait statements depend on
 	 */
-	private function processNodesForTraitUse(NodeScopeResolver $nodeScopeResolver, $node, ClassReflection $traitReflection, MutatingScope $scope, ExpressionResultStorage $storage, array $adaptations, callable $nodeCallback, array &$dependencies): void
+	private function processNodesForTraitUse(NodeScopeResolver $nodeScopeResolver, Node|int|float|string|bool|array|null $node, ClassReflection $traitReflection, MutatingScope $scope, ExpressionResultStorage $storage, array $adaptations, callable $nodeCallback, array &$dependencies): void
 	{
 		if ($node instanceof Node) {
 			if ($node instanceof Node\Stmt\Trait_ && $traitReflection->getName() === (string) $node->namespacedName && $traitReflection->getNativeReflection()->getStartLine() === $node->getStartLine()) {

src/Analyser/ThrowPoint.php

@@ -55,10 +55,7 @@ public function getType(): Type
 		return $this->type;
 	}
 
-	/**
-	 * @return Node\Expr|Node\Stmt
-	 */
-	public function getNode()
+	public function getNode(): Node\Expr|Node\Stmt
 	{
 		return $this->node;
 	}

src/Analyser/VariableFlow.php

@@ -83,8 +83,7 @@ public static function arrow(ArrowFunction $arrow, ?self $body, ?self $outputs):
 		return new VariableControlFlow(self::ARROW, [$body, $outputs], arrow: $arrow);
 	}
 
-	/** @param int|string|null $offset */
-	public static function read(string $name, ?int $targetId = null, bool $container = false, $offset = null): ?self
+	public static function read(string $name, ?int $targetId = null, bool $container = false, int|string|null $offset = null): ?self
 	{
 		if ($name === 'this' || in_array($name, Scope::SUPERGLOBAL_VARIABLES, true)) {
 			return null;

src/Analyser/VariableLivenessResolver.php

@@ -489,8 +489,7 @@ private function passBindingProbes(array $next, string $name, ?VariableWrite $wr
 		return $next;
 	}
 
-	/** @param int|string $offset */
-	private static function offsetKey($offset): string
+	private static function offsetKey(int|string $offset): string
 	{
 		return (is_int($offset) ? 'i:' : 's:') . $offset;
 	}

src/Analyser/VariableWriteOffset.php

@@ -17,10 +17,7 @@
 final class VariableWriteOffset
 {
 
-	/**
-	 * @return int|string|null
-	 */
-	public static function fromType(Type $dimType)
+	public static function fromType(Type $dimType): int|string|null
 	{
 		$keyType = $dimType->toArrayKey();
 		if (!$keyType->isConstantScalarValue()->yes()) {

src/Cache/ArenaCache.php

@@ -61,20 +61,16 @@ public static function lookup(string $key): mixed
 
 	/**
 	 * Publishes the value unless the key already has one - the first publish wins.
-	 *
-	 * @param mixed $value
 	 */
-	public static function publish(string $key, $value): void
+	public static function publish(string $key, mixed $value): void
 	{
 	}
 
 	/**
 	 * Publishes the value in place of the one the key has - for a value just
 	 * computed because the published one was out of date.
-	 *
-	 * @param mixed $value
 	 */
-	public static function replace(string $key, $value): void
+	public static function replace(string $key, mixed $value): void
 	{
 	}
 

src/Cache/Cache.php

@@ -20,10 +20,8 @@ public function __construct(
 
 	/**
 	 * @param non-empty-string $key
-	 *
-	 * @return mixed|null
 	 */
-	public function load(string $key, string $variableKey)
+	public function load(string $key, string $variableKey): mixed
 	{
 		// Every entry is also shared across the run's parallel workers
 		// through the shared-memory arena (turbo extension only; the seam is
@@ -58,10 +56,8 @@ public function load(string $key, string $variableKey)
 
 	/**
 	 * @param non-empty-string $key
-	 *
-	 * @param mixed $data
 	 */
-	public function save(string $key, string $variableKey, $data): void
+	public function save(string $key, string $variableKey, mixed $data): void
 	{
 		$this->storage->save($key, $variableKey, $data);
 

src/Cache/CacheItem.php

@@ -5,10 +5,7 @@
 final class CacheItem
 {
 
-	/**
-	 * @param mixed $data
-	 */
-	public function __construct(private string $variableKey, private $data)
+	public function __construct(private string $variableKey, private mixed $data)
 	{
 	}
 
@@ -17,10 +14,7 @@ public function isVariableKeyValid(string $variableKey): bool
 		return $this->variableKey === $variableKey;
 	}
 
-	/**
-	 * @return mixed
-	 */
-	public function getData()
+	public function getData(): mixed
 	{
 		return $this->data;
 	}

src/Cache/CacheStorage.php

@@ -14,9 +14,7 @@ public function load(string $key, string $variableKey);
 
 	/**
 	 * @param non-empty-string $key
-	 *
-	 * @param mixed $data
 	 */
-	public function save(string $key, string $variableKey, $data): void;
+	public function save(string $key, string $variableKey, mixed $data): void;
 
 }

src/Cache/FileCacheStorage.php

@@ -53,10 +53,7 @@ public function __construct(private string $directory)
 	{
 	}
 
-	/**
-	 * @return mixed|null
-	 */
-	public function load(string $key, string $variableKey)
+	public function load(string $key, string $variableKey): mixed
 	{
 		[,, $filePath] = $this->getFilePaths($key);
 
@@ -88,10 +85,9 @@ public function load(string $key, string $variableKey)
 	}
 
 	/**
-	 * @param mixed $data
 	 * @throws DirectoryCreatorException
 	 */
-	public function save(string $key, string $variableKey, $data): void
+	public function save(string $key, string $variableKey, mixed $data): void
 	{
 		[$firstDirectory, $secondDirectory, $path] = $this->getFilePaths($key);
 		DirectoryCreator::ensureDirectoryExists($this->directory, 0777);

src/Cache/MemoryCacheStorage.php

@@ -8,10 +8,7 @@ final class MemoryCacheStorage implements CacheStorage
 	/** @var array<string, CacheItem> */
 	private array $storage = [];
 
-	/**
-	 * @return mixed|null
-	 */
-	public function load(string $key, string $variableKey)
+	public function load(string $key, string $variableKey): mixed
 	{
 		if (!isset($this->storage[$key])) {
 			return null;
@@ -25,10 +22,7 @@ public function load(string $key, string $variableKey)
 		return $item->getData();
 	}
 
-	/**
-	 * @param mixed $data
-	 */
-	public function save(string $key, string $variableKey, $data): void
+	public function save(string $key, string $variableKey, mixed $data): void
 	{
 		$this->storage[$key] = new CacheItem($variableKey, $data);
 	}