Make SavedObjectFinder backward compatible

examples/content_management_examples/kibana.jsonc

@@ -12,6 +12,7 @@
       "developerExamples",
       "kibanaReact",
       "savedObjectsTaggingOss"
-    ]
+    ],
+    "requiredBundles": ["savedObjectsFinder"]
   }
 }

examples/content_management_examples/public/examples/finder/finder_app.tsx

@@ -0,0 +1,49 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0 and the Server Side Public License, v 1; you may not use this file except
+ * in compliance with, at your election, the Elastic License 2.0 or the Server
+ * Side Public License, v 1.
+ */
+
+import React from 'react';
+import { ContentClientProvider, type ContentClient } from '@kbn/content-management-plugin/public';
+import type { CoreStart } from '@kbn/core/public';
+import { I18nProvider } from '@kbn/i18n-react';
+import { SavedObjectTaggingOssPluginStart } from '@kbn/saved-objects-tagging-oss-plugin/public';
+import { SavedObjectFinder } from '@kbn/saved-objects-finder-plugin/public';
+
+export const FinderApp = (props: {
+  contentClient: ContentClient;
+  core: CoreStart;
+  savedObjectsTagging: SavedObjectTaggingOssPluginStart;
+}) => {
+  return (
+    <ContentClientProvider contentClient={props.contentClient}>
+      <I18nProvider>
+        <SavedObjectFinder
+          showFilter={true}
+          services={{
+            savedObjectsTagging: props.savedObjectsTagging.getTaggingApi(),
+            contentClient: props.contentClient,
+          }}
+          onChoose={(...args) => {
+            alert(JSON.stringify(args));
+          }}
+          savedObjectMetaData={[
+            {
+              type: `search`,
+              getIconForSavedObject: () => 'discoverApp',
+              name: 'Saved search',
+            },
+            {
+              type: 'index-pattern',
+              getIconForSavedObject: () => 'indexPatternApp',
+              name: 'Data view',
+            },
+          ]}
+        />
+      </I18nProvider>
+    </ContentClientProvider>
+  );
+};

src/plugins/saved_objects_finder/server/plugin.test.mocks.ts → examples/content_management_examples/public/examples/finder/index.tsx

@@ -6,7 +6,4 @@
  * Side Public License, v 1.
  */
 
-export const registerRoutesMock = jest.fn();
-jest.doMock('./routes', () => ({
-  registerRoutes: registerRoutesMock,
-}));
+export { FinderApp } from './finder_app';

examples/content_management_examples/public/examples/index.tsx

@@ -16,6 +16,7 @@ import { AppMountParameters, CoreStart } from '@kbn/core/public';
 import { StartDeps } from '../types';
 import { TodoApp } from './todos';
 import { MSearchApp } from './msearch';
+import { FinderApp } from './finder';
 
 export const renderApp = (
   core: CoreStart,
@@ -45,6 +46,12 @@ export const renderApp = (
                       'data-test-subj': 'msearchExample',
                       href: '/app/contentManagementExamples/msearch',
                     },
+                    {
+                      id: 'finder',
+                      name: 'Finder',
+                      'data-test-subj': 'finderExample',
+                      href: '/app/contentManagementExamples/finder',
+                    },
                   ],
                 },
               ]}
@@ -64,6 +71,13 @@ export const renderApp = (
                   savedObjectsTagging={savedObjectsTaggingOss}
                 />
               </Route>
+              <Route path="/finder">
+                <FinderApp
+                  contentClient={contentManagement.client}
+                  core={core}
+                  savedObjectsTagging={savedObjectsTaggingOss}
+                />
+              </Route>
             </Routes>
           </EuiPageTemplate.Section>
         </EuiPageTemplate>

examples/content_management_examples/tsconfig.json

@@ -29,5 +29,6 @@
     "@kbn/content-management-table-list-view-table",
     "@kbn/content-management-table-list-view",
     "@kbn/shared-ux-router",
+    "@kbn/saved-objects-finder-plugin",
   ]
 }

packages/kbn-content-management-utils/src/saved_object_content_storage.ts

