[Console] Fix issue with reserved keyword "command" as argument name

src/Symfony/Component/Console/CHANGELOG.md

@@ -5,6 +5,7 @@ CHANGELOG
 ---
 
  * Add `ArgvInput::getRawTokens()`
+ * Throws `InvalidArgumentException` if an argument was created with the reserved keyword `command` as name and is not the `command name` argument.
 
 7.0
 ---

src/Symfony/Component/Console/Command/Command.php

@@ -407,7 +407,7 @@ public function getNativeDefinition(): InputDefinition
      *
      * @return $this
      *
-     * @throws InvalidArgumentException When argument mode is not valid
+     * @throws InvalidArgumentException When argument name is 'command' or mode is not valid
      */
     public function addArgument(string $name, ?int $mode = null, string $description = '', mixed $default = null, array|\Closure $suggestedValues = []): static
     {

src/Symfony/Component/Console/Input/InputArgument.php

@@ -50,7 +50,7 @@ class InputArgument
      * @param string|bool|int|float|array|null                                              $default         The default value (for self::OPTIONAL mode only)
      * @param array|\Closure(CompletionInput,CompletionSuggestions):list<string|Suggestion> $suggestedValues The values used for input completion
      *
-     * @throws InvalidArgumentException When argument mode is not valid
+     * @throws InvalidArgumentException When argument name is 'command' or mode is not valid
      */
     public function __construct(
         private string $name,
@@ -59,6 +59,9 @@ public function __construct(
         string|bool|int|float|array|null $default = null,
         private \Closure|array $suggestedValues = [],
     ) {
+        if ('command' === $name && 'The command to execute' !== $description) {
+            throw new InvalidArgumentException('Reserved keyword "command" was used as argument name.');
+        }
         if (null === $mode) {
             $mode = self::OPTIONAL;
         } elseif ($mode >= (self::IS_ARRAY << 1) || $mode < 1) {

src/Symfony/Component/Console/Tests/ApplicationTest.php

@@ -1303,7 +1303,7 @@ public function testAddingAlreadySetDefinitionElementData($def)
     public static function getAddingAlreadySetDefinitionElementData(): array
     {
         return [
-            [new InputArgument('command', InputArgument::REQUIRED)],
+            [new InputArgument('command', InputArgument::REQUIRED, 'The command to execute')],
             [new InputOption('quiet', '', InputOption::VALUE_NONE)],
             [new InputOption('query', 'q', InputOption::VALUE_NONE)],
         ];

src/Symfony/Component/Console/Tests/Input/ArgvInputTest.php

@@ -316,7 +316,7 @@ public static function provideInvalidInput(): array
             ],
             [
                 ['cli.php', 'acme:foo', 'bar'],
-                new InputDefinition([new InputArgument('command', InputArgument::REQUIRED)]),
+                new InputDefinition([new InputArgument('command', InputArgument::REQUIRED, 'The command to execute')]),
                 'No arguments expected for "acme:foo" command, got "bar"',
             ],
             [

src/Symfony/Component/Console/Tests/Input/InputArgumentTest.php

@@ -15,6 +15,7 @@
 use Symfony\Component\Console\Completion\CompletionInput;
 use Symfony\Component\Console\Completion\CompletionSuggestions;
 use Symfony\Component\Console\Completion\Suggestion;
+use Symfony\Component\Console\Exception\InvalidArgumentException;
 use Symfony\Component\Console\Exception\LogicException;
 use Symfony\Component\Console\Input\InputArgument;
 
@@ -26,6 +27,13 @@ public function testConstructor()
         $this->assertEquals('foo', $argument->getName(), '__construct() takes a name as its first argument');
     }
 
+    public function testReservedArgumentName()
+    {
+        self::expectException(InvalidArgumentException::class);
+        self::expectExceptionMessage('Reserved keyword "command" was used as argument name.');
+        new InputArgument('command');
+    }
+
     public function testModes()
     {
         $argument = new InputArgument('foo');

src/Symfony/Component/Console/Tests/Tester/CommandTesterTest.php

@@ -16,6 +16,7 @@
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\HelperSet;
 use Symfony\Component\Console\Helper\QuestionHelper;
+use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Output\Output;
 use Symfony\Component\Console\Question\ChoiceQuestion;
 use Symfony\Component\Console\Question\Question;
@@ -30,7 +31,7 @@ class CommandTesterTest extends TestCase
     protected function setUp(): void
     {
         $this->command = new Command('foo');
-        $this->command->addArgument('command');
+        $this->command->addArgument('command', InputArgument::REQUIRED, 'The command to execute');
         $this->command->addArgument('foo');
         $this->command->setCode(function ($input, $output) { $output->writeln('foo'); });
 
@@ -231,7 +232,7 @@ public function testSymfonyStyleCommandWithInputs()
     public function testErrorOutput()
     {
         $command = new Command('foo');
-        $command->addArgument('command');
+        $command->addArgument('command', InputArgument::REQUIRED, 'The command to execute');
         $command->addArgument('foo');
         $command->setCode(function ($input, $output) {
             $output->getErrorOutput()->write('foo');