Skip to content

fix: skip notification validation without Copilot app id - #1344

Draft
cursor[bot] wants to merge 3 commits into
mainfrom
cursor/OUT-3903-cursor-automated-triage-response-9451
Draft

fix: skip notification validation without Copilot app id#1344
cursor[bot] wants to merge 3 commits into
mainfrom
cursor/OUT-3903-cursor-automated-triage-response-9451

Conversation

@cursor

@cursor cursor Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Support COPILOT_APP_ID as the notification app identifier, with the existing COPILOT_APP_API_KEY value as fallback.
  • Skip client notification count validation when no app id is configured, avoiding raw ZodError crashes and preventing unsafe repair work against unfiltered notifications.
  • Filter Copilot notifications using the resolved app id without parsing an undefined env value.

Testing Criteria

  • yarn test src/utils/CopilotAPI.notifications.test.ts src/app/_fetchers/ValidateNotificationCountFetcher.test.tsx --runInBand covers missing app-id no-op behavior and configured app-id filtering.
  • yarn lint:check exits successfully with existing warnings only.
  • yarn tsc was run; it still fails on pre-existing missing SVG module declarations in src/icons/index.ts, with no diagnostics from the changed files.

Notes

  • Linear and Sentry MCP access were unavailable in this automation session (auth_revoked / needsAuth), so the issue source and stack trace could not be directly re-read from Linear/Sentry. The implemented fix targets the current Tasks code path that can produce the observed ZodError: [ title when notification app-id env is absent.

Impact & Surface Area of Change

  • Affects client notification count validation on /client and /api/notification/validate-count.
  • If no app id is configured, validation is skipped rather than throwing or mutating notification state.
  • If an app id is configured, existing validation behavior is preserved while filtering by that id.
Open in Web View Automation 

cursoragent and others added 3 commits June 24, 2026 13:03
Co-authored-by: Neil Raina <makeitraina@users.noreply.github.com>
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 Jun 24, 2026

Copy link
Copy Markdown

OUT-3903

@vercel

vercel Bot commented Jun 24, 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 24, 2026 1:06pm

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