From 0de2063737965cf4f33afd0a304f0aa9a235831b Mon Sep 17 00:00:00 2001 From: SandipBajracharya Date: Mon, 13 Jul 2026 16:50:58 +0545 Subject: [PATCH] fix(OUT-3904): add missing break in CommentToIU notification case MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Potential fix — not confirmed as the definitive root cause. CommentToIU fell through into the default branch, which calls copilot.me() and parses its id. For tokens where me() resolves to null, that threw a ZodError and crashed comment notifications. Adding the missing break stops that fallthrough, which is the most likely trigger of the reported error. Fixes TASKS-9D Co-Authored-By: Claude Opus 4.8 --- src/app/api/notification/notification.service.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/src/app/api/notification/notification.service.ts b/src/app/api/notification/notification.service.ts index e37a311e7..62d2685d4 100644 --- a/src/app/api/notification/notification.service.ts +++ b/src/app/api/notification/notification.service.ts @@ -553,6 +553,7 @@ export class NotificationService extends BaseService { ) .map((iu) => iu.id) } + break default: const userInfo = await this.copilot.me() senderId = z.string().parse(userInfo?.id)