Npm: download the required binaries during installation

.gitignore

@@ -12,7 +12,9 @@ dist/
 .rubygems/pkg/
 .rubygems/libexec/
 .npm/bin/
+!.npm/bin/*.js
 package.json
+!.npm/package.json
 node_modules/
 yarn.lock
 package-lock.json

.npm/bin/index.js

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
 
 var spawn = require('child_process').spawn;
-const { getExePath } = require('../get-exe');
+const path = require("path")
+const exePath = path.join(__dirname, `lefthook${["win32", "cygwin"].includes(process.platform) ? ".exe" : ""}`)
 
 var command_args = process.argv.slice(2);
-
 var child = spawn(
-    getExePath(),
+    exePath,
     command_args,
     { stdio: "inherit" });
 

.npm/package.json

@@ -30,5 +30,8 @@
   ],
   "scripts": {
     "postinstall": "node postinstall.js"
+  },
+  "dependencies": {
+    "node-downloader-helper": "^1.0.18"
   }
 }

.npm/postinstall.js

@@ -1,10 +1,69 @@
-if (!process.env.CI) {
-  const { spawnSync } = require('child_process');
-  const { getExePath } = require('./get-exe');
+const { spawnSync } = require("child_process")
 
+async function install() {
+  if (process.env.CI) {
+    return
+  }
+  const exePath = await downloadBinary()
   // run install
-  spawnSync(getExePath(), ['install', '-f'], {
+  spawnSync(exePath, ["install", "-f"], {
     cwd: process.env.INIT_CWD || process.cwd,
-    stdio: 'inherit',
-  });
+    stdio: "inherit",
+  })
 }
+
+function getDownloadURL() {
+  // Detect OS
+  // https://nodejs.org/api/process.html#process_process_platform
+  let goOS = process.platform
+  let extension = ""
+  if (["win32", "cygwin"].includes(process.platform)) {
+    goOS = "windows"
+    extension = ".exe"
+  }
+
+  // Convert the goOS to the os name in the download URL
+  let downloadOS = goOS === "darwin" ? "macOS" : goOS
+  downloadOS = `${downloadOS.charAt(0).toUpperCase()}${downloadOS.slice(1)}`
+
+  // Detect architecture
+  // https://nodejs.org/api/process.html#process_process_arch
+  let arch = process.arch
+  switch (process.arch) {
+    case "x64": {
+      arch = "x86_64"
+      break
+    }
+    case "x32":
+    case "ia32": {
+      arch = "i386"
+      break
+    }
+  }
+  const version = require("./package.json").version
+
+  return `https://github.com/evilmartians/lefthook/releases/download/v${version}/lefthook_${version}_${downloadOS}_${arch}${extension}`
+}
+
+const { DownloaderHelper } = require("node-downloader-helper")
+const path = require("path")
+
+async function downloadBinary() {
+  // TODO zip the binaries to reduce the download size
+  const downloadURL = getDownloadURL()
+  const extension = path.extname(downloadURL).slice(1)
+  const fileName = `lefthook.${extension}`
+  const binDir = path.join(__dirname, "bin")
+  const dl = new DownloaderHelper(downloadURL, binDir, {
+    fileName,
+    retry: { maxRetries: 5, delay: 50 },
+  })
+  dl.on("end", () => console.log("lefthook binary was downloaded"))
+  await dl.start()
+  return path.join(binDir, fileName)
+}
+
+// start:
+install().catch((e) => {
+  throw e
+})

cmd/templates.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"embed"
 	"text/template"
+	"runtime"
 
 	"github.com/spf13/afero"
 )
@@ -14,6 +15,7 @@ var templatesFS embed.FS
 type hookTmplData struct {
 	AutoInstall string
 	HookName    string
+	Extension 	string
 }
 
 func hookTemplate(hookName string, fs afero.Fs) []byte {
@@ -22,6 +24,7 @@ func hookTemplate(hookName string, fs afero.Fs) []byte {
 	err := t.Execute(buf, hookTmplData{
 		AutoInstall: autoInstall(hookName, fs),
 		HookName:    hookName,
+		Extension: 	 getExtension(),
 	})
 	check(err)
 
@@ -40,5 +43,13 @@ func autoInstall(hookName string, fs afero.Fs) string {
 		return ""
 	}
 
-	return "# lefthook_version: " + configChecksum(fs) + "\n\ncall_lefthook \"lefthook install\""
+	return "# lefthook_version: " + configChecksum(fs) + "\n\ncall_lefthook \"install\""
+}
+
+func getExtension() string {
+	if (runtime.GOOS == "windows") {
+		return ".exe"
+	} else {
+		return ""
+	}
 }

cmd/templates/hook.tmpl

@@ -14,24 +14,24 @@ call_lefthook()
 {
   if lefthook -h >/dev/null 2>&1
   then
-    eval $1
-  elif test -f "$dir/node_modules/@arkweid/lefthook/bin/lefthook"
+    eval lefthook $1
+  elif test -f "$dir/node_modules/@arkweid/lefthook/bin/lefthook{{.Extension}}"
   then
-    eval $dir/node_modules/@arkweid/lefthook/bin/$1
+    eval "$dir/node_modules/@arkweid/lefthook/bin/lefthook{{.Extension}} $1"
   elif bundle exec lefthook -h >/dev/null 2>&1
   then
-    bundle exec $1
+    bundle exec lefthook $1
   elif npx lefthook -h >/dev/null 2>&1
   then
-    npx $1
+    npx lefthook $1
   elif yarn lefthook -h >/dev/null 2>&1
   then
-    yarn $1
+    yarn lefthook $1
   else
     echo "Can't find lefthook in PATH"
   fi
 }
 
 {{.AutoInstall}}
 
-call_lefthook "lefthook run {{.HookName}} $@"
+call_lefthook "run {{.HookName}} $@"

spec/fixtures/pre-commit

@@ -14,24 +14,24 @@ call_lefthook()
 {
   if lefthook -h >/dev/null 2>&1
   then
-    eval $1
+    eval lefthook $1
   elif test -f "$dir/node_modules/@arkweid/lefthook/bin/lefthook"
   then
-    eval $dir/node_modules/@arkweid/lefthook/bin/$1
+    eval "$dir/node_modules/@arkweid/lefthook/bin/lefthook $1"
   elif bundle exec lefthook -h >/dev/null 2>&1
   then
-    bundle exec $1
+    bundle exec lefthook $1
   elif npx lefthook -h >/dev/null 2>&1
   then
-    npx $1
+    npx lefthook $1
   elif yarn lefthook -h >/dev/null 2>&1
   then
-    yarn $1
+    yarn lefthook $1
   else
     echo "Can't find lefthook in PATH"
   fi
 }
 
 
 
-call_lefthook "lefthook run pre-commit $@"
+call_lefthook "run pre-commit $@"

spec/fixtures/pre-push

@@ -14,24 +14,24 @@ call_lefthook()
 {
   if lefthook -h >/dev/null 2>&1
   then
-    eval $1
+    eval lefthook $1
   elif test -f "$dir/node_modules/@arkweid/lefthook/bin/lefthook"
   then
-    eval $dir/node_modules/@arkweid/lefthook/bin/$1
+    eval "$dir/node_modules/@arkweid/lefthook/bin/lefthook $1"
   elif bundle exec lefthook -h >/dev/null 2>&1
   then
-    bundle exec $1
+    bundle exec lefthook $1
   elif npx lefthook -h >/dev/null 2>&1
   then
-    npx $1
+    npx lefthook $1
   elif yarn lefthook -h >/dev/null 2>&1
   then
-    yarn $1
+    yarn lefthook $1
   else
     echo "Can't find lefthook in PATH"
   fi
 }
 
 
 
-call_lefthook "lefthook run pre-push $@"
+call_lefthook "run pre-push $@"