Skip to content

Handle comment activity refetch errors - #1317

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/OUT-3871-cursor-automated-triage-response-ff06
Draft

Handle comment activity refetch errors#1317
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/OUT-3871-cursor-automated-triage-response-ff06

Conversation

@cursor

@cursor cursor Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Await the ActivityWrapper comment creation SWR mutation so post-comment activity-log refetch failures are handled by the existing catch block instead of surfacing as unhandled promise rejections.
  • Addresses the first-party frame seen in Sentry issue TASKS-97 (src/app/detail/ui/ActivityWrapper.tsx -> src/utils/fetcher.ts) after a successful comment post/navigation in Safari.

Testing Criteria

  • yarn lint:check passed.
  • yarn tsc was attempted, but the repo currently fails before this change on missing SVG module declarations/imports in src/icons/index.ts (for example ./inreview.svg, ./humans.svg, and many other SVG imports).

Notes

Impact & Surface Area of Change

  • Comment creation on the task detail activity section.
  • Optimistic SWR mutation error handling for activity-log re-fetch after comment submission.
Open in Web View Automation 

Co-authored-by: Neil Raina <makeitraina@users.noreply.github.com>
@linear-code

linear-code Bot commented Jun 12, 2026

Copy link
Copy Markdown

OUT-3871

@vercel

vercel Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tasks-app Ready Ready Preview, Comment Jun 12, 2026 3:26pm

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant