[chukwudiikeh]

Fixes #382

key generate and key restore repeated the same steps to derive the master key from a mnemonic and passphrase, differing only in where the mnemonic comes from. That duplication is how issue #372 happened: one copy dropped the passphrase.

This PR moves the shared steps into key_from_mnemonic, which both commands call, and replaces the manual fold in key generate with the Display impl of Mnemonic, as key restore already did. Display joins the words with single spaces, the same output as the fold; I compared generate and restore output for 40 mnemonics (12 and 24 words) and all matched.No behaviour change. The existing key tests cover both paths and pass unchanged.

[chukwudiikeh]

@vadim-anfv @tvpeter this is the follow-up from issue #373, ready for review whenever you have time. No behaviour change the existing key tests cover both paths.

[emma31-dev]

This overlaps with #384 . If #384 gets merged first this change is then redundant. It is not an exact duplicate but It's worth deciding which lands first and rebasing the other on top.

[codecov]

Codecov Report

❌ Patch coverage is 90.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.96%. Comparing base (5558555) to head (a302315).
⚠️ Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
src/handlers/key.rs	90.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #383      +/-   ##
==========================================
- Coverage   60.00%   59.96%   -0.05%     
==========================================
  Files          25       25              
  Lines        4736     4726      -10     
==========================================
- Hits         2842     2834       -8     
+ Misses       1894     1892       -2     

Flag	Coverage Δ
rust	59.96% <90.00%> (-0.05%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[chukwudiikeh]

Thanks @emma31-dev . I stacked it on purpose: #384 is built on top of this PR #382 first, then the #381 rename, so this one should land first. I've rebased #384 onto the current commit here so it only adds the rename on top.