Skip to content

feat(WPN-1486): remove Solana Snap assets tracking after direct migration (Lane B - PR 6b) - #640

Draft
ulissesferreira wants to merge 1 commit into
WPN-1476-hardcode-core-assets-adapterfrom
WPN-1486-remove-solana-assets-tracking
Draft

feat(WPN-1486): remove Solana Snap assets tracking after direct migration (Lane B - PR 6b)#640
ulissesferreira wants to merge 1 commit into
WPN-1476-hardcode-core-assets-adapterfrom
WPN-1486-remove-solana-assets-tracking

Conversation

@ulissesferreira

Copy link
Copy Markdown
Contributor

Summary

  • Remove SnapAssetsAdapter balance fetch/persist code and AssetsRepository
  • Remove assetEntities from snap state
  • AssetsService is Core-only for all balance reads; clean up wiring in snapContext

Test plan

  • yarn workspace @metamask/solana-wallet-snap test:core (891 passed)

@ulissesferreira
ulissesferreira force-pushed the WPN-1476-hardcode-core-assets-adapter branch 3 times, most recently from f87679d to b39f84d Compare July 31, 2026 23:01
@ulissesferreira
ulissesferreira force-pushed the WPN-1486-remove-solana-assets-tracking branch from 0ce6db4 to 0ccec27 Compare July 31, 2026 23:06
@ulissesferreira
ulissesferreira force-pushed the WPN-1476-hardcode-core-assets-adapter branch from b39f84d to 3569f70 Compare August 2, 2026 23:41
@ulissesferreira
ulissesferreira force-pushed the WPN-1486-remove-solana-assets-tracking branch from 0ccec27 to a014762 Compare August 2, 2026 23:42
@ulissesferreira
ulissesferreira force-pushed the WPN-1476-hardcode-core-assets-adapter branch from 3569f70 to ced85b8 Compare August 2, 2026 23:49
@ulissesferreira
ulissesferreira force-pushed the WPN-1486-remove-solana-assets-tracking branch 2 times, most recently from 5b99836 to b9a39d1 Compare August 3, 2026 00:02
@ulissesferreira
ulissesferreira force-pushed the WPN-1476-hardcode-core-assets-adapter branch from ced85b8 to 7aeabb2 Compare August 3, 2026 00:13
@ulissesferreira
ulissesferreira force-pushed the WPN-1486-remove-solana-assets-tracking branch from b9a39d1 to 0edfaf4 Compare August 3, 2026 00:14
@sonarqubecloud

sonarqubecloud Bot commented Aug 3, 2026

Copy link
Copy Markdown

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.

1 participant