@@ -135,6 +135,7 @@ export interface SOContentStorageConstrutorParams<Types extends CMCrudTypes> {
   updateArgsToSoUpdateOptions?: UpdateArgsToSoUpdateOptions<Types>;
   searchArgsToSOFindOptions?: SearchArgsToSOFindOptions<Types>;
   enableMSearch?: boolean;
+  mSearchAdditionalSearchFields?: string[];
 }
 
 export abstract class SOContentStorage<Types extends CMCrudTypes>
@@ -153,6 +154,7 @@ export abstract class SOContentStorage<Types extends CMCrudTypes>
     searchArgsToSOFindOptions,
     enableMSearch,
     allowedSavedObjectAttributes,
+    mSearchAdditionalSearchFields,
   }: SOContentStorageConstrutorParams<Types>) {
     this.savedObjectType = savedObjectType;
     this.cmServicesDefinition = cmServicesDefinition;
@@ -166,6 +168,7 @@ export abstract class SOContentStorage<Types extends CMCrudTypes>
     if (enableMSearch) {
       this.mSearch = {
         savedObjectType: this.savedObjectType,
+        additionalSearchFields: mSearchAdditionalSearchFields,
         toItemResult: (ctx: StorageContext, savedObject: SavedObjectsFindResult): Types['Item'] => {
           const transforms = ctx.utils.getTransforms(this.cmServicesDefinition);
 
@@ -201,6 +204,7 @@ export abstract class SOContentStorage<Types extends CMCrudTypes>
   mSearch?: {
     savedObjectType: string;
     toItemResult: (ctx: StorageContext, savedObject: SavedObjectsFindResult) => Types['Item'];
+    additionalSearchFields?: string[];
   };
 
   async get(ctx: StorageContext, id: string): Promise<Types['GetOut']> {

packages/kbn-event-annotation-components/components/event_annotation_group_saved_object_finder.tsx

@@ -8,12 +8,10 @@
 
 import React, { useEffect, useState } from 'react';
 import { i18n } from '@kbn/i18n';
-import type { CoreStart } from '@kbn/core/public';
 import { FormattedMessage } from '@kbn/i18n-react';
-import type { IUiSettingsClient } from '@kbn/core-ui-settings-browser';
 import type { SavedObjectCommon } from '@kbn/saved-objects-finder-plugin/common';
 import { SavedObjectFinder } from '@kbn/saved-objects-finder-plugin/public';
-import type { SavedObjectsManagementPluginStart } from '@kbn/saved-objects-management-plugin/public';
+import type { ContentClient } from '@kbn/content-management-plugin/public';
 import {
   EuiButton,
   EuiEmptyPrompt,
@@ -26,17 +24,13 @@ import { css } from '@emotion/react';
 import { EVENT_ANNOTATION_GROUP_TYPE } from '@kbn/event-annotation-common';
 
 export const EventAnnotationGroupSavedObjectFinder = ({
-  uiSettings,
-  http,
-  savedObjectsManagement,
   fixedPageSize = 10,
   checkHasAnnotationGroups,
   onChoose,
   onCreateNew,
+  contentClient,
 }: {
-  uiSettings: IUiSettingsClient;
-  http: CoreStart['http'];
-  savedObjectsManagement: SavedObjectsManagementPluginStart;
+  contentClient: ContentClient;
   fixedPageSize?: number;
   checkHasAnnotationGroups: () => Promise<boolean>;
   onChoose: (value: {
@@ -115,9 +109,7 @@ export const EventAnnotationGroupSavedObjectFinder = ({
       }
       savedObjectMetaData={savedObjectMetaData}
       services={{
-        uiSettings,
-        http,
-        savedObjectsManagement,
+        contentClient,
       }}
     />
   );

packages/kbn-event-annotation-components/tsconfig.json

@@ -29,13 +29,12 @@
     "@kbn/content-management-table-list-view-table",
     "@kbn/content-management-tabbed-table-list-view",
     "@kbn/event-annotation-common",
-    "@kbn/core",
     "@kbn/i18n-react",
     "@kbn/saved-objects-finder-plugin",
-    "@kbn/saved-objects-management-plugin",
     "@kbn/core-notifications-browser-mocks",
     "@kbn/core-notifications-browser",
     "@kbn/core-saved-objects-api-browser",
     "@kbn/expressions-plugin",
+    "@kbn/content-management-plugin",
   ]
 }

src/plugins/content_management/public/mocks.ts

@@ -23,6 +23,7 @@ const createStartContract = (): ContentManagementPublicStart => {
       update: jest.fn(),
       delete: jest.fn(),
       search: jest.fn(),
+      mSearch: jest.fn(),
     } as unknown as ContentManagementPublicStart['client'],
     registry: {
       get: jest.fn(),

src/plugins/dashboard/public/dashboard_actions/index.ts

@@ -32,18 +32,14 @@ export const buildAllDashboardActions = async ({
   plugins,
   allowByValueEmbeddables,
 }: BuildAllDashboardActionsProps) => {
-  const { uiSettings } = core;
-  const { uiActions, share, presentationUtil, savedObjectsManagement, savedObjectsTaggingOss } =
-    plugins;
+  const { uiActions, share, presentationUtil, savedObjectsTaggingOss, contentManagement } = plugins;
 
   const clonePanelAction = new ClonePanelAction(core.savedObjects);
   uiActions.registerAction(clonePanelAction);
   uiActions.attachAction(CONTEXT_MENU_TRIGGER, clonePanelAction.id);
 
   const SavedObjectFinder = getSavedObjectFinder(
-    uiSettings,
-    core.http,
-    savedObjectsManagement,
+    contentManagement.client,
     savedObjectsTaggingOss?.getTaggingApi()
   );
   const changeViewAction = new ReplacePanelAction(SavedObjectFinder);

src/plugins/dashboard/public/services/content_management/content_management_service.stub.ts

@@ -0,0 +1,17 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0 and the Server Side Public License, v 1; you may not use this file except
+ * in compliance with, at your election, the Elastic License 2.0 or the Server
+ * Side Public License, v 1.
+ */
+
+import { PluginServiceFactory } from '@kbn/presentation-util-plugin/public';
+import { ContentManagementPublicStart } from '@kbn/content-management-plugin/public';
+import { contentManagementMock } from '@kbn/content-management-plugin/public/mocks';
+
+export type ContentManagementServiceFactory = PluginServiceFactory<ContentManagementPublicStart>;
+
+export const contentManagementServiceFactory: ContentManagementServiceFactory = () => {
+  return contentManagementMock.createStartContract();
+};

src/plugins/dashboard/public/services/content_management/content_management_service.ts

@@ -0,0 +1,24 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0 and the Server Side Public License, v 1; you may not use this file except
+ * in compliance with, at your election, the Elastic License 2.0 or the Server
+ * Side Public License, v 1.
+ */
+
+import { KibanaPluginServiceFactory } from '@kbn/presentation-util-plugin/public';
+import { ContentManagementPublicStart } from '@kbn/content-management-plugin/public';
+import { DashboardStartDependencies } from '../../plugin';
+
+export type ContentManagementServiceFactory = KibanaPluginServiceFactory<
+  ContentManagementPublicStart,
+  DashboardStartDependencies
+>;
+
+export const contentManagementServiceFactory: ContentManagementServiceFactory = ({
+  startPlugins,
+}) => {
+  const { contentManagement } = startPlugins;
+
+  return contentManagement;
+};

src/plugins/dashboard/public/services/plugin_services.stub.ts

@@ -40,6 +40,7 @@ import { visualizationsServiceFactory } from './visualizations/visualizations.st
 import { dashboardContentManagementServiceFactory } from './dashboard_content_management/dashboard_content_management.stub';
 import { customBrandingServiceFactory } from './custom_branding/custom_branding.stub';
 import { savedObjectsManagementServiceFactory } from './saved_objects_management/saved_objects_management_service.stub';
+import { contentManagementServiceFactory } from './content_management/content_management_service.stub';
 
 export const providers: PluginServiceProviders<DashboardServices> = {
   dashboardContentManagement: new PluginServiceProvider(dashboardContentManagementServiceFactory),
@@ -68,6 +69,7 @@ export const providers: PluginServiceProviders<DashboardServices> = {
   visualizations: new PluginServiceProvider(visualizationsServiceFactory),
   customBranding: new PluginServiceProvider(customBrandingServiceFactory),
   savedObjectsManagement: new PluginServiceProvider(savedObjectsManagementServiceFactory),
+  contentManagement: new PluginServiceProvider(contentManagementServiceFactory),
 };
 
 export const registry = new PluginServiceRegistry<DashboardServices>(providers);

src/plugins/dashboard/public/services/plugin_services.ts

@@ -41,6 +41,7 @@ import { analyticsServiceFactory } from './analytics/analytics_service';
 import { customBrandingServiceFactory } from './custom_branding/custom_branding_service';
 import { savedObjectsManagementServiceFactory } from './saved_objects_management/saved_objects_management_service';
 import { dashboardContentManagementServiceFactory } from './dashboard_content_management/dashboard_content_management_service';
+import { contentManagementServiceFactory } from './content_management/content_management_service';
 
 const providers: PluginServiceProviders<DashboardServices, DashboardPluginServiceParams> = {
   dashboardContentManagement: new PluginServiceProvider(dashboardContentManagementServiceFactory, [
@@ -82,6 +83,7 @@ const providers: PluginServiceProviders<DashboardServices, DashboardPluginServic
   visualizations: new PluginServiceProvider(visualizationsServiceFactory),
   customBranding: new PluginServiceProvider(customBrandingServiceFactory),
   savedObjectsManagement: new PluginServiceProvider(savedObjectsManagementServiceFactory),
+  contentManagement: new PluginServiceProvider(contentManagementServiceFactory),
 };
 
 export const pluginServices = new PluginServices<DashboardServices>();

src/plugins/dashboard/public/services/types.ts

@@ -9,6 +9,7 @@
 import { PluginInitializerContext } from '@kbn/core/public';
 import { KibanaPluginServiceParams } from '@kbn/presentation-util-plugin/public';
 import { SavedObjectsManagementPluginStart } from '@kbn/saved-objects-management-plugin/public';
+import { ContentManagementPublicStart } from '@kbn/content-management-plugin/public';
 
 import { DashboardStartDependencies } from '../plugin';
 import { DashboardAnalyticsService } from './analytics/types';
@@ -68,4 +69,5 @@ export interface DashboardServices {
   visualizations: DashboardVisualizationsService;
   customBranding: DashboardCustomBrandingService;
   savedObjectsManagement: SavedObjectsManagementPluginStart;
+  contentManagement: ContentManagementPublicStart;
 }

src/plugins/data_views/server/content_management/data_views_storage.ts

@@ -31,6 +31,7 @@ export class DataViewsStorage extends SOContentStorage<DataViewCrudTypes> {
         'allowNoIndex',
         'name',
       ],
+      mSearchAdditionalSearchFields: ['name'],
     });
   }
 }

src/plugins/discover/kibana.jsonc

@@ -25,7 +25,8 @@
       "dataViewEditor",
       "expressions",
       "unifiedSearch",
-      "unifiedHistogram"
+      "unifiedHistogram",
+      "contentManagement"
     ],
     "optionalPlugins": [
       "home",
@@ -36,13 +37,7 @@
       "savedObjectsTaggingOss",
       "lens"
     ],
-    "requiredBundles": [
-      "kibanaUtils",
-      "kibanaReact",
-      "unifiedSearch"
-    ],
-    "extraPublicDirs": [
-      "common"
-    ]
+    "requiredBundles": ["kibanaUtils", "kibanaReact", "unifiedSearch"],
+    "extraPublicDirs": ["common"]
   }
 }

src/plugins/discover/public/application/main/components/top_nav/open_search_panel.tsx

@@ -30,14 +30,7 @@ interface OpenSearchPanelProps {
 }
 
 export function OpenSearchPanel(props: OpenSearchPanelProps) {
-  const {
-    addBasePath,
-    capabilities,
-    core,
-    uiSettings,
-    savedObjectsManagement,
-    savedObjectsTagging,
-  } = useDiscoverServices();
+  const { addBasePath, capabilities, savedObjectsTagging, contentClient } = useDiscoverServices();
   const hasSavedObjectPermission =
     capabilities.savedObjectsManagement?.edit || capabilities.savedObjectsManagement?.delete;
 
@@ -56,10 +49,8 @@ export function OpenSearchPanel(props: OpenSearchPanelProps) {
       <EuiFlyoutBody>
         <SavedObjectFinder
           services={{
-            http: core.http,
-            uiSettings,
-            savedObjectsManagement,
             savedObjectsTagging,
+            contentClient,
           }}
           noItemsMessage={
             <FormattedMessage