From 73e04398d1bdbb5de54a5c8b51b36deb614692de Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Sat, 22 Aug 2026 04:06:39 +0000 Subject: [PATCH] fix(POR-22648): handle platform apps entity errors in notification-center Wrap notification fetch in try/catch so platform errors like 'No entity type for apps found' show a user-friendly SilentError instead of an unhandled request error in Sentry. Also use safeParse for CTA params and return early when token payload lacks notificationId. Co-authored-by: Neil Raina --- src/app/notification-center/page.tsx | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/src/app/notification-center/page.tsx b/src/app/notification-center/page.tsx index e39b77851..364d5b8c5 100644 --- a/src/app/notification-center/page.tsx +++ b/src/app/notification-center/page.tsx @@ -9,9 +9,9 @@ async function getNotificationDetail(token: string) { const copilot = new CopilotAPI(token) const tokenPayload = await copilot.getTokenPayload() - if (!tokenPayload) throw new Error('Failed to get token payload') + if (!tokenPayload?.notificationId) return null - return await copilot.getIUNotification(z.string().parse(tokenPayload.notificationId), tokenPayload.workspaceId) // notification "id" is expected in tokenPayload + return await copilot.getIUNotification(tokenPayload.notificationId, tokenPayload.workspaceId) } export default async function NotificationCenter(props: { searchParams: Promise<{ token: string }> }) { @@ -21,13 +21,22 @@ export default async function NotificationCenter(props: { searchParams: Promise< return } - const notificationDetail = await getNotificationDetail(token) + let notificationDetail + try { + notificationDetail = await getNotificationDetail(token) + } catch (error) { + console.warn('notification-center: failed to load notification', error) + return + } + if (!notificationDetail) return - const params = NotificationInProductCtaParamsSchema.parse(notificationDetail.deliveryTargets?.inProduct?.ctaParams) + const params = NotificationInProductCtaParamsSchema.safeParse(notificationDetail.deliveryTargets?.inProduct?.ctaParams) + if (!params.success) { + return + } - redirectIfTaskCta({ ...params, ...searchParams }, UserType.INTERNAL_USER, true) + redirectIfTaskCta({ ...params.data, ...searchParams }, UserType.INTERNAL_USER, true) - // Silent Error is shown if redirect fails. Only possible reason for redirect to not work can be of the taskId not found return }