fix: Delete invalid SelectedNetworkController state

[Gudahtt]

Description

The SelectedNetworkController state is cleared if any invalid networkConfigurationIds are found in state. We are seeing reports of this happening in production in v12.0.1.

The suspected cause is NetworkController state corruption. We resolved a few cases of this in v12.0.1, but for users that were affected by this, the invalid IDs may have propogated to the SelectedNetworkController state already. That is what this migration intends to fix.

Related issues

Fixes #26309

Manual testing steps

We don't have clear reproduction steps for the bug itself. To artificially reproduce the scenario by changing extension state, this should work:

• Create a dev build from v12.0.2
• Install the dev build from the dist/chrome directory and proceed through onboarding
• Visit the test dapp, refresh, and connect to it.
• Run this command in the background console:

  chrome.storage.local.get(
    null,
    (state) => {
      state.data.SelectedNetworkController.domains['https://metamask.github.io'] = '123';
      chrome.storage.local.set(state, () => chrome.runtime.reload());
    }
  );
  

  • The linked error should now appear in the console of the popup
• Disable the extension
• Switch to this branch and create a dev build
• Enable and reload the extension
  • The error should no longer appear.

Screenshots/Recordings

N/A

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.02%. Comparing base (c5c13d5) to head (70795ce).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #26428      +/-   ##
===========================================
+ Coverage    69.99%   70.02%   +0.03%     
===========================================
  Files         1424     1425       +1     
  Lines        49984    50028      +44     
  Branches     13872    13883      +11     
===========================================
+ Hits         34986    35031      +45     
+ Misses       14998    14997       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[metamaskbot]

Builds ready [70795ce]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (76 ± 16 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	73	281	110	44	21
		domContentLoaded	9	87	27	17	8
		load	41	197	76	34	16
		domInteractive	9	87	27	17	8

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 0 Bytes (0.00%)
• common: 0 Bytes (0.00%)

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[DDDDDanica]

LGTM ! The tests are very solid

[metamaskbot]

Builds ready [26f7fa1]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (144 ± 159 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	73	167	104	28	13
		domContentLoaded	9	71	24	12	6
		load	40	1582	144	331	159
		domInteractive	9	71	24	12	6

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 0 Bytes (0.00%)
• common: 0 Bytes (0.00%)