@thoughtspot/visual-embed-sdk 1.50.0 → 1.51.0

package/src/embed/liveboard.ts

@@ -402,11 +402,12 @@ export interface LiveboardViewConfig extends BaseViewConfig, LiveboardOtherViewC
     /**
      * Enables the 'what you see is what you get' PDF export for Liveboards. Each tab is rendered on a single page
      * following the exact UI layout, instead of splitting visualizations across multiple A4 pages.
-     * This feature is GA from version 26.5.0.cl. It is disabled by default in embed deployments.
+     * This feature is GA from SDK version 1.51.0 and ThoughtSpot version 26.8.0.cl.
      *
      * Supported embed types: `AppEmbed`, `LiveboardEmbed`
      * @type {boolean}
      * @version SDK: 1.48.0 | ThoughtSpot: 26.5.0.cl
+     * @default true
      * @example
      * ```js
      * // Replace <EmbedComponent> with embed component name. For example, AppEmbed or LiveboardEmbed
@@ -419,10 +420,12 @@ export interface LiveboardViewConfig extends BaseViewConfig, LiveboardOtherViewC
     isContinuousLiveboardPDFEnabled?: boolean;
     /**
      * This flag is used to enable/disable the XLSX/CSV download option for Liveboards
+     * This feature is GA from SDK version 1.51.0 and ThoughtSpot version 26.6.0.cl.
      *
      * Supported embed types: `AppEmbed`, `LiveboardEmbed`
      * @type {boolean}
      * @version SDK: 1.46.0 | ThoughtSpot: 26.3.0.cl
+     * @default true
      * @example
      * ```js
      * // Replace <EmbedComponent> with embed component name. For example, AppEmbed or LiveboardEmbed
@@ -435,10 +438,12 @@ export interface LiveboardViewConfig extends BaseViewConfig, LiveboardOtherViewC
     isLiveboardXLSXCSVDownloadEnabled?: boolean;
     /**
      * This flag is used to enable/disable the granular XLSX/CSV schedules feature
+     * This feature is GA from SDK version 1.51.0 and ThoughtSpot version 26.6.0.cl.
      *
      * Supported embed types: `AppEmbed`, `LiveboardEmbed`
      * @type {boolean}
      * @version SDK: 1.46.0 | ThoughtSpot: 26.3.0.cl
+     * @default true
      * @example
      * ```js
      * // Replace <EmbedComponent> with embed component name. For example, AppEmbed or LiveboardEmbed
@@ -529,6 +534,21 @@ export interface LiveboardViewConfig extends BaseViewConfig, LiveboardOtherViewC
      * ```
      */
     updatedSpotterChatPrompt?: boolean;
+    /**
+     * showSpotterRadiance : Controls the radiance on the Spotter page.
+     *
+     * Supported embed types: `LiveboardEmbed`
+     * @version SDK: 1.52.0 | ThoughtSpot Cloud: 26.9.0.cl
+     * @default false
+     * @example
+     * ```js
+     * const embed = new LiveboardEmbed('#tsEmbed', {
+     *    ... //other embed view config
+     *    showSpotterRadiance : true,
+     * })
+     * ```
+     */
+    showSpotterRadiance?: boolean;
     /**
      * Enables the stop answer generation button in the Spotter embed UI,
      * allowing users to interrupt an ongoing answer generation.
@@ -580,6 +600,12 @@ export interface LiveboardViewConfig extends BaseViewConfig, LiveboardOtherViewC
      *            { label: 'Tip', text: 'try asking about revenue by region' },
      *            { label: 'Tip', text: 'use natural language' },
      *        ],
+     *        // liveboardBrandName, spotterBrandName, insightTileBrandName, insightTileViewPlanLabel and insightTileLoaderText require SDK: 1.52.0 | ThoughtSpot Cloud: 26.9.0.cl
+     *        liveboardBrandName: 'Reports',
+     *        spotterBrandName: 'Analyst',
+     *        insightTileBrandName: 'Insight card',
+     *        insightTileViewPlanLabel: 'View plan',
+     *        insightTileLoaderText: 'Generating insight',
      *    },
      * })
      * ```
@@ -598,6 +624,21 @@ export interface LiveboardViewConfig extends BaseViewConfig, LiveboardOtherViewC
      * ```
      */
     enableLiveboardDataCache?: boolean;
