[Security Assistant] Fix Security Assistant settings link behavior and adjust initial settings tab (#197323)

stephmilovic · web-flow · commit a6dc47ddeb15 · 2024-10-23T11:06:47.000-05:00
diff --git a/x-pack/packages/kbn-elastic-assistant/impl/assistant/conversations/conversation_settings/translations.ts b/x-pack/packages/kbn-elastic-assistant/impl/assistant/conversations/conversation_settings/translations.ts
@@ -59,7 +59,7 @@ export const SETTINGS_PROMPT_HELP_TEXT_TITLE = i18n.translate(
 export const STREAMING_TITLE = i18n.translate(
   'xpack.elasticAssistant.assistant.conversations.settings.streamingTitle',
   {
-    defaultMessage: 'STREAMING',
+    defaultMessage: 'Streaming',
   }
 );
 
diff --git a/x-pack/packages/kbn-elastic-assistant/impl/assistant/settings/assistant_settings_management.tsx b/x-pack/packages/kbn-elastic-assistant/impl/assistant/settings/assistant_settings_management.tsx
@@ -37,7 +37,7 @@ interface Props {
   dataViews: DataViewsContract;
   selectedConversation: Conversation;
   onTabChange?: (tabId: string) => void;
-  currentTab?: SettingsTabs;
+  currentTab: SettingsTabs;
 }
 
 /**
@@ -65,14 +65,14 @@ export const AssistantSettingsManagement: React.FC<Props> = React.memo(
 
     const tabsConfig = useMemo(
       () => [
-        {
-          id: CONNECTORS_TAB,
-          label: i18n.CONNECTORS_MENU_ITEM,
-        },
         {
           id: CONVERSATIONS_TAB,
           label: i18n.CONVERSATIONS_MENU_ITEM,
         },
+        {
+          id: CONNECTORS_TAB,
+          label: i18n.CONNECTORS_MENU_ITEM,
+        },
         {
           id: SYSTEM_PROMPTS_TAB,
           label: i18n.SYSTEM_PROMPTS_MENU_ITEM,
diff --git a/x-pack/packages/kbn-elastic-assistant/impl/assistant/settings/settings_context_menu/settings_context_menu.tsx b/x-pack/packages/kbn-elastic-assistant/impl/assistant/settings/settings_context_menu/settings_context_menu.tsx
@@ -12,6 +12,7 @@ import {
   EuiContextMenuPanel,
   EuiContextMenuItem,
   EuiConfirmModal,
+  EuiIcon,
   EuiNotificationBadge,
   EuiPopover,
   EuiButtonIcon,
@@ -68,6 +69,7 @@ export const SettingsContextMenu: React.FC<Params> = React.memo(
       () =>
         navigateToApp('management', {
           path: 'kibana/securityAiAssistantManagement',
+          openInNewTab: true,
         }),
       [navigateToApp]
     );
@@ -81,6 +83,7 @@ export const SettingsContextMenu: React.FC<Params> = React.memo(
       () =>
         navigateToApp('management', {
           path: `kibana/securityAiAssistantManagement?tab=${KNOWLEDGE_BASE_TAB}`,
+          openInNewTab: true,
         }),
       [navigateToApp]
     );
@@ -101,6 +104,13 @@ export const SettingsContextMenu: React.FC<Params> = React.memo(
           data-test-subj={'ai-assistant-settings'}
         >
           {i18n.AI_ASSISTANT_SETTINGS}
+          <EuiIcon
+            css={css`
+              margin-left: ${euiThemeVars.euiSizeXS};
+            `}
+            size="s"
+            type="popout"
+          />
         </EuiContextMenuItem>,
         <EuiContextMenuItem
           aria-label={'knowledge-base'}
@@ -109,6 +119,13 @@ export const SettingsContextMenu: React.FC<Params> = React.memo(
           data-test-subj={'knowledge-base'}
         >
           {i18n.KNOWLEDGE_BASE}
+          <EuiIcon
+            css={css`
+              margin-left: ${euiThemeVars.euiSizeXS};
+            `}
+            size="s"
+            type="popout"
+          />
         </EuiContextMenuItem>,
         <EuiContextMenuItem
           aria-label={'anonymization'}
diff --git a/x-pack/plugins/security_solution/public/assistant/stack_management/management_settings.tsx b/x-pack/plugins/security_solution/public/assistant/stack_management/management_settings.tsx
@@ -19,7 +19,7 @@ import {
 import { useConversation } from '@kbn/elastic-assistant/impl/assistant/use_conversation';
 import type { FetchConversationsResponse } from '@kbn/elastic-assistant/impl/assistant/api';
 import { SECURITY_AI_SETTINGS } from '@kbn/elastic-assistant/impl/assistant/settings/translations';
-import { CONNECTORS_TAB } from '@kbn/elastic-assistant/impl/assistant/settings/const';
+import { CONVERSATIONS_TAB } from '@kbn/elastic-assistant/impl/assistant/settings/const';
 import type { SettingsTabs } from '@kbn/elastic-assistant/impl/assistant/settings/types';
 import { useKibana } from '../../common/lib/kibana';
 
@@ -68,7 +68,7 @@ export const ManagementSettings = React.memo(() => {
 
   const [searchParams] = useSearchParams();
   const currentTab = useMemo(
-    () => (searchParams.get('tab') as SettingsTabs) ?? CONNECTORS_TAB,
+    () => (searchParams.get('tab') as SettingsTabs) ?? CONVERSATIONS_TAB,
     [searchParams]
   );
 

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ export const SETTINGS_PROMPT_HELP_TEXT_TITLE = i18n.translate(`
`59`	`59`	`export const STREAMING_TITLE = i18n.translate(`
`60`	`60`	`'xpack.elasticAssistant.assistant.conversations.settings.streamingTitle',`
`61`	`61`	`{`
`62`		`- defaultMessage: 'STREAMING',`
	`62`	`+ defaultMessage: 'Streaming',`
`63`	`63`	`}`
`64`	`64`	`);`
`65`	`65`