RB
← All projectsRust Bitcoin

rust-bitcoin

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

BitcoinCryptographic librariesNormal
Repository coverage

2313 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.

532security candidates511second-pass queue2206AI analyses
133commits · 30 days
267commits · 60 days
1103commits · 180 days
2024commits · 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.

66/100 average clarity
510Strong · 80–100
1085Adequate · 60–79
567Thin · 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.
Andrew Poelstra23179157290
Mitchell Bagot649193645068
Tobin C. Harding41566410063
jrakibi944994068
Nick Johnson19121190060
Jamil Lambert, PhD11919116061
satsfy (Renato Britto)381527066
Fmt Bot331431045
Trevor Arjeski111111069
Shing Him Ng31731056
Martin Habovstiak30628068
Ismail Daif22622050
Analysis record

Published AI watches

Last scanned 52 minutes ago

Moderate 62 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6954: units: serialize unsigned amounts as u64

This commit fixes a mismatch in how unsigned Bitcoin amounts were serialized versus deserialized when using certain compact binary formats. Previously, an unsigned amount (like 100 satoshis) was written as a signed number, which caused for…

Data integrity bug: serialized values decode to different numeric values in varint binary formatsRange-check failure: Amount::MAX and large values near the cap fail deserialization after round-tripSerde serialize/deserialize hint mismatch for unsigned amount types
295c9d8aby Andrew Poelstra+66−112 files
No security note in commit
Low 25 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6955: key_expression: preserve master-key invariants in Xpub Arbitrary

This change fixes a bug in test-only code that generates random fake Bitcoin extended public keys (xpubs). Previously, when generating a master xpub (depth 0), the code could pick random values for the parent fingerprint and child number, …

BIP32 master-key invariant violation in generated test dataEncode/decode round-trip failure for generated master xpubsFix aligns Xpub::arbitrary with existing Xpriv::arbitrary behavior
4116ecc6by Andrew Poelstra+35−31 file
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6947: build(deps): bump cargo-bins/cargo-binstall from 1.21.0 to 1.21.1

This commit updates the version of a helper tool (cargo-binstall) used only inside GitHub Actions automation. It is a routine dependency bump by Dependabot and does not change any code that ships to users. There is no indication of a secur…

c1be49cbby Andrew Poelstra+2−22 files
No security note in commit
High 70 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6919: Sanitize serde size hints before allocating

This commit fixes a denial-of-service weakness in how the library deserializes lists of Bitcoin data (witnesses, amounts, fee rates) from untrusted input. Before the fix, a few bytes of attacker-controlled data could claim a list would con…

Untrusted serde size hint fed directly into Vec::with_capacityPotential memory exhaustion / OOM kill from small malicious inputDenial-of-service vector in deserialization paths
55ddbc0cby Andrew Poelstra+88−105 files
Vendor flagged security relevance
Moderate 60 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6945: bitcoin: handle OP_CODESEPARATOR in legacy

This commit fixes how the Rust Bitcoin library calculates old-style (legacy) transaction signatures when the spending script contains a special opcode called OP_CODESEPARATOR. Previously the library did not handle this opcode at all, which…

Protocol correctness fix for legacy sighash serializationOP_CODESEPARATOR handling added to match Bitcoin Core consensus behaviorPreviously omitted test vectors restored, indicating prior non-compliance
5b815281by Andrew Poelstra+600−3093 files
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6948: build(deps): bump taiki-e/install-action from 2.83.2 to 2.85.4

This is a routine update by Dependabot to the version of a third-party GitHub Action used in the project's automated testing workflows. The change only affects internal continuous integration (CI) scripts, not the actual Bitcoin library co…

d1431904by Andrew Poelstra+2−22 files
No security note in commit
Low 33 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6946: Fix integer overflow in `get_array`

This commit fixes a small but real bug in a Rust helper that reads fixed-size chunks from a data slice. The helper was supposed to safely return 'nothing' when asked to read past the end of the data, but it accidentally added two numbers t…

Integer overflow in bounds-checking helperContract violation: method documented to return None on out-of-bounds access could panic insteadDebug-build panic (denial of service) possible
c6e80843by Andrew Poelstra+2−11 file
Vendor flagged security relevance
Moderate 62 AI analysisMessage 73 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Fix integer overflow in `get_array`

This commit fixes a bug in a Rust helper method called `get_array`, which is meant to safely read a fixed-size chunk from a slice and return nothing if the requested range is out of bounds. The bug was that the code added the caller's offs…

Integer overflow in bounds calculationPotential panic due to violated internal length expectationCaller-controlled arithmetic used for memory access bounds
56fb1287by Martin Habovstiak+2−11 file
Vendor flagged security relevance
High 71 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6915: primitives: Fix `Witness` handling of oversized items

This commit fixes a bug in how the Rust Bitcoin library counts and compares transaction witness data when a witness contains an oversized item. Previously, several functions relied on an iterator that silently skips oversized items, causin…

Inconsistent serialization/iterator behavior for oversized witness itemswtxid collision risk between transactions differing only in oversized witness bytesIncorrect witness equality for oversized single-item stacks
e1ed5884by Andrew Poelstra+106−273 files
Vendor flagged security relevance
Informational 19 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6922: Use `try_fold` instead of `fold` in `Sum` impl

This is a code-quality and performance improvement, not a security fix. It changes how the library adds up lists of Bitcoin amounts so that it stops early once an overflow is detected, rather than continuing to process the rest of the list…

No security-relevant signal in commit message or diffRefactor preserves overflow-checking behavior (short-circuits instead of continuing)New API method `NumOpResult::from_result` is a pure inverse of existing `into_result`
86e4d5daby Andrew Poelstra+60−562 files
No security note in commit
Moderate 52 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6893: units: Reject malformed amount strings

This update fixes a bug in how the library reads Bitcoin amount strings like '1.5 BTC'. Previously, certain malformed inputs such as '.', '._', '1_', '1_.0', and '1._0' were incorrectly accepted and treated as valid amounts (often zero), i…

Input validation bypass in amount parserMalformed strings silently parsed as zero or ordinary amountsUnderscore separator placement not enforced
fcb14622by Andrew Poelstra+88−343 files
Vendor flagged security relevance
Low 48 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6921: units: fix div_by_fee_rate_ceil precision

This commit fixes a rounding bug in how the rust-bitcoin library calculates the minimum transaction weight needed to pay a given fee at a given fee rate. The old code rounded the fee rate up too early, which could produce a weight slightly…

Incorrect fee-weight calculation due to premature integer roundingPotential transaction fee shortfall when using div_by_fee_rate_ceilOverflow protection added for Amount::MAX * 4_000_000 intermediate value
b31212e0by Andrew Poelstra+38−82 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6898: Release tracking PR: `consensus-encoding 1.3.0`

This is a routine release-management commit that bumps the version number of the `bitcoin-consensus-encoding` crate from 1.2.0 to 1.3.0 and updates lock files accordingly. It contains no code changes that fix or introduce a security issue.…

0cfc7908by Andrew Poelstra+37−349 files
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6909: build(deps): bump actions/labeler from 6.2.0 to 7.0.0

This commit updates a GitHub Actions automation tool (actions/labeler) used to automatically tag pull requests with labels. It is a routine dependency version bump from 6.2.0 to 7.0.0, with no indication of a security fix or vulnerability.…

4ed7c068by Andrew Poelstra+1−11 file
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6910: build(deps): bump actions/checkout from 7.0.0 to 7.0.1

This commit is a routine update to the GitHub Actions checkout tool used by the project's automated workflows. It only changes version numbers in configuration files and does not alter the actual Bitcoin library code that users run. There …

328c4ae9by Andrew Poelstra+37−3717 files
No security note in commit
Informational 15 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6911: build(deps): bump astral-sh/setup-uv from 8.3.2 to 9.0.0

This commit updates a GitHub Actions helper used to install a Python tool called uv, which runs the zizmor security scanner. The change only bumps the pinned version of the helper from 8.3.2 to 9.0.0. The new version's release notes mentio…

No security-relevant signals in commit or upstream release notesDependency bump in CI only, not in library codeNo CVE or advisory referenced
67600795by Andrew Poelstra+2−22 files
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6912: build(deps): bump github/codeql-action/upload-sarif from 4.37.0 to 4.37.3

This is a routine Dependabot update that changes the pinned version of GitHub's official CodeQL upload-sarif action from 4.37.0 to 4.37.3 in a single CI workflow. The action only uploads static analysis results to GitHub; it does not touch…

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

Merge rust-bitcoin/rust-bitcoin#6913: build(deps): bump dtolnay/rust-toolchain from 6c977a6ca4077a0ceb28ffbe03f59d46e9ac8772 to 02cb101ec7c40f2c49e1d9714d64511d8e1b74de

This is a routine update to a GitHub Actions helper used to install Rust during automated testing. It only changes the pinned version of the dtolnay/rust-toolchain action in workflow files. There is no change to the actual rust-bitcoin lib…

90330d15by Andrew Poelstra+8−84 files
No security note in commit
Informational 20 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6906: consensus_encoding, primitives: expose exact encoding size for block and transaction

This commit adds a way to ask, in advance, exactly how many bytes a Bitcoin block or transaction will take when serialized. It is a feature addition for the library's encoding system, not a fix for a vulnerability. There is no indication i…

No security-relevant signals in commit message or diffFeature addition: expose exact encoded sizeNo mention of vulnerability, CVE, bug bounty, or security report
1a365d53by Andrew Poelstra+129−1068 files
No security note in commit
Informational 15 AI analysisMessage 88 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

build(deps): bump dtolnay/rust-toolchain

This is a routine update by Dependabot that changes which version of a popular GitHub Action (dtolnay/rust-toolchain) is used to install Rust in automated CI workflows. The commit only updates pinned commit hashes in workflow files; it doe…

a31e0b0eby dependabot[bot]+8−84 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-priorityAutomated update to Github CI to rustc nightly-2026-03-13by Update Nightly Rustc Bot · e451625e · Mar 14, 2026 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Update Nightly Rustc Bot

Automated update to Github CI to rustc nightly-2026-03-13

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit simply updates the version of the Rust nightly compiler used by the project's GitHub CI (continuous integration) from one nightly build to a newer one. It changes only one line in a configuration file. There is no code change, no bug fix, and no security-relevant change visible in the diff.

AI review queuedFix decoder error Display and source implsby busayo-OD · 7bfeae68 · Mar 13, 2026 · 6 filesMessage 45 · ThinInformational 18Details
Commit message · busayo-OD

Fix decoder error Display and source impls

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

This commit fixes how several Bitcoin data-decoding error types print messages and report their underlying cause. The changes are cosmetic/correctness improvements to error formatting and the `source()` chain used by Rust's standard error handling. There is no evidence of a security vulnerability, remote exploit, or memory-safety bug in the diff.

Security candidateRemove Display from PrivateKeyby Mitchell Bagot · c04f2dd2 · Mar 13, 2026 · 3 filesMessage 58 · ThinLow 38Details
Commit message · Mitchell Bagot

Remove Display from PrivateKey

As part of the work in hiding secret data in keys, we want to remove
the Display impl for PrivateKey. Users may still display the secret
data using to_wif, or by calling display_secret on the inner key.

Remove Display from PrivateKey.

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

This change removes the ability to accidentally print a Bitcoin private key using Rust's standard '{}' formatting. Before, code like println!("{}", private_key) would expose the secret key. Now developers must explicitly call .to_wif() or similar methods, making secret leakage less likely through casual logging or error messages. It is a defensive hardening improvement, not a fix for an active exploit.

Security candidateRemove Index<RangeFull> from PrivateKeyby Mitchell Bagot · e4d46398 · Mar 13, 2026 · 1 fileMessage 68 · AdequateLow 29Details
Commit message · Mitchell Bagot

Remove Index<RangeFull> from PrivateKey

The Index trait allows for indexing on the PrivateKey type. Since we're
working towards hiding secret data in the PrivateKey, this trait should
not be included on the PrivateKey type.

Remove Index<RangeFull> impl on PrivateKey.

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 29/100

This commit removes the ability to use square-bracket slicing (like `private_key[..]`) on a Bitcoin private key. The change is part of a longer-term effort to prevent secret key material from being accidentally exposed through Rust's indexing features. It is a hardening cleanup rather than a fix for an active security flaw.

Security candidateAdd hex Display formatting for taproot::Signatureby Mitchell Bagot · bf3d769c · Mar 13, 2026 · 1 fileMessage 83 · StrongInformational 19Details
Commit message · Mitchell Bagot

Add hex Display formatting for taproot::Signature

The ecdsa signature has display formatting which roundtrips with the
corresponding FromStr impl. The taproot signature has no such display
capabilities.

