expensify-common 2.0.171 → 2.0.174

package/dist/ExpensiMark.d.ts

@@ -31,6 +31,7 @@ type CommonRule = {
     rawInputReplacement?: Replacement;
     pre?: (input: string) => string;
     post?: (input: string) => string;
+    shouldSkipProcessing?: (textToCheck: string) => boolean;
 };
 type RuleWithRegex = CommonRule & {
     regex: RegExp;

package/dist/ExpensiMark.js

@@ -128,6 +128,11 @@ class ExpensiMark {
                     const extraAttrs = attrCache && attrCache[videoSource];
                     return `<video data-expensify-source="${str_1.default.sanitizeURL(videoSource)}" data-raw-href="${videoSource}" data-link-variant="${typeof videoName === 'string' ? 'labeled' : 'auto'}" ${extraAttrs || ''}>${videoName ? `${videoName}` : ''}</video>`;
                 },
+                shouldSkipProcessing: (textToCheck) => {
+                    const missingSpecialCharacters = !textToCheck.includes('!') || !textToCheck.includes('(') || !textToCheck.includes(')');
+                    const missingVideoExtension = !Constants.CONST.VIDEO_EXTENSIONS.some((extension) => textToCheck.includes(`.${extension}`));
+                    return missingSpecialCharacters || missingVideoExtension;
+                },
             },
             /**
              * Apply inline code-block to avoid applying any other formatting rules inside of it,
@@ -208,6 +213,9 @@ class ExpensiMark {
                     const extraAttrs = attrCache && attrCache[imgSource];
                     return `<img src="${str_1.default.sanitizeURL(imgSource)}"${imgAlt ? ` alt="${this.escapeAttributeContent(imgAlt)}"` : ''} data-raw-href="${imgSource}" data-link-variant="${typeof imgAlt === 'string' ? 'labeled' : 'auto'}" ${extraAttrs || ''}/>`;
                 },
+                shouldSkipProcessing: (textToCheck) => {
+                    return !textToCheck.includes('!') || !textToCheck.includes('(') || !textToCheck.includes(')');
+                },
             },
             /**
              * Converts markdown style links to anchor tags e.g. [Expensify](https://www.expensify.com)
@@ -229,6 +237,9 @@ class ExpensiMark {
                     }
                     return `<a href="${str_1.default.sanitizeURL(g2)}" data-raw-href="${g2}" data-link-variant="labeled" target="_blank" rel="noreferrer noopener">${g1}</a>`;
                 },
+                shouldSkipProcessing: (textToCheck) => {
+                    return !textToCheck.includes('.');
+                },
             },
             /**
              * Apply the hereMention first because the string @here is still a valid mention for the userMention regex.
@@ -313,6 +324,9 @@ class ExpensiMark {
                     const href = str_1.default.sanitizeURL(g2);
                     return `${g1}<a href="${href}" data-raw-href="${g2}" data-link-variant="auto" target="_blank" rel="noreferrer noopener">${g2}</a>${g1}`;
                 },
+                shouldSkipProcessing: (textToCheck) => {
+                    return !textToCheck.includes('.');
+                },
             },
             {
                 name: 'quote',
@@ -807,6 +821,9 @@ class ExpensiMark {
      * If not provided, all available rules will be applied. If provided, the rules in the array will be skipped.
      */
     replace(text, { filterRules = [], shouldEscapeText = true, shouldKeepRawInput = false, disabledRules = [], extras = EXTRAS_DEFAULT } = {}) {
+        if (!text) {
+            return '';
+        }
         // This ensures that any html the user puts into the comment field shows as raw html
         let replacedText = shouldEscapeText ? Utils.escapeText(text) : text;
         const rules = this.getHtmlRuleset(filterRules, disabledRules, shouldKeepRawInput);
@@ -815,6 +832,9 @@ class ExpensiMark {
             if (rule.pre) {
                 replacedText = rule.pre(replacedText);
             }
+            if (rule.shouldSkipProcessing && rule.shouldSkipProcessing(replacedText)) {
+                return;
+            }
             const replacement = shouldKeepRawInput && rule.rawInputReplacement ? rule.rawInputReplacement : rule.replacement;
             if ('process' in rule) {
                 replacedText = rule.process(replacedText, replacement, shouldKeepRawInput);
@@ -1056,6 +1076,9 @@ class ExpensiMark {
      * Replaces HTML with markdown
      */
     htmlToMarkdown(htmlString, extras = EXTRAS_DEFAULT) {
+        if (!htmlString) {
+            return '';
+        }
         let generatedMarkdown = htmlString;
         const body = /<(body)(?:"[^"]*"|'[^']*'|[^'"><])*>(?:\n|\r\n)?([\s\S]*?)(?:\n|\r\n)?<\/\1>(?![^<]*(<\/pre>|<\/code>))/im;
         const parseBodyTag = generatedMarkdown.match(body);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expensify-common",
-  "version": "2.0.171",
+  "version": "2.0.174",
   "author": "Expensify, Inc.",
   "description": "Expensify libraries and components shared across different repos",
   "homepage": "https://expensify.com",