[Due for payment 2025-07-30] LHN - LHN preview incorrectly shows "Room renamed to "y" after changing report title

[jponikarchuk]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Version Number: 9.1.75-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail: #64733
Email or phone of affected tester (no customers): sdjiosdnjsdoisjdoi@gmail.com
Issue reported by: Applause Internal Team
Device used: Mac 15.5 / Chrome
App Component: Left Hand Navigation (LHN)

Action Performed:

1. Go to staging.new.expensify.com
2. Go to workspace chat.
3. Submit an expense.
4. Open the expense report.
5. Click Submit.
6. Click on the report header.
7. Click Title.
8. Change the title and save it.
9. Close the RHP.

Expected Result:

LHN preview will show "renamed to "y" (previously "x")", which is the same as the system message in the report.

Actual Result:

LHN preview shows "Room renamed to "y".

Workaround:

Unknown

Platforms:

• Android: App
• Android: mWeb Chrome
• iOS: App
• iOS: mWeb Safari
• iOS: mWeb Chrome
• Windows: Chrome
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021940915845829900379
• Upwork Job ID: 1940915845829900379
• Last Price Increase: 2025-07-03
• Automatic offers:
• QichenZhu | Contributor | 107923133
• thelullabyy | Contributor | 107941645

Issue Owner

Current Issue Owner: @twisterdotcom

[melvin-bot]

Triggered auto assignment to @deetergp (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

[melvin-bot]

Triggered auto assignment to @twisterdotcom (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[github-actions]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[melvin-bot]

💬 A slack conversation has been started in #expensify-open-source

[BlameGPT]

Possible culprit PRs for this issue

• LHN doesn't show the message sent or the system message #64733: Changed getLastMessageTextForReport logic in OptionsListUtils which determines LHN preview text, likely causing rename system message to be truncated.
• fix: last actor display name not update on actionableMentionWhisper #64204: Altered OptionsListUtils and SidebarUtils affecting how last actor and action data are read for LHN previews, potentially removing the '(previously "x")' segment.

If not above, check these

• Fix - Remove getTransaction #62434: This PR refactored ReportPreviewActionUtils (which generates LHN preview text) and likely dropped the logic that included the previous report title, causing the preview to now show only “Room renamed to ‘y’”.

[samranahm]

Proposal

Please re-state the problem that we are trying to solve in this issue.

LHN preview incorrectly shows "Room renamed to "y" after changing report title

What is the root cause of that problem?

when we submit an expense an open that expense report, that is money request report, and we show isRenamedAction based on there checks that include isPolicyExpenseChat but not isMoneyRequestReport

App/src/libs/SidebarUtils.ts

Line 660 in 1a4ee85

if ((result.isChatRoom || result.isPolicyExpenseChat || result.isThread || result.isTaskReport || isThreadMessage || isGroupChat) && !result.private_isArchived) {

so we end up here

App/src/libs/SidebarUtils.ts

Lines 805 to 806 in 1a4ee85

	if (shouldShowLastActorDisplayName(report, lastActorDetails, lastAction) && !isArchivedReport(reportNameValuePairs)) {
	result.alternateText = `${lastActorDisplayName}: ${formatReportLastMessageText(Parser.htmlToText(lastMessageText))}`;

displaying lastMessageText from report that is Room renamed to ...

What changes do you think we should make in order to solve the problem?

We should add this check result.isMoneyRequestReport here

App/src/libs/SidebarUtils.ts

Line 660 in 1a4ee85

if ((result.isChatRoom || result.isPolicyExpenseChat || result.isThread || result.isTaskReport || isThreadMessage || isGroupChat) && !result.private_isArchived) {

like

if ((result.isChatRoom || result.isPolicyExpenseChat || result.isMoneyRequestReport || result.isThread || result.isTaskReport || isThreadMessage || isGroupChat) && !result.private_isArchived) {

so that we can reach to isRenamedAction here

App/src/libs/SidebarUtils.ts

Lines 664 to 665 in 1a4ee85

	if (isRenamedAction(lastAction)) {
	result.alternateText = getRenamedAction(lastAction, isExpense, lastActorDisplayName);

that will display correct copy of renamed report action

What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?

N/A

What alternative solutions did you explore? (Optional)

Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.