fix: sonarlint issues

Signed-off-by: Adam Setch <[email protected]>

Author: setchy

src/renderer/components/filters/TokenSearchInput.tsx

@@ -19,7 +19,7 @@ interface TokenSearchInputProps {
   onRemove: (token: SearchToken) => void;
 }
 
-const INPUT_KEY_EVENTS = ['Enter', 'Tab', ' ', ','];
+const INPUT_KEY_EVENTS: Set<string> = new Set(['Enter', 'Tab', ' ', ',']);
 
 export const TokenSearchInput: FC<TokenSearchInputProps> = ({
   label,
@@ -51,7 +51,7 @@ export const TokenSearchInput: FC<TokenSearchInputProps> = ({
   }
 
   function onKeyDown(e: React.KeyboardEvent<HTMLInputElement>) {
-    if (INPUT_KEY_EVENTS.includes(e.key)) {
+    if (INPUT_KEY_EVENTS.has(e.key)) {
       tryAddToken(e);
       setShowSuggestions(false);
     } else if (e.key === 'ArrowDown') {

src/renderer/components/metrics/MetricPill.tsx

@@ -6,14 +6,15 @@ import { Label, Stack, Text } from '@primer/react';
 import { type IconColor, Size } from '../../types';
 
 export interface IMetricPill {
-  key?: string;
   title: string;
   metric?: number;
   icon: Icon;
   color: IconColor;
 }
 
 export const MetricPill: FC<IMetricPill> = (props: IMetricPill) => {
+  const Icon = props.icon;
+
   return (
     <Label
       className="hover:bg-gitify-notification-pill-hover"
@@ -22,7 +23,7 @@ export const MetricPill: FC<IMetricPill> = (props: IMetricPill) => {
       variant="secondary"
     >
       <Stack align="center" direction="horizontal" gap="none">
-        <props.icon className={props.color} size={Size.XSMALL} />
+        <Icon className={props.color} size={Size.XSMALL} />
         {props.metric ? (
           <Text className="text-xxs px-1">{props.metric}</Text>
         ) : null}

src/renderer/components/settings/AppearanceSettings.tsx

@@ -59,7 +59,6 @@ export const AppearanceSettings: FC = () => {
           <FieldLabel label="Theme:" name="theme" />
           <Select
             data-testid="settings-theme"
-            id="theme"
             onChange={(evt) =>
               updateSetting('theme', evt.target.value as Theme)
             }

src/renderer/routes/__snapshots__/Settings.test.tsx.snap

@@ -77,7 +77,7 @@ export function actionsURL(repositoryURL: string, filters: string[]): Link {
   }
 
   // Note: the GitHub Actions UI cannot handle encoded '+' characters.
-  return url.toString().replace(/%2B/g, '+') as Link;
+  return url.toString().replaceAll('%2B', '+') as Link;
 }
 
 async function getDiscussionUrl(notification: Notification): Promise<Link> {

src/renderer/utils/helpers.ts

@@ -139,13 +139,13 @@ export async function getLatestReviewForReviewers(
       (review) => review.state === prReview.state,
     );
 
-    if (!reviewerFound) {
+    if (reviewerFound) {
+      reviewerFound.users.push(prReview.user.login);
+    } else {
       reviewers.push({
         state: prReview.state,
         users: [prReview.user.login],
       });
-    } else {
-      reviewerFound.users.push(prReview.user.login);
     }
   }
 

src/renderer/utils/notifications/handlers/pullRequest.ts

@@ -22,12 +22,13 @@ export const triggerNativeNotifications = (
         return accountNotifications.notifications;
       }
 
-      const accountPreviousNotificationsIds =
-        accountPreviousNotifications.notifications.map((item) => item.id);
+      const accountPreviousNotificationsIds = new Set<string>(
+        accountPreviousNotifications.notifications.map((item) => item.id),
+      );
 
       const accountNewNotifications = accountNotifications.notifications.filter(
         (item) => {
-          return !accountPreviousNotificationsIds.includes(`${item.id}`);
+          return !accountPreviousNotificationsIds.has(`${item.id}`);
         },
       );
 

src/renderer/utils/notifications/native.ts

@@ -42,7 +42,7 @@ function getNotifications(state: GitifyState) {
 export async function getAllNotifications(
   state: GitifyState,
 ): Promise<AccountNotifications[]> {
-  const responses = await Promise.all([...getNotifications(state)]);
+  const responses = await Promise.all(getNotifications(state));
 
   const notifications: AccountNotifications[] = await Promise.all(
     responses

src/renderer/utils/notifications/notifications.ts

@@ -16,8 +16,8 @@ export function removeNotifications(
   }
 
   const removeNotificationAccount = notificationsToRemove[0].account;
-  const removeNotificationIDs = notificationsToRemove.map(
-    (notification) => notification.id,
+  const removeNotificationIDs = new Set<string>(
+    notificationsToRemove.map((notification) => notification.id),
   );
 
   const accountIndex = allNotifications.findIndex(
@@ -31,7 +31,7 @@ export function removeNotifications(
     updatedNotifications[accountIndex] = {
       ...updatedNotifications[accountIndex],
       notifications: updatedNotifications[accountIndex].notifications.filter(
-        (notification) => !removeNotificationIDs.includes(notification.id),
+        (notification) => !removeNotificationIDs.has(notification.id),
       ),
     };
     return updatedNotifications;

src/renderer/utils/notifications/remove.ts

@@ -7,7 +7,7 @@ const MULTIPLIER = 2;
  * @returns zoomLevel -2 to 0.5
  */
 export const zoomPercentageToLevel = (percentage: number): number => {
-  if (typeof percentage === 'undefined') {
+  if (percentage === undefined) {
     return 0;
   }
 
@@ -20,7 +20,7 @@ export const zoomPercentageToLevel = (percentage: number): number => {
  * @returns percentage 0-150
  */
 export const zoomLevelToPercentage = (zoom: number): number => {
-  if (typeof zoom === 'undefined') {
+  if (zoom === undefined) {
     return 100;
   }