OUT-4109 | Handle SWR mutate rejections instead of letting them reach Sentry (#1439)

* OUT-4027 | Fix 500 when deleting a task whose label row is missing (#1397)

Author: priosshrsth

src/app/configure-tasks-app/ui/Subtemplates.tsx

@@ -53,7 +53,8 @@ export const Subtemplates = ({ template_id, token }: { template_id: string; toke
 
   const { mutate } = useSWRConfig()
 
-  const _debounceMutate = async (cacheKey: string) => await mutate(cacheKey)
+  const _debounceMutate = (cacheKey: string) =>
+    mutate(cacheKey).catch((error) => console.error('Failed to revalidate subtemplates:', error))
   const debounceMutate = useDebounce(_debounceMutate, 200)
 
   useEffect(() => {
@@ -68,7 +69,7 @@ export const Subtemplates = ({ template_id, token }: { template_id: string; toke
     debounceMutate(cacheKey)
   }, [activeTemplate?.subTaskTemplates])
 
-  const handleSubtemplateCreation = (payload: CreateTemplateRequest) => {
+  const handleSubtemplateCreation = async (payload: CreateTemplateRequest) => {
     const tempId = generateRandomString('temp-template')
     setOptimisticUpdates((prev) => [
       ...prev,
@@ -90,7 +91,7 @@ export const Subtemplates = ({ template_id, token }: { template_id: string; toke
     const optimisticData = sortTaskByDescendingOrder([...currentSubtemplates, tempSubtemplate])
 
     try {
-      mutate(
+      await mutate(
         cacheKey,
         async () => {
           const subTask = await createSubTemplate(token, template_id, payload)

src/app/detail/ui/ActivityWrapper.tsx

@@ -55,7 +55,8 @@ export const ActivityWrapper = ({
 
   useScrollToElement('commentId')
 
-  const _debounceMutate = async (cacheKey: string) => await mutate(cacheKey)
+  const _debounceMutate = (cacheKey: string) =>
+    mutate(cacheKey).catch((error) => console.error('Failed to revalidate activity logs:', error))
   const debounceMutate = useDebounce(_debounceMutate, 300)
 
   const shouldRefetchRef = useRef(true) //preventing double fetching from comment apis. Due to optimistic update revalidation, we are already fetching logs there. So no need to refetch in case for comment creation and deletion.
@@ -102,7 +103,7 @@ export const ActivityWrapper = ({
     const optimisticData = getOptimisticData(postCommentPayload, activities.data, tempLog)
 
     try {
-      mutate(
+      await mutate(
         cacheKey,
         async () => {
           shouldRefetchRef.current = false

src/app/detail/ui/Subtasks.tsx

@@ -67,7 +67,8 @@ export const Subtasks = ({
 
   const { mutate } = useSWRConfig()
 
-  const _debounceMutate = async (cacheKey: string) => await mutate(cacheKey)
+  const _debounceMutate = (cacheKey: string) =>
+    mutate(cacheKey).catch((error) => console.error('Failed to revalidate subtasks:', error))
   const debounceMutate = useDebounce(_debounceMutate, 200)
 
   useEffect(() => {
@@ -84,7 +85,7 @@ export const Subtasks = ({
     setLastUpdated(activeTask?.lastSubtaskUpdated)
   }, [activeTask?.lastSubtaskUpdated])
 
-  const handleSubTaskCreation = (payload: CreateTaskRequest) => {
+  const handleSubTaskCreation = async (payload: CreateTaskRequest) => {
     const tempId = generateRandomString('temp-task')
     setOptimisticUpdates((prev) => [
       ...prev,
@@ -104,7 +105,7 @@ export const Subtasks = ({
     )
     const optimisticData = subTasks?.tasks ? sortSubtasksByPriority([...subTasks.tasks, tempSubtask]) : [tempSubtask]
     try {
-      mutate(
+      await mutate(
         cacheKey,
         async () => {
           const subTask = await handleCreate(token, payload, { disableSubtaskTemplates: true })