Introduce Display impl on taproot::Signature.
Add test for hex roundtrip with Display + FromStr and FromStr error.

83/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Names security-relevant behavior explicitly
Why it was queued
signing boundarycryptography-sensitive path
AI analysis · Informational 19/100

This commit adds a human-readable hex display format for Taproot signatures in the rust-bitcoin library, matching a similar feature already available for ECDSA signatures. It is a straightforward feature addition with tests and does not fix any security bug.

Security candidateAdd hex string parsing for taproot::Signatureby Mitchell Bagot · e31b80ee · Mar 13, 2026 · 1 fileMessage 73 · AdequateInformational 19Details
Commit message · Mitchell Bagot

Add hex string parsing for taproot::Signature

The ecdsa::Signature can roundtrip to hex, but taproot::Signature
cannot.

Add FromStr for taproot::Signature and ParseSignatureError as an error
type.

73/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Names security-relevant behavior explicitly
Why it was queued
signing boundarycryptography-sensitive path
AI analysis · Informational 19/100

This commit adds a new feature: the ability to parse a Taproot cryptographic signature from a hexadecimal text string. Previously only ECDSA signatures could be parsed this way. The change is purely additive and does not fix any known bug or vulnerability. It simply exposes a new parsing function to users of the library.

Lower-priorityp2p: Add CommandStringEncoder newtypeby Mitchell Bagot · b25cbb22 · Mar 13, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Mitchell Bagot

p2p: Add CommandStringEncoder newtype

The CommandString type implements encoding::Encodable using the
ArrayEncoder type. In order to provide semantic clarity and match
standard implementations, this should be named CommandStringEncoder.
Due to issues with lifetimes on the CommandString and encoder in the
V1NetworkMessage, it is necessary to implement the newtype encoder
manually without using the encoder_newtype macro.

Add CommandStringEncoder encoder type that wraps ArrayEncoder<12> and
replace relevant uses of ArrayEncoder with it.

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

This commit is a simple internal code cleanup in the Bitcoin peer-to-peer networking code. It renames a generic array-based encoder to a more specific 'CommandStringEncoder' for the 12-byte command strings used in Bitcoin network messages. There is no change to behavior, data formats, or security properties.

Security candidatehashes: conservative upgrade to workspace lintsby Nick Johnson · 28ad2d31 · Mar 13, 2026 · 22 filesMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

hashes: conservative upgrade to workspace lints

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

This commit is a code-quality and linting cleanup for the `hashes` crate in rust-bitcoin. It moves the crate to use shared workspace lint rules, adds clippy allow attributes, fixes minor style issues (semicolons, doc formatting, import lists), and adds documentation comments. There are no functional changes to hash algorithms or public APIs, and no security bug is introduced or fixed.

Lower-priorityhashes: drop unused cfg flagby Nick Johnson · 44e3ff8f · Mar 13, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

hashes: drop unused cfg flag

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

This commit removes an unused compiler configuration flag from a linting rule in the package manifest. It is a cleanup change with no security relevance.

Security candidateMove impl_array_newtype_stringify to includeby Mitchell Bagot · 6df5edae · Mar 12, 2026 · 2 filesMessage 58 · ThinInformational 15Details
Commit message · Mitchell Bagot

Move impl_array_newtype_stringify to include

The impl_array_newtype_stringify macro is an internal macro in bitcoin
that is used in bip32 and other logic. As part of the separation of
bip32 functionality into a new crate, this macro will be needed.

Split impl_array_newtype_stringify into a module in include/ directory
and include! in existing location in bitcoin.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
Why it was queued
secret or key material
AI analysis · Informational 15/100

This commit simply moves an existing internal Rust macro from one file to another shared location. The code itself is unchanged; it is a pure refactoring to prepare for splitting a crate. There is no security-relevant change.

Lower-priorityunits: Fix holes in API surface testby Tobin C. Harding · 61dccf11 · Mar 12, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Tobin C. Harding

units: Fix holes in API surface test

Use claude to check if we missed anything in the API surface test and
it turns out we did.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit only adds missing test coverage for public API surface. It adds a new error type to an existing test struct and a new test that checks public types from the `result` module can be imported. There is no code behavior change, no bug fix, and no security relevance.

