build(cdn): Ensure we properly polyfill ES5 methods

Author: mydea

packages/eslint-config-sdk/src/index.js

@@ -161,6 +161,9 @@ module.exports = {
 
         // All imports should be accounted for
         'import/no-extraneous-dependencies': 'error',
+
+        // Do not allow usage of functions we do not polyfill for ES5
+        '@sentry-internal/sdk/no-unsupported-es6-methods': 'error',
       },
     },
     {

packages/eslint-plugin-sdk/src/index.js

@@ -13,5 +13,6 @@ module.exports = {
     'no-optional-chaining': require('./rules/no-optional-chaining'),
     'no-nullish-coalescing': require('./rules/no-nullish-coalescing'),
     'no-eq-empty': require('./rules/no-eq-empty'),
+    'no-unsupported-es6-methods': require('./rules/no-unsupported-es6-methods'),
   },
 };

packages/eslint-plugin-sdk/src/rules/no-unsupported-es6-methods.js

@@ -0,0 +1,35 @@
+'use strict';
+
+/**
+ * Taken and adapted from https://github.com/nkt/eslint-plugin-es5/blob/master/src/rules/no-es6-methods.js
+ */
+
+module.exports = {
+  meta: {
+    docs: {
+      description: 'Forbid methods added in ES6 which are not polyfilled by Sentry.',
+    },
+    schema: [],
+  },
+  create(context) {
+    return {
+      CallExpression(node) {
+        if (!node.callee || !node.callee.property) {
+          return;
+        }
+        const functionName = node.callee.property.name;
+
+        const es6ArrayFunctions = ['copyWithin', 'values', 'fill'];
+        const es6StringFunctions = ['repeat'];
+
+        const es6Functions = [].concat(es6ArrayFunctions, es6StringFunctions);
+        if (es6Functions.indexOf(functionName) > -1) {
+          context.report({
+            node: node.callee.property,
+            message: `ES6 methods not allowed: ${functionName}`,
+          });
+        }
+      },
+    };
+  },
+};

packages/node/.eslintrc.js

@@ -6,5 +6,6 @@ module.exports = {
   rules: {
     '@sentry-internal/sdk/no-optional-chaining': 'off',
     '@sentry-internal/sdk/no-nullish-coalescing': 'off',
+    '@sentry-internal/sdk/no-unsupported-es6-methods': 'off',
   },
 };

packages/overhead-metrics/.eslintrc.cjs

@@ -10,6 +10,7 @@ module.exports = {
         'import/no-unresolved': 'off',
         '@sentry-internal/sdk/no-optional-chaining': 'off',
         '@sentry-internal/sdk/no-nullish-coalescing': 'off',
+        '@sentry-internal/sdk/no-unsupported-es6-methods': 'off',
         'jsdoc/require-jsdoc': 'off',
       },
     },

packages/replay/.eslintrc.js

@@ -7,21 +7,8 @@ module.exports = {
   extends: ['../../.eslintrc.js'],
   overrides: [
     {
-      files: ['worker/**/*.ts'],
-      parserOptions: {
-        // TODO: figure out if we need a worker-specific tsconfig
-        project: ['tsconfig.worker.json'],
-      },
-      rules: {
-        // We cannot use backticks, as that conflicts with the stringified worker
-        'prefer-template': 'off',
-      },
-    },
-    {
-      files: ['src/worker/**/*.js'],
-      parserOptions: {
-        sourceType: 'module',
-      },
+      files: ['src/**/*.ts'],
+      rules: {},
     },
     {
       files: ['jest.setup.ts', 'jest.config.ts'],

packages/svelte/.eslintrc.js

@@ -3,4 +3,7 @@ module.exports = {
     browser: true,
   },
   extends: ['../../.eslintrc.js'],
+  rules: {
+    '@sentry-internal/sdk/no-unsupported-es6-methods': 'off',
+  },
 };

packages/vue/src/components.ts

@@ -9,6 +9,7 @@ const ANONYMOUS_COMPONENT_NAME = '<Anonymous>';
 
 const repeat = (str: string, n: number): string => {
   // string.repeat() is not supported by IE11, we fall back to just using the string in that case
+  // eslint-disable-next-line @sentry-internal/sdk/no-unsupported-es6-methods
   return str.repeat ? str.repeat(n) : str;
 };
 

rollup/bundleHelpers.js

@@ -17,6 +17,7 @@ import {
   makeTerserPlugin,
   makeTSPlugin,
   makeSetSDKSourcePlugin,
+  getEs5Polyfills,
 } from './plugins/index.js';
 import { mergePlugins } from './utils';
 
@@ -25,6 +26,8 @@ const BUNDLE_VARIANTS = ['.js', '.min.js', '.debug.min.js'];
 export function makeBaseBundleConfig(options) {
   const { bundleType, entrypoints, jsVersion, licenseTitle, outputFileBase, packageSpecificConfig } = options;
 
+  const isEs5 = jsVersion.toLowerCase() === 'es5';
+
   const nodeResolvePlugin = makeNodeResolvePlugin();
   const sucrasePlugin = makeSucrasePlugin();
   const cleanupPlugin = makeCleanupPlugin();
@@ -42,6 +45,10 @@ export function makeBaseBundleConfig(options) {
     output: {
       format: 'iife',
       name: 'Sentry',
+      outro: () => {
+        // Add polyfills for ES6 array/string methods at the end of the bundle
+        return isEs5 ? getEs5Polyfills() : '';
+      },
     },
     context: 'window',
     plugins: [markAsBrowserBuildPlugin],
@@ -101,10 +108,9 @@ export function makeBaseBundleConfig(options) {
       strict: false,
       esModule: false,
     },
-    plugins:
-      jsVersion === 'es5'
-        ? [tsPlugin, nodeResolvePlugin, cleanupPlugin, licensePlugin]
-        : [sucrasePlugin, nodeResolvePlugin, cleanupPlugin, licensePlugin],
+    plugins: isEs5
+      ? [tsPlugin, nodeResolvePlugin, cleanupPlugin, licensePlugin]
+      : [sucrasePlugin, nodeResolvePlugin, cleanupPlugin, licensePlugin],
     treeshake: 'smallest',
   };
 

rollup/plugins/bundlePlugins.js

@@ -8,6 +8,9 @@
  * Typescript plugin docs: https://github.com/ezolenko/rollup-plugin-typescript2
  */
 
+import * as fs from 'fs';
+import * as path from 'path';
+
 import commonjs from '@rollup/plugin-commonjs';
 import deepMerge from 'deepmerge';
 import license from 'rollup-plugin-license';
@@ -38,6 +41,11 @@ export function makeLicensePlugin(title) {
   return plugin;
 }
 
+export function getEs5Polyfills() {
+  // Note: __dirname resolves to e.g. packages/browser or packages/tracing
+  return fs.readFileSync(path.join(__dirname, '../../rollup/polyfills/es5.js'), 'utf-8');
+}
+
 /**
  * Create a plugin to set the value of the `__SENTRY_DEBUG__` magic string.
  *