@metamask-previews/assets-controller 9.0.2-preview-8633f7f2a → 9.0.2-preview-9de9aa4

package/CHANGELOG.md

@@ -7,16 +7,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-### Added
-
-- Add `priceFreshnessTtlMs` option to `PriceDataSourceConfig` controlling how long a fetched price is considered fresh before it is re-fetched (defaults to the poll interval, 60 000 ms) ([#9189](https://github.com/MetaMask/core/pull/9189))
-- Add `PriceDataSource.invalidatePriceCache()` to force the next fetch to bypass the freshness cache; `AssetsController` now calls it when the selected currency changes so cached prices in the previous currency are refreshed ([#9189](https://github.com/MetaMask/core/pull/9189))
-
-### Fixed
-
-- Deduplicate price fetches in `PriceDataSource` so overlapping triggers (enrichment middleware, subscription polls, and manual refreshes) no longer issue duplicate price API requests for the same asset; assets fetched within the freshness TTL are skipped and concurrent in-flight fetches are joined ([#9189](https://github.com/MetaMask/core/pull/9189))
-- Remove a redundant one-time balance fetch in `AssetsController` when the enabled network list changes, since subscription refresh and the enabled-networks handler already cover those fetches ([#9189](https://github.com/MetaMask/core/pull/9189))
-
 ## [9.0.2]
 
 ### Changed

package/dist/AssetsController.cjs

@@ -748,8 +748,6 @@ class AssetsController extends base_controller_1.BaseController {
         if (!__classPrivateFieldGet(this, _AssetsController_isBasicFunctionality, "f").call(this)) {
             return;
         }
-        // Currency changed — old cached prices are in the wrong currency.
-        __classPrivateFieldGet(this, _AssetsController_priceDataSource, "f").invalidatePriceCache();
         this.getAssets(__classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_getSelectedAccounts).call(this), {
             forceUpdate: true,
             dataTypes: ['price'],
@@ -1159,12 +1157,23 @@ _AssetsController_isEnabled = new WeakMap(), _AssetsController_isBasicFunctional
     const addedChains = activeChains.filter((ch) => !previousSet.has(ch));
     const removedChains = previous.filter((ch) => !activeChains.includes(ch));
     if (addedChains.length > 0 || removedChains.length > 0) {
-        // Refresh subscriptions to use updated data source availability.
-        // No one-time fetch needed here — #handleEnabledNetworksChanged
-        // handles fetches when the user enables a network, and
-        // #subscribeAssets re-subscribes with the correct chain assignment.
+        // Refresh subscriptions to use updated data source availability
         __classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_subscribeAssets).call(this);
     }
+    // If chains were added and we have selected accounts, do one-time fetch
+    if (addedChains.length > 0 && __classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_getSelectedAccounts).call(this).length > 0) {
+        const addedEnabledChains = addedChains.filter((chain) => __classPrivateFieldGet(this, _AssetsController_enabledChains, "f").has(chain));
+        if (addedEnabledChains.length > 0) {
+            log('Fetching balances for newly added chains', { addedEnabledChains });
+            this.getAssets(__classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_getSelectedAccounts).call(this), {
+                chainIds: addedEnabledChains,
+                forceUpdate: true,
+                updateMode: 'merge',
+            }).catch((error) => {
+                log('Failed to fetch balance for added chains', { error });
+            });
+        }
+    }
 }, _AssetsController_executeMiddlewares = 
 // ============================================================================
 // MIDDLEWARE EXECUTION