[Security Solution] remove automatic troubleshooting tech preview

x-pack/platform/packages/shared/kbn-elastic-assistant-common/impl/capabilities/index.ts

@@ -20,6 +20,5 @@ export type AssistantFeatureKey = keyof AssistantFeatures;
  */
 export const defaultAssistantFeatures = Object.freeze({
   assistantModelEvaluation: false,
-  defendInsights: true,
   defendInsightsPolicyResponseFailure: false,
 });

x-pack/platform/packages/shared/kbn-elastic-assistant-common/impl/schemas/capabilities/get_capabilities_route.gen.ts

@@ -19,5 +19,5 @@ import { z } from '@kbn/zod';
 export type GetCapabilitiesResponse = z.infer<typeof GetCapabilitiesResponse>;
 export const GetCapabilitiesResponse = z.object({
   assistantModelEvaluation: z.boolean(),
-  defendInsights: z.boolean(),
+  defendInsightsPolicyResponseFailure: z.boolean(),
 });

x-pack/platform/packages/shared/kbn-elastic-assistant-common/impl/schemas/capabilities/get_capabilities_route.schema.yaml

@@ -22,11 +22,11 @@ paths:
                 properties:
                   assistantModelEvaluation:
                     type: boolean
-                  defendInsights:
+                  defendInsightsPolicyResponseFailure:
                     type: boolean
                 required:
                   - assistantModelEvaluation
-                  - defendInsights
+                  - defendInsightsPolicyResponseFailure
         '400':
           description: Generic Error
           content:

x-pack/solutions/security/packages/features/src/security/v2_features/kibana_sub_features.ts

