feat: price API v3 upgrade

app/scripts/controller-init/messengers/assets/token-rates-controller-messenger.ts

@@ -1,14 +1,8 @@
 import { Messenger } from '@metamask/messenger';
 import {
   NetworkControllerGetStateAction,
-  NetworkControllerGetNetworkClientByIdAction,
   NetworkControllerStateChangeEvent,
 } from '@metamask/network-controller';
-import {
-  AccountsControllerGetSelectedAccountAction,
-  AccountsControllerGetAccountAction,
-  AccountsControllerSelectedEvmAccountChangeEvent,
-} from '@metamask/accounts-controller';
 import {
   TokensControllerGetStateAction,
   TokensControllerStateChangeEvent,
@@ -19,18 +13,11 @@ import {
 } from '../../../controllers/preferences-controller';
 import { RootMessenger } from '../../../lib/messenger';
 
-type Actions =
-  | TokensControllerGetStateAction
-  | NetworkControllerGetNetworkClientByIdAction
-  | NetworkControllerGetStateAction
-  | AccountsControllerGetAccountAction
-  | AccountsControllerGetSelectedAccountAction;
+type Actions = TokensControllerGetStateAction | NetworkControllerGetStateAction;
 
 type Events =
-  | NetworkControllerStateChangeEvent
-  | AccountsControllerSelectedEvmAccountChangeEvent
-  | PreferencesControllerStateChangeEvent
-  | TokensControllerStateChangeEvent;
+  | TokensControllerStateChangeEvent
+  | NetworkControllerStateChangeEvent;
 
 export type TokenRatesControllerMessenger = ReturnType<
   typeof getTokenRatesControllerMessenger
@@ -57,19 +44,8 @@ export function getTokenRatesControllerMessenger(
   });
   messenger.delegate({
     messenger: controllerMessenger,
-    actions: [
-      'TokensController:getState',
-      'NetworkController:getNetworkClientById',
-      'NetworkController:getState',
-      'AccountsController:getAccount',
-      'AccountsController:getSelectedAccount',
-    ],
-    events: [
-      'NetworkController:stateChange',
-      'AccountsController:selectedEvmAccountChange',
-      'PreferencesController:stateChange',
-      'TokensController:stateChange',
-    ],
+    actions: ['TokensController:getState', 'NetworkController:getState'],
+    events: ['TokensController:stateChange', 'NetworkController:stateChange'],
   });
   return controllerMessenger;
 }

lavamoat/browserify/beta/policy.json

@@ -842,7 +842,7 @@
         "@metamask/keyring-snap-client": true,
         "@metamask/metamask-eth-abis": true,
         "@metamask/phishing-controller": true,
-        "@metamask/bridge-controller>@metamask/polling-controller": true,
+        "@metamask/assets-controllers>@metamask/polling-controller": true,
         "@metamask/rpc-errors": true,
         "@metamask/snaps-utils": true,
         "@metamask/utils": true,
@@ -1682,6 +1682,18 @@
         "punycode": true
       }
     },
+    "@metamask/assets-controllers>@metamask/polling-controller": {
+      "globals": {
+        "clearTimeout": true,
+        "console.error": true,
+        "setTimeout": true
+      },
+      "packages": {
+        "@metamask/base-controller": true,
+        "@metamask/snaps-utils>fast-json-stable-stringify": true,
+        "uuid": true
+      }
+    },
     "@metamask/bridge-controller>@metamask/polling-controller": {
       "globals": {
         "clearTimeout": true,

lavamoat/browserify/experimental/policy.json

@@ -842,7 +842,7 @@
         "@metamask/keyring-snap-client": true,
         "@metamask/metamask-eth-abis": true,
         "@metamask/phishing-controller": true,
-        "@metamask/bridge-controller>@metamask/polling-controller": true,
+        "@metamask/assets-controllers>@metamask/polling-controller": true,
         "@metamask/rpc-errors": true,
         "@metamask/snaps-utils": true,
         "@metamask/utils": true,
@@ -1682,6 +1682,18 @@
         "punycode": true
       }
     },
+    "@metamask/assets-controllers>@metamask/polling-controller": {
+      "globals": {
+        "clearTimeout": true,
+        "console.error": true,
+        "setTimeout": true
+      },
+      "packages": {
+        "@metamask/base-controller": true,
+        "@metamask/snaps-utils>fast-json-stable-stringify": true,
+        "uuid": true
+      }
+    },
     "@metamask/bridge-controller>@metamask/polling-controller": {
       "globals": {
         "clearTimeout": true,

lavamoat/browserify/flask/policy.json

@@ -842,7 +842,7 @@
         "@metamask/keyring-snap-client": true,
         "@metamask/metamask-eth-abis": true,
         "@metamask/phishing-controller": true,
-        "@metamask/bridge-controller>@metamask/polling-controller": true,
+        "@metamask/assets-controllers>@metamask/polling-controller": true,
         "@metamask/rpc-errors": true,
         "@metamask/snaps-utils": true,
         "@metamask/utils": true,
@@ -1682,6 +1682,18 @@
         "punycode": true
       }
     },
