From 110f02e4e3ce320d7bf3df4cd2104ced258450c5 Mon Sep 17 00:00:00 2001 From: adlee-was-taken Date: Fri, 26 Jun 2026 00:19:55 -0400 Subject: [PATCH] refactor(cli): drop unused param from resolve_second_factor_secret (simplify pass) Co-Authored-By: Claude Opus 4.8 Claude-Session: https://claude.ai/code/session_01McMF5pUJbCMricmFEnf2sG --- crates/relicario-cli/src/commands/recovery_qr.rs | 2 +- crates/relicario-cli/src/session.rs | 4 +--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/crates/relicario-cli/src/commands/recovery_qr.rs b/crates/relicario-cli/src/commands/recovery_qr.rs index 4525a56..f1adc1a 100644 --- a/crates/relicario-cli/src/commands/recovery_qr.rs +++ b/crates/relicario-cli/src/commands/recovery_qr.rs @@ -17,7 +17,7 @@ fn cmd_recovery_qr_generate() -> Result<()> { let root = crate::helpers::vault_dir()?; let pf = crate::session::read_params(&root)?; - let secret = crate::session::resolve_second_factor_secret(&root, pf.second_factor)?; + let secret = crate::session::resolve_second_factor_secret(pf.second_factor)?; let passphrase = if let Some(p) = crate::test_passphrase_override() { Zeroizing::new(p) diff --git a/crates/relicario-cli/src/session.rs b/crates/relicario-cli/src/session.rs index 63acf75..c6fde19 100644 --- a/crates/relicario-cli/src/session.rs +++ b/crates/relicario-cli/src/session.rs @@ -36,7 +36,7 @@ impl UnlockedVault { let pf = read_params(&root)?; let params = pf.to_kdf_params(); - let secret = resolve_second_factor_secret(&root, pf.second_factor)?; + let secret = resolve_second_factor_secret(pf.second_factor)?; let passphrase = if let Some(p) = crate::test_passphrase_override() { Zeroizing::new(p) @@ -210,10 +210,8 @@ pub fn get_keyfile_path() -> Result { /// Resolve the vault's 32-byte second-factor secret per the params hint: /// extract it from the reference image, or decode it from the key file. pub(crate) fn resolve_second_factor_secret( - root: &Path, second_factor: SecondFactor, ) -> Result> { - let _ = root; // retained in the API for future vault-relative resolution match second_factor { SecondFactor::Image => { let image_path = get_image_path()?;