fix(hook): handle node module path os agnostically (#114)

* fix(hook): detemine hook name path separator agnostically

Former `hook` name determination was a simple regex using *nix style path separator `/`, using `path.basename`, the required last part gets returned, independent of an os specific path separator.

* fix(hook): correctly escape rendered `node_modules_path` string

Author: ta2edchimp

lib/hook.template.js

@@ -1,10 +1,9 @@
 const fs = require('fs')
-const resolve = require('path').resolve
-const path = require('path')
+const {resolve, join} = require('path')
 
 exports.generatedMessage = 'Generated by ghooks. Do not edit this file.'
 
 exports.content = fs
   .readFileSync(resolve(`${__dirname}/hook.template.raw`), 'UTF-8')
   .replace('{{generated_message}}', exports.generatedMessage)
-  .replace('{{node_modules_path}}', path.join(process.cwd(), '..'))
+  .replace('{{node_modules_path}}', join(process.cwd(), '..').replace(/\\/g, '\\'))

lib/runner.js

@@ -1,7 +1,7 @@
 const clone = require('lodash.clone')
 const managePath = require('manage-path')
 const spawn = require('spawn-command')
-const resolve = require('path').resolve
+const {resolve, basename} = require('path')
 const findup = require('findup')
 const fs = require('fs')
 
@@ -13,7 +13,7 @@ module.exports = function run(nodeModulesPath, filename, env) {
 }
 
 function hook(filename) {
-  return filename.match(/\/([^\/]+)\/?$/)[1]
+  return basename(filename)
 }
 
 // replace any instance of $1 or $2 etc. to that item as an process.argv