RB
← All projectsRust Bitcoin

rust-bitcoin

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

BitcoinCryptographic librariesNormal
Repository coverage

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

459security candidates448second-pass queue2070AI analyses
181commits · 30 days
400commits · 60 days
1283commits · 180 days
2053commits · 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
311Strong · 80–100
1054Adequate · 60–79
555Thin · 40–59
150Opaque · 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 Bagot640192640068
Tobin C. Harding40965409063
jrakibi944994068
Nick Johnson18920189059
Andrew Poelstra512051075
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 37 minutes ago

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
Informational 15 AI analysisMessage 45 · Thin
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

release bitcoin_hashes 1.2.0

This commit is a routine version bump for the bitcoin_hashes library from version 1.1.0 to 1.2.0. It only updates lock files, the package version number, and the changelog. There is no code change and no security relevance visible in the c…

8899dfd2by Andrew Poelstra+27−224 files
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

build(deps): bump astral-sh/setup-uv from 8.1.0 to 8.2.0

This is a routine automated update by Dependabot that changes the pinned version of a GitHub Action used to install the 'uv' Python tool in two workflow files. The new commit hash corresponds to the legitimate 8.2.0 release of astral-sh/se…

1c438632by dependabot[bot]+2−22 files
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

build(deps): bump github/codeql-action/upload-sarif

This is a routine automated update by Dependabot that changes the pinned version of a GitHub-provided security scanning helper used in a workflow. It does not modify the project's actual Bitcoin library code, its cryptographic logic, or an…

5293bee4by dependabot[bot]+1−11 file
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

build(deps): bump taiki-e/install-action from 2.79.12 to 2.81.4

This is a routine Dependabot update that bumps a GitHub Actions helper used in automated testing workflows from version 2.79.12 to 2.81.4. It only changes which version of the action is invoked during CI runs; it does not modify the Rust B…

eb9544f9by dependabot[bot]+2−22 files
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

build(deps): bump actions/checkout from 6.0.2 to 6.0.3

This is a routine automated update by Dependabot that bumps the GitHub Actions 'actions/checkout' dependency from version 6.0.2 to 6.0.3 across all project workflow files. It only changes pinned commit hashes and version comments for the a…

Routine patch-level dependency bumpNo changes to permissions, inputs, or workflow logicpersist-credentials: false already present on all checkout steps
ae935fadby dependabot[bot]+29−2915 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-priorityImplement serde functions in as_consensusby Tobin C. Harding · 129c7acf · Mar 19, 2026 · 5 filesMessage 80 · StrongInformational 17Details
Commit message · Tobin C. Harding

Implement serde functions in as_consensus

Add a module for serde stuff and add a sub-module that provides
implementations of the `serde` traits by way of consensus encoding.

Update the `serde` example in `bitcoin` to use the new logic instead
of the old consensus encoding stuff.

Note that the code in `bitcoin::consensus::serde` is a bit more
complicated than what is added here. I _think_ that is because of
having to handle the I/O error in the old consensus code.

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
AI analysis · Informational 17/100

This commit adds a new helper module that lets Rust Bitcoin types be serialized and deserialized using Bitcoin's standard binary encoding, exposed through the popular serde serialization framework. It is a feature/refactoring change: it introduces a cleaner way to get the same behavior that previously required more verbose code. There is no indication in the commit that it fixes a security bug or vulnerability.

Security candidateIntroduce WifKey for private keys with networkby Mitchell Bagot · 2f7b79b6 · Mar 19, 2026 · 8 filesMessage 68 · AdequateInformational 17Details
Commit message · Mitchell Bagot

Introduce WifKey for private keys with network

In bitcoin, the PrivateKey type holds a secp SecretKey, a
compressedness flag and a NetworkKind. The NetworkKind field is only
ever used when converting to/from a WIF key string. Since a PrivateKey
can (or will) be usable for functions beyond just WIF import/export, it
is necessary to remove the dependence on network for the key type.

Introduce WifKey which holds a PrivateKey and a NetworkKind.
Remove network field from PrivateKey.
Replace uses of PrivateKey involving WIF logic with WifKey.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
secret or key materialcryptography-sensitive pathsigning or wallet path
AI analysis · Informational 17/100

This commit is a routine API refactor in the rust-bitcoin library. It splits the old PrivateKey type into two parts: a simpler PrivateKey that only stores the secret key and compression flag, and a new WifKey type that pairs a PrivateKey with a Bitcoin network for WIF import/export. The change removes the network field from PrivateKey and moves WIF-specific parsing, serialization, and formatting onto WifKey. There is no security fix here; it is a design cleanup to make the key types more general-purpose.

Security candidateRemove private key from create-p2wpkh-address exampleby Mitchell Bagot · d6b18416 · Mar 19, 2026 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Remove private key from create-p2wpkh-address example

The create-p2wpkh-address example demonstrates how to construct a
P2WPKH address from a secp PublicKey, going through the
CompressedPublicKey type. Currently, it also logs out the private key
required to spend from that address. Since the example is to
demonstrate address construction, the private key is not needed.

Remove the private key logging from the create-p2wpk-address example.

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

This commit removes a line from a code example that printed a randomly-generated private key. The example now only prints the Bitcoin address. It is a documentation/example cleanup, not a fix for a software vulnerability in the library itself.

AI review queuedRe-export serde and arbitrary when they appear in public APIby Mitchell Bagot · e59a0040 · Mar 19, 2026 · 7 filesMessage 85 · StrongInformational 19Details
Commit message · Mitchell Bagot

Re-export serde and arbitrary when they appear in public API

In various crates, we have types from external crates in the public
APIs, such as serde's Serialize or arbitrary's Arbitrary. In order to
simplify dependency handling, we should re-export these crates which
have types appearing in the public API.

Add pub extern crate for serde and arbitrary in bitcoin, network, p2p,
units and primitives as needed.

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 19/100

This change makes two helper libraries (serde and arbitrary) publicly visible from several rust-bitcoin crates. It is a dependency-management convenience for downstream developers and does not fix or introduce any security vulnerability.

Lower-priorityprimitives: Replace double alloc in script formatting with iteratorsby Mitchell Bagot · 51267c03 · Mar 19, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

primitives: Replace double alloc in script formatting with iterators

In Script in primitives, the to_hex_string_prefixed function uses a
double allocation to first encode the script to a vector, before
converting it to a hex string. With the changes to EncodableByteIter,
the first vector allocation can be entirely bypassed in favour of the
iterator.

Replace vector allocation in to_hex_string_prefixed with
EncodableByteIter.

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

This is a routine performance cleanup, not a security fix. It removes an unnecessary intermediate memory allocation when converting a Bitcoin script to a hex string by using an iterator directly. There is no change to behavior, parsing, validation, or cryptographic handling.

Lower-priorityconsensus_encoding: Allow ?Sized for EncodableByteIterby Mitchell Bagot · a180da82 · Mar 19, 2026 · 4 filesMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

consensus_encoding: Allow ?Sized for EncodableByteIter

The EncodableByteIter currently takes any Sized Encodable and yields
bytes in an iterator. While useful, the Sized bound makes this more
restrictive than the other helper functionality in consensus_encoding
like encode_to_vec, which all explicitly permit ?Sized encodables.

Loosen trait bound for EncodableByteIter generic to accept ?Sized
encodable types.

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

This change relaxes a Rust type-system restriction so a particular byte-iterator helper can work with trait objects and slices, not just fixed-size types. It is a routine API ergonomics improvement with no security relevance visible in the commit.

Lower-priorityExclude RFC7539 tests from chacha20_poly1305_fuzz buildsby Abeeujah · b8fe28bb · Mar 18, 2026 · 3 filesMessage 83 · StrongInformational 15Details
Commit message · Abeeujah

Exclude RFC7539 tests from chacha20_poly1305_fuzz builds

The rfc7539 and related tests `new_from_block` require the
implementation to be spec-compliant. However under the
`chacha20_poly1305_fuzz` cfg flag, the `apply_keystream` is a no-op
that leaves ciphertext unmodified.

This intentional deviation from the RFC causes these specific tests
to fail. Gating them prevents false positives in
`--cfg=chacha20_poly1305_fuzz` builds.

83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 15/100

