Fix #532, improve lazy Sentry setup

Sentry was being included directly in the popup, but not properly configured. This adds catcherAsyncSetup, and makes corrections where that script expected Raven

extension/catcher.js

@@ -54,7 +54,7 @@ this.catcher = (function() {
       },
     };
     if (typeof Sentry === "undefined") {
-      window.SENTRY_DSK = sentryOptions;
+      window.SENTRY_SDK = sentryOptions;
       catcherAsyncSetup();
     } else {
       Sentry.init(sentryOptions);

extension/popup/popup.html

@@ -11,7 +11,7 @@
     <script src="../js/vendor/react-dom.production.min.js"></script>
     <script src="../buildSettings.js"></script>
     <script src="../log.js"></script>
-    <script src="../js/vendor/sentry.js"></script>
+    <script src="../catcherAsyncSetup.js"></script>
     <script src="../catcher.js"></script>
     <script src="../util.js"></script>
     <script src="../settings.js"></script>

extension/recorder/recorder.html

@@ -34,6 +34,7 @@ <h1>Error</h1>
     <script src="../buildSettings.js"></script>
     <script src="../log.js"></script>
     <script src="../js/vendor/sentry.js"></script>
+    <script src="../catcherAsyncSetup.js"></script>
     <script src="../catcher.js"></script>
     <script src="../util.js"></script>
     <script src="../popup/vad.js"></script>

package.json

@@ -59,7 +59,7 @@
     "start-extension": "mkdir -p ${PROFILE:-Profile} && web-ext run --firefox-profile ${PROFILE:-Profile}/ --keep-profile-changes --firefox \"${FIREFOX:-nightly}\" --source-dir extension/ --browser-console",
     "test": "npm-run-all build:manifest lint",
     "build:markdown": "mkdir -p extension/views && node bin/substitute-changelog.js",
-    "build:deps": "mkdir -p extension/js/vendor/ extension/css/vendor/ ; for file in lottie-web/build/player/lottie.min.js webextension-polyfill/dist/browser-polyfill.min.js moment/min/moment.min.js fuse.js/dist/fuse.js raven-js/dist/raven.js react/umd/react.production.min.js react-dom/umd/react-dom.production.min.js; do cp node_modules/$file extension/js/vendor/ ; done ; for file in tailwindcss/dist/tailwind.min.css ; do cp node_modules/$file extension/css/vendor/ ; done ; cp \"node_modules/@sentry/browser/build/bundle.es6.min.js\" extension/js/vendor/sentry.js",
+    "build:deps": "mkdir -p extension/js/vendor/ extension/css/vendor/ ; for file in lottie-web/build/player/lottie.min.js webextension-polyfill/dist/browser-polyfill.min.js moment/min/moment.min.js fuse.js/dist/fuse.js react/umd/react.production.min.js react-dom/umd/react-dom.production.min.js; do cp node_modules/$file extension/js/vendor/ ; done ; for file in tailwindcss/dist/tailwind.min.css ; do cp node_modules/$file extension/css/vendor/ ; done ; cp \"node_modules/@sentry/browser/build/bundle.es6.min.js\" extension/js/vendor/sentry.js",
     "build:manifest": "node bin/substitute-manifest.js",
     "build:jsx": "babel --plugins @babel/plugin-transform-react-jsx,@babel/plugin-proposal-class-properties extension/popup/*.jsx --out-dir extension/popup",
     "watch": "babel --plugins @babel/plugin-transform-react-jsx,@babel/plugin-proposal-class-properties extension/popup/*.jsx --watch --out-dir extension/popup",