refactor: 💡 move DynamicActionManager to x-pack

src/plugins/embeddable/public/lib/embeddables/embeddable.tsx

@@ -22,12 +22,7 @@ import { Adapters, ViewMode } from '../types';
 import { IContainer } from '../containers';
 import { EmbeddableInput, EmbeddableOutput, IEmbeddable } from './i_embeddable';
 import { TriggerContextMapping } from '../ui_actions';
-import { EmbeddableActionStorage } from './embeddable_action_storage';
-import {
-  UiActionsDynamicActionManager,
-  UiActionsStart,
-} from '../../../../../plugins/ui_actions/public';
-import { EmbeddableContext } from '../triggers';
+import { UiActionsStart } from '../../../../../plugins/ui_actions/public';
 
 function getPanelTitle(input: EmbeddableInput, output: EmbeddableOutput) {
   return input.hidePanelTitles ? '' : input.title === undefined ? output.defaultTitle : input.title;
@@ -60,28 +55,9 @@ export abstract class Embeddable<
   // to update input when the parent changes.
   private parentSubscription?: Rx.Subscription;
 
-  private storageSubscription?: Rx.Subscription;
-
   // TODO: Rename to destroyed.
   private destoyed: boolean = false;
 
-  private storage = new EmbeddableActionStorage((this as unknown) as Embeddable);
-
-  private cachedDynamicActions?: UiActionsDynamicActionManager;
-  public get dynamicActions(): UiActionsDynamicActionManager | undefined {
-    if (!this.params.uiActions) return undefined;
-    if (!this.cachedDynamicActions) {
-      this.cachedDynamicActions = new UiActionsDynamicActionManager({
-        isCompatible: async (context: unknown) =>
-          (context as EmbeddableContext).embeddable.runtimeId === this.runtimeId,
-        storage: this.storage,
-        uiActions: this.params.uiActions,
-      });
-    }
-
-    return this.cachedDynamicActions;
-  }
-
   constructor(
     input: TEmbeddableInput,
     output: TEmbeddableOutput,
@@ -111,18 +87,6 @@ export abstract class Embeddable<
         this.onResetInput(newInput);
       });
     }
