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
184commits · 30 days
409commits · 60 days
1296commits · 180 days
2064commits · 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 20 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-priorityunits: Remove old encoding test casesby Mitchell Bagot · 133cf323 · Jan 24, 2026 · 3 filesMessage 78 · AdequateInformational 15Details
Commit message · Mitchell Bagot

units: Remove old encoding test cases

The test cases for encoding/decoding of Sequence, BlockHeight and
BlockTime are no longer required for coverage with the introduction
of the encoding test module.

Remove redundant unit tests for encoding/decoding of Sequence,
BlockHeight and BlockTime.

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

This commit simply deletes redundant unit tests for encoding and decoding of Sequence, BlockHeight, and BlockTime. It does not change any production code, so it cannot introduce a security vulnerability or fix one. The remaining error-handling tests still exercise the encoding feature.

Lower-priorityunits: Add tests for encodable/decodable typesby Mitchell Bagot · b78a8806 · Jan 24, 2026 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · Mitchell Bagot

units: Add tests for encodable/decodable types

While some of the encodable and decodable types in units have tests
to cover the encoding and decoding behaviour, the code is largely
repeated across the types. It better lives in a distinct test file.

Introduce tests/encoding.rs which covers decoding and encoding
behaviour for all Encodable/Decodable types.

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

This commit only adds a new test file that checks whether various Bitcoin data types can be correctly encoded to bytes and decoded back again. It does not change any production code, so it cannot introduce a security vulnerability or fix one.

Lower-priorityclippy: modify config fileby Tobin C. Harding · 791476fc · Jan 23, 2026 · 1 fileMessage 58 · ThinInformational 15Details
Commit message · Tobin C. Harding

clippy: modify config file

Modify clippy config file to configure clippy to understand a couple
of domain specific terms. Without this the linter flags these as
requiring backticks thinking they are Rust identifiers.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit only changes the project's Clippy linter configuration file. It removes some general settings and tells the linter that 'SegWit' and 'OpenSSL' are valid terms in documentation so it won't nag about missing backticks. There is no code change and no security impact.

Security candidateCI: Use the new cargo rbmt toolby Tobin C. Harding · 7810f3a2 · Jan 23, 2026 · 48 filesMessage 90 · StrongInformational 15Details
Commit message · Tobin C. Harding

CI: Use the new cargo rbmt tool

Use the mad new `cargo rbmt` tool for CI. Note that for `bitcoin` the
`extra_tests.sh` test are rolled into `rbmt.toml` now that running
without default args is supported.

90/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100

This commit is a routine update to the project's automated testing setup. It replaces a set of shell scripts with a new Rust tool called `cargo rbmt` for running tests and lints in CI. There are no changes to the actual Bitcoin library code that users interact with, and no security issue is evident.

AI review queuedp2p: Allow too many argsby Tobin C. Harding · b997f142 · Jan 23, 2026 · 1 fileMessage 38 · OpaqueInformational 15Details
Commit message · Tobin C. Harding

p2p: Allow too many args

For now just shoosh the linter.

38/100 · OpaqueMessage clarity
✓ Subject identifies a change✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100

This commit adds a single Rust lint suppression comment to silence a Clippy warning about a function having too many arguments. It does not change any executable code, behavior, or data flow. There is no security relevance.

Security candidatehashes: Allow too many argsby Tobin C. Harding · 58495710 · Jan 23, 2026 · 3 filesMessage 60 · AdequateInformational 15Details
Commit message · Tobin C. Harding

hashes: Allow too many args

These are private crypto functions, just shoosh the linter.

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

This commit only adds Clippy lint-suppression attributes (`#[allow(clippy::too_many_arguments)]`) to three internal cryptographic helper functions in the `rust-bitcoin` hashes crate. It does not change any executable code, logic, or public API. There is no security relevance: it simply tells the Rust linter to stop warning that these functions have many parameters.

Lower-priorityp2p: Implement `encoding` traits for `GetBlocks/HeadersMessage`by rustaceanrob · 8476d6fb · Jan 23, 2026 · 1 fileMessage 73 · AdequateInformational 12Details
Commit message · rustaceanrob

