From ae4ff369979df4e9f8bfbc3dcdea5c6dd81b6622 Mon Sep 17 00:00:00 2001 From: pedrofrxncx Date: Fri, 4 Sep 2026 03:43:57 -0300 Subject: [PATCH] refactor(task-board): remove redundant defensive code from useTaskBoardItemPrs The initialData function's cast to 'TaskBoardItemPr[] | undefined' already includes undefined as a valid type, making the trailing '?? undefined' a no-op. Remove it to simplify the code and improve readability. --- apps/web/src/hooks/use-task-board-item-prs.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/apps/web/src/hooks/use-task-board-item-prs.ts b/apps/web/src/hooks/use-task-board-item-prs.ts index f3d5cf37e3..b4ba152c7b 100644 --- a/apps/web/src/hooks/use-task-board-item-prs.ts +++ b/apps/web/src/hooks/use-task-board-item-prs.ts @@ -51,9 +51,9 @@ export function useTaskBoardItemPrs(itemId: string | undefined) { // cached data, so the parse doesn't run on every render. initialData: () => itemId - ? ((readCachedTaskPrs(locator, itemId)?.data as + ? (readCachedTaskPrs(locator, itemId)?.data as | TaskBoardItemPr[] - | undefined) ?? undefined) + | undefined) : undefined, initialDataUpdatedAt: () => itemId ? readCachedTaskPrs(locator, itemId)?.updatedAt : undefined,