Use @embroider/macros instead of runInDebug for @sentry/ember

This ensures that the debug code is actually stripped from production builds.

Author: mydea

packages/ember/addon/index.ts

@@ -1,9 +1,9 @@
 import * as Sentry from '@sentry/browser';
 import { addGlobalEventProcessor, SDK_VERSION, BrowserOptions } from '@sentry/browser';
 import environmentConfig from 'ember-get-config';
-
+import { macroCondition, isDevelopingApp } from '@embroider/macros';
 import { next } from '@ember/runloop';
-import { assert, warn, runInDebug } from '@ember/debug';
+import { assert, warn } from '@ember/debug';
 import Ember from 'ember';
 import { timestampWithMs } from '@sentry/utils';
 
@@ -22,7 +22,7 @@ export function InitSentryForEmber(_runtimeConfig: BrowserOptions | undefined) {
 
   Sentry.init(initConfig);
 
-  runInDebug(() => {
+  if (macroCondition(isDevelopingApp())) {
     if (config.ignoreEmberOnErrorWarning) {
       return;
     }
@@ -35,7 +35,7 @@ export function InitSentryForEmber(_runtimeConfig: BrowserOptions | undefined) {
         },
       );
     });
-  });
+  }
 }
 
 export const getActiveTransaction = () => {

packages/ember/package.json

@@ -31,6 +31,7 @@
     "pack": "npm pack"
   },
   "dependencies": {
+    "@embroider/macros": ">= 0.25.0",
     "@sentry/browser": "5.24.2",
     "@sentry/tracing": "5.24.2",
     "@sentry/types": "5.24.2",