p2p: Implement `encoding` traits for `GetBlocks/HeadersMessage`

This is a rare case where this can be macro-ized, but I think the extra
pain of code duplication makes this easier to read than a macro.

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

This commit adds new ways to encode and decode two Bitcoin peer-to-peer network messages, `GetBlocks` and `GetHeaders`. It does not change the existing encoding logic; it only introduces an alternative implementation using newer internal traits. There is no indication this fixes a security bug or introduces a vulnerability.

Lower-priorityunits: Extend test coverage of amount typesby Mitchell Bagot · 76864db3 · Jan 23, 2026 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · Mitchell Bagot

units: Extend test coverage of amount types

The Amount and SignedAmount types have various functions that are not
covered by existing tests, including to_btc, display_dynamic,
checked_mul/div and div_by_weight.

Extend test coverage of amount types.

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

This commit only adds new unit tests for existing Bitcoin amount types. It does not change any production code, fix bugs, or alter behavior. There is no security relevance.

Lower-priorityImprove test coverage for WitnessMerkleNodeby Jamil Lambert, PhD · 0dbce2bd · Jan 22, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Jamil Lambert, PhD

Improve test coverage for WitnessMerkleNode

Add tests to increase the test coverage to 100% for witness_merkle_node.

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 new unit tests for an existing Bitcoin hash type. It does not change any production code, so it cannot introduce or fix a security vulnerability on its own.

Lower-priorityImprove test coverage in hash_typesby Jamil Lambert, PhD · 329c2ac4 · Jan 22, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Jamil Lambert, PhD

Improve test coverage in hash_types

Add a test to increase the test coverage to 100% for hash_types.
Excluding serde.

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 a new unit test to verify that different ways of reading a transaction hash (Txid) return the same underlying bytes. It does not change any production code, APIs, or behavior. There is no security relevance.

AI review queuedImprove test coverage for TxMerkleNodeby Jamil Lambert, PhD · 5fb07d04 · Jan 22, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Jamil Lambert, PhD

Improve test coverage for TxMerkleNode

Add two tests to increase the test coverage to 100% for
transaction_merkle_node.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only adds two new unit tests for the TxMerkleNode type. It does not change any production code, fix a bug, or alter behavior. There is no security relevance.

Lower-priorityImprove test coverage for block_hashby Jamil Lambert, PhD · 7766f03a · Jan 22, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Jamil Lambert, PhD

Improve test coverage for block_hash

Add two tests to increase the test coverage to 100% for block_hash.

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 new automated tests for the block hash decoder. It does not change any production code, so it cannot introduce or fix a security vulnerability on its own.

Lower-priorityMake test dummy txid hex a constby Jamil Lambert, PhD · 70f441fd · Jan 22, 2026 · 1 fileMessage 82 · StrongInformational 15Details
Commit message · Jamil Lambert, PhD

Make test dummy txid hex a const

Make the existing dummy txid a constant so that it can be used in
additional tests.

82/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Explains rationale or failure mode✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit is a minor internal test cleanup. It moves a hard-coded dummy transaction ID string used only in unit tests into a constant so it can be reused in future tests. There is no change to production code, no security fix, and no user-facing behavior change.

Lower-priorityadd aarch64 cross testing (needed for ARM SHA acceleration)by jrakibi · baaab030 · Jan 22, 2026 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · jrakibi

add aarch64 cross testing (needed for ARM SHA acceleration)

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only changes the project's GitHub Actions CI configuration to add automated cross-compilation testing for the aarch64 (64-bit ARM) Linux target, alongside the existing s390x target. It does not modify any library code, cryptographic logic, or user-facing behavior. There is no security issue here.

Lower-priorityp2p: Implement `encoding` traits for `Reject`by rustaceanrob · 73b14429 · Jan 22, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · rustaceanrob

p2p: Implement `encoding` traits for `Reject`

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

