invalid command line flags are ignored vs throwing error / usage


(William) #1

If an invalid command line flag / switch is provided, npm will continue, rather than throwing an error and giving the help text for that context.
% npm audit --format=asdfasfd --123 asdf
will simply act as if you had typed
% npm audit
vs returning usage and exiting non-0.

More generally, standardizing processing of both positional arguments and command line flags might be a big plus in terms of keeping the tool consistent and well documented.

It was suggested in that I bring it up here; also submitted:

(Rebecca Turner) #2

I really wish we could do this. We may be able to for commands like npm audit (but not npm audit fix), but doing so for npm install and it’s related family of commands, is vastly more difficult:

npm passes all of its config options and command line arguments together through to scripts that it runs in the environment. This includes unknown arguments. There are a number of build tools that take advantage of this so that you can just add their config to your .npmrc or pass it in as extra arguments to npm i and they get it automatically.

Erroring on invalid options would break these use cases.

(William) #3

What about npm run? I think you normally have to pass command line flags w/ -- if you want to pass them to a subcommand?
For example, I typically have to do
npm run xyz -- -y if I want to pass in the -y argument to the command that xyz is running.
I think that’s the usual convention for passing flags to a command run by another command, right?

(Rebecca Turner) #4

npm run is probably not a bad starter case either.