I discovered the same the last days.
Might also be intentional, as without arguments the command is no longer valid without an explicit default main command method.
I guess that one always has to run bin/cake my_command -h with help flag to get the desired overview and help output.
Without it it seems to be considered incomplete now.

Codecov Report

Merging #11179 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master   #11179      +/-   ##
============================================
+ Coverage     93.15%   93.15%   +<.01%     
- Complexity    12978    12979       +1     
============================================
  Files           437      437              
  Lines         33619    33622       +3     
============================================
+ Hits          31317    31320       +3     
  Misses         2302     2302

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6829ddd...6da6d0b. Read the comment docs.

Or should we show some error message before output help:

$this->err('foo');
$this->out($this->OptionParser->help($command));

see: CommandRunner.php#L166-L169

Looks like that would make sense. We should at least make sure we keep the exit code of 1 here.

Thanks!

I don't think this change is correct; when usage (help) output is printed due to missing arguments on Linux, that output should be send to stdErr to prevent the output from being piped to other commands when redirecting output.

I did a quick search to provide more information, and found this; https://www.jstorimer.com/blogs/workingwithcode/7766119-when-to-use-stderr-instead-of-stdout, but there should be other resources explaining this

@thaJeztah This change was to restore behaviour that has existed for quite some time. The reasons you cited were part of the reason I has used stderr. But those changes have caused people grief.

@markstory thanks for clarifying - my eye fell on this PR's title, and I know there's sometimes confusion around this, so thought to add the information above, but understand the reason to change if it broke people 👍

I too disagree with this change.

I think a better solution is:

diff --git a/src/Console/Shell.php b/src/Console/Shell.php
index d513038..8bd52aa 100644
--- a/src/Console/Shell.php
+++ b/src/Console/Shell.php
@@ -508,7 +508,7 @@ class Shell
             return $this->main(...$this->args);
         }
 
-        $this->err($this->OptionParser->help($command));
+        $this->_io->err($this->OptionParser->help($command));
 
         return false;
     }

This also correctly handles the <info /> tags as oppose to using the err() or out() wrapper

@rchavik You can make a follow up PR :)

@rchavik 👍