[Console] Preserve --help option when a command is not found

santysisi · santysisi · commit 632e02c980e9 · 2025-12-02T20:10:15.000-03:00
diff --git a/src/Symfony/Component/Console/Application.php b/src/Symfony/Component/Console/Application.php
@@ -732,15 +732,20 @@ public function find(string $name)
             $message = \sprintf('Command "%s" is not defined.', $name);
 
             if ($alternatives = $this->findAlternatives($name, $allCommands)) {
+                $wantHelps = $this->wantHelps;
+                $this->wantHelps = false;
                 // remove hidden commands
                 $alternatives = array_filter($alternatives, fn ($name) => !$this->get($name)->isHidden());
+                $this->wantHelps = $wantHelps;
 
-                if (1 == \count($alternatives)) {
-                    $message .= "\n\nDid you mean this?\n    ";
-                } else {
-                    $message .= "\n\nDid you mean one of these?\n    ";
+                if ($alternatives) {
+                    if (1 == \count($alternatives)) {
+                        $message .= "\n\nDid you mean this?\n    ";
+                    } else {
+                        $message .= "\n\nDid you mean one of these?\n    ";
+                    }
+                    $message .= implode("\n    ", $alternatives);
                 }
-                $message .= implode("\n    ", $alternatives);
             }
 
             throw new CommandNotFoundException($message, array_values($alternatives));
diff --git a/src/Symfony/Component/Console/Tests/ApplicationTest.php b/src/Symfony/Component/Console/Tests/ApplicationTest.php
@@ -2439,6 +2439,35 @@ public function testFindAmbiguousHiddenCommands()
         $application->find('t:f');
     }
 
+    public function testDoesNotFindHiddenCommandAsAlternativeIfHelpOptionIsPresent()
+    {
+        $application = new Application();
+        $application->setAutoExit(false);
+        $application->add(new \FooHiddenCommand());
+
+        $tester = new ApplicationTester($application);
+        $tester->run(['command' => 'foohidden', '--help' => true], ['interactive' => false]);
+
+        $this->assertStringContainsString('Command "foohidden" is not defined.', $tester->getDisplay(true));
+        $this->assertStringNotContainsString('Did you mean', $tester->getDisplay(true));
+        $this->assertStringNotContainsString('Do you want to run', $tester->getDisplay(true));
+    }
+
+    public function testsPreservedHelpOptionWhenItsAnAlternative()
+    {
+        $application = new Application();
+        $application->setAutoExit(false);
+        $application->add(new \FoobarCommand());
+
+        $tester = new ApplicationTester($application);
+        $tester->setInputs(['yes']);
+        $tester->run(['command' => 'foobarfoo', '--help' => true]);
+
+        $this->assertStringContainsString('Command "foobarfoo" is not defined.', $tester->getDisplay(true));
+        $this->assertStringContainsString('Do you want to run "foobar:foo" instead?', $tester->getDisplay(true));
+        $this->assertStringContainsString('The foobar:foo command', $tester->getDisplay(true));
+    }
+
     /**
      * Reads the private "signalHandlers" property of the SignalRegistry for assertions.
      */
