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 queue909AI analyses
188commits · 30 days
404commits · 60 days
1283commits · 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
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 Bagot640192315068
Tobin C. Harding40965150063
jrakibi944961068
Nick Johnson1892079059
Andrew Poelstra512031075
Jamil Lambert, PhD1141848061
Fmt Bot311420045
Trevor Arjeski111111069
Shing Him Ng31718056
Ismail Daif2268050
satsfy (Renato Britto)1769066
Martin Habovstiak27511069
Analysis record

Published AI watches

Last scanned 35 minutes ago

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

Update API files

This commit only updates generated API snapshot text files to remove two lines that documented a `From<u16>` conversion for a locktime type. No actual Rust source code was changed, so there is no runtime behavior change or security fix in …

6ea25bedby Mitchell Bagot+0−123 files
No security note in commit
Informational 15 AI analysisMessage 60 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

base58ck: Bump version to 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 describing unrelated feature changes. There are no code changes,…

8479b56eby Tobin C. Harding+22−168 files
No security note in commit
Informational 20 AI analysisMessage 58 · Thin
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Rename Error to DecodeCheckError

This commit is a routine code cleanup: it renames the `Error` type in the base58 module to the more descriptive `DecodeCheckError`, and adds a deprecated type alias so existing code using `base58::Error` continues to work. There is no chan…

b0eba4c0by Mitchell Bagot+45−395 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 snapshot text files to reflect that the remainder (modulo) operator for the Weight type now returns another Weight instead of a plain number. It contains no source code changes, no bug fix, and no sec…

351acf8eby Mitchell Bagot+6−63 files
No security note in commit
Informational 15 AI analysisMessage 73 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

consensus_encoding: decoder docs which are consensus specific

This commit only changes documentation comments in the consensus_encoding crate. It renames descriptions like 'Primitive decoders' to 'Primitive and combinator decoder types' and updates several function doc comments to say 'consensus deco…

5cdd5aa0by Nick Johnson+11−132 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.

AI review queuedAdd hex codec traits for Transactionby Mitchell Bagot · 7c2ddb17 · Jan 2, 2026 · 4 filesMessage 68 · AdequateInformational 18Details
Commit message · Mitchell Bagot

Add hex codec traits for Transaction

Currently, the Transaction type can be easily encoded into bytes using
consensus_encoding, but there exists no way to encode it to hex without
using the bitcoin crate's serialize_hex or a manual implementation.
Neither such solution is elegant or discoverable.
In implementing traits to hex, FromStr should also be introduced to
decode from a hex string.

Implement FromStr, Display, LowerHex, UpperHex and DisplayHex for
Transactions.

68/100 · AdequateMessage clarity
✓ 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 18/100

This commit adds convenience features to the rust-bitcoin library so that Bitcoin transactions can be easily converted to and from hexadecimal strings. It does not fix a bug, change security behavior, or introduce any obvious vulnerability. It is a routine API enhancement.

AI review queuedprimitives: reject transactions with invalid coinbase scriptSig lengthby jrakibi · c621401a · Jan 1, 2026 · 2 filesMessage 65 · AdequateModerate 52Details
Commit message · jrakibi

primitives: reject transactions with invalid coinbase scriptSig length

Coinbase input scriptSig must be between 2 and 100 bytes in length.
Add this validation rule to transaction decoder

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Moderate 52/100

This commit adds a missing validation rule to the library's Bitcoin transaction decoder: a coinbase transaction (the special first transaction in a block that creates new coins) must have an input script between 2 and 100 bytes long. Before this change, the library would accept shorter or longer coinbase scripts, which violates Bitcoin consensus rules and could let malformed transactions slip through. The fix rejects them with clear errors and includes tests for the boundary cases.

AI review queuedUpdate the API filesby Tobin C. Harding · 7cef310d · Dec 30, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · Tobin C. Harding

Update the API files

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

This commit only updates an automatically-generated API listing file. It records that a new test-helper trait implementation (for generating random 'relative::LockTime' values) is now part of the crate's public API surface. There is no code change, no bug fix, and no security-sensitive behavior introduced.

AI review queuedprimitives: prevent null prevout in non-coinbase transactionsby jrakibi · c71db422 · Dec 28, 2025 · 1 fileMessage 73 · AdequateModerate 60Details
Commit message · jrakibi

