Skip to content

Handle null Copilot notification responses#1274

Draft
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/OUT-3801-cursor-automated-triage-response-7dda
Draft

Handle null Copilot notification responses#1274
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/OUT-3801-cursor-automated-triage-response-7dda

Conversation

@cursor

@cursor cursor Bot commented May 30, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Treat null Copilot notification responses as expected for email-only delivery targets.
  • Keep in-product notification deliveries strict by throwing a clear error if Copilot returns no notification resource.
  • Guard notification persistence paths so nullable email-only responses are not written as notification records.
  • Add focused coverage for null email-only and null in-product responses.

Testing Criteria

  • yarn test src/utils/CopilotAPI.test.ts --runInBand
  • yarn lint:check (passes with existing warnings)
  • yarn tsc --noEmit --pretty false (blocked by pre-existing SVG module declaration errors in src/icons/index.ts)

Notes

  • Investigated Linear OUT-3801 / Sentry TASKS-8R. Sentry events are from the tasks project and the issue was created in Triage.
  • Linear MCP authentication is expired, so I could not post the requested ticket comment from this environment.

Impact & Surface Area of Change

  • Affects Copilot notification creation and downstream notification persistence paths.
  • Email-only reminder deliveries can now complete without raising a ZodError when Copilot returns null instead of a notification resource.
  • In-product notification flows still require a persisted Copilot notification object and will fail loudly if Copilot returns none.
Open in Web View Automation 

cursoragent and others added 2 commits May 30, 2026 00:12
Co-authored-by: Neil Raina <makeitraina@users.noreply.github.com>
Co-authored-by: Neil Raina <makeitraina@users.noreply.github.com>
@linear-code

linear-code Bot commented May 30, 2026

Copy link
Copy Markdown

OUT-3801

@vercel

vercel Bot commented May 30, 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 May 30, 2026 12:16am

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