feat(solana-wallet-snap): add read-only CoreAssetsAdapter and mapControllerAsset

eslint-suppressions.json

@@ -353,13 +353,10 @@
       "count": 2
     }
   },
-  "packages/solana-wallet-snap/src/core/services/assets/AssetsService.ts": {
+  "packages/solana-wallet-snap/src/core/services/assets/adapters/SnapAssetsAdapter.ts": {
     "@typescript-eslint/await-thenable": {
       "count": 1
     },
-    "@typescript-eslint/explicit-function-return-type": {
-      "count": 10
-    },
     "no-unused-private-class-members": {
       "count": 2
     }

packages/solana-wallet-snap/CHANGELOG.md

@@ -7,8 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Added
+
+- Add a read-only `CoreAssetsAdapter` and `mapControllerAsset` for AssetsController integration (wired unused until routing lands), including Core messenger plumbing (`coreMessenger`, `RemoteFeatureFlagsProvider`, `AssetsProvider`). Solana has no snap-owned assets, so the adapter does not fetch, persist, or publish balances, and the mapper does not derive associated token account addresses. ([#122](https://github.com/MetaMask/internal-snaps/pull/122))
+
 ### Changed
 
+- Extract Snap-owned assets domain logic into `SnapAssetsAdapter`; `AssetsService` is a thin facade that delegates metadata, market data, fetch, persist, and account asset reads through the adapter (no Core routing yet). ([#121](https://github.com/MetaMask/internal-snaps/pull/121))
 - Align `AssetsService` read API with `snap-networks-utils` / AssetsController shapes by adding `getAccountAssetByID`, `getAccountAssetsByIDs`, `getAccountAssetsByScope`, and `getAccountAssets`, and routing Keyring and Send through them (still Snap-owned storage). ([#120](https://github.com/MetaMask/internal-snaps/pull/120))
 
 ## [6.0.0]

packages/solana-wallet-snap/package.json

@@ -52,10 +52,14 @@
   },
   "devDependencies": {
     "@jest/globals": "^29.5.0",
+    "@metamask/assets-controller": "^13.0.0",
     "@metamask/auto-changelog": "^6.1.1",
     "@metamask/key-tree": "^10.1.1",
     "@metamask/keyring-api": "^23.7.0",
     "@metamask/keyring-snap-sdk": "^9.2.1",
+    "@metamask/messenger": "^2.0.0",
+    "@metamask/remote-feature-flag-controller": "^5.0.0",
+    "@metamask/snap-networks-utils": "^1.0.0",
     "@metamask/snaps-cli": "^8.4.1",
     "@metamask/snaps-jest": "^10.2.0",
     "@metamask/snaps-sdk": "^11.2.0",

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

@@ -7,7 +7,7 @@
     "url": "https://github.com/MetaMask/internal-snaps.git"
   },
   "source": {
-    "shasum": "YaPEFBNuMbbASorQCj4MFJKRSlqOjSOqJoOGUB9ET2I=",
+    "shasum": "2ZJZAnGhLs7mB0gCCb9ffito6qcx8KMGbteQ0wwtsYw=",
     "location": {
       "npm": {
         "filePath": "dist/bundle.js",
@@ -88,7 +88,15 @@
     "snap_manageAccounts": {},
     "snap_manageState": {},
     "snap_dialog": {},
-    "snap_getPreferences": {}
+    "snap_getPreferences": {},
+    "endowment:messenger": {
+      "actions": [
+        "RemoteFeatureFlagController:getState",
+        "AssetsController:getAccountAssetByID",
+        "AssetsController:getAccountAssetsByIDs",
+        "AssetsController:getAccountAssetsByScope"
+      ]
+    }
   },
   "platformVersion": "11.2.0",
   "manifestVersion": "0.1"

packages/solana-wallet-snap/src/core/services/assets/AssetsService.test.ts

@@ -24,6 +24,8 @@ import { mockLogger } from '../mocks/logger';
 import { createMockConnection } from '../mocks/mockConnection';
 import { MOCK_SOLANA_RPC_GET_TOKEN_ACCOUNTS_BY_OWNER_RESPONSE } from '../mocks/mockSolanaRpcResponses';
 import type { TokenPricesService } from '../token-prices/TokenPrices';
+import { CoreAssetsAdapter } from './adapters/CoreAssetsAdapter';
+import { SnapAssetsAdapter } from './adapters/SnapAssetsAdapter';
 import type { AssetsRepository } from './AssetsRepository';
 import { AssetsService } from './AssetsService';
 
@@ -33,6 +35,7 @@ jest.mock('@metamask/keyring-snap-sdk', () => ({
 
 describe('AssetsService', () => {
   let assetsService: AssetsService;
+  let snapAssetsAdapter: SnapAssetsAdapter;
   let mockConnection: SolanaConnection;
   let mockConfigProvider: ConfigProvider;
   let mockAssetsRepository: AssetsRepository;
@@ -87,7 +90,7 @@ describe('AssetsService', () => {
       findById: jest.fn().mockResolvedValue(MOCK_SOLANA_KEYRING_ACCOUNT_0),
     } as unknown as AccountsService;
 
-    assetsService = new AssetsService({
+    snapAssetsAdapter = new SnapAssetsAdapter({
       connection: mockConnection,
       logger: mockLogger,
       configProvider: mockConfigProvider,
@@ -98,6 +101,20 @@ describe('AssetsService', () => {
       cache: mockCache,
       nftApiClient: mockNftApiClient,
     });
+
+    const coreAdapter = new CoreAssetsAdapter({
+      getAccountAssetByID: jest.fn().mockResolvedValue(null),
+      getAccountAssetsByIDs: jest.fn().mockResolvedValue({}),
+      getAccountAssetsByScope: jest.fn().mockResolvedValue({}),
+      findAccountById: mockAccountsService.findById.bind(mockAccountsService),
+      getActiveNetworks:
+        mockConfigProvider.getActiveNetworks.bind(mockConfigProvider),
+    });
+
+    assetsService = new AssetsService({
+      snapAdapter: snapAssetsAdapter,
+      coreAdapter,
+    });
   });
 
   describe('fetch', () => {
@@ -173,6 +190,45 @@ describe('AssetsService', () => {
     });
   });
 
+  describe('getAssetsMetadata', () => {
+    it('fetches token metadata from the token API client', async () => {
+      const tokenAssetTypes = [
+        MOCK_ASSET_ENTITY_1.assetType,
+        MOCK_ASSET_ENTITY_2.assetType,
+      ];
+
+      const metadata = await assetsService.getAssetsMetadata(tokenAssetTypes);
+
+      expect(mockTokenApiClient.getTokensMetadata).toHaveBeenCalledWith(
+        tokenAssetTypes,
+      );
+      expect(metadata).toStrictEqual(SOLANA_MOCK_TOKEN_METADATA);
+    });
+  });
+
+  describe('fetchAssetsMarketData', () => {
+    it('delegates to the token prices service', async () => {
+      const assets = [
+        {
+          asset: MOCK_ASSET_ENTITY_0.assetType,
+          unit: MOCK_ASSET_ENTITY_0.assetType,
+        },
+      ];
+      const expected = { [MOCK_ASSET_ENTITY_0.assetType]: {} };
+
+      jest
+        .spyOn(mockTokenPricesService, 'getMultipleTokensMarketData')
+        .mockResolvedValueOnce(expected as never);
+
+      const result = await assetsService.fetchAssetsMarketData(assets);
+
+      expect(
+        mockTokenPricesService.getMultipleTokensMarketData,
+      ).toHaveBeenCalledWith(assets);
+      expect(result).toStrictEqual(expected);
+    });
+  });
+
   describe('save', () => {
     it('saves an asset', async () => {
       const spy = jest