feat(console): support variadic argument (#1572)

Author: Bapawe

packages/console/src/ConsoleInputBuilder.php

@@ -26,6 +26,18 @@ public function build(): array
         $argumentDefinitions = $this->command->getArgumentDefinitions();
 
         foreach ($argumentDefinitions as $argumentDefinition) {
+            if ($argumentDefinition->isVariadic) {
+                $arguments = $this->argumentBag->findForVariadicArgument($argumentDefinition);
+
+                if ($arguments === []) {
+                    $invalidArguments[] = $argumentDefinition;
+                } else {
+                    $validArguments = [...$validArguments, ...$arguments];
+                }
+
+                continue;
+            }
+
             $argument = $argumentDefinition->type === 'array'
                 ? $this->argumentBag->findArrayFor($argumentDefinition)
                 : $this->argumentBag->findFor($argumentDefinition);

packages/console/src/Input/ConsoleArgumentBag.php

@@ -129,6 +129,26 @@ public function findArrayFor(ConsoleArgumentDefinition $argumentDefinition): Con
         );
     }
 
+    /**
+     * @return list<ConsoleInputArgument>
+     */
+    public function findForVariadicArgument(ConsoleArgumentDefinition $argumentDefinition): array
+    {
+        $arguments = [];
+
+        foreach ($this->arguments as $argument) {
+            if ($argument->position >= $argumentDefinition->position) {
+                $arguments[] = new ConsoleInputArgument(
+                    name: $argumentDefinition->name,
+                    position: $argument->position,
+                    value: $this->resolveArgumentValue($argumentDefinition, $argument)->value,
+                );
+            }
+        }
+
+        return $arguments;
+    }
+
     public function add(ConsoleInputArgument $argument): self
     {
         $this->arguments[] = $argument;

packages/console/src/Input/ConsoleArgumentDefinition.php

@@ -18,6 +18,7 @@ public function __construct(
         public mixed $default,
         public bool $hasDefault,
         public int $position,
+        public bool $isVariadic = false,
         public ?string $description = null,
         public array $aliases = [],
         public ?string $help = null,
@@ -36,6 +37,7 @@ public static function fromParameter(ParameterReflector $parameter): ConsoleArgu
             default: $default,
             hasDefault: $parameter->isDefaultValueAvailable(),
             position: $parameter->getPosition(),
+            isVariadic: $parameter->isVariadic(),
             description: $attribute?->description,
             aliases: $attribute->aliases ?? [],
             help: $attribute?->help,

tests/Integration/Console/ConsoleArgumentBagTest.php

@@ -190,4 +190,14 @@ public function test_name_mapping(): void
             ->assertSee('foo')
             ->assertSee('true');
     }
+
+    #[TestWith(['variadic-string-argument foo bar baz', '["foo","bar","baz"]'])]
+    #[TestWith(['variadic-integer-argument 1 2 3', '[1,2,3]'])]
+    #[TestWith(['variadic-backed-enum-argument a b c', '["a","b","c"]'])]
+    public function test_variadic_argument(string $command, string $jsonOutput): void
+    {
+        $this->console
+            ->call($command)
+            ->assertSee($jsonOutput);
+    }
 }

tests/Integration/Console/Fixtures/VariadicCommand.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Tempest\Integration\Console\Fixtures;
+
+use Tempest\Console\ConsoleCommand;
+use Tempest\Console\HasConsole;
+
+final readonly class VariadicCommand
+{
+    use HasConsole;
+
+    #[ConsoleCommand('variadic-string-argument')]
+    public function string(string ...$input): void
+    {
+        $this->writeln(json_encode($input));
+    }
+
+    #[ConsoleCommand('variadic-integer-argument')]
+    public function integer(int ...$input): void
+    {
+        $this->writeln(json_encode($input));
+    }
+
+    #[ConsoleCommand('variadic-backed-enum-argument')]
+    public function backedEnum(TestStringEnum ...$input): void
+    {
+        $this->writeln(json_encode($input));
+    }
+}