build: Remove @sentry-internal/sdk/no-async-await rule

Author: AbhiPrasad

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

@@ -143,9 +143,6 @@ module.exports = {
           },
         ],
 
-        // We want to prevent async await usage in our files to prevent uncessary bundle size. Turned off in tests.
-        '@sentry-internal/sdk/no-async-await': 'error',
-
         // JSDOC comments are required for classes and methods. As we have a public facing codebase, documentation,
         // even if it may seems excessive at times, is important to emphasize. Turned off in tests.
         'jsdoc/require-jsdoc': [
@@ -177,7 +174,6 @@ module.exports = {
         '@typescript-eslint/no-explicit-any': 'off',
         '@typescript-eslint/no-non-null-assertion': 'off',
         '@typescript-eslint/no-empty-function': 'off',
-        '@sentry-internal/sdk/no-async-await': 'off',
       },
     },
     {

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

@@ -10,7 +10,6 @@
 
 module.exports = {
   rules: {
-    'no-async-await': require('./rules/no-async-await'),
     'no-eq-empty': require('./rules/no-eq-empty'),
   },
 };

packages/eslint-plugin-sdk/src/rules/no-async-await.js

@@ -8,9 +8,6 @@ module.exports = {
   },
   ignorePatterns: ['test/integration/**'],
   extends: ['../../.eslintrc.js'],
-  rules: {
-    '@sentry-internal/sdk/no-async-await': 'off',
-  },
   overrides: [
     {
       files: ['scripts/**/*.ts'],

packages/eslint-plugin-sdk/test/lib/rules/no-async-await.js

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

packages/nextjs/.eslintrc.js

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

packages/node/.eslintrc.js

@@ -8,9 +8,6 @@ module.exports = {
   },
   ignorePatterns: ['playwright.config.ts', 'test/integration/**'],
   extends: ['../../.eslintrc.js'],
-  rules: {
-    '@sentry-internal/sdk/no-async-await': 'off',
-  },
   overrides: [
     {
       files: ['scripts/**/*.ts'],

packages/opentelemetry-node/.eslintrc.js

@@ -20,13 +20,6 @@ module.exports = {
         sourceType: 'module',
       },
     },
-    {
-      files: ['*.ts', '*.tsx', '*.d.ts'],
-      rules: {
-        // Since we target only es6 here, we can leave this off
-        '@sentry-internal/sdk/no-async-await': 'off',
-      },
-    },
     {
       files: ['jest.setup.ts', 'jest.config.ts'],
       parserOptions: {

packages/remix/.eslintrc.js

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

packages/replay/.eslintrc.js

@@ -15,7 +15,6 @@ import { _nullishCoalesce } from './_nullishCoalesce';
  * @param rhsFn A function returning the value of the expression to the right of the `??`
  * @returns The LHS value, unless it's `null` or `undefined`, in which case, the RHS value
  */
-// eslint-disable-next-line @sentry-internal/sdk/no-async-await
 export async function _asyncNullishCoalesce(lhs: unknown, rhsFn: () => unknown): Promise<unknown> {
   return _nullishCoalesce(lhs, rhsFn);
 }

packages/serverless/.eslintrc.js

@@ -10,7 +10,6 @@ import type { GenericFunction } from './types';
  * @param ops Array result of expression conversion
  * @returns The value of the expression
  */
-// eslint-disable-next-line @sentry-internal/sdk/no-async-await
 export async function _asyncOptionalChain(ops: unknown[]): Promise<unknown> {
   let lastAccessLHS: unknown = undefined;
   let value = ops[0];

packages/utils/src/buildPolyfills/_asyncNullishCoalesce.ts

@@ -12,7 +12,6 @@ import { _asyncOptionalChain } from './_asyncOptionalChain';
  * property (one which can't be deleted or turned into an accessor, and whose enumerability can't be changed), in which
  * case `false`.
  */
-// eslint-disable-next-line @sentry-internal/sdk/no-async-await
 export async function _asyncOptionalChainDelete(ops: unknown[]): Promise<boolean> {
   const result = (await _asyncOptionalChain(ops)) as Promise<boolean | null>;
   // If `result` is `null`, it means we didn't get to the end of the chain and so nothing was deleted (in which case,