Skip to content

fix: unify desktop Sentry environment handling#303

Merged
mrcfps merged 1 commit intomainfrom
dev-0320-pr
Mar 20, 2026
Merged

fix: unify desktop Sentry environment handling#303
mrcfps merged 1 commit intomainfrom
dev-0320-pr

Conversation

@nettee
Copy link
Copy Markdown
Contributor

@nettee nettee commented Mar 20, 2026

Summary

This PR ensures desktop build and release pipelines pass an explicit SENTRY_ENV so packaged builds and their sourcemap uploads record the intended environment and DSN, while refreshing the diagnostics/environment docs to match the current tooling.

Changes

  • Require sentry_env input for desktop-build and wire it into build-config generation and release builds
  • Preserve NEXU_SENTRY_ENV through the mac dist script and upload-sourcemaps so project selection is based on environment instead of build source
  • Remove the redundant English diagnostics doc and update the diagnostics/environment rails to match the latest guidance

- ensure the desktop build workflow requires  and plumbs it into generated config

- keep  in the mac dist script so CI builds preserve the runtime context

- change sourcemap upload metadata to derive projects from  instead of build sources

- refresh diagnostics and environment docs to match the new recommendations
@mrcfps mrcfps merged commit bdbeac8 into main Mar 20, 2026
6 checks passed
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.

2 participants