fix: minor issues

Author: xabg2

src/App.tsx

@@ -24,7 +24,7 @@ function App() {
   };
 
   useEffect(() => {
-    initializeUser();
+    void initializeUser();
   }, []);
 
   return (

src/components/user-chip/index.tsx

@@ -14,7 +14,8 @@ const UserChip = ({ avatar, name, email, onRemove }: UserChipProps) => {
   const [position, setPosition] = useState<{ top: number; left: number } | null>(null);
   const ref = useRef<HTMLDivElement>(null);
   const tooltipId = useId();
-  const userName = name.split(' ')[0] || email.split('@')[0];
+  const normalizedName = name.trim();
+  const userName = normalizedName ? normalizedName.split(/\s+/)[0] : email.split('@')[0];
 
   const handleMouseEnter = () => {
     const rect = ref.current?.getBoundingClientRect();

src/errors/mail/index.ts

@@ -4,7 +4,6 @@ export class FetchMailboxesInfoError extends Error {
     public requestId?: string,
   ) {
     super('Error while fetching mailboxes info: ' + errorMsg);
-    this.requestId = requestId;
 
     Object.setPrototypeOf(this, FetchMailboxesInfoError.prototype);
   }
@@ -16,7 +15,6 @@ export class FetchListFolderError extends Error {
     public requestId?: string,
   ) {
     super('Error while fetching list folder: ' + errorMsg);
-    this.requestId = requestId;
 
     Object.setPrototypeOf(this, FetchListFolderError.prototype);
   }
@@ -28,7 +26,6 @@ export class FetchMessageError extends Error {
     public requestId?: string,
   ) {
     super('Error while fetching message: ' + errorMsg);
-    this.requestId = requestId;
 
     Object.setPrototypeOf(this, FetchMessageError.prototype);
   }
@@ -41,7 +38,6 @@ export class UpdateMailError extends Error {
     public requestId?: string,
   ) {
     super(`Error while updating mail when ${action}: ` + errorMsg);
-    this.requestId = requestId;
 
     Object.setPrototypeOf(this, UpdateMailError.prototype);
   }

src/features/mail/MailView.tsx

@@ -7,6 +7,7 @@ import Settings from './components/settings';
 import { useGetListFolderQuery, useGetMailMessageQuery, useMarkAsReadMutation } from '@/store/queries/mail/mail.query';
 import { DateService } from '@/services/date';
 import { DEFAULT_FOLDER_LIMIT } from '@/constants';
+import { ErrorService } from '@/services/error';
 
 interface MailViewProps {
   folder: FolderType;
@@ -40,10 +41,15 @@ const MailView = ({ folder }: MailViewProps) => {
 
     if (isRead) return;
 
-    await markAsRead({
-      emailId: id,
-      query,
-    });
+    try {
+      await markAsRead({
+        emailId: id,
+        query,
+      });
+    } catch (error) {
+      const err = ErrorService.instance.castError(error);
+      console.error(`Error while marking as read the email ${id}: `, err);
+    }
   };
 
   return (

src/services/sdk/mail/index.ts

@@ -33,6 +33,12 @@ export class MailService {
     return this.client.listEmails(query);
   }
 
+  /**
+   * Returns a specific email by its ID.
+   *
+   * @param {string} emailId - The ID of the email to fetch
+   * @returns A promise that resolves with an EmailResponse object
+   */
   async getMessage(emailId: string): Promise<EmailResponse> {
     return this.client.getEmail(emailId);
   }

src/store/queries/mail/mail.query.test.ts

@@ -1,5 +1,3 @@
-// store/queries/__tests__/storage.query.test.ts
-
 import { describe, test, expect, vi, beforeEach } from 'vitest';
 import { ErrorService } from '@/services/error';
 import { FetchListFolderError, FetchMailboxesInfoError, FetchMessageError, UpdateMailError } from '@/errors';

src/store/queries/mail/mail.query.ts

@@ -57,15 +57,20 @@ export const mailQuery = createApi({
         }
       },
       async onQueryStarted({ emailId, query }, { dispatch, queryFulfilled }) {
+        let shouldUpdateUnreadCount = false;
+
         const patchEmailList = dispatch(
           mailQuery.util.updateQueryData('getListFolder', query, (draft) => {
             const mail = draft.emails.find((m) => m.id === emailId);
             if (mail && !mail.isRead) {
               mail.isRead = true;
+              shouldUpdateUnreadCount = true;
             }
           }),
         );
 
+        if (!shouldUpdateUnreadCount) return;
+
         const patchMailboxes = dispatch(
           mailQuery.util.updateQueryData('getMailboxesInfo', undefined, (draft) => {
             const entry = draft.find((m) => m.type === query?.mailbox);