ref!: Don't polyfill optional chaining and nullish coalescing (#14603)

Author: Luca Forstner

dev-packages/rollup-utils/npmHelpers.mjs

@@ -15,7 +15,6 @@ import { defineConfig } from 'rollup';
 import {
   makeCleanupPlugin,
   makeDebugBuildStatementReplacePlugin,
-  makeExtractPolyfillsPlugin,
   makeImportMetaUrlReplacePlugin,
   makeNodeResolvePlugin,
   makeRrwebBuildPlugin,
@@ -44,7 +43,6 @@ export function makeBaseNPMConfig(options = {}) {
   const debugBuildStatementReplacePlugin = makeDebugBuildStatementReplacePlugin();
   const importMetaUrlReplacePlugin = makeImportMetaUrlReplacePlugin();
   const cleanupPlugin = makeCleanupPlugin();
-  const extractPolyfillsPlugin = makeExtractPolyfillsPlugin();
   const rrwebBuildPlugin = makeRrwebBuildPlugin({
     excludeShadowDom: undefined,
     excludeIframe: undefined,
@@ -121,10 +119,6 @@ export function makeBaseNPMConfig(options = {}) {
     ],
   };
 
-  if (addPolyfills) {
-    defaultBaseConfig.plugins.push(extractPolyfillsPlugin);
-  }
-
   return deepMerge(defaultBaseConfig, packageSpecificConfig, {
     // Plugins have to be in the correct order or everything breaks, so when merging we have to manually re-order them
     customMerge: key => (key === 'plugins' ? mergePlugins : undefined),

dev-packages/rollup-utils/plugins/extractPolyfillsPlugin.mjs

@@ -173,5 +173,3 @@ export function makeCodeCovPlugin() {
     uploadToken: process.env.CODECOV_TOKEN,
   });
 }
-
-export { makeExtractPolyfillsPlugin } from './extractPolyfillsPlugin.mjs';

dev-packages/rollup-utils/plugins/npmPlugins.mjs

@@ -3,12 +3,4 @@ module.exports = {
     node: true,
   },
   extends: ['../../.eslintrc.js'],
-  overrides: [
-    {
-      files: ['**/*.ts'],
-      rules: {
-        '@sentry-internal/sdk/no-optional-chaining': 'off',
-      },
-    },
-  ],
 };

dev-packages/test-utils/.eslintrc.js

@@ -11,12 +11,5 @@ module.exports = {
         project: ['tsconfig.test.json'],
       },
     },
-    {
-      files: ['src/integration/**', 'src/server/**'],
-      rules: {
-        '@sentry-internal/sdk/no-optional-chaining': 'off',
-        '@sentry-internal/sdk/no-nullish-coalescing': 'off',
-      },
-    },
   ],
 };

packages/astro/.eslintrc.cjs

@@ -3,9 +3,6 @@ module.exports = {
     node: true,
   },
   extends: ['../../.eslintrc.js'],
-  rules: {
-    '@sentry-internal/sdk/no-optional-chaining': 'off',
-  },
   overrides: [
     {
       files: ['scripts/**/*.ts'],

packages/aws-serverless/.eslintrc.js

@@ -4,8 +4,6 @@ module.exports = {
   },
   extends: ['../../.eslintrc.js'],
   rules: {
-    '@sentry-internal/sdk/no-optional-chaining': 'off',
-    '@sentry-internal/sdk/no-nullish-coalescing': 'off',
     '@sentry-internal/sdk/no-class-field-initializers': 'off',
   },
 };

packages/bun/.eslintrc.js

@@ -4,8 +4,6 @@ module.exports = {
   },
   extends: ['../../.eslintrc.js'],
   rules: {
-    '@sentry-internal/sdk/no-optional-chaining': 'off',
-    '@sentry-internal/sdk/no-nullish-coalescing': 'off',
     '@sentry-internal/sdk/no-class-field-initializers': 'off',
   },
 };