more review comments

Author: s1gr1d

MIGRATION.md

@@ -114,9 +114,9 @@ Sentry SDK requires the fetch API to be available in the environment.
 
 ## Integrations
 
-We moved optional integrations from their own package (`@sentry/integrations`) to `@sentry/browser` and `@sentry/node`.
+We moved pluggable integrations from their own package (`@sentry/integrations`) to `@sentry/browser` and `@sentry/node`.
 
-Integrations that are now exported from `@sentry/browser` :
+Integrations that are now exported from `@sentry/browser` (or framework-specific packages like `@sentry/react`):
 
 - httpClientIntegration
 - contextLinesIntegration
@@ -127,10 +127,10 @@ Integrations that are now exported from `@sentry/node` and `@sentry/browser` (or
 
 - captureConsoleIntegration
 - debugIntegration
-- dedupeIntegration
 - extraErrorDataIntegration
 - rewriteFramesIntegration
 - sessionTimingIntegration
+- dedupeIntegration (enabled by default, not pluggable)
 
 # Deprecations in 7.x
 

packages/browser/rollup.bundle.config.mjs

@@ -10,7 +10,14 @@ if (targets.some(target => target !== 'es5' && target !== 'es6')) {
 
 const browserPluggableIntegrationFiles = ['contextlines', 'httpclient', 'reportingobserver'];
 
-const coreIntegrationFiles = ['captureconsole', 'debug', 'dedupe', 'extraerrordata', 'rewriteframes', 'sessiontiming'];
+const corePluggableIntegrationFiles = [
+  'captureconsole',
+  'debug',
+  'dedupe',
+  'extraerrordata',
+  'rewriteframes',
+  'sessiontiming',
+];
 
 targets.forEach(jsVersion => {
   const baseBundleConfig = makeBaseBundleConfig({
@@ -41,7 +48,7 @@ targets.forEach(jsVersion => {
     builds.push(...makeBundleConfigVariants(integrationsBundleConfig));
   });
 
-  coreIntegrationFiles.forEach(integrationName => {
+  corePluggableIntegrationFiles.forEach(integrationName => {
     const integrationsBundleConfig = makeBaseBundleConfig({
       bundleType: 'addon',
       entrypoints: [`src/integrations-bundle/index.${integrationName}.ts`],