RB
← All projectsRust Bitcoin

rust-bitcoin

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

BitcoinCryptographic librariesNormal
Repository coverage

2313 commits in the local evidence base

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

532security candidates511second-pass queue2206AI analyses
130commits · 30 days
248commits · 60 days
1102commits · 180 days
2009commits · 365 days
Backfill bands
Aug 5 → Feb 6787 seen32 candidatesComplete
Feb 6 → Jun 6878 seen53 candidatesComplete
Jun 6 → Jul 6211 seen15 candidatesComplete
Jul 6 → Aug 5184 seen2 candidatesComplete
Commit communication

Does the history explain itself?

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

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

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
Andrew Poelstra23179157290
Mitchell Bagot649193645068
Tobin C. Harding41566410063
jrakibi944994068
Nick Johnson19121190060
Jamil Lambert, PhD11919116061
satsfy (Renato Britto)381527066
Fmt Bot331431045
Trevor Arjeski111111069
Shing Him Ng31731056
Martin Habovstiak30628068
Ismail Daif22622050
Analysis record

Published AI watches

Last scanned 54 minutes ago

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Fix integer overflow in `get_array`

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

build(deps): bump dtolnay/rust-toolchain

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

a31e0b0eby dependabot[bot]+8−84 files
No security note in commit
Repository ledger

Explore captured commits

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

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

Lower-prioritySplit Network functionality into extension traitby Mitchell Bagot · dd528968 · Jan 19, 2026 · 2 filesMessage 68 · AdequateInformational 18Details
Commit message · Mitchell Bagot

Split Network functionality into extension trait