Security candidateMake crate headers uniformby Tobin C. Harding · 36db3222 · Mar 12, 2026 · 7 filesMessage 66 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Make crate headers uniform

I incorrectly churned this in #4667 which was pointed out in #5720 but
only fixed in the stabalizing crates. Now its been done in `hashes`
and again confused me so just do the whole repo already.

66/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100

This commit only changes the wording and formatting of documentation comments at the top of seven source files. It makes crate header titles uniform by adding '#' markers and adjusting capitalization. There are no code changes, no security fixes, and no behavior changes.

Lower-priorityprimitives: Fix holes in API surface testby Tobin C. Harding · f5ffbd62 · Mar 12, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Tobin C. Harding

primitives: Fix holes in API surface test

Use claude to check if we missed anything in the API surface test and
it turns out we did.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit only updates an internal test file that checks the public API surface of the `bitcoin_primitives` crate. It adds missing types (like `Ntxid`, `BlockChecked`, script buffer types) to compile-time API tests and enables a previously commented-out iterator field. There is no change to production code, no bug fix, and no security relevance.

Lower-priorityUpdate the API text filesby Tobin C. Harding · 0022a290 · Mar 12, 2026 · 7 filesMessage 45 · ThinInformational 15Details
Commit message · Tobin C. Harding

Update the API text files

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only updates generated API text files to reflect that a new public dependency called `encoding` is now exposed by several Rust Bitcoin crates. There are no code changes, no bug fixes, and no security-related content.

Lower-priorityRe-export the consensus-encoding crateby Tobin C. Harding · f2162fa5 · Mar 12, 2026 · 5 filesMessage 60 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Re-export the consensus-encoding crate

For all crates upstream of `bitcoin` re-export `encoding` at the crate
root.

This is by convention.

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

This commit simply makes an internal helper crate (consensus-encoding) visible at the top level of several related Rust crates. It is a routine API-organization change with no security implications.

Security candidatedocs: Favour Constructs over Createsby Tobin C. Harding · bab4f99c · Mar 12, 2026 · 2 filesMessage 72 · AdequateInformational 15Details
Commit message · Tobin C. Harding

docs: Favour Constructs over Creates

We decided a while back to favour `/// Constructs ...` but some
`/// Creates ...` have snuck in again.

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100

This commit only changes two words in code comments, replacing 'Creates' with 'Constructs' in documentation. There is no change to actual program logic, no security fix, and no behavior change.

Lower-priorityActually put the encoder trait impls firstby Tobin C. Harding · 849e35be · Mar 12, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Actually put the encoder trait impls first

In commit:

84c51a8ea1c05b1ff1b8249793f9c1e0491aaee0 BlockHash: Move Encodable and Decodable trait impls

I wrote:

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.

As correctly pointed out by Yancy this description does not correctly
describe the state of the code after that patch was merged. The
encoder newtype was put first but should be moved _below_ the trait
impls.

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

This commit simply reorders code in a single Rust source file. It moves the implementation of encoding/decoding traits above the helper 'encoder newtype' struct so the file matches the author's earlier stated intention. There is no functional change, no bug fix, and no security relevance.

Security candidateexamples: Use NetworkKind in call to PrivateKey::from_secby Tobin C. Harding · 6012d8a4 · Mar 11, 2026 · 1 fileMessage 90 · StrongInformational 15Details
Commit message · Tobin C. Harding

examples: Use NetworkKind in call to PrivateKey::from_sec

Guide users to the `NetworkKind` by calling `PrivateKey::from_secp`
with it. Done because I was confused the other day why private key
needed the network and I've been hacking on this library for 5 years
now ...

90/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Names security-relevant behavior explicitly
Why it was queued
secret or key material
AI analysis · Informational 15/100

This is a tiny documentation-style change to an example file. It swaps one way of saying 'use the Bitcoin main network' for another equivalent way, with no effect on actual code behavior or security.

Lower-priorityAdd error Debug API testby Jamil Lambert, PhD · c6952895 · Mar 11, 2026 · 1 fileMessage 48 · ThinInformational 12Details
Commit message · Jamil Lambert, PhD

Add error Debug API test

48/100 · ThinMessage clarity
✓ Subject identifies a change✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 12/100

This commit only adds a new automated test that checks the Debug text representation of two public error types is never empty. It does not change any library code, fix a bug, or alter behavior. There is no security issue in the diff itself.

Lower-priorityAdd std::error::Error impl API testby Jamil Lambert, PhD · 5c3e2297 · Mar 11, 2026 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · Jamil Lambert, PhD

Add std::error::Error impl API test

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

This commit only adds a new automated test that checks two existing error types implement Rust's standard Error trait. It does not change any production code, fix any bug, or alter behavior. There is no security relevance.

Lower-priorityImprove sync/send test coverageby Jamil Lambert, PhD · 0cfe5308 · Mar 11, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Jamil Lambert, PhD

Improve sync/send test coverage

Add missing Hkdf to the sync/send tests.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit only adds two lines to a test file, checking that a type called Hkdf can be safely used across threads (Send/Sync). It is purely a test-coverage improvement with no changes to production code, no bug fixes, and no security-relevant behavior changes.

AI review queuedhashes: Introduce hex stable and remove unecessary uses of hex_unstableby Mitchell Bagot · ba7cab15 · Mar 11, 2026 · 11 filesMessage 73 · AdequateInformational 18Details
Commit message · Mitchell Bagot

hashes: Introduce hex stable and remove unecessary uses of hex_unstable

The hashes crate provides various cross-crate macros, which expose
unstable hash errors in the public API of the crates that use them. In
order to remove these unstable errors from the bitcoin API, the hashes
crate needs to use stable hex_conservative errors in these macros.

Introduce hex_conservative 1.0.0 as a dependency for hashes and replace
uses of FromHex with decode_to_vec and decode_to_array.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 18/100

This commit is a routine dependency and API cleanup in the rust-bitcoin project's hashing crate. It swaps an unstable hex-parsing helper for a newly released stable version and updates the error types that downstream code sees. There is no direct evidence of a security vulnerability being fixed; it is best understood as a maintainability and API-stability improvement.

AI review queuedp2p: Introduce hex stable and remove unecessary uses of hex_unstableby Mitchell Bagot · 26ccfd92 · Mar 11, 2026 · 7 filesMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

p2p: Introduce hex stable and remove unecessary uses of hex_unstable

While the p2p crate doesn't directly expose unstable hex errors, it has
various uses of the unstable hex crate where the stable hex crate would
be sufficient.

Introduce hex_conservative 1.0.0 as a dependency for p2p and replace
uses of FromHex with decode_to_vec and decode_to_array.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 15/100

This commit is a routine internal cleanup in the rust-bitcoin project's peer-to-peer (p2p) crate. It swaps some test and internal code from using an unstable version of a hex-encoding helper library to a newly released stable version. There is no user-facing behavior change, no bug fix, and no security fix visible in the diff.

Security candidatebitcoin: Remove uses of HexToArrayErrorby Mitchell Bagot · 9f3eca94 · Mar 11, 2026 · 3 filesMessage 80 · StrongInformational 17Details
Commit message · Mitchell Bagot

bitcoin: Remove uses of HexToArrayError

The HexToArrayError in hex_conservative comes from the unstable version
of the crate. Since we want to avoid unstable hex conservative in the
public API, we should remove all uses of this error.

Remove all direct uses of HexToArrayError in bitcoin.

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 · Informational 17/100

This commit is a routine internal cleanup in the rust-bitcoin library. It swaps one hex-decoding helper for another equivalent one so the library no longer depends on an 'unstable' public API from a dependency. There is no indication this fixes a security bug or changes behavior visible to users in a dangerous way.

Security candidatebitcoin: Remove uses of HexToBytesErrorby Mitchell Bagot · 22140d7a · Mar 11, 2026 · 10 filesMessage 80 · StrongInformational 19Details
Commit message · Mitchell Bagot

bitcoin: Remove uses of HexToBytesError

The HexToBytesError in hex_conservative comes from the unstable version
of the crate. Since we want to avoid unstable hex conservative in the
public API, we should remove all uses of this error.

Remove all direct uses of HexToBytesError in bitcoin and remaining
uses of unstable FromHex trait.

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 pathsigning or wallet path
AI analysis · Informational 19/100

This commit swaps out an unstable, experimental hex-decoding helper for a stable, project-owned one. It is a cleanup/refactoring change to keep the public API from depending on a crate version that is not yet finalized. There is no direct security bug being fixed; the main risk is that if done incorrectly, error handling could change slightly for users parsing hex strings.