From 6cbb8cca9438c24812acd2ef218d1002c49c985b Mon Sep 17 00:00:00 2001 From: Teknium <127238744+teknium1@users.noreply.github.com> Date: Thu, 23 Jul 2026 12:01:35 -0700 Subject: [PATCH] fix(desktop): add MoA preset enabled toggle Salvaged from PR #59743. Original author email was malformed (sr@samirusani, not resolvable to a GitHub account), so the commit is re-authored with credit via trailer. Co-authored-by: Sami Rusani --- .../src/app/settings/model-settings.test.tsx | 21 +++++++++++++++++++ .../src/app/settings/model-settings.tsx | 9 ++++++++ 2 files changed, 30 insertions(+) diff --git a/apps/desktop/src/app/settings/model-settings.test.tsx b/apps/desktop/src/app/settings/model-settings.test.tsx index 985c6a018b6..1feb2848b04 100644 --- a/apps/desktop/src/app/settings/model-settings.test.tsx +++ b/apps/desktop/src/app/settings/model-settings.test.tsx @@ -435,4 +435,25 @@ describe('ModelSettings MoA preset editor', () => { vi.useRealTimers() } }) + + it('autosaves the selected preset when its enabled switch is toggled', async () => { + vi.useFakeTimers({ shouldAdvanceTime: true }) + + try { + await openReferenceEditor() + + fireEvent.click(screen.getByRole('switch', { name: 'Enabled' })) + await vi.advanceTimersByTimeAsync(700) + + expect(saveMoaModels).toHaveBeenCalledWith( + expect.objectContaining({ + presets: expect.objectContaining({ + default: expect.objectContaining({ enabled: false }) + }) + }) + ) + } finally { + vi.useRealTimers() + } + }) }) diff --git a/apps/desktop/src/app/settings/model-settings.tsx b/apps/desktop/src/app/settings/model-settings.tsx index c0f76175e49..52066eab35a 100644 --- a/apps/desktop/src/app/settings/model-settings.tsx +++ b/apps/desktop/src/app/settings/model-settings.tsx @@ -993,6 +993,15 @@ export function ModelSettings({ onMainModelChanged }: ModelSettingsProps) { ))} +