primitives: prevent null prevout in non-coinbase transactions

Non-coinbase transactions must not contain a null prevout (coinbase prevout)
Add this validation to the transaction decoder to reject txs
that violate this rule.

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 · Moderate 60/100

This commit adds a safety check in the Rust Bitcoin library's transaction decoder. It ensures that any transaction with more than one input cannot contain a 'coinbase-style' empty previous transaction reference. Such a reference is only allowed in the special coinbase transaction that creates new coins. Without this check, a malformed or malicious transaction could slip through and potentially cause incorrect behavior in software using this library to parse Bitcoin transactions.

AI review queuedFix decoder bug when ending before decoding prefixby Shing Him Ng · 5d4f9cff · Dec 26, 2025 · 1 fileMessage 73 · AdequateModerate 50Details
Commit message · Shing Him Ng

Fix decoder bug when ending before decoding prefix

Before this fix, calling `ByteVecDecoder.end()` on a decoder that hadn't
finished reading in the full prefix would result in a valid result of an
empty vec. This should instead result in an error, since the decoder
shouldn't be able to decode something with an incomplete prefix.

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

This commit fixes a bug in a Bitcoin data decoder. Previously, if you stopped decoding early—before the full length prefix was read—the decoder would incorrectly report success with an empty result instead of reporting an error. The fix makes the decoder correctly return an 'unexpected end of data' error in those cases. This could matter for anyone parsing Bitcoin protocol data from partial or truncated inputs.

AI review queuedMove internals::compact_size encoding into CompactSizeEncoderby Mitchell Bagot · 46097693 · Dec 22, 2025 · 15 filesMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Move internals::compact_size encoding into CompactSizeEncoder

Currently, the CompactSizeEncoder calls into the internals::compact_size
module to perform the actual encoding process. This creates a strange
API where there are two ways to do the same thing, one of which differs
from how other encoders work.

Move encode and encoded_size from internals::compact_size to the
CompactSizeEncoder in consensus_encoding. Replace usage of
compact_size::encode with pub(crate) function in primitives and
direct usage of CompactSizeEncoder in bitcoin.

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

This commit is a routine internal code cleanup. It moves the logic for encoding Bitcoin 'compact size' numbers from one internal helper module into a dedicated encoder type, then updates call sites to use the new location. There is no security fix or behavior change intended.

AI review queued2025-12-21 automated rustfmt nightlyby Fmt Bot · 1da9343f · Dec 21, 2025 · 8 filesMessage 45 · ThinInformational 15Details
Commit message · Fmt Bot

2025-12-21 automated rustfmt nightly

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

This commit is an automated code-formatting run by rustfmt. It only changes whitespace, line breaks, and the order of import statements. There are no functional changes, no bug fixes, and no security-related alterations.

AI review queuedMove `TxIdentifier` into `bip152`by rustaceanrob · b7f035be · Dec 17, 2025 · 2 filesMessage 58 · ThinInformational 18Details
Commit message · rustaceanrob

Move `TxIdentifier` into `bip152`

This trait was used in `bitcoin/blockdata/transaction` only to be used
in `p2p`. Move it to `p2p` and drop it in `bitcoin`. I opted to include
it directly in BIP-152, unless another context arises where these IDs
are interchangable.

Credit @mpbagot for the find

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 18/100

This commit is a simple code reorganization: a small trait called TxIdentifier that was defined in the main bitcoin crate is moved into the p2p crate's BIP-152 module because that was its only real user. There is no security fix, behavior change, or vulnerability being patched. It is purely a refactoring/cleanup change.

AI review queuedUse usize for CompactSizeDecoder and add new_with_limitby Mitchell Bagot · 4f80f6d0 · Dec 16, 2025 · 8 filesMessage 73 · AdequateLow 33Details
Commit message · Mitchell Bagot

Use usize for CompactSizeDecoder and add new_with_limit

Following the change to CompactSizeEncoder::new to take a usize, the
return types and ranges of the CompactSizeEncoder and CompactSizeDecoder
now differ. Since a device's memory addressing is limited to usize,
returning values beyond that size is largely useless. Converting the
CompactSizeDecoder to return a usize also allows the
cast_to_usize_if_valid function to be merged into the decoder itself.

