getsentry / getsentry/sentry-javascript

Convert `@sentry/ember` addon to V2 addon format

Open
#15,835 25 comments 1 reaction 0 assignees View on GitHub
Ember Feature Meta: Help Wanted
Dominant language
TypeScript
Stars
8.7k
Forks
1.8k
Avg merge
1d 17h
Merged PRs (30d)
515

Description

### Problem Statement

Modern Ember build tools (`@embroider/vite` and `@embroider/webpack`) benefit greatly from static, pre-built addons when bundling the app.

The current `@sentry/ember` package is currently on the old ember addon-format. It works, but it is not optimized for the modern ember build tooling, as it does currently needs be processed/rewritten by `@embroider` on every build.

Also, it is currently sorted with warnings in the group of other legacy V1 addons when running e.g. `npx ember-addon-v2-scanner@latest`:

| NPM Name | Version | Current version | On V2 Addon | V2 Addon available |
|----------|:-------------:|------:|------:|------:|
| '@sentry/ember' | '9.9.0' | '9.9.0' |❌ |❌ |

(related to https://github.com/getsentry/sentry-javascript/issues/12336, but this is more specific)

### Solution Brainstorm

Here is a [guide for porting an addon to the V2 format ](https://github.com/embroider-build/embroider/blob/main/docs/porting-addons-to-v2.md).

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.