getsentry / getsentry/sentry-javascript
(breaking) Change default for `sourcemaps.ignore` to `["node_modules"]`
Open
Improvement
javascript
Meta: Breaking
- Dominant language
- TypeScript
- Stars
- 8.7k
- Forks
- 1.8k
- Avg merge
- 1d 17h
- Merged PRs (30d)
- 515
Description
Currently, the CLI applies defaults for the `ignore` value. To align the behavior better, we would move this default decision one level up to the bundler plugins.
How it is currently:
| CLI | Bundler Plugins | Meta-Framework SDK |
|---|---|---|
| adds `'node_modules'` as default | uses CLI in plugin manager (no defaults) | uses bundler plugins (no defaults) |
More context on this in this thread: https://github.com/getsentry/sentry-javascript-bundler-plugins/pull/836#discussion_r2694358688
Contributor guide
Assessment
This issue has not been assessed yet.