@metamask-previews/network-controller 25.0.0-preview-a345102 → 25.0.0-preview-317c0a9f

package/CHANGELOG.md

@@ -11,22 +11,8 @@ 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,21 +335,14 @@ function validateInitialState(state) {
  */
 function correctInitialState(state, messenger) {
     const networkConfigurationsSortedByChainId = getNetworkConfigurations(state).sort((a, b) => a.chainId.localeCompare(b.chainId));
-    const availableNetworkClientIds = getAvailableNetworkClientIds(networkConfigurationsSortedByChainId);
-    const invalidNetworkClientIdsWithMetadata = Object.keys(state.networksMetadata).filter((networkClientId) => !availableNetworkClientIds.includes(networkClientId));
+    const networkClientIds = getAvailableNetworkClientIds(networkConfigurationsSortedByChainId);
     return (0, immer_1.produce)(state, (newState) => {
-        if (!availableNetworkClientIds.includes(state.selectedNetworkClientId)) {
+        if (!networkClientIds.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'));
-        }
     });
 }
 /**
@@ -969,9 +962,6 @@ 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);
     }
@@ -990,8 +980,6 @@ 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();
     }
@@ -1248,8 +1236,6 @@ 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 {