Skip to content

Commit 68b151f

Browse files
arpandhakalclaude
andcommitted
fix(notifications): break CommentToIU so it doesn't fall through to default
getNotificationParties' CommentToIU case had no break and fell through to default, which strict-parses me().id and task.assigneeId — throwing (ZodError "Required") when me() returns null or the task is unassigned. The comment job only consumes recipientIds/senderCompanyId (both set in the CommentToIU case), so the fallthrough's values were unused anyway. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent a716d21 commit 68b151f

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

src/app/api/notification/notification.service.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -601,6 +601,7 @@ export class NotificationService extends BaseService {
601601
)
602602
.map((iu) => iu.id)
603603
}
604+
break
604605
default:
605606
const userInfo = await this.copilot.me()
606607
senderId = z.string().parse(userInfo?.id)

0 commit comments

Comments
 (0)