@metamask-previews/assets-controller 3.1.1-preview-55f19615e → 3.2.0-preview-4f74a56

package/CHANGELOG.md

@@ -7,21 +7,35 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [3.2.0]
+
 ### Added
 
 - Add Sentry traces for Assets Health dashboard ([#8310](https://github.com/MetaMask/core/pull/8310))
-  - `Assets Data Source Timing` — per-source latency for each middleware in the fetch pipeline
-  - `Assets Data Source Error` — tracks middleware failures with source names and error counts
-  - `Assets Full Fetch` — end-to-end fetch timing with asset/price/chain/account counts
-  - `Assets Update Pipeline` — enrichment pipeline timing for pushed data source updates
-  - `Assets Subscription Error` — subscription failure tracking per data source
-  - `Assets State Size` — entry counts for balances, metadata, prices, and custom assets (once on app start)
-  - `Aggregated Balance Selector` — balance selector computation time with asset/network/account counts
+  - `AssetsDataSourceTiming` — per-source latency for each middleware in the fetch pipeline
+  - `AssetsDataSourceError` — tracks middleware failures with source names and error counts
+  - `AssetsFullFetch` — end-to-end fetch timing with asset/price/chain/account counts
+  - `AssetsUpdatePipeline` — enrichment pipeline timing for pushed data source updates
+  - `AssetsSubscriptionError` — subscription failure tracking per data source
+  - `AssetsStateSize` — entry counts for balances, metadata, prices, custom assets, unique assets, and network count (once on app start)
+  - `AggregatedBalanceSelector` — balance selector computation time with asset/network/account counts
   - Add optional `trace` parameter to `getAggregatedBalanceForAccount` selector
 
 ### Changed
 
-- Bump `@metamask/transaction-controller` from `^63.1.0` to `^63.2.0` ([#8301](https://github.com/MetaMask/core/pull/8301))
+- Bump `@metamask/account-tree-controller` from `^5.0.1` to `^6.0.0` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/assets-controllers` from `^101.0.1` to `^102.0.0` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/base-controller` from `^9.0.0` to `^9.0.1` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/client-controller` from `^1.0.0` to `^1.0.1` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/core-backend` from `^6.2.0` to `^6.2.1` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/keyring-controller` from `^25.1.0` to `^25.1.1` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/messenger` from `^0.3.0` to `^1.0.0` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/network-controller` from `^30.0.0` to `^30.0.1` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/network-enablement-controller` from `^5.0.0` to `^5.0.1` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/permission-controller` from `^12.2.1` to `^12.3.0` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/polling-controller` from `^16.0.3` to `^16.0.4` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/preferences-controller` from `^23.0.0` to `^23.1.0` ([#8317](https://github.com/MetaMask/core/pull/8317))
+- Bump `@metamask/transaction-controller` from `^63.1.0` to `^63.3.1` ([#8301](https://github.com/MetaMask/core/pull/8301), [#8313](https://github.com/MetaMask/core/pull/8313), [#8317](https://github.com/MetaMask/core/pull/8317))
 
 ## [3.1.1]
 
@@ -224,7 +238,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Refactor `RpcDataSource` to delegate polling to `BalanceFetcher` and `TokenDetector` services ([#7709](https://github.com/MetaMask/core/pull/7709))
 - Refactor `BalanceFetcher` and `TokenDetector` to extend `StaticIntervalPollingControllerOnly` for independent polling management ([#7709](https://github.com/MetaMask/core/pull/7709))
 
-[Unreleased]: https://github.com/MetaMask/core/compare/@metamask/assets-controller@3.1.1...HEAD
+[Unreleased]: https://github.com/MetaMask/core/compare/@metamask/assets-controller@3.2.0...HEAD
+[3.2.0]: https://github.com/MetaMask/core/compare/@metamask/assets-controller@3.1.1...@metamask/assets-controller@3.2.0
 [3.1.1]: https://github.com/MetaMask/core/compare/@metamask/assets-controller@3.1.0...@metamask/assets-controller@3.1.1
 [3.1.0]: https://github.com/MetaMask/core/compare/@metamask/assets-controller@3.0.0...@metamask/assets-controller@3.1.0
 [3.0.0]: https://github.com/MetaMask/core/compare/@metamask/assets-controller@2.4.0...@metamask/assets-controller@3.0.0

package/dist/AssetsController.cjs

@@ -56,13 +56,13 @@ const DEFAULT_POLLING_INTERVAL_MS = 30000;
 // ============================================================================
 // TRACE NAMES — used in Sentry spans (search these strings in Discover)
 // ============================================================================
-const TRACE_FIRST_INIT_FETCH = 'AssetsController First Init Fetch';
-const TRACE_FULL_FETCH = 'Assets Full Fetch';
-const TRACE_DATA_SOURCE_TIMING = 'Assets Data Source Timing';
-const TRACE_DATA_SOURCE_ERROR = 'Assets Data Source Error';
-const TRACE_UPDATE_PIPELINE = 'Assets Update Pipeline';
-const TRACE_SUBSCRIPTION_ERROR = 'Assets Subscription Error';
-const TRACE_STATE_SIZE = 'Assets State Size';
+const TRACE_FIRST_INIT_FETCH = 'AssetsControllerFirstInitFetch';
+const TRACE_FULL_FETCH = 'AssetsFullFetch';
+const TRACE_DATA_SOURCE_TIMING = 'AssetsDataSourceTiming';
+const TRACE_DATA_SOURCE_ERROR = 'AssetsDataSourceError';
+const TRACE_UPDATE_PIPELINE = 'AssetsUpdatePipeline';
+const TRACE_SUBSCRIPTION_ERROR = 'AssetsSubscriptionError';
+const TRACE_STATE_SIZE = 'AssetsStateSize';
 const log = (0, logger_1.createModuleLogger)(logger_1.projectLogger, CONTROLLER_NAME);
 /**
  * Returns the default state for AssetsController.
@@ -347,7 +347,7 @@ class AssetsController extends base_controller_1.BaseController {
             customAssets.push(...accountCustomAssets);
         }
         if (options?.forceUpdate) {
-            const startTime = Date.now();
+            const startTime = performance.now();
             const request = __classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_buildDataRequest).call(this, accounts, chainIds, {
                 assetTypes,
                 dataTypes,
@@ -381,7 +381,7 @@ class AssetsController extends base_controller_1.BaseController {
                 options.chainIds.length < __classPrivateFieldGet(this, _AssetsController_enabledChains, "f").size;
             const updateMode = options?.updateMode ?? (isPartialChainFetch ? 'merge' : 'full');
             await __classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_updateState).call(this, { ...response, updateMode });
-            const durationMs = Date.now() - startTime;
+            const durationMs = performance.now() - startTime;
             // Emit trace for every full fetch (Assets Health dashboard)
             __classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_emitTrace).call(this, TRACE_FULL_FETCH, {
                 duration_ms: durationMs,
@@ -728,7 +728,7 @@ class AssetsController extends base_controller_1.BaseController {
      * @param request - Optional original request for context when enriching
      */
     async handleAssetsUpdate(response, sourceId, request) {
-        const updateStart = Date.now();
+        const updateStart = performance.now();
         log('Assets updated from data source', {
             sourceId,
             hasBalance: Boolean(response.assetsBalance),
@@ -750,7 +750,7 @@ class AssetsController extends base_controller_1.BaseController {
         await __classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_updateState).call(this, enrichedResponse);
         __classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_emitTrace).call(this, TRACE_UPDATE_PIPELINE, {
             source: sourceId,
-            duration_ms: Date.now() - updateStart,
+            duration_ms: performance.now() - updateStart,
             has_balance: Boolean(response.assetsBalance),
             has_price: Boolean(response.assetsPrice),
             has_metadata: Boolean(enrichedResponse.assetsInfo),
@@ -800,23 +800,30 @@ _AssetsController_isEnabled = new WeakMap(), _AssetsController_isBasicFunctional
     if (!__classPrivateFieldGet(this, _AssetsController_trace, "f")) {
         return;
     }
-    try {
-        __classPrivateFieldGet(this, _AssetsController_trace, "f").call(this, { name, data, tags }, () => undefined).catch(() => {
-            // Telemetry failure must not break.
-        });
-    }
-    catch {
+    __classPrivateFieldGet(this, _AssetsController_trace, "f").call(this, { name, data, tags }, () => undefined).catch(() => {
         // Telemetry failure must not break.
-    }
+    });
 }, _AssetsController_emitStateSizeTrace = function _AssetsController_emitStateSizeTrace() {
     if (!__classPrivateFieldGet(this, _AssetsController_trace, "f") || __classPrivateFieldGet(this, _AssetsController_stateSizeReported, "f")) {
         return;
     }
     __classPrivateFieldSet(this, _AssetsController_stateSizeReported, true, "f");
     const { assetsBalance: balances, customAssets, assetsInfo, assetsPrice, } = this.state;
+    // Count balance entries and collect unique asset IDs / chain IDs in one pass.
     let balanceEntries = 0;
+    const uniqueAssets = new Set();
+    const uniqueNetworks = new Set();
     for (const acct of Object.values(balances)) {
-        balanceEntries += Object.keys(acct).length;
+        const assetIds = Object.keys(acct);
+        balanceEntries += assetIds.length;
+        for (const assetId of assetIds) {
+            uniqueAssets.add(assetId);
+            // CAIP-19 format: "eip155:1/slip44:60" — chainId is everything before "/"
+            const slash = assetId.indexOf('/');
+            if (slash > 0) {
+                uniqueNetworks.add(assetId.slice(0, slash));
+            }
+        }
     }
     let customAssetEntries = 0;
     for (const ids of Object.values(customAssets)) {
@@ -827,6 +834,8 @@ _AssetsController_isEnabled = new WeakMap(), _AssetsController_isBasicFunctional
     __classPrivateFieldGet(this, _AssetsController_instances, "m", _AssetsController_emitTrace).call(this, TRACE_STATE_SIZE, {
         balance_entries: balanceEntries,
         balance_accounts: Object.keys(balances).length,
+        unique_asset_count: uniqueAssets.size,
+        network_count: uniqueNetworks.size,
         metadata_entries: Object.keys(assetsInfo).length,
         price_entries: Object.keys(assetsPrice).length,
         custom_asset_entries: customAssetEntries,
@@ -953,12 +962,12 @@ async function _AssetsController_executeMiddlewares(sources, request, initialRes
     const middlewares = sources.map((source) => source.assetsMiddleware);
     const inclusive = [];
     const wrapped = middlewares.map((middleware, i) => (async (ctx, next) => {
-        const start = Date.now();
+        const start = performance.now();
         try {
             return await middleware(ctx, next);
         }
         finally {
-            inclusive[i] = Date.now() - start;
+            inclusive[i] = performance.now() - start;
         }
     }));
     const middlewareErrors = [];