@metamask-previews/network-controller 25.0.0-preview-2e88eaea → 25.0.0-preview-c96ff8f

package/CHANGELOG.md

@@ -11,8 +11,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - **BREAKING:** Use `InternalProvider` instead of `SafeEventEmitterProvider` ([#6796](https://github.com/MetaMask/core/pull/6796))
   - Providers accessible either via network clients or global proxies no longer emit events (or inherit from EventEmitter, for that matter).
+- **BREAKING:** Make `Provider` type more specific ([#7061](https://github.com/MetaMask/core/pull/7061))
+  - The `Provider` type is now an `InternalProvider` with a context type of `{ origin: string; skipCache: boolean } & Record<string, unknown>`.
+- **BREAKING:** Stop retrying `undefined` results for methods that include a block tag parameter ([#7001](https://github.com/MetaMask/core/pull/7001))
+  - The network client middleware, via `@metamask/eth-json-rpc-middleware`, will now throw an error if it encounters an
+    `undefined` result when dispatching a request with a later block number than the originally requested block number.
+  - In practice, this should happen rarely if ever.
+- **BREAKING:** Migrate `NetworkClient` to `JsonRpcEngineV2` ([#7065](https://github.com/MetaMask/core/pull/7065))
+  - This ought to be unobservable, but we mark it as breaking out of an abundance of caution.
 - Bump `@metamask/controller-utils` from `^11.14.1` to `^11.15.0` ([#7003](https://github.com/MetaMask/core/pull/7003))
 
+### Fixed
+
+- Ensure `networksMetadata` never references old network client IDs ([#7047](https://github.com/MetaMask/core/pull/7047))
+  - When removing a network configuration, ensure that metadata for all RPC endpoints in the network configuration are also removed from `networksMetadata`
+  - When initializing the controller, remove metadata for RPC endpoints in `networksMetadata` that are not present in a network configuration
+
 ## [25.0.0]
 
 ### Changed

package/dist/NetworkController.cjs

@@ -335,14 +335,21 @@ function validateInitialState(state) {
  */
 function correctInitialState(state, messenger) {
     const networkConfigurationsSortedByChainId = getNetworkConfigurations(state).sort((a, b) => a.chainId.localeCompare(b.chainId));
-    const networkClientIds = getAvailableNetworkClientIds(networkConfigurationsSortedByChainId);
+    const availableNetworkClientIds = getAvailableNetworkClientIds(networkConfigurationsSortedByChainId);
+    const invalidNetworkClientIdsWithMetadata = Object.keys(state.networksMetadata).filter((networkClientId) => !availableNetworkClientIds.includes(networkClientId));
     return (0, immer_1.produce)(state, (newState) => {
-        if (!networkClientIds.includes(state.selectedNetworkClientId)) {
+        if (!availableNetworkClientIds.includes(state.selectedNetworkClientId)) {
             const firstNetworkConfiguration = networkConfigurationsSortedByChainId[0];
             const newSelectedNetworkClientId = firstNetworkConfiguration.rpcEndpoints[firstNetworkConfiguration.defaultRpcEndpointIndex].networkClientId;
             messenger.call('ErrorReportingService:captureException', new Error(`\`selectedNetworkClientId\` '${state.selectedNetworkClientId}' does not refer to an RPC endpoint within a network configuration; correcting to '${newSelectedNetworkClientId}'`));
             newState.selectedNetworkClientId = newSelectedNetworkClientId;
         }
+        if (invalidNetworkClientIdsWithMetadata.length > 0) {
+            for (const invalidNetworkClientId of invalidNetworkClientIdsWithMetadata) {
+                delete newState.networksMetadata[invalidNetworkClientId];
+            }
+            messenger.call('ErrorReportingService:captureException', new Error('`networksMetadata` had invalid network client IDs, which have been removed'));
+        }
     });
 }
 /**
@@ -962,6 +969,9 @@ class NetworkController extends base_controller_1.BaseController {
                 mode: 'remove',
                 existingNetworkConfiguration,
             });
+            for (const rpcEndpoint of existingNetworkConfiguration.rpcEndpoints) {
+                delete state.networksMetadata[rpcEndpoint.networkClientId];
+            }
         });
         this.messenger.publish('NetworkController:networkRemoved', existingNetworkConfiguration);
     }
@@ -980,6 +990,8 @@ class NetworkController extends base_controller_1.BaseController {
      *
      * In-progress requests will not be aborted.
      */
+    // We're intentionally changing the signature of an extended method.
+    // eslint-disable-next-line @typescript-eslint/no-misused-promises
     async destroy() {
         await __classPrivateFieldGet(this, _NetworkController_blockTrackerProxy, "f")?.destroy();
     }
@@ -1236,6 +1248,8 @@ async function _NetworkController_lookupSelectedNetwork() {
     return new Promise((resolve, reject) => {
         ethQuery.sendAsync({ method: 'eth_getBlockByNumber', params: ['latest', false] }, (error, block) => {
             if (error) {
+                // This error comes from JsonRpcEngine, we don't control it.
+                // eslint-disable-next-line @typescript-eslint/prefer-promise-reject-errors
                 reject(error);
             }
             else {