ref: Dedupe is now optional, Fix integration build

Author: HazAT

CHANGELOG.md

@@ -44,6 +44,7 @@ since we removed some methods from the public API and removed some classes from
 - [all] feat: Prefix all private methods with `_`
 - [all] build: Use terser instead of uglify
 - [opentracing] feat: Introduce `@sentry/opentracing` providing functions to attach opentracing data to Sentry Events
+- **breaking** [core] ref: `Dedupe` Integration is now optional, it is no longer enabled by default.
 
 ## 4.6.4
 

dangerfile.ts

@@ -48,6 +48,6 @@ export default async () => {
 
   schedule(async () => {
     const result = (await promisify(exec)('cd packages/browser; yarn size:check')).stdout;
-    message(`@sentry/browser gzip'ed minified size: ${result.split('\n')[1]}`);
+    message(`@sentry/browser bundle gzip'ed minified size: \n${result.split('\n')[2]}\n${result.split('\n')[3]}`);
   });
 };

packages/browser/src/sdk.ts

@@ -4,7 +4,6 @@ import { BrowserClient, ReportDialogOptions } from './client';
 import { Breadcrumbs, GlobalHandlers, LinkedErrors, TryCatch, UserAgent } from './integrations';
 
 export const defaultIntegrations = [
-  new CoreIntegrations.Dedupe(),
   new CoreIntegrations.InboundFilters(),
   new CoreIntegrations.FunctionToString(),
   new TryCatch(),

packages/browser/test/index.test.ts

@@ -128,15 +128,6 @@ describe('SentryBrowser', () => {
       captureEvent({ message: 'event' });
     });
 
-    it('should dedupe an event', async () => {
-      captureMessage('event222');
-      captureMessage('event222');
-
-      await flush(2000);
-
-      expect(beforeSend.calledOnce).to.be.true;
-    });
-
     it('should not dedupe an event on bound client', async () => {
       const localBeforeSend = spy();
       getCurrentHub().bindClient(

packages/browser/test/integration/common.js

@@ -117,6 +117,7 @@ function initSDK() {
   Sentry.init({
     dsn: 'https://public@example.com/1',
     // debug: true,
+    integrations: [new SentryIntegration.Dedupe()],
     attachStacktrace: true,
     transport: DummyTransport,
     ignoreErrors: ['ignoreErrorTest'],

packages/browser/test/integration/dedupe.js

@@ -0,0 +1 @@
+../../../integrations/build/dedupe.js

packages/browser/test/integration/dedupe.js.map

@@ -6,6 +6,7 @@
   <title></title>
   <script src="polyfills/es6-promise-4.2.5.auto.js"></script>
   <script src="polyfills/whatwg-fetch-3.0.0.js"></script>
+  <script src="dedupe.js"></script>
   <script src="common.js"></script>
   <script src="../../build/bundle.js"></script>
   <script src="init.js"></script>

packages/browser/test/integration/frame.html

@@ -6,6 +6,7 @@
   <title></title>
   <script src="polyfills/es6-promise-4.2.5.auto.js"></script>
   <script src="polyfills/whatwg-fetch-3.0.0.js"></script>
+  <script src="dedupe.js"></script>
   <script src="common.js"></script>
   <script src="../../src/loader.js" data-lazy="no"></script>
 </head>

packages/browser/test/integration/loader-lazy-no.html

@@ -5,6 +5,7 @@
     <title></title>
     <script src="polyfills/es6-promise-4.2.5.auto.js"></script>
     <script src="polyfills/whatwg-fetch-3.0.0.js"></script>
+    <script src="dedupe.js"></script>
     <script src="common.js"></script>
     <script src="../../src/loader.js" data-lazy="no"></script>
   </head>