This commit only changes test code. It adds a configuration gate so that certain cryptography specification tests are skipped when the code is built with a special fuzzing flag. The fuzzing flag intentionally disables encryption to make fuzz testing easier, which naturally causes those spec-compliance tests to fail. There is no change to production code and no security vulnerability is being fixed.

AI review queuedapi: update for CompactSizeU64Decoderby Nick Johnson · 85a6da3a · Mar 18, 2026 · 5 filesMessage 35 · OpaqueInformational 15Details
Commit message · Nick Johnson

api: update for CompactSizeU64Decoder

35/100 · OpaqueMessage clarity
✓ Descriptive subject! 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 functions) to reflect a new CompactSizeU64Decoder type and a moved CompactSizeEncoder module path. No actual source code, logic, or behavior changes are present in the diff, so it has no security relevance on its own.

AI review queuedconsensus_encoding: move compact size into its own moduleby Nick Johnson · a06bf033 · Mar 18, 2026 · 4 filesMessage 65 · AdequateInformational 15Details
Commit message · Nick Johnson

consensus_encoding: move compact size into its own module

Public API is unchanged; lib.rs re-exports all the same types.

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: security-sensitive path
AI analysis · Informational 15/100

This commit is a pure code reorganization: it moves the compact-size integer encoder and decoder into a new dedicated file/module. The public API is unchanged, the logic is unchanged, and no security-relevant behavior is modified.

Lower-priorityconsensus_encoding: add a u64 constructor to compact size encoderby Nick Johnson · 773b4a35 · Mar 18, 2026 · 4 filesMessage 50 · ThinInformational 18Details
Commit message · Nick Johnson

consensus_encoding: add a u64 constructor to compact size encoder

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

This commit adds a new way to create a compact-size encoder using a u64 value, and slightly tightens the existing usize constructor. It is a routine API improvement, not a security fix. There is no evidence in the commit or supplied references that this resolves any vulnerability.

Lower-priorityconsensus_encoding: add round trip test for compact size codecsby Nick Johnson · 46145158 · Mar 18, 2026 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · Nick Johnson

consensus_encoding: add round trip test for compact size codecs

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 adds new test code for the CompactSize encoding/decoding logic. It does not change any production code, so it cannot introduce a security vulnerability on its own. The tests verify that valid compact-size values encode and decode correctly, and that non-minimal encodings are rejected as errors.

Lower-priorityconsensus_encoding: add a u64 decoder for compact sizeby Nick Johnson · 77a60aea · Mar 18, 2026 · 2 filesMessage 50 · ThinModerate 50Details
Commit message · Nick Johnson

consensus_encoding: add a u64 decoder for compact size

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

