box-ui-elements 23.4.0-beta.12 → 23.4.0-beta.14

package/src/elements/content-sidebar/__tests__/useSidebarMetadataFetcher.test.tsx

@@ -36,6 +36,7 @@ const mockPreconditionFailedError = {
 const mockFile = {
     id: '123',
     permissions: { [FIELD_PERMISSIONS_CAN_UPLOAD]: true },
+    type: 'file',
 };
 
 const mockTemplates = [
@@ -423,5 +424,62 @@ describe('useSidebarMetadataFetcher', () => {
                 }),
             );
         });
+
+        test('should call extractStructured with custom AI agent ID', async () => {
+            const { result } = setupHook();
+            const agentId = 'custom-agent-123';
+
+            await result.current.extractSuggestions('templateKey', 'global', agentId);
+
+            expect(mockAPI.extractStructured).toHaveBeenCalledWith({
+                items: [{ id: mockFile.id, type: mockFile.type }],
+                metadata_template: { template_key: 'templateKey', scope: 'global', type: 'metadata_template' },
+                ai_agent: { type: 'ai_agent_id', id: agentId },
+            });
+        });
+
+        test('should not call extractStructured with custom AI agent ID', async () => {
+            const { result } = setupHook();
+
+            await result.current.extractSuggestions('templateKey', 'global');
+
+            // Assert that ai_agent is NOT present
+            expect(mockAPI.extractStructured).toHaveBeenCalledWith(
+                expect.not.objectContaining({
+                    ai_agent: expect.anything(),
+                }),
+            );
+            // Also verify what IS present
+            expect(mockAPI.extractStructured).toHaveBeenCalledWith({
+                items: [{ id: mockFile.id, type: mockFile.type }],
+                metadata_template: { template_key: 'templateKey', scope: 'global', type: 'metadata_template' },
+            });
+        });
+
+        test('should handle undefined agentIDs', async () => {
+            const { result } = setupHook();
+
+            await result.current.extractSuggestions('templateKey', 'global', undefined);
+
+            // Assert that ai_agent is NOT present
+            expect(mockAPI.extractStructured).toHaveBeenCalledWith(
+                expect.not.objectContaining({
+                    ai_agent: expect.anything(),
+                }),
+            );
+            // Also verify what IS present
+            expect(mockAPI.extractStructured).toHaveBeenCalledWith({
+                items: [{ id: mockFile.id, type: mockFile.type }],
+                metadata_template: { template_key: 'templateKey', scope: 'global', type: 'metadata_template' },
+            });
+            mockAPI.extractStructured.mockClear();
+
+            await result.current.extractSuggestions('templateKey', 'global', '');
+            expect(mockAPI.extractStructured).toHaveBeenCalledWith(
+                expect.not.objectContaining({
+                    ai_agent: expect.anything(),
+                }),
+            );
+        });
     });
 });

package/src/elements/content-sidebar/hooks/useSidebarMetadataFetcher.ts

@@ -29,6 +29,7 @@ import messages from '../../common/messages';
 
 import { type BoxItem } from '../../../common/types/core';
 import { type ErrorContextProps, type ExternalProps, type SuccessContextProps } from '../MetadataSidebarRedesign';