+    /**
+     * updatedSpotterExperience : Controls the updated Spotter experience.
+     *
+     * Supported embed types: `LiveboardEmbed`
+     * @version SDK: 1.52.0 | ThoughtSpot Cloud: 26.9.0.cl
+     * @default false
+     * @example
+     * ```js
+     * const embed = new LiveboardEmbed('#tsEmbed', {
+     *    ... //other embed view config
+     *    updatedSpotterExperience : true,
+     * })
+     * ```
+     */
+    updatedSpotterExperience?: boolean;
 }
 
 /**
@@ -673,34 +714,37 @@ export class LiveboardEmbed extends V1Embed {
             showLiveboardDescription,
             showLiveboardTitle,
             isLiveboardHeaderSticky = true,
-            isLiveboardCompactHeaderEnabled = false,
+            isLiveboardCompactHeaderEnabled,
             showLiveboardVerifiedBadge = true,
             showLiveboardReverifyBanner = true,
-            hideIrrelevantChipsInLiveboardTabs = false,
+            hideIrrelevantChipsInLiveboardTabs,
             showMaskedFilterChip = false,
-            isLiveboardMasterpiecesEnabled = false,
+            isLiveboardMasterpiecesEnabled,
             newChartsLibrary,
-            isEnhancedFilterInteractivityEnabled = false,
+            isEnhancedFilterInteractivityEnabled,
             enableAskSage,
             enable2ColumnLayout,
             dataPanelV2 = true,
+            updatedSpotterExperience,
             enableCustomColumnGroups = false,
             oAuthPollingInterval,
             isForceRedirect,
             dataSourceId,
-            coverAndFilterOptionInPDF = false,
+            coverAndFilterOptionInPDF,
             isLiveboardStylingAndGroupingEnabled,
             isPNGInScheduledEmailsEnabled = false,
-            isLiveboardXLSXCSVDownloadEnabled = false,
-            isGranularXLSXCSVSchedulesEnabled = false,
+            isLiveboardXLSXCSVDownloadEnabled,
+            isGranularXLSXCSVSchedulesEnabled,
             showSpotterLimitations,
             isCentralizedLiveboardFilterUXEnabled = false,
+            isScopedLiveboardFilteringEnabled,
             isLinkParametersEnabled,
             updatedSpotterChatPrompt,
+            showSpotterRadiance,
             enableStopAnswerGenerationEmbed,
             spotterChatConfig,
             isThisPeriodInDateFiltersEnabled,
-            isContinuousLiveboardPDFEnabled = false,
+            isContinuousLiveboardPDFEnabled,
             enableLiveboardDataCache,
         } = this.viewConfig;
 
@@ -726,6 +770,9 @@ export class LiveboardEmbed extends V1Embed {
         if (!isUndefined(updatedSpotterChatPrompt)) {
             params[Param.UpdatedSpotterChatPrompt] = !!updatedSpotterChatPrompt;
         }
+        if (!isUndefined(showSpotterRadiance)) {
+            params[Param.ShowSpotterRadiance] = !!showSpotterRadiance;
+        }
         if (!isUndefined(enableStopAnswerGenerationEmbed)) {
             params[Param.EnableStopAnswerGenerationEmbed] = !!enableStopAnswerGenerationEmbed;
         }
@@ -790,6 +837,10 @@ export class LiveboardEmbed extends V1Embed {
             params[Param.ShowSpotterLimitations] = showSpotterLimitations;
         }
 
+        if (!isUndefined(updatedSpotterExperience)) {
+            params[Param.UpdatedSpotterExperience] = !!updatedSpotterExperience;
+        }
+
         // Handle spotterChatConfig params
         if (spotterChatConfig) {
             const {
@@ -823,6 +874,12 @@ export class LiveboardEmbed extends V1Embed {
             ] = isCentralizedLiveboardFilterUXEnabled;
         }
 
+        if (isScopedLiveboardFilteringEnabled !== undefined) {
+            params[
+                Param.isScopedLiveboardFilteringEnabled
+            ] = isScopedLiveboardFilteringEnabled;
+        }
+
         if (isThisPeriodInDateFiltersEnabled !== undefined) {
             params[Param.IsThisPeriodInDateFiltersEnabled] = isThisPeriodInDateFiltersEnabled;
         }
@@ -840,16 +897,27 @@ export class LiveboardEmbed extends V1Embed {
         }
 
         params[Param.LiveboardHeaderSticky] = isLiveboardHeaderSticky;
-        params[Param.LiveboardHeaderV2] = isLiveboardCompactHeaderEnabled;
+        if (!isUndefined(isLiveboardCompactHeaderEnabled)) {
+            params[Param.LiveboardHeaderV2] = isLiveboardCompactHeaderEnabled;
+        }
         params[Param.ShowLiveboardVerifiedBadge] = showLiveboardVerifiedBadge;
         params[Param.ShowLiveboardReverifyBanner] = showLiveboardReverifyBanner;
-        params[Param.HideIrrelevantFiltersInTab] = hideIrrelevantChipsInLiveboardTabs;
+        if (!isUndefined(hideIrrelevantChipsInLiveboardTabs)) {
+            params[Param.HideIrrelevantFiltersInTab] = hideIrrelevantChipsInLiveboardTabs;
+        }
         params[Param.ShowMaskedFilterChip] = showMaskedFilterChip;
-        params[Param.IsLiveboardMasterpiecesEnabled] = isLiveboardMasterpiecesEnabled;
-        params[Param.IsEnhancedFilterInteractivityEnabled] = isEnhancedFilterInteractivityEnabled;
+        if (!isUndefined(isLiveboardMasterpiecesEnabled)) {
+            params[Param.IsLiveboardMasterpiecesEnabled] = isLiveboardMasterpiecesEnabled;
+        }
+        if (!isUndefined(isEnhancedFilterInteractivityEnabled)) {
+            params[Param.IsEnhancedFilterInteractivityEnabled] =
+                isEnhancedFilterInteractivityEnabled;
+        }
         params[Param.DataPanelV2Enabled] = dataPanelV2;
         params[Param.EnableCustomColumnGroups] = enableCustomColumnGroups;
-        params[Param.CoverAndFilterOptionInPDF] = coverAndFilterOptionInPDF;
+        if (!isUndefined(coverAndFilterOptionInPDF)) {
+            params[Param.CoverAndFilterOptionInPDF] = coverAndFilterOptionInPDF;
+        }
 
         const queryParams = getQueryParamString(params, true);
 

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

@@ -1,4 +1,8 @@
-import { resolveEnablePastConversationsSidebar, buildSpotterSidebarAppInitData } from './spotter-utils';
+import {
+    resolveEnablePastConversationsSidebar,
+    buildSpotterSidebarAppInitData,
+    buildSpotterShareConversationAppInitData,
+} from './spotter-utils';
 import { ErrorDetailsTypes, EmbedErrorCodes } from '../types';
 import { ERROR_MESSAGE } from '../errors';
 
@@ -36,6 +40,22 @@ describe('buildSpotterSidebarAppInitData', () => {
         });
     });
 
+    it('forwards a nested spotterChatPinConfig object through untouched', () => {
+        const spotterChatPinConfig = {
+            enabled: true,
+            pinLabel: 'Pin to top',
+            unpinLabel: 'Unpin',
+        };
+        const result = buildSpotterSidebarAppInitData(base, {
+            spotterSidebarConfig: {
+                enablePastConversationsSidebar: true,
+                spotterChatPinConfig,
+            },
+        }, noopError);
+        expect(result.embedParams?.spotterSidebarConfig?.spotterChatPinConfig)
+            .toEqual(spotterChatPinConfig);
+    });
+
     it('promotes standalone flag into spotterSidebarConfig.enablePastConversationsSidebar', () => {
         const result = buildSpotterSidebarAppInitData(base, { enablePastConversationsSidebar: true }, noopError);
         expect(result.embedParams?.spotterSidebarConfig?.enablePastConversationsSidebar).toBe(true);
@@ -106,3 +126,50 @@ describe('buildSpotterSidebarAppInitData', () => {
         expect(result.embedParams?.spotterSidebarConfig?.enablePastConversationsSidebar).toBe(true);
     });
 });
+
+describe('buildSpotterShareConversationAppInitData', () => {
+    const base = { type: 'APP_INIT' } as any;
+
+    it('returns base unchanged when no spotterShareConversationConfig', () => {
+        const result = buildSpotterShareConversationAppInitData(base, {});
+        expect(result).toBe(base);
+    });
+
+    it('nests spotterShareConversationConfig under embedParams', () => {
+        const result = buildSpotterShareConversationAppInitData(base, {
+            spotterShareConversationConfig: { enableShareConversation: true },
+        });
+        expect(result.embedParams?.spotterShareConversationConfig).toEqual({
+            enableShareConversation: true,
+        });
+    });
+
+    it('passes label override fields through untouched', () => {
+        const spotterShareConversationConfig = {
+            enableShareConversation: true,
+            spotterShareLabel: 'Share',
+            spotterShareModalTitle: 'Share conversation',
+            spotterShareConfirmLabel: 'Share',
+            spotterShareCancelLabel: 'Cancel',
+            spotterShareAddUsersLabel: 'Add users or groups',
+            spotterShareEmptyTitle: 'No users added yet',
+            spotterShareEmptySubtitle: 'Not shared with any user',
+            spotterShareIncludeNewMessagesLabel: 'Include new messages',
+            spotterShareUpToCurrentLabel: 'Share up to current moment',
+            spotterShareStaleInfoLabel: 'This snapshot may be stale',
+        };
+        const result = buildSpotterShareConversationAppInitData(base, {
+            spotterShareConversationConfig,
+        });
+        expect(result.embedParams?.spotterShareConversationConfig).toEqual(spotterShareConversationConfig);
+    });
+
+    it('preserves existing embedParams when nesting the share config', () => {
+        const withParams = { type: 'APP_INIT', embedParams: { existing: 'keep' } } as any;
+        const result = buildSpotterShareConversationAppInitData(withParams, {
+            spotterShareConversationConfig: { enableShareConversation: true },
+        });
+        expect(result.embedParams?.existing).toBe('keep');
+        expect(result.embedParams?.spotterShareConversationConfig?.enableShareConversation).toBe(true);
+    });
+});

package/src/embed/spotter-utils.ts

@@ -1,7 +1,7 @@
 import { DefaultAppInitData, ErrorDetailsTypes, EmbedErrorCodes } from '../types';
 import { validateHttpUrl } from '../utils';
 import { ERROR_MESSAGE } from '../errors';
-import type { SpotterSidebarViewConfig } from './conversation';
+import type { SpotterSidebarViewConfig, SpotterShareConversationConfig } from './conversation';
 import type { VisualizationOverrides } from '../types';
 
 /**
@@ -79,3 +79,18 @@ export function buildSpotterSidebarAppInitData<T extends DefaultAppInitData>(
         },
     };
 }
+
+export function buildSpotterShareConversationAppInitData<T extends DefaultAppInitData>(
+    initData: T,
+    viewConfig: { spotterShareConversationConfig?: SpotterShareConversationConfig },
+): T & { embedParams?: { spotterShareConversationConfig?: SpotterShareConversationConfig } } {
+    const { spotterShareConversationConfig } = viewConfig;
+    if (!spotterShareConversationConfig) return initData;
+    return {
+        ...initData,
+        embedParams: {
+            ...((initData as T & { embedParams?: Record<string, unknown> }).embedParams || {}),
+            spotterShareConversationConfig,
+        },
+    };
+}

package/src/embed/spotter-viz-utils.spec.ts

@@ -20,6 +20,23 @@ describe('buildSpotterVizAppInitData', () => {
         expect(result.embedParams?.spotterVizConfig?.brandHeadline).toBe("Hi, there! I'm");
     });
 
+    it('passes liveboardBrandName, spotterBrandName, insightTileBrandName, insightTileViewPlanLabel and insightTileLoaderText through spotterVizConfig', () => {
+        const spotterViz = {
+            brandName: 'MyBrand',
+            liveboardBrandName: 'Reports',
+            spotterBrandName: 'Analyst',
+            insightTileBrandName: 'Insight card',
+            insightTileViewPlanLabel: 'View plan',
+            insightTileLoaderText: 'Generating insight',
+        };
+        const result = buildSpotterVizAppInitData(base, { spotterViz });
+        expect(result.embedParams?.spotterVizConfig?.liveboardBrandName).toBe('Reports');
+        expect(result.embedParams?.spotterVizConfig?.spotterBrandName).toBe('Analyst');
+        expect(result.embedParams?.spotterVizConfig?.insightTileBrandName).toBe('Insight card');
+        expect(result.embedParams?.spotterVizConfig?.insightTileViewPlanLabel).toBe('View plan');
+        expect(result.embedParams?.spotterVizConfig?.insightTileLoaderText).toBe('Generating insight');
+    });
+
     it('preserves existing embedParams when adding spotterVizConfig', () => {
         const existing = { ...base, embedParams: { spotterSidebarConfig: { enablePastConversationsSidebar: true } } };
         const spotterViz = { brandName: 'MyBrand' };

package/src/embed/spotter-viz-utils.ts

@@ -38,6 +38,12 @@ export interface SpotterVizStarterPrompt {
  *            { label: 'Tip', text: 'try asking about revenue by region' },
  *            { label: 'Tip', text: 'use natural language' },
  *        ],
+ *        // liveboardBrandName, spotterBrandName, insightTileBrandName, insightTileViewPlanLabel and insightTileLoaderText require SDK: 1.52.0 | ThoughtSpot Cloud: 26.9.0.cl
+ *        liveboardBrandName: 'Reports',
+ *        spotterBrandName: 'Analyst',
+ *        insightTileBrandName: 'Insight card',
+ *        insightTileViewPlanLabel: 'View plan',
+ *        insightTileLoaderText: 'Generating insight',
  *    },
  * })
  * ```
@@ -94,6 +100,34 @@ export interface SpotterVizConfig {
      * @version SDK: 1.51.0 | ThoughtSpot Cloud: 26.8.0.cl
      */
     loaderTips?: SpotterVizLoaderTip[];
