refactor(tron-wallet-snap): aggressive assets cleanup with Core AssetsProvider routing

- Slim SnapAssetsAdapter to snap-owned fetch/save/read only (fetchSnapOwnedAssetsForAccount)
- Move handler logic (metadata, conversions, market data, historical prices) into AssetsService
- Add syncSnapOwnedAssets for cron sync; AccountsService.synchronizeAssets delegates to it
- Keep PR97 fungible read routing via AssetsProvider for getAccountAssetByID/ByIDs/ByScope
- Remove public saveMany, getAll, fetchAssetsAndBalancesForAccount, hasChanged from AssetsService
- Drop unused state from AssetsService constructor in context.ts
- Port and adapt unit tests; update eslint suppressions and changelog

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

Author: cursoragent

eslint-suppressions.json

@@ -269,6 +269,16 @@
       "count": 2
     }
   },
+  "packages/tron-wallet-snap/src/services/assets/AssetsRepository.ts": {
+    "import-x/no-extraneous-dependencies": {
+      "count": 1
+    }
+  },
+  "packages/tron-wallet-snap/src/services/assets/AssetsService.ts": {
+    "import-x/no-extraneous-dependencies": {
+      "count": 2
+    }
+  },
   "packages/tron-wallet-snap/src/services/assets/AssetsService.test.ts": {
     "@typescript-eslint/explicit-function-return-type": {
       "count": 3

packages/tron-wallet-snap/CHANGELOG.md

@@ -12,14 +12,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Wire Core messenger endowment and resolve assets migration stage from remote feature flags on each account asset read (no routing yet).
 - Route fungible asset reads through the shared `AssetsProvider` from `@metamask/snap-networks-utils` using account-scoped `AssetsController:getAccountAssetByID`, `AssetsController:getAccountAssetsByIDs`, and `AssetsController:getAccountAssetsByScope` actions based on migration stage (TRX, TRC10, TRC20). Protocol assets (energy, bandwidth, staking, lock/withdrawal, rewards) remain Snap-owned. Resolution order: remote feature flags → Off default.
 
-### Removed
-
-- Assets migration feature-flag routing. Fungible reads (`getAccountAssetByID`, `getAccountAssetsByIDs`, `getAccountAssetsByScope`) now always use Core `AssetsController` via `AssetsProvider`; snap-owned protocol assets remain on the Snap adapter. Removed `RemoteFeatureFlagController:getState` messenger endowment.
-
 ### Changed
 
+- Move assets handler logic (metadata, conversions, market data, historical prices) into `AssetsService`; slim `SnapAssetsAdapter` to snap-owned fetch/save/read only. Cron asset sync uses `syncSnapOwnedAssets` for protocol assets.
 - Update `snap.manifest.json` bundle shasum ([#82](https://github.com/MetaMask/internal-snaps/pull/82))
 
+### Removed
+
+- Assets migration feature-flag routing. Fungible reads (`getAccountAssetByID`, `getAccountAssetsByIDs`, `getAccountAssetsByScope`) now always use Core `AssetsController` via `AssetsProvider`; snap-owned protocol assets remain on the Snap adapter. Removed `RemoteFeatureFlagController:getState` messenger endowment.
+
 ## [2.0.0]
 
 ### Changed

packages/tron-wallet-snap/snap.manifest.json

@@ -7,7 +7,7 @@
     "url": "https://github.com/MetaMask/internal-snaps.git"
   },
   "source": {
-    "shasum": "asiK6MnbNlIxxdMCvfOEHEp+28D1ZKGXL7Dg3t2V0Xs=",
+    "shasum": "ac/9ewd8ZZXTLl5K5ZMg5PSsUeITzUwYtvfHPNjKQu4=",
     "location": {
       "npm": {
         "filePath": "dist/bundle.js",

packages/tron-wallet-snap/src/context.ts

@@ -110,7 +110,6 @@ const securityAlertsApiClient = new SecurityAlertsApiClient(
 // Business Services
 const assetsService = new AssetsService({
   logger,
-  state,
   assetsRepository,
   trongridApiClient,
   tronHttpClient,

packages/tron-wallet-snap/src/services/accounts/AccountsService.test.ts

@@ -21,7 +21,6 @@ import {
 
 import type { SnapClient } from '../../clients/snap/SnapClient';
 import { Network } from '../../constants';
-import type { NativeAsset } from '../../entities/assets';
 import type { TronKeyringAccount } from '../../entities/keyring-account';
 import type { ILogger } from '../../utils/logger';
 import { mockLogger } from '../../utils/mockLogger';
@@ -119,9 +118,7 @@ type WithAccountsServiceCallback = (payload: {
   >;
   mockConfigProvider: jest.Mocked<Pick<ConfigProvider, 'get'>>;
   mockLogger: ILogger;
-  mockAssetsService: jest.Mocked<
-    Pick<AssetsService, 'fetchAssetsAndBalancesForAccount' | 'saveMany'>
-  >;
+  mockAssetsService: jest.Mocked<Pick<AssetsService, 'syncSnapOwnedAssets'>>;
   mockSnapClient: jest.Mocked<
     Pick<SnapClient, 'getBip32Entropy' | 'listEntropySources'>
   >;
@@ -268,10 +265,9 @@ async function withAccountsService(
   };
 
   const mockAssetsService: jest.Mocked<
-    Pick<AssetsService, 'fetchAssetsAndBalancesForAccount' | 'saveMany'>
+    Pick<AssetsService, 'syncSnapOwnedAssets'>
   > = {
-    fetchAssetsAndBalancesForAccount: jest.fn().mockResolvedValue([]),
-    saveMany: jest.fn().mockResolvedValue(undefined),
+    syncSnapOwnedAssets: jest.fn().mockResolvedValue(undefined),
   };
 
   const mockTransactionsService: jest.Mocked<
@@ -1194,7 +1190,7 @@ describe('AccountsService', () => {
   });
 
   describe('synchronizeAssets', () => {
-    it('calls fetch for each account and scope, then saveMany', async () => {
+    it('calls syncSnapOwnedAssets with accounts and active networks', async () => {
       const account: TronKeyringAccount = {
         id: 'sync-asset-id',
         address: 'TSyncAsset12345678901234567',
@@ -1206,42 +1202,22 @@ describe('AccountsService', () => {
         derivationPath: "m/44'/195'/0'/0/0",
         index: 0,
       };
-      const mockAssets: NativeAsset[] = [
-        {
-          assetType: `${Network.Mainnet}/slip44:195`,
-          keyringAccountId: 'sync-asset-id',
-          network: Network.Mainnet,
-          symbol: 'TRX',
-          decimals: 6,
-          rawAmount: '1000000',
-          uiAmount: '1',
-          iconUrl: '',
-        },
-      ];
 
       await withAccountsService(
         async ({ accountsService, mockConfigProvider, mockAssetsService }) => {
           mockConfigProvider.get.mockReturnValue({
             ...MOCK_CONFIG,
             activeNetworks: [Network.Mainnet, Network.Shasta],
           });
-          mockAssetsService.fetchAssetsAndBalancesForAccount.mockResolvedValue(
-            mockAssets,
-          );
 
           await accountsService.synchronizeAssets([account]);
 
-          expect(
-            mockAssetsService.fetchAssetsAndBalancesForAccount,
-          ).toHaveBeenCalledTimes(2);
-          expect(
-            mockAssetsService.fetchAssetsAndBalancesForAccount,
-          ).toHaveBeenCalledWith(Network.Mainnet, account);
-          expect(
-            mockAssetsService.fetchAssetsAndBalancesForAccount,
-          ).toHaveBeenCalledWith(Network.Shasta, account);
-          expect(mockAssetsService.saveMany).toHaveBeenCalledWith(
-            expect.arrayContaining(mockAssets),
+          expect(mockAssetsService.syncSnapOwnedAssets).toHaveBeenCalledTimes(
+            1,
+          );
+          expect(mockAssetsService.syncSnapOwnedAssets).toHaveBeenCalledWith(
+            [account],
+            [Network.Mainnet, Network.Shasta],
           );
         },
       );
@@ -1266,10 +1242,10 @@ describe('AccountsService', () => {
 
           await accountsService.synchronizeAssets([account]);
 
-          expect(
-            mockAssetsService.fetchAssetsAndBalancesForAccount,
-          ).not.toHaveBeenCalled();
-          expect(mockAssetsService.saveMany).toHaveBeenCalledWith([]);
+          expect(mockAssetsService.syncSnapOwnedAssets).toHaveBeenCalledWith(
+            [account],
+            [],
+          );
         },
       );
     });
@@ -1358,9 +1334,10 @@ describe('AccountsService', () => {
 
           await accountsService.synchronize([account]);
 
-          expect(
-            mockAssetsService.fetchAssetsAndBalancesForAccount,
-          ).toHaveBeenCalledWith(Network.Mainnet, account);
+          expect(mockAssetsService.syncSnapOwnedAssets).toHaveBeenCalledWith(
+            [account],
+            [Network.Mainnet],
+          );
           expect(
             mockTransactionsService.fetchNewTransactionsForAccount,
           ).toHaveBeenCalledWith(Network.Mainnet, account);

packages/tron-wallet-snap/src/services/accounts/AccountsService.ts

@@ -522,24 +522,7 @@ export class AccountsService {
    */
   async synchronizeAssets(accounts: TronKeyringAccount[]): Promise<void> {
     const scopes = this.#configProvider.get().activeNetworks;
-    const combinations = accounts.flatMap((account) =>
-      scopes.map((scope) => ({ account, scope })),
-    );
-
-    const assetResponses = await Promise.allSettled(
-      combinations.map(async ({ account, scope }) => {
-        return this.#assetsService.fetchAssetsAndBalancesForAccount(
-          scope,
-          account,
-        );
-      }),
-    );
-
-    const assets = assetResponses.flatMap((response) =>
-      response.status === 'fulfilled' ? response.value : [],
-    );
-
-    await this.#assetsService.saveMany(assets);
+    await this.#assetsService.syncSnapOwnedAssets(accounts, scopes);
   }
 
   async synchronizeTransactions(accounts: TronKeyringAccount[]): Promise<void> {