feat(eslint): Make class member accessibility explicit

Add @typescript-eslint/explicit-member-accessibility rule to make
sure class members declarations have explicit accessibility (public,
private, protected).

Author: AbhiPrasad

packages/browser/src/transports/fetch.ts

@@ -81,7 +81,7 @@ export class FetchTransport extends BaseTransport {
    */
   private _fetch: typeof fetch;
 
-  constructor(options: TransportOptions, fetchImpl: FetchImpl = getNativeFetchImplementation()) {
+  public constructor(options: TransportOptions, fetchImpl: FetchImpl = getNativeFetchImplementation()) {
     super(options);
     this._fetch = fetchImpl;
   }

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

@@ -104,6 +104,10 @@ module.exports = {
         // instead of any. This is especially important for methods that expose a public API, as users
         // should know exactly what they have to provide to use those methods. Turned off in tests.
         '@typescript-eslint/no-unsafe-member-access': 'error',
+
+        // Be explicit about class member accessibility (public, private, protected). Turned off
+        // on tests for ease of use.
+        '@typescript-eslint/explicit-member-accessibility': ['error'],
       },
     },
     {
@@ -140,6 +144,7 @@ module.exports = {
         'no-unused-expressions': 'off',
         '@typescript-eslint/no-unused-expressions': 'off',
         '@typescript-eslint/no-unsafe-member-access': 'off',
+        '@typescript-eslint/explicit-member-accessibility': 'off',
       },
     },
     {

packages/gatsby/test/integration.test.tsx

@@ -1,6 +1,7 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { render } from '@testing-library/react';
 import { useEffect } from 'react';
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 import * as React from 'react';
 
 import { onClientEntry } from '../gatsby-browser';

packages/hub/src/session.ts

@@ -18,15 +18,15 @@ export class Session implements SessionInterface {
   public ipAddress?: string;
   public init: boolean = true;
 
-  constructor(context?: Omit<SessionContext, 'started' | 'status'>) {
+  public constructor(context?: Omit<SessionContext, 'started' | 'status'>) {
     if (context) {
       this.update(context);
     }
   }
 
   /** JSDoc */
   // eslint-disable-next-line complexity
-  update(context: SessionContext = {}): void {
+  public update(context: SessionContext = {}): void {
     if (context.user) {
       if (context.user.ip_address) {
         this.ipAddress = context.user.ip_address;
@@ -78,7 +78,7 @@ export class Session implements SessionInterface {
   }
 
   /** JSDoc */
-  close(status?: Exclude<SessionStatus, SessionStatus.Ok>): void {
+  public close(status?: Exclude<SessionStatus, SessionStatus.Ok>): void {
     if (status) {
       this.update({ status });
     } else if (this.status === SessionStatus.Ok) {
@@ -89,7 +89,7 @@ export class Session implements SessionInterface {
   }
 
   /** JSDoc */
-  toJSON(): {
+  public toJSON(): {
     init: boolean;
     sid: string;
     did?: string;

packages/nextjs/src/utils/metadataBuilder.ts

@@ -13,7 +13,7 @@ export class MetadataBuilder {
   private _options: NextjsOptions;
   private _packageNames: string[];
 
-  constructor(options: NextjsOptions, packages: string[]) {
+  public constructor(options: NextjsOptions, packages: string[]) {
     this._options = options;
     this._packageNames = packages;
   }

packages/tracing/test/integrations/mongo.test.ts

@@ -10,15 +10,15 @@ class Collection {
   public namespace: string = 'mockedNamespace';
 
   // Method that can have a callback as last argument, or return a promise otherwise.
-  insertOne(_doc: unknown, _options: unknown, callback?: () => void) {
+  public insertOne(_doc: unknown, _options: unknown, callback?: () => void) {
     if (typeof callback === 'function') {
       callback();
       return;
     }
     return Promise.resolve();
   }
   // Method that has no callback as last argument, and doesnt return promise.
-  initializeOrderedBulkOp() {
+  public initializeOrderedBulkOp() {
     return {};
   }
 }