PhpDocParser: support variadic parameters in @param tag

Author: JanTvrdik

doc/grammars/phpdoc-param.peg

@@ -1,9 +1,12 @@
 PhpDocParam
-  = AnnotationName Type ParameterName Description?
+  = AnnotationName Type IsVariadic? ParameterName Description?
 
 AnnotationName
   = '@param'
 
+IsVariaric
+  = '...'
+
 ParameterName
   = '$' [a-zA-Z_\127-\255][a-zA-Z0-9_\127-\255]*
 

src/Ast/PhpDoc/ParamTagValueNode.php

@@ -10,23 +10,28 @@ class ParamTagValueNode implements PhpDocTagValueNode
 	/** @var TypeNode */
 	public $type;
 
+	/** @var bool */
+	public $isVariadic;
+
 	/** @var string (may be empty) */
 	public $parameterName;
 
 	/** @var string (may be empty) */
 	public $description;
 
-	public function __construct(TypeNode $type, string $parameterName, string $description)
+	public function __construct(TypeNode $type, bool $isVariadic, string $parameterName, string $description)
 	{
 		$this->type = $type;
+		$this->isVariadic = $isVariadic;
 		$this->parameterName = $parameterName;
 		$this->description = $description;
 	}
 
 
 	public function __toString(): string
 	{
-		return trim("{$this->type} {$this->parameterName} {$this->description}");
+		$variadic = $this->isVariadic ? '...' : '';
+		return trim("{$this->type} {$variadic}{$this->parameterName} {$this->description}");
 	}
 
 }

src/Parser/PhpDocParser.php

@@ -122,9 +122,10 @@ private function parseTagValue(TokenIterator $tokens, string $tag): Ast\PhpDoc\P
 	private function parseParamTagValue(TokenIterator $tokens): Ast\PhpDoc\ParamTagValueNode
 	{
 		$type = $this->typeParser->parse($tokens);
+		$isVariadic = $tokens->tryConsumeTokenType(Lexer::TOKEN_VARIADIC);
 		$parameterName = $this->parseRequiredVariableName($tokens);
 		$description = $this->parseOptionalDescription($tokens);
-		return new Ast\PhpDoc\ParamTagValueNode($type, $parameterName, $description);
+		return new Ast\PhpDoc\ParamTagValueNode($type, $isVariadic, $parameterName, $description);
 	}
 
 

tests/PHPStan/Parser/PhpDocParserTest.php

@@ -95,6 +95,7 @@ public function provideParseData(): array
 						'@param',
 						new ParamTagValueNode(
 							new IdentifierTypeNode('Foo'),
+							false,
 							'$foo',
 							''
 						)
@@ -127,6 +128,22 @@ public function provideParseData(): array
 						'@param',
 						new ParamTagValueNode(
 							new IdentifierTypeNode('Foo'),
+							false,
+							'$foo',
+							'optional description '
+						)
+					),
+				]),
+			],
+			[
+				'/** @param Foo ...$foo optional description */',
+				new PhpDocNode([
+					new PhpDocTextNode(' '),
+					new PhpDocTagNode(
+						'@param',
+						new ParamTagValueNode(
+							new IdentifierTypeNode('Foo'),
+							true,
 							'$foo',
 							'optional description '
 						)
@@ -289,6 +306,7 @@ public function provideParseData(): array
 						'@param',
 						new ParamTagValueNode(
 							new IdentifierTypeNode('Foo'),
+							false,
 							'$foo',
 							'1st multi world description'
 						)
@@ -298,6 +316,7 @@ public function provideParseData(): array
 						'@param',
 						new ParamTagValueNode(
 							new IdentifierTypeNode('Bar'),
+							false,
 							'$bar',
 							'2nd multi world description'
 						)
@@ -317,6 +336,7 @@ public function provideParseData(): array
 						'@param',
 						new ParamTagValueNode(
 							new IdentifierTypeNode('Foo'),
+							false,
 							'$foo',
 							'1st multi world description'
 						)
@@ -326,6 +346,7 @@ public function provideParseData(): array
 						'@param',
 						new ParamTagValueNode(
 							new IdentifierTypeNode('Bar'),
+							false,
 							'$bar',
 							'2nd multi world description'
 						)