Repository navigation
[No QA] Extend search parity golden master with autocomplete call sites and Onyx updates (Part 2/2) - #105898
Conversation
|
@MelvinBot review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable findings in this review. Verified autocomplete configurations against production and checked warm-cache versus cold-build lifecycle assertions. Snapshot changes preserve existing entries. Tests: All 7 tests and 7 snapshots passed without updating snapshots ( |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
|
@eVoloshchak Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button] |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 14f0828d83
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Reviewer Checklist
Screenshots/Videos |
mountiny
left a comment
There was a problem hiding this comment.
Thanks for adding these.
|
🚀 Deployed to staging by https://github.com/mountiny in version: 9.5.34-0 🚀
|
|
🚀 Deployed to production by https://github.com/yuwenmemon in version: 9.5.34-1 🚀
Bundle Size Analysis (Sentry): |

Explanation of Change
Follow-up to #104510 and the concluding part of the search parity golden master series guarding the SearchRouter option index initiative. This PR completes the test harness by recording autocomplete call sites, query-side normalization forms, and the Onyx update lifecycle into the baseline snapshot.
useAutocompleteSuggestionscall sites:Records the
autocompleteParticipants(to:/from:, reading onlypersonalDetails) andautocompleteIn(in:, reading onlyrecentReports) call sites alongside the SearchRouter ones. Because the hook sharescreateFilteredOptionListandgetSearchOptionswithSearchAutocompleteList, these snapshots guarantee that upcoming refactors of the shared pipeline cannot silently regress syntax queries. The configurations mirror the production hook, with inline comments documenting intentional test-level simplifications.Adds 18 equivalent query forms (NFD accents, combining dots, lowercase sharp s
ß, Greek case and tonos, zero-width spaces, non-breaking spaces, word order permutations, and punctuation) directly toSEARCH_QUERIES.Replays 13 representative Onyx mutations in
SEARCH_FIXTURE_UPDATES(renames, archive/unarchive, deletes, new/removed contacts, unread states, recency updates, thread replies, drafts). Asserts that warm caches produce the exact same results as a cold build of the updated account, and verifies that every update mutates at least one result (hasChangedResults: true).Fixed Issues
$ #103148
PROPOSAL:
Tests
npm test -- tests/unit/OptionsListUtilsSearchParityTest.ts-u).Offline tests
N\A
QA Steps
N\A
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, I verified the components usingAvatarare working as expected)npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.