+import { type AiExtractStructured } from '../../../api/schemas/AiExtractStructured';
 
 export enum STATUS {
     IDLE = 'idle',
@@ -45,7 +46,7 @@ interface DataFetcher {
         | ERROR_CODE_METADATA_PRECONDITION_FAILED
         | ERROR_CODE_UNKNOWN
         | null;
-    extractSuggestions: (templateKey: string, scope: string) => Promise<MetadataTemplateField[]>;
+    extractSuggestions: (templateKey: string, scope: string, agentId?: string) => Promise<MetadataTemplateField[]>;
     file: BoxItem | null;
     handleCreateMetadataInstance: (
         templateInstance: MetadataTemplateInstance,
@@ -214,16 +215,19 @@ function useSidebarMetadataFetcher(
 
     const [, setError] = React.useState();
     const extractSuggestions = React.useCallback(
-        async (templateKey: string, scope: string): Promise<MetadataTemplateField[]> => {
+        async (templateKey: string, scope: string, agentId?: string): Promise<MetadataTemplateField[]> => {
             const aiAPI = api.getIntelligenceAPI();
             setExtractErrorCode(null);
-
             let answer = null;
+            const customAiAgent = agentId ? { ai_agent: { type: 'ai_agent_id', id: agentId } } : {};
+            const requestBody: AiExtractStructured = {
+                items: [{ id: file.id, type: file.type }],
+                metadata_template: { template_key: templateKey, scope, type: 'metadata_template' },
+                ...customAiAgent,
+            };
+
             try {
-                answer = (await aiAPI.extractStructured({
-                    items: [{ id: file.id, type: file.type }],
-                    metadata_template: { template_key: templateKey, scope, type: 'metadata_template' },
-                })) as Record<string, MetadataFieldValue>;
+                answer = (await aiAPI.extractStructured(requestBody)) as Record<string, MetadataFieldValue>;
             } catch (error) {
                 // Axios makes the status code nested under the response object
                 if (error.response?.status === 408) {

package/src/elements/index.js

@@ -3,6 +3,7 @@
 export { default as ContentExplorer } from './content-explorer';
 export { default as ContentPreview } from './content-preview';
 export { default as ContentPicker, ContentPickerPopup } from './content-picker';
+// $FlowFixMe
 export { default as ContentUploader, ContentUploaderPopup } from './content-uploader';
 export { default as ContentOpenWith } from './content-open-with';
 export { default as ContentSidebar } from './content-sidebar';

package/src/elements/wrappers/ContentUploader.js

@@ -9,7 +9,9 @@ import * as React from 'react';
 // eslint-disable-next-line react/no-deprecated
 import { render } from 'react-dom';
 import ES6Wrapper from './ES6Wrapper';
+// $FlowFixMe
 import ContentUploaderPopup from '../content-uploader/ContentUploaderPopup';
+// $FlowFixMe
 import WrappedContentUploaderComponent from '../content-uploader/ContentUploader';
 import type { UploadFileWithAPIOptions } from '../../common/types/upload';
 import type { BoxItem } from '../../common/types/core';

package/src/features/metadata-instance-fields/MetadataField.js

@@ -21,6 +21,7 @@ import {
     FIELD_TYPE_STRING,
     FIELD_TYPE_DATE,
     FIELD_TYPE_MULTISELECT,
+    FIELD_TYPE_TAXONOMY,
 } from './constants';
 
 type Props = {
@@ -160,6 +161,19 @@ const MetadataField = ({
                 />
             );
 
+        // The taxonomy field is a valid field type which,
+        // although not yet supported here, should not trigger an error message.
+        // For this reason, we are currently setting it to read-only.
+        case FIELD_TYPE_TAXONOMY:
+            return (
+                <ReadOnlyMetadataField
+                    dataValue={dataValue}
+                    description={description}
+                    displayName={displayName}
+                    type={type}
+                />
+            );
+
         default:
             return (
                 <InlineError title={type}>

package/src/features/metadata-instance-fields/__tests__/MetadataField.test.js

@@ -33,6 +33,12 @@ describe('features/metadata-instance-editor/fields/MetadataField', () => {
         );
         expect(wrapper).toMatchSnapshot();
     });
+    test('should correctly render a taxonomy field - for the time being, in read-only mode', () => {
+        const wrapper = shallow(
+            <MetadataField canEdit dataValue="value" onChange={onChange} onRemove={onRemove} type="taxonomy" />,
+        );
+        expect(wrapper).toMatchSnapshot();
+    });
     test('should correctly render a float field', () => {
         const wrapper = shallow(
             <MetadataField canEdit dataValue="value" onChange={onChange} onRemove={onRemove} type="float" />,

package/src/features/metadata-instance-fields/__tests__/__snapshots__/MetadataField.test.js.snap

@@ -37,6 +37,13 @@ exports[`features/metadata-instance-editor/fields/MetadataField should correctly
 />
 `;
 
+exports[`features/metadata-instance-editor/fields/MetadataField should correctly render a taxonomy field - for the time being, in read-only mode 1`] = `
+<ReadOnlyMetadataField
+  dataValue="value"
+  type="taxonomy"
+/>
+`;
+
 exports[`features/metadata-instance-editor/fields/MetadataField should correctly render a text field 1`] = `
 <TextMetadataField
   dataValue="value"