fix: disable extracting name from signature (#2486)

Author: malek10xdev

backend/src/services/signature/llm/index.ts

@@ -117,11 +117,11 @@ export const SignaturePrompt = {
             description:
               'Must always be "Person" as per schema.org type definition'
           },
-          name: {
-            type: 'string',
-            description:
-              'Full name exactly as written in the signature, preserving original spelling and capitalization'
-          },
+          // name: {
+          //   type: 'string',
+          //   description:
+          //     'Full name exactly as written in the signature, preserving original spelling and capitalization'
+          // },
           jobTitle: {
             type: 'string',
             description: 'Job title or position, only if explicitly stated'
@@ -158,7 +158,7 @@ export const SignaturePrompt = {
               'Array of social profile URLs (e.g., LinkedIn, Twitter); add https:// prefix if missing'
           }
         },
-        required: ['@type', 'name'],
+        required: ['@type'], // , 'name'
         additionalProperties: false
       }
     }
@@ -274,7 +274,7 @@ export class SignatureLLM implements ExtractSignature {
   private cleanOutput(signature: string, person: PersonLD): PersonLD | null {
     return removeFalsePositives(
       {
-        name: undefinedIfFalsy(parseString(person.name)),
+        // name: undefinedIfFalsy(parseString(person.name)),
         jobTitle: undefinedIfFalsy(parseString(person.jobTitle)),
         worksFor: undefinedIfFalsy(parseString(person.worksFor)),
         address: undefinedIfEmpty(parseStringArray(person.address) ?? []),

backend/src/workers/email-signature/handler.ts

@@ -129,8 +129,7 @@ export class EmailSignatureProcessor {
     body: string,
     messageDate: string
   ): Promise<void> {
-    const signature =
-      this.extractSignature(body);
+    const signature = this.extractSignature(body);
 
     if (!signature || !isUsefulSignatureContent(signature)) {
       this.logging.info('No signature found; skipping cache', {
@@ -211,12 +210,15 @@ export class EmailSignatureProcessor {
       const originalMessage = CleanQuotedForwardedReplies(text);
       const parsed = new EmailReplyParser().read(originalMessage);
       const sigFrag = parsed.fragments.filter((f) => f.isSignature()).pop();
-      return sigFrag?.getContent() ?? originalMessage.trim()
-        .split('\n')
-        .filter((l) => l.trim())
-        .slice(-4)
-        .join('\n');
-        
+      return (
+        sigFrag?.getContent() ??
+        originalMessage
+          .trim()
+          .split('\n')
+          .filter((l) => l.trim())
+          .slice(-4)
+          .join('\n')
+      );
     } catch (err) {
       this.logging.error('Failed to parse email body for signature', err);
       return null;

backend/src/workers/email-signature/utils.ts

@@ -37,7 +37,7 @@ export function isUsefulSignatureContent(signature: string): boolean {
 
   const words = text.split(/\s+/);
   const hasURL = /(https?:\/\/|www\.)\S+/i.test(text);
-  const hasDigits = /\d{2,}(?:[\s.,:\/-]?\d+)*/.test(text);
+  const hasDigits = /\d{2,}(?:[\s.,:/-]?\d+)*/.test(text);
   const hasSymbols = /[@+]/.test(text);
 
   const wordsMinMax =

micro-services/emails-fetcher/src/services/imap/ImapEmailsFetcher.ts

@@ -630,37 +630,21 @@ export default class ImapEmailsFetcher {
         encoding
       );
 
-      // decodeQuotedPrintable(
-      //   msg.bodyParts?.get(partId)?.toString() ?? '',
-      //   'utf-8'
-      // );
-
-      // if (headers && text?.length) {
-      //   try {
-      //     const { text: parsedText } = await simpleParser(
-      //       Buffer.concat([headers, text as Uint8Array<ArrayBufferLike>]),
-      //       {
-      //         skipHtmlToText: true,
-      //         skipTextToHtml: true,
-      //         skipImageLinks: true,
-      //         skipTextLinks: true
-      //       }
-      //     );
-      //     text = parsedText?.slice(0, this.EMAIL_TEXT_MAX_LENGTH) || '';
-      //   } catch {
-      //     text = '';
-      //   }
-      // }
-
       if (text.length && from && date) {
+        const { address, name } = from;
         await redisClient.xadd(
           this.signatureStream,
           '*',
           'message',
           JSON.stringify({
             type: 'email',
             data: {
-              header: { from, messageId, messageDate: date, rawHeader: header },
+              header: {
+                from: { address: address?.toLowerCase(), name },
+                messageId,
+                messageDate: date,
+                rawHeader: header
+              },
               body: text,
               seqNumber: seq,
               folderPath,