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,13 @@
 #!/usr/bin/env node
 
 var spawn = require('child_process').spawn;
-const { getExePath } = require('../get-exe');
+const path = require("path")
+const extension = ["win32", "cygwin"].includes(process.platform) ? ".exe" : ""
+const exePath = path.join(__dirname, `lefthook${extension}`)
 
 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,78 @@
-if (!process.env.CI) {
- const { spawnSync } = require('child_process');
- const { getExePath } = require('./get-exe');
+const { spawnSync } = require("child_process")
 
+const iswin = ["win32", "cygwin"].includes(process.platform)
+
+async function install() {
+ if (process.env.CI) {
+ return
+ }
+ const exePath = await downloadBinary()
+ if (!iswin) {
+ const { chmodSync } = require("fs")
+ chmodSync(exePath, "755")
+ }
  // 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 (iswin) {
+ 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 = iswin ? ".exe" : ""
+ const fileName = `lefthook${extension}`
+ const binDir = path.join(__dirname, "bin")
+ const dl = new DownloaderHelper(downloadURL, binDir, {
+ fileName,
+ retry: { maxRetries: 5, delay: 50 },
+ })
+ dl.on("error", (e) => {
+ e.message = `Failed to download ${fileName}: ${e.message} while fetching ${downloadURL}`
+ throw e
+ })
+ await dl.start()
+ return path.join(binDir, fileName)
+}
+
+// start:
+install().catch((e) => {
+ throw e
+})

cmd/templates.go

@@ -3,6 +3,7 @@ package cmd
 import (
  "bytes"
  "embed"
+ "runtime"
  "text/template"
 
  "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,12 @@ 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"
+ }
+ 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 $@"