@thoughtspot/visual-embed-sdk 1.44.4-HE-beta.1 → 1.44.4

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@thoughtspot/visual-embed-sdk",
-    "version": "1.44.4-HE-beta.1",
+    "version": "1.44.4",
     "description": "ThoughtSpot Embed SDK",
     "module": "lib/src/index.js",
     "main": "dist/tsembed.js",

package/src/embed/app.ts

@@ -19,7 +19,6 @@ import {
     AllEmbedViewConfig,
 } from '../types';
 import { V1Embed } from './ts-embed';
-import { PageContextOptions } from './hostEventClient/contracts';
 
 /**
  * Pages within the ThoughtSpot app that can be embedded.

package/src/embed/conversation.ts

@@ -3,7 +3,6 @@ import { ERROR_MESSAGE } from '../errors';
 import { Param, BaseViewConfig, RuntimeFilter, RuntimeParameter, ErrorDetailsTypes, EmbedErrorCodes } from '../types';
 import { TsEmbed } from './ts-embed';
 import { getQueryParamString, getFilterQuery, getRuntimeParameters } from '../utils';
-import { PageContextOptions } from './hostEventClient/contracts';
 
 /**
  * Configuration for search options
@@ -329,16 +328,6 @@ export class SpotterEmbed extends TsEmbed {
         await this.renderIFrame(src);
         return this;
     }
-
-    /**
-     * Get the current context of the embedded SpotterEmbed.
-     * @returns The current context object containing the page type and object ids.
-     * @version SDK: 1.46.0 | ThoughtSpot: 26.3.0.cl
-     */
-    public async getCurrentContext(): Promise<PageContextOptions> {
-        const context = await super.getCurrentContext();
-        return context;
-    }
 }
 
 /**

package/src/embed/hostEventClient/contracts.ts

@@ -1,4 +1,4 @@
-import { ContextType, HostEvent } from '../../types';
+import { HostEvent } from '../../types';
 
 export enum UIPassthroughEvent {
   PinAnswerToLiveboard = 'addVizToPinboard',
@@ -103,7 +103,7 @@ export type HostEventRequest<HostEventT extends HostEvent> =
     ? UIPassthroughRequest<EmbedApiHostEventMapping[HostEventT]>
     : any;
 
-export type HostEventResponse<HostEventT extends HostEvent, ContextT extends ContextType> =
+export type HostEventResponse<HostEventT extends HostEvent> =
   HostEventT extends keyof EmbedApiHostEventMapping
     ? UIPassthroughResponse<EmbedApiHostEventMapping[HostEventT]>
     : any;
@@ -111,24 +111,5 @@ export type HostEventResponse<HostEventT extends HostEvent, ContextT extends Con
 // trigger response and request
 export type TriggerPayload<PayloadT, HostEventT extends HostEvent> =
   PayloadT | HostEventRequest<HostEventT>;
-export type TriggerResponse<PayloadT, HostEventT extends HostEvent, ContextT extends ContextType> =
-  PayloadT extends HostEventRequest<HostEventT> ? HostEventResponse<HostEventT, ContextT> : any;
-
-export enum PageType {
-  PAGE = 'page',
-  DIALOG = 'dialog',
-}
-
-interface Objects {
-  answerId?: string;
-  liveboardId?: string;
-  vizIds?: string[];
-  dataModelIds?: string[];
-  modalTitle?: string;
-}
-
-export interface PageContextOptions {
-  page: ContextType;
-  pageType: PageType;
-  objects: Objects;
-}
+export type TriggerResponse<PayloadT, HostEventT extends HostEvent> =
+  PayloadT extends HostEventRequest<HostEventT> ? HostEventResponse<HostEventT> : any;

package/src/embed/hostEventClient/host-event-client.spec.ts

@@ -54,7 +54,6 @@ describe('HostEventClient', () => {
                     type: apiName,
                     parameters,
                 },
-                undefined,
             );
             expect(result).toEqual(await triggerResponse);
         });
@@ -149,7 +148,6 @@ describe('HostEventClient', () => {
                 HostEvent.UIPassthrough,
                 'http://localhost',
                 { parameters: payload, type: UIPassthroughEvent.PinAnswerToLiveboard },
-                undefined,
             );
             expect(result).toEqual(mockResponse.value);
         });
@@ -187,7 +185,6 @@ describe('HostEventClient', () => {
                     parameters: payload,
                     type: 'saveAnswer',
                 },
-                undefined,
             );
             expect(result).toEqual({ answerId: 'newAnswer', ...mockResponse[0].value });
         });
@@ -201,7 +198,7 @@ describe('HostEventClient', () => {
 
             const result = await client.triggerHostEvent(hostEvent, payload);
 
-            expect(client.hostEventFallback).toHaveBeenCalledWith(hostEvent, payload, undefined);
+            expect(client.hostEventFallback).toHaveBeenCalledWith(hostEvent, payload);
             expect(result).toEqual(mockResponse);
         });
 
@@ -226,7 +223,6 @@ describe('HostEventClient', () => {
                 HostEvent.Pin,
                 mockThoughtSpotHost,
                 {},
-                undefined,
             );
             expect(result).toEqual([mockResponse]);
         });
@@ -252,7 +248,6 @@ describe('HostEventClient', () => {
                 HostEvent.Save,
                 mockThoughtSpotHost,
                 {},
-                undefined,
             );
             expect(result).toEqual([mockResponse]);
         });
@@ -308,7 +303,6 @@ describe('HostEventClient', () => {
                     parameters: { ...payload, pinboardId: 'test', newPinboardName: 'testLiveboard' },
                     type: 'addVizToPinboard',
                 },
-                undefined,
             );
             expect(result).toEqual({
                 pinboardId: 'testLiveboard',

package/src/embed/hostEventClient/host-event-client.ts

@@ -1,4 +1,4 @@
-import { ContextType, HostEvent } from '../../types';
+import { HostEvent } from '../../types';
 import { processTrigger as processTriggerService } from '../../utils/processTrigger';
 import { getEmbedConfig } from '../embedConfig';
 import {
@@ -23,7 +23,7 @@ export class HostEventClient {
    * @param {any} data Data to send with the host event
    * @returns {Promise<any>} - the response from the process trigger
    */