This commit adds new serialization/deserialization code for the Bitcoin P2P 'reject' message using the project's newer encoding framework. It is a routine refactor/migration of an existing message type to a new encoder/decoder API. There is no indication in the commit or supplied references that this fixes a security bug, vulnerability, or crash.

Lower-priorityp2p: Implement `encoding` traits for `RejectReason`by rustaceanrob · 8c230783 · Jan 22, 2026 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · rustaceanrob

p2p: Implement `encoding` traits for `RejectReason`

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

This commit adds new encoding/decoding machinery for the RejectReason type used in Bitcoin peer-to-peer messages. It does not change existing behavior or fix any known bug; it is a routine feature addition that makes the type work with a newer internal encoding framework. There is no indication of a security issue.

Lower-priorityp2p: Implement `encoding` traits for `BlockTransactionsRequest`by rustaceanrob · b34d8825 · Jan 22, 2026 · 1 fileMessage 65 · AdequateInformational 12Details
Commit message · rustaceanrob

p2p: Implement `encoding` traits for `BlockTransactionsRequest`

Straightforward implementation with the introduction of `Offset`

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

This commit adds new encoding and decoding machinery for a Bitcoin peer-to-peer message type (BlockTransactionsRequest). It is a routine feature addition that mirrors an existing implementation and does not appear to fix or introduce a security problem on its own.

AI review queuedbip152: Add an `Offset` wrapper typeby rustaceanrob · 940602f0 · Jan 22, 2026 · 2 filesMessage 90 · StrongLow 36Details
Commit message · rustaceanrob

bip152: Add an `Offset` wrapper type

When implementing `encoding` with a list of items it's nice to have
`Encodable` implemented for the item that will go into the
`SliceEncoder` to avoid prep steps. I also think this type should be
revamped.

For one, having struct initialization available for this type allows for
creating completely invalid states that result in panics. This is shown
by a test that literally asserts creating the struct results in a panic
when trying to serialize it later. A simple case that is not currently
tested is when the user passes an unsorted vector in the initialization
and later tries to encode this type. I introduce two constructors here
that make it clear the client should sort the list themselves, or allow
the sort to happen on construction. The encoding functions here were
doing an immense amount of logic, which I think should be pushed out to
a designated method call.

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: broader security terminology
AI analysis · Low 36/100

This commit refactors how Bitcoin compact-block transaction indexes are stored and encoded. It replaces a public Vec<u64> field with a private Vec<Offset> field and adds constructors that require callers to provide sorted indices. The change removes a documented panic during serialization when u64::MAX was used, and moves overflow detection from encoding time to a new indices() accessor. It is a defensive hardening patch rather than a clear-cut vulnerability fix, because the previous panic required a caller to deliberately construct an invalid object and then serialize it.

AI review queuedAdjust error re-export doc inliningby Mitchell Bagot · 9209abdf · Jan 21, 2026 · 3 filesMessage 80 · StrongInformational 15Details
Commit message · Mitchell Bagot

Adjust error re-export doc inlining

Error re-exports in bitcoin and primitives should use #[doc(no_inline)]
to ensure that pure re-export types are rendered as re-exports and not
defined types of the module in docs.

Use #[doc(no_inline)] for BlockHashDecoderError in primitives and
PushBytesError, ParseTransactionError, and ParseOutPointError 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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only changes how Rust documentation is generated for certain error types. It switches some re-exports from being shown as if they were defined in the current module to being shown as re-exports from another module. There is no change to actual code behavior, no security fix, and no vulnerability.

Lower-priorityp2p: add v1 header decoderby yancy · 10c68edb · Jan 20, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · yancy

p2p: add v1 header decoder

Allow decoding a p2p network message header.

The header can then be read using the consensus_encoding crate:

```
let V1MessageHeader {command, ..} =
decode_from_read_unbuffered::<V1MessageHeader, _>(&mut r).unwrap()
```

Update p2p/src/message.rs

Co-authored-by: Mitchell Bagot <packof14eggies@gmail.com>

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

