Skip to content

Conversation

Lms24
Copy link
Member

@Lms24 Lms24 commented Jul 18, 2025

This PR bumps @sentry/cli to 2.49.0 which adds a new rejectOnError live mode setting to the execute and releases.uploadSourceMaps APIs.

This new mode now makes the CLI reject the promise it returns if the CLI binary process exits with a non-zero exit code. In the previous live mode, it never rejected but only piped the stdio output, meaning, the plugins didn't even know that the CLI failed.

This should finally fix the "Successfully uploaded source maps" message when in fact we failed during upload.

closes #670

@Lms24 Lms24 changed the title fix(core): Observe Sentry CLI sourcemap upload failures fix(core): Observe and handle Sentry CLI sourcemap upload failures Jul 18, 2025
@Lms24 Lms24 self-assigned this Jul 18, 2025
Copy link
Member

@andreiborza andreiborza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice, good stuff!

@Lms24 Lms24 merged commit 11c6e74 into main Jul 18, 2025
37 of 40 checks passed
@Lms24 Lms24 deleted the lms/cli-uploadSourceMaps-fail-rejectOnError branch July 18, 2025 10:16
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jul 27, 2025
| datasource | package             | from  | to    |
| ---------- | ------------------- | ----- | ----- |
| npm        | @sentry/vite-plugin | 3.5.0 | 3.6.1 |


## [v3.6.1](https://github.com/getsentry/sentry-javascript-bundler-plugins/blob/HEAD/CHANGELOG.md#361)

- fix(core): Observe and handle Sentry CLI sourcemap upload failures ([#763](getsentry/sentry-javascript-bundler-plugins#763))


## [v3.6.0](https://github.com/getsentry/sentry-javascript-bundler-plugins/blob/HEAD/CHANGELOG.md#360)

- feat(core): Don't add `debugIdUploadPlugin` when sourcemaps option is disabled ([#753](getsentry/sentry-javascript-bundler-plugins#753))
- fix(core): Avoid showing success message if upload was disabled or nothing was uploaded ([#757](getsentry/sentry-javascript-bundler-plugins#757))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bundler plugin says upload successful, when Sentry CLI fails
2 participants