-  protected async processTrigger(message: HostEvent, data: any, context?: ContextType): Promise<any> {
+  protected async processTrigger(message: HostEvent, data: any): Promise<any> {
       if (!this.iFrame) {
           throw new Error('Iframe element is not set');
       }
@@ -34,16 +34,14 @@ export class HostEventClient {
           message,
           thoughtspotHost,
           data,
-          context,
       );
   }
 
   public async handleHostEventWithParam<UIPassthroughEventT extends UIPassthroughEvent>(
       apiName: UIPassthroughEventT,
       parameters: UIPassthroughRequest<UIPassthroughEventT>,
-      context?: ContextType,
   ): Promise<UIPassthroughResponse<UIPassthroughEventT>> {
-      const response = (await this.triggerUIPassthroughApi(apiName, parameters, context as ContextType))
+      const response = (await this.triggerUIPassthroughApi(apiName, parameters))
           ?.filter?.((r) => r.error || r.value)[0];
 
       if (!response) {
@@ -67,9 +65,8 @@ export class HostEventClient {
   public async hostEventFallback(
       hostEvent: HostEvent,
       data: any,
-      context?: ContextType,
   ): Promise<any> {
-      return this.processTrigger(hostEvent, data, context);
+      return this.processTrigger(hostEvent, data);
   }
 
   /**
@@ -83,22 +80,20 @@ export class HostEventClient {
   public async triggerUIPassthroughApi<UIPassthroughEventT extends UIPassthroughEvent>(
       apiName: UIPassthroughEventT,
       parameters: UIPassthroughRequest<UIPassthroughEventT>,
-      context?: ContextType,
   ): Promise<UIPassthroughArrayResponse<UIPassthroughEventT>> {
       const res = await this.processTrigger(HostEvent.UIPassthrough, {
           type: apiName,
           parameters,
-      }, context);
+      });
 
       return res;
   }
 
   protected async handlePinEvent(
       payload: HostEventRequest<HostEvent.Pin>,
-      context?: ContextType,
-  ): Promise<HostEventResponse<HostEvent.Pin, ContextType>> {
+  ): Promise<HostEventResponse<HostEvent.Pin>> {
       if (!payload || !('newVizName' in payload)) {
-          return this.hostEventFallback(HostEvent.Pin, payload, context);
+          return this.hostEventFallback(HostEvent.Pin, payload);
       }
 
       const formattedPayload = {
@@ -109,7 +104,6 @@ export class HostEventClient {
 
       const data = await this.handleHostEventWithParam(
           UIPassthroughEvent.PinAnswerToLiveboard, formattedPayload,
-          context as ContextType,
       );
 
       return {
@@ -120,16 +114,14 @@ export class HostEventClient {
 
   protected async handleSaveAnswerEvent(
       payload: HostEventRequest<HostEvent.SaveAnswer>,
-      context?: ContextType,
   ): Promise<any> {
       if (!payload || !('name' in payload) || !('description' in payload)) {
           // Save is the fallback for SaveAnswer
-          return this.hostEventFallback(HostEvent.Save, payload, context);
+          return this.hostEventFallback(HostEvent.Save, payload);
       }
 
       const data = await this.handleHostEventWithParam(
           UIPassthroughEvent.SaveAnswer, payload,
-          context as ContextType,
       );
       return {
           ...data,
@@ -140,22 +132,19 @@ export class HostEventClient {
   public async triggerHostEvent<
     HostEventT extends HostEvent,
     PayloadT,
-    ContextT extends ContextType,
   >(
       hostEvent: HostEventT,
       payload?: TriggerPayload<PayloadT, HostEventT>,
-      context?: ContextT,
-  ): Promise<TriggerResponse<PayloadT, HostEventT, ContextType>> {
+  ): Promise<TriggerResponse<PayloadT, HostEventT>> {
       switch (hostEvent) {
           case HostEvent.Pin:
-              return this.handlePinEvent(payload as HostEventRequest<HostEvent.Pin>, context as ContextType) as any;
+              return this.handlePinEvent(payload as HostEventRequest<HostEvent.Pin>) as any;
           case HostEvent.SaveAnswer:
               return this.handleSaveAnswerEvent(
                   payload as HostEventRequest<HostEvent.SaveAnswer>,
-                  context as ContextType,
               ) as any;
           default:
-              return this.hostEventFallback(hostEvent, payload, context);
+              return this.hostEventFallback(hostEvent, payload);
       }
   }
 }

package/src/embed/liveboard.spec.ts

@@ -1495,7 +1495,7 @@ describe('Liveboard/viz embed tests', () => {
                 await liveboardEmbed.trigger(HostEvent.Save);
                 expect(mockProcessTrigger).toHaveBeenCalledWith(HostEvent.Save, {
                     vizId: 'testViz',
-                }, undefined);
+                });
             });
         });
     });

package/src/embed/liveboard.ts

@@ -22,13 +22,12 @@ import {
     LiveboardAppEmbedViewConfig,
     ErrorDetailsTypes,
     EmbedErrorCodes,
-    ContextType,
 } from '../types';
 import { calculateVisibleElementData, getQueryParamString, isUndefined, isValidCssMargin } from '../utils';
 import { getAuthPromise } from './base';
 import { TsEmbed, V1Embed } from './ts-embed';
 import { addPreviewStylesIfNotPresent } from '../utils/global-styles';
-import { TriggerPayload, TriggerResponse, PageContextOptions } from './hostEventClient/contracts';
+import { TriggerPayload, TriggerResponse } from './hostEventClient/contracts';
 import { logger } from '../utils/logger';
 
 
@@ -643,7 +642,6 @@ export class LiveboardEmbed extends V1Embed {
     }
 
     private sendFullHeightLazyLoadData = () => {
-        console.log('sendFullHeightLazyLoadData', this.iFrame);
         const data = calculateVisibleElementData(this.iFrame);
         this.trigger(HostEvent.VisibleEmbedCoordinates, data);
     }
@@ -802,11 +800,10 @@ export class LiveboardEmbed extends V1Embed {
      * @param {any} data The payload to send with the message
      * @returns A promise that resolves with the response from the embedded app
      */
-    public trigger<HostEventT extends HostEvent, PayloadT, ContextT extends ContextType>(
+    public trigger<HostEventT extends HostEvent, PayloadT>(
         messageType: HostEventT,
         data: TriggerPayload<PayloadT, HostEventT> = ({} as any),
-        context?: ContextT,
-    ): Promise<TriggerResponse<PayloadT, HostEventT, ContextT>> {
+    ): Promise<TriggerResponse<PayloadT, HostEventT>> {
         const dataWithVizId: any = data;
         if (messageType === HostEvent.SetActiveTab) {
             this.setActiveTab(data as { tabId: string });
@@ -815,7 +812,7 @@ export class LiveboardEmbed extends V1Embed {
         if (typeof dataWithVizId === 'object' && this.viewConfig.vizId) {
             dataWithVizId.vizId = this.viewConfig.vizId;
         }
-        return super.trigger(messageType, dataWithVizId, context);
+        return super.trigger(messageType, dataWithVizId);
     }
     /**
      * Destroys the ThoughtSpot embed, and remove any nodes from the DOM.
@@ -892,16 +889,6 @@ export class LiveboardEmbed extends V1Embed {
 
         return url;
     }
-
-    /**
-     * Get the current context of the embedded liveboard.
-     * @returns The current context object containing the page type and object ids.
-     * @version SDK: 1.46.0 | ThoughtSpot: 26.3.0.cl
-     */
-    public async getCurrentContext(): Promise<PageContextOptions> {
-        const context = await super.getCurrentContext();
-        return context;
-    } 
 }
 
 /**

package/src/embed/sage.ts

@@ -6,7 +6,6 @@
  * @author Mourya Balabhadra <mourya.balabhadra@thoughtspot.com>
  */
 
-import { PageContextOptions } from './hostEventClient/contracts';
 import { DOMSelector, Param, BaseViewConfig, SearchLiveboardCommonViewConfig } from '../types';
 import { getQueryParamString } from '../utils';
 import { V1Embed } from './ts-embed';
@@ -230,14 +229,4 @@ export class SageEmbed extends V1Embed {
 
         return this;
     }
-
-    /**
-     * Get the current context of the embedded SageEmbed.
-     * @returns The current context object containing the page type and object ids.
-     * @version SDK: 1.46.0 | ThoughtSpot: 26.3.0.cl
-     */
-    public async getCurrentContext(): Promise<PageContextOptions> {
-        const context = await super.getCurrentContext();
-        return context;
-    }
 }

package/src/embed/search-bar.tsx

@@ -2,7 +2,6 @@ import { SearchLiveboardCommonViewConfig, BaseViewConfig, DefaultAppInitData, Pa
 import { getQueryParamString } from '../utils';
 import { TsEmbed } from './ts-embed';
 import { SearchOptions } from './search';
-import { PageContextOptions } from './hostEventClient/contracts';
 
 /**
  * @group Embed components
@@ -199,14 +198,4 @@ export class SearchBarEmbed extends TsEmbed {
         const defaultAppInitData = await super.getAppInitData();
         return { ...defaultAppInitData, ...this.getSearchInitData() };
     }
-
-    /**
-     * Get the current context of the embedded search bar.
-     * @returns The current context object containing the page type and object ids.
-     * @version SDK: 1.46.0 | ThoughtSpot: 26.3.0.cl
-     */
-    public async getCurrentContext(): Promise<PageContextOptions> {
-        const context = await super.getCurrentContext();
-        return context;
-    }
 }

package/src/embed/search.ts

@@ -27,7 +27,6 @@ import { getAuthPromise } from './base';
 import { getReleaseVersion } from '../auth';
 import { getEmbedConfig } from './embedConfig';
 import { getInterceptInitData } from '../api-intercept';
-import { PageContextOptions } from './hostEventClient/contracts';
 
 /**
  * Configuration for search options.
@@ -524,14 +523,4 @@ export class SearchEmbed extends TsEmbed {
         });
         return this;
     }
-
-    /**
-     * Get the current context of the embedded search.
-     * @returns The current context object containing the page type and object ids.
-     * @version SDK: 1.46.0 | ThoughtSpot: 26.3.0.cl
-     */
-    public async getCurrentContext(): Promise<PageContextOptions> {
-        const context = await super.getCurrentContext();
-        return context;
-    }
 }

package/src/embed/ts-embed.spec.ts

@@ -204,7 +204,6 @@ describe('Unit test case for ts embed', () => {
                         parameters: payload,
                         type: UIPassthroughEvent.PinAnswerToLiveboard,
                     },
-                    undefined,
                 );
             });
         });
