Handling partial success for multiple values in add / set commands

[arjunumenon]

There are commands which accepts multiple values as parameters for add or set commands. In these scenarios, when you have multiple values and some of the values are invalid, the command still gives us a positive output.
For e.g., when you set multiple owners to aad group using, aad o365group set, even if some of the usernames are invalid, command still give you positive result.

What should be the behavior of the output on these scenarios. Should we,

• Continue giving positive results eventhough some values are invalid
• Manipulate the output with added information along with added flag (True / False) so that executed result will show how many values were added and how were not?
• Any other ideas??

Requesting your thoughts, @pnp/cli-for-microsoft-365-maintainers

Following are some of the commands which we have identified so far which we have these scenarios.

1. aad o365group set
2. spo group user add

Creating the discussion based on this conversation.

[garrytrinder]

I seem to recall that being unable to write to both stdout and stderr in the same command execution was a limitation we have? I could be wrong. @waldekmastykarz

[waldekmastykarz]

Correct, it's impossible to write to stdout and stderr at the same time. That said, if you run a command to perform multiple actions, and not all of them complete, then I'd consider the execution faulty and expect the command to fail.

[arjunumenon]

Thanks @waldekmastykarz , @garrytrinder for the clarification. So in that case, do you suggest to make the execution as failed if some of the values of the multi-value parameters are not valid.?
If you agree, I can create an enhancement to handle that scenario for the below commands.

1. aad o365group set
2. spo group user add

[waldekmastykarz]

So in that case, do you suggest to make the execution as failed if some of the values of the multi-value parameters are not valid.?

I'd say so

[garrytrinder]

Agreed.

[arjunumenon]

Thanks @garrytrinder @waldekmastykarz for the suggestion.
Based on that, I have created enhancement for the commands, #2653 ,#2654.