fix: unknown-options-as-args now respects short-option-groups

Author: shadowspawn

lib/yargs-parser.ts

@@ -975,7 +975,7 @@ export class YargsParser {
       // ignore negative numbers
       if (arg.match(negative)) { return false }
       // if this is a short option group and all of them are configured, it isn't unknown
-      if (hasAllShortFlags(arg)) { return false }
+      if (configuration['short-option-groups'] && hasAllShortFlags(arg)) { return false }
       // e.g. '--count=2'
       const flagWithEquals = /^-+([^=]+?)=[\s\S]*$/
       // e.g. '-a' or '--arg'

test/yargs-parser.mjs

@@ -3218,6 +3218,18 @@ describe('yargs-parser', function () {
           v: true
         })
       })
+      it('should not parse known short option groups when short-option-groups:false', function () {
+        const argv = parser('-known', {
+          boolean: ['k', 'n', 'o', 'w', 'n'],
+          configuration: {
+            'unknown-options-as-args': true,
+            'short-option-groups': false
+          }
+        })
+        argv.should.deep.equal({
+          _: ['-known']
+        })
+      })
       it('should parse negative numbers', function () {
         const argv = parser('-k -33', {
           boolean: ['k'],