@@ -225,7 +224,6 @@ describe('Unit test case for ts embed', () => {
                     HostEvent.Save,
                     'http://tshost',
                     {},
-                    undefined,
                 );
             });
         });
@@ -247,7 +245,6 @@ describe('Unit test case for ts embed', () => {
                     HostEvent.Save,
                     'http://tshost',
                     false,
-                    undefined,
                 );
             });
         });
@@ -1334,7 +1331,6 @@ describe('Unit test case for ts embed', () => {
                     HostEvent.InfoSuccess,
                     'http://tshost',
                     expect.objectContaining({ info: expect.any(Object) }),
-                    undefined,
                 );
             });
         });
@@ -1471,7 +1467,6 @@ describe('Unit test case for ts embed', () => {
                     HostEvent.InfoSuccess,
                     'http://tshost',
                     expect.objectContaining({ info: expect.any(Object) }),
-                    undefined,
                 );
             });
         });
@@ -1486,7 +1481,6 @@ describe('Unit test case for ts embed', () => {
                     HostEvent.InfoSuccess,
                     'http://tshost',
                     expect.objectContaining({ info: expect.any(Object) }),
-                    undefined,
                 );
             });
         });
@@ -1501,7 +1495,6 @@ describe('Unit test case for ts embed', () => {
                     HostEvent.InfoSuccess,
                     'http://tshost',
                     expect.objectContaining({ info: expect.any(Object) }),
-                    undefined,
                 );
             });
         });
