feat(remix): Add debugid injection and map deletion to sourcemaps script

Author: onurtemizkan

packages/remix/package.json

@@ -27,7 +27,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@sentry/cli": "2.2.0",
+    "@sentry/cli": "2.20.4",
     "@sentry/core": "7.64.0",
     "@sentry/node": "7.64.0",
     "@sentry/react": "7.64.0",

packages/remix/scripts/createRelease.js

@@ -1,19 +1,31 @@
+/* eslint-disable no-console */
 const SentryCli = require('@sentry/cli');
+
+const { deleteSourcemaps } = require('./deleteSourcemaps');
+
 const sentry = new SentryCli();
 
-async function createRelease(argv, DEFAULT_URL_PREFIX, DEFAULT_BUILD_PATH) {
+async function createRelease(argv, URL_PREFIX, BUILD_PATH) {
   const RELEASE = argv.release || (await sentry.releases.proposeVersion());
-  const URL_PREFIX = argv.urlPrefix || DEFAULT_URL_PREFIX;
-  const BUILD_PATH = argv.buildPath || DEFAULT_BUILD_PATH;
 
   await sentry.releases.new(RELEASE);
 
   await sentry.releases.uploadSourceMaps(RELEASE, {
     urlPrefix: URL_PREFIX,
     include: [BUILD_PATH],
+    useArtifactBundle: !argv.disableDebugIds,
   });
 
   await sentry.releases.finalize(RELEASE);
+
+  if (argv.deleteAfterUpload) {
+    try {
+      deleteSourcemaps(BUILD_PATH);
+    } catch (error) {
+      console.warn(`Failed to delete sourcemaps in build directory: ${BUILD_PATH}`);
+      console.error(error);
+    }
+  }
 }
 
 module.exports = {

packages/remix/scripts/deleteSourcemaps.js

@@ -0,0 +1,21 @@
+/* eslint-disable no-console */
+const fs = require('fs');
+const path = require('path');
+
+function deleteSourcemaps(buildPath) {
+  console.info(`Deleting sourcemaps from ${buildPath}`);
+
+  const files = fs.readdirSync(buildPath);
+
+  files.forEach(file => {
+    if (file.endsWith('.map')) {
+      fs.unlinkSync(path.join(buildPath, file));
+
+      console.info(`Deleted ${file}`);
+    }
+  });
+}
+
+module.exports = {
+  deleteSourcemaps,
+};

packages/remix/scripts/injectDebugId.js

@@ -0,0 +1,15 @@
+/* eslint-disable no-console */
+const { execSync } = require('child_process');
+
+function injectDebugId(buildPath) {
+  try {
+    execSync(`node ./node_modules/@sentry/cli/bin/sentry-cli sourcemaps inject ${buildPath}`);
+  } catch (error) {
+    console.warn('Failed to inject debug ids.');
+    console.error(error);
+  }
+}
+
+module.exports = {
+  injectDebugId,
+};

packages/remix/scripts/sentry-upload-sourcemaps.js

@@ -1,7 +1,9 @@
 #!/usr/bin/env node
+/* eslint-disable no-console */
 const yargs = require('yargs');
 
 const { createRelease } = require('./createRelease');
+const { injectDebugId } = require('./injectDebugId');
 
 const DEFAULT_URL_PREFIX = '~/build/';
 const DEFAULT_BUILD_PATH = 'public/build';
@@ -24,16 +26,35 @@ const argv = yargs(process.argv.slice(2))
     describe: 'The path to the build directory',
     default: DEFAULT_BUILD_PATH,
   })
+  .option('disableDebugIds', {
+    type: 'boolean',
+    describe: 'Disable the injection and upload of debug ids',
+    default: false,
+  })
+  .option('deleteAfterUpload', {
+    type: 'boolean',
+    describe: 'Delete sourcemaps after uploading',
+    default: true,
+  })
   .usage(
     'Usage: $0\n' +
       '  [--release RELEASE]\n' +
       '  [--urlPrefix URL_PREFIX]\n' +
       '  [--buildPath BUILD_PATH]\n\n' +
+      '  [--disableDebugIds true|false]\n\n' +
+      '  [--deleteAfterUpload true|false]\n\n' +
       'This CLI tool will upload sourcemaps to Sentry for the given release.\n' +
       'It has defaults for URL prefix and build path for Remix builds, but you can override them.\n\n' +
       'If you need a more advanced configuration, you can use `sentry-cli` instead.\n' +
       'https://github.com/getsentry/sentry-cli',
   )
   .wrap(120).argv;
 
