@sentry/cli 1.74.3 → 2.0.1

package/bin/sentry-cli

@@ -3,17 +3,17 @@
 'use strict';
 
 const childProcess = require('child_process');
-const cli = require('../js');
+const SentryCli = require('../js');
 
 const child = childProcess
-  .spawn(cli.getPath(), process.argv.slice(2), {
+  .spawn(SentryCli.getPath(), process.argv.slice(2), {
     stdio: 'inherit',
   })
-  .on('error', err => {
+  .on('error', (err) => {
     console.error(err); // eslint-disable-line no-console
     process.exit(1);
   })
-  .on('exit', code => process.exit(code));
+  .on('exit', (code) => process.exit(code));
 
 process.on('SIGTERM', () => child.kill('SIGTERM'));
 process.on('SIGINT', () => child.kill('SIGINT'));

package/checksums.txt

@@ -1,9 +1,9 @@
-sentry-cli-Darwin-arm64=da33835c6c68e007747784cc191731fd47b6c1d0984fc5373aad60f3025e5ae9
-sentry-cli-Darwin-universal=ec665920219317a728093b30fd9c009dd64a8f49001d1d8434a46aa8ba5d0f12
-sentry-cli-Darwin-x86_64=d5f0e2380a79852a1ebaa5abab4c4d0e9681e5cb080d92b5b2801034649b27e5
-sentry-cli-Linux-aarch64=5f981d63bf6644973b85cf514381196b7a7089fec07e580be7530522728cf95f
-sentry-cli-Linux-armv7=c852fd2e1f402c3db0b269df17da7d14e8098c1c066ee09d342901c39862a36a
-sentry-cli-Linux-i686=f1db03614ec2b15e9cafcd7f3f7a9aaee5291f420fe4981ea220dc1bb8ec479e
-sentry-cli-Linux-x86_64=2d2b4acc6bb644aef5d07bd8c96ffaa16207719355fbf1578833c6eeee912554
-sentry-cli-Windows-i686.exe=8fbef71c26cc3a034043eaa4d2efd9830c17c2500f069411283e2fa3f7a0f9db
-sentry-cli-Windows-x86_64.exe=0d2f372d98f53ea4d4df26161f1f821d5322b00a0227ce84ec939bf271c720ad
+sentry-cli-Darwin-arm64=6bf38158c0b3d198802a22ec2d2021d619e5fe1a125f2d21395c6b0b54f5808d
+sentry-cli-Darwin-universal=4ded776b94d1c1e26fbb9e8811fdf003e7f51116bd4720e627a3bc2b9a814e74
+sentry-cli-Darwin-x86_64=a3add22156f70fba60c4f2eb47880362c704bd1fbc6f455f7f9f941a3e269754
+sentry-cli-Linux-aarch64=d05bf0aba980795fb046c62a693bd6c152421de1f85c1c851f94e206cabbe62f
+sentry-cli-Linux-armv7=3e64b3b61dca0cae6c6845dfbfe7ec19a208cc04480ec1dca5a343d2a5a8f3f5
+sentry-cli-Linux-i686=2ac8358c52f5a8936bade1729fb85e3b8be604d60bd195f88609cb44e6fb8f06
+sentry-cli-Linux-x86_64=4f378d823e03a40560cb0438b3eb3ee54ea828aff6087a9ef63dfeb31fb064ab
+sentry-cli-Windows-i686.exe=83965514cc737d6d96cd04d39d0ba91a26fa7881e3bcd325ac89a60d615f4de1
+sentry-cli-Windows-x86_64.exe=00df30fa383c90e27a14f8ddb151f8b29a7edb78507e8832350260bd0e131209

package/js/helper.js

@@ -133,7 +133,7 @@ function getPath() {
  * @param {Object} [config] More configuration to pass to the CLI
  * @returns {Promise.<string>} A promise that resolves to the standard output.
  */
-function execute(args, live, silent, configFile, config = {}) {
+async function execute(args, live, silent, configFile, config = {}) {
   const env = { ...process.env };
   if (configFile) {
     env.SENTRY_PROPERTIES = configFile;

package/js/releases/index.js

@@ -50,7 +50,7 @@ class Releases {
    * @returns {Promise} A promise that resolves when the release has been created.
    * @memberof SentryReleases
    */
-  new(release, options) {
+  async new(release, options) {
     const args = ['releases', 'new', release].concat(helper.getProjectFlagsFromOptions(options));
     return this.execute(args, null);
   }
@@ -76,7 +76,7 @@ class Releases {
    * @returns {Promise} A promise that resolves when the commits have been associated
    * @memberof SentryReleases
    */
-  setCommits(release, options) {
+  async setCommits(release, options) {
     if (!options || (!options.auto && (!options.repo || !options.commit))) {
       throw new Error('options.auto, or options.repo and options.commit must be specified');
     }
@@ -95,10 +95,6 @@ class Releases {
       commitFlags.push('--ignore-missing');
     }
 
-    if (options.ignoreEmpty) {
-      commitFlags.push('--ignore-empty');
-    }
-
     return this.execute(['releases', 'set-commits', release].concat(commitFlags));
   }
 
@@ -110,7 +106,7 @@ class Releases {
    * @returns {Promise} A promise that resolves when the release has been finalized.
    * @memberof SentryReleases
    */
-  finalize(release) {
+  async finalize(release) {
     return this.execute(['releases', 'finalize', release], null);
   }
 
@@ -121,10 +117,9 @@ class Releases {
    * @returns {Promise.<string>} A promise that resolves to the version string.
    * @memberof SentryReleases
    */
-  proposeVersion() {
-    return this.execute(['releases', 'propose-version'], null).then(
-      version => version && version.trim()
-    );
+  async proposeVersion() {
+    const version = await this.execute(['releases', 'propose-version'], null);
+    return version.trim();
   }
 
   /**
@@ -159,7 +154,7 @@ class Releases {
    * @returns {Promise} A promise that resolves when the upload has completed successfully.
    * @memberof SentryReleases
    */
-  uploadSourceMaps(release, options) {
+  async uploadSourceMaps(release, options) {
     if (!options || !options.include || !Array.isArray(options.include)) {
       throw new Error(
         '`options.include` must be a vaild array of paths and/or path descriptor objects.'
@@ -169,7 +164,7 @@ class Releases {
     // Each entry in the `include` array will map to an array of promises, which
     // will in turn contain one promise per literal path value. Thus `uploads`
     // will be an array of Promise arrays, which we'll flatten later.
-    const uploads = options.include.map(includeEntry => {
+    const uploads = options.include.map((includeEntry) => {
       let pathOptions;
       let uploadPaths;
 
@@ -200,7 +195,7 @@ class Releases {
         .concat(helper.getProjectFlagsFromOptions(options))
         .concat(['files', release, 'upload-sourcemaps']);
 
-      return uploadPaths.map(path =>
+      return uploadPaths.map((path) =>
         // `execute()` is async and thus we're returning a promise here
         this.execute(helper.prepareCommand([...args, path], SOURCEMAPS_SCHEMA, newOptions), true)
       );
@@ -221,7 +216,7 @@ class Releases {
    * @returns {Promise} A promise that resolves when the list comes back from the server.
    * @memberof SentryReleases
    */
-  listDeploys(release) {
+  async listDeploys(release) {
     return this.execute(['releases', 'deploys', release, 'list'], null);
   }
 
@@ -247,7 +242,7 @@ class Releases {
    * @returns {Promise} A promise that resolves when the deploy has been created.
    * @memberof SentryReleases
    */
-  newDeploy(release, options) {
+  async newDeploy(release, options) {
     if (!options || !options.env) {
       throw new Error('options.env must be a vaild name');
     }
@@ -261,7 +256,7 @@ class Releases {
    * @param {boolean} live We inherit stdio to display `sentry-cli` output directly.
    * @returns {Promise.<string>} A promise that resolves to the standard output.
    */
-  execute(args, live) {
+  async execute(args, live) {
     return helper.execute(args, live, this.options.silent, this.configFile, this.options);
   }
 }

package/package.json

@@ -1,28 +1,34 @@
 {
   "name": "@sentry/cli",
-  "version": "1.74.3",
+  "version": "2.0.1",
   "description": "A command line utility to work with Sentry. https://docs.sentry.io/hosted/learn/cli/",
+  "repository": "git://github.com/getsentry/sentry-cli.git",
   "homepage": "https://docs.sentry.io/hosted/learn/cli/",
+  "author": "Sentry",
   "license": "BSD-3-Clause",
-  "keywords": [
-    "sentry",
-    "sentry-cli",
-    "cli"
-  ],
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/getsentry/sentry-cli"
-  },
-  "bugs": {
-    "url": "https://github.com/getsentry/sentry-cli/issues"
-  },
   "engines": {
-    "node": ">= 8"
+    "node": ">= 12"
   },
   "main": "js/index.js",
+  "types": "js/index.d.ts",
   "bin": {
     "sentry-cli": "bin/sentry-cli"
   },
+  "dependencies": {
+    "https-proxy-agent": "^5.0.0",
+    "node-fetch": "^3.2.3",
+    "npmlog": "^6.0.1",
+    "progress": "^2.0.3",
+    "proxy-from-env": "^1.1.0",
+    "which": "^2.0.2"
+  },
+  "devDependencies": {
+    "eslint": "^8.13.0",
+    "eslint-config-prettier": "^8.5.0",
+    "jest": "^27.5.1",
+    "npm-run-all": "^4.1.5",
+    "prettier": "^2.6.2"
+  },
   "scripts": {
     "install": "node ./scripts/install.js",
     "fix": "npm-run-all fix:eslint fix:prettier",
@@ -34,29 +40,11 @@
     "test:eslint": "eslint bin/* scripts/**/*.js js/**/*.js",
     "test:prettier": "prettier --check  bin/* scripts/**/*.js js/**/*.js"
   },
-  "dependencies": {
-    "https-proxy-agent": "^5.0.0",
-    "mkdirp": "^0.5.5",
-    "node-fetch": "^2.6.7",
-    "npmlog": "^4.1.2",
-    "progress": "^2.0.3",
-    "proxy-from-env": "^1.1.0",
-    "which": "^2.0.2"
-  },
-  "devDependencies": {
-    "eslint": "^6.8.0",
-    "eslint-config-airbnb-base": "^14.1.0",
-    "eslint-config-prettier": "^6.10.1",
-    "eslint-plugin-import": "^2.20.2",
-    "jest": "^25.3.0",
-    "npm-run-all": "^4.1.5",
-    "prettier": "^1.19.1"
-  },
   "jest": {
     "collectCoverage": true,
     "testEnvironment": "node",
     "testPathIgnorePatterns": [
-      "src/utils"
+      "<rootDir>/src"
     ]
   }
 }

package/scripts/install.js

@@ -11,12 +11,10 @@ const zlib = require('zlib');
 const stream = require('stream');
 const process = require('process');
 
+const fetch = (...args) => import('node-fetch').then(({ default: fetch }) => fetch(...args));
 const HttpsProxyAgent = require('https-proxy-agent');
-const fetch = require('node-fetch');
 const ProgressBar = require('progress');
 const Proxy = require('proxy-from-env');
-// NOTE: Can be dropped in favor of `fs.mkdirSync(path, { recursive: true })` once we stop supporting Node 8.x
-const mkdirp = require('mkdirp');
 const npmLog = require('npmlog');
 const which = require('which');
 
@@ -46,10 +44,9 @@ function getLogStream(defaultStream) {
 }
 
 function shouldRenderProgressBar() {
-  const silentFlag = process.argv.some(v => v === '--silent');
+  const silentFlag = process.argv.some((v) => v === '--silent');
   const silentConfig = process.env.npm_config_loglevel === 'silent';
-  // Leave `SENTRY_NO_PROGRESS_BAR` for backwards compatibility
-  const silentEnv = process.env.SENTRYCLI_NO_PROGRESS_BAR || process.env.SENTRY_NO_PROGRESS_BAR;
+  const silentEnv = process.env.SENTRYCLI_NO_PROGRESS_BAR;
   const ciEnv = process.env.CI === 'true';
   // If any of possible options is set, skip rendering of progress bar
   return !(silentFlag || silentConfig || silentEnv || ciEnv);
@@ -111,7 +108,7 @@ function createProgressBar(name, total) {
   let pct = null;
   let current = 0;
   return {
-    tick: length => {
+    tick: (length) => {
       current += length;
       const next = Math.round((current / total) * 100);
       if (next > pct) {
@@ -133,11 +130,7 @@ function npmCache() {
 }
 
 function getCachedPath(url) {
-  const digest = crypto
-    .createHash('md5')
-    .update(url)
-    .digest('hex')
-    .slice(0, 6);
+  const digest = crypto.createHash('md5').update(url).digest('hex').slice(0, 6);
 
   return path.join(
     npmCache(),
@@ -147,9 +140,7 @@ function getCachedPath(url) {
 }
 
 function getTempFile(cached) {
-  return `${cached}.${process.pid}-${Math.random()
-    .toString(16)
-    .slice(2)}.tmp`;
+  return `${cached}.${process.pid}-${Math.random().toString(16).slice(2)}.tmp`;
 }
 
 function validateChecksum(tempPath, name) {
@@ -176,10 +167,7 @@ function validateChecksum(tempPath, name) {
     return;
   }
 
-  const currentHash = crypto
-    .createHash('sha256')
-    .update(fs.readFileSync(tempPath))
-    .digest('hex');
+  const currentHash = crypto.createHash('sha256').update(fs.readFileSync(tempPath)).digest('hex');
 
   if (storedHash !== currentHash) {
     fs.unlinkSync(tempPath);
@@ -191,7 +179,7 @@ function validateChecksum(tempPath, name) {
   }
 }
 
-function downloadBinary() {
+async function downloadBinary() {
   const arch = os.arch();
   const platform = os.platform();
   const outputPath = helper.getPath();
@@ -212,14 +200,14 @@ function downloadBinary() {
 
   const downloadUrl = getDownloadUrl(platform, arch);
   if (!downloadUrl) {
-    return Promise.reject(new Error(`Unsupported target ${platform}-${arch}`));
+    throw new Error(`Unsupported target ${platform}-${arch}`);
   }
 
   const cachedPath = getCachedPath(downloadUrl);
   if (fs.existsSync(cachedPath)) {
     npmLog.info('sentry-cli', `Using cached binary: ${cachedPath}`);
     fs.copyFileSync(cachedPath, outputPath);
-    return Promise.resolve();
+    return;
   }
 
   const proxyUrl = Proxy.getProxyForUrl(downloadUrl);
@@ -231,74 +219,77 @@ function downloadBinary() {
     npmLog.info('sentry-cli', `Using proxy URL: ${proxyUrl}`);
   }
 
-  return fetch(downloadUrl, {
-    agent,
-    compress: false,
-    headers: {
-      'accept-encoding': 'gzip, deflate, br',
-    },
-    redirect: 'follow',
-  })
-    .then(response => {
-      if (!response.ok) {
-        throw new Error(
-          `Unable to download sentry-cli binary from ${downloadUrl}.\nServer returned ${response.status}: ${response.statusText}.`
-        );
-      }
-
-      const contentEncoding = response.headers.get('content-encoding');
-      let decompressor;
-      if (/\bgzip\b/.test(contentEncoding)) {
-        decompressor = zlib.createGunzip();
-      } else if (/\bdeflate\b/.test(contentEncoding)) {
-        decompressor = zlib.createInflate();
-      } else if (/\bbr\b/.test(contentEncoding)) {
-        decompressor = zlib.createBrotliDecompress();
-      } else {
-        decompressor = new stream.PassThrough();
-      }
-      const name = downloadUrl.match(/.*\/(.*?)$/)[1];
-      const total = parseInt(response.headers.get('content-length'), 10);
-      const progressBar = createProgressBar(name, total);
-      const tempPath = getTempFile(cachedPath);
-      mkdirp.sync(path.dirname(tempPath));
-
-      return new Promise((resolve, reject) => {
-        response.body
-          .on('error', e => reject(e))
-          .on('data', chunk => progressBar.tick(chunk.length))
-          .pipe(decompressor)
-          .pipe(fs.createWriteStream(tempPath, { mode: '0755' }))
-          .on('error', e => reject(e))
-          .on('close', () => resolve());
-      }).then(() => {
-        if (process.env.SENTRYCLI_SKIP_CHECKSUM_VALIDATION !== '1') {
-          validateChecksum(tempPath, name);
-        }
-        fs.copyFileSync(tempPath, cachedPath);
-        fs.copyFileSync(tempPath, outputPath);
-        fs.unlinkSync(tempPath);
-      });
-    })
-    .catch(error => {
-      if (error instanceof fetch.FetchError) {
-        throw new Error(
-          `Unable to download sentry-cli binary from ${downloadUrl}.\nError code: ${error.code}`
-        );
-      } else {
-        throw error;
-      }
+  let response;
+  try {
+    response = await fetch(downloadUrl, {
+      agent,
+      compress: false,
+      headers: {
+        'accept-encoding': 'gzip, deflate, br',
+      },
+      redirect: 'follow',
     });
-}
-
-function checkVersion() {
-  return helper.execute(['--version']).then(output => {
-    const version = output.replace('sentry-cli ', '').trim();
-    const expected = process.env.SENTRYCLI_LOCAL_CDNURL ? 'DEV' : pkgInfo.version;
-    if (version !== expected) {
-      throw new Error(`Unexpected sentry-cli version "${version}", expected "${expected}"`);
+  } catch (error) {
+    if (error instanceof fetch.FetchError) {
+      throw new Error(
+        `Unable to download sentry-cli binary from ${downloadUrl}.\nError code: ${error.code}`
+      );
+    } else {
+      throw error;
     }
+  }
+
+  if (!response.ok) {
+    throw new Error(
+      `Unable to download sentry-cli binary from ${downloadUrl}.\nServer returned ${
+        response.status
+      }${response.statusText ? `: ${response.statusText}` : ''}.`
+    );
+  }
+
+  const contentEncoding = response.headers.get('content-encoding');
+  let decompressor;
+  if (/\bgzip\b/.test(contentEncoding)) {
+    decompressor = zlib.createGunzip();
+  } else if (/\bdeflate\b/.test(contentEncoding)) {
+    decompressor = zlib.createInflate();
+  } else if (/\bbr\b/.test(contentEncoding)) {
+    decompressor = zlib.createBrotliDecompress();
+  } else {
+    decompressor = new stream.PassThrough();
+  }
+  const name = downloadUrl.match(/.*\/(.*?)$/)[1];
+  const total = parseInt(response.headers.get('content-length'), 10);
+  const progressBar = createProgressBar(name, total);
+  const tempPath = getTempFile(cachedPath);
+  fs.mkdirSync(path.dirname(tempPath), { recursive: true });
+
+  await new Promise((resolve, reject) => {
+    response.body
+      .on('error', (e) => reject(e))
+      .on('data', (chunk) => progressBar.tick(chunk.length))
+      .pipe(decompressor)
+      .pipe(fs.createWriteStream(tempPath, { mode: '0755' }))
+      .on('error', (e) => reject(e))
+      .on('close', () => resolve());
   });
+
+  if (process.env.SENTRYCLI_SKIP_CHECKSUM_VALIDATION !== '1') {
+    validateChecksum(tempPath, name);
+  }
+
+  fs.copyFileSync(tempPath, cachedPath);
+  fs.copyFileSync(tempPath, outputPath);
+  fs.unlinkSync(tempPath);
+}
+
+async function checkVersion() {
+  const output = await helper.execute(['--version']);
+  const version = output.replace('sentry-cli ', '').trim();
+  const expected = process.env.SENTRYCLI_LOCAL_CDNURL ? 'DEV' : pkgInfo.version;
+  if (version !== expected) {
+    throw new Error(`Unexpected sentry-cli version "${version}", expected "${expected}"`);
+  }
 }
 
 if (process.env.SENTRYCLI_LOCAL_CDNURL) {
@@ -323,11 +314,14 @@ if (process.env.SENTRYCLI_SKIP_DOWNLOAD === '1') {
   process.exit(0);
 }
 
-downloadBinary()
-  .then(() => checkVersion())
-  .then(() => process.exit(0))
-  .catch(e => {
+(async () => {
+  try {
+    await downloadBinary();
+    await checkVersion();
+    process.exit(0);
+  } catch (e) {
     // eslint-disable-next-line no-console
     console.error(e.toString());
     process.exit(1);
-  });
+  }
+})();