From 265d4967d6238589b47f1fa195e766279f6dd2c4 Mon Sep 17 00:00:00 2001
From: dominictb <tb-dominic@outlook.com>
Date: Mon, 5 Aug 2024 12:47:14 +0700
Subject: [PATCH] fix: no trimming for link name

---
 __tests__/ExpensiMark-HTML-test.js | 10 ++++++----
 lib/ExpensiMark.ts                 |  9 ++++-----
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/__tests__/ExpensiMark-HTML-test.js b/__tests__/ExpensiMark-HTML-test.js
index ab3cd854..f514229e 100644
--- a/__tests__/ExpensiMark-HTML-test.js
+++ b/__tests__/ExpensiMark-HTML-test.js
@@ -389,7 +389,8 @@ test('Test critical markdown style links', () => {
         '[link with [brackets] inside of it](https://google.com) ' +
         '[link with smart quotes ‘’“”](https://google.com) ' +
         '[link with someone@expensify.com email in it](https://google.com)' +
-        '[Localhost](http://a:3030)';
+        '[Localhost](http://a:3030)' + 
+        '[ name with rear spaces ](https://example.com)';
     const resultString =
         'Testing ' +
         '<a href="https://expensify.com" target="_blank" rel="noreferrer noopener"><del>strikethrough</del> <strong>bold</strong> <em>italic</em></a> ' +
@@ -407,7 +408,8 @@ test('Test critical markdown style links', () => {
         '<a href="https://google.com" target="_blank" rel="noreferrer noopener">link with [brackets] inside of it</a> ' +
         '<a href="https://google.com" target="_blank" rel="noreferrer noopener">link with smart quotes ‘’“”</a> ' +
         '<a href="https://google.com" target="_blank" rel="noreferrer noopener">link with someone@expensify.com email in it</a>' +
-        '<a href="http://a:3030" target="_blank" rel="noreferrer noopener">Localhost</a>';
+        '<a href="http://a:3030" target="_blank" rel="noreferrer noopener">Localhost</a>' +
+        '<a href="https://example.com" target="_blank" rel="noreferrer noopener"> name with rear spaces </a>';
     expect(parser.replace(testString)).toBe(resultString);
 });
 
@@ -1032,7 +1034,7 @@ test('Test markdown and url links with inconsistent starting and closing parens'
         '[Text text] more text (<a href="https://www.google.com" target="_blank" rel="noreferrer noopener">link [square brackets within] here</a>)' +
         '[Text text] more text (<a href="https://www.google.com" target="_blank" rel="noreferrer noopener">link (parenthesis within) here</a>)' +
         '[Text text] more text <a href="https://www.google.com" target="_blank" rel="noreferrer noopener">link here</a>' +
-        '[Text text] more text (<a href="https://www.google.com" target="_blank" rel="noreferrer noopener">link here</a>)' +
+        '[Text text] more text (<a href="https://www.google.com" target="_blank" rel="noreferrer noopener">link here  </a>)' +
         '[Text text] more text ((<a href="https://www.google.com" target="_blank" rel="noreferrer noopener">link here</a>))' +
         '[Text text] more text [(<a href="https://www.google.com" target="_blank" rel="noreferrer noopener">link here</a>)]' +
         '[Text text] more text (<a href="https://www.google.com" target="_blank" rel="noreferrer noopener">link here</a>)[Text text] more text (<a href="https://www.google.com" target="_blank" rel="noreferrer noopener">link here</a>)';
@@ -2270,7 +2272,7 @@ describe('room mentions', () => {
 
     test('room mention with space inside link should not be rendered', () => {
         const testString = '[ #room](google.com/sub#111)';
-        const resultString = '<a href="https://google.com/sub#111" target="_blank" rel="noreferrer noopener">#room</a>';
+        const resultString = '<a href="https://google.com/sub#111" target="_blank" rel="noreferrer noopener"> #room</a>';
         expect(parser.replace(testString)).toBe(resultString);
     });
 
diff --git a/lib/ExpensiMark.ts b/lib/ExpensiMark.ts
index 34b437ed..9b63f4bb 100644
--- a/lib/ExpensiMark.ts
+++ b/lib/ExpensiMark.ts
@@ -261,13 +261,13 @@ export default class ExpensiMark {
                     if (!g1.trim()) {
                         return match;
                     }
-                    return `<a href="${Str.sanitizeURL(g2)}" target="_blank" rel="noreferrer noopener">${g1.trim()}</a>`;
+                    return `<a href="${Str.sanitizeURL(g2)}" target="_blank" rel="noreferrer noopener">${g1}</a>`;
                 },
                 rawInputReplacement: (_extras, match, g1, g2) => {
                     if (!g1.trim()) {
                         return match;
                     }
-                    return `<a href="${Str.sanitizeURL(g2)}" data-raw-href="${g2}" data-link-variant="labeled" target="_blank" rel="noreferrer noopener">${g1.trim()}</a>`;
+                    return `<a href="${Str.sanitizeURL(g2)}" data-raw-href="${g2}" data-link-variant="labeled" target="_blank" rel="noreferrer noopener">${g1}</a>`;
                 },
             },
 
@@ -298,10 +298,9 @@ export default class ExpensiMark {
             {
                 name: 'reportMentions',
 
-                regex: /(?<![^ \n*~_])(#[\p{Ll}0-9-]{1,80})(?![^<]*(?:<\/pre>|<\/code>))/gimu,
-                replacement: '<mention-report>$1</mention-report>',
+                regex: /(?<![^ \n*~_])(#[\p{Ll}0-9-]{1,80})(?![^<]*(?:<\/pre>|<\/code>|<\/a>))/gimu,
+                replacement: '<mention-report>$1</mention-report>'
             },
-
             /**
              * This regex matches a valid user mention in a string.
              * A user mention is a string that starts with the '@' symbol and is followed by a valid user's primary