RB
← All projectsRust Bitcoin

rust-bitcoin

Rust library for Bitcoin data structures, serialization, consensus encoding, and scripts.

BitcoinCryptographic librariesNormal
Repository coverage

2084 commits in the local evidence base

Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.

461security candidates452second-pass queue2076AI analyses
195commits · 30 days
409commits · 60 days
1296commits · 180 days
2066commits · 365 days
Backfill bands
Aug 5 → Feb 6787 seen32 candidatesComplete
Feb 6 → Jun 6878 seen53 candidatesComplete
Jun 6 → Jul 6211 seen15 candidatesComplete
Jul 6 → Aug 5184 seen2 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

64/100 average clarity
318Strong · 80–100
1058Adequate · 60–79
557Thin · 40–59
151Opaque · 0–39
20security candidates with opaque commit messaging
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
Mitchell Bagot643192641068
Tobin C. Harding41366410063
jrakibi944994068
Andrew Poelstra582155077
Nick Johnson18920189059
Jamil Lambert, PhD11418114061
Fmt Bot311431045
Trevor Arjeski111111069
Shing Him Ng31731056
Ismail Daif22622050
satsfy (Renato Britto)17617066
Martin Habovstiak27527069
Analysis record

Published AI watches

Last scanned 18 minutes ago

Informational 20 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6678: units: Add `Weight::to_vb_*` functions, deprecating `to_vbytes_*`

This is a routine API cleanup in a Rust Bitcoin library. It adds new method names (to_vb_floor, to_vb_ceil) for converting transaction 'weight' to 'virtual bytes' and marks the old names (to_vbytes_floor, to_vbytes_ceil) as deprecated. The…

7f614781by Andrew Poelstra+30−86 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6597: Release tracking PR: `crypto 0.3.0`

This commit is a routine release-tracking merge. It only bumps version numbers (bitcoin-crypto to 0.3.0 and bitcoin-primitives to 0.103.1), updates dependency version requirements, refreshes lock files, and adds changelog entries. There ar…

1fddd49eby Andrew Poelstra+34−1210 files
No security note in commit
Informational 15 AI analysisMessage 28 · Opaque
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Update API files

This commit only updates generated API listing files (text snapshots of public functions) to reflect a newly added public function, `to_consensus_u32`, on the `CompactTarget` type. It does not change any source code, logic, or behavior, an…

a75c53c8by Mitchell Bagot+6−03 files
No security note in commit
Informational 15 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6675: units: Remove unnecessary track_caller

This commit removes two Rust compiler hints (`#[track_caller]`) from helper functions that simply return a value or call a user-provided fallback. These functions never panic, so the hints were unnecessary and had no security effect. The c…

ce894f36by Andrew Poelstra+0−21 file
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6644: primitives: Add serde impls for `WitnessVersion`

This commit adds standard serialization/deserialization support for the WitnessVersion type when the optional serde feature is enabled. It is a routine feature addition with no security relevance visible in the code or commit message.

4981faa7by Andrew Poelstra+50−02 files
No security note in commit
Informational 15 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6671: units: reword docs from height to count in relative locktime

This commit only changes documentation comments and variable names in the rust-bitcoin library. It rewords references to 'block height' as 'block count' for relative locktime, because a relative locktime measures elapsed blocks since a coi…

6ea4ff92by Andrew Poelstra+20−172 files
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6670: ci: fix labeller permissions

This is a routine GitHub Actions workflow fix. The change gives the automated PR labeler workflow permission to download artifacts and explicitly tells the GitHub CLI which repository to act on. It does not change the Bitcoin library code,…

CI workflow permission fixNo source code changesNo cryptographic or consensus changes
276e0a61by Andrew Poelstra+5−01 file
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6657: Release tracking PR: `base58ck 0.5.0`

This commit is a routine version bump for the base58ck crate from 0.4.0 to 0.5.0. It only updates version numbers in package manifests, lock files, and adds a changelog entry. There are no code changes and no security relevance.

9deb5de1by Andrew Poelstra+22−168 files
No security note in commit
Informational 21 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6440: base58: Introduce `decode_check_to_array` for alloc-less decoding

This commit is a routine feature addition to the rust-bitcoin base58 crate. It adds a new no-allocator function to decode short base58-check strings into fixed-size byte arrays, and renames the existing error type while keeping a deprecate…

