Default alerts as data docs for legacy siem notification rule type

x-pack/plugins/security_solution/server/lib/detection_engine/routes/__mocks__/request_responses.ts

@@ -46,7 +46,7 @@ import {
 } from '../../../../../common/api/detection_engine/signals_migration/mocks';
 
 // eslint-disable-next-line no-restricted-imports
-import type { LegacyRuleNotificationAlertType } from '../../rule_actions_legacy';
+import type { LegacyRuleNotificationRuleType } from '../../rule_actions_legacy';
 import type { RuleAlertType, RuleParams } from '../../rule_schema';
 import { getQueryRuleParams } from '../../rule_schema/mocks';
 
@@ -520,7 +520,7 @@ export const legacyGetNotificationResult = ({
 }: {
   id?: string;
   ruleId?: string;
-} = {}): LegacyRuleNotificationAlertType => ({
+} = {}): LegacyRuleNotificationRuleType => ({
   id,
   name: 'Notification for Rule Test',
   tags: [],
@@ -567,7 +567,7 @@ export const legacyGetNotificationResult = ({
  */
 export const legacyGetFindNotificationsResultWithSingleHit = (
   ruleId = '123'
-): FindHit<LegacyRuleNotificationAlertType> => ({
+): FindHit<LegacyRuleNotificationRuleType> => ({
   page: 1,
   perPage: 1,
   total: 1,

x-pack/plugins/security_solution/server/lib/detection_engine/rule_actions_legacy/api/create_legacy_notification/route.ts

@@ -14,7 +14,7 @@ import { legacyUpdateOrCreateRuleActionsSavedObject } from '../../logic/rule_act
 // eslint-disable-next-line no-restricted-imports
 import { legacyReadNotifications } from '../../logic/notifications/legacy_read_notifications';
 // eslint-disable-next-line no-restricted-imports
-import type { LegacyRuleNotificationAlertTypeParams } from '../../logic/notifications/legacy_types';
+import type { LegacyRuleNotificationRuleTypeParams } from '../../logic/notifications/legacy_types';
 // eslint-disable-next-line no-restricted-imports
 import { legacyCreateNotifications } from '../../logic/notifications/legacy_create_notifications';
 import { UPDATE_OR_CREATE_LEGACY_ACTIONS } from '../../../../../../common/constants';
@@ -75,7 +75,7 @@ export const legacyCreateLegacyNotificationRoute = (
             ruleAlertId,
           });
           if (notification != null) {
-            await rulesClient.update<LegacyRuleNotificationAlertTypeParams>({
+            await rulesClient.update<LegacyRuleNotificationRuleTypeParams>({
               id: notification.id,
               data: {
                 tags: [],

x-pack/plugins/security_solution/server/lib/detection_engine/rule_actions_legacy/index.ts

@@ -8,13 +8,13 @@
 export * from './api/register_routes';
 
 // eslint-disable-next-line no-restricted-imports
-export { legacyRulesNotificationAlertType } from './logic/notifications/legacy_rules_notification_alert_type';
+export { legacyRulesNotificationRuleType } from './logic/notifications/legacy_rules_notification_rule_type';
 // eslint-disable-next-line no-restricted-imports
-export { legacyIsNotificationAlertExecutor } from './logic/notifications/legacy_types';
+export { isLegacyNotificationRuleExecutor } from './logic/notifications/legacy_types';
 // eslint-disable-next-line no-restricted-imports
 export type {
-  LegacyRuleNotificationAlertType,
-  LegacyRuleNotificationAlertTypeParams,
+  LegacyRuleNotificationRuleType,
+  LegacyRuleNotificationRuleTypeParams,
 } from './logic/notifications/legacy_types';
 export type { NotificationRuleTypeParams } from './logic/notifications/schedule_notification_actions';
 export { scheduleNotificationActions } from './logic/notifications/schedule_notification_actions';

x-pack/plugins/security_solution/server/lib/detection_engine/rule_actions_legacy/logic/notifications/legacy_create_notifications.ts

@@ -10,7 +10,7 @@ import { SERVER_APP_ID, LEGACY_NOTIFICATIONS_ID } from '../../../../../../common
 // eslint-disable-next-line no-restricted-imports
 import type {
   CreateNotificationParams,
-  LegacyRuleNotificationAlertTypeParams,
+  LegacyRuleNotificationRuleTypeParams,
 } from './legacy_types';
 
 /**
@@ -23,8 +23,8 @@ export const legacyCreateNotifications = async ({
   ruleAlertId,
   interval,
   name,
-}: CreateNotificationParams): Promise<SanitizedRule<LegacyRuleNotificationAlertTypeParams>> =>
-  rulesClient.create<LegacyRuleNotificationAlertTypeParams>({
+}: CreateNotificationParams): Promise<SanitizedRule<LegacyRuleNotificationRuleTypeParams>> =>
+  rulesClient.create<LegacyRuleNotificationRuleTypeParams>({
     data: {
       name,
       tags: [],

x-pack/plugins/security_solution/server/lib/detection_engine/rule_actions_legacy/logic/notifications/legacy_read_notifications.ts

@@ -9,7 +9,7 @@ import type { RuleTypeParams, SanitizedRule } from '@kbn/alerting-plugin/common'
 // eslint-disable-next-line no-restricted-imports
 import type { LegacyReadNotificationParams } from './legacy_types';
 // eslint-disable-next-line no-restricted-imports
-import { legacyIsAlertType } from './legacy_types';
+import { isLegacyRuleType } from './legacy_types';
 // eslint-disable-next-line no-restricted-imports
 import { legacyFindNotifications } from './legacy_find_notifications';
 
@@ -24,7 +24,7 @@ export const legacyReadNotifications = async ({
   if (id != null) {
     try {
       const notification = await rulesClient.get({ id });
-      if (legacyIsAlertType(notification)) {
+      if (isLegacyRuleType(notification)) {
         return notification;
       } else {
         return null;
@@ -43,10 +43,7 @@ export const legacyReadNotifications = async ({
       filter: `alert.attributes.params.ruleAlertId: "${ruleAlertId}"`,
       page: 1,
     });
-    if (
-      notificationFromFind.data.length === 0 ||
-      !legacyIsAlertType(notificationFromFind.data[0])
-    ) {
+    if (notificationFromFind.data.length === 0 || !isLegacyRuleType(notificationFromFind.data[0])) {
       return null;
     } else {
       return notificationFromFind.data[0];