ref: Remove tslint-disable comments

Author: AbhiPrasad

.eslintrc.js

@@ -16,18 +16,21 @@ module.exports = {
     },
     {
       // Configuration for typescript files
-      files: ['*.ts', '*.tsx'],
+      files: ['*.ts', '*.tsx', '*.d.ts'],
       extends: ['plugin:@typescript-eslint/recommended', 'prettier/@typescript-eslint'],
       plugins: ['@typescript-eslint'],
       parser: '@typescript-eslint/parser',
+      parserOptions: {
+        project: './tsconfig.json',
+      },
       rules: {
-        // We want to prevent async await usage in our files to prevent uncessary bundle size.
+        // We want to prevent async await usage in our files to prevent uncessary bundle size. Turned off in tests.
         'sentry-sdk/no-async-await': 'error',
 
-        // Make sure variables marked with _ are ignored (ex. _varName)
+        // Unused variables should be removed unless they are marked with and underscore (ex. _varName).
         '@typescript-eslint/no-unused-vars': ['warn', { argsIgnorePattern: '^_' }],
 
-        // Make sure that all ts-ignore comments are given a description
+        // Make sure that all ts-ignore comments are given a description.
         '@typescript-eslint/ban-ts-comment': [
           'warn',
           {
@@ -37,7 +40,7 @@ module.exports = {
 
         // Types usage should be explicit as possible, so we prevent usage of inferrable types.
         // This is especially important because we have a public API, so usage needs to be as
-        // easy to understand as possible
+        // easy to understand as possible.
         '@typescript-eslint/no-inferrable-types': 'off',
 
         // Enforce type annotations to maintain consistency. This is especially important as
@@ -48,6 +51,10 @@ module.exports = {
         // Consistent ordering of fields, methods and constructors for classes should be enforced
         '@typescript-eslint/member-ordering': 'error',
 
+        // Enforce that unbound methods are called within an expected scope. As we frequently pass data between classes
+        // in SDKs, we should make sure that we are correctly preserving class scope.
+        '@typescript-eslint/unbound-method': 'error',
+
         // Private and protected members of a class should be prefixed with a leading underscore
         '@typescript-eslint/naming-convention': [
           'error',
@@ -65,7 +72,8 @@ module.exports = {
           },
         ],
 
-        // JSDOC comments are required for classes and methods
+        // 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': [
           'error',
           { require: { ClassDeclaration: true, MethodDefinition: true }, checkConstructors: false },
@@ -81,6 +89,7 @@ module.exports = {
       rules: {
         'sentry-sdk/no-async-await': 'off',
         'jsdoc/require-jsdoc': 'off',
+        'max-lines': 'off',
       },
     },
     {
@@ -97,8 +106,19 @@ module.exports = {
     // We want to prevent usage of unary operators outside of for loops
     'no-plusplus': ['error', { allowForLoopAfterthoughts: true }],
 
-    // disallow usage of console and alert
+    // Disallow usage of console and alert
     'no-console': 'error',
     'no-alert': 'error',
+
+    // Prevent reassignment of function parameters, but still allow object properties to be
+    // reassigned. We want to enforce immutability when possible, but we shouldn't sacrifice
+    // too much efficiency
+    'no-param-reassign': ['error', { props: false }],
+
+    // Prefer use of template expression over string literal concatenation
+    'prefer-template': 'error',
+
+    // Limit maximum file size to reduce complexity. Turned off in tests.
+    'max-lines': 'error',
   },
 };

packages/browser/.eslintrc.js

@@ -10,11 +10,19 @@ module.exports = {
   extends: ['../../.eslintrc.js'],
   ignorePatterns: ['build/**/*', 'dist/**/*', 'esm/**/*', 'examples/**/*', 'scripts/**/*', 'src/loader.js'],
   overrides: [
+    {
+      files: ['*.ts', '*.tsx', '*.d.ts'],
+      parserOptions: {
+        project: './tsconfig.json',
+      },
+    },
     {
       files: ['test/**/*'],
       rules: {
         'jsdoc/require-jsdoc': 'off',
         'no-console': 'off',
+        'max-lines': 'off',
+        'prefer-template': 'off',
       },
     },
     {

packages/browser/src/eventbuilder.ts

@@ -71,6 +71,7 @@ export function eventFromUnknownInput(
   if (isErrorEvent(exception as ErrorEvent) && (exception as ErrorEvent).error) {
     // If it is an ErrorEvent with `error` property, extract it to get actual Error
     const errorEvent = exception as ErrorEvent;
+    // eslint-disable-next-line no-param-reassign
     exception = errorEvent.error;
     event = eventFromStacktrace(computeStackTrace(exception as Error));
     return event;

packages/browser/src/helpers.ts

@@ -38,7 +38,6 @@ export function wrap(
   before?: WrappedFunction,
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
 ): any {
-  // tslint:disable-next-line:strict-type-predicates
   if (typeof fn !== 'function') {
     return fn;
   }
@@ -65,9 +64,7 @@ export function wrap(
   const sentryWrapped: WrappedFunction = function(this: any): void {
     const args = Array.prototype.slice.call(arguments);
 
-    // tslint:disable:no-unsafe-any
     try {
-      // tslint:disable-next-line:strict-type-predicates
       if (before && typeof before === 'function') {
         before.apply(this, arguments);
       }
@@ -87,7 +84,6 @@ export function wrap(
       //       means the sentry.javascript SDK caught an error invoking your application code. This
       //       is expected behavior and NOT indicative of a bug with sentry.javascript.
       return fn.apply(this, wrappedArguments);
-      // tslint:enable:no-unsafe-any
     } catch (ex) {
       ignoreNextOnError();
 
@@ -211,6 +207,7 @@ export function injectReportDialog(options: ReportDialogOptions = {}): void {
   script.src = new API(options.dsn).getReportDialogEndpoint(options);
 
   if (options.onLoad) {
+    // eslint-disable-next-line @typescript-eslint/unbound-method
     script.onload = options.onLoad;
   }
 

packages/browser/src/index.ts

@@ -9,12 +9,10 @@ import * as Transports from './transports';
 let windowIntegrations = {};
 
 // This block is needed to add compatibility with the integrations packages when used with a CDN
-// tslint:disable: no-unsafe-any
 const _window = getGlobalObject<Window>();
 if (_window.Sentry && _window.Sentry.Integrations) {
   windowIntegrations = _window.Sentry.Integrations;
 }
-// tslint:enable: no-unsafe-any
 
 const INTEGRATIONS = {
   ...windowIntegrations,

packages/browser/src/integrations/breadcrumbs.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-lines */
 import { getCurrentHub } from '@sentry/core';
 import { Event, Integration, Severity } from '@sentry/types';
 import {
@@ -291,11 +292,9 @@ export class Breadcrumbs implements Integration {
     // Use only the path component of the URL if the URL matches the current
     // document (almost all the time when using pushState)
     if (parsedLoc.protocol === parsedTo.protocol && parsedLoc.host === parsedTo.host) {
-      // tslint:disable-next-line:no-parameter-reassignment
       to = parsedTo.relative;
     }
     if (parsedLoc.protocol === parsedFrom.protocol && parsedLoc.host === parsedFrom.host) {
-      // tslint:disable-next-line:no-parameter-reassignment
       from = parsedFrom.relative;
     }
 

packages/browser/src/integrations/trycatch.ts

@@ -162,7 +162,6 @@ export class TryCatch implements Integration {
         options?: boolean | AddEventListenerOptions,
       ): (eventName: string, fn: EventListenerObject, capture?: boolean, secure?: boolean) => void {
         try {
-          // tslint:disable-next-line:no-unbound-method strict-type-predicates
           if (typeof fn.handleEvent === 'function') {
             fn.handleEvent = wrap(fn.handleEvent.bind(fn), {
               mechanism: {

packages/browser/src/integrations/tslint.json

@@ -22,7 +22,6 @@ export function exceptionFromStacktrace(stacktrace: TraceKitStackTrace): Excepti
     exception.stacktrace = { frames };
   }
 
-  // tslint:disable-next-line:strict-type-predicates
   if (exception.type === undefined && exception.value === '') {
     exception.value = 'Unrecoverable error caught';
   }

packages/browser/src/parsers.ts

@@ -164,5 +164,5 @@ export function close(timeout?: number): PromiseLike<boolean> {
  */
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function wrap(fn: (...args: any) => any): any {
-  return internalWrap(fn)(); // tslint:disable-line:no-unsafe-any
+  return internalWrap(fn)();
 }