lint

Author: billyvg

packages/replay/src/replay.ts

@@ -268,7 +268,7 @@ export class ReplayContainer implements ReplayContainerInterface {
       this.stopRecording();
 
       // Flush event buffer before stopping
-      await this.flushImmediate()
+      await this.flushImmediate();
 
       this._shouldFinalFlush = false;
 
@@ -279,7 +279,6 @@ export class ReplayContainer implements ReplayContainerInterface {
       // Clear session from session storage, note this means if a new session
       // is started after, it will not have `previousSessionId`
       clearSession(this);
-
     } catch (err) {
       this._handleException(err);
     } finally {
@@ -489,7 +488,7 @@ export class ReplayContainer implements ReplayContainerInterface {
     this.session = session;
 
     if (!this.session.sampled) {
-      this.stop('session unsampled');
+      void this.stop('session unsampled');
       return false;
     }
 

packages/replay/src/session/clearSession.ts

@@ -1,7 +1,10 @@
 import { REPLAY_SESSION_KEY, WINDOW } from '../../src/constants';
 import type { ReplayContainer } from '../../src/types';
 
-export function clearSession(replay: ReplayContainer) {
+/**
+ *
+ */
+export function clearSession(replay: ReplayContainer): void {
   deleteSession();
   replay.session = undefined;
 }

packages/replay/test/integration/errorSampleRate.test.ts

@@ -9,14 +9,14 @@ import {
   WINDOW,
 } from '../../src/constants';
 import type { ReplayContainer } from '../../src/replay';
+import { clearSession } from '../../src/session/clearSession';
 import { addEvent } from '../../src/util/addEvent';
 import { PerformanceEntryResource } from '../fixtures/performanceEntry/resource';
 import type { RecordMock } from '../index';
 import { BASE_TIMESTAMP } from '../index';
 import { resetSdkMock } from '../mocks/resetSdkMock';
 import type { DomHandler } from '../types';
 import { useFakeTimers } from '../utils/use-fake-timers';
-import { clearSession } from '../../src/session/clearSession';
 
 useFakeTimers();
 

packages/replay/test/integration/events.test.ts

@@ -2,13 +2,13 @@ import { getCurrentHub } from '@sentry/core';
 
 import { WINDOW } from '../../src/constants';
 import type { ReplayContainer } from '../../src/replay';
+import { clearSession } from '../../src/session/clearSession';
 import { addEvent } from '../../src/util/addEvent';
 import { PerformanceEntryResource } from '../fixtures/performanceEntry/resource';
 import type { RecordMock } from '../index';
 import { BASE_TIMESTAMP } from '../index';
 import { resetSdkMock } from '../mocks/resetSdkMock';
 import { useFakeTimers } from '../utils/use-fake-timers';
-import { clearSession } from '../../src/session/clearSession';
 
 useFakeTimers();
 

packages/replay/test/integration/flush.test.ts

@@ -2,14 +2,14 @@ import * as SentryUtils from '@sentry/utils';
 
 import { DEFAULT_FLUSH_MIN_DELAY, WINDOW } from '../../src/constants';
 import type { ReplayContainer } from '../../src/replay';
+import { clearSession } from '../../src/session/clearSession';
 import type { EventBuffer } from '../../src/types';
 import * as AddMemoryEntry from '../../src/util/addMemoryEntry';
 import { createPerformanceEntries } from '../../src/util/createPerformanceEntries';
 import { createPerformanceSpans } from '../../src/util/createPerformanceSpans';
 import * as SendReplay from '../../src/util/sendReplay';
 import { BASE_TIMESTAMP, mockRrweb, mockSdk } from '../index';
 import { useFakeTimers } from '../utils/use-fake-timers';
-import { clearSession } from '../../src/session/clearSession';
 
 useFakeTimers();
 

packages/replay/test/integration/rateLimiting.test.ts

@@ -3,11 +3,11 @@ import type { Transport } from '@sentry/types';
 
 import { DEFAULT_FLUSH_MIN_DELAY } from '../../src/constants';
 import type { ReplayContainer } from '../../src/replay';
+import { clearSession } from '../../src/session/clearSession';
 import * as SendReplayRequest from '../../src/util/sendReplayRequest';
 import { BASE_TIMESTAMP, mockSdk } from '../index';
 import { mockRrweb } from '../mocks/mockRrweb';
 import { useFakeTimers } from '../utils/use-fake-timers';
-import { clearSession } from '../../src/session/clearSession';
 
 useFakeTimers();
 

packages/replay/test/integration/sendReplayEvent.test.ts

@@ -4,11 +4,11 @@ import * as SentryUtils from '@sentry/utils';
 
 import { DEFAULT_FLUSH_MIN_DELAY, WINDOW } from '../../src/constants';
 import type { ReplayContainer } from '../../src/replay';
+import { clearSession } from '../../src/session/clearSession';
 import { addEvent } from '../../src/util/addEvent';
 import * as SendReplayRequest from '../../src/util/sendReplayRequest';
 import { BASE_TIMESTAMP, mockRrweb, mockSdk } from '../index';
 import { useFakeTimers } from '../utils/use-fake-timers';
-import { clearSession } from '../../src/session/clearSession';
 
 useFakeTimers();
 

packages/replay/test/integration/session.test.ts

@@ -9,14 +9,14 @@ import {
   WINDOW,
 } from '../../src/constants';
 import type { ReplayContainer } from '../../src/replay';
+import { clearSession } from '../../src/session/clearSession';
 import type { Session } from '../../src/types';
 import { addEvent } from '../../src/util/addEvent';
 import { createPerformanceSpans } from '../../src/util/createPerformanceSpans';
 import { BASE_TIMESTAMP } from '../index';
 import type { RecordMock } from '../mocks/mockRrweb';
 import { resetSdkMock } from '../mocks/resetSdkMock';
 import { useFakeTimers } from '../utils/use-fake-timers';
-import { clearSession } from '../../src/session/clearSession';
 
 useFakeTimers();
 

packages/replay/test/integration/stop.test.ts

@@ -3,11 +3,11 @@ import * as SentryUtils from '@sentry/utils';
 import type { Replay } from '../../src';
 import { WINDOW } from '../../src/constants';
 import type { ReplayContainer } from '../../src/replay';
+import { clearSession } from '../../src/session/clearSession';
 import { addEvent } from '../../src/util/addEvent';
 // mock functions need to be imported first
 import { BASE_TIMESTAMP, mockRrweb, mockSdk } from '../index';
 import { useFakeTimers } from '../utils/use-fake-timers';
-import { clearSession } from '../../src/session/clearSession';
 
 useFakeTimers();
 
@@ -97,7 +97,7 @@ describe('Integration | stop', () => {
     integration.start();
 
     // will be different session
-    expect(replay.session?.id).not.toEqual(previousSessionId)
+    expect(replay.session?.id).not.toEqual(previousSessionId);
 
     jest.advanceTimersByTime(ELAPSED);
 
@@ -155,9 +155,7 @@ describe('Integration | stop', () => {
 
     expect(replay.eventBuffer).toBe(null);
     expect(replay).toHaveLastSentReplay({
-      recordingData: JSON.stringify([
-        TEST_EVENT,
-      ]),
+      recordingData: JSON.stringify([TEST_EVENT]),
     });
   });
 

packages/replay/test/utils/setupReplayContainer.ts

@@ -1,7 +1,7 @@
 import { createEventBuffer } from '../../src/eventBuffer';
 import { ReplayContainer } from '../../src/replay';
-import type { RecordingOptions, ReplayPluginOptions } from '../../src/types';
 import { clearSession } from '../../src/session/clearSession';
+import type { RecordingOptions, ReplayPluginOptions } from '../../src/types';
 
 export function setupReplayContainer({
   options,