fix(sentry): set consistent Sentry release name (#464)

sentry.js

@@ -0,0 +1,9 @@
+/* eslint-disable no-var-requires, @typescript-eslint/no-var-requires, @typescript-eslint/ban-ts-ignore */
+const { version } = require('./package.json');
+
+// @ts-ignore : This file is used in plain JS in Webpack config
+const getRelease = buildType => `webext@${version}-${buildType}`;
+
+module.exports = {
+  getRelease
+};

src/app/content/index.ts

@@ -13,7 +13,7 @@ if (!(window as CustomWindow).__BULLES__CONTENT_SCRIPT_INJECTED__) {
   } = require('../utils/sentry');
 
   try {
-    if (!process.env.FIREFOX_BUILD) {
+    if ((process.env as AppEnv).BUILD !== 'firefox') {
       initSentry();
       configureSentryScope((scope: Scope) => {
         scope.setTag('context', 'content');

src/app/utils/sentry.ts

@@ -6,7 +6,7 @@ import {
   captureMessage as sentryCaptureMessage,
   captureException as sentryCaptureException
 } from '@sentry/browser';
-import { version } from '../../../package.json';
+import { getRelease } from '../../../sentry';
 import Logger from './Logger';
 
 let sentryInitialized = false;
@@ -18,7 +18,7 @@ export const initSentry = () => {
       init({
         dsn: process.env.SENTRY_DSN,
         environment: process.env.NODE_ENV,
-        release: `${version}-${process.env.BUILD}`,
+        release: getRelease(process.env.BUILD),
         integrations: integrations =>
           integrations.filter(i => !blacklist.includes(i.name))
       });

src/types/index.d.ts

@@ -7,6 +7,7 @@ interface AppEnv extends NodeJS.ProcessEnv {
   SEND_CONTRIBUTION_TO: string;
   SEND_IN_BLUE_TOKEN: string;
   SENTRY_DSN: string;
+  BUILD: 'firefox' | 'chromium' | 'dev' | 'build';
 }
 
 declare module '*.png' {

webpack/config.plugins.js

@@ -12,6 +12,7 @@ const { BundleAnalyzerPlugin } = require('webpack-bundle-analyzer');
 const basePlugins = require('../webpack/config.plugins.base');
 const manifests = require('../manifest');
 const { version } = require('../package.json');
+const { getRelease } = require('../sentry');
 
 const BUILD_CONFIG = {
   dev: {
@@ -90,8 +91,7 @@ module.exports = (env = {}, argv = {}, outputPath) => {
         ...getBuildConfig(env.build, env.api),
         ...processENVVarsToInject(process.env),
         BUILD: JSON.stringify(env.build),
-        SENTRY_ENABLE: env.sentry ? 'true' : 'false',
-        FIREFOX_BUILD: env.build === 'firefox' ? 'true' : 'false'
+        SENTRY_ENABLE: env.sentry ? 'true' : 'false'
       }
     }),
     new ModuleNotFoundPlugin(path.resolve(__dirname, '..')),
@@ -117,7 +117,7 @@ module.exports = (env = {}, argv = {}, outputPath) => {
       new SentryWebpackPlugin({
         include: `./build/${env.build}/js`,
         ignore: ['test.*.js*'],
-        release: `web-extension@${version}-${env.build}`
+        release: getRelease(env.build)
       })
     );
   }