No security-relevant bug fix is described in the commit message or diff.New decoding path uses a fixed 128-byte scratch buffer (`ArrayVec`) and rejects oversized inputs.Checksum verification and invalid-character handling are preserved from the existing alloc implementation.
c5fd1832by Andrew Poelstra+330−775 files
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6646: hashes: add several methods to `Midstate` and release 1.2.0

This commit is a routine feature release for the bitcoin_hashes crate. It adds new public methods and a constant to the SHA-256 Midstate type so developers can compute SHA-256 midstates in const contexts. There is no indication of a securi…

0f68fcb3by Andrew Poelstra+157−4210 files
No security note in commit
Informational 15 AI analysisMessage 73 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

units: reword docs to block count in relative locktime

This commit only rewords documentation comments and variable names in the codebase. It changes references from 'block height' to 'block count' for relative lock times, which is a terminology correction with no functional code changes. Ther…

8867fe2eby satsfy (Renato Britto)+20−172 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6667: Make the `WitnessesEncoder` private

This commit makes a small internal helper type called WitnessesEncoder private. It was previously exported as public API but is only used inside the transaction encoder. The change removes it from the public API surface and moves the code …

07d91f7fby Andrew Poelstra+54−1324 files
No security note in commit
Informational 12 AI analysisMessage 75 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

ci: fix labeller permissions

This commit adjusts a GitHub Actions workflow that automatically labels pull requests based on CI results. It adds a built-in repository token, tells the artifact download step to merge single-file artifacts, and explicitly sets the reposi…

Workflow-only change with no library code modificationsUses the default GITHUB_TOKEN rather than a custom secretRuns in workflow_run context, which already has elevated repository access by design
715f2bd6by Nick Johnson+5−01 file
No security note in commit
Informational 19 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6661: units: Remove `From<u16>` from `NumberOfBlocks`

This is a routine API cleanup, not a security fix. The developers removed a shortcut that let programmers create a 'number of blocks' value directly from any u16 number, replacing it with an existing named constructor called from_height. T…

ed5f1b68by Andrew Poelstra+18−358 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6668: Scrub the transaction encoding logic

This commit is a pure code reorganization (refactor) in the rust-bitcoin library. It moves transaction encoding and decoding definitions around within a single file so the code follows the project's preferred layout. No logic, behavior, or…

c5588ba5by Andrew Poelstra+57−571 file
No security note in commit
Informational 18 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6662: build(deps): bump actions/checkout from 6.0.2 to 6.0.3

This commit is a routine update to the GitHub Actions checkout tool used by the project's automated workflows. It changes the pinned version from 6.0.2 to 6.0.3 across many workflow files. The newer version fixes minor bugs related to SHA-…

Dependency bump of actions/checkout to a patch releaseUpstream fixes for SHA-256 repository checkout and merge commit regexAll affected workflows already disable persisted credentials
f7d364cdby Andrew Poelstra+29−2915 files
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6663: build(deps): bump taiki-e/install-action from 2.79.12 to 2.81.4

This is a routine Dependabot update that bumps the version of a third-party GitHub Action used only in automated testing workflows. The change does not touch the project's actual Bitcoin library code, and there is no indication it fixes or…

5c92c0b0by Andrew Poelstra+2−22 files
No security note in commit
Informational 15 AI analysisMessage 45 · Thin
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Update the API text files

This commit only updates two generated API text files that list the public surface of the Rust crate. It removes entries for a `WitnessesEncoder` type, reflecting that the type is no longer part of the public API. There is no source code c…

9c391145by Tobin C. Harding+0−782 files
No security note in commit
Informational 15 AI analysisMessage 68 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Scrub the transaction encoding logic

This commit is a pure code reorganization: it moves existing transaction encoding and decoding definitions around within a single file so they follow the project's preferred layout. No logic was changed, no security bug was fixed, and no n…

80dcef15by Tobin C. Harding+57−571 file
No security note in commit
Informational 15 AI analysisMessage 58 · Thin
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Make the WitnessesEncoder private

This commit simply hides an internal helper type called WitnessesEncoder from public view. It is a routine API-cleanup change with no security relevance visible in the code or commit message.

b37accc3by Tobin C. Harding+54−542 files
No security note in commit
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

Lower-priorityconsensus_encoding: expose vis fragment on exposed macrosby Nick Johnson · f0180702 · Feb 24, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Nick Johnson

consensus_encoding: expose vis fragment on exposed macros

Following the C-MACRO-VIS API convention, expose a vis fragment to allow
the caller to choose the output's visibility. This change is backwards
compatible.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit is a routine API ergonomics improvement. It changes two Rust macros so that the caller can specify the visibility of the generated struct and its constructor (e.g., public, private, or crate-visible), instead of always forcing public visibility. The change is explicitly described as backwards compatible and follows a documented Rust API convention.

