[Due for payment 2026-01-09] [$250] Remove updateOptimisticReportNamesFromUpdates middleware

[neil-marcellini]

Problem

We have this middleware updateOptimisticReportNamesFromUpdates which optimistically updates computed report names based on Onyx updates. It was added as part of our original design for optimistic custom report names on New Expensify. However, we recently decided that it is not worth the effort to build this complete system for optimistic updates, given that the backend can generally take care of it when users are online, and it has been taking a really longtime to build it out perfectly without any bugs.

Solution

Remove the middleware function and all code related to optimistically computing report names based on Onyx updates. We still want to keep the code to compute a report name given its formula. In the future we're going to make sure that custom report names are optimistically computed only when reports are created, and probably just for the most important formula parts.

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021998500208056442603
• Upwork Job ID: 1998500208056442603
• Last Price Increase: 2025-12-09
• Automatic offers:
• QichenZhu | Reviewer | 109763302
• truph01 | Contributor | 109763306

Issue Owner

Current Issue Owner: @CortneyOfstad

[truph01]

Proposal

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

• Remove updateOptimisticReportNamesFromUpdates middleware

What is the root cause of that problem?

• Feature request

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

• Remove this function

  App/src/libs/OptimisticReportNames.ts

  Line 345 in 68c4a93

  function updateOptimisticReportNamesFromUpdates(optimisticData: OnyxUpdate[], context: UpdateContext, successData?: OnyxUpdate[], failureData?: OnyxUpdate[]): ProcessedUpdates {

• Remove the places where the function is being used:

App/src/libs/API/index.ts

Line 99 in 7c65499

const processedOptimisticData = OptimisticReportNames.updateOptimisticReportNamesFromUpdates(optimisticData, context);

and all the related tests.

What alternative solutions did you explore? (Optional)

[neil-marcellini]

Ok @truph01 pls go ahead. I should probably stay focused on other work vs doing this myself.

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~021998500208056442603

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @QichenZhu (External)