From 208e1fb5bc9d846d48f570a7acc8488ec1885441 Mon Sep 17 00:00:00 2001 From: Flea Flicker Date: Fri, 27 Mar 2026 13:29:41 +0000 Subject: [PATCH] fix(gro-47): add non-null assertions on Drizzle RETURNING results Drizzle's update().returning() types the array element as T | undefined. After the if (!appt) guard, updated is still typed as possibly undefined because RETURNING can succeed with no rows. Add ! assertions since we already guard with the existence check. Co-Authored-By: Paperclip --- apps/api/src/routes/portal.ts | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/apps/api/src/routes/portal.ts b/apps/api/src/routes/portal.ts index 4d6284f..663b87c 100644 --- a/apps/api/src/routes/portal.ts +++ b/apps/api/src/routes/portal.ts @@ -135,10 +135,10 @@ portalRouter.post("/appointments/:id/confirm", async (c) => { .returning(); return c.json({ - id: updated.id, - confirmationStatus: updated.confirmationStatus, - confirmedAt: updated.confirmedAt, - updatedAt: updated.updatedAt, + id: updated!.id, + confirmationStatus: updated!.confirmationStatus, + confirmedAt: updated!.confirmedAt, + updatedAt: updated!.updatedAt, }); }); @@ -195,11 +195,11 @@ portalRouter.post("/appointments/:id/cancel", async (c) => { .returning(); return c.json({ - id: updated.id, - status: updated.status, - confirmationStatus: updated.confirmationStatus, - cancelledAt: updated.cancelledAt, - updatedAt: updated.updatedAt, + id: updated!.id, + status: updated!.status, + confirmationStatus: updated!.confirmationStatus, + cancelledAt: updated!.cancelledAt, + updatedAt: updated!.updatedAt, }); });