Convert the CompactSizeDecoder to return a usize. Remove
cast_to_usize_if_valid and introduce its functionality into
CompactSizeDecoder.

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

This commit tightens how the library decodes Bitcoin 'compact size' length fields. Previously the decoder could return a 64-bit value, which callers then had to convert to a machine-usable size. Now the decoder itself returns a usize and enforces a configurable upper limit. This is a defensive hardening change: it reduces the chance that a maliciously large length value causes memory problems or integer-conversion bugs later on. It is not a fix for a known active exploit.

AI review queuedbip158: Remove doc_auto_cfgby Tobin C. Harding · 829c94aa · Dec 16, 2025 · 1 fileMessage 35 · OpaqueInformational 15Details
Commit message · Tobin C. Harding

bip158: Remove doc_auto_cfg

Quick before someone has a seizure.

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100

This commit removes a documentation-only Rust compiler feature flag (`doc_auto_cfg`) from the BIP-158 crate. It does not change any executable code, runtime behavior, or security-sensitive logic. The change only affects how documentation is generated on docs.rs and appears to be a cleanup because the feature was unstable or causing issues.

AI review queuedDepend on primitives instead of bitcoin where type definitions overlapby yancy · f81e8d27 · Dec 15, 2025 · 6 filesMessage 73 · AdequateInformational 18Details
Commit message · yancy

Depend on primitives instead of bitcoin where type definitions overlap

Consolidate dependency tree for downstream crates that depend on
`bitcoin-p2p-messages` and `bitcoin-primitives` by reducing overlapping
dependencies. For example, consider crate A which depends on
`bitcoin-primitives` and `bitcoin-p2p-messages`. For types defined by
both `bitcoin-primitives` and `bitcoin`, Crate A may still need to
depend on `bitcoin` if a type of `bitcoin-p2p-messages` depends on
`bitcoin` instead of `bitcoin-primitives`.

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 change is a routine internal dependency cleanup. The developers moved some shared type definitions (like block and transaction identifiers) from a large crate called `bitcoin` into a smaller, more focused crate called `bitcoin-primitives`. This reduces duplicate dependencies for downstream projects but does not change what the code does or fix any security problem.

AI review queuedFix push_slice() testsby Nadav Ivgi · 0b9094c6 · Dec 14, 2025 · 1 fileMessage 38 · OpaqueInformational 15Details
Commit message · Nadav Ivgi

Fix push_slice() tests

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

This commit only updates test code. It removes an 'ignore' marker from a test and corrects the expected byte output for pushing a single zero byte onto a Bitcoin script. There is no change to production code and no security issue.

AI review queuedExtend test coverage for BlockTimeby Mitchell Bagot · 781586d2 · Dec 14, 2025 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Extend test coverage for BlockTime

The encoding/decoding and serde functionality of BlockTime is not
covered in existing tests.

Add tests to cover BlockTimeEncoder, BlockTimeDecoder and the
Serialize + Deserialize traits on BlockTime.

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 15/100

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

AI review queuedp2p: Remove `OutPointExt` from `p2p` testsby rustaceanrob · 54fd0c72 · Dec 13, 2025 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · rustaceanrob

p2p: Remove `OutPointExt` from `p2p` tests

Remove a pointless dependency on `bitcoin` WRT #5331

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 15/100

This is a tiny test-only cleanup commit. It removes an unused trait import and replaces a helper constructor with direct struct field assignment in a unit test. There is no security relevance.

AI review queuedprimitives: Remove arrayvec dependencyby Mitchell Bagot · 9a11819a · Dec 13, 2025 · 4 filesMessage 70 · AdequateInformational 15Details
Commit message · Mitchell Bagot

primitives: Remove arrayvec dependency

The primitives crate only uses the arrayvec crate to provide Display
trait for Header. In order to minimise the use of external
dependencies, it should be removed.

Convert Header Display impl to use fmt_hex_exact! and remove the
arrayvec dependency from primitives.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 15/100

This commit removes a third-party dependency called `arrayvec` from the `bitcoin-primitives` crate and rewrites how a Bitcoin block header is printed as text. The old code built the text in a small fixed-size buffer from `arrayvec`; the new code streams the bytes directly through an internal formatting helper. There is no security fix here—this is a routine dependency-reduction refactor.