This commit adds a new decoder for reading very large Bitcoin-style numbers (up to 64 bits) and refactors the existing length decoder to share the same parsing logic. The change itself is defensive: it separates 'length' decoding (which has a safety cap to prevent memory exhaustion) from 'raw number' decoding (which has no cap because it's not used for allocation). The commit does not fix a known exploit, but it strengthens the library's design so future callers are less likely to accidentally use an uncapped decoder for lengths.

Lower-priorityconsensus_encoding: const the compact size prefix valuesby Nick Johnson · bb7bf48d · Mar 18, 2026 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Nick Johnson

consensus_encoding: const the compact size prefix values

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

This commit is a simple code cleanup: it replaces hardcoded numbers (0xFD, 0xFE, 0xFF) with named constants (PREFIX_U16, PREFIX_U32, PREFIX_U64) in the compact-size integer encoder/decoder. The behavior of the code is unchanged. There is no security fix or vulnerability here.

Lower-priorityp2p: add V1MessageHeader encoding traitsby yancy · f62f81ed · Mar 18, 2026 · 1 fileMessage 76 · AdequateInformational 15Details
Commit message · yancy

p2p: add V1MessageHeader encoding traits

`V1MessageHeader` has traits for decoding a message header, however the
traits to encode a header are missing.

The four fields that are encoded are as follows:
* magic.encoder()
* command.encoder()
* length is cast using to_le_bytes per protocol docs
* no casting is needed for checksum since it's already a u8[]

ref: https://en.bitcoin.it/wiki/Protocol_documentation

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

This commit adds the missing ability to encode (serialize) Bitcoin P2P v1 message headers, matching the existing decoding support. It is a straightforward feature addition with no security-relevant behavior change.

Security candidateImplement `Arbitrary` for psbt typesby Shing Him Ng · 37c9fce2 · Mar 18, 2026 · 17 filesMessage 45 · ThinInformational 19Details
Commit message · Shing Him Ng

Implement `Arbitrary` for psbt types

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

This commit adds optional support for the `arbitrary` crate, a Rust fuzz-testing helper, across several data types in the rust-bitcoin library. It does not change normal runtime behavior; the new code is only compiled when the optional `arbitrary` feature is enabled. There is no direct security vulnerability in the diff, but it slightly expands the library's dependency tree and public API surface for users who opt into the feature.

Security candidatepsbt: Add unit test case for key byte size handlingby Mitchell Bagot · 3c05450d · Mar 18, 2026 · 1 fileMessage 95 · StrongLow 35Details
Commit message · Mitchell Bagot

psbt: Add unit test case for key byte size handling

Recently, a change ws introduced in psbt to fix key parsing when the
key has a total byte size value that is so small as to fail to cover
the type compact size. In this case, a panic/underflow could occur.
This has been fixed, but a test case should be included to prevent
regressions.

Introduce unit test to verify that small key bytes throw errors instead
of panicing due to underflow.

95/100 · StrongMessage clarity
✓ Specific, 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
signing boundarysigning or wallet path
AI analysis · Low 35/100

This commit only adds a new unit test; it does not change production code. The test checks that parsing a malformed PSBT key with an impossibly small byte-size field returns a proper error instead of crashing. The underlying panic/underflow bug was already fixed in an earlier change, so this patch is a regression test rather than a fix.

Lower-priorityunits: Add test file for string parsing of integer typesby Mitchell Bagot · 4429fb88 · Mar 18, 2026 · 1 fileMessage 95 · StrongInformational 15Details
Commit message · Mitchell Bagot

units: Add test file for string parsing of integer types

With the integer wrapper types all not implementing hex parsing,
they should be tested to ensure correct function and prevent
regressions.

Add test file for testing string parsing for integer types.

95/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit only adds a new test file that checks how integer wrapper types parse hexadecimal strings. It does not change any production code, so it cannot introduce a security vulnerability on its own. It is a routine quality-assurance addition.

Lower-priorityunits: Update API filesby Mitchell Bagot · aec52a08 · Mar 18, 2026 · 3 filesMessage 51 · ThinInformational 15Details
Commit message · Mitchell Bagot

units: Update API files

The previous patches have introduced new functions on various types in
the public API.

Run just check-api to update units api files.

51/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only updates automatically-generated API listing files (text files that record what functions exist in the public interface). No actual source code was changed, and nothing was fixed or removed. It is purely a documentation/tracking update for newly added public functions introduced by earlier commits.

Lower-priorityunits: Change hex parsing on Height and MedianTimePastby Mitchell Bagot · 64988240 · Mar 18, 2026 · 2 filesMessage 85 · StrongInformational 18Details
Commit message · Mitchell Bagot

units: Change hex parsing on Height and MedianTimePast

Height and MedianTimePast both have an existing from_hex function that
parses from an optionally prefixed hex string. In order to match the
other integer wrapper types in units, this should be split to from_hex
and from_unprefixed_hex.

Change from_hex to only parse prefixed hex in Height and
MedianTimePast. Introduce from_unprefixed_hex in Height and
MedianTimePast to parse unprefixed hex. Adjust ParseTimeError and
ParseHeightError accordingly.

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
AI analysis · Informational 18/100

This commit changes how two Bitcoin locktime types (block height and median-time-past) parse hexadecimal strings. Previously, a single function accepted hex with or without a '0x' prefix. Now there are two separate functions: one that requires the '0x' prefix and one that rejects it. This is an API-consistency cleanup, not a fix for an exploitable vulnerability. Existing callers that relied on the old flexible behavior could break at compile time or runtime if they still pass unprefixed hex to from_hex, but the change is intentional and documented.

Lower-priorityunits: Add hex parsing for numeric wrapper typesby Mitchell Bagot · fe483ad9 · Mar 18, 2026 · 4 filesMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

units: Add hex parsing for numeric wrapper types

units contains various other numeric wrapper types that can be
formatted as numeric data. To complement this, they should also be able
to be parsed from data like hex with the same units.

Introduce from_hex and from_hex_unprefixed to all remaining numeric
wrapper types aside from FeeRate.

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

This commit adds new public helper functions that let several numeric wrapper types in the rust-bitcoin 'units' crate be created from hexadecimal strings (with or without a '0x' prefix). It is a straightforward feature addition with no security-relevant behavior change; the underlying hex parsing already existed in the crate.

AI review queuedunits: Add hex parsing for Amount and SignedAmountby Mitchell Bagot · 7abbd16f · Mar 18, 2026 · 3 filesMessage 73 · AdequateInformational 12Details
Commit message · Mitchell Bagot

units: Add hex parsing for Amount and SignedAmount

The Amount and SignedAmount types are numeric wrapper types that should
include functionality for parsing from hex. Since the types can error
during construction for non-hex reasons, the existing ParseAmountError
needs to be extended

Add additional variants to ParseAmountErrorInner to handle hex parsing.
Introduce from_hex and from_hex_unprefixed to Amount and SignedAmount.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 12/100

This commit adds new public functions that let users create Bitcoin amount values from hexadecimal strings. It is a feature addition, not a fix for a known bug or vulnerability. There is no evidence in the commit or supplied references that this change addresses a security issue.

Lower-priorityp2p: add pong message typeby yancy · 87c691a7 · Mar 17, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · yancy

p2p: add pong message type

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

This commit adds support for the Bitcoin network's 'pong' message type to the library. A pong is the standard reply to a ping, used to check that a peer is still alive. The change only adds new code for encoding, decoding, and testing this message type; it does not alter existing behavior or fix any vulnerability.

Lower-priorityp2p: add ping message typeby yancy · 9fb559d5 · Mar 17, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · yancy

p2p: add ping message type

Remove tests for previous serialize implementation of ping since this
encoder/decoder replaces it.

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 adds a new way to encode and decode Bitcoin 'ping' network messages in the rust-bitcoin library. It is a routine feature addition: it introduces a dedicated Ping message type with proper serialization/deserialization and replaces older serialization tests with a round-trip test. There is no indication of a security bug being fixed.

Lower-priorityAdd fuzzing support for chacha20_poly1305 via cfg flagby Abeeujah · fdf898d8 · Mar 17, 2026 · 4 filesMessage 85 · StrongInformational 24Details
Commit message · Abeeujah

Add fuzzing support for chacha20_poly1305 via cfg flag

Introduce `chacha20_poly1305_fuzz` cfg to replace ChaCha20 and Poly1305
primitives with deterministic no-op implementations for fuzzing.

- ChaCha20: keystream returns zeroed 64-byte blocks; apply_keystream
is a no-op.
- Poly1305: stores first 16 bytes of key directly as the tag (trivial
construction avoiding crypto bottlenecks).
- AEAD: derives poly_key directly from raw key bytes, bypassing
ChaCha20 keystream entirely.
- Key: change visibility to pub(super).

Register cfg in workspace lints to prevent unexpected_cfg violations.

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 24/100

This commit adds a special build-time flag named `chacha20_poly1305_fuzz` that swaps out real ChaCha20/Poly1305 cryptography with fake, deterministic no-op versions intended only for fuzz testing. The fake versions do not encrypt data and do not authenticate it; they are meant to be used only when explicitly compiling with the fuzz flag. The change itself is a test/infrastructure addition, not a normal production code path. The main risk is accidental misuse: if someone builds or ships production code with `chacha20_poly1305_fuzz` enabled, encryption and authentication would be silently disabled. The commit does not claim to fix a vulnerability and does not appear to introduce one in normal builds.

Lower-priorityjustfile: simplify with new cargo-rbmt featuresby Nick Johnson · 74902e1a · Mar 17, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

justfile: simplify with new cargo-rbmt features

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

This commit only edits the project's justfile, a developer convenience script. It removes explicit Rust toolchain arguments because a newer version of the cargo-rbmt tool now handles toolchains internally. There is no change to the actual Bitcoin library code, no user-facing behavior change, and no security relevance.