Treat a refused tool update as a failed save

frontend/src/settings/ToolConfig.test.tsx

@@ -185,6 +185,28 @@ describe('ToolConfig', () => {
     expect(alert?.className).toContain('text-destructive');
   });
 
+  it('shows a save the server refused as failed', async () => {
+    updateTool.mockResolvedValue({ ok: false, status: 400 });
+    await render({ ...userTool, customName: '' });
+    const name = container.querySelector<HTMLInputElement>(
+      'input[placeholder="settings.tools.customNamePlaceholder"]',
+    );
+    await act(async () => {
+      const setter = Object.getOwnPropertyDescriptor(
+        HTMLInputElement.prototype,
+        'value',
+      )?.set;
+      setter?.call(name, 'Renamed');
+      name?.dispatchEvent(new Event('input', { bubbles: true }));
+    });
+    await act(async () => {
+      buttonByText('settings.tools.save')?.click();
+    });
+    expect(
+      container.querySelector<HTMLElement>('[role="alert"]')?.textContent,
+    ).toBe('settings.tools.saveFailed');
+  });
+
   it('renders the API tool header actions as outline-primary pills', async () => {
     await render(apiTool);
     for (const label of [

frontend/src/settings/ToolConfig.tsx

@@ -257,12 +257,10 @@ export default function ToolConfig({
       actions: 'actions' in tool ? tool.actions : [],
       status: tool.status,
     };
-    if (tool.id) {
-      await userService.updateTool({ id: tool.id, ...payload }, token);
-      return;
-    }
-    const response = await userService.createTool(payload, token);
-    if (!response.ok) throw new Error('create failed');
+    const response = tool.id
+      ? await userService.updateTool({ id: tool.id, ...payload }, token)
+      : await userService.createTool(payload, token);
+    if (!response.ok) throw new Error('save failed');
   };
 
   const handleSaveChanges = async () => {