AI review queuedImplement Decodable for Witness using encoding crateby Tobin C. Harding · f9a6b37e · Dec 13, 2025 · 7 filesMessage 85 · StrongLow 34Details
Commit message · Tobin C. Harding

Implement Decodable for Witness using encoding crate

Use the new `consensus_encoding` crate (by way of the `io` crate
decoding function) to implement `consensus_decode` for `Witness`.

This is ugly a bit ugly because we remove equality derives from
`ParseError` so that we can jam the witness decoding error into it.
Cleanly it should not go there but in order to keep
`encode::deserialize_partial` working we have to be able to return
a `ParseError` and we have to be able to include the witness decoding
error if there was one.

Each subsequent time we try to do this in other decoding impls we will
add a new error variant each time also. This encoding code is on its
way out and will hopefully be deprecated soon so I believe this is ok.

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 · Low 34/100

This commit replaces the custom code that reads Bitcoin transaction witness data with a shared decoding library. The change is described by the author as a temporary, slightly messy refactor to keep an older decoding helper working while a newer encoding crate takes over. There is no claim in the commit that this fixes a security bug, and the diff itself is mostly code removal and plumbing a new error variant. It could affect how malformed witness data is rejected, but no vulnerability is disclosed or demonstrated.

AI review queuedPut feature gate below test attributeby Tobin C. Harding · a7aa12be · Dec 11, 2025 · 5 filesMessage 78 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Put feature gate below test attribute

For the stabilising crates we would like test code to be treated as a
first class citizen. In my book that means clean AF.

Audit all three crates and put the feature gate attributes below the
`#[test]` attribute, based on two things:

- Uniformity (we could arbitrarily pick either)
- Puts the more important thing closer to the code (subjective)

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only reorders two Rust attribute lines on test functions so that #[test] appears above #[cfg(...)]. It is a cosmetic/style change with no effect on what code runs or how it behaves. There is no security relevance.

AI review queuedbitcoin: Update changelogby Tobin C. Harding · 2714cded · Dec 11, 2025 · 1 fileMessage 35 · OpaqueInformational 15Details
Commit message · Tobin C. Harding

bitcoin: Update changelog

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 edits the project's changelog file. It documents an upcoming beta release and lists previously merged changes, but it does not modify any source code, build scripts, tests, or configuration. There is no security-relevant code change to review.

AI review queuedp2p: remove HeaderExt dependencyby Nick Johnson · 6710addb · Dec 10, 2025 · 1 fileMessage 68 · AdequateLow 34Details
Commit message · Nick Johnson

p2p: remove HeaderExt dependency

This breaks another link between p2p and bitcoin. p2p is focused on
encoding and decoding the peer to peer messages, the POW verification
should probably be on the caller or perhaps added to the primitives
decoder.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
defensive validationsecond-pass: broader security terminology
AI analysis · Low 34/100

This commit removes a helper function called all_targets_satisfied() from the peer-to-peer (p2p) message handling code. That function used to check whether every block header in a 'headers' message met its own proof-of-work target. The change is framed by the author as a design cleanup: p2p should only encode and decode network messages, while proof-of-work verification should be done by the caller or elsewhere. Removing the check does not by itself create a vulnerability, but any downstream code that relied on this function to validate headers before processing them will no longer get that protection automatically. Those callers must now add the check themselves or use a different validation path.

AI review queuedCreate empty `bip158` crateby rustaceanrob · 47fe9ceb · Dec 9, 2025 · 9 filesMessage 66 · AdequateInformational 15Details
Commit message · rustaceanrob

Create empty `bip158` crate

As part of discussion #5331, `bitcoin` and `p2p` should not depend on
each other and should instead mutually depend on `primitives`, `hashes`,
etc. BIP-158 is the last remaining dependency from `bitcoin` in `p2p`.
It is not a peer-to-peer specification, but it is not an essential
module of `bitcoin` either. IMO the best option for this module is to
release it as a crate.

Blaming the module shows it hasn't changed architecturally in many
years. A new crate would allow us to revisit some of the design choices.
Particularly the heavy use of generics, dependency on `io` that could be
replaced by `consensus_encoding`, and perhaps an opportunity to benchmark
and improve performance.

66/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 15/100

