build: configure static globals to control bundle/debug code

Author: mitsuhiko

globals.d.ts

@@ -0,0 +1,2 @@
+declare const SENTRY_BROWSER_BUNDLE: boolean | undefined;
+declare const SENTRY_NDBEUG: boolean | undefined;

packages/browser/rollup.config.js

@@ -9,6 +9,12 @@ const commitHash = require('child_process')
   .trim();
 
 const terserInstance = terser({
+  compress: {
+    global_defs: {
+      SENTRY_BROWSER_BUNDLE: true,
+      SENTRY_NDEBUG: true,
+    },
+  },
   mangle: {
     // captureExceptions and captureMessage are public API methods and they don't need to be listed here
     // as mangler doesn't touch user-facing thing, however sentryWrapped is not, and it would be mangled into a minified version.

packages/browser/tsconfig.json

@@ -1,7 +1,7 @@
 {
   "extends": "../../tsconfig.json",
 
-  "include": ["src/**/*"],
+  "include": ["src/**/*", "../../globals.d.ts"],
 
   "compilerOptions": {
     // package-specific options

packages/utils/src/global.ts

@@ -24,6 +24,8 @@ interface SentryGlobal {
     hub: any;
     logger: any;
   };
+  SENTRY_BROWSER_BUNDLE?: boolean;
+  SENTRY_NDEBUG?: boolean;
 }
 
 const fallbackGlobalObject = {};

packages/utils/src/node.ts

@@ -9,7 +9,11 @@
  * @returns Answer to given question
  */
 export function isNodeEnv(): boolean {
-  return Object.prototype.toString.call(typeof process !== 'undefined' ? process : 0) === '[object process]';
+  if (SENTRY_BROWSER_BUNDLE) {
+    return false;
+  } else {
+    return Object.prototype.toString.call(typeof process !== 'undefined' ? process : 0) === '[object process]';
+  }
 }
 
 /**

packages/utils/tsconfig.json

@@ -1,7 +1,7 @@
 {
   "extends": "../../tsconfig.json",
 
-  "include": ["src/**/*"],
+  "include": ["src/**/*", "../../globals.d.ts"],
 
   "compilerOptions": {
     // package-specific options

tsconfig.json

@@ -5,5 +5,8 @@
     "allowSyntheticDefaultImports": true,
     "types": ["node"],
     "noErrorTruncation": true // move me up to @sentry/typescript
-  }
+  },
+  "include": [
+    "./globals.d.ts"
+  ]
 }