expensify-common 2.0.170 → 2.0.173

package/dist/ExpensiMark.js

@@ -807,6 +807,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);
@@ -1056,6 +1059,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/dist/Logger.d.ts

@@ -12,12 +12,14 @@ type LogLine = {
     parameters: Parameters;
     onlyFlushWithOthers?: boolean;
     timestamp: Date;
+    email?: string | null;
 };
 type LoggerOptions = {
     serverLoggingCallback: ServerLoggingCallback;
     isDebug: boolean;
     clientLoggingCallback: ClientLoggingCallBack;
     maxLogLinesBeforeFlush?: number;
+    getContextEmail?: () => string | null;
 };
 export default class Logger {
     logLines: LogLine[];
@@ -25,7 +27,8 @@ export default class Logger {
     clientLoggingCallback: ClientLoggingCallBack;
     isDebug: boolean;
     maxLogLinesBeforeFlush: number;
-    constructor({ serverLoggingCallback, isDebug, clientLoggingCallback, maxLogLinesBeforeFlush }: LoggerOptions);
+    getContextEmail?: () => string | null;
+    constructor({ serverLoggingCallback, isDebug, clientLoggingCallback, maxLogLinesBeforeFlush, getContextEmail }: LoggerOptions);
     /**
      * Ask the server to write the log message
      */

package/dist/Logger.js

@@ -2,13 +2,14 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const MAX_LOG_LINES_BEFORE_FLUSH = 50;
 class Logger {
-    constructor({ serverLoggingCallback, isDebug, clientLoggingCallback, maxLogLinesBeforeFlush }) {
+    constructor({ serverLoggingCallback, isDebug, clientLoggingCallback, maxLogLinesBeforeFlush, getContextEmail }) {
         // An array of log lines that limits itself to a certain number of entries (deleting the oldest)
         this.logLines = [];
         this.serverLoggingCallback = serverLoggingCallback;
         this.clientLoggingCallback = clientLoggingCallback;
         this.isDebug = isDebug;
         this.maxLogLinesBeforeFlush = maxLogLinesBeforeFlush || MAX_LOG_LINES_BEFORE_FLUSH;
+        this.getContextEmail = getContextEmail;
         // Public Methods
         this.info = this.info.bind(this);
         this.alert = this.alert.bind(this);
@@ -51,11 +52,21 @@ class Logger {
      * @param onlyFlushWithOthers A request will never be sent to the server if all loglines have this set to true
      */
     add(message, parameters, forceFlushToServer, onlyFlushWithOthers = false, extraData = '') {
+        // Capture the user's email at the moment this specific log line is created
+        // This ensures the log retains user context even if the session is cleared before sending
+        let email = null;
+        try {
+            email = this.getContextEmail ? this.getContextEmail() : null;
+        }
+        catch (_a) {
+            // Silently fail if getContextEmail throws - logging should not crash
+        }
         const length = this.logLines.push({
             message,
             parameters,
             onlyFlushWithOthers,
             timestamp: new Date(),
+            email,
         });
         if (this.isDebug) {
             this.client(`${message} - ${JSON.stringify(parameters)}`, extraData);

package/package.json

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