[8.x] [Lens][ES|QL] Going to Discover without carrying the dashboard filters (#208041) (#208676)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Lens][ES|QL] Going to Discover without carrying the dashboard
filters (#208041)](#208041)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Stratoula
Kalafateli","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-29T07:00:22Z","message":"[Lens][ES|QL]
Going to Discover without carrying the dashboard filters
(#208041)","sha":"d97b6fa7a114192606b638b974e6b83d19922954","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Feature:Lens","v9.0.0","Feature:ES|QL","backport:version","v8.18.0"],"title":"[Lens][ES|QL]
Going to Discover without carrying the dashboard
filters","number":208041,"url":"https://github.com/elastic/kibana/pull/208041","mergeCommit":{"message":"[Lens][ES|QL]
Going to Discover without carrying the dashboard filters
(#208041)","sha":"d97b6fa7a114192606b638b974e6b83d19922954"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/208041","number":208041,"mergeCommit":{"message":"[Lens][ES|QL]
Going to Discover without carrying the dashboard filters
(#208041)","sha":"d97b6fa7a114192606b638b974e6b83d19922954"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Stratoula Kalafateli <[email protected]>

Author: kibanamachine

x-pack/platform/plugins/shared/lens/public/trigger_actions/open_in_discover_action.test.ts

@@ -135,4 +135,48 @@ describe('open in discover action', () => {
     expect(locator.getRedirectUrl).toHaveBeenCalledWith(viewUnderlyingDataArgs);
     expect(globalThis.open).toHaveBeenCalledWith(discoverUrl, '_blank');
   });
+
+  it('navigates to discover for an ES|QL chart but without the filters', async () => {
+    const viewUnderlyingDataArgs = {
+      dataViewSpec: { id: 'index-pattern-id' },
+      timeRange: {},
+      filters: [{ meta: { type: 'range' } }],
+      query: undefined,
+      columns: [],
+    };
+
+    const embeddable = {
+      ...compatibleEmbeddableApi,
+      getViewUnderlyingDataArgs: jest.fn(() => viewUnderlyingDataArgs),
+      isTextBasedLanguage: jest.fn(() => true),
+    };
+
+    const discoverUrl = 'https://discover-redirect-url';
+    const locator = {
+      getRedirectUrl: jest.fn(() => discoverUrl),
+    } as unknown as DiscoverAppLocator;
+
+    globalThis.open = jest.fn();
+
+    await createOpenInDiscoverAction(
+      locator,
+      {
+        get: () => ({
+          isTimeBased: () => true,
+          toSpec: () => ({ id: 'index-pattern-id' }),
+        }),
+      } as unknown as DataViewsService,
+      true
+    ).execute({
+      embeddable,
+    } as ActionExecutionContext<EmbeddableApiContext>);
+
+    expect(embeddable.getViewUnderlyingDataArgs).toHaveBeenCalled();
+    const viewUnderlyingDataArgsWithoutFilters = {
+      ...viewUnderlyingDataArgs,
+      filters: [],
+    };
+    expect(locator.getRedirectUrl).toHaveBeenCalledWith(viewUnderlyingDataArgsWithoutFilters);
+    expect(globalThis.open).toHaveBeenCalledWith(discoverUrl, '_blank');
+  });
 });

x-pack/platform/plugins/shared/lens/public/trigger_actions/open_in_discover_helpers.ts

@@ -59,7 +59,10 @@ async function getDiscoverLocationParams({
     throw new Error('Underlying data is not ready');
   }
   const dataView = await dataViews.get(args.dataViewSpec.id!);
-  let filtersToApply = [...(filters || []), ...args.filters];
+  // we don't want to pass the DSL filters when navigating from an ES|SQL embeddable
+  let filtersToApply = embeddable.isTextBasedLanguage()
+    ? []
+    : [...(filters || []), ...args.filters];
   let timeRangeToApply = args.timeRange;
   // if the target data view is time based, attempt to split out a time range from the provided filters
   if (dataView.isTimeBased() && dataView.timeFieldName === timeFieldName) {