@@ -1516,7 +1509,6 @@ describe('Unit test case for ts embed', () => {
                     HostEvent.InfoSuccess,
                     'http://tshost',
                     expect.objectContaining({ info: expect.any(Object) }),
-                    undefined,
                 );
             });
         });
@@ -4063,7 +4055,7 @@ describe('Destroy error handling', () => {
         }).not.toThrow();
         
         expect(logSpy).toHaveBeenCalledWith('Error destroying TS Embed', expect.any(Error));
-        logSpy.mockReset();
+        logSpy.mockRestore();
     });
 });
 
@@ -4106,12 +4098,11 @@ describe('Fullscreen change handler behavior', () => {
         document.dispatchEvent(event);
         
         await executeAfterWait(() => {
-            expect(mockProcessTrigger).toHaveBeenLastCalledWith(
+            expect(mockProcessTrigger).toHaveBeenCalledWith(
                 expect.any(Object),
                 HostEvent.ExitPresentMode,
                 expect.any(String),
                 expect.any(Object),
-                undefined,
             );
         });
     });
@@ -4204,14 +4195,13 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
         embed2.showPreRender();
 
         await executeAfterWait(() => {
-            expect(mockProcessTrigger).toHaveBeenLastCalledWith(
+            expect(mockProcessTrigger).toHaveBeenCalledWith(
                 expect.any(Object),
                 HostEvent.UpdateEmbedParams,
                 expect.any(String),
                 expect.objectContaining({
                     liveboardId: 'updated-lb',
                 }),
-                undefined,
             );
         });
     });
