[Bug Report] Multiple words negated options

[meteorlxy]

Issue Type

• Bug Report
• Feature Request
• Other

Steps to repro

// test.js
const cli = require('cac')();

cli.option('--no-aaa-bbb').option('--no-ccc');

const parsed = cli.parse();

console.log(JSON.stringify(parsed, null, 2));

$ node test.js
{
  "args": [],
  "options": {
    "--": [],
    "aaa-bbb": true,
    "ccc": true
  }
}

$ node test.js --no-aaa-bbb
{
  "args": [],
  "options": {
    "--": [],
    "aaa-bbb": true,
    "ccc": true,
    "aaaBbb": false
  }
}

$ node test.js --no-ccc
{
  "args": [],
  "options": {
    "--": [],
    "aaa-bbb": true,
    "ccc": false
  }
}

The "aaa-bbb": true is always there.

Info

• CAC version: 6.5.3

[egoist]

I updated the docs for this: https://github.com/cacjs/cac#dash-in-option-names

Maybe we should delete the options['foo-bar'] to reduce confusion

[meteorlxy]

Well, the point is that:

• options without dash have default value true
• options with dash have default value undefined

They are inconsistent

[egoist]

I see, I'll fix it.

[egoist]

🎉 This issue has been resolved in version 6.5.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