Lower-priorityio: Add flush_to_writer functionby Mitchell Bagot · e44744ce · Feb 24, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Mitchell Bagot

io: Add flush_to_writer function

In some cases, we may have an Encoder instance, without a corresponding
Encodable. In these cases, having a way to flush the encoder to an io
writer is extremely useful to avoid having to replicate the logic of
encode_to_writer inline.

Add flush_to_writer function to match flush_to_writer from
consensus_encoding.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Explains rationale or failure mode
AI analysis · Informational 15/100

This commit adds a new helper function that lets developers flush an existing data encoder directly to an output writer. It is a straightforward code refactor that splits an existing function into two pieces; there is no indication of a security bug or fix.

Security candidatebitcoin: reject 65 bytes signature with sighash 0x00by jrakibi · 371b17e3 · Feb 24, 2026 · 1 fileMessage 98 · StrongModerate 60Details
Commit message · jrakibi

bitcoin: reject 65 bytes signature with sighash 0x00

According to BIP341, if taproot signature is 65 bytes long, the last byte (sighash)
must be different from 0x00, otherwise, it is invalid.
currently, we are accepting it as a valid signature.
this might also break the roundtrip of from_slice -> serialize

ref: https://github.com/bitcoin/bips/blob/master/bip-0341.mediawiki#taproot-key-path-spending-signature-validation

98/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Links an issue, advisory, or supporting reference✓ Names security-relevant behavior explicitly
Why it was queued
signing boundarycryptography-sensitive path
AI analysis · Moderate 60/100

This commit fixes a bug where the Rust Bitcoin library incorrectly accepted 65-byte Taproot signatures whose final byte was 0x00. Under Bitcoin's BIP-341 rules, such signatures are invalid. Accepting them could let invalid transactions or signatures slip through, and it could break the library's own round-trip serialization (reading a signature in and writing it back out).

AI review queuedconsensus_encoding: add track_caller to panic-able sitesby Nick Johnson · 0276325e · Feb 23, 2026 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · Nick Johnson

consensus_encoding: add track_caller to panic-able sites

Defensively added to the Decoder trait itself since the performance
impact is negligible.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
parser or protocol pathsecond-pass: broader security terminologysecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit adds Rust's #[track_caller] attribute to two methods in a decoding trait. This is a diagnostic improvement: if the code ever panics, the error message will point to the caller's location instead of deep inside the library. It does not change what the code does, what data it accepts, or whether it panics. There is no security fix here.

AI review queuedRe-release without rc suffixby Tobin C. Harding · 418685a9 · Feb 23, 2026 · 20 filesMessage 90 · StrongInformational 20Details
Commit message · Tobin C. Harding

Re-release without rc suffix

Turns out that the `-rc.0` suffix causes way more problems than it
solves because of how `cargo` resolves the version numbers and what we
intended on using the RC releases for.

In brief

- We wanted to be able to do breaking changes if required
- We wanted to signal that these releases were almost there (TM)
- We wanted to be able to do downstream testing including releasing
downstream crates with the RC releases as part of their public API.

In hindsite we messed up and should have just kept iterating as normal
until we were ready.

Re-release the whole stack without any rc suffix's. However keep
`bitcoin 0.33.0-beta` because we want 0.32.0 to be the latest stable
release and its important that it shows as such on docs.rs

Also, for pre-1.0 crates that had an rc release just jump to the next
version i.e., `io 0.4.0-rc.0` goes to `io 0.5.0`. Just for good
measure.


crate | latest stable | latest RC | with this applied
-----------------------------------------------------------
consensus_encoding 0.0.0 1.0.0-rc.3 0.1.0
units 0.2.0 1.0.0-rc.4 0.3.0
primitives 0.101.0 1.0.0-rc.2 0.102.0
hashes 0.19.0 - 0.20.0
io 0.3.0 0.4.0-rc.0 0.5.0
base58ck 0.3.0 - 0.4.0


bitcoin - 0.33.0-beta.0 to be yanked. Release as 0.33.0-beta

p2p - updated deps, unrelased so no other changes.
internals - not touched (currently 0.5.0)
chacha20_poly1305 - not touched

90/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
Why it was queued
second-pass: unusually broad change
AI analysis · Informational 20/100

