fix: ensure PATH is always set (#93)

netlify · Aug 10, 2022 · 2f71c57 · 2f71c57
1 parent 9ffa88b
commit 2f71c57
Show file tree

Hide file tree

Showing 3 changed files with 76 additions and 25 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -87,6 +87,7 @@
     "node-fetch": "^3.1.1",
     "node-stream-zip": "^1.15.0",
     "p-wait-for": "^4.1.0",
+    "path-key": "^4.0.0",
     "semver": "^7.3.5",
     "tmp-promise": "^3.0.3",
     "uuid": "^8.3.2"

diff --git a/src/bridge.ts b/src/bridge.ts
@@ -3,6 +3,7 @@ import path from 'path'
 import process from 'process'
 
 import { execa, ExecaChildProcess, Options } from 'execa'
+import pathKey from 'path-key'
 import semver from 'semver'
 
 import { download } from './downloader.js'
@@ -199,6 +200,9 @@ class DenoBridge {
       env.DENO_DIR = this.denoDir
     }
 
+    // Ensure PATH is always set as otherwise we are not able to find the global deno binary
+    env[pathKey()] = inputEnv[pathKey({ env: inputEnv })] || process.env[pathKey()]
+
     return env
   }