OUT-4109 | Catch debounced SWR revalidation rejections and drop token from fetcher errors

The unawaited mutate() fix missed the fire-and-forget debounceMutate(cacheKey)
paths, which reject the same way when revalidation fails, and it made two
non-async handlers await, so the branch did not typecheck.

The enriched error also embedded the full URL, which carries the Copilot
session token — that would have shipped a live credential to Sentry on every
failed fetch. Log the path only and attach the status, which swr-config's
onErrorRetry already reads to skip retrying 404s.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y4tF84tzJW7Bo1DehsDX22

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,

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.

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,

src/utils/fetcher.test.ts

@@ -17,18 +17,28 @@ describe('fetcher', () => {
     await expect(fetcher('/api/tasks/?token=test')).resolves.toEqual({ tasks: [] })
   })
 
-  it('throws an error with status, url, and response body for failed responses', async () => {
+  it('throws an error with status, path, and response body, without the token', async () => {
     global.fetch = jest.fn().mockResolvedValue({
       ok: false,
       status: 500,
       text: async () => 'Internal server error',
     })
 
-    await expect(fetcher('/api/tasks/?token=test')).rejects.toThrow(
-      'An error occurred while fetching the data. [500] /api/tasks/?token=test: Internal server error',
+    await expect(fetcher('/api/tasks/?token=secret-token')).rejects.toThrow(
+      'An error occurred while fetching the data. [500] /api/tasks/: Internal server error',
     )
   })
 
+  it('attaches the response status to the error so onErrorRetry can skip 404s', async () => {
+    global.fetch = jest.fn().mockResolvedValue({
+      ok: false,
+      status: 404,
+      text: async () => '',
+    })
+
+    await expect(fetcher('/api/tasks/?token=test')).rejects.toMatchObject({ status: 404 })
+  })
+
   it('returns undefined when url is null', async () => {
     await expect(fetcher(null)).resolves.toBeUndefined()
   })

src/utils/fetcher.ts

@@ -4,9 +4,13 @@ export const fetcher = async (url: string | null) => {
   const res = await fetch(url)
 
   if (!res.ok) {
-    const responseText = await res.text().catch(() => '')
-    const detail = responseText ? `: ${responseText.slice(0, 200)}` : ''
-    throw new Error(`An error occurred while fetching the data. [${res.status}] ${url}${detail}`)
+    const body = await res.text().catch(() => '')
+    // url carries the Copilot session token in its query string — keep it out of error reporting
+    const path = url.split('?')[0]
+    const error = new Error(
+      `An error occurred while fetching the data. [${res.status}] ${path}${body ? `: ${body.slice(0, 200)}` : ''}`,
+    )
+    throw Object.assign(error, { status: res.status })
   }
 
   return res.json()