Adding #42: Add options.array

getopts.d.ts

@@ -20,6 +20,7 @@ declare namespace getopts {
     alias?: { [key: string]: string | string[] }
     string?: string[]
     boolean?: string[]
+    array?: string[]
     default?: { [key: string]: any }
     unknown?: (optionName: string) => boolean
     stopEarly?: boolean

index.js

@@ -92,7 +92,7 @@ const parseOptions = function(aliases, options, value) {
   return out
 }
 
-const write = function(out, key, value, aliases, unknown) {
+const write = function(out, key, value, aliases, unknown, inArray) {
   let i,
     prev,
     alias = aliases[key],
@@ -102,7 +102,7 @@ const write = function(out, key, value, aliases, unknown) {
     prev = out[key]
 
     if (prev === undefined) {
-      out[key] = value
+      out[key] = inArray ? [value] : value
     } else {
       if (isArray(prev)) {
         prev.push(value)
@@ -123,6 +123,7 @@ const getopts = function(argv, opts) {
     strings = parseOptions(aliases, opts.string, ""),
     values = parseDefault(aliases, opts.default),
     bools = parseOptions(aliases, opts.boolean, false),
+    arrays = parseOptions(aliases, opts.array, true),
     stopEarly = opts.stopEarly,
     _ = [],
     out = { _ },
@@ -167,7 +168,7 @@ const getopts = function(argv, opts) {
             ? parseValue(argv[++i])
             : argv[++i])
       }
-      write(out, key, value, aliases, unknown)
+      write(out, key, value, aliases, unknown, arrays[key])
     } else {
       SHORTSPLIT.lastIndex = 2
       match = SHORTSPLIT.exec(arg)
@@ -189,7 +190,8 @@ const getopts = function(argv, opts) {
             : bools[key] !== undefined ||
               (strings[key] === undefined ? parseValue(value) : value),
           aliases,
-          unknown
+          unknown,
+          arrays[key]
         )
       }
     }

test/array.test.js

@@ -0,0 +1,34 @@
+const Parse = require("./Parse").Parse
+
+exports.default = {
+  "opts.array": [
+    {
+      name: "with short options",
+      argv: ["-a1", "-b2", "-c1.5", "-c", "2.5", "-doh", "-d", "k"],
+      opts: {
+        string: ["d"],
+        array: ["a", "d"]
+      },
+      expected: {
+        _: [],
+        a: [1],
+        b: 2,
+        c: [1.5, 2.5],
+        d: ["oh", "k"]
+      }
+    },
+    {
+      name: "with long options",
+      argv: ["--foo=001", "--bar=001", "--bar", ".5", "--baz"],
+      opts: {
+        array: ["foo", "bar"]
+      },
+      expected: {
+        _: [],
+        foo: [1],
+        bar: [1, .5],
+        baz: true
+      }
+    }
+  ].map(Parse)
+}