Since bitcoin-network-kind now exists, we can use its types in bitcoin.
However, some functionality on the current bitcoin Network type doesn't
(and can't) exist in the bitcoin-network-kind crate.

Split functionality of Network into NetworkExt extension trait.

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

This commit is a routine internal code reorganization. It moves some methods of the Network type into a new 'extension trait' called NetworkExt so that the underlying Network type can be shared with a separate crate. There is no security bug being fixed here.

Lower-priorityUnify docstrings for Target::MAX_ATTAINABLE_* constsby Mitchell Bagot · 5b637bdf · Jan 19, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Unify docstrings for Target::MAX_ATTAINABLE_* consts

The docstrings on the MAX_ATTAINABLE constants on Target differ from
one another. These should use the same wording to clarify that they
are they represent the same value on different networks.

Adjust docstring wording for Target::MAX_ATTAINABLE_* constants.

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

This commit only changes documentation comments (docstrings) for several constants in the proof-of-work module. It rewords descriptions to be consistent and adds a source reference comment. No code behavior is altered, so there is no security impact.

Lower-priorityInclude Core link in U256::inverse()by Mitchell Bagot · f0ed5c68 · Jan 19, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Include Core link in U256::inverse()

The docstring in Core outlines an algorithm from Core for computing
the inverse of a 256-bit integer, which is ultimately used to convert
Work to/from Target.

Introduce a permalink to the code in Core on Github.

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

This commit only adds a documentation link inside a code comment. It does not change any executable code, behavior, or security properties of the library.

Security candidateIntroduce wrapper type for secp256k1::Keypairby Mitchell Bagot · 3ef1b4c0 · Jan 18, 2026 · 8 filesMessage 80 · StrongInformational 19Details
Commit message · Mitchell Bagot

Introduce wrapper type for secp256k1::Keypair

In order to solve the issue of imprecise types for from_str in
XOnlyPublicKey, a wrapper type was implemented, alongside a
corresponding narrow error type. This solution provided a cleaner
interface for such keys, in place of directly working with secp256k1
XOnlyPublicKey types.

Add a wrapper type for secp256k1::Keypair to tighten the error type
for from_str.

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

This commit is a routine code-quality refactor in the rust-bitcoin library. It wraps the underlying secp256k1 keypair type in a library-specific type so that parsing errors can be more precise and the public API is cleaner. There is no direct security vulnerability being fixed here; it is an interface improvement that may indirectly reduce misuse.

AI review queuedAdd *_with_max difficulty functions to Targetby Mitchell Bagot · 2f842a42 · Jan 18, 2026 · 1 fileMessage 80 · StrongInformational 15Details
Commit message · Mitchell Bagot

Add *_with_max difficulty functions to Target

In order to allow for Target to eventually be moved out from bitcoin,
it is necessary to decouple its dependency on Params. Since it is
necessary to maintain the difficulty calculations, a Params-less
variation is introduced that the original version can call through to.

Add difficulty_with_max and difficulty_float_with_max to Target.

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 code refactoring in the rust-bitcoin library. It splits the existing difficulty calculation functions into two parts: new lower-level functions that take a maximum target directly, and the original functions that now simply call these new functions with the network's maximum target from the Params object. There is no change to the actual math or behavior, and no security issue is present.

AI review queued2026-01-18 automated rustfmt nightlyby Fmt Bot · 4d2fee84 · Jan 18, 2026 · 4 filesMessage 45 · ThinInformational 15Details
Commit message · Fmt Bot

2026-01-18 automated rustfmt nightly

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

This commit is purely a code-formatting cleanup performed by an automated rustfmt nightly run. It reorders imports, collapses or expands brace blocks, removes blank lines, and adjusts line wrapping in test code. No program logic, APIs, or security behavior was changed.

Lower-priorityconsensus_encoding: implement batched allocation for WitnessDecoderby jrakibi · 9bf15224 · Jan 17, 2026 · 2 filesMessage 91 · StrongModerate 59Details
Commit message · jrakibi

consensus_encoding: implement batched allocation for WitnessDecoder

Add batched allocation for witness decoder (data area).
we allocate incrementally in ~1 MB batches

The current approach automatically addresses the concerns raised in
https://github.com/rust-bitcoin/rust-bitcoin/issues/5258 and
https://github.com/rust-bitcoin/rust-bitcoin/pull/5239#discussion_r2501954586

we aslo add `reserve_batch` to Mutants exclusion list
Mutation testing in WitnessDecoder::reserve_batch changes the buffer
allocation logic and they cause infinite loop during decoding

91/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
AI analysis · Moderate 59/100

This commit changes how the Rust Bitcoin library allocates memory when decoding a Bitcoin transaction's witness data. Previously, the decoder could double its internal buffer repeatedly, which could let a maliciously crafted transaction force the program to reserve far more memory than needed. The new code allocates memory in roughly 1 MB batches instead. A test was added that feeds a 4,000,000-element witness and checks that the decoder only allocates about 16–17.5 MB, suggesting the change is meant to limit memory blow-up during decoding.

Security candidateprimitives: reject txs with output sum > MAX_MONEYby jrakibi · 8973524e · Jan 17, 2026 · 1 fileMessage 73 · AdequateModerate 68Details
Commit message · jrakibi

primitives: reject txs with output sum > MAX_MONEY

Rejects transactions with total output value exceeds MAX_MONEY
See CVE-2010-5139

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Links an issue, advisory, or supporting reference
Why it was queued
explicit security languagesigning or wallet path
AI analysis · Moderate 68/100

This commit adds a safety check in the rust-bitcoin library so that when a Bitcoin transaction is being decoded, the total value of all its outputs is rejected if it exceeds the protocol's maximum allowed money supply (MAX_MONEY). This prevents a class of bugs similar to the historic Bitcoin Core overflow issue referenced as CVE-2010-5139, where an impossibly large total output value could cause incorrect accounting or consensus problems.

AI review queuedprimitives: test individual output value > MAX_MONEYby jrakibi · 75682547 · Jan 17, 2026 · 1 fileMessage 60 · AdequateInformational 12Details
Commit message · jrakibi

primitives: test individual output value > MAX_MONEY

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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 12/100

This commit only adds a new unit test. It does not change any production code. The test checks that the library rejects Bitcoin transactions containing an output worth more than the maximum allowed money (MAX_MONEY + 1 satoshi) when decoding transaction bytes. It is a defensive test borrowed from Bitcoin Core's test data, confirming an existing safety check works.

Lower-priorityconsensus_encoding: Add flush_to_* functionsby Mitchell Bagot · a6c3d826 · Jan 17, 2026 · 4 filesMessage 70 · AdequateInformational 16Details
Commit message · Mitchell Bagot

consensus_encoding: Add flush_to_* functions

In some cases, we may have an Encoder instance, without a corresponding
Encodable. In these cases, having a way to flush the encoder to a vec
and/or a writer is extremely useful to avoid having to replicate
the logic of encode_to_* functions.

Add flush_to_vec and flush_to_writer functionality to flush Encoder
objects to a Vec or Write instance respectively.

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

This commit adds two new helper functions, flush_to_vec and flush_to_writer, to the consensus_encoding crate. They let developers take an already-created Encoder object and write its contents out to a Vec or an I/O writer. The existing encode_to_vec and encode_to_writer functions are refactored to use these new helpers internally. There is no security fix or behavior change; it is a straightforward API convenience addition.

Lower-priorityprimitives: Remove BlockTime decoder from root exportby Mitchell Bagot · 825cdf24 · Jan 16, 2026 · 6 filesMessage 73 · AdequateInformational 19Details
Commit message · Mitchell Bagot

primitives: Remove BlockTime decoder from root export

Encoder/decoder types are not typically exported outside of the module
they are defined in. The BlockTimeDecoder (and associated error) are.

Remove BlockTimeDecoder and BlockTimeDecoderError from root level
re-export in primitives.

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

This commit is a routine cleanup of the public API surface. It stops two internal helper types (BlockTimeDecoder and BlockTimeDecoderError) from being exported at the top level of the primitives crate. The types are still available inside the crate where they are actually used. There is no security fix here and no indication of a vulnerability.

Lower-priorityp2p: Switch to network crateby rustaceanrob · 7e9df7ab · Jan 16, 2026 · 5 filesMessage 60 · AdequateInformational 15Details
Commit message · rustaceanrob

p2p: Switch to network crate

Introduce `bitcoin-network-kind` to `p2p` and update lock files.

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

This is a routine internal code reorganization. The project is moving the definition of Bitcoin network types (like Mainnet and Testnet) from one internal crate to another newly created crate. There are no user-facing behavior changes and no security fixes.

Lower-priorityp2p: Remove `Magic::from_params` methodby rustaceanrob · 16cc484e · Jan 16, 2026 · 1 fileMessage 68 · AdequateInformational 16Details
Commit message · rustaceanrob

p2p: Remove `Magic::from_params` method

`Params` is not included in the `bitcoin-network-kind` crate, and this
method simply takes the network from params and calls `ok`. This
conversion is already possible via `TryFrom`.

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

This commit removes a small helper method called Magic::from_params from the rust-bitcoin peer-to-peer networking crate. The method was redundant because the same conversion can already be done through Rust's standard TryFrom trait. There is no security issue here; it is a routine code cleanup to simplify the public API and remove an unnecessary dependency on the Params type.

Lower-priorityunits: Correctly group depsby Tobin C. Harding · 31ba7e78 · Jan 15, 2026 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Tobin C. Harding

units: Correctly group deps

By convention optional dependencies are grouped separately to
non-option ones.

50/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit is purely cosmetic: it moves one optional dependency line down by one position in a configuration file so that optional and non-optional dependencies are grouped separately, matching project convention. There is no code change and no security relevance.

Lower-priorityAdd tests for error variantsby Jamil Lambert, PhD · 32de2578 · Jan 15, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Jamil Lambert, PhD

Add tests for error variants

Test the error variants without checking the error message string
contents.

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 error types in the Bitcoin block primitives. It does not change any production code, fix bugs, or alter behavior. There is no security issue here.

Lower-priorityAdd test non coinbase returns noneby Jamil Lambert, PhD · 110bbca8 · Jan 15, 2026 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Jamil Lambert, PhD

Add test non coinbase returns none

Add a test that witness_commitment_from_coinbase() returns none when
the transaction is not a coinbase transaction.

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 automated test to verify that a helper function correctly returns nothing when given a non-coinbase Bitcoin transaction. It does not change any production code, fix a bug, or alter behavior. It is purely a test addition.

Lower-priorityAdd tests of version encoder and decoderby Jamil Lambert, PhD · d2491028 · Jan 15, 2026 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · Jamil Lambert, PhD

Add tests of version encoder and decoder

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

This commit only adds new unit tests for encoding and decoding the block version number. It does not change any production code, so it cannot introduce or fix a security vulnerability on its own.

Lower-priorityExtend from_hex and from_unprefixed_hex docsby Mitchell Bagot · cb891294 · Jan 15, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Extend from_hex and from_unprefixed_hex docs

The docs for from_hex and from_unprefixed_hex don't include details
on the potential errors that can be thrown. In stabilising crates,
this information is required in docstrings.

Add error information to from_hex and from_unprefixed_hex docs.

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

This commit only updates documentation comments for two existing Rust functions, adding details about what errors they can return. No code logic was changed, so it cannot introduce or fix a security vulnerability on its own.

Security candidateFix clippy warningsby Tobin C. Harding · 6bb4c7e3 · Jan 15, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Fix clippy warnings

Fix two unused warnings, one because of incorrect feature gating in
test code and the other because `super::*` is not needed.

73/100 · AdequateMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100

This commit only fixes compiler lint warnings. It adds missing feature gates so that test code compiles cleanly when certain optional features are enabled or disabled. There is no change to runtime behavior, no bug fix, and no security relevance.

Security candidatebitcoin: Add default arg to bip32 exampleby Tobin C. Harding · 84522b1b · Jan 15, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Tobin C. Harding

bitcoin: Add default arg to bip32 example

Use the documented seed hex string as default if one is not passed in.
This allows us to simplify running this example in CI.

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

This change only touches an example program used for documentation and CI demos. It makes the seed argument optional by providing a documented default value. There is no security issue: the seed was already hard-coded in the comment above, and the example is not production code.

Lower-priorityAdd tests for Block and Header read limitsby Jamil Lambert, PhD · 5504035b · Jan 14, 2026 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · Jamil Lambert, PhD

Add tests for Block and Header read limits

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

This commit only adds two new automated tests that check how many bytes a Block and Header decoder expects to read. It does not change any production code, fix a bug, or alter behavior. There is no security issue here.