From 6fd6966ed8225935feee247b04b4988424b47c88 Mon Sep 17 00:00:00 2001
From: Jatin Kathuria <jatin.kathuria@elastic.co>
Date: Fri, 13 Oct 2023 08:22:06 +0200
Subject: [PATCH] [Security Solution] Fixes ES|QL Tab resetting to KQL Bar
 (#168716)

## Summary

Handles #168431

This PR fixes a race condition which was causing Discover ES|QL tab to
revert to it KQL query bar. See demo below:

| Before | After |
|---|---|
| <video
src="https://github.com/elastic/kibana/assets/7485038/24ff8ca9-30ce-4581-94d7-52908a2c81fd"
/>|<video
src="https://github.com/elastic/kibana/assets/7485038/746946ad-bd69-412f-bd38-d3d79b570c68"
/>|
---
 .../timelines/components/timeline/esql_tab_content/index.tsx  | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/x-pack/plugins/security_solution/public/timelines/components/timeline/esql_tab_content/index.tsx b/x-pack/plugins/security_solution/public/timelines/components/timeline/esql_tab_content/index.tsx
index 97737a8ebfc3b..b3697421906ef 100644
--- a/x-pack/plugins/security_solution/public/timelines/components/timeline/esql_tab_content/index.tsx
+++ b/x-pack/plugins/security_solution/public/timelines/components/timeline/esql_tab_content/index.tsx
@@ -181,6 +181,10 @@ export const DiscoverTabContent: FC<DiscoverTabContentProps> = ({ timelineId })
     if (!savedObjectId) return;
     if (!status || status === 'draft') return;
     const latestState = getCombinedDiscoverSavedSearchState();
+    const index = latestState?.searchSource.getField('index');
+    /* when a new timeline is loaded, a new discover instance is loaded which first emits
+     * discover's initial state which is then updated in the saved search. We want to avoid that.*/
+    if (!index) return;
     if (!latestState || combinedDiscoverSavedSearchStateRef.current === latestState) return;
     if (isEqualWith(latestState, savedSearchById, savedSearchComparator)) return;
     debouncedUpdateSavedSearch(latestState, timelineId);