+    /**
+     * Custom term used to replace "Liveboard" in the agent's responses.
+     * @version SDK: 1.52.0 | ThoughtSpot Cloud: 26.9.0.cl
+     * @default ''
+     */
+    liveboardBrandName?: string;
+    /**
+     * Custom term used to replace "Spotter" in the agent's responses.
+     * @version SDK: 1.52.0 | ThoughtSpot Cloud: 26.9.0.cl
+     * @default ''
+     */
+    spotterBrandName?: string;
+    /**
+     * Custom term used to replace "Insight tile" in the UI and in the agent's responses.
+     * @version SDK: 1.52.0 | ThoughtSpot Cloud: 26.9.0.cl
+     * @default ''
+     */
+    insightTileBrandName?: string;
+    /**
+     * Custom term used to replace "View plan" in the insight tile menu.
+     * @version SDK: 1.52.0 | ThoughtSpot Cloud: 26.9.0.cl
+     */
+    insightTileViewPlanLabel?: string;
+    /**
+     * Custom loader text shown on the insight tile.
+     * @version SDK: 1.52.0 | ThoughtSpot Cloud: 26.9.0.cl
+     */
+    insightTileLoaderText?: string;
 }
 
 /**

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

@@ -1409,6 +1409,40 @@ describe('Unit test case for ts embed', () => {
             const { mockGetPreauthInfo } = await setup(true, 123);
             expect(mockGetPreauthInfo).toHaveBeenCalledTimes(0);
         });
+
+        test('should NOT trigger InfoSuccess when getPreauthInfo resolves with no .info field', async () => {
+            jest.spyOn(window, 'addEventListener').mockImplementationOnce(
+                (event, handler) => {
+                    (handler as EventListener)({
+                        data: { type: 'xyz' },
+                        ports: [3000],
+                        source: null,
+                    } as any);
+                },
+            );
+            mockProcessTrigger.mockResolvedValueOnce({ session: 'test' });
+            jest.spyOn(authService, 'fetchPreauthInfoService').mockResolvedValueOnce({
+                ok: true,
+                headers: new Headers({ 'content-type': 'application/json' }),
+                json: async () => ({}), // no .info field
+            } as any);
+            const iFrame: any = document.createElement('div');
+            jest.spyOn(baseInstance, 'getAuthPromise').mockResolvedValueOnce(true as any);
+            const tsEmbed = new SearchEmbed(getRootEl(), {});
+            iFrame.contentWindow = { postMessage: jest.fn() };
+            jest.spyOn(iFrame, 'addEventListener').mockImplementationOnce(
+                (event, handler) => { (handler as EventListener)({} as Event); },
+            );
+            jest.spyOn(document, 'createElement').mockReturnValueOnce(iFrame);
+            await tsEmbed.render();
+
+            await executeAfterWait(() => {
+                const infoSuccessCall = mockProcessTrigger.mock.calls.find(
+                    (call) => call[1] === HostEvent.InfoSuccess,
+                );
+                expect(infoSuccessCall).toBeUndefined();
+            });
+        });
     });
 
     describe('Preauth Cache for FullAppEmbed with PrimaryNavBar', () => {
@@ -2281,6 +2315,46 @@ describe('Unit test case for ts embed', () => {
             });
         });
 
+        it('Sets the overrideHistoryState param', async () => {
+            const appEmbed = new AppEmbed(getRootEl(), {
+                frameParams: {
+                    width: '100%',
+                    height: '100%',
+                },
+                overrideHistoryState: true,
+            });
+            await appEmbed.render();
+            expectUrlToHaveParamsWithValues(getIFrameSrc(), {
+                overrideHistoryState: true,
+            });
+        });
+
+        it('Sets the overrideHistoryState param to false', async () => {
+            const appEmbed = new AppEmbed(getRootEl(), {
+                frameParams: {
+                    width: '100%',
+                    height: '100%',
+                },
+                overrideHistoryState: false,
+            });
+            await appEmbed.render();
+            expectUrlToHaveParamsWithValues(getIFrameSrc(), {
+                overrideHistoryState: false,
+            });
+        });
+
+        it('Should not add overrideHistoryState param when it is undefined', async () => {
+            const appEmbed = new AppEmbed(getRootEl(), {
+                frameParams: {
+                    width: '100%',
+                    height: '100%',
+                },
+            });
+            await appEmbed.render();
+            const url = getIFrameSrc();
+            expect(url).not.toContain('overrideHistoryState');
+        });
+
         it('Should not add contextMenuEnabledOnWhichClick flag to the iframe src when it is not passed', async () => {
             const liveboardEmbed = new LiveboardEmbed(getRootEl(), {
                 ...defaultViewConfig,
@@ -4570,6 +4644,28 @@ describe('Trigger method edge cases', () => {
             expect(result).toBeNull();
         });
     });
+
+    test('isValidationError catch: handleError called with embedErrorDetails and error is rethrown', async () => {
+        const searchEmbed = new SearchEmbed(getRootEl(), defaultViewConfig);
+        const handleErrorSpy = jest.spyOn(searchEmbed as any, 'handleError').mockImplementation(() => {});
+        await searchEmbed.render();
+
+        // First processTrigger call is getAvailableUIPassthroughKeys for DrillDown
+        mockProcessTrigger.mockResolvedValueOnce([
+            { value: { keys: Object.values(UIPassthroughEvent) } },
+        ]);
+
+        // DrillDown with no points → throwDrillDownValidationError → isValidationError=true
+        await expect(
+            searchEmbed.trigger(HostEvent.DrillDown, {} as any),
+        ).rejects.toThrow('DrillDown requires a valid points object');
+
+        expect(handleErrorSpy).toHaveBeenCalledWith(
+            expect.objectContaining({
+                data: expect.objectContaining({ code: EmbedErrorCodes.HOST_EVENT_VALIDATION }),
+            }),
+        );
+    });
 });
 
 describe('PreRender replaceExistingPreRender scenarios', () => {
@@ -4580,6 +4676,10 @@ describe('PreRender replaceExistingPreRender scenarios', () => {
         });
     });
 
+    beforeEach(() => {
+        document.body.innerHTML = getDocumentBody();
+    });
+
     afterEach(() => {
         const rootEle = document.getElementById('myRoot');
         rootEle?.remove();
@@ -4725,6 +4825,52 @@ describe('Fullscreen change handler behavior', () => {
             );
         });
     });
+
+    test('disableFullscreenPresentation undefined defaults to true — no fullscreenchange listener registered', () => {
+        init({
+            thoughtSpotHost: 'tshost',
+            authType: AuthType.None,
+            // disableFullscreenPresentation not set → defaults to true via ?? true
+        });
+
+        const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+        const addEventListenerSpy = jest.spyOn(document, 'addEventListener');
+
+        embed['setupFullscreenChangeHandler']();
+
+        expect(addEventListenerSpy).not.toHaveBeenCalledWith('fullscreenchange', expect.any(Function));
+
+        addEventListenerSpy.mockRestore();
+        // Restore for other tests in this describe
+        init({ thoughtSpotHost: 'tshost', authType: AuthType.None, disableFullscreenPresentation: false });
+    });
+
+    test('fullscreenchange fires but iFrame.contentWindow is null — trigger NOT called, debug log emitted', () => {
+        const embed = new SearchEmbed(getRootEl(), defaultViewConfig);
+        // Give the embed a fake iFrame with null contentWindow
+        (embed as any).iFrame = { contentWindow: null };
+
+        const addEventListenerSpy = jest.spyOn(document, 'addEventListener');
+        const loggerDebugSpy = jest.spyOn(logger, 'debug').mockImplementation(() => {});
+        const triggerSpy = jest.spyOn(embed, 'trigger' as any).mockImplementation(() => Promise.resolve(null));
+
+        embed['setupFullscreenChangeHandler']();
+
+        // Capture the registered handler directly instead of dispatching to document
+        const fullscreenCall = addEventListenerSpy.mock.calls.find((c) => c[0] === 'fullscreenchange');
+        expect(fullscreenCall).toBeTruthy();
+        const handler = fullscreenCall[1] as EventListener;
+
+        Object.defineProperty(document, 'fullscreenElement', { value: null, writable: true, configurable: true });
+        handler(new Event('fullscreenchange'));
+
+        expect(triggerSpy).not.toHaveBeenCalled();
+        expect(loggerDebugSpy).toHaveBeenCalledWith(expect.stringContaining('Skipping ExitPresentMode'));
+
+        addEventListenerSpy.mockRestore();
+        loggerDebugSpy.mockRestore();
+        triggerSpy.mockRestore();
+    });
 });
 
 describe('ShowPreRender with UpdateEmbedParams', () => {
@@ -5315,5 +5461,45 @@ describe('ShowPreRender with UpdateEmbedParams', () => {
 
             embed.destroy();
         });
+
+        // showPreRender located the stale placeholder with a subtree-wide
+        // querySelector but removed it with hostElement.removeChild(), which
+        // only accepts direct children. With fullHeight the placeholder can be
+        // nested deeper than a direct child, so removeChild threw
+        // "NotFoundError: Failed to execute 'removeChild' on 'Node'".
+        it('showPreRender removes a placeholder nested below a direct child without throwing', async () => {
+            createRootEleForEmbed();
+            const embed = new LiveboardEmbed('#tsEmbedDiv', {
+                preRenderId: 'ph-nested-test',
+                liveboardId: 'lb-id',
+                fullHeight: true,
+            } as any);
+            await embed.preRender();
+            await waitFor(
+                () => !!document.querySelector('#tsEmbed-pre-render-child-ph-nested-test'),
+            );
+
+            const hostElement = document.querySelector('#tsEmbedDiv') as HTMLElement;
+            const placeHolderId = 'tsEmbed-pre-render-placeholder-ph-nested-test';
+            // Simulate the fullHeight layout where the placeholder ends up as
+            // a grandchild of the host element rather than a direct child.
+            const intermediateWrapper = document.createElement('div');
+            const staleNestedPlaceholder = document.createElement('div');
+            staleNestedPlaceholder.id = placeHolderId;
+            intermediateWrapper.appendChild(staleNestedPlaceholder);
+            hostElement.appendChild(intermediateWrapper);
+
+            expect(hostElement.querySelector(`#${placeHolderId}`)).toBe(staleNestedPlaceholder);
+            // Before the fix this threw NotFoundError.
+            await expect(embed.showPreRender()).resolves.toBeDefined();
+            // The stale nested placeholder is gone; the fresh one is a direct child.
+            expect(staleNestedPlaceholder.isConnected).toBe(false);
+            const currentPlaceholder = (
+                embed as any
+            ).getPreRenderPlaceHolderElement() as HTMLDivElement;
+            expect(currentPlaceholder.parentElement).toBe(hostElement);
+
+            embed.destroy();
+        });
     });
 });