This commit creates a brand-new, empty placeholder crate named `bitcoin-bip158` inside the rust-bitcoin workspace. It adds no executable code, no network functionality, no cryptographic operations, and no changes to existing crates. It is purely a project-structure refactor in preparation for later moving BIP-158 filter code out of the main `bitcoin` crate.

AI review queuedunits: Set TBD in deprecate attributeby Tobin C. Harding · d19605cb · Dec 7, 2025 · 7 filesMessage 60 · AdequateInformational 15Details
Commit message · Tobin C. Harding

units: Set TBD in deprecate attribute

Set deprecation version. Note that this deprecated code is only going
to exist until the do the non-RC i.e., `v1.0.0`.

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

This commit only fills in placeholder version numbers in deprecation warnings. It does not change any program behavior, fix any bug, or introduce any security issue. It is a routine documentation/metadata cleanup.

AI review queuedinternals: Bump version to 0.4.2by Tobin C. Harding · fb65b565 · Dec 7, 2025 · 5 filesMessage 80 · StrongInformational 15Details
Commit message · Tobin C. Harding

internals: Bump version to 0.4.2

In preparation for releasing `bitcoin 0.33.0-rc.0` we need do a point
release of `internals` because we move a macro out of `bitcoin` and
over to `internals`. No other crate uses the macro so far so we only
update the `bitcoin` dependency on `internals` leaving the other
crates as they are.

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
second-pass: broader security terminology
AI analysis · Informational 15/100

This commit is a routine version bump for an internal helper crate. It only changes version numbers in package files and a changelog entry. There is no code change, no bug fix, and no security issue visible in the diff.

AI review queuedp2p: remove io::Error from exposed APIby Nick Johnson · 213b85ba · Dec 6, 2025 · 1 fileMessage 60 · AdequateInformational 19Details
Commit message · Nick Johnson

p2p: remove io::Error from exposed API

Part of the process to break the p2p package's dependency on the old io
package.

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 19/100

This commit refactors how Bitcoin peer-to-peer address errors are reported. It replaces a generic input/output error type with a new, more specific error type for addresses that cannot be routed over normal internet connections (Tor, I2P, CJDNS). This is a routine API cleanup, not a fix for an active security vulnerability. The change is part of a larger effort to reduce internal package dependencies.

AI review queuedMove `bip152` to `p2p`by rustaceanrob · e242dd61 · Dec 6, 2025 · 6 filesMessage 59 · ThinInformational 15Details
Commit message · rustaceanrob

Move `bip152` to `p2p`

Replaces #5333 and #5365 by manually implementing `hex` traits for the
`ShortId` type. `FromStr` is omitted as I can't think of a logical use
of that method in this context. To make any sense of a string short ID
one needs the nonce and siphash keys.

This module is only relevant between two peers relaying blocks over the
peer to peer network.

59/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
signing or wallet pathsecond-pass: broader security terminologysecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a routine code reorganization: it moves the BIP-152 compact block module from the main `bitcoin` crate into the `p2p` crate, where it is more logically located. It also replaces an auto-derived hex formatting trait with a manual implementation for one small type. There is no security fix or behavior change visible in the diff.

AI review queuedRemove alloc feature gate for MerkleNodeby Mitchell Bagot · cc0da4ca · Dec 5, 2025 · 6 filesMessage 68 · AdequateInformational 18Details
Commit message · Mitchell Bagot

Remove alloc feature gate for MerkleNode

With the introduction of pop() to ArrayVec, the
MerkleNode::calculate_root logic can be implemented using ArrayVec
instead of alloc::vec::Vec. This allows for use of the functionality
without the alloc feature, albeit with an upper iterator length bound
of 32768 (exclusive) items.

Introduce ArrayVec bounded variation to MerkleNode::calculate_root,
and remove alloc feature gate from MerkleNode.

68/100 · AdequateMessage clarity
✓ 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 18/100

This commit refactors how Bitcoin Merkle roots are calculated so the code works even when the optional 'alloc' memory allocator feature is disabled. It replaces a growable vector with a fixed-size 15-slot array, which limits the no-alloc path to at most 32,767 transaction IDs. The change is a feature-portability improvement, not a security fix, and it explicitly returns None when the no-alloc limit is exceeded rather than overflowing or corrupting state.