@@ -131,9 +131,6 @@ export const getSecurityV2SubFeaturesMap = ({
   if (!experimentalFeatures.endpointManagementSpaceAwarenessEnabled) {
     securitySubFeaturesMap.delete(SecuritySubFeatureId.globalArtifactManagement);
   }
-  if (!experimentalFeatures.defendInsights) {
-    securitySubFeaturesMap.delete(SecuritySubFeatureId.workflowInsights);
-  }
 
   return Object.freeze(securitySubFeaturesMap);
 };

x-pack/solutions/security/packages/features/src/security/v3_features/kibana_sub_features.ts

@@ -78,9 +78,6 @@ export const getSecurityV3SubFeaturesMap = ({
   );
 
   // Remove disabled experimental features
-  if (!experimentalFeatures.defendInsights) {
-    securitySubFeaturesMap.delete(SecuritySubFeatureId.workflowInsights);
-  }
   if (!experimentalFeatures.trustedDevices) {
     securitySubFeaturesMap.delete(SecuritySubFeatureId.trustedDevices);
   }

x-pack/solutions/security/plugins/elastic_assistant/public/src/context/assistant_context/assistant_provider.test.tsx

@@ -70,7 +70,7 @@ describe('AssistantProvider', () => {
         }),
         assistantFeatures: expect.objectContaining({
           assistantModelEvaluation: expect.any(Boolean),
-          defendInsights: expect.any(Boolean),
+          defendInsightsPolicyResponseFailure: expect.any(Boolean),
         }),
         assistantStreamingEnabled: expect.any(Boolean),
         assistantTelemetry: expect.any(Object),

x-pack/solutions/security/plugins/elastic_assistant/server/routes/defend_insights/helpers.ts

@@ -68,24 +68,6 @@ function addGenerationInterval(
   return newGenerationIntervals;
 }
 
-export function isDefendInsightsEnabled({
-  request,
-  logger,
-  assistantContext,
-}: {
-  request: KibanaRequest;
-  logger: Logger;
-  assistantContext: ElasticAssistantApiRequestHandlerContext;
-}): boolean {
-  const pluginName = getPluginNameFromRequest({
-    request,
-    logger,
-    defaultPluginName: DEFAULT_PLUGIN_NAME,
-  });
-
-  return assistantContext.getRegisteredFeatures(pluginName).defendInsights;
-}
-
 export function isDefendInsightsPolicyResponseFailureEnabled({
   request,
   logger,

x-pack/solutions/security/plugins/elastic_assistant/server/services/app_context.test.ts

@@ -64,7 +64,6 @@ describe('AppContextService', () => {
       appContextService.start(mockAppContext);
       appContextService.registerFeatures('super', {
         assistantModelEvaluation: true,
-        defendInsights: true,
       });
       appContextService.stop();
 
@@ -126,7 +125,6 @@ describe('AppContextService', () => {
       const features: AssistantFeatures = {
         ...defaultAssistantFeatures,
         assistantModelEvaluation: true,
-        defendInsights: true,
       };
 
       appContextService.start(mockAppContext);
@@ -142,13 +140,11 @@ describe('AppContextService', () => {
       const featuresOne: AssistantFeatures = {
         ...defaultAssistantFeatures,
         assistantModelEvaluation: true,
-        defendInsights: true,
       };
       const pluginTwo = 'plugin2';
       const featuresTwo: AssistantFeatures = {
         ...defaultAssistantFeatures,
         assistantModelEvaluation: false,
-        defendInsights: false,
       };
 
       appContextService.start(mockAppContext);
@@ -164,12 +160,10 @@ describe('AppContextService', () => {
       const featuresOne: AssistantFeatures = {
         ...defaultAssistantFeatures,
         assistantModelEvaluation: true,
-        defendInsights: true,
       };
       const featuresTwo: AssistantFeatures = {
         ...defaultAssistantFeatures,
         assistantModelEvaluation: false,
-        defendInsights: false,
       };
 
       appContextService.start(mockAppContext);
@@ -191,7 +185,6 @@ describe('AppContextService', () => {
       const pluginName = 'pluginName';
       const featuresSubset: Partial<AssistantFeatures> = {
         assistantModelEvaluation: true,
-        defendInsights: true,
       };
 
       appContextService.start(mockAppContext);

x-pack/solutions/security/plugins/security_solution/common/experimental_features.ts

@@ -257,11 +257,6 @@ export const allowedExperimentalValues = Object.freeze({
    */
   siemMigrationsDisabled: false,
 
-  /**
-   * Enables the Defend Insights feature
-   */
-  defendInsights: true,
-
   /**
    * Enables the Defend Insights Policy Response Failure feature
    */

x-pack/solutions/security/plugins/security_solution/public/management/pages/endpoint_hosts/view/details/components/insights/workflow_insights.tsx

@@ -5,23 +5,12 @@
  * 2.0.
  */
 
-import {
-  EuiAccordion,
-  EuiSpacer,
-  EuiText,
-  EuiBetaBadge,
-  EuiFlexItem,
-  EuiFlexGroup,
-} from '@elastic/eui';
+import { EuiAccordion, EuiSpacer, EuiText, EuiFlexItem, EuiFlexGroup } from '@elastic/eui';
 import React, { useState, useEffect, useCallback, useMemo } from 'react';
 import moment from 'moment';
 
 import type { DefendInsightType } from '@kbn/elastic-assistant-common';
 import { ActionType } from '../../../../../../../../common/endpoint/types/workflow_insights';
-import {
-  TECHNICAL_PREVIEW_TOOLTIP,
-  TECHNICAL_PREVIEW,
-} from '../../../../../../../common/translations';
 import { useIsExperimentalFeatureEnabled } from '../../../../../../../common/hooks/use_experimental_features';
 import { useFetchInsights } from '../../../hooks/insights/use_fetch_insights';
 import { useTriggerScan } from '../../../hooks/insights/use_trigger_scan';
@@ -177,16 +166,6 @@ export const WorkflowInsights = React.memo(({ endpointId }: WorkflowInsightsProp
                 <h4>{WORKFLOW_INSIGHTS.title}</h4>
               </EuiText>
             </EuiFlexItem>
-            <EuiFlexItem grow={false}>
-              <EuiBetaBadge
-                alignment={'middle'}
-                label={TECHNICAL_PREVIEW}
-                tooltipContent={TECHNICAL_PREVIEW_TOOLTIP}
-                size="s"
-                iconType={'beaker'}
-                data-test-subj={'workflow-insights-tech-preview-badge'}
-              />
-            </EuiFlexItem>
           </EuiFlexGroup>
         }
         initialIsOpen

x-pack/solutions/security/plugins/security_solution/public/management/pages/endpoint_hosts/view/details/endpoint_details_content.tsx

@@ -15,7 +15,6 @@ import {
 } from '@elastic/eui';
 import React, { memo, useMemo } from 'react';
 import { FormattedMessage } from '@kbn/i18n-react';
-import { useIsExperimentalFeatureEnabled } from '../../../../../common/hooks/use_experimental_features';
 import { WorkflowInsights } from './components/insights/workflow_insights';
 import { isPolicyOutOfDate } from '../../utils';
 import { AgentStatus } from '../../../../../common/components/endpoint/agents/agent_status';
@@ -45,14 +44,7 @@ interface EndpointDetailsContentProps {
 export const EndpointDetailsContent = memo<EndpointDetailsContentProps>(
   ({ hostInfo, policyInfo }) => {
     // Access control
-    const isWorkflowInsightsFeatureFlagEnabled = useIsExperimentalFeatureEnabled('defendInsights');
     const { canReadWorkflowInsights } = useUserPrivileges().endpointPrivileges;
-    const canAccessWorkflowInsights = useMemo(() => {
-      if (!isWorkflowInsightsFeatureFlagEnabled) {
-        return false;
-      }
-      return canReadWorkflowInsights;
-    }, [canReadWorkflowInsights, isWorkflowInsightsFeatureFlagEnabled]);
 
     const queryParams = useEndpointSelector(uiQueryParams);
     const policyStatus = useMemo(
@@ -195,7 +187,7 @@ export const EndpointDetailsContent = memo<EndpointDetailsContentProps>(
     }, [hostInfo, policyInfo, missingPolicies, policyStatus, policyStatusClickHandler]);
     return (
       <div>
-        {canAccessWorkflowInsights && <WorkflowInsights endpointId={hostInfo.metadata.agent.id} />}
+        {canReadWorkflowInsights && <WorkflowInsights endpointId={hostInfo.metadata.agent.id} />}
         <EuiDescriptionList
           columnWidths={[1, 3]}
           compressed

x-pack/solutions/security/plugins/security_solution/server/endpoint/routes/workflow_insights/index.ts

@@ -16,8 +16,6 @@ export const registerWorkflowInsightsRoutes = (
   config: ConfigType,
   endpointContext: EndpointAppContext
 ) => {
-  if (config.experimentalFeatures.defendInsights) {
-    registerGetInsightsRoute(router, endpointContext);
-    registerUpdateInsightsRoute(router, endpointContext);
-  }
+  registerGetInsightsRoute(router, endpointContext);
+  registerUpdateInsightsRoute(router, endpointContext);
 };

x-pack/solutions/security/plugins/security_solution/server/endpoint/services/workflow_insights/index.test.ts

@@ -149,7 +149,6 @@ describe('SecurityWorkflowInsightsService', () => {
       securityWorkflowInsightsService.setup({
         kibanaVersion: kibanaPackageJson.version,
         logger,
-        isFeatureEnabled: true,
         endpointContext: mockEndpointAppContextService,
       });
 
@@ -166,7 +165,6 @@ describe('SecurityWorkflowInsightsService', () => {
       securityWorkflowInsightsService.setup({
         kibanaVersion: kibanaPackageJson.version,
         logger,
-        isFeatureEnabled: true,
         endpointContext: mockEndpointAppContextService,
       });
 
@@ -191,7 +189,6 @@ describe('SecurityWorkflowInsightsService', () => {
       securityWorkflowInsightsService.setup({
         kibanaVersion: kibanaPackageJson.version,
         logger,
-        isFeatureEnabled: true,
         endpointContext: mockEndpointAppContextService,
       });
       expect(createDatastreamMock).toHaveBeenCalledTimes(1);
@@ -218,7 +215,6 @@ describe('SecurityWorkflowInsightsService', () => {
       securityWorkflowInsightsService.setup({
         kibanaVersion: kibanaPackageJson.version,
         logger,
-        isFeatureEnabled: true,
         endpointContext: mockEndpointAppContextService,
       });
 
@@ -518,7 +514,6 @@ describe('SecurityWorkflowInsightsService', () => {
       securityWorkflowInsightsService.setup({
         kibanaVersion: kibanaPackageJson.version,
         logger,
-        isFeatureEnabled: true,
         endpointContext: mockEndpointAppContextService,
       });
 

x-pack/solutions/security/plugins/security_solution/server/endpoint/services/workflow_insights/index.ts

@@ -44,7 +44,6 @@ const DEFAULT_PAGE_SIZE = 10;
 interface SetupInterface {
   kibanaVersion: string;
   logger: Logger;
-  isFeatureEnabled: boolean;
   endpointContext: EndpointAppContextService;
 }
 
@@ -64,7 +63,6 @@ class SecurityWorkflowInsightsService {
   private _isInitialized: Promise<[void, void]> = firstValueFrom(
     combineLatest<[void, void]>([this.setup$, this.start$])
   );
-  private isFeatureEnabled = false;
 
   /**
    * Lifecycle
@@ -74,12 +72,7 @@ class SecurityWorkflowInsightsService {
     return this._isInitialized;
   }
 
-  public setup({ kibanaVersion, logger, isFeatureEnabled, endpointContext }: SetupInterface) {
-    this.isFeatureEnabled = isFeatureEnabled;
-    if (!isFeatureEnabled) {
-      return;
-    }
-
+  public setup({ kibanaVersion, logger, endpointContext }: SetupInterface) {
     this._logger = logger;
     this._endpointContext = endpointContext;
 
@@ -94,10 +87,6 @@ class SecurityWorkflowInsightsService {
   }
 
   public async start({ esClient, registerDefendInsightsCallback }: StartInterface) {
-    if (!this.isFeatureEnabled) {
-      return;
-    }
-
     this._esClient = esClient;
     await firstValueFrom(this.setup$);
 

x-pack/solutions/security/plugins/security_solution/server/plugin.ts

@@ -582,7 +582,6 @@ export class Plugin implements ISecuritySolutionPlugin {
     securityWorkflowInsightsService.setup({
       kibanaVersion: pluginContext.env.packageInfo.version,
       logger: this.logger,
-      isFeatureEnabled: config.experimentalFeatures.defendInsights,
       endpointContext: this.endpointContext.service,
     });