feat: add suggestions to watchlist on marketlist UI#31650
Conversation
PR template — items to address before "Ready for review"Warnings — informational, address before merging:
See docs/readme/ready-for-review.md for the full Definition of Ready for Review. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c049f1d. Configure here.
geositta
left a comment
There was a problem hiding this comment.
Requesting changes unless we have a product decision not to support search? The navigation override and focused tests look good, but the watchlist marketlist search regression should be fixed before merge IMO.
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
Tag selection rationale:
The navigation change (push vs navigate) is a behavioral fix that could affect the back-navigation flow in Perps, warranting E2E validation. The watchlist search filtering is new functionality that needs testing. Performance Test Selection: |
geositta
left a comment
There was a problem hiding this comment.
Tested latest changes and looks good!
⚡ Performance Test Results
✅ All tests passed · 2 tests · 1 device 📱 Devices tested (1)Android: Google Pixel 8 Pro (v14.0) ✅ Passed Tests (2)
Branch: |

Description
Add suggestions to the watchlist on market list UI. Should match behavior on Perps Home, minus the collapsible "show more" button.
Fixes back navigation from asset details screen so that it navigates correctly to Perps Home, or Market List, contextually.
Fixes search bar functionality on Market List Watchlist
Changelog
CHANGELOG entry: add suggested markets to watchlist on marketlist UI
Related issues
Fixes: n/a
Manual testing steps
Screenshots/Recordings
Screen.Recording.2026-06-12.at.9.31.00.AM.mov
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Low Risk
UI-only perps market list and watchlist component changes behind the existing watchlist flag, with broad test coverage and no auth or data-layer changes.
Overview
Perps market list watchlist mode now mirrors Perps Home: when the watchlist feature flag is on and the watchlist filter is active, the screen always renders
PerpsWatchlistMarketswith saved markets and suggested markets—not only when the watchlist is empty. The list view passesenableShowMore={false},showHeader={false}, and wires row taps through the existinghandleMarketPresssoonMarketSelect,navigateToMarketDetails, and routetransactionActiveAbTestsbehave like the main market list.PerpsWatchlistMarketsgains optionalonMarketPress(parent-owned navigation instead of internalnavigation.navigate) andenableShowMore(defaulttrue; whenfalse, all watchlist rows show with no expand/collapse). Both V1 and V2 honoronMarketPress.Tests cover the new market-list watchlist flows and the new component props.
Reviewed by Cursor Bugbot for commit 6ffb46b. Bugbot is set up for automated code reviews on this repo. Configure here.