Skip to content

[HIGH] Investigate Search for reports hanging as user types #62106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mountiny opened this issue May 15, 2025 · 5 comments
Open

[HIGH] Investigate Search for reports hanging as user types #62106

mountiny opened this issue May 15, 2025 · 5 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2

Comments

@mountiny
Copy link
Contributor

mountiny commented May 15, 2025

Shared in Slack quality room

When typing to search for the reports, the ui can hang. Collected logs and profile traces shared in slack post to investigate

Issue OwnerCurrent Issue Owner: @sosek108
@mountiny mountiny added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels May 15, 2025
Copy link

melvin-bot bot commented May 15, 2025

Triggered auto assignment to @mallenexpensify (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.

@mallenexpensify mallenexpensify changed the title Investigate Search for reports hanging as user types [HIGH] Investigate Search for reports hanging as user types May 15, 2025
@mallenexpensify
Copy link
Contributor

Hey.. opened the bug from the thread and it's assigned to me!
Starting HIGH cuz Vit said he's seen it before and I suspect others have encountered too. Looks like Vit was on iOS

  • HIGH -  UX Reliability, API Reliability and Performance issues that are part of main flows and affect a subset of users

@sosek108
Copy link
Contributor

sosek108 commented May 16, 2025

No significant update for now.

I saw no such error on Android emulator so I left this one for now and focused on iOS on which there are significant problems with Search input.
I see that each typed letter causes rerender of whole SearchRouter which is slighly significant and heavy rerener of AutocompleteList

Image

Issues that may be connected:

@sosek108
Copy link
Contributor

sosek108 commented May 20, 2025

No big update.

I've confirmed already what is said in other tasks. There are couple of places that could be improved and I see no "one quick fix" that will resolve that, but I'm trying - I'm doing POC of some refactror in SearchAutocompleteList to check this components bottlenecks.

  • SearchInput is Markdown which alongside with parser gives unpleasent lag. Parser is generated by peggy so I am not sure if it could be impreoved
  • After almost every keystroke FastSearch is created from scratch
  • API call is send very often. I've got 2 API calls during quick 4 keystrokes. And each API response causes rerender of Search components
  • SearchAutocompleteList render cost is very high (even when FastSearch is disabled)

Next:

@mountiny
Copy link
Contributor Author

Thanks for the updated, could we introduce more caching or debounce the search calls a bit more?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2
Projects
Status: HIGH
Development

No branches or pull requests

3 participants