fix: bump accounts controller and migration to fix undefined selectedAccount

[montelaidev]

Description

This PR bumps the AccountsController and introduces a new migration. The updateAccounts methods from the AccountsController now checks if the selectedAccount is undefined and recovers from this. The migration updates the selectedAccount values that are not defined.

Related issues

Fixes: #26377

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

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.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[socket-security]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@metamask/[email protected]	None	0	391 kB	metamaskbot
npm/@metamask/[email protected]	None	0	250 kB	metamaskbot

🚮 Removed packages: npm/@metamask/[email protected]), npm/@metamask/[email protected])

View full report↗︎

[montelaidev]

@metamaskbot update-policies

[metamaskbot]

Policies updated

[codecov]

Codecov Report

Attention: Patch coverage is 95.65217% with 1 line in your changes missing coverage. Please review.

Project coverage is 70.05%. Comparing base (590f0a9) to head (e6a2d32).
Report is 19 commits behind head on develop.

Files	Patch %	Lines
...nents/multichain/pages/connections/connections.tsx	88.89%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #26573      +/-   ##
===========================================
+ Coverage    70.02%   70.05%   +0.02%     
===========================================
  Files         1406     1411       +5     
  Lines        49076    49219     +143     
  Branches     13725    13757      +32     
===========================================
+ Hits         34364    34476     +112     
- Misses       14712    14743      +31     

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

[metamaskbot]

Builds ready [7d5955c]

• 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
  • common: 0, 1, 10, 11, 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 ± 7 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	74	124	101	14	7
		domContentLoaded	50	95	72	13	6
		load	50	102	76	14	7
		domInteractive	9	53	28	9	4

Bundle size diffs [🚨 Warning! Bundle size has increased!]

• background: 48.78 KiB (1.37%)
• ui: -5 Bytes (-0.00%)
• common: 26.6 KiB (0.32%)

[gantunesr]

Looks like, left some minor comments

[sonarcloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

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

See analysis details on SonarCloud

[metamaskbot]

Builds ready [e6a2d32]

• 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
  • common: 0, 1, 10, 11, 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 (82 ± 18 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	65	289	118	52	25
		domContentLoaded	39	201	79	39	19
		load	46	201	82	38	18
		domInteractive	9	87	31	18	9

Bundle size diffs [🚨 Warning! Bundle size has increased!]

• background: 48.78 KiB (1.37%)
• ui: -5 Bytes (-0.00%)
• common: 26.6 KiB (0.32%)

[k-g-j]

Adding a note here as a TODO for us to remove the need of mergeAccounts