feat(run): support pre/post script

lib/install/resolver/fetchers/remote.js

@@ -1,4 +1,3 @@
-const path = require('path')
 const request = require('request')
 const tar = require('tar-stream')
 const gunzip = require('gunzip-maybe')

lib/run.js

@@ -1,34 +1,15 @@
-const exec = require('child_process').exec
-const fs = require('fs')
-const npmPath = require('npm-path')
 const path = require('path')
-const spawn = require('cross-spawn')
-const which = require('which')
-const nm = require('./utils/nm')
+const list = require('./run/list')
+const runner = require('./run/runner')
 
 const run = (argv) => {
   argv._handled = true
   const pkgJSON = require(path.join(process.cwd(), 'package.json'))
-  const scripts = pkgJSON.scripts
+  if (!pkgJSON.scripts) return
   if (argv._.length === 1) {
-    process.stdout.write(
-      'Available scripts via `dep run`\n\n' +
-      Object.keys(scripts).map((key) => {
-        return 'dep run ' + key + ':\n  ' + scripts[key]
-      }).join('\n') + '\n'
-    )
+    list(pkgJSON)
   } else {
-    var env = process.env
-    var newPath = npmPath.getSync({})
-    env[npmPath.PATH] = newPath
-    const args = argv._.slice(1)
-    var cmds = scripts[args.shift()] || ''
-    cmds = cmds.split(' ')
-    const cmd = cmds.shift()
-    const script = spawn(cmd, args.concat(cmds), {env: env})
-    script.stdout.on('data', (data) => {
-      process.stdout.write(data)
-    })
+    runner(argv._, pkgJSON)
   }
 }
 

lib/run/list.js

@@ -0,0 +1,9 @@
+module.exports = (pkg) => {
+  const scripts = pkg.scripts
+  process.stdout.write(
+    'Available scripts via `dep run`\n\n' +
+    Object.keys(scripts).map((key) => {
+      return 'dep run ' + key + ':\n  ' + scripts[key]
+    }).join('\n') + '\n'
+  )
+}

lib/run/runner.js

@@ -0,0 +1,35 @@
+const spawn = require('cross-spawn')
+const npmPath = require('npm-path')
+const each = require('promise-each')
+
+module.exports = (_, pkg) => {
+  const args = _.slice(1)
+  const scripts = pkg.scripts
+  const key = args.shift()
+  var cmds = Object.keys(scripts).filter((script) => {
+    return script === 'pre' + key
+      || script === key
+      || script === 'post' + key
+    }).map((script) => {
+      return scripts[script].split(' ')
+    })
+  var env = process.env
+  var newPath = npmPath.getSync({})
+  env[npmPath.PATH] = newPath
+
+  Promise.resolve(cmds).then(each((item) => {
+    return new Promise((resolve, reject) => {
+      const cmd = item.shift()
+      const script = spawn(cmd, args.concat(item), {env: env})
+      script.stdout.on('data', (data) => {
+        process.stdout.write(data)
+      })
+      script.stderr.on('data', (data) => {
+        reject(data)
+      })
+      script.on('close', (data) => {
+        resolve()
+      })
+    })
+  }))
+}