From 704bbf2757bc267675daf5b9d8fd8a368f30e737 Mon Sep 17 00:00:00 2001 From: Joe Au-Yeung <65426560+joeauyeung@users.noreply.github.com> Date: Wed, 30 Jul 2025 20:47:38 -0400 Subject: [PATCH] chore: Refactor logs (#22824) * Refactor logs * Add specific info to log --- packages/app-store/_utils/getCalendar.ts | 1 - .../bookings/lib/handleNewBooking/ensureAvailableUsers.ts | 2 +- packages/lib/CalendarManager.ts | 5 ++++- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/packages/app-store/_utils/getCalendar.ts b/packages/app-store/_utils/getCalendar.ts index 399f92ac75..9fb865c203 100644 --- a/packages/app-store/_utils/getCalendar.ts +++ b/packages/app-store/_utils/getCalendar.ts @@ -49,7 +49,6 @@ export const getCalendar = async ( log.warn(`calendar of type ${calendarType} is not implemented`); return null; } - log.info("Got calendarApp", calendarApp.lib.CalendarService); const CalendarService = calendarApp.lib.CalendarService; return new CalendarService(credential); }; diff --git a/packages/features/bookings/lib/handleNewBooking/ensureAvailableUsers.ts b/packages/features/bookings/lib/handleNewBooking/ensureAvailableUsers.ts index 81e0ad84de..439257c3a0 100644 --- a/packages/features/bookings/lib/handleNewBooking/ensureAvailableUsers.ts +++ b/packages/features/bookings/lib/handleNewBooking/ensureAvailableUsers.ts @@ -222,7 +222,7 @@ const _ensureAvailableUsers = async ( if (!dateRanges.length) { loggerWithEventDetails.error( - `User does not have availability at this time.`, + `User ${user.id} does not have availability at this time.`, piiFreeInputDataForLogging ); return; diff --git a/packages/lib/CalendarManager.ts b/packages/lib/CalendarManager.ts index 1d03d886cf..581c8bafa3 100644 --- a/packages/lib/CalendarManager.ts +++ b/packages/lib/CalendarManager.ts @@ -277,7 +277,10 @@ export const getBusyCalendarTimes = async ( ); } } catch (e) { - log.warn(safeStringify(e)); + log.warn(`Error getting calendar availability`, { + selectedCalendarIds: selectedCalendars.map((calendar) => calendar.externalId), + error: safeStringify(e), + }); } return results.reduce((acc, availability) => acc.concat(availability), []); };