@@ -4240,7 +4230,7 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
         embed2.showPreRender();
 
         await executeAfterWait(() => {
-            expect(mockProcessTrigger).toHaveBeenLastCalledWith(
+            expect(mockProcessTrigger).toHaveBeenCalledWith(
                 expect.any(Object),
                 HostEvent.UpdateEmbedParams,
                 expect.any(String),
@@ -4260,7 +4250,6 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
                         },
                     ],
                 }),
-                undefined,
             );
         });
     });
@@ -4293,7 +4282,7 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
         embed2.showPreRender();
 
         await executeAfterWait(() => {
-            expect(mockProcessTrigger).toHaveBeenLastCalledWith(
+            expect(mockProcessTrigger).toHaveBeenCalledWith(
                 expect.any(Object),
                 HostEvent.UpdateEmbedParams,
                 expect.any(String),
@@ -4308,7 +4297,6 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
                         },
                     ],
                 }),
-                undefined,
             );
         });
     });

package/src/embed/ts-embed.ts

@@ -60,7 +60,6 @@ import {
     EmbedErrorDetailsEvent,
     ErrorDetailsTypes,
     EmbedErrorCodes,
-    ContextType,
 } from '../types';
 import { uploadMixpanelEvent, MIXPANEL_EVENT } from '../mixpanel-service';
 import { processEventData, processAuthFailure } from '../utils/processData';