-
-    if (this.dynamicActions) {
-      this.dynamicActions.start().catch(error => {
-        /* eslint-disable */
-        console.log('Failed to start embeddable dynamic actions', this);
-        console.error(error);
-        /* eslint-enable */
-      });
-      this.storageSubscription = this.input$.subscribe(() => {
-        this.storage.reload$.next();
-      });
-    }
   }
 
   public getIsContainer(): this is IContainer {
@@ -202,19 +166,6 @@ export abstract class Embeddable<
   public destroy(): void {
     this.destoyed = true;
 
-    if (this.dynamicActions) {
-      this.dynamicActions.stop().catch(error => {
-        /* eslint-disable */
-        console.log('Failed to stop embeddable dynamic actions', this);
-        console.error(error);
-        /* eslint-enable */
-      });
-    }
-
-    if (this.storageSubscription) {
-      this.storageSubscription.unsubscribe();
-    }
-
     this.input$.complete();
     this.output$.complete();
 

src/plugins/embeddable/public/lib/embeddables/i_embeddable.ts

@@ -18,7 +18,6 @@
  */
 
 import { Observable } from 'rxjs';
-import { UiActionsDynamicActionManager } from '../../../../../plugins/ui_actions/public';
 import { Adapters } from '../types';
 import { IContainer } from '../containers/i_container';
 import { ViewMode } from '../types';
@@ -96,11 +95,6 @@ export interface IEmbeddable<
    */
   enhancements?: object;
 
-  /**
-   * Default implementation of dynamic action API for embeddables.
-   */
-  dynamicActions?: UiActionsDynamicActionManager;
-
   /**
    * A functional representation of the isContainer variable, but helpful for typescript to
    * know the shape if this returns true

src/plugins/embeddable/public/lib/panel/embeddable_panel.tsx

@@ -203,13 +203,15 @@ export class EmbeddablePanel extends React.Component<Props, State> {
       this.props.embeddable.render(this.embeddableRoot.current);
     }
 
-    const dynamicActions = this.props.embeddable.dynamicActions;
+    const dynamicActions = (this.props.embeddable.enhancements as any)?.dynamicActions;
     if (dynamicActions) {
       this.setState({ eventCount: dynamicActions.state.get().events.length });
-      this.eventCountSubscription = dynamicActions.state.state$.subscribe(({ events }) => {
-        if (!this.mounted) return;
-        this.setState({ eventCount: events.length });
-      });
+      this.eventCountSubscription = dynamicActions.state.state$.subscribe(
+        ({ events }: { events: unknown[] }) => {
+          if (!this.mounted) return;
+          this.setState({ eventCount: events.length });
+        }
+      );
     }
   }
 

src/plugins/ui_actions/public/actions/index.ts

@@ -23,6 +23,4 @@ export * from './action_factory_definition';
 export * from './action_factory';
 export * from './create_action';
 export * from './incompatible_action_error';
-export * from './dynamic_action_storage';
-export * from './dynamic_action_manager';
 export * from './types';

src/plugins/ui_actions/public/actions/types.ts

@@ -22,3 +22,9 @@ export interface SerializedAction<Config> {
   readonly name: string;
   readonly config: Config;
 }
+
+export interface SerializedEvent {
+  eventId: string;
+  triggers: string[];
+  action: SerializedAction<unknown>;
+}

src/plugins/ui_actions/public/index.ts

@@ -31,11 +31,7 @@ export {
   ActionDefinition as UiActionsActionDefinition,
   ActionFactoryDefinition as UiActionsActionFactoryDefinition,
   ActionInternal as UiActionsActionInternal,
-  ActionStorage as UiActionsActionStorage,
-  AbstractActionStorage as UiActionsAbstractActionStorage,
   createAction,
-  DynamicActionManager,
-  DynamicActionManagerState,
   IncompatibleActionError,
   SerializedAction as UiActionsSerializedAction,
   SerializedEvent as UiActionsSerializedEvent,
@@ -57,4 +53,4 @@ export {
   applyFilterTrigger,
 } from './triggers';
 export { TriggerContextMapping, TriggerId, ActionContextMapping, ActionType } from './types';
-export { ActionByType, DynamicActionManager as UiActionsDynamicActionManager } from './actions';
+export { ActionByType } from './actions';

src/plugins/ui_actions/public/actions/dynamic_action_manager.test.ts → x-pack/plugins/advanced_ui_actions/public/dynamic_actions/dynamic_action_manager.test.ts

@@ -1,29 +1,18 @@
 /*
- * Licensed to Elasticsearch B.V. under one or more contributor
- * license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright
- * ownership. Elasticsearch B.V. licenses this file to you under
- * the Apache License, Version 2.0 (the "License"); you may
- * not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
  */
 
 import { DynamicActionManager } from './dynamic_action_manager';
 import { ActionStorage, MemoryActionStorage, SerializedEvent } from './dynamic_action_storage';
-import { UiActionsService } from '../service';
-import { ActionFactoryDefinition } from './action_factory_definition';
-import { ActionRegistry } from '../types';
-import { SerializedAction } from './types';
-import { of } from '../../../kibana_utils';
+import {
+  UiActionsService,
+  UiActionsActionFactoryDefinition as ActionFactoryDefinition,
+  UiActionsSerializedAction as SerializedAction,
+  UiActionsActionInternal as ActionInternal,
+} from '../../../../../src/plugins/ui_actions/public';
+import { of } from '../../../../../src/plugins/kibana_utils';
 
 const actionFactoryDefinition1: ActionFactoryDefinition = {
   id: 'ACTION_FACTORY_1',
@@ -82,7 +71,7 @@ const event3: SerializedEvent = {
 const setup = (events: readonly SerializedEvent[] = []) => {
   const isCompatible = async () => true;
   const storage: ActionStorage = new MemoryActionStorage(events);
-  const actions: ActionRegistry = new Map();
+  const actions = new Map<string, ActionInternal>();
   const uiActions = new UiActionsService({
     actions,
   });

src/plugins/ui_actions/public/actions/dynamic_action_manager.ts → x-pack/plugins/advanced_ui_actions/public/dynamic_actions/dynamic_action_manager.ts

@@ -1,31 +1,20 @@
 /*
- * Licensed to Elasticsearch B.V. under one or more contributor
- * license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright
- * ownership. Elasticsearch B.V. licenses this file to you under
- * the Apache License, Version 2.0 (the "License"); you may
- * not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
  */
 
 import { v4 as uuidv4 } from 'uuid';
 import { Subscription } from 'rxjs';
 import { ActionStorage, SerializedEvent } from './dynamic_action_storage';
-import { UiActionsService } from '../service';
-import { SerializedAction } from './types';
-import { TriggerContextMapping } from '../types';
-import { ActionDefinition } from './action';
+import {
+  UiActionsService,
+  UiActionsSerializedAction as SerializedAction,
+  TriggerContextMapping,
+  UiActionsActionDefinition as ActionDefinition,
+} from '../../../../../src/plugins/ui_actions/public';
 import { defaultState, transitions, selectors, State } from './dynamic_action_manager_state';
-import { StateContainer, createStateContainer } from '../../../kibana_utils';
+import { StateContainer, createStateContainer } from '../../../../../src/plugins/kibana_utils';
 
 const compareEvents = (
   a: ReadonlyArray<{ eventId: string }>,

src/plugins/ui_actions/public/actions/dynamic_action_manager_state.ts → x-pack/plugins/advanced_ui_actions/public/dynamic_actions/dynamic_action_manager_state.ts

@@ -1,20 +1,7 @@
 /*
- * Licensed to Elasticsearch B.V. under one or more contributor
- * license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright
- * ownership. Elasticsearch B.V. licenses this file to you under
- * the Apache License, Version 2.0 (the "License"); you may
- * not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
  */
 
 import { SerializedEvent } from './dynamic_action_storage';

src/plugins/ui_actions/public/actions/dynamic_action_storage.ts → x-pack/plugins/advanced_ui_actions/public/dynamic_actions/dynamic_action_storage.ts

@@ -1,26 +1,13 @@
 /*
- * Licensed to Elasticsearch B.V. under one or more contributor
- * license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright
- * ownership. Elasticsearch B.V. licenses this file to you under
- * the Apache License, Version 2.0 (the "License"); you may
- * not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
  */
 
 /* eslint-disable max-classes-per-file */
 
 import { Observable, Subject } from 'rxjs';
-import { SerializedAction } from './types';
+import { UiActionsSerializedAction as SerializedAction } from '../../../../../src/plugins/ui_actions/public';
 
 /**
  * Serialized representation of event-action pair, used to persist in storage.

x-pack/plugins/advanced_ui_actions/public/dynamic_actions/index.ts

@@ -0,0 +1,9 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
+ */
+
+export * from './dynamic_action_storage';
+export * from './dynamic_action_manager_state';
+export * from './dynamic_action_manager';

x-pack/plugins/advanced_ui_actions/public/index.ts

@@ -26,3 +26,10 @@ export {
   Configurable as AdvancedUiActionsConfigurable,
   CollectConfigProps as AdvancedUiActionsCollectConfigProps,
 } from './util';
+
+export {
+  AbstractActionStorage as UiActionsEnhancedAbstractActionStorage,
+  DynamicActionManager as UiActionsEnhancedDynamicActionManager,
+  DynamicActionManagerParams as UiActionsEnhancedDynamicActionManagerParams,
+  DynamicActionManagerState as UiActionsEnhancedDynamicActionManagerState,
+} from './dynamic_actions';

x-pack/plugins/dashboard_enhanced/public/services/drilldowns/actions/flyout_create_drilldown/flyout_create_drilldown.tsx

@@ -10,6 +10,7 @@ import { CoreStart } from 'src/core/public';
 import { ActionByType } from '../../../../../../../../src/plugins/ui_actions/public';
 import { toMountPoint } from '../../../../../../../../src/plugins/kibana_react/public';
 import { DrilldownsStart } from '../../../../../../drilldowns/public';
+import { isEnhancedEmbeddable } from '../../../../../../embeddable_enhanced/public';
 import { EmbeddableContext } from '../../../../../../../../src/plugins/embeddable/public';
 
 export const OPEN_FLYOUT_ADD_DRILLDOWN = 'OPEN_FLYOUT_ADD_DRILLDOWN';
@@ -37,7 +38,7 @@ export class FlyoutCreateDrilldownAction implements ActionByType<typeof OPEN_FLY
   }
 
   private isEmbeddableCompatible(context: EmbeddableContext) {
-    if (!context.embeddable.dynamicActions) return false;
+    if (!isEnhancedEmbeddable(context.embeddable)) return false;
     const supportedTriggers = context.embeddable.supportedTriggers();
     if (!supportedTriggers || !supportedTriggers.length) return false;
     return supportedTriggers.indexOf('VALUE_CLICK_TRIGGER') > -1;
@@ -51,10 +52,12 @@ export class FlyoutCreateDrilldownAction implements ActionByType<typeof OPEN_FLY
   public async execute(context: EmbeddableContext) {
     const overlays = await this.params.overlays();
     const drilldowns = await this.params.drilldowns();
-    const dynamicActionManager = context.embeddable.dynamicActions;
+    const { embeddable } = context;
 
-    if (!dynamicActionManager) {
-      throw new Error(`Can't execute FlyoutCreateDrilldownAction without dynamicActionsManager`);
+    if (!isEnhancedEmbeddable(embeddable)) {
+      throw new Error(
+        'Need embeddable to be EnhancedEmbeddable to execute FlyoutCreateDrilldownAction.'
+      );
     }
 
     const handle = overlays.openFlyout(
@@ -63,7 +66,7 @@ export class FlyoutCreateDrilldownAction implements ActionByType<typeof OPEN_FLY
           onClose={() => handle.close()}
           placeContext={context}
           viewMode={'create'}
-          dynamicActionManager={dynamicActionManager}
+          dynamicActionManager={embeddable.enhancements.dynamicActions}
         />
       ),
       {