From f7064bce5499c73335d476c73b466ebaaf9f8c1e Mon Sep 17 00:00:00 2001
From: Adam Setch <adam.setch@outlook.com>
Date: Mon, 5 Jan 2026 08:18:21 -0500
Subject: [PATCH] refactor(group): isGroupByDate helper util

Signed-off-by: Adam Setch <adam.setch@outlook.com>
---
 .../components/notifications/NotificationHeader.tsx |  7 +++----
 .../components/notifications/NotificationRow.tsx    |  7 +++----
 src/renderer/utils/notifications/group.test.ts      | 13 +++++++++++++
 src/renderer/utils/notifications/group.ts           |  7 +++++++
 4 files changed, 26 insertions(+), 8 deletions(-)

diff --git a/src/renderer/components/notifications/NotificationHeader.tsx b/src/renderer/components/notifications/NotificationHeader.tsx
index 55c6178a1..f2941f70e 100644
--- a/src/renderer/components/notifications/NotificationHeader.tsx
+++ b/src/renderer/components/notifications/NotificationHeader.tsx
@@ -3,9 +3,10 @@ import type { FC, MouseEvent } from 'react';
 import { Stack } from '@primer/react';
 
 import { useAppContext } from '../../context/App';
-import { type GitifyNotification, GroupBy, Opacity, Size } from '../../types';
+import { type GitifyNotification, Opacity, Size } from '../../types';
 import { cn } from '../../utils/cn';
 import { openRepository } from '../../utils/links';
+import { isGroupByDate } from '../../utils/notifications/group';
 import { AvatarWithFallback } from '../avatars/AvatarWithFallback';
 
 interface NotificationHeaderProps {
@@ -23,10 +24,8 @@ export const NotificationHeader: FC<NotificationHeaderProps> = ({
     ? `#${notification.subject.number}`
     : '';
 
-  const groupByDate = settings.groupBy === GroupBy.DATE;
-
   return (
-    groupByDate && (
+    isGroupByDate(settings) && (
       <div className="py-0.5">
         <Stack align="center" direction="horizontal" gap="condensed">
           <button
diff --git a/src/renderer/components/notifications/NotificationRow.tsx b/src/renderer/components/notifications/NotificationRow.tsx
index 088e6b26f..be8fb33e3 100644
--- a/src/renderer/components/notifications/NotificationRow.tsx
+++ b/src/renderer/components/notifications/NotificationRow.tsx
@@ -4,10 +4,11 @@ import { BellSlashIcon, CheckIcon, ReadIcon } from '@primer/octicons-react';
 import { Stack, Text, Tooltip } from '@primer/react';
 
 import { useAppContext } from '../../context/App';
-import { type GitifyNotification, GroupBy, Opacity, Size } from '../../types';
+import { type GitifyNotification, Opacity, Size } from '../../types';
 import { cn } from '../../utils/cn';
 import { isMarkAsDoneFeatureSupported } from '../../utils/features';
 import { openNotification } from '../../utils/links';
+import { isGroupByDate } from '../../utils/notifications/group';
 import { createNotificationHandler } from '../../utils/notifications/handlers';
 import { HoverButton } from '../primitives/HoverButton';
 import { HoverGroup } from '../primitives/HoverGroup';
@@ -68,8 +69,6 @@ export const NotificationRow: FC<NotificationRowProps> = ({
   const notificationNumber = handler.formattedNotificationNumber(notification);
   const notificationTitle = handler.formattedNotificationTitle(notification);
 
-  const groupByDate = settings.groupBy === GroupBy.DATE;
-
   const isNotificationRead = !notification.unread;
 
   return (
@@ -125,7 +124,7 @@ export const NotificationRow: FC<NotificationRowProps> = ({
               className={cn(
                 'text-xxs ml-auto mr-2',
                 Opacity.READ,
-                (groupByDate || !settings.showNumber) && 'hidden',
+                (isGroupByDate(settings) || !settings.showNumber) && 'hidden',
               )}
             >
               {notificationNumber}
diff --git a/src/renderer/utils/notifications/group.test.ts b/src/renderer/utils/notifications/group.test.ts
index 33d3e0c09..9623dca7e 100644
--- a/src/renderer/utils/notifications/group.test.ts
+++ b/src/renderer/utils/notifications/group.test.ts
@@ -5,10 +5,23 @@ import { GroupBy } from '../../types';
 import {
   getFlattenedNotificationsByRepo,
   groupNotificationsByRepository,
+  isGroupByDate,
   isGroupByRepository,
 } from './group';
 
 describe('renderer/utils/notifications/group.ts', () => {
+  describe('isGroupByDate', () => {
+    it('returns true when groupBy is DATE', () => {
+      const settings = { ...mockSettings, groupBy: GroupBy.DATE };
+      expect(isGroupByDate(settings)).toBe(true);
+    });
+
+    it('returns false when groupBy is REPOSITORY', () => {
+      const settings = { ...mockSettings, groupBy: GroupBy.REPOSITORY };
+      expect(isGroupByDate(settings)).toBe(false);
+    });
+  });
+
   describe('isGroupByRepository', () => {
     it('returns true when groupBy is REPOSITORY', () => {
       const settings = { ...mockSettings, groupBy: GroupBy.REPOSITORY };
diff --git a/src/renderer/utils/notifications/group.ts b/src/renderer/utils/notifications/group.ts
index 951de0b5c..d0265cfbb 100644
--- a/src/renderer/utils/notifications/group.ts
+++ b/src/renderer/utils/notifications/group.ts
@@ -1,5 +1,12 @@
 import type { GitifyNotification, SettingsState } from '../../types';
 
+/**
+ * Returns true when settings say to group by date.
+ */
+export function isGroupByDate(settings: SettingsState) {
+  return settings.groupBy === 'DATE';
+}
+
 /**
  * Returns true when settings say to group by repository.
  */