add read access check to fs.accessSync & use es6 destructuring

lib/clean.js

@@ -1,5 +1,5 @@
-module.exports = function(Spike, args) {
-  const project = new Spike({ root: args.path })
+module.exports = function(Spike, { path }) {
+  const project = new Spike({ root: path })
   process.nextTick(() => project.clean())
   return project
 }

lib/compile.js

@@ -1,5 +1,5 @@
-module.exports = function(Spike, args) {
-  const project = new Spike({ root: args.path, env: args.env })
+module.exports = function(Spike, { path, env }) {
+  const project = new Spike({ root: path, env })
   process.nextTick(() => project.compile())
   return project
 }

lib/index.js

@@ -6,7 +6,7 @@ const EventEmitter = require('events')
 const GA = require('universal-analytics')
 const ArgumentParser = require('argparse').ArgumentParser
 const reduce = require('lodash.reduce')
-const pkg = require('../package.json')
+const { version, description } = require('../package.json')
 const path = require('path')
 const fs = require('fs')
 let analytics
@@ -21,12 +21,7 @@ module.exports = class CLI extends EventEmitter {
   constructor() {
     super()
 
-    this.parser = new ArgumentParser({
-      version: pkg.version,
-      description: pkg.description,
-      addHelp: true
-    })
-
+    this.parser = new ArgumentParser({ version, description, addHelp: true })
     this.sub = this.parser.addSubparsers()
 
     this.addCompile()
@@ -61,7 +56,7 @@ module.exports = class CLI extends EventEmitter {
     // try to load up a local spike instance first
     let spikePath = args.path && path.join(args.path, 'node_modules/spike-core')
     try {
-      fs.accessSync(spikePath)
+      fs.accessSync(spikePath, fs.constants.F_OK | fs.constants.R_OK)
     } catch (_) {
       spikePath = 'spike-core'
     }

lib/new.js

@@ -1,23 +1,23 @@
-const path = require('path')
+const { resolve } = require('path')
 const EventEmitter = require('events').EventEmitter
 const inquirer = require('inquirer')
 
-module.exports = function(Spike, args) {
+module.exports = function(Spike, { path, overrides, template }) {
   const emitter = new EventEmitter()
 
   emitter.on('done', project => {
     emitter.emit(
       'success',
-      `project created at ${path.resolve(project.config.context)}`
+      `project created at ${resolve(project.config.context)}`
     )
   })
 
   process.nextTick(() =>
     Spike.new({
-      root: args.path,
-      emitter: emitter,
-      locals: args.overrides,
-      template: args.template,
+      emitter,
+      template,
+      root: path,
+      locals: overrides,
       inquirer: inquirer.prompt.bind(inquirer)
     })
   )

lib/template/add.js

@@ -2,8 +2,6 @@ const EventEmitter = require('events')
 
 module.exports = function(Spike, args) {
   const emitter = new EventEmitter()
-  process.nextTick(() => {
-    Spike.template.add(Object.assign(args, { emitter: emitter }))
-  })
+  process.nextTick(() => Spike.template.add(Object.assign(args, { emitter })))
   return emitter
 }

lib/template/default.js

@@ -2,8 +2,8 @@ const EventEmitter = require('events')
 
 module.exports = function(Spike, args) {
   const emitter = new EventEmitter()
-  process.nextTick(() => {
-    Spike.template.default(Object.assign(args, { emitter: emitter }))
-  })
+  process.nextTick(() =>
+    Spike.template.default(Object.assign(args, { emitter }))
+  )
   return emitter
 }

lib/template/list.js

@@ -4,10 +4,10 @@ module.exports = function(Spike, args) {
   const emitter = new EventEmitter()
   const e2 = new EventEmitter()
   process.nextTick(() => {
-    emitter.on('success', res => {
+    emitter.on('success', res =>
       e2.emit('success', 'Your Templates:\n- ' + Object.keys(res).join('\n- '))
-    })
-    Spike.template.list(Object.assign(args, { emitter: emitter }))
+    )
+    Spike.template.list(Object.assign(args, { emitter }))
   })
   return e2
 }

lib/template/remove.js

@@ -2,8 +2,8 @@ const EventEmitter = require('events')
 
 module.exports = function(Spike, args) {
   const emitter = new EventEmitter()
-  process.nextTick(() => {
-    Spike.template.remove(Object.assign(args, { emitter: emitter }))
-  })
+  process.nextTick(() =>
+    Spike.template.remove(Object.assign(args, { emitter }))
+  )
   return emitter
 }

lib/watch.js

@@ -1,9 +1,5 @@
-module.exports = function(Spike, args) {
-  const project = new Spike({
-    root: args.path,
-    env: args.env,
-    server: { port: args.port }
-  })
+module.exports = function(Spike, { path, env, port }) {
+  const project = new Spike({ env, root: path, server: { port } })
   process.nextTick(() => project.watch())
   return project
 }