This commit adds a new decoder for Bitcoin peer-to-peer network message headers in the rust-bitcoin library. It is a straightforward feature addition that exposes a way to parse v1 message headers using the existing consensus encoding framework. There is no indication of a security bug, fix, or vulnerability.

Lower-priorityp2p: Implement `encoding` traits for `FilterLoad`by rustaceanrob · 89bfc055 · Jan 20, 2026 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · rustaceanrob

p2p: Implement `encoding` traits for `FilterLoad`

Another low-sakes encoding implementation building on the previous
commit.

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

This commit adds a new way to encode and decode the Bitcoin 'filterload' P2P network message using a new internal encoding framework. It is a routine refactor that mirrors how other message types were already handled. There is no indication of a security bug being fixed or introduced.

Lower-priorityp2p: Implement `encoding` traits for `BloomFlags`by rustaceanrob · fca56538 · Jan 20, 2026 · 1 fileMessage 68 · AdequateInformational 13Details
Commit message · rustaceanrob

p2p: Implement `encoding` traits for `BloomFlags`

Here is an example of a field with a single byte representation. Does it
make sense to use the `ArrayEncoder` here? I think so, unless there is
something I'm missing.

I also took the liberty of adding a variant to the unknown bloom flag
error that actually reports the unknown flag back. Not that it should
see any use.

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

This commit adds new encoding helpers for a small network-message flag type called BloomFlags. It does not change existing behavior; it only wires up a new internal encoding trait implementation and improves an error message to report the actual unknown flag value. There is no indication this fixes or introduces a security issue.

Security candidateunits: Extend test coverage for NumOpResultby Mitchell Bagot · 0bb519bc · Jan 20, 2026 · 2 filesMessage 78 · AdequateInformational 15Details
Commit message · Mitchell Bagot

units: Extend test coverage for NumOpResult

NumOpResult has a similar API to core's Result type, most of which is
untested. Other functionality, like the is_overflow() function are also
not tested by existing tests.

Implement tests for unwrap, expect, ok, map and other similar functions
in NumOpResult. Adjust tests to cover is_overflow and is_div_by_zero.

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
memory safety
AI analysis · Informational 15/100

This commit only adds new unit tests for an existing Rust type called NumOpResult. It does not change any production code, fix bugs, or alter behavior. There is no security issue here.

Lower-priorityTest MAX_ATTAINABLE_* constant values against Core valuesby Mitchell Bagot · 7f73089d · Jan 19, 2026 · 1 fileMessage 95 · StrongInformational 15Details
Commit message · Mitchell Bagot

Test MAX_ATTAINABLE_* constant values against Core values

The MAX_ATTAINABLE_* constants in Target represent compact lossy values
of the target limits in Bitcoin Core. In order to ensure that these are
correctly converted here, we should compare against the original
known-correct values from Bitcoin Core.

Introduce a test case to assert that converting to then from
CompactTarget yields the MAX_ATTAINABLE_* consts.

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 unit test. It does not change any production code, so it cannot introduce a security vulnerability or fix one. The test verifies that certain difficulty-target constants in the rust-bitcoin library match the values used in Bitcoin Core after a compact/lossy conversion.

Lower-prioritybitcoin: Replace Network typesby Mitchell Bagot · 349ca990 · Jan 19, 2026 · 6 filesMessage 58 · ThinInformational 18Details
Commit message · Mitchell Bagot

bitcoin: Replace Network types

With the introduction of bitcoin-network-kind, the network types can
all be removed from bitcoin in favour of re-exports from the network
crate.

Remove Network, NetworkKind, TestnetVersion and ParseNetworkError from
bitcoin::network, and re-export the types from bitcoin-network-kind.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
AI analysis · Informational 18/100

This commit is a routine internal code reorganization. It removes the definitions of Bitcoin network types (like Mainnet, Testnet, Signet) from the main `bitcoin` crate and instead re-exports them from a new dedicated `bitcoin-network-kind` crate. It also adds fallback handling for any future testnet versions. There is no indication this fixes a security vulnerability.