fix(automl/configure, autorag/configure): address coderabbit

Author: Daniel Duong

packages/automl/frontend/src/app/pages/__tests__/AutomlConfigurePage.spec.tsx

@@ -32,13 +32,13 @@ jest.mock('mod-arch-core', () => ({
 }));
 
 jest.mock('~/app/hooks/mutations', () => ({
-  usePipelineRunsMutation: jest.fn(() => ({
+  useCreatePipelineRunMutation: jest.fn(() => ({
     mutateAsync: mockMutateAsync,
   })),
 }));
 
 jest.mock('~/app/hooks/queries', () => ({
-  useFilesQuery: jest.fn(() => ({
+  useS3GetFileSchemaQuery: jest.fn(() => ({
     data: [
       { name: 'column1', type: 'string' },
       { name: 'column2', type: 'int64' },
@@ -404,9 +404,13 @@ describe('AutomlConfigurePage', () => {
       const backButton = await screen.findByRole('button', { name: 'Back' });
       await user.click(backButton);
 
+      // Re-query inputs after navigation (DOM elements are remounted)
+      const nameInputAfterBack = await screen.findByLabelText(/Name/i);
+      const descriptionInputAfterBack = await screen.findByLabelText(/Description/i);
+
       // Verify form data is preserved
-      expect(nameInput).toHaveValue('Preserved Name');
-      expect(descriptionInput).toHaveValue('Preserved Description');
+      expect(nameInputAfterBack).toHaveValue('Preserved Name');
+      expect(descriptionInputAfterBack).toHaveValue('Preserved Description');
     });
   });
 

packages/automl/frontend/src/app/schemas/configure.schema.ts

@@ -22,7 +22,7 @@ function createConfigureSchema() {
   return createSchema({
     schema: z.object({
       // Common fields
-      display_name: z.string().default(''),
+      display_name: z.string().min(1).default(''),
       description: z.string().default('').optional(),
       task_type: z.enum(TASK_TYPES).default(TASK_TYPE_BINARY),
       train_data_secret_name: z.string().min(1).default(''),

packages/autorag/frontend/src/app/components/configure/AutoragConfigure.tsx

@@ -194,9 +194,8 @@ function AutoragConfigure(): React.JSX.Element {
                                   value={selectedSecret?.uuid}
                                   onChange={(secret) => {
                                     if (!secret) {
-                                      setValue('input_data_secret_name', '', {
-                                        shouldValidate: true,
-                                      });
+                                      setSelectedSecret(undefined);
+                                      onChange('');
                                       return;
                                     }