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';
 
 declare module '@ember/debug' {
@@ -21,7 +21,7 @@ export function InitSentryForEmber(_runtimeConfig: BrowserOptions | undefined) {
 
   Sentry.init(initConfig);
 
-  runInDebug(() => {
+  if (macroCondition(isDevelopingApp())) {
     if (config.ignoreEmberOnErrorWarning) {
       return;
     }
@@ -34,7 +34,7 @@ export function InitSentryForEmber(_runtimeConfig: BrowserOptions | undefined) {
         },
       );
     });
-  });
+  }
 }
 
 function createEmberEventProcessor(): void {

packages/ember/package.json

@@ -30,9 +30,11 @@
     "pack": "npm pack"
   },
   "dependencies": {
+    "@embroider/macros": "^0.22.0",
     "@sentry/browser": "5.22.3",
     "@sentry/types": "5.22.3",
     "@sentry/utils": "5.22.3",
+    "@types/babel__core": "^7.1.9",
     "ember-auto-import": "^1.6.0",
     "ember-cli-babel": "^7.20.5",
     "ember-cli-htmlbars": "^5.1.2",