This commit is a routine release-management change for the rust-bitcoin project. It re-releases several crates without the '-rc' (release candidate) suffix in their version numbers and updates internal dependency version requirements accordingly. There are no code logic changes, bug fixes, or security patches in the diff—only version numbers, lock files, changelogs, and README text.

Security candidatebitcoin: preserve parity for XOnlyPublicKeyby jrakibi · f4e9fcae · Feb 22, 2026 · 1 fileMessage 80 · StrongLow 49Details
Commit message · jrakibi

bitcoin: preserve parity for XOnlyPublicKey

both From impls for XOnlyPublicKey lose parity because they call `from_secp(`,
which defaults to Parity::Even. we now preserve parity by extracting it from
x_only_public_key() instead

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
cryptography-sensitive path
AI analysis · Low 49/100

This commit fixes a bug in how X-only public keys were created from full public keys. An X-only key is just the X coordinate of a point, but a point can have an even or odd Y coordinate (parity). The old code always assumed even parity, which could silently produce the wrong key for odd-parity inputs. The fix preserves the actual parity from the source key.

Lower-priorityconsensus_encoding: fix up doc linkby Nick Johnson · 2098b582 · Feb 21, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

consensus_encoding: fix up doc link

Confrom to C-LINK API conventions.

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This is a one-word documentation fix. A comment that referenced another type was changed from plain text to a proper Rust doc link so that documentation tools can create a clickable link. There is no code behavior change and no security relevance.

Lower-priorityconsensus_encoding: add conventinal manifest metadataby Nick Johnson · e001c89d · Feb 21, 2026 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · Nick Johnson

consensus_encoding: add conventinal manifest metadata

Follows C-METADATA from the API guidelines.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit only adds two standard metadata fields (homepage and documentation URLs) to a Rust package manifest file. It does not change any executable code, cryptographic logic, or network behavior. There is no security relevance.

AI review queuedUpdate apiby Nick Johnson · c9cdde0b · Feb 21, 2026 · 1 fileMessage 0 · OpaqueInformational 15Details
Commit message · Nick Johnson

Update api

0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
documentation-only discountsecond-pass: opaque commit message
AI analysis · Informational 15/100

This commit updates a generated API snapshot file to reflect a minor relaxation of trait bounds on an error type. It is a documentation/tracking file change, not a code change, and has no security relevance.

AI review queuedconsensus_encoding: remove redundant trait boundsby Nick Johnson · 3ea57579 · Feb 20, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

consensus_encoding: remove redundant trait bounds

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This is a minor code cleanup in a Rust Bitcoin library. It removes unnecessary requirements (trait bounds) from an error-handling implementation. The change does not affect security, runtime behavior, or how the library processes Bitcoin data.

Lower-priorityFix Unknown NetworkMessage encodingby Mitchell Bagot · cf1a9fee · Feb 20, 2026 · 1 fileMessage 58 · ThinLow 37Details
Commit message · Mitchell Bagot

Fix Unknown NetworkMessage encoding

The NetworkMessageEncoder and old Encodable implementation for
NetworkMessage make use of the Vec<u8> encoding for the raw payload
data. This introduces a vector length prefix to the start of the
encoding which causes the roundtrip for Unknown NetworkMessage types to
fail. In practice, NetworkMessage will only ever be decoded through the
RawNetworkMessage, where a payload length will be known. As such, this
length prefix is not necessary in the encoding and can be trivially
removed.

Change Unknown NetworkMessage encoding to directly write the vector of
bytes instead of using Vec<u8> encoding logic.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
AI analysis · Low 37/100

This commit fixes a bug in how custom or unknown Bitcoin peer-to-peer messages were being serialized. Previously, the code accidentally added an extra length number to the front of the raw payload bytes, so if you decoded an unknown message and re-encoded it, the bytes would not match. The fix writes the raw payload bytes directly without the extra length prefix, and adds a test to confirm round-trip encoding works.

Security candidateEncapsulate PublicKey and PrivateKeyby Mitchell Bagot · 118b8987 · Feb 20, 2026 · 1 fileMessage 58 · ThinInformational 17Details
Commit message · Mitchell Bagot

Encapsulate PublicKey and PrivateKey

With the previous changes to the PublicKey and PrivateKey types, both
can now be encapsulated to control the scope of access to the inner
fields.

Move PublicKey and PrivateKey to encapsulated module and add them to
the pub use.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive path
AI analysis · Informational 17/100

This commit is a routine internal code reorganization. It moves the PublicKey and PrivateKey type definitions into a private 'encapsulate' module and re-exports them publicly. The fields were already private before this change, so no new access restrictions are introduced. There is no indication this fixes a security bug or changes behavior visible to users of the library.

Security candidateRemove public access to all fields of PublicKey and PrivateKeyby Mitchell Bagot · 41d946ff · Feb 20, 2026 · 3 filesMessage 73 · AdequateInformational 18Details
Commit message · Mitchell Bagot

Remove public access to all fields of PublicKey and PrivateKey

As with the inner fields, the compressed and network fields of PublicKey
and PrivateKey allows public access for read/write. To allow for
encapsulation, read access should be limited to an accessor function.

Introduce compressed() accessor function for PublicKey and compressed()
and network() accessor functions for PrivateKey. Replace all direct
access to inner fields with use of the accessor functions.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive pathsigning or wallet path
AI analysis · Informational 18/100

This commit hides the internal fields of two key types (PublicKey and PrivateKey) so outside code can no longer read or change them directly. Instead, it provides read-only accessor methods. This is a normal defensive-coding change that improves encapsulation and makes future misuse less likely, but the commit itself does not fix a known active bug or vulnerability.

Security candidateRemove public access to inner field of PublicKey and PrivateKeyby Mitchell Bagot · 9506d45c · Feb 20, 2026 · 6 filesMessage 73 · AdequateLow 35Details
Commit message · Mitchell Bagot

Remove public access to inner field of PublicKey and PrivateKey

While all construction of PublicKey and PublicKey now goes through the
from_secp constructors, read access to the inner fields are still
commonplace.

Introduce as_inner function for PrivateKey and to_inner for PublicKey
and remove all direct access to the inner fields.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive pathsigning or wallet path
AI analysis · Low 35/100

This commit hides the internal cryptographic key fields of PublicKey and PrivateKey from outside code, replacing direct access with controlled getter methods. It is a defensive hardening change that reduces the chance future callers will accidentally misuse or mutate the raw secp256k1 keys, but the commit itself does not fix a known exploitable bug.

Security candidateIntroduce from_secp constructors for PublicKey and PrivateKeyby Mitchell Bagot · 2f74f3ea · Feb 20, 2026 · 8 filesMessage 73 · AdequateInformational 18Details
Commit message · Mitchell Bagot

Introduce from_secp constructors for PublicKey and PrivateKey

The PublicKey and PrivateKey types rely on public access to the inner
fields in many places to instantiate the type. Since we wish to
encapsulate the type, this inner access should instead be wrapped
through constructors.

Introduce from_secp, and from_secp_uncompressed constructors and
replace all instantiation of PublicKey and PrivateKey with calls to
these.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive pathsigning or wallet path
AI analysis · Informational 18/100

This is a routine code cleanup in the rust-bitcoin library. It introduces new constructor functions named from_secp for creating public and private keys, and replaces direct internal field access with calls to those constructors. The old constructors are kept but marked as deprecated. There is no security bug being fixed here; the change is about improving code structure and preparing to hide internal details in the future.

Lower-priorityprimitives: Unit test hex featureby Tobin C. Harding · 771ebf2d · Feb 20, 2026 · 1 fileMessage 86 · StrongInformational 15Details
Commit message · Tobin C. Harding

primitives: Unit test hex feature

