Skip to content

feat(tron-wallet-snap): route asset reads through Core AssetsController - #127

Draft
ulissesferreira wants to merge 8 commits into
mainfrom
ulissesferreira/wpn-1497-core-adapter-routing-9863
Draft

feat(tron-wallet-snap): route asset reads through Core AssetsController#127
ulissesferreira wants to merge 8 commits into
mainfrom
ulissesferreira/wpn-1497-core-adapter-routing-9863

Conversation

@ulissesferreira

Copy link
Copy Markdown
Contributor

Summary

  • Route fungible asset reads (TRX, TRC10, TRC20) based on resolved migration stage (Off / ReadWithFallback / Read) via AssetsProvider
  • Add mapControllerAsset helper and update SnapAssetsAdapter for stage-aware sync/saveMany
  • Snap-owned assets always use SnapAssetsAdapter
  • Manifest keeps endowment:messenger actions for AssetsController reads and RemoteFeatureFlagController

Part 2 of the Tron Assets migration stack (replaces closed #96 after branch rebuild onto #95).

PR chain

#126#95this PR#97

Jira

Test plan

  • yarn workspace @metamask/tron-wallet-snap run jest --no-coverage src/services/assets/
  • CI green
Open in Web Open in Cursor 

@cursor
cursor Bot force-pushed the ulissesferreira/wpn-1497-core-adapter-routing-9863 branch from bfa488d to 327dfab Compare August 5, 2026 18:29
@cursor
cursor Bot force-pushed the WPN-1497-messenger-plumbing branch 2 times, most recently from 6587ae2 to 9428a52 Compare August 5, 2026 19:34
@cursor
cursor Bot force-pushed the ulissesferreira/wpn-1497-core-adapter-routing-9863 branch 2 times, most recently from 6769ec9 to 4d766ab Compare August 5, 2026 19:34
@ulissesferreira ulissesferreira changed the title feat(tron-wallet-snap): route fungible reads through Core AssetsController feat(tron-wallet-snap): route asset reads through Core AssetsController Aug 6, 2026
@cursor
cursor Bot force-pushed the WPN-1497-messenger-plumbing branch from 9428a52 to 489bab0 Compare August 6, 2026 08:14
Base automatically changed from WPN-1497-messenger-plumbing to main August 6, 2026 08:31
@ulissesferreira

Copy link
Copy Markdown
Contributor Author

@cursoragent rebase with latest main

@cursor

cursor Bot commented Aug 6, 2026

Copy link
Copy Markdown

Rebased ulissesferreira/wpn-1497-core-adapter-routing-9863 onto latest main and force-pushed.

What happened during rebase:

  • Resolved conflict in .github/workflows/lint-build-test.yml — kept main’s upload-artifact@v7 / download-artifact@v8 (your branch had older pinned versions).
  • Dropped build: run workspace builds in topological-dev order — already on main.
  • Skipped feat(tron-wallet-snap): add Core messenger plumbing — already merged via feat(tron-wallet-snap): add Core messenger plumbing #95.

Result: 8 commits ahead of main (down from 11), including the core routing work and follow-up fixes.

PR #127 is updated: #127

Open in Web Open in Cursor 

cursoragent and others added 8 commits August 6, 2026 09:27
Co-authored-by: Ulisses Ferreira <ulisses@hey.com>
Co-authored-by: Ulisses Ferreira <ulisses@hey.com>
…oller

Introduce migration-stage routing for fungible asset reads (Off /
ReadWithFallback / Read) via AssetsProvider, with mapControllerAsset and
SnapAssetsAdapter updates. Snap-owned assets remain on the Snap adapter.

Co-authored-by: Ulisses Ferreira <ulisses@hey.com>
Co-authored-by: Ulisses Ferreira <ulisses@hey.com>
…link

Co-authored-by: Ulisses Ferreira <ulisses@hey.com>
Co-authored-by: Ulisses Ferreira <ulisses@hey.com>
Co-authored-by: Ulisses Ferreira <ulisses@hey.com>
Co-authored-by: Ulisses Ferreira <ulisses@hey.com>
@cursor
cursor Bot force-pushed the ulissesferreira/wpn-1497-core-adapter-routing-9863 branch from 4d766ab to af82730 Compare August 6, 2026 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants