Default value type not applied to aliases

[andrewiggins]

Hey Luke 👋

Related: lukeed/sade#17

Was using sade in a project where I wanted an option to be coerced to a string always, so I specified that a default value of "". However I noticed that when using the alias of the option, the value was not turned into a string (repo below). For me, the use case is for simple file matching, where some file names may begin with a number. So the option may be passed a value of "01" and I want that as a string, not the number 1.

I think lukeed/sade#17 is related to what I want here, but seeing as how that issue is old and closed without discussion I thought maybe opening a new issue focused around the use case would be helpful. Apologies if I thought wrong here 😬

Repo

test.js:

const sade = require("sade"); // v1.7.3

// Using `isOne` just to simplify this program
const { args } = sade("test", true)
  .option("-a --arg", "Test option", "")
  .parse(process.argv, { lazy: true });

console.log(args);

Output with --arg:

$ node .\test.js --arg 01
[ { _: [], arg: '01', a: '01' } ]

Output with -a alias:
(expected to be the same as --arg output)

$ node .\test.js -a 01   
[ { _: [], a: 1, arg: 1 } ]

If you think this is something that should/can be fixed, happy to open a PR. My assumption is that we need to pass defaults for option names and aliases to mri?

Thanks!

P.S. Love all awesome libraries you build - thanks for giving these to us 😁

[lukeed]

Thank you :)

Can't remember why I closed that. I think I thought I fixed it? Clearly not, though.

This is actually a mri bug & so I'll transfer this over there. Thanks!

[lukeed]

Released within [email protected] and should be picked up in your next sade install too.

Thanks!