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

getsentry · Dec 13, 2024 · 1c9544b · 1c9544b
1 parent 8b81476
commit 1c9544b
Show file tree

Hide file tree

Showing 41 changed files with 0 additions and 1,073 deletions.
diff --git a/dev-packages/rollup-utils/npmHelpers.mjs b/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),

diff --git a/dev-packages/rollup-utils/plugins/extractPolyfillsPlugin.mjs b/dev-packages/rollup-utils/plugins/extractPolyfillsPlugin.mjs
diff --git a/dev-packages/rollup-utils/plugins/npmPlugins.mjs b/dev-packages/rollup-utils/plugins/npmPlugins.mjs
@@ -173,5 +173,3 @@ export function makeCodeCovPlugin() {
     uploadToken: process.env.CODECOV_TOKEN,
   });
 }
-
-export { makeExtractPolyfillsPlugin } from './extractPolyfillsPlugin.mjs';
diff --git a/dev-packages/test-utils/.eslintrc.js b/dev-packages/test-utils/.eslintrc.js
@@ -3,12 +3,4 @@ module.exports = {
     node: true,
   },
   extends: ['../../.eslintrc.js'],
-  overrides: [
-    {
-      files: ['**/*.ts'],
-      rules: {
-        '@sentry-internal/sdk/no-optional-chaining': 'off',
-      },
-    },
-  ],
 };
diff --git a/packages/astro/.eslintrc.cjs b/packages/astro/.eslintrc.cjs
@@ -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',
-      },
-    },
   ],
 };
diff --git a/packages/aws-serverless/.eslintrc.js b/packages/aws-serverless/.eslintrc.js
@@ -3,9 +3,6 @@ module.exports = {
     node: true,
   },
   extends: ['../../.eslintrc.js'],
-  rules: {
-    '@sentry-internal/sdk/no-optional-chaining': 'off',
-  },
   overrides: [
     {
       files: ['scripts/**/*.ts'],

diff --git a/packages/bun/.eslintrc.js b/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',
   },
 };
diff --git a/packages/cloudflare/.eslintrc.js b/packages/cloudflare/.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',
   },
 };
diff --git a/packages/core/src/utils-hoist/buildPolyfills/README.md b/packages/core/src/utils-hoist/buildPolyfills/README.md
diff --git a/packages/core/src/utils-hoist/buildPolyfills/_asyncNullishCoalesce.ts b/packages/core/src/utils-hoist/buildPolyfills/_asyncNullishCoalesce.ts
-Original file line number
+Diff line change
@@ Expand Up / @@ -173,5 +173,3 @@ export function makeCodeCovPlugin() { @@
         uploadToken: process.env.CODECOV_TOKEN,
       });
     }
-    export { makeExtractPolyfillsPlugin } from './extractPolyfillsPlugin.mjs';