box-ui-elements 23.5.0-beta.2 → 23.5.0-beta.3

package/src/features/metadata-instance-editor/CascadePolicy.js

@@ -27,7 +27,6 @@ const AI_LINK = 'https://www.box.com/ai';
 type Props = {
     canEdit: boolean,
     canUseAIFolderExtraction: boolean,
-    canUseAIFolderExtractionAgentSelector: boolean,
     cascadePolicyConfiguration?: MetadataCascadePolicyConfiguration,
     isAIFolderExtractionEnabled: boolean,
     isCascadingEnabled: boolean,
@@ -44,7 +43,6 @@ type Props = {
 const CascadePolicy = ({
     canEdit,
     canUseAIFolderExtraction,
-    canUseAIFolderExtractionAgentSelector,
     cascadePolicyConfiguration,
     isCascadingEnabled,
     isCascadingOverwritten,
@@ -194,13 +192,14 @@ const CascadePolicy = ({
                                 <FormattedMessage {...messages.aiAutofillLearnMore} />
                             </Link>
                         </div>
-                        {canUseAIFolderExtractionAgentSelector && isAIFolderExtractionEnabled && (
+                        {isAIFolderExtractionEnabled && (
                             <div className="metadata-cascade-ai-agent-selector">
                                 <BoxAiAgentSelectorWithApiContainer
                                     disabled={isExistingCascadePolicy}
                                     fetcher={agentFetcher}
                                     onSelectAgent={handleAgentSelect}
                                     recordAction={() => {}}
+                                    selectorAlignment="left"
                                 />
                             </div>
                         )}

package/src/features/metadata-instance-editor/Instance.js

@@ -55,7 +55,6 @@ import './Instance.scss';
 type Props = {
     canEdit: boolean,
     canUseAIFolderExtraction?: boolean,
-    canUseAIFolderExtractionAgentSelector?: boolean,
     cascadePolicy?: MetadataCascadePolicy, // eslint-disable-line
     data: MetadataFields,
     hasError: boolean,
@@ -642,7 +641,6 @@ class Instance extends React.PureComponent<Props, State> {
     render() {
         const {
             canUseAIFolderExtraction = false,
-            canUseAIFolderExtractionAgentSelector = false,
             cascadePolicy = {},
             isDirty,
             isCascadingPolicyApplicable,
@@ -703,9 +701,6 @@ class Instance extends React.PureComponent<Props, State> {
                                             cascadePolicyConfiguration={cascadePolicy?.cascadePolicyConfiguration}
                                             canEdit={isEditing && !!cascadePolicy.canEdit}
                                             canUseAIFolderExtraction={canUseAIFolderExtraction}
-                                            canUseAIFolderExtractionAgentSelector={
-                                                canUseAIFolderExtractionAgentSelector
-                                            }
                                             isAIFolderExtractionEnabled={isAIFolderExtractionEnabled}
                                             isCascadingEnabled={isCascadingEnabled}
                                             isCascadingOverwritten={isCascadingOverwritten}

package/src/features/metadata-instance-editor/Instances.js

@@ -7,7 +7,6 @@ import type { JSONPatchOperations } from '../../common/types/api';
 
 type Props = {
     canUseAIFolderExtraction?: boolean,
-    canUseAIFolderExtractionAgentSelector?: boolean,
     editors?: Array<MetadataEditor>,
     isCascadingPolicyApplicable?: boolean,
     onModification?: (id: string, isDirty: boolean) => void,
@@ -23,7 +22,6 @@ type Props = {
 
 const Instances = ({
     canUseAIFolderExtraction = false,
-    canUseAIFolderExtractionAgentSelector = false,
     isCascadingPolicyApplicable = false,
     editors = [],
     onModification,
@@ -39,7 +37,6 @@ const Instances = ({
                 <Instance
                     canEdit={instance.canEdit}
                     canUseAIFolderExtraction={canUseAIFolderExtraction}
-                    canUseAIFolderExtractionAgentSelector={canUseAIFolderExtractionAgentSelector}
                     cascadePolicy={instance.cascadePolicy}
                     data={instance.data}
                     hasError={hasError}

package/src/features/metadata-instance-editor/MetadataInstanceEditor.js

@@ -15,7 +15,6 @@ type Props = {
     blurExceptionClassNames?: Array<string>,
     canAdd: boolean,
     canUseAIFolderExtraction?: boolean,
-    canUseAIFolderExtractionAgentSelector?: boolean,
     editors?: Array<MetadataEditor>,
     isCascadingPolicyApplicable?: boolean,
     isDropdownBusy?: boolean,
@@ -38,7 +37,6 @@ const MetadataInstanceEditor = ({
     blurExceptionClassNames,
     canAdd,
     canUseAIFolderExtraction = false,
-    canUseAIFolderExtractionAgentSelector = false,
     isCascadingPolicyApplicable = false,
     isDropdownBusy,
     editors = [],
@@ -66,7 +64,6 @@ const MetadataInstanceEditor = ({
                 <ScrollWrapper>
                     <Instances
                         canUseAIFolderExtraction={canUseAIFolderExtraction}
-                        canUseAIFolderExtractionAgentSelector={canUseAIFolderExtractionAgentSelector}
                         editors={editors}
                         isCascadingPolicyApplicable={isCascadingPolicyApplicable}
                         onModification={onModification}

package/src/features/metadata-instance-editor/__tests__/CascadePolicy.test.js

@@ -118,7 +118,6 @@ describe('features/metadata-instance-editor/CascadePolicy', () => {
                 <CascadePolicy
                     canEdit
                     canUseAIFolderExtraction
-                    canUseAIFolderExtractionAgentSelector
                     shouldShowCascadeOptions
                     isAIFolderExtractionEnabled
                     onAIFolderExtractionToggle={jest.fn()}
@@ -133,18 +132,12 @@ describe('features/metadata-instance-editor/CascadePolicy', () => {
             expect(screen.getByRole('combobox', { name: 'Standard' })).toBeInTheDocument();
         });
 
-        test('should not render AI agent selector when canUseAIFolderExtractionAgentSelector is false', () => {
-            render(<CascadePolicy canEdit canUseAIFolderExtraction shouldShowCascadeOptions />);
-            expect(screen.queryByRole('combobox', { name: 'Standard' })).not.toBeInTheDocument();
-        });
-
         test('should call onAIAgentSelect when an agent is selected', async () => {
             const onAIAgentSelect = jest.fn();
             render(
                 <CascadePolicy
                     canEdit
                     canUseAIFolderExtraction
-                    canUseAIFolderExtractionAgentSelector
                     shouldShowCascadeOptions
                     isAIFolderExtractionEnabled
                     onAIAgentSelect={onAIAgentSelect}
@@ -185,7 +178,6 @@ describe('features/metadata-instance-editor/CascadePolicy', () => {
                 <CascadePolicy
                     canEdit
                     canUseAIFolderExtraction
-                    canUseAIFolderExtractionAgentSelector
                     shouldShowCascadeOptions
                     isAIFolderExtractionEnabled
                     cascadePolicyConfiguration={cascadePolicyConfiguration}
@@ -207,7 +199,6 @@ describe('features/metadata-instance-editor/CascadePolicy', () => {
                 <CascadePolicy
                     canEdit
                     canUseAIFolderExtraction
-                    canUseAIFolderExtractionAgentSelector
                     shouldShowCascadeOptions
                     isAIFolderExtractionEnabled
                     onAIFolderExtractionToggle={jest.fn()}

package/src/features/metadata-instance-editor/__tests__/Instance.test.js

@@ -771,7 +771,6 @@ const getBaseProps = (props = {}) => ({
         cascadePolicyType: 'regular', // default to non-AI
     },
     canUseAIFolderExtraction: true, // Assume feature flag is on
-    canUseAIFolderExtractionAgentSelector: false,
     ...props,
 });
 
@@ -862,33 +861,6 @@ describe('Instance Component - React Testing Library', () => {
     });
 
     describe('Props passed to CascadePolicy', () => {
-        test('should pass canUseAIFolderExtractionAgentSelector to CascadePolicy', async () => {
-            render(
-                <Instance
-                    {...getBaseProps({
-                        canUseAIFolderExtractionAgentSelector: true,
-                        cascadePolicy: {
-                            id: 'policy-1',
-                            canEdit: true,
-                            isEnabled: true,
-                            cascadePolicyType: CASCADE_POLICY_TYPE_AI_EXTRACT,
-                        },
-                    })}
-                />,
-            );
-
-            const editButton = screen.queryByRole('button', { name: 'Edit Metadata' });
-            if (editButton) await userEvent.click(editButton); // Enter edit mode to ensure CascadePolicy options are visible
-
-            const cascadeToggle = screen.getByRole('switch', { name: 'Enable Cascade Policy' });
-            expect(cascadeToggle).toBeChecked();
-
-            const aiToggle = screen.getByRole('switch', { name: 'Box AI Autofill' });
-            expect(aiToggle).toBeChecked();
-
-            expect(screen.getByRole('combobox', { name: 'Standard' })).toBeInTheDocument();
-        });
-
         test('should disable CascadePolicy options when a cascade already exists', async () => {
             render(<Instance {...getBaseProps()} />);
 

package/src/features/metadata-instance-editor/__tests__/Instances.test.js

@@ -253,11 +253,9 @@ describe('features/metadata-editor-editor/Instances', () => {
     });
 });
 
-describe('Instances component - canUseAIFolderExtractionAgentSelector prop', () => {
-    test('should pass canUseAIFolderExtractionAgentSelector to child Instance components, showing agent selector', async () => {
-        const props = getInstancesBaseProps({
-            canUseAIFolderExtractionAgentSelector: true,
-        });
+describe('Instances component -  AI agent selector', () => {
+    test('should show AI agent selector when canUseAIFolderExtraction is true', async () => {
+        const props = getInstancesBaseProps();
         props.editors[0].instance.cascadePolicy.cascadePolicyType = CASCADE_POLICY_TYPE_AI_EXTRACT;
         render(<Instances {...props} />);
 
@@ -273,11 +271,11 @@ describe('Instances component - canUseAIFolderExtractionAgentSelector prop', ()
         expect(screen.getByRole('combobox', { name: 'Standard' })).toBeInTheDocument();
     });
 
-    test('should not show agent selector in child Instance if canUseAIFolderExtractionAgentSelector is false', async () => {
+    test('should not show agent selector in child Instance if canUseAIFolderExtraction is false', async () => {
         render(
             <Instances
                 {...getInstancesBaseProps({
-                    canUseAIFolderExtractionAgentSelector: false,
+                    canUseAIFolderExtraction: false,
                 })}
             />,
         );
@@ -288,9 +286,9 @@ describe('Instances component - canUseAIFolderExtractionAgentSelector prop', ()
         expect(screen.queryByRole('combobox', { name: 'Standard' })).not.toBeInTheDocument();
     });
 
-    test('should not show agent selector in child Instance if canUseAIFolderExtractionAgentSelector is undefined', async () => {
+    test('should not show agent selector in child Instance if canUseAIFolderExtraction is undefined', async () => {
         const props = getInstancesBaseProps();
-        delete props.canUseAIFolderExtractionAgentSelector;
+        delete props.canUseAIFolderExtraction;
         render(<Instances {...props} />);
 
         const editButton = screen.getByRole('button', { name: 'Edit Metadata' });

package/src/features/metadata-instance-editor/__tests__/MetadataInstanceEditor.test.js

@@ -534,57 +534,10 @@ describe('features/metadata-editor-editor/MetadataInstanceEditor', () => {
     });
 });
 
-describe('MetadataInstanceEditor - canUseAIFolderExtractionAgentSelector prop', () => {
-    test('should propagate canUseAIFolderExtractionAgentSelector, showing agent selector', async () => {
-        const props = getMetadataEditorBaseProps({
-            canUseAIFolderExtraction: true,
-            canUseAIFolderExtractionAgentSelector: true,
-        });
-        props.editors[0].instance.cascadePolicy.cascadePolicyType = 'ai_extract';
-        render(<MetadataInstanceEditor {...props} />);
-
-        const editButton = await screen.findByRole('button', { name: 'Edit Metadata' }, { timeout: 3000 });
-        await userEvent.click(editButton);
-
-        const cascadeToggle = screen.getByRole('switch', { name: 'Enable Cascade Policy' });
-        expect(cascadeToggle).toBeChecked();
-
-        const aiToggle = screen.getByRole('switch', { name: 'Box AI Autofill' });
-        expect(aiToggle).toBeChecked();
-
-        expect(screen.getByRole('combobox', { name: 'Standard' })).toBeInTheDocument();
-    });
-
-    test('should not show agent selector if canUseAIFolderExtractionAgentSelector is false', async () => {
-        render(
-            <MetadataInstanceEditor
-                {...getMetadataEditorBaseProps({
-                    canUseAIFolderExtractionAgentSelector: false,
-                })}
-            />,
-        );
-
-        const editButton = await screen.findByRole('button', { name: 'Edit Metadata' });
-        await userEvent.click(editButton);
-
-        expect(screen.queryByRole('combobox', { name: 'Standard' })).not.toBeInTheDocument();
-    });
-
-    test('should not show agent selector if canUseAIFolderExtractionAgentSelector is undefined', async () => {
-        const props = getMetadataEditorBaseProps();
-        delete props.canUseAIFolderExtractionAgentSelector;
-        render(<MetadataInstanceEditor {...props} />);
-
-        const editButton = await screen.findByRole('button', { name: 'Edit Metadata' });
-        await userEvent.click(editButton);
-
-        expect(screen.queryByRole('combobox', { name: 'Standard' })).not.toBeInTheDocument();
-    });
-
+describe('MetadataInstanceEditor agent selector', () => {
     test('should show "Enhanced" in the combobox when the second option is selected', async () => {
         const props = getMetadataEditorBaseProps({
             canUseAIFolderExtraction: true,
-            canUseAIFolderExtractionAgentSelector: true,
         });
         props.editors[0].instance.cascadePolicy.cascadePolicyType = 'ai_extract';
         props.editors[0].instance.cascadePolicy.id = null;

package/src/features/metadata-instance-editor/__tests__/__snapshots__/Instance.test.js.snap

@@ -416,7 +416,6 @@ exports[`features/metadata-instance-editor/fields/Instance should correctly rend
         >
           <CascadePolicy
             canUseAIFolderExtraction={false}
-            canUseAIFolderExtractionAgentSelector={false}
             isAIFolderExtractionEnabled={false}
             isCascadingEnabled={true}
             isCascadingOverwritten={false}

package/src/features/metadata-instance-editor/__tests__/__snapshots__/Instances.test.js.snap

@@ -5,7 +5,6 @@ exports[`features/metadata-editor-editor/Instances should correctly render edito
   <injectIntl(Instance)
     canEdit={true}
     canUseAIFolderExtraction={false}
-    canUseAIFolderExtractionAgentSelector={false}
     data={
       {
         "datefield": "2018-06-20T00:00:00.000Z",
@@ -111,7 +110,6 @@ exports[`features/metadata-editor-editor/Instances should correctly render edito
   <injectIntl(Instance)
     canEdit={false}
     canUseAIFolderExtraction={false}
-    canUseAIFolderExtractionAgentSelector={false}
     data={
       {
         "datefield": "2018-06-20T00:00:00.000Z",

package/src/features/metadata-instance-editor/__tests__/__snapshots__/MetadataInstanceEditor.test.js.snap

@@ -772,7 +772,6 @@ exports[`features/metadata-editor-editor/MetadataInstanceEditor should correctly
     >
       <Instances
         canUseAIFolderExtraction={false}
-        canUseAIFolderExtractionAgentSelector={false}
         editors={
           [
             {

package/src/features/metadata-instance-editor/stories/tests/CascadePolicy-visual.stories.js

@@ -5,7 +5,6 @@ import CascadePolicy from '../../CascadePolicy';
 const baseProps = {
     canEdit: true,
     canUseAIFolderExtraction: true,
-    canUseAIFolderExtractionAgentSelector: true,
     isAIFolderExtractionEnabled: false,
     isCascadingEnabled: true,
     isCascadingOverwritten: false,