add session aggregate typing

Author: AbhiPrasad

packages/core/src/baseclient.ts

@@ -10,6 +10,7 @@ import {
   Integration,
   IntegrationClass,
   NewTransport,
+  SessionAggregates,
   Severity,
   SeverityLevel,
   Transport,
@@ -186,7 +187,7 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
     if (!(typeof session.release === 'string')) {
       IS_DEBUG_BUILD && logger.warn('Discarded session because of missing or non-string release');
     } else {
-      this._sendSession(session);
+      this.sendSession(session);
       // After sending, we set init false to indicate it's not the first occurrence
       session.update({ init: false });
     }
@@ -271,7 +272,7 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
   /**
    * @inheritDoc
    */
-  public sendSession(session: Session): void {
+  public sendSession(session: Session | SessionAggregates): void {
     if (this._dsn) {
       const [env] = createSessionEnvelope(session, this._dsn, this._options._metadata, this._options.tunnel);
       this.sendEnvelope(env);
@@ -324,12 +325,6 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
     }
   }
 
-  /** Deliver captured session to Sentry */
-  // TODO(v7): should this be deleted?
-  protected _sendSession(session: Session): void {
-    this.sendSession(session);
-  }
-
   /**
    * Determine if the client is finished processing. Returns a promise because it will wait `timeout` ms before saying
    * "no" (resolving to `false`) in order to give the client a chance to potentially finish first.

packages/hub/src/sessionflusher.ts

@@ -27,21 +27,14 @@ export class SessionFlusher implements SessionFlusherLike {
     this._sessionAttrs = attrs;
   }
 
-  /** Sends session aggregates to Transport */
-  public sendSessionAggregates(sessionAggregates: SessionAggregates): void {
-    void this._client.sendSession(sessionAggregates).then(null, reason => {
-      IS_DEBUG_BUILD && logger.error('Error while sending session:', reason);
-    });
-  }
-
   /** Checks if `pendingAggregates` has entries, and if it does flushes them by calling `sendSessions` */
   public flush(): void {
     const sessionAggregates = this.getSessionAggregates();
     if (sessionAggregates.aggregates.length === 0) {
       return;
     }
     this._pendingAggregates = {};
-    this.sendSessionAggregates(sessionAggregates);
+    this._client.sendSession(sessionAggregates);
   }
 
   /** Massages the entries in `pendingAggregates` and returns aggregated sessions */