+    "@metamask/assets-controllers>@metamask/polling-controller": {
+      "globals": {
+        "clearTimeout": true,
+        "console.error": true,
+        "setTimeout": true
+      },
+      "packages": {
+        "@metamask/base-controller": true,
+        "@metamask/snaps-utils>fast-json-stable-stringify": true,
+        "uuid": true
+      }
+    },
     "@metamask/bridge-controller>@metamask/polling-controller": {
       "globals": {
         "clearTimeout": true,

lavamoat/browserify/main/policy.json

@@ -842,7 +842,7 @@
         "@metamask/keyring-snap-client": true,
         "@metamask/metamask-eth-abis": true,
         "@metamask/phishing-controller": true,
-        "@metamask/bridge-controller>@metamask/polling-controller": true,
+        "@metamask/assets-controllers>@metamask/polling-controller": true,
         "@metamask/rpc-errors": true,
         "@metamask/snaps-utils": true,
         "@metamask/utils": true,
@@ -1682,6 +1682,18 @@
         "punycode": true
       }
     },
+    "@metamask/assets-controllers>@metamask/polling-controller": {
+      "globals": {
+        "clearTimeout": true,
+        "console.error": true,
+        "setTimeout": true
+      },
+      "packages": {
+        "@metamask/base-controller": true,
+        "@metamask/snaps-utils>fast-json-stable-stringify": true,
+        "uuid": true
+      }
+    },
     "@metamask/bridge-controller>@metamask/polling-controller": {
       "globals": {
         "clearTimeout": true,

lavamoat/webpack/mv2/policy.json

@@ -865,7 +865,7 @@
         "@metamask/keyring-snap-client": true,
         "@metamask/metamask-eth-abis": true,
         "@metamask/phishing-controller": true,
-        "@metamask/bridge-controller>@metamask/polling-controller": true,
+        "@metamask/assets-controllers>@metamask/polling-controller": true,
         "@metamask/rpc-errors": true,
         "@metamask/snaps-utils": true,
         "@metamask/utils": true,
@@ -1753,6 +1753,18 @@
         "punycode": true
       }
     },
+    "@metamask/assets-controllers>@metamask/polling-controller": {
+      "globals": {
+        "clearTimeout": true,
+        "console.error": true,
+        "setTimeout": true
+      },
+      "packages": {
+        "@metamask/base-controller": true,
+        "@metamask/snaps-utils>fast-json-stable-stringify": true,
+        "uuid": true
+      }
+    },
     "@metamask/bridge-controller>@metamask/polling-controller": {
       "globals": {
         "clearTimeout": true,

package.json

@@ -274,7 +274,7 @@
     "@metamask/address-book-controller": "^7.0.0",
     "@metamask/announcement-controller": "^8.0.0",
     "@metamask/approval-controller": "^8.0.0",
-    "@metamask/assets-controllers": "patch:@metamask/assets-controllers@patch%3A@metamask/assets-controllers@npm%253A89.0.1%23~/.yarn/patches/@metamask-assets-controllers-npm-89.0.1-02fa7acd54.patch%3A%3Aversion=89.0.1&hash=6be0d3#~/.yarn/patches/@metamask-assets-controllers-patch-7c7d711c8c.patch",
+    "@metamask/assets-controllers": "patch:@metamask/assets-controllers@npm%3A91.0.0#~/.yarn/patches/@metamask-assets-controllers-npm-91.0.0-ea998cb0bd.patch",
     "@metamask/base-controller": "^9.0.0",
     "@metamask/bitcoin-wallet-snap": "^1.7.0",
     "@metamask/bridge-controller": "patch:@metamask/bridge-controller@npm%3A61.0.0#~/.yarn/patches/@metamask-bridge-controller-npm-61.0.0-8c413c463f.patch",

ui/hooks/useTokenRatesPolling.ts

@@ -1,11 +1,5 @@
 import { useSelector } from 'react-redux';
-import {
-  getChainIdsToPoll,
-  getMarketData,
-  getTokenExchangeRates,
-  getTokensMarketData,
-  getUseCurrencyRateCheck,
-} from '../selectors';
+import { getChainIdsToPoll, getUseCurrencyRateCheck } from '../selectors';
 import { getEnabledChainIds } from '../selectors/multichain/networks';
 import {
   tokenRatesStartPolling,
@@ -26,11 +20,6 @@ const useTokenRatesPolling = () => {
   const chainIds = useSelector(getChainIdsToPoll);
   const enabledChainIds = useSelector(getEnabledChainIds);
 
-  // Selectors returning state updated by the polling
-  const tokenExchangeRates = useSelector(getTokenExchangeRates);
-  const tokensMarketData = useSelector(getTokensMarketData);
-  const marketData = useSelector(getMarketData);
-
   const enabled = completedOnboarding && isUnlocked && useCurrencyRateCheck;
 
   const pollableChains = isGlobalNetworkSelectorRemoved
@@ -44,12 +33,6 @@ const useTokenRatesPolling = () => {
     stopPollingByPollingToken: tokenRatesStopPollingByPollingToken,
     input: enabled ? [pollableChains] : [],
   });
-
-  return {
-    tokenExchangeRates,
-    tokensMarketData,
-    marketData,
-  };
 };
 
 export default useTokenRatesPolling;