Make the alerts plugin support generics

x-pack/plugins/alerting_builtins/server/alert_types/index_threshold/action_context.ts

@@ -6,7 +6,7 @@
 
 import { i18n } from '@kbn/i18n';
 import { Params } from './alert_type_params';
-import { AlertExecutorOptions } from '../../../../alerts/server';
+import { AlertExecutorOptions, AlertInstanceContext } from '../../../../alerts/server';
 
 // alert type context provided to actions
 
@@ -19,7 +19,7 @@ export interface ActionContext extends BaseActionContext {
   message: string;
 }
 
-export interface BaseActionContext {
+export interface BaseActionContext extends AlertInstanceContext {
   // the aggType used in the alert
   // the value of the aggField, if used, otherwise 'all documents'
   group: string;

x-pack/plugins/alerting_builtins/server/alert_types/index_threshold/alert_type.ts

@@ -7,7 +7,7 @@
 import { i18n } from '@kbn/i18n';
 import { AlertType, AlertExecutorOptions } from '../../types';
 import { Params, ParamsSchema } from './alert_type_params';
-import { BaseActionContext, addMessages } from './action_context';
+import { ActionContext, BaseActionContext, addMessages } from './action_context';
 import { TimeSeriesQuery } from './lib/time_series_query';
 import { Service } from '../../types';
 import { BUILT_IN_ALERTS_FEATURE_ID } from '../../../common';
@@ -18,7 +18,7 @@ const ActionGroupId = 'threshold met';
 const ComparatorFns = getComparatorFns();
 export const ComparatorFnNames = new Set(ComparatorFns.keys());
 
-export function getAlertType(service: Service): AlertType {
+export function getAlertType(service: Service): AlertType<Params, {}, {}, ActionContext> {
   const { logger } = service;
 
   const alertTypeName = i18n.translate('xpack.alertingBuiltins.indexThreshold.alertTypeTitle', {
@@ -88,9 +88,8 @@ export function getAlertType(service: Service): AlertType {
     producer: BUILT_IN_ALERTS_FEATURE_ID,
   };
 
-  async function executor(options: AlertExecutorOptions) {
-    const { alertId, name, services } = options;
-    const params: Params = options.params as Params;
+  async function executor(options: AlertExecutorOptions<Params, {}, {}, ActionContext>) {
+    const { alertId, name, services, params } = options;
 
     const compareFn = ComparatorFns.get(params.thresholdComparator);
     if (compareFn == null) {

x-pack/plugins/alerts/common/alert.ts

@@ -6,6 +6,11 @@
 
 import { SavedObjectAttributes } from 'kibana/server';
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+export type AlertTypeState = Record<string, any>;
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+export type AlertTypeParams = Record<string, any>;
+
 export interface IntervalSchedule extends SavedObjectAttributes {
   interval: string;
 }
@@ -28,9 +33,7 @@ export interface Alert {
   consumer: string;
   schedule: IntervalSchedule;
   actions: AlertAction[];
-  // This will have to remain `any` until we can extend Alert Executors with generics
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  params: Record<string, any>;
+  params: AlertTypeParams;
   scheduledTaskId?: string;
   createdBy: string | null;
   updatedBy: string | null;

x-pack/plugins/alerts/common/alert_instance.ts

@@ -17,6 +17,9 @@ export type AlertInstanceMeta = t.TypeOf<typeof metaSchema>;
 const stateSchema = t.record(t.string, t.unknown);
 export type AlertInstanceState = t.TypeOf<typeof stateSchema>;
 
+const contextSchema = t.record(t.string, t.unknown);
+export type AlertInstanceContext = t.TypeOf<typeof contextSchema>;
+
 export const rawAlertInstance = t.partial({
   state: stateSchema,
   meta: metaSchema,

x-pack/plugins/alerts/server/alert_instance/alert_instance.ts

@@ -8,24 +8,26 @@ import {
   AlertInstanceState,
   RawAlertInstance,
   rawAlertInstance,
+  AlertInstanceContext,
 } from '../../common';
 
-import { State, Context } from '../types';
 import { parseDuration } from '../lib';
 
-interface ScheduledExecutionOptions {
-  actionGroup: string;
-  context: Context;
-  state: State;
-}
 export type AlertInstances = Record<string, AlertInstance>;
-export class AlertInstance {
-  private scheduledExecutionOptions?: ScheduledExecutionOptions;
+export class AlertInstance<
+  State extends AlertInstanceState = AlertInstanceState,
+  Context extends AlertInstanceContext = AlertInstanceContext
+> {
+  private scheduledExecutionOptions?: {
+    actionGroup: string;
+    context: Context;
+    state: State;
+  };
   private meta: AlertInstanceMeta;
-  private state: AlertInstanceState;
+  private state: State;
 
-  constructor({ state = {}, meta = {} }: RawAlertInstance = {}) {
-    this.state = state;
+  constructor({ state, meta = {} }: RawAlertInstance = {}) {
+    this.state = (state || {}) as State;
     this.meta = meta;
   }
 
@@ -62,11 +64,15 @@ export class AlertInstance {
     return this.state;
   }
 
-  scheduleActions(actionGroup: string, context: Context = {}) {
+  scheduleActions(actionGroup: string, context?: Context) {
     if (this.hasScheduledActions()) {
       throw new Error('Alert instance execution has already been scheduled, cannot schedule twice');
     }
-    this.scheduledExecutionOptions = { actionGroup, context, state: this.state };
+    this.scheduledExecutionOptions = {
+      actionGroup,
+      context: (context || {}) as Context,
+      state: this.state,
+    };
     return this;
   }
 

x-pack/plugins/alerts/server/alert_type_registry.ts

@@ -10,7 +10,13 @@ import { schema } from '@kbn/config-schema';
 import typeDetect from 'type-detect';
 import { RunContext, TaskManagerSetupContract } from '../../task_manager/server';
 import { TaskRunnerFactory } from './task_runner';
-import { AlertType } from './types';
+import {
+  AlertType,
+  AlertTypeParams,
+  AlertTypeState,
+  AlertInstanceState,
+  AlertInstanceContext,
+} from './types';
 
 interface ConstructorOptions {
   taskManager: TaskManagerSetupContract;
@@ -59,7 +65,12 @@ export class AlertTypeRegistry {
     return this.alertTypes.has(id);
   }
 
-  public register(alertType: AlertType) {
+  public register<
+    Params extends AlertTypeParams = AlertTypeParams,
+    State extends AlertTypeState = AlertTypeState,
+    AlertInstanceStateType extends AlertInstanceState = AlertInstanceState,
+    AlertInstanceContextType extends AlertInstanceContext = AlertInstanceContext
+  >(alertType: AlertType<Params, State, AlertInstanceStateType, AlertInstanceContextType>) {
     if (this.has(alertType.id)) {
       throw new Error(
         i18n.translate('xpack.alerts.alertTypeRegistry.register.duplicateAlertTypeError', {
@@ -71,18 +82,23 @@ export class AlertTypeRegistry {
       );
     }
     alertType.actionVariables = normalizedActionVariables(alertType.actionVariables);
-    this.alertTypes.set(alertIdSchema.validate(alertType.id), { ...alertType });
+    this.alertTypes.set(alertIdSchema.validate(alertType.id), { ...alertType } as AlertType);
     this.taskManager.registerTaskDefinitions({
       [`alerting:${alertType.id}`]: {
         title: alertType.name,
         type: `alerting:${alertType.id}`,
         createTaskRunner: (context: RunContext) =>
-          this.taskRunnerFactory.create(alertType, context),
+          this.taskRunnerFactory.create({ ...alertType } as AlertType, context),
       },
     });
   }
 
-  public get(id: string): AlertType {
+  public get<
+    Params extends AlertTypeParams = AlertTypeParams,
+    State extends AlertTypeState = AlertTypeState,
+    AlertInstanceStateType extends AlertInstanceState = AlertInstanceState,
+    AlertInstanceContextType extends AlertInstanceContext = AlertInstanceContext
+  >(id: string): AlertType<Params, State, AlertInstanceStateType, AlertInstanceContextType> {
     if (!this.has(id)) {
       throw Boom.badRequest(
         i18n.translate('xpack.alerts.alertTypeRegistry.get.missingAlertTypeError', {
@@ -93,7 +109,12 @@ export class AlertTypeRegistry {
         })
       );
     }
-    return this.alertTypes.get(id)!;
+    return this.alertTypes.get(id)! as AlertType<
+      Params,
+      State,
+      AlertInstanceStateType,
+      AlertInstanceContextType
+    >;
   }
 
   public list(): Set<RegistryAlertType> {

x-pack/plugins/alerts/server/index.ts

@@ -17,8 +17,11 @@ export {
   AlertExecutorOptions,
   AlertActionParams,
   AlertServices,
-  State,
+  AlertTypeState,
+  AlertTypeParams,
   PartialAlert,
+  AlertInstanceState,
+  AlertInstanceContext,
 } from './types';
 export { PluginSetupContract, PluginStartContract } from './plugin';
 export { FindResult } from './alerts_client';

x-pack/plugins/alerts/server/task_runner/create_execution_handler.ts

@@ -5,7 +5,7 @@
  */
 
 import { map } from 'lodash';
-import { AlertAction, State, Context, AlertType } from '../types';
+import { AlertAction, AlertInstanceState, AlertInstanceContext, AlertType } from '../types';
 import { Logger, KibanaRequest } from '../../../../../src/core/server';
 import { transformActionParams } from './transform_action_params';
 import { PluginStartContract as ActionsPluginStartContract } from '../../../actions/server';
@@ -29,8 +29,8 @@ interface CreateExecutionHandlerOptions {
 interface ExecutionHandlerOptions {
   actionGroup: string;
   alertInstanceId: string;
-  context: Context;
-  state: State;
+  context: AlertInstanceContext;
+  state: AlertInstanceState;
 }
 
 export function createExecutionHandler({

x-pack/plugins/alerts/server/task_runner/transform_action_params.ts

@@ -6,7 +6,7 @@
 
 import Mustache from 'mustache';
 import { isString, cloneDeepWith } from 'lodash';
-import { AlertActionParams, State, Context } from '../types';
+import { AlertActionParams, AlertInstanceState, AlertInstanceContext } from '../types';
 
 interface TransformActionParamsOptions {
   alertId: string;
@@ -15,8 +15,8 @@ interface TransformActionParamsOptions {
   tags?: string[];
   alertInstanceId: string;
   actionParams: AlertActionParams;
-  state: State;
-  context: Context;
+  state: AlertInstanceState;
+  context: AlertInstanceContext;
 }
 
 export function transformActionParams({

x-pack/plugins/alerts/server/types.ts

@@ -7,22 +7,25 @@
 import { AlertInstance } from './alert_instance';
 import { AlertTypeRegistry as OrigAlertTypeRegistry } from './alert_type_registry';
 import { PluginSetupContract, PluginStartContract } from './plugin';
-import { Alert, AlertActionParams, ActionGroup } from '../common';
 import { AlertsClient } from './alerts_client';
-export * from '../common';
 import {
   ILegacyClusterClient,
   ILegacyScopedClusterClient,
   KibanaRequest,
   SavedObjectAttributes,
   SavedObjectsClientContract,
 } from '../../../../src/core/server';
+import {
+  Alert,
+  AlertActionParams,
+  ActionGroup,
+  AlertTypeParams,
+  AlertTypeState,
+  AlertInstanceContext,
+  AlertInstanceState,
+} from '../common';
 
-// This will have to remain `any` until we can extend Alert Executors with generics
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-export type State = Record<string, any>;
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-export type Context = Record<string, any>;
+export * from '../common';
 export type WithoutQueryAndParams<T> = Pick<T, Exclude<keyof T, 'query' | 'params'>>;
 export type GetServicesFunction = (request: KibanaRequest) => Services;
 export type GetBasePathFunction = (spaceId?: string) => string;
@@ -43,18 +46,26 @@ export interface Services {
   getLegacyScopedClusterClient(clusterClient: ILegacyClusterClient): ILegacyScopedClusterClient;
 }
 
-export interface AlertServices extends Services {
-  alertInstanceFactory: (id: string) => AlertInstance;
+export interface AlertServices<
+  AlertInstanceStateType extends AlertInstanceState = AlertInstanceState,
+  AlertInstanceContextType extends AlertInstanceContext = AlertInstanceContext
+> extends Services {
+  alertInstanceFactory: (
+    id: string
+  ) => AlertInstance<AlertInstanceStateType, AlertInstanceContextType>;
 }
 
-export interface AlertExecutorOptions {
+export interface AlertExecutorOptions<
+  Params extends AlertTypeParams = AlertTypeParams,
+  State extends AlertTypeState = AlertTypeState,
+  AlertInstanceStateType extends AlertInstanceState = AlertInstanceState,
+  AlertInstanceContextType extends AlertInstanceContext = AlertInstanceContext
+> {
   alertId: string;
   startedAt: Date;
   previousStartedAt: Date | null;
-  services: AlertServices;
-  // This will have to remain `any` until we can extend Alert Executors with generics
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  params: Record<string, any>;
+  services: AlertServices<AlertInstanceStateType, AlertInstanceContextType>;
+  params: Params;
   state: State;
   spaceId: string;
   namespace?: string;
@@ -69,15 +80,29 @@ export interface ActionVariable {
   description: string;
 }
 
-export interface AlertType {
+export interface AlertType<
+  Params extends AlertTypeParams = AlertTypeParams,
+  State extends AlertTypeState = AlertTypeState,
+  AlertInstanceStateType extends AlertInstanceState = AlertInstanceState,
+  AlertInstanceContextType extends AlertInstanceContext = AlertInstanceContext
+> {
   id: string;
   name: string;
   validate?: {
-    params?: { validate: (object: unknown) => AlertExecutorOptions['params'] };
+    params?: { validate: (object: unknown) => Params };
   };
   actionGroups: ActionGroup[];
   defaultActionGroupId: ActionGroup['id'];
-  executor: ({ services, params, state }: AlertExecutorOptions) => Promise<State | void>;
+  executor: ({
+    services,
+    params,
+    state,
+  }: AlertExecutorOptions<
+    Params,
+    State,
+    AlertInstanceStateType,
+    AlertInstanceContextType
+  >) => Promise<State | void>;
   producer: string;
   actionVariables?: {
     context?: ActionVariable[];

x-pack/plugins/monitoring/server/alerts/types.d.ts

@@ -4,13 +4,14 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 import { AlertMessageTokenType, AlertSeverity } from '../../common/enums';
+import { AlertInstanceState as BaseAlertInstanceState } from '../../../alerts/server';
 
 export interface AlertEnableAction {
   id: string;
   config: { [key: string]: any };
 }
 
-export interface AlertInstanceState {
+export interface AlertInstanceState extends BaseAlertInstanceState {
   alertStates: AlertState[];
 }
 

x-pack/plugins/security_solution/server/lib/detection_engine/notifications/types.ts

@@ -8,7 +8,7 @@ import {
   AlertsClient,
   PartialAlert,
   AlertType,
-  State,
+  AlertTypeState,
   AlertExecutorOptions,
 } from '../../../../../alerts/server';
 import { Alert } from '../../../../../alerts/common';
@@ -102,5 +102,9 @@ export const isNotificationAlertExecutor = (
 };
 
 export type NotificationAlertTypeDefinition = Omit<AlertType, 'executor'> & {
-  executor: ({ services, params, state }: NotificationExecutorOptions) => Promise<State | void>;
+  executor: ({
+    services,
+    params,
+    state,
+  }: NotificationExecutorOptions) => Promise<AlertTypeState | void>;
 };