[BUG]: Subcommands in the --help stdout don't look like the options stdout

[orowith2os]

Describe the bug

The subcommands seem to have a bit of a typo, they're missing a comma to separate the pacman-style commands from plain english.

To reproduce the bug

Run ame --help

Expected behavior

The subcommands --help area would look similar to the options area.

Screenshots

No response

Additional context

❯ ame --help
Amethyst 4.0.0 (Funky Fish)
A fast and efficient AUR helper

USAGE:
    ame [OPTIONS] [SUBCOMMAND]

OPTIONS:
    -c, --cachedir <CACHEDIR>    Sets a custom AUR clone and build directory for the specified operation
    -h, --help                   Print help information
        --noconfirm              Complete operation without prompting user
    -q, --quiet                  Make some commands have less output
        --sudoloop               Loops sudo in the background to ensure it doesn't time out during long builds
    -v, --verbose                Sets the level of verbosity
    -V, --version                Print version information

SUBCOMMANDS:
    clean -C      Removes all orphaned packages
    diff -d       Runs pacdiff
    gencomp -G    Generates shell completions for supported shells (bash, fish, elvish, pwsh)
    help          Print this message or the help of the given subcommand(s)
    query -Q      Queries installed packages
    remove -R     Removes a previously installed package [aliases: rm]
    search        Searches for packages matching a provided pattern in the AUR/repos [aliases: -Ss]
    sync -S       Installs or searches for a package in either the AUR or the Pacman-defined repositories
                      [aliases: install, i]
    upgrade       Upgrades locally installed packages to their latest versions (Default) [aliases: -Syu]

This should instead be:

❯ ame --help
Amethyst 4.0.0 (Funky Fish)
A fast and efficient AUR helper

USAGE:
    ame [OPTIONS] [SUBCOMMAND]

OPTIONS:
    -c, --cachedir <CACHEDIR>    Sets a custom AUR clone and build directory for the specified operation
    -h, --help                   Print help information
        --noconfirm              Complete operation without prompting user
    -q, --quiet                  Make some commands have less output
        --sudoloop               Loops sudo in the background to ensure it doesn't time out during long builds
    -v, --verbose                Sets the level of verbosity
    -V, --version                Print version information

SUBCOMMANDS:
    clean, -C      Removes all orphaned packages
    diff, -d       Runs pacdiff
    gencomp, -G    Generates shell completions for supported shells (bash, fish, elvish, pwsh)
    help          Print this message or the help of the given subcommand(s)
    query, -Q      Queries installed packages
    remove, -R     Removes a previously installed package [aliases: rm]
    search        Searches for packages matching a provided pattern in the AUR/repos [aliases: -Ss]
    sync, -S       Installs or searches for a package in either the AUR or the Pacman-defined repositories
                      [aliases: install, i]
    upgrade       Upgrades locally installed packages to their latest versions (Default) [aliases: -Syu]

[fries1234]

looks like this is a clap thing i dont think we can do much about this 😔

[orowith2os]

Should be fixed soon, for now just gonna tough it out.

[fries1234]

uh can we use git HEAD for main and wait for 4.0.0 release or would that be a bad idea

[orowith2os]

Why not backport the fix and use our own crate for it, then use upstream again when v4 is released? Less risk of breakage due to an unstable lib.

[fries1234]

i guess we could do that

[Trivernis]

As this is just a minor visual bug I wouldn't go the lengths of maintaining a fork of clap until it's fixed or rather until clap v4 is released

[orowith2os]

I'll close this issue then, see yall on the update lol