clap icon indicating copy to clipboard operation
clap copied to clipboard

Prevent specifying global args multiple times

Open cd-work opened this issue 2 years ago • 2 comments

Please complete the following tasks

Clap Version

3.2.8

Describe your use case

When making an argument global with Arg::global(true), it will be available to both the parent and child subcommand. To me this implies that this option is a single flag that can be modified from anywhere.

However when specifying the global twice (e.g. cmd --global x subcmd --global y), the parent's assignment of the variable will be discarded in favor of the subcommand's. While this definitely is a possible way to handle this edgecase, I think this command invocation indicates that the user is doing something wrong and it would be better if clap could tell the user that what he's doing doesn't make any sense.

In my specific scenario, I have a command which defaults to one of it subcommands, so cmd --flag x is the same as cmd list --flag x, while there's also other subcommands like cmd foo --flag x. One flag is shared between all subcommands (which I've tried making global), while some others are only available to cmd and cmd list. This would solve the issue of having an option that can be specified twice for that one shared flag, but I'd still need another solution preventing people from using the other cmd list-specific flags which can also be used with just cmd when another subcommand like cmd foo is used. So it's possible that this feature request is just a band-aid rather than solving the root issue.

Describe the solution you'd like

Emit an error when specifying the same global argument multiple times.

Alternatives, if applicable

No response

Additional Context

No response

cd-work avatar Jul 15 '22 16:07 cd-work

I'd still need another solution preventing people from using the other cmd list-specific flags which can also be used with just cmd when another subcommand like cmd foo is used. So it's possible that this feature request is just a band-aid rather than solving the root issue.

I've found the args_conflicts_with_subcommands option, so I guess that solves my root issue in a more general way. Though the error is rather... unfortunate.

cd-work avatar Jul 15 '22 17:07 cd-work

Just reading the description,

  • I would expect global arguments to be merged or overwritten based on their Action (granted, due to the current structure, we might not be able to handle this cleanly when custom actions are implemented).
  • Changing this would likely be considered a breaking change

epage avatar Jul 16 '22 02:07 epage