fix: Correctly detect local binary when installing via npm

getsentry · Jul 26, 2023 · d26f7cd · d26f7cd
1 parent 23cdcec
commit d26f7cd
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,12 @@
 
 "You know what they say. Fool me once, strike one, but fool me twice... strike three." — Michael Scott
 
+## 2.20.1
+
+### Various fixes and improvements
+
+- fix: Correctly detect local binary when installing via npm (#1695)
+
 ## 2.20.0
 
 ### Various fixes and improvements

diff --git a/scripts/install.js b/scripts/install.js
@@ -193,7 +193,9 @@ async function downloadBinary() {
 
   if (process.env.SENTRYCLI_USE_LOCAL === '1') {
     try {
-      const binPath = which.sync('sentry-cli');
+      const binPaths = which.sync('sentry-cli', { all: true });
+      if (!binPaths.length) throw new Error('Binary not found');
+      const binPath = binPaths[binPaths.length - 1];
       logger.log(`Using local binary: ${binPath}`);
       fs.copyFileSync(binPath, outputPath);
       return Promise.resolve();