What the developer wrote
Authored by Trevor Arjeski
refactor: remove DerivationError
What changed, and why it matters
This commit removes a deprecated type alias named `DerivationError` from the public API, replacing it with the already-existing `DeriveXpubError`. It is a straightforward cleanup with no functional code changes and no security implications.
No security action needed. Users relying on the deprecated `DerivationError` alias should migrate to `DeriveXpubError`.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch deletes a pub use re-export and a pub type alias for DerivationError, which was marked #[deprecated] and simply pointed to DeriveXpubError. No logic, error handling, or behavior changes; only a public API surface reduction.
Changed components
key_expression/src/bip32.rs public APIInspect captured patch +0 / −7
diff --git a/key_expression/src/bip32.rs b/key_expression/src/bip32.rs
index 0d80ee40..172f7d63 100644
--- a/key_expression/src/bip32.rs
+++ b/key_expression/src/bip32.rs
@@ -26,9 +26,6 @@ pub use self::error::{
ParseAbsoluteDerivationPathError, ParseChildNumberError, ParseRelativeDerivationPathError,
ParseXprivError, ParseXpubError, XkeyDecodeError, XprivDecodeError, XpubDecodeError
};
-#[doc(no_inline)]
-#[allow(deprecated_in_future)]
-pub use self::error::DerivationError;
/// Version bytes for extended public keys on the Bitcoin network.
const VERSION_BYTES_MAINNET_PUBLIC: [u8; 4] = [0x04, 0x88, 0xB2, 0x1E];
@@ -1449,10 +1446,6 @@ pub mod error {
}
}
- /// Deprecated name for errors deriving an extended public key.
- #[deprecated(since = "TBD", note = "use `DeriveXpubError` instead")]
- pub type DerivationError = DeriveXpubError;
-
/// Out-of-range index when constructing a child number.
///
/// *Indices* are always in the range [0, 2^31 - 1]. Normal child numbers have the
Why this scored 19/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.