Refactor @bugsnag/browser

dockerfiles/Dockerfile.browser

@@ -10,6 +10,7 @@ COPY babel.config.js lerna.json .eslintignore .eslintrc.js jest.config.js tsconf
 ADD min_packages.tar .
 COPY .rollup ./.rollup
 COPY bin ./bin
+COPY jest ./jest
 COPY packages ./packages
 
 RUN npm install

dockerfiles/Dockerfile.ci

@@ -12,6 +12,7 @@ COPY .rollup ./.rollup
 COPY bin ./bin
 COPY scripts ./scripts
 COPY test ./test
+COPY jest ./jest
 COPY packages ./packages
 
 RUN npm install --unsafe-perm

dockerfiles/Dockerfile.node

@@ -10,6 +10,7 @@ COPY babel.config.js lerna.json .eslintignore .eslintrc.js jest.config.js tsconf
 ADD min_packages.tar .
 COPY .rollup ./.rollup
 COPY bin ./bin
+COPY jest ./jest
 COPY packages ./packages
 
 RUN npm install

jest.config.js

@@ -49,7 +49,9 @@ module.exports = {
       'plugin-simple-throttle',
       'plugin-console-breadcrumbs',
       'plugin-browser-session'
-    ]),
+    ], {
+      setupFiles: ['<rootDir>/jest/setup/mockEventTarget.js']
+    }),
     project('react native', [
       'react-native',
       'delivery-react-native',

jest/setup/mockEventTarget.js

@@ -0,0 +1,40 @@
+// copy the code from https://developer.mozilla.org/en-US/docs/Web/API/EventTarget#Simple_implementation_of_EventTarget
+var EventTarget = function () {
+  this.listeners = {}
+}
+
+EventTarget.prototype.listeners = null
+EventTarget.prototype.addEventListener = function (type, callback) {
+  if (!(type in this.listeners)) {
+    this.listeners[type] = []
+  }
+  this.listeners[type].push(callback)
+}
+
+EventTarget.prototype.removeEventListener = function (type, callback) {
+  if (!(type in this.listeners)) {
+    return
+  }
+  var stack = this.listeners[type]
+  for (var i = 0, l = stack.length; i < l; i++) {
+    if (stack[i] === callback) {
+      stack.splice(i, 1)
+      return
+    }
+  }
+}
+
+EventTarget.prototype.dispatchEvent = function (event) {
+  if (!(event.type in this.listeners)) {
+    return true
+  }
+  var stack = this.listeners[event.type].slice()
+
+  for (var i = 0, l = stack.length; i < l; i++) {
+    stack[i].call(this, event)
+  }
+  return !event.defaultPrevented
+}
+
+// make the EventTarget global
+global.EventTarget = EventTarget

packages/browser/package.json

@@ -1,8 +1,15 @@
 {
   "name": "@bugsnag/browser",
   "version": "8.1.2",
-  "main": "dist/bugsnag.js",
+  "main": "dist/notifier.js",
   "types": "types/bugsnag.d.ts",
+  "exports": {
+    ".": {
+      "types": "./types/bugsnag.d.ts",
+      "default": "./dist/notifier.js",
+      "import": "./dist/notifier.mjs"
+    }
+  },
   "description": "Bugsnag error reporter for browser JavaScript",
   "homepage": "https://www.bugsnag.com/",
   "repository": {
@@ -22,7 +29,8 @@
   "scripts": {
     "size": "../../bin/size dist/bugsnag.min.js",
     "clean": "rm -fr dist && mkdir dist",
-    "build": "npm run clean && npm run build:dist && npm run build:dist:min",
+    "build": "npm run clean && npm run build:npm",
+    "build:npm": "rollup --config rollup.config.npm.mjs",
     "build:dist": "cross-env NODE_ENV=production bash -c '../../bin/bundle src/notifier.js --standalone=Bugsnag | ../../bin/extract-source-map dist/bugsnag.js'",
     "build:dist:min": "cross-env NODE_ENV=production bash -c '../../bin/bundle src/notifier.js --standalone=Bugsnag | ../../bin/minify dist/bugsnag.min.js'",
     "cdn-upload": "../../bin/cdn-upload dist/*"

packages/browser/rollup.config.npm.mjs

@@ -0,0 +1,33 @@
+import createRollupConfig from "../../.rollup/index.mjs";
+
+export default createRollupConfig({
+  input: "src/notifier.ts",
+  external: [
+    "@bugsnag/core/client",
+    "@bugsnag/core/event",
+    "@bugsnag/core/session",
+    "@bugsnag/core/breadcrumb",
+    "@bugsnag/core/config",
+    "@bugsnag/core/types",
+    "@bugsnag/core/lib/es-utils/map",
+    "@bugsnag/core/lib/es-utils/keys",
+    "@bugsnag/core/lib/es-utils/assign",
+    "@bugsnag/plugin-window-onerror",
+    "@bugsnag/plugin-window-unhandled-rejection",
+    "@bugsnag/plugin-app-duration",
+    "@bugsnag/plugin-browser-device",
+    "@bugsnag/plugin-browser-context",
+    "@bugsnag/plugin-browser-request",
+    "@bugsnag/plugin-simple-throttle",
+    "@bugsnag/plugin-console-breadcrumbs",
+    "@bugsnag/plugin-network-breadcrumbs",
+    "@bugsnag/plugin-navigation-breadcrumbs",
+    "@bugsnag/plugin-interaction-breadcrumbs",
+    "@bugsnag/plugin-inline-script-content",
+    "@bugsnag/plugin-browser-session",
+    "@bugsnag/plugin-client-ip",
+    "@bugsnag/plugin-strip-query-string",
+    "@bugsnag/delivery-x-domain-request",
+    "@bugsnag/delivery-xml-http-request"
+  ],
+});

packages/browser/src/config.ts

@@ -0,0 +1,24 @@
+import { schema } from '@bugsnag/core/config'
+import assign from '@bugsnag/core/lib/es-utils/assign'
+import getPrefixedConsole from './get-prefixed-console'
+
+const config = {
+  releaseStage: assign({}, schema.releaseStage, {
+    defaultValue: () => {
+      if (/^localhost(:\d+)?$/.test(window.location.host)) return 'development'
+      return 'production'
+    }
+  }),
+  appType: assign({}, schema.appType, {
+    defaultValue: () => 'browser'
+  }),
+  logger: assign({}, schema.logger, {
+    defaultValue: () =>
+      // set logger based on browser capability
+      (typeof console !== 'undefined' && typeof console.debug === 'function')
+        ? getPrefixedConsole()
+        : undefined
+  })
+}
+
+export default config

packages/browser/src/get-prefixed-console.ts

@@ -0,0 +1,17 @@
+import map from '@bugsnag/core/lib/es-utils/map'
+
+type LoggerMethod = 'debug' | 'info' | 'warn' | 'error'
+
+const getPrefixedConsole = () => {
+  const logger: Record<string, unknown> = {}
+  const consoleLog = console.log
+  map(['debug', 'info', 'warn', 'error'], (method: LoggerMethod) => {
+    const consoleMethod = console[method]
+    logger[method] = typeof consoleMethod === 'function'
+      ? consoleMethod.bind(console, '[bugsnag]')
+      : consoleLog.bind(console, '[bugsnag]')
+  })
+  return logger
+}
+
+export default getPrefixedConsole