fix: assistant list UX improvements

src/containers/Assistants/AssistantDetail/AssistantDetail.test.tsx

@@ -20,10 +20,10 @@ const notificationSpy = vi.spyOn(Notification, 'setNotification');
 const renderAssistantDetail = (mocks: any = ASSISTANT_DETAIL_MOCKS, assistantId = '1') =>
   render(
     <MockedProvider mocks={mocks}>
-      <MemoryRouter initialEntries={[`/assistant-new/${assistantId}`]}>
+      <MemoryRouter initialEntries={[`/assistants/${assistantId}`]}>
         <Routes>
-          <Route path="/assistant-new/:assistantId" element={<AssistantDetail />} />
-          <Route path="/assistants-new" element={<div data-testid="assistants-page" />} />
+          <Route path="/assistants/:assistantId" element={<AssistantDetail />} />
+          <Route path="/assistants" element={<div data-testid="assistants-page" />} />
         </Routes>
       </MemoryRouter>
     </MockedProvider>
@@ -32,10 +32,10 @@ const renderAssistantDetail = (mocks: any = ASSISTANT_DETAIL_MOCKS, assistantId
 const renderCreateMode = (mocks: any = []) =>
   render(
     <MockedProvider mocks={mocks}>
-      <MemoryRouter initialEntries={['/assistant-new/add']}>
+      <MemoryRouter initialEntries={['/assistants/add']}>
         <Routes>
-          <Route path="/assistant-new/:assistantId" element={<AssistantDetail />} />
-          <Route path="/assistants-new" element={<div data-testid="assistants-page" />} />
+          <Route path="/assistants/:assistantId" element={<AssistantDetail />} />
+          <Route path="/assistants" element={<div data-testid="assistants-page" />} />
         </Routes>
       </MemoryRouter>
     </MockedProvider>
@@ -165,7 +165,7 @@ test('create mode shows Cancel and Save buttons', async () => {
   });
 });
 
-test('cancel button in create mode navigates to /assistants-new', async () => {
+test('cancel button in create mode navigates to /assistants', async () => {
   renderCreateMode();
 
   await waitFor(() => {
@@ -283,6 +283,32 @@ test('Leave button in unsaved changes modal closes the modal', async () => {
   });
 });
 
+test('Leave button switches to the pending version', async () => {
+  renderAssistantDetail();
+
+  await waitFor(() => {
+    expect(screen.getByText('Assistant-405db438 / Version 1')).toBeInTheDocument();
+  });
+
+  // Modify form to trigger unsaved changes
+  const textareas = screen.getAllByRole('textbox');
+  fireEvent.change(textareas[0], { target: { value: 'Changed' } });
+
+  // Click version 2 card — should open modal
+  fireEvent.click(screen.getAllByTestId('versionCard')[0]);
+
+  await waitFor(() => {
+    expect(screen.getByTestId('version-switch-leave')).toBeInTheDocument();
+  });
+
+  fireEvent.click(screen.getByTestId('version-switch-leave'));
+
+  await waitFor(() => {
+    expect(screen.queryByTestId('version-switch-leave')).not.toBeInTheDocument();
+    expect(screen.getByText('Assistant-405db438 / Version 2')).toBeInTheDocument();
+  });
+});
+
 test('save button in edit mode triggers notification on success', async () => {
   renderAssistantDetail(ASSISTANT_DETAIL_SAVE_MOCKS);
 
@@ -319,9 +345,9 @@ test('create mode save navigates to the new assistant page', async () => {
   render(
     <MockedProvider
       mocks={[createAssistantSuccessMock, getAssistant('99')]}>
-      <MemoryRouter initialEntries={['/assistant-new/add']}>
+      <MemoryRouter initialEntries={['/assistants/add']}>
         <Routes>
-          <Route path="/assistant-new/:assistantId" element={<AssistantDetail />} />
+          <Route path="/assistants/:assistantId" element={<AssistantDetail />} />
         </Routes>
       </MemoryRouter>
     </MockedProvider>

src/containers/Assistants/AssistantDetail/AssistantDetail.tsx

@@ -35,7 +35,7 @@ export const AssistantDetail = () => {
 
   useEffect(() => {
     if (!assistantId) {
-      navigate('/assistants-new');
+      navigate('/assistants');
     }
   }, [assistantId]);
 
@@ -62,7 +62,7 @@ export const AssistantDetail = () => {
 
   const handleSaved = (newId?: string) => {
     if (isCreateMode && newId) {
-      navigate(`/assistant-new/${newId}`);
+      navigate(`/assistants/${newId}`);
       return;
     }
     setHasUnsavedChanges(false);
@@ -93,7 +93,7 @@ export const AssistantDetail = () => {
     <div className={styles.Page} data-testid="assistantDetailContainer">
       <Heading
         formTitle={isCreateMode ? t('Create New Assistant') : (assistantData?.name ?? '')}
-        backLink="/assistants-new"
+        backLink="/assistants"
       />
 
       {!isCreateMode && assistantData?.assistantId && (
@@ -118,7 +118,7 @@ export const AssistantDetail = () => {
             vectorStore={null}
             newVersionInProgress={false}
             onSaved={handleSaved}
-            onCancel={() => navigate('/assistants-new')}
+            onCancel={() => navigate('/assistants')}
             createMode
           />
         </div>

src/containers/Assistants/AssistantDetail/ConfigEditor.test.tsx

@@ -4,7 +4,7 @@ import { vi } from 'vitest';
 
 import * as Notification from 'common/notification';
 import {
-  ASSISTANT_DETAIL_SAVE_MOCKS,
+  CONFIG_EDITOR_SAVE_MOCKS,
   createAssistantConfigMock,
   createAssistantErrorMock,
   mockVersions,
@@ -206,7 +206,7 @@ describe('ConfigEditor — edit mode', () => {
   });
 
   it('calls updateAssistant mutation and shows notification on save', async () => {
-    renderEdit({}, ASSISTANT_DETAIL_SAVE_MOCKS);
+    renderEdit({}, CONFIG_EDITOR_SAVE_MOCKS);
 
     const textareas = screen.getAllByRole('textbox');
     const instructionsField = textareas.find((el) => el.getAttribute('name') === 'instructions');
@@ -248,6 +248,21 @@ describe('ConfigEditor — edit mode', () => {
   });
 });
 
+describe('ConfigEditor — settings parsing', () => {
+  beforeEach(() => vi.clearAllMocks());
+
+  it('populates temperature correctly when settings is a JSON string', () => {
+    renderEdit({
+      version: {
+        ...mockVersion,
+        settings: JSON.stringify({ temperature: 0.7 }),
+      },
+    });
+
+    expect(screen.getByRole('sliderDisplay')).toHaveValue(0.7);
+  });
+});
+
 describe('ConfigEditor — expand instructions modal', () => {
   beforeEach(() => vi.clearAllMocks());
 

src/containers/Assistants/AssistantDetail/ConfigEditor.tsx

@@ -84,6 +84,7 @@ export const ConfigEditor = ({
     enableReinitialize: false,
     onSubmit: (values) => {
       const payload: Record<string, any> = {
+        name: assistantName,
         instructions: values.instructions,
         model: values.model?.label,
         temperature: values.temperature,
@@ -145,7 +146,8 @@ export const ConfigEditor = ({
   useEffect(() => {
     if (!version) return;
     const modelValue = version.model ? { id: version.model, label: version.model } : modelOptions[0];
-    const settings = version.settings ?? {};
+    const rawSettings = version.settings ?? {};
+    const settings = typeof rawSettings === 'string' ? JSON.parse(rawSettings) : rawSettings;
 
     formik.resetForm({
       values: {
@@ -340,7 +342,7 @@ export const ConfigEditor = ({
                     data-testid="saveVersionButton"
                     onClick={formik.submitForm}
                     loading={savingChanges || creating}
-                    disabled={newVersionInProgress || savingChanges || creating}
+                    disabled={newVersionInProgress || savingChanges || creating || !hasUnsavedChanges}
                   >
                     {t('Save')}
                   </Button>
@@ -372,11 +374,6 @@ export const ConfigEditor = ({
                 <Field key={field.name} {...field} />
               </div>
             ))}
-            {!createMode && (
-              <span className={styles.NoEvals} data-testid="noEvalsLink">
-                {t('No evals run. Start New Eval >')}
-              </span>
-            )}
           </div>
         </form>
 

src/containers/Assistants/AssistantList/AssistantList.module.css

@@ -26,11 +26,27 @@
   color: #1f2937;
 }
 
+.DisplayIdRow {
+  display: flex;
+  align-items: center;
+  gap: 2px;
+}
+
 .DisplayId {
   font-size: 0.75rem;
   color: #6b7280;
 }
 
+.CopyButton {
+  padding: 2px !important;
+  color: #6b7280;
+
+  svg {
+    width: 12px;
+    height: 12px;
+  }
+}
+
 .VersionBadge {
   color: #374151;
 }

src/containers/Assistants/AssistantList/AssistantList.test.tsx

@@ -16,11 +16,11 @@
 const renderAssistantList = (mocks: any[] = [filterAssistantsMock, countAssistantsMock]) =>
   render(
     <MockedProvider mocks={mocks}>
-      <MemoryRouter initialEntries={['/assistants-new']}>
+      <MemoryRouter initialEntries={['/assistants']}>
         <Routes>
-          <Route path="/assistants-new" element={<AssistantList />} />
-          <Route path="/assistant-new/add" element={<div data-testid="create-page" />} />
-          <Route path="/assistant-new/:id" element={<div data-testid="edit-page" />} />
+          <Route path="/assistants" element={<AssistantList />} />
+          <Route path="/assistants/add" element={<div data-testid="create-page" />} />
+          <Route path="/assistants/:id" element={<div data-testid="edit-page" />} />
         </Routes>
       </MemoryRouter>
     </MockedProvider>
@@ -37,7 +37,7 @@
 test('renders assistant rows with name and live version', async () => {
   renderAssistantList();
 
   await waitFor(() => {
     expect(screen.getByText('Assistant-1')).toBeInTheDocument();
     expect(screen.getByText('Version 3')).toBeInTheDocument();
     expect(screen.getByText('Assistant-2')).toBeInTheDocument();
@@ -48,29 +48,29 @@
 test('renders assistant display ID below name', async () => {
   renderAssistantList();
 
   await waitFor(() => {
     expect(screen.getByText('asst_abc123')).toBeInTheDocument();
   });
 });
 
-test('Create New Assistant button navigates to /assistant-new/add', async () => {
+test('Create New Assistant button navigates to /assistants/add', async () => {
   renderAssistantList();
 
   await waitFor(() => {
     expect(screen.getByText('Create New Assistant')).toBeInTheDocument();
   });
 
   fireEvent.click(screen.getByText('Create New Assistant'));
 
   await waitFor(() => {
     expect(screen.getByTestId('create-page')).toBeInTheDocument();
   });
 });
 
-test('edit icon navigates to /assistant-new/:id', async () => {
+test('edit icon navigates to /assistants/:id', async () => {
   renderAssistantList();
 
   await waitFor(() => {
     expect(screen.getAllByTestId('edit-icon')).toHaveLength(2);
   });
 
@@ -84,7 +84,7 @@
 test('clone icon opens confirmation dialog', async () => {
   renderAssistantList();
 
   await waitFor(() => {
     expect(screen.getAllByTestId('copy-icon')).toHaveLength(2);
   });
 
@@ -100,7 +100,7 @@
 test('clone dialog cancel closes without calling API', async () => {
   renderAssistantList([filterAssistantsMock, countAssistantsMock]);
 
   await waitFor(() => {
     expect(screen.getAllByTestId('copy-icon')).toHaveLength(2);
   });
 
@@ -120,7 +120,7 @@
 test('clone non-legacy assistant passes versionId to API', async () => {
   renderAssistantList([filterAssistantsMock, countAssistantsMock, cloneAssistantFromListMock]);
 
   await waitFor(() => {
     expect(screen.getAllByTestId('copy-icon')).toHaveLength(2);
   });
 
@@ -141,7 +141,7 @@
 test('clone legacy assistant does not pass versionId to API', async () => {
   renderAssistantList([filterAssistantsMock, countAssistantsMock, cloneLegacyAssistantFromListMock]);
 
   await waitFor(() => {
     expect(screen.getAllByTestId('copy-icon')).toHaveLength(2);
   });
 
@@ -162,7 +162,7 @@
 test('clone API returns errors shows error message', async () => {
   renderAssistantList([filterAssistantsMock, countAssistantsMock, cloneAssistantFromListErrorMock]);
 
   await waitFor(() => {
     expect(screen.getAllByTestId('copy-icon')).toHaveLength(2);
   });
 

src/containers/Assistants/AssistantList/AssistantList.tsx

@@ -5,9 +5,13 @@ import { useMutation } from '@apollo/client';
 import dayjs from 'dayjs';
 import relativeTime from 'dayjs/plugin/relativeTime';
 
+import { IconButton, Tooltip } from '@mui/material';
+
 import EditIcon from 'assets/images/icons/Edit.svg?react';
 import CopyIcon from 'assets/images/icons/Settings/Copy.svg?react';
 
+import { copyToClipboard } from 'common/utils';
+
 import { FILTER_ASSISTANTS, GET_ASSISTANTS_COUNT } from 'graphql/queries/Assistant';
 import { CLONE_ASSISTANT, DELETE_ASSISTANT } from 'graphql/mutations/Assistant';
 import { List } from 'containers/List/List';
@@ -22,7 +26,22 @@ dayjs.extend(relativeTime);
 const getAssistantName = (name: string, assistantDisplayId: string) => (
   <div className={styles.NameCell}>
     <span className={styles.Name}>{name}</span>
-    <span className={styles.DisplayId}>{assistantDisplayId}</span>
+    <span className={styles.DisplayIdRow}>
+      <Tooltip title="Copy assistant ID" placement="top">
+        <IconButton
+          size="small"
+          className={styles.CopyButton}
+          onClick={(e) => {
+            e.stopPropagation();
+            copyToClipboard(assistantDisplayId);
+          }}
+          data-testid="copyAssistantId"
+        >
+          <CopyIcon />
+        </IconButton>
+      </Tooltip>
+      <span className={styles.DisplayId}>{assistantDisplayId}</span>
+    </span>
   </div>
 );
 
@@ -60,7 +79,7 @@ export const AssistantList = () => {
   const [cloneAssistant, { loading: cloning }] = useMutation(CLONE_ASSISTANT);
 
   const handleEdit = (id: string) => {
-    navigate(`/assistant-new/${id}`);
+    navigate(`/assistants/${id}`);
   };
 
   const handleCloneClick = (_id: string, item: any) => {
@@ -104,7 +123,7 @@ export const AssistantList = () => {
   const columnNames = [
     { name: 'name', label: t('Assistant Name') },
     { label: t('Live Version') },
-    { label: t('Last Updated') },
+    { name: 'updated_at', label: t('Last Updated'), sort: true, order: 'desc' },
     { label: t('Actions') },
   ];
 
@@ -140,14 +159,16 @@ export const AssistantList = () => {
         dialogMessage={t("You won't be able to use this assistant.")}
         {...queries}
         {...columnAttributes}
-        searchParameter={['name']}
+        searchParameter={['name', 'assistantId']}
         additionalAction={additionalAction}
         button={{
           show: true,
           label: t('Create New Assistant'),
-          action: () => navigate('/assistant-new/add'),
+          action: () => navigate('/assistants/add'),
         }}
         editSupport={false}
+        deleteModifier={{ variables: (id: string) => ({ deleteAssistantId: id }) }}
+        sortConfig={{ sortBy: 'updated_at', sortOrder: 'desc' }}
       />
 
       {cloneDialogOpen && selectedAssistant && (