When I added the `hex` feature (#5213) I forgot to tie it into CI.

Test the feature both with and without `std` enabled.

86/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 15/100

This commit only changes a CI configuration file to add the 'hex' feature to automated testing matrices. It does not modify any runtime code, APIs, or dependencies that could affect security. There is no security issue or fix here.

AI review queuedUpdate the API filesby jrakibi · d65b496d · Feb 19, 2026 · 3 filesMessage 28 · OpaqueInformational 15Details
Commit message · jrakibi

Update the API files

28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
documentation-only discountsecond-pass: opaque commit message
AI analysis · Informational 15/100

This commit only updates generated API snapshot files (text listings of public types and methods). It does not change any source code, behavior, or fix any bug. The added lines reflect new methods like non_secure_erase and Drop that already exist in the actual library code; the removed line shows Hkdf no longer implements Copy. These are documentation/snapshot updates, not a security patch.

Security candidatehashes: add non_secure_erase for HMAC and HKDFby jrakibi · 5c844315 · Feb 19, 2026 · 3 filesMessage 68 · AdequateLow 30Details
Commit message · jrakibi

hashes: add non_secure_erase for HMAC and HKDF

HKDF and HMAC hold secret data (prk, ipad/opad). we add
non_secure_erase to overwrite it with zeros (same approach as
rust-secp256k1 non_secure_erase)

this is a best-effort to erase secret data, it does not guarantee
that material never remains in memory. the compiler may still copy or move
data to other places in memory.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive path
AI analysis · Low 30/100

This commit adds a best-effort memory-clearing helper for two cryptographic building blocks (HMAC and HKDF) in the rust-bitcoin library. It tries to overwrite secret intermediate data with zeros when those objects are destroyed, but the authors explicitly note this is not a guaranteed security fix because the Rust compiler may still leave copies of the data elsewhere in memory. The change is defensive hardening rather than a fix for a known exploitable bug.

Lower-priorityunits: add optional encoding dep to feature flag test matrixby Nick Johnson · 574f7e9e · Feb 19, 2026 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · Nick Johnson

units: add optional encoding dep to feature flag test matrix

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only changes a test configuration file (rbmt.toml) to include an optional 'encoding' feature in the automated feature-flag testing matrix. It does not modify any production code, APIs, or security-sensitive logic. There is no security issue here.

Lower-priorityBlockHash: Inline all the consensus encoding stuffby Tobin C. Harding · 5bc28cee · Feb 19, 2026 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · Tobin C. Harding

BlockHash: Inline all the consensus encoding stuff

Add `inline` attribute to everything except the error code.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit only adds Rust compiler hints (`#[inline]`) to a few small functions related to encoding and decoding block hashes. It does not change what the code does, only gives the compiler permission to embed these tiny functions directly at call sites for potential performance gains. There is no security relevance.

Lower-priorityBlockHash: Move Encodable and Decodable trait implsby Tobin C. Harding · 84c51a8e · Feb 19, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Tobin C. Harding

BlockHash: Move Encodable and Decodable trait impls

Put the trait impls first. They are the most important thing in
regards to the encoding/decoding logic, put them first and the
supporting types and code next.

Code move only, no logic change.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit simply reorders two blocks of code in a single Rust source file. It moves the implementation of the Decodable trait for BlockHash to appear earlier in the file, before the definition of its helper decoder type. The commit message and the diff confirm there is no change to program logic, behavior, or security.

Lower-priorityUse encoding:: prefix on encoding traitby Tobin C. Harding · bca1bb95 · Feb 19, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Use encoding:: prefix on encoding trait

It makes it a bit more clear that the trait is from another crate if
we use one level of path. Also this seems to be consistent with other
usage repo wide lately for `encoding` stuff at least.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This is a trivial code-style change. A developer replaced a direct import of the Encodable trait with its full path (encoding::Encodable) to make it clearer that the trait comes from another crate. No behavior, logic, or security properties changed.

Lower-priorityFix display msg for BlockHashDecoderErrorby Tobin C. Harding · 63fce3f2 · Feb 19, 2026 · 1 fileMessage 60 · AdequateInformational 16Details
Commit message · Tobin C. Harding

Fix display msg for BlockHashDecoderError

Looks like a cut'n'paste error; this is the block hash decoder not the
sequence decoder.

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 16/100

This commit fixes a typo in an error message. The error message for a block hash decoding failure incorrectly said 'sequence decoder error' instead of 'block hash decoder error'. It is a cosmetic string correction with no security or functional impact.

Lower-priorityp2p: implement encoding for Addressby yancy · 3c19767f · Feb 19, 2026 · 1 fileMessage 86 · StrongInformational 17Details
Commit message · yancy

p2p: implement encoding for Address

port and address are both converted to an array type &[u8] as required
by `ArrayEncoder` using `to_be_bytes()`. Big endian byte order is
specified in:
https://developer.bitcoin.org/reference/p2p_networking.html

Since address is an array of `[u16; 8]`, each respective `u16` is split
into a `[u8; 2]`, then both elements are filled into the resulting
`[u8; 32]`.

In the test cases, replace `serialize` with `encoding::encode_to_vec`
since the push encoding here is planned to replace the existing
`serialize` implementation.

86/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 17/100

This commit adds a new way to serialize (convert to bytes) Bitcoin peer-to-peer network addresses in the rust-bitcoin library. It does not appear to fix or introduce a security vulnerability; it is a routine implementation change replacing one serialization method with another while preserving the documented big-endian byte order.

Lower-prioritytest: add roundtrip test for encoding and decoding an Addressby yancy · 4218190d · Feb 19, 2026 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · yancy

test: add roundtrip test for encoding and decoding an Address

72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This commit adds a new automated test that checks an Address object can be encoded to bytes and then decoded back to the same object. It does not change any production code, fix any bug, or alter behavior visible to users.