Skip HTML parsing for task report names when content is plain text

src/libs/Parser.ts

@@ -7,7 +7,7 @@
 const accountIDToNameMap: Record<string, string> = {};
 
 const reportIDToNameMap: Record<string, string> = {};
 Onyx.connect({
     key: ONYXKEYS.COLLECTION.REPORT,
     waitForCollectionCallback: true,
     callback: (value) => {
@@ -24,7 +24,7 @@
     },
 });
 
 Onyx.connect({
     key: ONYXKEYS.PERSONAL_DETAILS_LIST,
     callback: (personalDetailsList) => {
         for (const personalDetails of Object.values(personalDetailsList ?? {})) {
@@ -61,6 +61,10 @@
         });
     }
 
+    isHTML(text: string): boolean {
+        return /<[^>]+>/.test(text) || /&[#\w]+;/.test(text);
+    }
+
     truncateHTML(htmlString: string, limit: number, extras?: {ellipsis: string | undefined}): string {
         return super.truncateHTML(htmlString, limit, extras);
     }

src/libs/ReportNameUtils.ts

@@ -140,7 +140,7 @@
 
 let allPersonalDetails: OnyxEntry<PersonalDetailsList>;
 
 Onyx.connect({
     key: ONYXKEYS.PERSONAL_DETAILS_LIST,
     callback: (value) => {
         allPersonalDetails = value;
@@ -776,7 +776,9 @@
     }
 
     if (isTaskReport(report)) {
-        return Parser.htmlToText(report?.reportName ?? '').trim();
+        const taskName = report?.reportName ?? '';
+
+        return Parser.isHTML(taskName) ? Parser.htmlToText(taskName).trim() : taskName.trim();
     }
 
     const privateIsArchivedValue = privateIsArchived ?? allReportNameValuePairs?.[`${ONYXKEYS.COLLECTION.REPORT_NAME_VALUE_PAIRS}${report.reportID}`]?.private_isArchived;

src/libs/ReportUtils.ts

@@ -1013,7 +1013,7 @@
 };
 
 let conciergeReportIDOnyxConnect: OnyxEntry<string>;
 Onyx.connect({
     key: ONYXKEYS.CONCIERGE_REPORT_ID,
     callback: (value) => {
         conciergeReportIDOnyxConnect = value;
@@ -1021,7 +1021,7 @@
 });
 
 const defaultAvatarBuildingIconTestID = 'SvgDefaultAvatarBuilding Icon';
 Onyx.connect({
     key: ONYXKEYS.SESSION,
     callback: (value) => {
         // When signed out, val is undefined
@@ -1039,7 +1039,7 @@
 let allPersonalDetails: OnyxEntry<PersonalDetailsList>;
 let allPersonalDetailLogins: string[];
 let currentUserPersonalDetails: OnyxEntry<PersonalDetails>;
 Onyx.connect({
     key: ONYXKEYS.PERSONAL_DETAILS_LIST,
     callback: (value) => {
         if (currentUserAccountID) {
@@ -1051,7 +1051,7 @@
 });
 
 let allReportsDraft: OnyxCollection<Report>;
 Onyx.connect({
     key: ONYXKEYS.COLLECTION.REPORT_DRAFT,
     waitForCollectionCallback: true,
     callback: (value) => (allReportsDraft = value),
@@ -1060,7 +1060,7 @@
 let allPolicies: OnyxCollection<Policy>;
 let hasPolicies: boolean;
 let policiesArray: Policy[] = [];
 Onyx.connect({
     key: ONYXKEYS.COLLECTION.POLICY,
     waitForCollectionCallback: true,
     callback: (value) => {
@@ -1071,7 +1071,7 @@
 });
 
 let allPolicyDrafts: OnyxCollection<Policy>;
 Onyx.connect({
     key: ONYXKEYS.COLLECTION.POLICY_DRAFTS,
     waitForCollectionCallback: true,
     callback: (value) => (allPolicyDrafts = value),
@@ -1079,7 +1079,7 @@
 
 let allReports: OnyxCollection<Report>;
 let reportsByPolicyID: ReportByPolicyMap;
 Onyx.connect({
     key: ONYXKEYS.COLLECTION.REPORT,
     waitForCollectionCallback: true,
     callback: (value) => {
@@ -5757,7 +5757,8 @@
     }
 
     if (isTaskReport(report)) {
-        return Parser.htmlToText(report?.reportName ?? '').trim();
+        const taskName = report?.reportName ?? '';
+        return Parser.isHTML(taskName) ? Parser.htmlToText(taskName).trim() : taskName.trim();
     }
 
     if (isChatThread(report)) {

tests/unit/ParserTest.ts

@@ -0,0 +1,27 @@
+import Parser from '@libs/Parser';
+
+describe('Parser', () => {
+    describe('isHTML', () => {
+        test('returns true for strings containing HTML tags', () => {
+            expect(Parser.isHTML('<b>bold</b>')).toBe(true);
+            expect(Parser.isHTML('<a href="https://example.com">link</a>')).toBe(true);
+            expect(Parser.isHTML('<h1>heading</h1>')).toBe(true);
+            expect(Parser.isHTML('text with <em>emphasis</em> inside')).toBe(true);
+            expect(Parser.isHTML('<br/>')).toBe(true);
+            expect(Parser.isHTML('<p>paragraph</p>')).toBe(true);
+        });
+
+        test('returns false for plain text', () => {
+            expect(Parser.isHTML('just plain text')).toBe(false);
+            expect(Parser.isHTML('Fix the login bug')).toBe(false);
+            expect(Parser.isHTML('')).toBe(false);
+            expect(Parser.isHTML('100 > 50 and 20 < 30')).toBe(false);
+        });
+
+        test('returns false for strings with angle brackets that are not HTML', () => {
+            expect(Parser.isHTML('a > b')).toBe(false);
+            expect(Parser.isHTML('x < y')).toBe(false);
+            expect(Parser.isHTML('<>')).toBe(false);
+        });
+    });
+});

tests/unit/ReportNameUtilsTest.ts

@@ -284,6 +284,64 @@ describe('ReportNameUtils', () => {
             );
             expect(name).toBe('heading with link');
         });
+
+        test('Returns plain text title without HTML conversion', () => {
+            const plainTaskTitle = 'Fix the login bug on Android';
+            const report: Report = {
+                ...createRegularTaskReport(41, currentUserAccountID),
+                reportName: plainTaskTitle,
+            };
+
+            const name = computeReportName(
+                report,
+                emptyCollections.reports,
+                emptyCollections.policies,
+                undefined,
+                undefined,
+                participantsPersonalDetails,
+                emptyCollections.reportActions,
+                currentUserAccountID,
+            );
+            expect(name).toBe('Fix the login bug on Android');
+        });
+
+        test('Trims whitespace from plain text title', () => {
+            const report: Report = {
+                ...createRegularTaskReport(42, currentUserAccountID),
+                reportName: '  Expense report review  ',
+            };
+
+            const name = computeReportName(
+                report,
+                emptyCollections.reports,
+                emptyCollections.policies,
+                undefined,
+                undefined,
+                participantsPersonalDetails,
+                emptyCollections.reportActions,
+                currentUserAccountID,
+            );
+            expect(name).toBe('Expense report review');
+        });
+
+        test('Returns empty string for undefined reportName', () => {
+            const report: Report = {
+                ...createRegularTaskReport(43, currentUserAccountID),
+                reportName: undefined,
+            };
+
+            const name = computeReportName(
+                report,
+                emptyCollections.reports,
+                emptyCollections.policies,
+                undefined,
+                undefined,
+                participantsPersonalDetails,
+                emptyCollections.reportActions,
+                currentUserAccountID,
+            );
+            expect(name).toBe('');
+        });
     });
 
     describe('computeReportName - Thread report action names', () => {