refactor: handler enhancements

Signed-off-by: Adam Setch <adam.setch@outlook.com>

Author: setchy

src/renderer/components/notifications/__snapshots__/AccountNotifications.test.tsx.snap

@@ -54,4 +54,78 @@ describe('renderer/utils/notifications/handlers/default.ts', () => {
       expect(defaultHandler.iconColor(subject)).toBe(expected);
     });
   });
+
+  describe('formattedNotificationType', () => {
+    it('formats state and type with proper casing and spacing', () => {
+      const notification = partialMockNotification({
+        title: 'Sample',
+        type: 'PullRequest',
+        state: 'open',
+      });
+
+      expect(defaultHandler.formattedNotificationType(notification)).toBe(
+        'Open Pull Request',
+      );
+    });
+
+    it('handles missing state (null) gracefully', () => {
+      const notification = partialMockNotification({
+        title: 'Sample',
+        type: 'Issue',
+        state: null,
+      });
+
+      expect(defaultHandler.formattedNotificationType(notification)).toBe(
+        'Issue',
+      );
+    });
+  });
+
+  describe('formattedNotificationNumber', () => {
+    it('returns formatted number when present', () => {
+      const notification = partialMockNotification({
+        title: 'Sample',
+        type: 'Issue',
+        state: 'open',
+      });
+      notification.subject.number = 42;
+      expect(defaultHandler.formattedNotificationNumber(notification)).toBe(
+        '#42',
+      );
+    });
+
+    it('returns empty string when number absent', () => {
+      const notification = partialMockNotification({
+        title: 'Sample',
+        type: 'Issue',
+        state: 'open',
+      });
+      expect(defaultHandler.formattedNotificationNumber(notification)).toBe('');
+    });
+  });
+
+  describe('formattedNotificationTitle', () => {
+    it('appends number in brackets when present', () => {
+      const notification = partialMockNotification({
+        title: 'Fix bug',
+        type: 'Issue',
+        state: 'open',
+      });
+      notification.subject.number = 101;
+      expect(defaultHandler.formattedNotificationTitle(notification)).toBe(
+        'Fix bug [#101]',
+      );
+    });
+
+    it('returns title unchanged when number missing', () => {
+      const notification = partialMockNotification({
+        title: 'Improve docs',
+        type: 'Issue',
+        state: 'open',
+      });
+      expect(defaultHandler.formattedNotificationTitle(notification)).toBe(
+        'Improve docs',
+      );
+    });
+  });
 });

src/renderer/utils/notifications/handlers/default.test.ts

@@ -36,5 +36,6 @@ export function formatForDisplay(text: string[]): string {
     .replace(/\w+/g, (word) => {
       // Convert to proper case (capitalize first letter of each word)
       return word.charAt(0).toUpperCase() + word.slice(1).toLowerCase();
-    });
+    })
+    .trim();
 }