Revert "feat: calculate total usage for plugin"

100mslive · Mar 15, 2024 · 83c45af · 83c45af
1 parent 764c784
commit 83c45af
Show file tree

Hide file tree

Showing 7 changed files with 1 addition and 81 deletions.
diff --git a/packages/hms-video-store/src/analytics/AnalyticsEventFactory.ts b/packages/hms-video-store/src/analytics/AnalyticsEventFactory.ts
@@ -3,7 +3,6 @@ import { AdditionalAnalyticsProperties } from './AdditionalAnalyticsProperties';
 import AnalyticsEvent from './AnalyticsEvent';
 import { AnalyticsEventLevel } from './AnalyticsEventLevel';
 import { IAnalyticsPropertiesProvider } from './IAnalyticsPropertiesProvider';
-import { pluginUsageTracker } from '../common';
 import { HMSException } from '../error/HMSException';
 import { DeviceMap, SelectedDevices } from '../interfaces';
 import { HMSTrackSettings } from '../media/settings/HMSTrackSettings';
@@ -236,23 +235,6 @@ export default class AnalyticsEventFactory {
     });
   }
 
-  static getKrispUsage(sessionID: string) {
-    const duration = pluginUsageTracker.getPluginUsage('HMSKrispPlugin', sessionID);
-    return new AnalyticsEvent({
-      name: 'krisp.usage',
-      level: AnalyticsEventLevel.INFO,
-      properties: { duration },
-    });
-  }
-
-  static transportLeave() {
-    return new AnalyticsEvent({
-      name: 'transport.leave',
-      level: AnalyticsEventLevel.INFO,
-      properties: {},
-    });
-  }
-
   private static eventNameFor(name: string, ok: boolean) {
     const suffix = ok ? 'success' : 'failed';
     return `${name}.${suffix}`;

diff --git a/packages/hms-video-store/src/common/PluginUsageTracker.ts b/packages/hms-video-store/src/common/PluginUsageTracker.ts
diff --git a/packages/hms-video-store/src/common/index.ts b/packages/hms-video-store/src/common/index.ts
diff --git a/packages/hms-video-store/src/index.ts b/packages/hms-video-store/src/index.ts
@@ -53,7 +53,6 @@ export type {
   HMSQuizLeaderboardSummary,
 } from './internal';
 
-export { pluginUsageTracker } from './common';
 export { HMSReactiveStore } from './reactive-store/HMSReactiveStore';
 export { HMSPluginUnsupportedTypes, HMSRecordingState, HLSPlaylistType } from './internal';
 export type {

diff --git a/packages/hms-video-store/src/plugins/audio/AudioPluginsAnalytics.ts b/packages/hms-video-store/src/plugins/audio/AudioPluginsAnalytics.ts
@@ -24,7 +24,6 @@ export class AudioPluginsAnalytics {
     this.addedTimestamps[name] = Date.now();
     this.initTime[name] = 0;
     this.pluginSampleRate[name] = sampleRate;
-    this.eventBus.analytics.publish(MediaPluginsAnalyticsFactory.added(name, this.addedTimestamps[name]));
   }
 
   removed(name: string) {

diff --git a/packages/hms-video-store/src/sdk/index.ts b/packages/hms-video-store/src/sdk/index.ts
@@ -12,7 +12,6 @@ import { HMSAnalyticsLevel } from '../analytics/AnalyticsEventLevel';
 import { AnalyticsEventsService } from '../analytics/AnalyticsEventsService';
 import { AnalyticsTimer, TimedEvent } from '../analytics/AnalyticsTimer';
 import { AudioSinkManager } from '../audio-sink-manager';
-import { pluginUsageTracker } from '../common/PluginUsageTracker';
 import { DeviceManager } from '../device-manager';
 import { AudioOutputManager } from '../device-manager/AudioOutputManager';
 import { DeviceStorageManager } from '../device-manager/DeviceStorage';
@@ -182,6 +181,7 @@ export class HMSSdk implements HMSInterface {
     );
     this.sessionStore = new SessionStore(this.transport);
     this.interactivityCenter = new InteractivityCenter(this.transport, this.store, this.listener);
+
     /**
      * Note: Subscribe to events here right after creating stores and managers
      * to not miss events that are published before the handlers are subscribed.
@@ -498,7 +498,6 @@ export class HMSSdk implements HMSInterface {
     this.errorListener?.onError(error);
   };
 
-  // eslint-disable-next-line complexity
   async join(config: HMSConfig, listener: HMSUpdateListener) {
     validateMediaDevicesExistence();
     validateRTCPeerConnection();
@@ -572,8 +571,6 @@ export class HMSSdk implements HMSInterface {
       throw error;
     }
     HMSLogger.timeEnd(`join-room-${roomId}`);
-    const sessionID = this.store.getRoom()?.sessionId || '';
-    this.eventBus.analytics.subscribe(e => pluginUsageTracker.updatePluginUsageData(e, sessionID));
   }
 
   private stringifyMetadata(config: HMSConfig) {

diff --git a/packages/hms-video-store/src/transport/index.ts b/packages/hms-video-store/src/transport/index.ts
@@ -495,9 +495,6 @@ export default class HMSTransport {
     this.joinParameters = undefined;
     HMSLogger.d(TAG, 'leaving in transport');
     try {
-      const sessionID = this.store.getRoom()?.sessionId || '';
-      this.eventBus.analytics.publish(AnalyticsEventFactory.getKrispUsage(sessionID));
-      this.eventBus.analytics.publish(AnalyticsEventFactory.transportLeave());
       this.state = TransportState.Leaving;
       this.publishStatsAnalytics?.stop();
       this.subscribeStatsAnalytics?.stop();