fix: remove lookbehind, strip user names from the input instead

Author: madsroskar

package/src/components/Message/MessageSimple/utils/parseLinks.ts

@@ -9,8 +9,7 @@
 const emailUserName = '[\\w+\\.~$_-]+';
 const schema = `(\\w{2,15}:\\/\\/)`;
 // something.tld OR 123.123.123.123
-const userName = '^@\\w+\\.?\\w*';
-const domain = `((?:\\w+\\.[a-zA-Z]+(?<!${userName}))+(?:[^:\\/\\s]+)|(?:\\d{1,3}\\.\\d{1,3}\\.\\d{1,3}\\.\\d{1,3}))`;
+const domain = `((?:\\w+\\.[a-zA-Z]+)+(?:[^:\\/\\s]+)|(?:\\d{1,3}\\.\\d{1,3}\\.\\d{1,3}\\.\\d{1,3}))`;
 const port = `(:[0-9]{1,5})`;
 const path = `((?:\\/)?[^?\\s]+)`;
 const queryString = `(\\?[^#\\s]+)`;
@@ -44,10 +43,19 @@ interface Link {
  * */
 const removeMarkdownLinksFromText = (input: string) => input.replace(/\[[\w\s]+\]\(.*\)/g, '');
 
+/**
+ * Hermes doesn't support lookbehind, so this is done separately to avoid
+ * parsing user names as links.
+ * */
+const removeUserNamesFromText = (input: string) => input.replace(/^@\w+\.?\w/, '');
+
 export const parseLinksFromText = (input: string): Link[] => {
   let matches;
 
-  const inputWithoutMarkdownLinks = removeMarkdownLinksFromText(input);
+  const strippedInput = [removeMarkdownLinksFromText, removeUserNamesFromText].reduce(
+    (acc, fn) => fn(acc),
+    input,
+  );
 
   const results: Link[] = [];
 
@@ -62,7 +70,7 @@ export const parseLinksFromText = (input: string): Link[] => {
    * to avoid overlapping matches being duplicated in the output.
    * */
   const linkRegex = new RegExp(`${fqdnLinkPattern}|${schemePrefixedLinkPattern}`, 'gi');
-  while ((matches = linkRegex.exec(inputWithoutMarkdownLinks)) !== null) {
+  while ((matches = linkRegex.exec(strippedInput)) !== null) {
     const [
       raw,
       fqdnScheme = '',