@@ -76,7 +75,6 @@ import { ERROR_MESSAGE } from '../errors';
 import { getPreauthInfo } from '../utils/sessionInfoService';
 import { HostEventClient } from './hostEventClient/host-event-client';
 import { getInterceptInitData, handleInterceptEvent, processApiInterceptResponse, processLegacyInterceptResponse } from '../api-intercept';
-import { getHostEventsConfig } from '../utils';
 
 const { version } = pkgInfo;
 
@@ -482,7 +480,6 @@ export class TsEmbed {
             hiddenListColumns: this.viewConfig.hiddenListColumns || [],
             customActions: customActionsResult.actions,
             ...getInterceptInitData(this.viewConfig),
-            ...getHostEventsConfig(this.viewConfig),
         };
 
         return baseInitData;
@@ -1350,11 +1347,10 @@ export class TsEmbed {
      * @param {any} data The payload to send with the message
      * @returns A promise that resolves with the response from the embedded app
      */
-    public async trigger<HostEventT extends HostEvent, PayloadT, ContextT extends ContextType>(
+    public async trigger<HostEventT extends HostEvent, PayloadT>(
         messageType: HostEventT,
         data: TriggerPayload<PayloadT, HostEventT> = {} as any,
-        context?: ContextT,
-    ): Promise<TriggerResponse<PayloadT, HostEventT, ContextT>> {
+    ): Promise<TriggerResponse<PayloadT, HostEventT>> {
         uploadMixpanelEvent(`${MIXPANEL_EVENT.VISUAL_SDK_TRIGGER}-${messageType}`);
 
         if (!this.isRendered) {
@@ -1387,7 +1383,7 @@ export class TsEmbed {
         }
 
         // send an empty object, this is needed for liveboard default handlers
-        return this.hostEventClient.triggerHostEvent(messageType, data, context);
+        return this.hostEventClient.triggerHostEvent(messageType, data);
     }
 
     /**
@@ -1429,20 +1425,6 @@ export class TsEmbed {
         return this.render();
     }
 
-    /**
-     * Get the current context of the embedded TS component.
-     * @returns The current context object containing the page type and object ids.
-     * @version SDK: 1.46.0 | ThoughtSpot: 26.3.0.cl
-     */
-    public async getCurrentContext(): Promise<any> {
-        return new Promise((resolve) => {
-            this.executeAfterEmbedContainerLoaded(async () => {
-                const context = await this.trigger(HostEvent.GetPageContext, {});
-                resolve(context);
-            });
-        });
-    }
-
     /**
      * Creates the preRender shell
      * @param showPreRenderByDefault - Show the preRender after render, hidden by default

package/src/index.ts

@@ -68,7 +68,6 @@ import {
     InterceptedApiType,
     EmbedErrorCodes,
     ErrorDetailsTypes,
-    ContextType,
 } from './types';
 import { CustomCssVariables } from './css-variables';
 import { SageEmbed, SageViewConfig } from './embed/sage';
@@ -122,7 +121,6 @@ export {
     RuntimeFilterOp,
     EmbedEvent,
     HostEvent,
-    ContextType,
     DataSourceVisualMode,
     Action,
     ContextMenuTriggerOptions,