[console] [7.3] Add implicit boolean type

Author: TomasVotruba

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

@@ -47,7 +47,7 @@ final class SomeCommand
 {
     public function __invoke(#[\Symfony\Component\Console\Attribute\Argument(name: 'argument', description: 'Argument description')]
     string $argument, #[\Symfony\Component\Console\Attribute\Option(name: 'option', shortcut: 'o', mode: InputOption::VALUE_NONE, description: 'Option description')]
-    $option): int
+    bool $option = false): int
     {
         $someArgument = $argument;
         $someOption = $option;

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

@@ -48,7 +48,7 @@ final class SomeCommandWithMethodChaining
 {
     public function __invoke(#[\Symfony\Component\Console\Attribute\Argument(name: 'argument', description: 'Argument description')]
     string $argument, #[\Symfony\Component\Console\Attribute\Option(name: 'option', shortcut: 'o', mode: InputOption::VALUE_NONE, description: 'Option description')]
-    $option): int
+    bool $option = false): int
     {
         $someArgument = $argument;
         $someOption = $option;

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

@@ -53,7 +53,7 @@ final class SomeCommandWithSetHelp
 
     public function __invoke(#[\Symfony\Component\Console\Attribute\Argument(name: 'argument', description: 'Argument description')]
     string $argument, #[\Symfony\Component\Console\Attribute\Option(name: 'option', shortcut: 'o', mode: InputOption::VALUE_NONE, description: 'Option description')]
-    $option): int
+    bool $option = false): int
     {
         $someArgument = $argument;
         $someOption = $option;

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

@@ -57,9 +57,9 @@ final class WithMultipleArgumentsOptionsFluent
         #[\Symfony\Component\Console\Attribute\Argument(name: 'argument2', description: 'Argument2 description')]
         string $argument2,
         #[\Symfony\Component\Console\Attribute\Option(name: 'option1', shortcut: 'o', mode: InputOption::VALUE_NONE, description: 'Option1 description')]
-        $option1,
+        bool $option1 = false,
         #[\Symfony\Component\Console\Attribute\Option(name: 'option2', shortcut: 'p', mode: InputOption::VALUE_NONE, description: 'Option2 description')]
-        $option2
+        bool $option2 = false
     ): int
     {
         $arg1 = $argument1;

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

@@ -62,9 +62,9 @@ final class WithMultipleArgumentsOptionsNotFluent
         #[\Symfony\Component\Console\Attribute\Argument(name: 'argument2', description: 'Argument2 description')]
         string $argument2,
         #[\Symfony\Component\Console\Attribute\Option(name: 'option1', shortcut: 'o', mode: InputOption::VALUE_NONE, description: 'Option1 description')]
-        $option1,
+        bool $option1 = false,
         #[\Symfony\Component\Console\Attribute\Option(name: 'option2', shortcut: 'p', mode: InputOption::VALUE_NONE, description: 'Option2 description')]
-        $option2
+        bool $option2 = false
     ): int
     {
         $arg1 = $argument1;

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

@@ -47,7 +47,7 @@ final class WithOptionalArgument
 {
     public function __invoke(#[\Symfony\Component\Console\Attribute\Argument(name: 'argument', description: 'Argument description')]
     ?string $argument, #[\Symfony\Component\Console\Attribute\Option(name: 'option', shortcut: 'o', mode: InputOption::VALUE_NONE, description: 'Option description')]
-    $option): int
+    bool $option = false): int
     {
         $someArgument = $argument;
         $someOption = $option;

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

@@ -51,7 +51,7 @@ final class WithOverride
         #[\Symfony\Component\Console\Attribute\Argument(name: 'argument', description: 'Argument description')]
         string $argument,
         #[\Symfony\Component\Console\Attribute\Option(name: 'option', shortcut: 'o', mode: InputOption::VALUE_NONE, description: 'Option description')]
-        $option
+        bool $option = false
     ): int
     {
         $someArgument = $argument;

rules/Symfony73/NodeAnalyzer/CommandOptionsResolver.php

@@ -36,6 +36,8 @@ public function resolve(ClassMethod $configureClassMethod): array
 
             $optionName = $this->valueResolver->getValue($addOptionArgs[0]->value);
 
+            $isImplicitBoolean = $this->isImplicitBoolean($addOptionArgs);
+
             $commandOptions[] = new CommandOption(
                 $optionName,
                 $addOptionArgs[0]->value,
@@ -44,6 +46,7 @@ public function resolve(ClassMethod $configureClassMethod): array
                 $addOptionArgs[3]->value ?? null,
                 $addOptionArgs[4]->value ?? null,
                 $this->isArrayMode($addOptionArgs),
+                $isImplicitBoolean,
                 $this->resolveDefaultType($addOptionArgs)
             );
         }
@@ -75,7 +78,19 @@ private function isArrayMode(array $args): bool
         }
 
         $modeValue = $this->valueResolver->getValue($modeExpr);
-        // binary check for InputOptions::VALUE_IS_ARRAY
+        // binary check for InputOption::VALUE_IS_ARRAY
         return (bool) ($modeValue & 8);
     }
+
+    private function isImplicitBoolean(array $args): bool
+    {
+        $modeExpr = $args[2]->value ?? null;
+        if (! $modeExpr instanceof Expr) {
+            return false;
+        }
+
+        $modeValue = $this->valueResolver->getValue($modeExpr);
+        // binary check for InputOption::VALUE_NONE
+        return (bool) ($modeValue & 1);
+    }
 }

rules/Symfony73/NodeFactory/CommandInvokeParamsFactory.php

@@ -99,6 +99,8 @@ private function createOptionParams(array $commandOptions): array
 
             if ($commandOption->getDefault() instanceof Expr) {
                 $optionParam->default = $commandOption->getDefault();
+            } elseif ($commandOption->isImplicitBoolean()) {
+                $optionParam->default = new Expr\ConstFetch(new Node\Name('false'));
             }
 
             $this->decorateParamType($optionParam, $commandOption);
@@ -165,6 +167,11 @@ private function decorateParamType(
         Param $argumentParam,
         CommandArgument|CommandOption $commandArgumentOrOption
     ): void {
+        if ($commandArgumentOrOption instanceof CommandOption && $commandArgumentOrOption->isImplicitBoolean()) {
+            $argumentParam->type = new Identifier('bool');
+            return;
+        }
+
         if ($commandArgumentOrOption->isArray()) {
             $argumentParam->type = new Identifier('array');
             return;

rules/Symfony73/ValueObject/CommandOption.php

@@ -17,6 +17,7 @@ public function __construct(
         private ?Expr $description,
         private ?Expr $default,
         private bool $isArray,
+        private bool $isImplicitBoolean,
         private ?Type $defaultType
     ) {
     }
@@ -60,4 +61,9 @@ public function isArray(): bool
     {
         return $this->isArray;
     }
+
+    public function isImplicitBoolean(): bool
+    {
+        return $this->isImplicitBoolean;
+    }
 }