fix(25182): reloading page during SRP creation breaks flow

[vinnyhoward]

Description

Problem

Fixed an issue in the onboarding flow where refreshing the page during the Secret Recovery Phrase (SRP) confirmation step would result in a broken UI state. The solution redirects users to the unlock screen when SRP is needed but not available, retrieves the phrase after password verification, and returns them to their previous location in the flow. This ensures a smooth recovery from page refreshes during the onboarding process.

The Solution

• Detects when user is on SRP-dependent routes
• Redirects to unlock screen when SRP is needed
• Retrieves SRP after password verification
• Returns user to their previous step in the flow
• This maintains the security of the SRP while improving the user experience during onboarding.

Related issues

Fixes: #25182

Manual testing steps

1. Start the Onboarding flow
2. Create Wallet
3. Click Secure your wallet
4. Reveal your secret recovery phrase
5. Go to Confirm Secret Recovery phrase
6. Refresh the page
7. See just 1 empty input and cannot perform any action + several errors in the ui console

Screenshots/Recordings

Before

before.mov

After

after.mov

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.

[metamaskbot]

Builds ready [d697706]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

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

Page Load Metrics (1858 ± 120 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	328	2242	1615	560	269
		domContentLoaded	1435	2352	1834	247	118
		load	1449	2358	1858	251	120
		domInteractive	24	109	58	32	15
		backgroundConnect	10	66	23	17	8
		firstReactRender	15	98	39	28	13
		getState	4	54	19	19	9
		initialActions	0	1	0	0	0
		loadScripts	1009	1782	1355	208	100
		setupStore	7	57	16	16	8
		uiStartup	1628	3008	2146	346	166

Bundle size diffs

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

[metamaskbot]

Builds ready [b03f532]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

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

Page Load Metrics (1666 ± 59 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	204	1833	1587	332	159
		domContentLoaded	1410	2001	1646	123	59
		load	1454	2004	1666	123	59
		domInteractive	24	99	41	23	11
		backgroundConnect	10	60	23	17	8
		firstReactRender	15	77	38	25	12
		getState	5	54	17	17	8
		initialActions	0	1	0	0	0
		loadScripts	1030	1422	1181	95	46
		setupStore	6	73	12	14	7
		uiStartup	1649	2308	1927	152	73

Bundle size diffs

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

[darkwing]

Awesome work!

[metamaskbot]

Builds ready [3d3cc68]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

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

Page Load Metrics (2086 ± 222 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	257	3001	1880	664	319
		domContentLoaded	1385	2978	2054	456	219
		load	1414	2998	2086	462	222
		domInteractive	23	172	64	42	20
		backgroundConnect	12	94	39	27	13
		firstReactRender	14	94	37	24	12
		getState	4	66	19	19	9
		initialActions	0	1	0	0	0
		loadScripts	1003	1997	1500	330	159
		setupStore	7	53	19	16	8
		uiStartup	1609	3244	2343	494	237

Bundle size diffs

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

[EtherWizard33]

LGTM!

[metamaskbot]

Builds ready [6b28089]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background:
  • common:
  • ui:
  • content-script:
  • offscreen:

[metamaskbot]

Builds ready [d08d319]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

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

Page Load Metrics (1736 ± 111 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	337	2303	1637	369	177
		domContentLoaded	1422	2252	1710	233	112
		load	1445	2305	1736	231	111
		domInteractive	22	202	57	44	21
		backgroundConnect	9	79	31	21	10
		firstReactRender	14	75	36	23	11
		getState	5	64	15	17	8
		initialActions	0	1	0	0	0
		loadScripts	1004	1784	1280	200	96
		setupStore	7	66	22	21	10
		uiStartup	1702	3008	2003	314	151

Bundle size diffs

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