shards --help invokes install subcommand

[HertzDevil]

After #631, if the command line invocation is exactly shards --help, or it is shards --help -- followed by more arguments, then Shards behaves as if install is the selected command and ignores the --help entirely. This does not seem to be intended behavior.

[luislavena]

Hello @HertzDevil, just noticed you submitted this issue and definitely I didn't test it properly to catch it 😢

Before #631, shards --help or shards install --help had always the same result: it listed all the commands, like no command was provided:

/app # shards --version
Shards 0.18.0 (2024-03-28)

/app # shards --help
shards [<options>...] [<command>]

Commands:
    build [<targets>] [<build_options>]  - Build the specified <targets> in `bin` path, all build_options are delegated to `crystal build`.
    check                                - Verify all dependencies are installed.
    init                                 - Initialize a `shard.yml` file.
    install                              - Install dependencies, creating or using the `shard.lock` file.
    list [--tree]                        - List installed dependencies.
    lock [--update] [<shards>...]        - Lock dependencies in `shard.lock` but doesn't install them.
    outdated [--pre]                     - List dependencies that are outdated.
    prune                                - Remove unused dependencies from `lib` folder.
    run [<target>] [<options>]           - Build and run specified target
    update [<shards>...]                 - Update dependencies and `shard.lock`.
    version [<path>]                     - Print the current version of the shard.

General options:
    --no-color                       Disable colored output.
    --version                        Print the `shards` version.
    --frozen                         Strictly installs locked versions from shard.lock.
    --without-development            Does not install development dependencies.
    --production                     same as `--frozen --without-development`
    --skip-postinstall               Does not run postinstall of dependencies
    --skip-executables               Does not install executables
    --local                          Don't update remote repositories, use the local cache only.
    --jobs=N                         Number of repository downloads to perform in parallel (default: 8). Currently only for git.
    --ignore-crystal-version         Has no effect. Kept for compatibility, to be removed in the future.
    -v, --verbose                    Increase the log verbosity, printing all debug statements.
    -q, --quiet                      Decrease the log verbosity, printing only warnings and errors.
    -h, --help                       Print usage synopsis.

/app # shards install --help
shards [<options>...] [<command>]

Commands:
    build [<targets>] [<build_options>]  - Build the specified <targets> in `bin` path, all build_options are delegated to `crystal build`.
    check                                - Verify all dependencies are installed.
    init                                 - Initialize a `shard.yml` file.
    install                              - Install dependencies, creating or using the `shard.lock` file.
    list [--tree]                        - List installed dependencies.
    lock [--update] [<shards>...]        - Lock dependencies in `shard.lock` but doesn't install them.
    outdated [--pre]                     - List dependencies that are outdated.
    prune                                - Remove unused dependencies from `lib` folder.
    run [<target>] [<options>]           - Build and run specified target
    update [<shards>...]                 - Update dependencies and `shard.lock`.
    version [<path>]                     - Print the current version of the shard.

General options:
    --no-color                       Disable colored output.
    --version                        Print the `shards` version.
    --frozen                         Strictly installs locked versions from shard.lock.
    --without-development            Does not install development dependencies.
    --production                     same as `--frozen --without-development`
    --skip-postinstall               Does not run postinstall of dependencies
    --skip-executables               Does not install executables
    --local                          Don't update remote repositories, use the local cache only.
    --jobs=N                         Number of repository downloads to perform in parallel (default: 8). Currently only for git.
    --ignore-crystal-version         Has no effect. Kept for compatibility, to be removed in the future.
    -v, --verbose                    Increase the log verbosity, printing all debug statements.
    -q, --quiet                      Decrease the log verbosity, printing only warnings and errors.
    -h, --help                       Print usage synopsis.

Perhaps we shouldn't assume install command is the default if --help is provided.

I will try to take a stab at that. Thank you for reporting it.
❤️ ❤️ ❤️