fix(schedule-actions): navigate to list before refetching deleted schedule

Skip describeSchedule invalidation on delete so the detail page does not
404-refetch and hit the error boundary before router.push completes.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: Assem-Uber

src/views/schedule-actions/config/schedule-actions.config.ts

@@ -80,12 +80,12 @@ const deleteScheduleActionConfig: ScheduleAction<DeleteScheduleResponse> = {
   httpMethod: 'DELETE',
   renderSuccessMessage: () => 'Schedule deleted.',
   onSuccess: ({ queryClient, params, router }) => {
-    queryClient.invalidateQueries({
-      queryKey: ['listSchedules', { domain: params.domain, cluster: params.cluster }],
-    });
     router.push(
       `/domains/${encodeURIComponent(params.domain)}/${encodeURIComponent(params.cluster)}/schedules`
     );
+    queryClient.invalidateQueries({
+      queryKey: ['listSchedules', { domain: params.domain, cluster: params.cluster }],
+    });
   },
 };
 

src/views/schedule-actions/schedule-actions-modal-content/__tests__/schedule-actions-modal-content.test.tsx

@@ -1,4 +1,5 @@
 import { HttpResponse } from 'msw';
+import { QueryClient } from '@tanstack/react-query';
 
 import { render, screen, userEvent, waitFor } from '@/test-utils/rtl';
 
@@ -135,6 +136,29 @@ describe(ScheduleActionsModalContent.name, () => {
       );
       expect(mockOnClose).toHaveBeenCalled();
     });
+
+    it('does not invalidate describeSchedule after delete', async () => {
+      const invalidateQueriesSpy = jest.spyOn(
+        QueryClient.prototype,
+        'invalidateQueries'
+      );
+
+      const { user, waitForRequest } = setup({
+        actionConfig: mockScheduleActionsConfig[2],
+      });
+
+      await user.click(
+        screen.getByRole('button', { name: 'Delete schedule' })
+      );
+
+      await waitForRequest();
+
+      expect(invalidateQueriesSpy).not.toHaveBeenCalledWith(
+        expect.objectContaining({
+          queryKey: ['describeSchedule', mockScheduleParams],
+        })
+      );
+    });
   });
 });
 

src/views/schedule-actions/schedule-actions-modal-content/schedule-actions-modal-content.tsx

@@ -68,11 +68,14 @@ export default function ScheduleActionsModalContent<
         }).then((res) => res.json() as Result);
       },
       onSuccess: (result, mutationParams) => {
-        queryClient.invalidateQueries({
-          queryKey: ['describeSchedule', params],
-        });
-
-        action.onSuccess?.({ queryClient, params, router });
+        if (action.id === 'delete') {
+          action.onSuccess?.({ queryClient, params, router });
+        } else {
+          queryClient.invalidateQueries({
+            queryKey: ['describeSchedule', params],
+          });
+          action.onSuccess?.({ queryClient, params, router });
+        }
 
         onCloseModal();
         enqueue({