feat: implement onAssetsMarketData handler

Author: ulissesferreira

package.json

@@ -36,7 +36,7 @@
     ]
   },
   "resolutions": {
-    "@metamask/snaps-sdk": "8.1.0"
+    "@metamask/snaps-sdk": "9.0.0"
   },
   "devDependencies": {
     "@commitlint/cli": "^17.7.1",
@@ -69,7 +69,7 @@
     "sharp": "^0.33.5",
     "typescript": "^5.6.3"
   },
-  "packageManager": "yarn@4.9.2",
+  "packageManager": "yarn@3.8.6",
   "engines": {
     "node": ">=20.17.0"
   },

packages/snap/package.json

@@ -45,7 +45,7 @@
     "@metamask/keyring-snap-sdk": "^4.0.0",
     "@metamask/snaps-cli": "^8.1.0",
     "@metamask/snaps-jest": "9.2.0",
-    "@metamask/snaps-sdk": "^8.1.0",
+    "@metamask/snaps-sdk": "^9.0.0",
     "@metamask/superstruct": "^3.1.0",
     "@metamask/utils": "11.4.0",
     "@noble/ed25519": "2.1.0",

packages/snap/snap.manifest.json

@@ -7,7 +7,7 @@
     "url": "https://github.com/MetaMask/snap-solana-wallet.git"
   },
   "source": {
-    "shasum": "g/jfpUCvmwu6piUwA21tuyxrmaZZnZl3JdLeVJysYFM=",
+    "shasum": "7CefbKb7jCuxLkSz1ySQYgmQIh8ih3OBOVXeVpfIirU=",
     "location": {
       "npm": {
         "filePath": "dist/bundle.js",
@@ -100,6 +100,6 @@
     "snap_dialog": {},
     "snap_getPreferences": {}
   },
-  "platformVersion": "8.1.0",
+  "platformVersion": "9.0.0",
   "manifestVersion": "0.1"
-}
+}

packages/snap/src/core/handlers/onAssetHistoricalPrice/onAssetHistoricalPrice.ts

@@ -15,23 +15,15 @@ import logger from '../../utils/logger';
 export const onAssetHistoricalPrice: OnAssetHistoricalPriceHandler = async (
   params,
 ) => {
-  try {
-    logger.log('[📈 onAssetHistoricalPrice]', params);
+  logger.log('[📈 onAssetHistoricalPrice]', params);
 
-    const { from, to } = params;
-    assert(from, CaipAssetTypeStruct);
-    assert(to, CaipAssetTypeStruct);
+  const { from, to } = params;
+  assert(from, CaipAssetTypeStruct);
+  assert(to, CaipAssetTypeStruct);
 
-    const historicalPrice = await tokenPricesService.getHistoricalPrice(
-      from,
-      to,
-    );
+  const historicalPrice = await tokenPricesService.getHistoricalPrice(from, to);
 
-    return {
-      historicalPrice,
-    };
-  } catch (error: any) {
-    logger.error('[📈 onAssetHistoricalPrice]', error);
-    return null;
-  }
+  return {
+    historicalPrice,
+  };
 };

packages/snap/src/core/handlers/onAssetsConversion/onAssetsConversion.ts

@@ -1,25 +1,17 @@
-import {
-  InternalError,
-  type OnAssetsConversionHandler,
-} from '@metamask/snaps-sdk';
+import { type OnAssetsConversionHandler } from '@metamask/snaps-sdk';
 
 import { tokenPricesService } from '../../../snapContext';
 import logger from '../../utils/logger';
 
 export const onAssetsConversion: OnAssetsConversionHandler = async (params) => {
-  try {
-    logger.log('[💱 onAssetsConversion]', params);
+  logger.log('[💱 onAssetsConversion]', params);
 
-    const { conversions } = params;
+  const { conversions } = params;
 
-    const conversionRates =
-      await tokenPricesService.getMultipleTokenConversions(conversions);
+  const conversionRates =
+    await tokenPricesService.getMultipleTokenConversions(conversions);
 
-    return {
-      conversionRates,
-    };
-  } catch (error: any) {
-    logger.error('[💱 onAssetsConversion]', error);
-    throw new InternalError(error) as Error;
-  }
+  return {
+    conversionRates,
+  };
 };

packages/snap/src/core/handlers/onAssetsMarketData/onAssetsMarketData.ts

@@ -1,20 +1,14 @@
 import type { OnAssetsMarketDataHandler } from '@metamask/snaps-sdk';
-import { InternalError } from '@metamask/snaps-sdk';
 
 import { assetsService } from '../../../snapContext';
 import logger from '../../utils/logger';
 
 export const onAssetsMarketData: OnAssetsMarketDataHandler = async (params) => {
-  try {
-    logger.log('[💰 onAssetsMarketData]', params);
+  logger.log('[💰 onAssetsMarketData]', params);
 
-    const { assets } = params;
+  const { assets } = params;
 
-    const marketData = await assetsService.getAssetsMarketData(assets);
+  const marketData = await assetsService.getAssetsMarketData(assets);
 
-    return { marketData };
-  } catch (error: any) {
-    logger.error('[💰 onAssetsMarketData]', error);
-    throw new InternalError(error) as Error;
-  }
+  return { marketData };
 };