From 2e4a4caae8d19f514dd14e394a80c830a60c3558 Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Sat, 22 Aug 2026 04:33:31 +0000 Subject: [PATCH] fix(POR-22659): handle platform LineItems validation errors in notification-center Wrap notification fetch in try/catch so platform billing validation errors (e.g. LineItems total below minimum) show a user-friendly SilentError instead of an unhandled request error in Sentry. Co-authored-by: Neil Raina --- src/app/notification-center/page.tsx | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/src/app/notification-center/page.tsx b/src/app/notification-center/page.tsx index e39b77851..57cdada72 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,24 @@ 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 }