Skip to content

Commit e05692d

Browse files
committed
hide ai assitant button when in preview mode
1 parent 6c80a08 commit e05692d

File tree

3 files changed

+22
-4
lines changed

3 files changed

+22
-4
lines changed

x-pack/solutions/security/plugins/security_solution/public/flyout/entity_details/generic_right/footer.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ export const GenericEntityFlyoutFooter = ({
4242
const { showAssistant, showAssistantOverlay } = useAssetInventoryAssistant({
4343
entityId,
4444
entityFields,
45+
isPreviewMode,
4546
});
4647

4748
const openDocumentFlyout = useCallback(() => {

x-pack/solutions/security/plugins/security_solution/public/flyout/entity_details/generic_right/hooks/use_asset_inventory_assistant.test.tsx

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ const entityId = 'test-entity-id';
3030

3131
const renderUseAssetInventoryAssistant = () =>
3232
renderHook((props: UseAssetInventoryAssistantParams) => useAssetInventoryAssistant(props), {
33-
initialProps: { entityId, entityFields: mockEntityFields },
33+
initialProps: { entityId, entityFields: mockEntityFields, isPreviewMode: false },
3434
});
3535

3636
const useAssistantOverlayMock = useAssistantOverlay as jest.Mock;
@@ -151,6 +151,17 @@ describe('useAssetInventoryAssistant', () => {
151151
expect(hookResult.result.current.showAssistant).toEqual(false);
152152
});
153153

154+
it('should return showAssistant false when isPreviewMode is true', () => {
155+
const hookResultWithPreviewMode = renderHook(
156+
(props: UseAssetInventoryAssistantParams) => useAssetInventoryAssistant(props),
157+
{
158+
initialProps: { entityId, entityFields: mockEntityFields, isPreviewMode: true },
159+
}
160+
);
161+
162+
expect(hookResultWithPreviewMode.result.current.showAssistant).toEqual(false);
163+
});
164+
154165
it('returns entity fields as prompt context data', async () => {
155166
hookResult = renderUseAssetInventoryAssistant();
156167

@@ -169,15 +180,15 @@ describe('useAssetInventoryAssistant', () => {
169180

170181
it('falls back to default conversation ID when entityId is not provided', () => {
171182
renderHook((props: UseAssetInventoryAssistantParams) => useAssetInventoryAssistant(props), {
172-
initialProps: { entityId: '', entityFields: mockEntityFields },
183+
initialProps: { entityId: '', entityFields: mockEntityFields, isPreviewMode: false },
173184
});
174185

175186
expect(useAssistantOverlayMock.mock.calls[0][1]).toEqual('Entity Summary');
176187
});
177188

178189
it('returns empty prompt context when entityFields is empty', async () => {
179190
renderHook((props: UseAssetInventoryAssistantParams) => useAssetInventoryAssistant(props), {
180-
initialProps: { entityId, entityFields: {} },
191+
initialProps: { entityId, entityFields: {}, isPreviewMode: false },
181192
});
182193

183194
const getPromptContext = (useAssistantOverlay as jest.Mock).mock.calls[0][3];

x-pack/solutions/security/plugins/security_solution/public/flyout/entity_details/generic_right/hooks/use_asset_inventory_assistant.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ const useAssistantNoop = () => ({
2626
export interface UseAssetInventoryAssistantParams {
2727
entityId: EntityEcs['id'];
2828
entityFields: Record<string, string[]>;
29+
isPreviewMode: boolean;
2930
}
3031

3132
export interface UseAssetInventoryAssistantResult {
@@ -49,6 +50,7 @@ export interface UseAssetInventoryAssistantResult {
4950
export const useAssetInventoryAssistant = ({
5051
entityId,
5152
entityFields,
53+
isPreviewMode,
5254
}: UseAssetInventoryAssistantParams): UseAssetInventoryAssistantResult => {
5355
const { hasAssistantPrivilege, isAssistantEnabled, isAssistantVisible } =
5456
useAssistantAvailability();
@@ -81,7 +83,11 @@ export const useAssetInventoryAssistant = ({
8183

8284
return {
8385
showAssistant:
84-
isAssistantEnabled && hasAssistantPrivilege && promptContextId !== null && isAssistantVisible,
86+
isAssistantEnabled &&
87+
hasAssistantPrivilege &&
88+
promptContextId !== null &&
89+
isAssistantVisible &&
90+
!isPreviewMode,
8591
showAssistantOverlay,
8692
promptContextId: promptContextId || '',
8793
};

0 commit comments

Comments
 (0)