Fix dialogs closing on mutation error instead of showing submission errors

Use mutateAsync instead of mutate and move dialog close to onSuccess,
so SubmissionError propagates back to withSubmissionError in forms.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: fushar

judgels-client/src/routes/admin/archives/ArchiveCreateDialog/ArchiveCreateDialog.jsx

@@ -29,13 +29,13 @@ export function ArchiveCreateDialog() {
     </>
   );
 
-  const createArchive = data => {
-    createArchiveMutation.mutate(data, {
+  const createArchive = async data => {
+    await createArchiveMutation.mutateAsync(data, {
       onSuccess: () => {
+        setIsDialogOpen(false);
         toastActions.showSuccessToast('Archive created.');
       },
     });
-    setIsDialogOpen(false);
   };
 
   return (

judgels-client/src/routes/admin/archives/ArchiveEditDialog/ArchiveEditDialog.jsx

@@ -21,13 +21,13 @@ export function ArchiveEditDialog({ archive, isOpen, onCloseDialog }) {
     </>
   );
 
-  const updateArchive = data => {
-    updateArchiveMutation.mutate(data, {
+  const updateArchive = async data => {
+    await updateArchiveMutation.mutateAsync(data, {
       onSuccess: () => {
+        onCloseDialog();
         toastActions.showSuccessToast('Archive updated.');
       },
     });
-    onCloseDialog();
   };
 
   const initialValues = archive && {

judgels-client/src/routes/admin/chapters/ChapterCreateDialog/ChapterCreateDialog.jsx

@@ -29,13 +29,13 @@ export function ChapterCreateDialog() {
     </>
   );
 
-  const createChapter = data => {
-    createChapterMutation.mutate(data, {
+  const createChapter = async data => {
+    await createChapterMutation.mutateAsync(data, {
       onSuccess: () => {
+        setIsDialogOpen(false);
         toastActions.showSuccessToast('Chapter created.');
       },
     });
-    setIsDialogOpen(false);
   };
 
   return (

judgels-client/src/routes/admin/chapters/ChapterEditDialog/ChapterEditDialog.jsx

@@ -21,13 +21,13 @@ export function ChapterEditDialog({ chapter, isOpen, onCloseDialog }) {
     </>
   );
 
-  const updateChapter = data => {
-    updateChapterMutation.mutate(data, {
+  const updateChapter = async data => {
+    await updateChapterMutation.mutateAsync(data, {
       onSuccess: () => {
+        onCloseDialog();
         toastActions.showSuccessToast('Chapter updated.');
       },
     });
-    onCloseDialog();
   };
 
   const initialValues = chapter && {

judgels-client/src/routes/admin/chapters/ChapterLessonEditDialog/ChapterLessonEditDialog.jsx

@@ -32,14 +32,14 @@ export function ChapterLessonEditDialog({ isOpen, chapter, onCloseDialog }) {
     setIsEditing(prev => !prev);
   };
 
-  const updateLessons = data => {
+  const updateLessons = async data => {
     const lessons = deserializeLessons(data.lessons);
-    setLessonsMutation.mutate(lessons, {
+    await setLessonsMutation.mutateAsync(lessons, {
       onSuccess: () => {
+        setIsEditing(false);
         toastActions.showSuccessToast('Chapter lessons updated.');
       },
     });
-    setIsEditing(false);
   };
 
   const renderDialogContent = () => {

judgels-client/src/routes/admin/chapters/ChapterProblemEditDialog/ChapterProblemEditDialog.jsx

@@ -33,14 +33,14 @@ export function ChapterProblemEditDialog({ isOpen, chapter, onCloseDialog }) {
     setIsEditing(prev => !prev);
   };
 
-  const updateProblems = data => {
+  const updateProblems = async data => {
     const problems = deserializeProblems(data.problems);
-    setProblemsMutation.mutate(problems, {
+    await setProblemsMutation.mutateAsync(problems, {
       onSuccess: () => {
+        setIsEditing(false);
         toastActions.showSuccessToast('Chapter problems updated.');
       },
     });
-    setIsEditing(false);
   };
 
   const renderDialogContent = () => {

judgels-client/src/routes/admin/courses/CourseCreateDialog/CourseCreateDialog.jsx

@@ -29,13 +29,13 @@ export function CourseCreateDialog() {
     </>
   );
 
-  const createCourse = data => {
-    createCourseMutation.mutate(data, {
+  const createCourse = async data => {
+    await createCourseMutation.mutateAsync(data, {
       onSuccess: () => {
+        setIsDialogOpen(false);
         toastActions.showSuccessToast('Course created.');
       },
     });
-    setIsDialogOpen(false);
   };
 
   return (

judgels-client/src/routes/admin/courses/CourseEditDialog/CourseEditDialog.jsx

@@ -21,13 +21,13 @@ export function CourseEditDialog({ course, isOpen, onCloseDialog }) {
     </>
   );
 
-  const updateCourse = data => {
-    updateCourseMutation.mutate(data, {
+  const updateCourse = async data => {
+    await updateCourseMutation.mutateAsync(data, {
       onSuccess: () => {
+        onCloseDialog();
         toastActions.showSuccessToast('Course updated.');
       },
     });
-    onCloseDialog();
   };
 
   const initialValues = course && {

judgels-client/src/routes/admin/problemsets/ProblemSetCreateDialog/ProblemSetCreateDialog.jsx

@@ -29,8 +29,8 @@ export function ProblemSetCreateDialog() {
     </>
   );
 
-  const createProblemSet = data => {
-    createProblemSetMutation.mutate(
+  const createProblemSet = async data => {
+    await createProblemSetMutation.mutateAsync(
       {
         slug: data.slug,
         name: data.name,
@@ -40,11 +40,11 @@ export function ProblemSetCreateDialog() {
       },
       {
         onSuccess: () => {
+          setIsDialogOpen(false);
           toastActions.showSuccessToast('Problemset created.');
         },
       }
     );
-    setIsDialogOpen(false);
   };
 
   const initialValues = {

judgels-client/src/routes/admin/problemsets/ProblemSetEditDialog/ProblemSetEditDialog.jsx

@@ -21,8 +21,8 @@ export function ProblemSetEditDialog({ isOpen, problemSet, archiveSlug, onCloseD
     </>
   );
 
-  const updateProblemSet = data => {
-    updateProblemSetMutation.mutate(
+  const updateProblemSet = async data => {
+    await updateProblemSetMutation.mutateAsync(
       {
         slug: data.slug,
         name: data.name,
@@ -32,11 +32,11 @@ export function ProblemSetEditDialog({ isOpen, problemSet, archiveSlug, onCloseD
       },
       {
         onSuccess: () => {
+          onCloseDialog();
           toastActions.showSuccessToast('Problemset updated.');
         },
       }
     );
-    onCloseDialog();
   };
 
   const initialValues = problemSet && {