get rid of argparse nulls once and for all

Author: Jeff Escalante

lib/index.js

@@ -4,6 +4,7 @@
 
 const EventEmitter = require('events').EventEmitter
 const ArgumentParser = require('argparse').ArgumentParser
+const reduce = require('lodash.reduce')
 // TODO: this should reflect spike version, or cli should always mirror
 const pkg = require('../package.json')
 let Spike = require('spike-core')
@@ -43,6 +44,10 @@ module.exports = class CLI extends EventEmitter {
     const fn = require(`./${args.fn}`)
     delete args.fn
 
+    // argparse uses `null` which doesn't get along with joi, so we remove
+    // null values here
+    args = reduce(args, (m, v, k) => { if (v) m[k] = v; return m }, {})
+
     // all CLI modules must return an event emitter of some sort
     // they should also wait to execute anything until nextTick
     const emitter = fn(Spike, args)

package.json

@@ -14,7 +14,8 @@
   "dependencies": {
     "argparse": "^1.0.6",
     "chalk": "^1.1.3",
-    "spike-core": "^0.2.0"
+    "lodash.reduce": "^4.4.0",
+    "spike-core": "^0.2.1"
   },
   "devDependencies": {
     "ava": "^0.14.0",