Fix tree shaking

Author: spalmurray

src/background/main.ts

@@ -1,5 +1,9 @@
 import browser from "webextension-polyfill";
-import * as Sentry from "@sentry/browser";
+import {
+  init as sentryInit,
+  browserTracingIntegration,
+  startSpan,
+} from "@sentry/browser";
 
 import { MessageType } from "src/types";
 import { Codecov } from "src/service";
@@ -11,12 +15,12 @@ import {
 async function main(): Promise<void> {
   browser.runtime.onMessage.addListener(handleMessages);
 
-  Sentry.init({
+  sentryInit({
     // @ts-ignore SENTRY_DSN is populated by Webpack at build time
     dsn: SENTRY_DSN,
 
     integrations: [
-      Sentry.browserTracingIntegration({
+      browserTracingIntegration({
         // disable automatic span creation
         instrumentNavigation: false,
         instrumentPageLoad: false,
@@ -33,7 +37,7 @@ async function handleMessages(message: {
   referrer?: string;
 }) {
   const codecov = new Codecov();
-  return Sentry.startSpan({ name: message.type }, async () => {
+  return startSpan({ name: message.type }, async () => {
     switch (message.type) {
       case MessageType.FETCH_COMMIT_REPORT:
         return codecov.fetchCommitReport(message.payload, message.referrer!);

src/content/common/sentry.ts

@@ -1,33 +1,35 @@
 import {
+  breadcrumbsIntegration,
+  browserApiErrorsIntegration,
   BrowserClient,
   defaultStackParser,
-  getDefaultIntegrations,
+  globalHandlersIntegration,
   makeFetchTransport,
+  dedupeIntegration,
   Scope,
-} from '@sentry/browser';
+} from "@sentry/browser";
 
 // Sentry config
-// Browser extensions must initialize Sentry a bit differently to avoid 
+// Browser extensions must initialize Sentry a bit differently to avoid
 // conflicts between Sentry instances should the site the extension is running
 // on also use Sentry. Read more here:
 // https://docs.sentry.io/platforms/javascript/best-practices/browser-extensions/
 
-const sentryIntegrations = getDefaultIntegrations({}).filter(defaultIntegration => {
-  return !['BrowserApiErrors', 'TryCatch', 'Breadcrumbs', 'GlobalHandlers'].includes(
-    defaultIntegration.name
-  );
-});
-
 const sentryClient = new BrowserClient({
   // @ts-ignore SENTRY_DSN is populated by Webpack at build time
   dsn: SENTRY_DSN,
   transport: makeFetchTransport,
   stackParser: defaultStackParser,
-  integrations: sentryIntegrations,
+  integrations: [
+    breadcrumbsIntegration,
+    browserApiErrorsIntegration,
+    globalHandlersIntegration,
+    dedupeIntegration,
+  ],
 });
 
 const Sentry = new Scope();
 Sentry.setClient(sentryClient);
 sentryClient.init();
 
-export default Sentry
+export default Sentry;

tsconfig.json

@@ -1,7 +1,7 @@
 {
   "compilerOptions": {
     "strict": true,
-    "module": "commonjs",
+    "module": "es6",
     "target": "es6",
     "esModuleInterop": true,
     "sourceMap": false,
@@ -15,6 +15,7 @@
     "outDir": "dist/js",
     "noEmitOnError": true,
     "jsx": "react",
-    "typeRoots": ["node_modules/@types"]
+    "typeRoots": ["node_modules/@types"],
+    "moduleResolution": "node"
   }
 }