From 2d4a4fcd618b08bae02bd07f72f3a64b5d768c5f Mon Sep 17 00:00:00 2001 From: Harrison Ngo Date: Thu, 26 Feb 2026 13:28:38 -0500 Subject: [PATCH] fix: wrap notifyGuardianOfAccessRequest in try-catch for graceful degradation Co-Authored-By: Claude --- .../runtime/routes/inbound-message-handler.ts | 20 +++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/assistant/src/runtime/routes/inbound-message-handler.ts b/assistant/src/runtime/routes/inbound-message-handler.ts index a5b39afc9eb..9e7aa00e459 100644 --- a/assistant/src/runtime/routes/inbound-message-handler.ts +++ b/assistant/src/runtime/routes/inbound-message-handler.ts @@ -275,14 +275,18 @@ export async function handleChannelInbound( // Notify the guardian about the access request so they can approve/deny. // Only fires when a guardian binding exists and no duplicate pending // request already exists for this requester. - notifyGuardianOfAccessRequest({ - canonicalAssistantId, - sourceChannel, - externalChatId, - senderExternalUserId: body.senderExternalUserId, - senderName: body.senderName, - senderUsername: body.senderUsername, - }); + try { + notifyGuardianOfAccessRequest({ + canonicalAssistantId, + sourceChannel, + externalChatId, + senderExternalUserId: body.senderExternalUserId, + senderName: body.senderName, + senderUsername: body.senderUsername, + }); + } catch (err) { + log.error({ err, sourceChannel, externalChatId }, 'Failed to notify guardian of access request'); + } return Response.json({ accepted: true, denied: true, reason: 'not_a_member' }); }