Order of optional parameters

rules-tests/Symfony73/Rector/Class_/InvokableCommandInputAttributeRector/Fixture/DefaultValue/option_with_array_type.php.inc

@@ -16,8 +16,8 @@ class OptionWithOptionalValue extends Command
 {
     protected function configure(): void
     {
-        $this->addOption('some-array', null, InputOption::VALUE_IS_ARRAY, '', ['third value']);
         $this->addOption('no-default-array', null, InputOption::VALUE_IS_ARRAY);
+        $this->addOption('some-array', null, InputOption::VALUE_IS_ARRAY, '', ['third value']);
     }
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
@@ -43,10 +43,10 @@ use Symfony\Component\Console\Output\OutputInterface;
 class OptionWithOptionalValue
 {
     public function __invoke(
-        #[\Symfony\Component\Console\Attribute\Option(name: 'some-array', mode: InputOption::VALUE_IS_ARRAY)]
-        array $someArray = ['third value'],
         #[\Symfony\Component\Console\Attribute\Option(name: 'no-default-array', mode: InputOption::VALUE_IS_ARRAY)]
-        array $noDefaultArray
+        array $noDefaultArray,
+        #[\Symfony\Component\Console\Attribute\Option(name: 'some-array', mode: InputOption::VALUE_IS_ARRAY)]
+        array $someArray = ['third value']
     ): int
     {
     }

rules-tests/Symfony73/Rector/Class_/InvokableCommandInputAttributeRector/Fixture/with_default_and_without_default_value.php.inc

@@ -0,0 +1,63 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\Class_\InvokableCommandInputAttributeRector\Fixture;
+
+use Symfony\Component\Console\Attribute\AsCommand;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputOption;
+
+#[AsCommand(name: 'some_name')]
+final class WithDefaultAndWithoutDefaultValue extends Command
+{
+    protected function configure()
+    {
+        $this->addOption('option', 'o', InputOption::VALUE_NONE, 'Option description', false);
+        $this->addOption('another', 'a', InputOption::VALUE_REQUIRED, 'No default value');
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $someOption = $input->getOption('option');
+        $another = $input->getOption('another');
+        $output->writeln('Using output too');
+        // ...
+        return 1;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\Class_\InvokableCommandInputAttributeRector\Fixture;
+
+use Symfony\Component\Console\Attribute\AsCommand;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputOption;
+
+#[AsCommand(name: 'some_name')]
+final class WithDefaultAndWithoutDefaultValue
+{
+    public function __invoke(
+        #[\Symfony\Component\Console\Attribute\Option(name: 'another', shortcut: 'a', mode: InputOption::VALUE_REQUIRED, description: 'No default value')]
+        $another,
+        OutputInterface $output,
+        #[\Symfony\Component\Console\Attribute\Option(name: 'option', shortcut: 'o', mode: InputOption::VALUE_NONE, description: 'Option description')]
+        bool $option = false
+    ): int
+    {
+        $someOption = $option;
+        $another = $another;
+        $output->writeln('Using output too');
+        // ...
+        return 1;
+    }
+}
+
+?>

rules/Symfony73/Rector/Class_/InvokableCommandInputAttributeRector.php

@@ -160,7 +160,13 @@ public function refactor(Node $node): ?Class_
 
             $invokeParams = $this->createInvokeParams($node);
 
-            $invokeClassMethod->params = array_merge($invokeParams, [$executeClassMethod->params[1]]);
+            $executeClassMethodParams = array_merge($invokeParams, [$executeClassMethod->params[1]]);
+
+            // Ensure that optional parameters are listed last in the argument list
+            $invokeClassMethod->params = array_merge(
+                array_filter($executeClassMethodParams, fn(Param $param) => is_null($param->default)),
+                array_filter($executeClassMethodParams, fn(Param $param) => !is_null($param->default)),
+            );
 
             // 6. remove parent class
             $node->extends = null;