fix: remove duplicate SOL_BIP44_PREFIX constant
What the developer wrote
Authored by ww3512687
fix: remove duplicate SOL_BIP44_PREFIX constant
What changed, and why it matters
This commit removes an accidentally duplicated constant definition for Solana's BIP44 derivation path prefix. In Rust, defining the same constant twice in the same scope is a compile-time error, so the duplicate would have prevented the code from compiling. The fix simply deletes the extra line, restoring normal compilation. There is no runtime security risk.
No security action required. Treat as a routine build fix / code cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff deletes a duplicate const SOL_BIP44_PREFIX: &str = "44'/501'"; line in rust/apps/wallets/src/keystone_connect.rs. Rust does not allow two items with the same name in the same scope, making this a build-breaking syntax/semantic error rather than a logic bug. The change is a one-line cleanup with no functional or security implications.
Changed components
rust/apps/wallets/src/keystone_connect.rsInspect captured patch +0 / −1
diff --git a/rust/apps/wallets/src/keystone_connect.rs b/rust/apps/wallets/src/keystone_connect.rs
index 2e401be..2484535 100644
--- a/rust/apps/wallets/src/keystone_connect.rs
+++ b/rust/apps/wallets/src/keystone_connect.rs
@@ -36,7 +36,6 @@ const XRP_PREFIX: &str = "44'/144'/0'";
const LTC_PREFIX: &str = "49'/2'/0'";
const LTC_NATIVE_SEGWIT_PREFIX: &str = "84'/2'/0'";
const SOL_BIP44_PREFIX: &str = "44'/501'";
-const SOL_BIP44_PREFIX: &str = "44'/501'";
const BCH_PREFIX: &str = "44'/145'/0'";
const ZEC_PREFIX: &str = "44'/133'/0'";
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.