From 8729b639c5b5af63bc45e7c75c17497a17930b02 Mon Sep 17 00:00:00 2001 From: s1gr1d <32902192+s1gr1d@users.noreply.github.com> Date: Tue, 28 Apr 2026 17:02:55 +0200 Subject: [PATCH 1/2] docs: Add deprecation notices to `bin` scripts --- packages/profiling-node/README.md | 6 ++++-- .../scripts/prune-profiler-binaries.js | 4 ++++ packages/remix/README.md | 6 +++--- packages/remix/scripts/sentry-upload-sourcemaps.js | 12 +++++++++++- 4 files changed, 22 insertions(+), 6 deletions(-) diff --git a/packages/profiling-node/README.md b/packages/profiling-node/README.md index 51e447640c14..b5957f023b10 100644 --- a/packages/profiling-node/README.md +++ b/packages/profiling-node/README.md @@ -238,8 +238,10 @@ Once you run `node esbuild.serverless.js` esbuild wil bundle and output the file the binaries will be copied. This is wasteful as you will likely only need one of these libraries to be available during runtime. -To prune the other libraries, profiling-node ships with a small utility script that helps you prune unused binaries. The -script can be invoked via `sentry-prune-profiler-binaries`: +> **Deprecation notice:** This script will be removed in the next major version. If you depend on it, please comment on +> [this issue](https://github.com/getsentry/sentry-javascript/issues/20567). + +To prune the other libraries, profiling-node ships with a small utility script that helps you prune unused binaries: ```bash npx --package=@sentry/profiling-node sentry-prune-profiler-binaries diff --git a/packages/profiling-node/scripts/prune-profiler-binaries.js b/packages/profiling-node/scripts/prune-profiler-binaries.js index 4314c2cb7fb2..11e8dc7f05f4 100755 --- a/packages/profiling-node/scripts/prune-profiler-binaries.js +++ b/packages/profiling-node/scripts/prune-profiler-binaries.js @@ -56,6 +56,10 @@ Arguments:\n process.exit(0); } +console.warn( + '[Sentry] Warning: This script will be removed in the next major version. See: https://github.com/getsentry/sentry-javascript/issues/20567', +); + const ARGV_ERRORS = []; const NODE_TO_ABI = { diff --git a/packages/remix/README.md b/packages/remix/README.md index 2589ed9f7e6b..ddb0c643a869 100644 --- a/packages/remix/README.md +++ b/packages/remix/README.md @@ -122,13 +122,13 @@ Sentry.captureEvent({ The Remix SDK provides a script that automatically creates a release and uploads sourcemaps. To generate sourcemaps with Remix, you need to call `remix build` with the `--sourcemap` option. -On release, call `sentry-upload-sourcemaps` to upload source maps and create a release: +On release, call the upload sourcemaps command to upload source maps and create a release: ```bash -npx --package=@sentry/remix sentry-upload-sourcemaps +npx @sentry/remix --upload-sourcemaps ``` -To see more details on how to use the command, call `npx --package=@sentry/remix sentry-upload-sourcemaps --help`. +To see more details on how to use the command, run `npx @sentry/remix --upload-sourcemaps --help`. For more advanced configuration, [directly use `sentry-cli` to upload source maps.](https://github.com/getsentry/sentry-cli). diff --git a/packages/remix/scripts/sentry-upload-sourcemaps.js b/packages/remix/scripts/sentry-upload-sourcemaps.js index 625526af5a8a..2ed0e1d1078e 100755 --- a/packages/remix/scripts/sentry-upload-sourcemaps.js +++ b/packages/remix/scripts/sentry-upload-sourcemaps.js @@ -8,6 +8,10 @@ const DEFAULT_URL_PREFIX = '~/build/'; const DEFAULT_BUILD_PATH = 'public/build'; const argv = yargs(process.argv.slice(2)) + .option('upload-sourcemaps', { + type: 'boolean', + describe: 'Specifies the upload sourcemaps command. Recommended for forward compatibility.', + }) .option('release', { type: 'string', describe: @@ -48,7 +52,7 @@ const argv = yargs(process.argv.slice(2)) default: true, }) .usage( - 'Usage: $0\n' + + 'Usage: npx @sentry/remix --upload-sourcemaps\n' + ' [--release RELEASE]\n' + ' [--org ORG]\n' + ' [--project PROJECT]\n' + @@ -64,6 +68,12 @@ const argv = yargs(process.argv.slice(2)) ) .wrap(120).argv; +if (!argv.uploadSourcemaps) { + process.stderr.write( + '[Sentry] Warning: Calling this script without --upload-sourcemaps is deprecated. Use: `npx @sentry/remix --upload-sourcemaps`', + ); +} + const buildPath = argv.buildPath || DEFAULT_BUILD_PATH; const urlPrefix = argv.urlPrefix || DEFAULT_URL_PREFIX; From ae9b8c61312add0778e97311626c2793218df0cc Mon Sep 17 00:00:00 2001 From: s1gr1d <32902192+s1gr1d@users.noreply.github.com> Date: Tue, 28 Apr 2026 17:16:35 +0200 Subject: [PATCH 2/2] add trailing newline --- packages/remix/scripts/sentry-upload-sourcemaps.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/remix/scripts/sentry-upload-sourcemaps.js b/packages/remix/scripts/sentry-upload-sourcemaps.js index 2ed0e1d1078e..ceb41d3ac14c 100755 --- a/packages/remix/scripts/sentry-upload-sourcemaps.js +++ b/packages/remix/scripts/sentry-upload-sourcemaps.js @@ -70,7 +70,7 @@ const argv = yargs(process.argv.slice(2)) if (!argv.uploadSourcemaps) { process.stderr.write( - '[Sentry] Warning: Calling this script without --upload-sourcemaps is deprecated. Use: `npx @sentry/remix --upload-sourcemaps`', + '[Sentry] Warning: Calling this script without --upload-sourcemaps is deprecated. Use: `npx @sentry/remix --upload-sourcemaps`\n', ); }