chore: social - traders list leaderboard#28170
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
@SocketSecurity ignore npm/@metamask-previews/social-controllers@0.0.0-preview-ac7a6cc51 |
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. Ignoring alerts on:
|
|
@SocketSecurity ignore npm/@metamask/social-controllers@0.1.0 |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ 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 5f9424f. Configure here.
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: Performance Test Selection: |
|
|
✅ E2E Fixture Validation — Schema is up to date |




Description
Wires up the Social Leaderboard to live data from the Social API.
Integrate the social-controllers (SocialService + SocialController) into the Engine, adds useTopTraders hook using useQuery from @metamask/react-data-query to fetch leaderboard data through the Data Services pattern.
Also adds the homepage TopTradersSection (horizontal card carousel) and the full TopTradersView (ranked list) with skeleton loading states.
Changelog
CHANGELOG entry: null
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Medium Risk
Adds a new Social data service/controller to the Engine and wires UI screens to fetch live leaderboard data from a new
SOCIAL_API_URL, which could impact app startup/state persistence and introduce new network failure modes.Overview
Adds live Social Leaderboard integration by introducing Engine-managed
SocialServiceandSocialController(new messengers, init wiring, persisted state, and data-service registration) and configuring a newSOCIAL_API_URLbuild/env constant.Updates the homepage
TopTradersSectionto fetch and render the top 3 traders (cards + skeleton loading, refresh via ref, hide when empty/flag off) and upgradesTopTradersViewto display the full ranked list with a network filter entry point and follow toggles.Includes new UI components (
TopTraderCard,TraderRow, skeletons,NetworkFilterButton), aformatPnlformatter, theuseTopTradershook built on@metamask/react-data-query, plus expanded unit tests and updated mocks/strings.Reviewed by Cursor Bugbot for commit 5881e59. Bugbot is set up for automated code reviews on this repo. Configure here.