feat: if the bin is in PATH, then use that rather than the full path

package.json

@@ -66,6 +66,7 @@
     "rollup-plugin-node-resolve": "^3.0.0",
     "rollup-plugin-uglify": "^2.0.1",
     "rollup-watch": "^4.3.1",
+    "which": "^1.3.0",
     "yargs-parser": "^7.0.0"
   },
   "repository": {

src/config/lintstagedrc.js

@@ -1,15 +1,19 @@
+const {resolveBin} = require('../utils')
+
+const kcdScripts = resolveBin('kcd-scripts')
+
 module.exports = {
   linters: {
     '**/*.+(js|json|less|css|ts)': [
-      'kcd-scripts format',
-      'kcd-scripts lint',
-      'kcd-scripts test --findRelatedTests',
+      `${kcdScripts} format`,
+      `${kcdScripts} lint`,
+      `${kcdScripts} test --findRelatedTests`,
       'git add',
     ],
     '.all-contributorsrc': [
       // lint-staged passes arguments to the scripts.
       // to avoid passing these arguments, we do the echo thing
-      'kcd-scripts contributors generate',
+      `${kcdScripts} contributors generate`,
       'git add README.md',
     ],
   },

src/utils.js

@@ -3,20 +3,39 @@ const path = require('path')
 const arrify = require('arrify')
 const has = require('lodash.has')
 const readPkgUp = require('read-pkg-up')
+const which = require('which')
 
 const {pkg, path: pkgPath} = readPkgUp.sync({
   cwd: fs.realpathSync(process.cwd()),
 })
 const appDirectory = path.dirname(pkgPath)
 
+// eslint-disable-next-line complexity
 function resolveBin(modName, {executable = modName} = {}) {
-  const modPkgPath = require.resolve(`${modName}/package.json`)
-  const modPkgDir = path.dirname(modPkgPath)
-  const {bin} = require(modPkgPath)
-  if (typeof bin === 'string') {
-    return path.join(modPkgDir, bin)
+  let pathFromWhich
+  try {
+    pathFromWhich = which.sync(executable)
+  } catch (_error) {
+    // ignore _error
+  }
+  try {
+    const modPkgPath = require.resolve(`${modName}/package.json`)
+    const modPkgDir = path.dirname(modPkgPath)
+    const {bin} = require(modPkgPath)
+    if (typeof bin === 'string') {
+      return path.join(modPkgDir, bin)
+    }
+    const fullPathToBin = path.join(modPkgDir, bin[executable])
+    if (fullPathToBin === pathFromWhich) {
+      return executable
+    }
+    return fullPathToBin
+  } catch (error) {
+    if (pathFromWhich) {
+      return executable
+    }
+    throw error
   }
-  return path.join(modPkgDir, bin[executable])
 }
 
 const fromRoot = (...p) => path.join(appDirectory, ...p)