`yargs` doesn't enforce `strictOptions` for options passed without `=` about yargs-parser HOT 6 CLOSED

Just to round this out in case anybody has a similar issue. For us, the solution was just to change our import for yargs from a default import of the sort:
import yargs from 'yargs'
to a namespace import:
import * as yargs from 'yargs'.

Thanks again for the help! Great project and we just added yargs autocomplete to our CLI tool as a bonus (since I didn't know it existed before).

from yargs-parser.

I was not able to reproduce this behaviour in a small test program. Both forms shows the help with an error Unknown argument: unknownOption.

In the linked issue, it mentions "unknown CLI params" and "unknown commands" and "unknown keyword arguments are being caught as expected but positional ones are not". To be clear, is the observed problem with an "option" starting with a - or --, or perhaps a non-option argument type?

from yargs-parser.

It's with what y'all call "options". Sorry for the lack of clarity. I'm wondering where things go awry because when I took just our yargs definition that starts here and copy it into a new script and try to run with the noted --datadir (from our issue), the unknownOption error appears. It's just when we run it with our compiled Typescript code where it doesn't show up.

from yargs-parser.

I managed to reproduce your problem using your tsconfig.json. The problem is the legacy yargs export does some tricky things, and TypeScript does some tricky things, and they don't always go together.

You can confirm by calling your broken program with unknown option and --help, and see if ----unknownOption appears in the help [sic].

node index.js --unknownOption unknownValue --help

See yargs/yargs#2010 (comment) for some research, which does not cover the settings you are using.

As a work-around, you could use a special TypeScript import style that has worked with all the tsconfig settings I have tried:

// import yargs from 'yargs'
import yargs = require("yargs");

See: https://www.typescriptlang.org/docs/handbook/modules.html#export--and-import--require

from yargs-parser.

Thank you for the guidance! I will research #2010 further and see if any of those options could work for us as I don't think the import = require... will work for us with our current build setup. Will circle back here and close things out once I find a solution.

from yargs-parser.

(Thanks for circling back.)

from yargs-parser.