ref(replay): Re-activate naming-convention EsLint rule

Author: Lms24

packages/replay/.eslintrc.js

@@ -25,22 +25,6 @@ module.exports = {
       rules: {
         // TODO (high-prio): Re-enable this after migration
         '@typescript-eslint/explicit-member-accessibility': 'off',
-        // TODO (high-prio): Remove this exception from naming convention after migration
-        '@typescript-eslint/naming-convention': [
-          'error',
-          {
-            selector: 'memberLike',
-            modifiers: ['private'],
-            format: ['camelCase'],
-            leadingUnderscore: 'allow',
-          },
-          {
-            selector: 'memberLike',
-            modifiers: ['protected'],
-            format: ['camelCase'],
-            leadingUnderscore: 'allow',
-          },
-        ],
         // TODO (high-prio): Re-enable this after migration
         '@sentry-internal/sdk/no-async-await': 'off',
         // TODO (medium-prio): Re-enable this after migration

packages/replay/src/eventBuffer.ts

@@ -80,12 +80,12 @@ class EventBufferArray implements IEventBuffer {
 
 // exporting for testing
 export class EventBufferCompressionWorker implements IEventBuffer {
-  private worker: null | Worker;
-  private eventBufferItemLength: number = 0;
+  private _worker: null | Worker;
+  private _eventBufferItemLength: number = 0;
   private _id: number = 0;
 
   constructor(worker: Worker) {
-    this.worker = worker;
+    this._worker = worker;
   }
 
   /**
@@ -113,7 +113,7 @@ export class EventBufferCompressionWorker implements IEventBuffer {
         }
 
         // At this point, we'll always want to remove listener regardless of result status
-        this.worker?.removeEventListener('message', listener);
+        this._worker?.removeEventListener('message', listener);
 
         if (!data.success) {
           // TODO: Do some error handling, not sure what
@@ -136,8 +136,8 @@ export class EventBufferCompressionWorker implements IEventBuffer {
 
       // Note: we can't use `once` option because it's possible it needs to
       // listen to multiple messages
-      this.worker?.addEventListener('message', listener);
-      this.worker?.postMessage({ id, method, args: stringifiedArgs });
+      this._worker?.addEventListener('message', listener);
+      this._worker?.postMessage({ id, method, args: stringifiedArgs });
     });
   }
 
@@ -148,16 +148,16 @@ export class EventBufferCompressionWorker implements IEventBuffer {
 
   destroy(): void {
     __DEBUG_BUILD__ && logger.log('[Replay] Destroying compression worker');
-    this.worker?.terminate();
-    this.worker = null;
+    this._worker?.terminate();
+    this._worker = null;
   }
 
   /**
    * Note that this may not reflect what is actually in the event buffer. This
    * is only a local count of the buffer size since `addEvent` is async.
    */
   get length(): number {
-    return this.eventBufferItemLength;
+    return this._eventBufferItemLength;
   }
 
   async addEvent(event: RecordingEvent, isCheckout?: boolean): Promise<string | Uint8Array> {
@@ -182,7 +182,7 @@ export class EventBufferCompressionWorker implements IEventBuffer {
     });
 
     // XXX: See note in `get length()`
-    this.eventBufferItemLength++;
+    this._eventBufferItemLength++;
 
     return promise;
   };
@@ -191,7 +191,7 @@ export class EventBufferCompressionWorker implements IEventBuffer {
     const promise = this.postMessage({ id, method: 'finish', args: [] });
 
     // XXX: See note in `get length()`
-    this.eventBufferItemLength = 0;
+    this._eventBufferItemLength = 0;
 
     return promise as Promise<Uint8Array>;
   };