Fix sentry error ignoring

Author: CompuIves

package.json

@@ -85,9 +85,9 @@
     "husky": "^2.2.0",
     "lerna": "^3.16.4",
     "lint-staged": "^9.2.5",
-    "prettier": "1.18.2",
+    "prettier": "1.19.0",
     "pretty-quick": "^1.10.0",
-    "typescript": "3.7.1-rc",
+    "typescript": "3.7.2",
     "username": "^5.1.0"
   },
   "husky": {

packages/app/package.json

@@ -312,7 +312,7 @@
     "terser": "^4.1.4",
     "terser-webpack-plugin": "^1.4.1",
     "thread-loader": "^2.1.2",
-    "typescript": "3.7.1-rc",
+    "typescript": "3.7.2",
     "url-loader": "1.0.1",
     "webpack": "^4.36.1",
     "webpack-bundle-analyzer": "^2.13.1",

packages/codesandbox-api/package.json

@@ -94,7 +94,7 @@
     "tslint-config-prettier": "^1.1.0",
     "tslint-config-standard": "^6.0.0",
     "typedoc": "^0.7.1",
-    "typescript": "3.7.1-rc",
+    "typescript": "3.7.2",
     "uuid": "^3.3.2",
     "validate-commit-msg": "^2.12.2"
   },

packages/common/package.json

@@ -108,7 +108,7 @@
     "jest-styled-components": "^6.3.3",
     "jest-svg-transformer": "^1.0.0",
     "rimraf": "^2.6.3",
-    "typescript": "3.7.1-rc",
+    "typescript": "3.7.2",
     "yarn": "^1.17.3"
   }
 }

packages/common/src/utils/analytics/sentry.ts

@@ -1,9 +1,9 @@
 import VERSION from '../../version';
 import { DO_NOT_TRACK_ENABLED } from './utils';
 
-let _Sentry;
+let _Sentry: typeof import('@sentry/browser');
 
