Potential fix

Author: florisluiten

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/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;