-createRelease(argv, DEFAULT_URL_PREFIX, DEFAULT_BUILD_PATH);
+const buildPath = argv.buildPath || DEFAULT_BUILD_PATH;
+const urlPrefix = argv.urlPrefix || DEFAULT_URL_PREFIX;
+
+if (!argv.disableDebugIds) {
+  injectDebugId(buildPath);
+}
+
+createRelease(argv, urlPrefix, buildPath);

packages/remix/test/scripts/upload-sourcemaps.test.ts

@@ -36,6 +36,7 @@ describe('createRelease', () => {
     expect(uploadSourceMapsMock).toHaveBeenCalledWith('0.1.2.3', {
       urlPrefix: '~/build/',
       include: ['public/build'],
+      useArtifactBundle: true,
     });
     expect(finalizeMock).toHaveBeenCalledWith('0.1.2.3');
   });
@@ -48,6 +49,7 @@ describe('createRelease', () => {
     expect(uploadSourceMapsMock).toHaveBeenCalledWith('0.1.2.3.4', {
       urlPrefix: '~/build/',
       include: ['public/build'],
+      useArtifactBundle: true,
     });
     expect(finalizeMock).toHaveBeenCalledWith('0.1.2.3.4');
   });
@@ -67,6 +69,7 @@ describe('createRelease', () => {
     expect(uploadSourceMapsMock).toHaveBeenCalledWith('0.1.2.3.4', {
       urlPrefix: '~/build/subfolder',
       include: ['public/build/subfolder'],
+      useArtifactBundle: true,
     });
     expect(finalizeMock).toHaveBeenCalledWith('0.1.2.3.4');
   });

yarn.lock

@@ -4428,14 +4428,13 @@
     unplugin "1.0.1"
     webpack-sources "3.2.3"
 
-"@sentry/cli@2.2.0":
-  version "2.2.0"
-  resolved "https://registry.yarnpkg.com/@sentry/cli/-/cli-2.2.0.tgz#0cf4d529d87e290dea54d7e58fa5ff87ea200e4e"
-  integrity sha512-ywFtB8VHyWN248LuM67fsRtdMLif/SOHYY3zyef5WybvnAmRLDmGTWK//hSUCebsHBpehRIkmt4iMiyUXwgd5w==
+"@sentry/cli@2.20.4":
+  version "2.20.4"
+  resolved "https://registry.yarnpkg.com/@sentry/cli/-/cli-2.20.4.tgz#755719aeaa58dbd3910ee23cc020decc16f5a5bf"
+  integrity sha512-mycBCKQmLdaJpSKpXc3IXrQwb5HLxrdjgJgLGbb8saFHIkXl/1ePH171j+REq4dIP1uetu0bdDgEYj6AQjkleQ==
   dependencies:
     https-proxy-agent "^5.0.0"
     node-fetch "^2.6.7"
-    npmlog "^6.0.1"
     progress "^2.0.3"
     proxy-from-env "^1.1.0"
     which "^2.0.2"
@@ -20552,7 +20551,7 @@ npmlog@^4.1.2:
     gauge "~2.7.3"
     set-blocking "~2.0.0"
 
-npmlog@^6.0.0, npmlog@^6.0.1, npmlog@^6.0.2:
+npmlog@^6.0.0, npmlog@^6.0.2:
   version "6.0.2"
   resolved "https://registry.yarnpkg.com/npmlog/-/npmlog-6.0.2.tgz#c8166017a42f2dea92d6453168dd865186a70830"
   integrity sha512-/vBvz5Jfr9dT/aFWd0FIRf+T/Q2WBsLENygUaFUqstqsycmZAP/t5BvFJTK0viFmSUxiUKTUplWy5vt+rvKIxg==