-function getSentry() {
+function getSentry(): Promise<typeof import('@sentry/browser')> {
   return import(/* webpackChunkName: 'sentry' */ '@sentry/browser');
 }
 
@@ -18,7 +18,12 @@ export async function initialize(dsn: string) {
         'Custom Object', // Called for errors coming from sandbox (https://sentry.io/organizations/codesandbox/issues/965255074/?project=155188&query=is%3Aunresolved&statsPeriod=14d)
         'TypeScript Server Error', // Called from the TSC server
         /^Canceled$/, // Used by VSCode to stop currently running actions
+
+        // Chrome extensions
+        /extensions\//i,
+        /^chrome:\/\//i,
       ],
+      whitelistUrls: [/https?:\/\/((uploads|www)\.)?codesandbox\.io/],
       /**
        * Don't send messages from the sandbox, so don't send from eg.
        * new.codesandbox.io or new.csb.app
@@ -29,10 +34,7 @@ export async function initialize(dsn: string) {
         /.*\.csb\.app/,
       ],
       beforeSend: (event, hint) => {
-        if (
-          event?.stacktrace?.frames &&
-          event.stacktrace.frames[0]
-        ) {
+        if (event?.stacktrace?.frames && event.stacktrace.frames[0]) {
           const { filename } = event.stacktrace.frames[0];
 
           if (
@@ -41,7 +43,7 @@ export async function initialize(dsn: string) {
             event.message.includes('too much recursion')
           ) {
             // https://sentry.io/organizations/codesandbox/issues/1293123855/events/b01ee0feb7e3415a8bb81b6a9df19152/?project=155188&query=is%3Aunresolved&statsPeriod=14d
-            return undefined;
+            return null;
           }
 
           if (
@@ -51,13 +53,13 @@ export async function initialize(dsn: string) {
             // This is the spammy event that doesn't do anything: https://sentry.io/organizations/codesandbox/issues/1054971728/?project=155188&query=is%3Aunresolved
             // Don't do anything with it right now, I can't seem to reproduce it for some reason.
             // We need to add sourcemaps
-            return undefined;
+            return null;
           }
 
           if (filename.includes('react-devtools-inline')) {
             // Outside of our scope for now, but we definitely want to check this out.
             // TODO: check what's happening here: https://sentry.io/organizations/codesandbox/issues/1239466583/?project=155188&query=is%3Aunresolved+release%3APROD-1573653062-4134efc0a
-            return undefined;
+            return null;
           }
         }
 
@@ -70,19 +72,17 @@ export async function initialize(dsn: string) {
           event.message.startsWith('Non-Error exception captured')
         ) {
           // This is an error coming from the sandbox, return with no error.
-          return undefined;
+          return null;
         }
 
         if (
           event.message &&
           event.message.includes('Unexpected frame by generating stack.')
         ) {
           // A firefox error with error-polyfill, not critical. Referenced here: https://sentry.io/organizations/codesandbox/issues/1293236389/?project=155188&query=is%3Aunresolved
-          return undefined;
+          return null;
         }
 
-
-
         return event;
       },
     });
@@ -91,30 +91,30 @@ export async function initialize(dsn: string) {
   return Promise.resolve();
 }
 
-export const captureException = (err) => {
+export const captureException = err => {
   if (_Sentry) {
     _Sentry.captureException(err);
   }
-}
+};
 
-export const configureScope = (cb) => {
+export const configureScope = cb => {
   if (_Sentry) {
     _Sentry.configureScope(cb);
-  } 
-}
+  }
+};
 
-export const setUserId = (userId) => {
+export const setUserId = userId => {
   if (_Sentry) {
     _Sentry.configureScope(scope => {
       scope.setUser({ id: userId });
     });
   }
-}
+};
 
 export const resetUserId = () => {
   if (_Sentry) {
     _Sentry.configureScope(scope => {
       scope.setUser({ id: undefined });
     });
   }
-}
+};

packages/executors/package.json

@@ -39,7 +39,6 @@
     "@types/jest": "^24.0.13",
     "@types/socket.io-client": "^1.4.32",
     "husky": "^2.4.0",
-    "prettier": "^1.18.2",
     "pretty-quick": "^1.11.0",
     "tsdx": "^0.6.1",
     "tslib": "^1.9.3",

standalone-packages/react-sandpack/package.json

@@ -126,7 +126,7 @@
     "tslint-config-prettier": "^1.1.0",
     "tslint-config-standard": "^7.0.0",
     "typedoc": "^0.10.0",
-    "typescript": "3.7.1-rc",
+    "typescript": "3.7.2",
     "validate-commit-msg": "^2.12.2"
   },
   "peerDependencies": {

standalone-packages/sandpack/package.json

@@ -95,7 +95,7 @@
     "tslint-config-prettier": "^1.1.0",
     "tslint-config-standard": "^7.0.0",
     "typedoc": "^0.11.1",
-    "typescript": "3.7.1-rc",
+    "typescript": "3.7.2",
     "validate-commit-msg": "^2.12.2"
   },
   "engines": {

yarn.lock

@@ -20725,7 +20725,12 @@ preserve@^0.2.0:
   version "0.2.0"
   resolved "https://registry.yarnpkg.com/preserve/-/preserve-0.2.0.tgz#815ed1f6ebc65926f865b310c0713bcb3315ce4b"
 
-[email protected], prettier@^1.14.2, prettier@^1.14.3, prettier@^1.16.4, prettier@^1.17.0, prettier@^1.18.2:
+[email protected]:
+  version "1.19.0"
+  resolved "https://registry.yarnpkg.com/prettier/-/prettier-1.19.0.tgz#3bec4489d5eebcd52b95ddd2c22467b5c852fde1"
+  integrity sha512-GlAIjk6DjkNT6u/Bw5QCWrbzh9YlLKwwmJT//1YiCR3WDpZDnyss64aXHQZgF8VKeGlWnX6+tGsKSVxsZT/gtA==
+
+prettier@^1.14.2, prettier@^1.14.3, prettier@^1.16.4, prettier@^1.17.0:
   version "1.18.2"
   resolved "https://registry.yarnpkg.com/prettier/-/prettier-1.18.2.tgz#6823e7c5900017b4bd3acf46fe9ac4b4d7bda9ea"
   integrity sha512-OeHeMc0JhFE9idD4ZdtNibzY0+TPHSpSSb9h8FqtP+YnoZZ1sl8Vc9b1sasjfymH3SonAF4QcA2+mzHPhMvIiw==
@@ -25841,10 +25846,10 @@ [email protected]:
   resolved "https://registry.yarnpkg.com/typescript/-/typescript-2.3.4.tgz#3d38321828231e434f287514959c37a82b629f42"
   integrity sha1-PTgyGCgjHkNPKHUUlZw3qCtin0I=
 
-[email protected].1-rc:
-  version "3.7.1-rc"
-  resolved "https://registry.yarnpkg.com/typescript/-/typescript-3.7.1-rc.tgz#2054b872d67f8dc732e36c1df397f9327f37ada0"
-  integrity sha512-2rMtWppLsaPvmpXsoIAXWDBQVnJMw1ITGGSnidMuayLj9iCmMRT69ncKZw/Mk5rXfJkilApKucWQZxproALoRw==
+[email protected].2:
+  version "3.7.2"
+  resolved "https://registry.yarnpkg.com/typescript/-/typescript-3.7.2.tgz#27e489b95fa5909445e9fef5ee48d81697ad18fb"
+  integrity sha512-ml7V7JfiN2Xwvcer+XAf2csGO1bPBdRbFCkYBczNZggrBZ9c7G3riSUeJmqEU5uOtXNPMhE3n+R4FA/3YOAWOQ==
 
 typography-breakpoint-constants@^0.15.10:
   version "0.15.10"