RB
← All projectsRust Bitcoin

rust-bitcoin

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

BitcoinCryptographic librariesNormal
Repository coverage

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

531security candidates510second-pass queue2204AI analyses
136commits · 30 days
269commits · 60 days
1108commits · 180 days
2034commits · 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
507Strong · 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 Poelstra22878155290
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 47 minutes ago

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

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

This is a routine patch-version update of a GitHub-maintained action used only to upload static-analysis results (SARIF files) from a scheduled CI job. The change does not touch any project source code, cryptographic logic, or user-facing …

2cac6e38by 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 astral-sh/setup-uv from 8.3.2 to 9.0.0

This is a routine automated update by Dependabot that changes the pinned version of a GitHub Action used to install a Python tool called 'uv' in two workflow files. The new version is a major release of the setup-uv action itself, but the …

7d7e7269by dependabot[bot]+2−22 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-priorityconsensus_encoding: Improve the API testby Tobin C. Harding · be9ffcab · Apr 14, 2026 · 1 fileMessage 98 · StrongInformational 15Details
Commit message · Tobin C. Harding

consensus_encoding: Improve the API test

Go over the API test and try to ensure it covers what we want it to.
Remember these API test files are mainly so devs can look and check
stuff is sane. They also kind of work as documentation of what the
crate does all in one spot.

I did this by looking at the state of `units/test/api.rs` after #5890
is applied (because I reworked that file the other day).

98/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 15/100

This commit only expands the project's API test file. It adds more type coverage, checks that Debug output is non-empty for additional types, and verifies that a specific error type can be formatted with Debug. There are no changes to production code, no bug fixes, and no security-related behavior changes.

Lower-priorityprimitives: enable batched merkle root computation for x86by jrakibi · 0c8ba186 · Apr 14, 2026 · 1 fileMessage 50 · ThinInformational 18Details
Commit message · jrakibi

primitives: enable batched merkle root computation for x86

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

This commit simply turns on an existing, faster way to calculate Merkle roots for Intel/AMD (x86 and x86_64) processors. It was already enabled for ARM64 (aarch64). There is no security bug here; it is a small performance portability change.

Lower-priorityFix AddrV1 Decoder Typographical Errorby Abeeujah · d059b9f0 · Apr 14, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Abeeujah

Fix AddrV1 Decoder Typographical Error

Correct the doc comment on `AddrV1MessageDecoder` to correctly mention
the `AddrV1Message` as the message format it is decoding, and not
`AddrV2Message`.

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

This commit fixes a typo in a code comment. The comment on the decoder for the older Bitcoin address message format (AddrV1) incorrectly said it decoded the newer format (AddrV2). No actual code behavior changed.

Security candidatehashes: Add x86 SHA-NI 2-way dispatchby jrakibi · 3be195ed · Apr 14, 2026 · 1 fileMessage 60 · AdequateInformational 17Details
Commit message · jrakibi

hashes: Add x86 SHA-NI 2-way dispatch

add x86 sha256d dispatcher that handles 2-way SHA-NI

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

This commit adds a performance optimization for computing SHA-256 hashes on modern x86 processors that support Intel's SHA-NI instructions. It is a routine feature addition, not a security fix or vulnerability patch. There is no indication in the commit that it addresses a security issue.

Security candidatehashes: Interleave two independent hashes for 2-way SHA256dby jrakibi · ee04a988 · Apr 14, 2026 · 1 fileMessage 81 · StrongInformational 17Details
Commit message · jrakibi

hashes: Interleave two independent hashes for 2-way SHA256d

x86 SHA256RNDS2 has a latency of 4-6 cycles depending on
the architecture. the next SHA256RNDS2 needs the result
of the current one as input, so the CPU has to wait until
it is ready. (in Transform 2 for eg, 3 out of every 4 cycles
are wasted doing nothing)

we fill those wasted cycles by computing a second independent
hash alongside the first.

See https://uops.info/html-instr/SHA256RNDS2_XMM_XMM.html
for per-architecture instruction timings.

81/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
cryptography-sensitive path
AI analysis · Informational 17/100

This commit is a performance optimization for Bitcoin's double-SHA256 hashing on modern Intel/AMD processors. It rewrites a low-level routine to compute two independent hashes at once instead of one, filling idle CPU cycles. There is no security fix or vulnerability here; it is purely about making hashing faster.

Security candidatehashes: Add optimized x86 SHA256d for 64-byte inputby jrakibi · 301c3c16 · Apr 14, 2026 · 1 fileMessage 73 · AdequateInformational 16Details
Commit message · jrakibi

hashes: Add optimized x86 SHA256d for 64-byte input

Add the three transforms for sha256(sha256(64_bytes))

Transforms 1 & 2 compute the inner `sha256(64_bytes)`:

- Transform 1: 64 bytes already fill the first block (no
padding needed), so we apply 64 rounds of compression
normally.
After T1: state = state + initial_state

- Transform 2: the message schedule at this step is constant
and known in advance (64 bytes of padding), so
we precompute W[i]+K[i] into MIDS and apply 64 rounds (no
schedule expansion needed).
After T2: state = state + saved_T1_state
(state now contains sha256(64_bytes))

- Transform 3 computes sha256(output_of_T1_and_T2). The output
is 32 bytes, so the block contains 32 bytes and the rest is
known padding. we precompute the message
schedule for the known words (w8-w15) into FINS.
After T3: state = state + initial_state
(state now contains the sha256d result)

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive path
AI analysis · Informational 16/100

This commit adds a new, highly optimized x86 implementation for computing a double SHA-256 hash (SHA256d) on exactly 64 bytes of input. It uses Intel SHA-NI CPU instructions and precomputed constants to speed up Bitcoin-style hashing. There is no indication in the commit that this fixes a security bug; it appears to be a pure performance optimization.

Lower-priorityunits: Re-export all the public amount error typesby Tobin C. Harding · e2eed8e3 · Apr 14, 2026 · 1 fileMessage 83 · StrongInformational 16Details
Commit message · Tobin C. Harding

units: Re-export all the public amount error types

Now we are adding `errors` modules everywhere we can re-export all the
public `units::amount` errors. In times past we were only going to
re-export the non-niche ones.

This will be tested later in `tests/api.rs`.

83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 16/100

This change is purely about making more error types publicly visible from a Rust library module. It does not alter how errors are handled, what data they contain, or any runtime behavior. There is no security issue here.

Lower-priorityunits: Correctly feature guard test codeby Tobin C. Harding · 02dfb738 · Apr 14, 2026 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · Tobin C. Harding

units: Correctly feature guard test code

We recently started putting all cfg's on their own lines. Fix one we
missed and also feature gate the `ToString` import correctly clearing
a lint warning when building with `alloc` feature.

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 is a minor code-quality fix inside test code only. It adjusts the way optional Rust features are checked so that a warning about an unused import disappears when building with only the 'alloc' feature. There is no change to production code and no security impact.

Security candidatehashes: use consistent round numbering in sha256 commentsby jrakibi · d61ad42b · Apr 14, 2026 · 2 filesMessage 73 · AdequateInformational 15Details
Commit message · jrakibi

hashes: use consistent round numbering in sha256 comments

the 1-way ARM and x86 SHA-NI functions used Rounds 0-3 while
the 2-way ARM function used Rounds 1-4 in the comments.

use this consistently everywhere.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100

This commit only changes comments in the SHA-256 code. It makes the round-number labels consistent between different versions of the optimized code (ARM and x86). No actual program logic, math, or behavior was changed, so it cannot affect security or correctness.

Lower-priorityExtend bitcoin re-export script to check against unitsby Mitchell Bagot · 87ff6899 · Apr 14, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Extend bitcoin re-export script to check against units

The public API files generated by the rbmt check-api command provide a
semi-comprehensive set of public API interfaces. In the case where a
crate pub use's a module from another crate (e.g. primitives pub use
units::absolute), the API files fail to outline all content within the
re-exported module. As such, checking bitcoin's re-exports against
primitives alone, misses many type re-exports.

Adjust bitcoin re-export check script to also compare against units.

Assisted by Claude Opus 4.6

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

This commit updates an internal shell script used to generate a Rust test file. The script now also checks that types from a secondary crate (`bitcoin_units`) are correctly re-exported by the main `bitcoin` crate. It is a build/test tooling improvement with no runtime code changes and no security relevance.

Lower-priorityFix re-exports in bitcoin crateby Mitchell Bagot · e037b016 · Apr 14, 2026 · 2 filesMessage 80 · StrongInformational 19Details
Commit message · Mitchell Bagot

Fix re-exports in bitcoin crate

Due to existing limitations in the re-export check scripts, various
types are not correctly re-exported in bitcoin to satisfy the
bitcoin -> primitives -> units superset requirements.

Re-export all missing types from bitcoin.

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

This commit adds missing public re-exports of types in the main 'bitcoin' crate so that downstream users can access types that are defined in the internal 'primitives' and 'units' crates. It is a build/API consistency fix with no apparent security relevance.

Lower-priorityp2p: add encoding trait impls to NetworkMessage encoder/decoderby Peter Zafonte · e8b7b15f · Apr 13, 2026 · 1 fileMessage 73 · AdequateLow 32Details
Commit message · Peter Zafonte

p2p: add encoding trait impls to NetworkMessage encoder/decoder

Migrate NetworkMessageEncoder and NetworkMessageDecoder to use
encoding::encode_to_vec and encoding::decode_from_slice rather than
the legacy bitcoin::consensus::encode traits.

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

This commit refactors how Bitcoin peer-to-peer network messages are encoded and decoded in the rust-bitcoin library. It replaces an older, all-in-one encoding approach with a new system that delegates encoding and decoding to type-specific handlers for each message variant. The change is primarily a code modernization and cleanup, but because it touches the core message parsing path, any mistake could affect how the library reads untrusted network data. The diff itself does not appear to introduce an obvious vulnerability, but it is a large rewrite of security-sensitive parsing code, so the risk comes from potential subtle behavioral differences rather than from a clear bug.

Security candidatehashes: split sha256 crypto int separate filesby jrakibi · a8e9c992 · Apr 13, 2026 · 4 filesMessage 68 · AdequateInformational 15Details
Commit message · jrakibi

hashes: split sha256 crypto int separate files

since we are planning to add other implementations for sha256: 2-way x86 shani,
4-way SSE4.1, and 8-way AVX2, we can split the file so each impl has
its own file without ending up with a single file that is hard to review.

this is just a code move, no logic change.

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

This commit is a pure file reorganization: it splits the SHA-256 cryptographic implementation from one large file into several smaller files. The author explicitly states there is no logic change, and the diff shows the same code being moved into new modules with only minor module wiring adjustments. There is no security-relevant change visible.

Lower-priorityunits: Remove TODO commentby Tobin C. Harding · 09e5f96b · Apr 13, 2026 · 1 fileMessage 46 · ThinInformational 15Details
Commit message · Tobin C. Harding

units: Remove TODO comment

There is a fix in flight for this: #5875

In case we don't want to wait for that PR to land just remove the
comment so the pre-release job on #5891 can pass.

46/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference! Contains work-in-progress language
AI analysis · Informational 15/100

This commit simply removes a two-line TODO comment from a test file. No code behavior changes, no security fix or vulnerability is introduced.

Lower-priorityunits: Update serde_regression testby Mitchell Bagot · e7ae1cf1 · Apr 12, 2026 · 2 filesMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

units: Update serde_regression test

The serde_regression test case in units checks for regressions in the
serde implementations for any type in units. Since new types with serde
impls have been added to units, this needs to be updated.

Update the serde_bincode and add Target and Work to Serde test struct
in serde tests.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 15/100

This commit only updates a test file. It adds two new data types (Target and Work) to an existing regression test that checks serialization behavior, and refreshes a binary test fixture. There is no change to production code, no security fix, and no vulnerability.

Lower-priorityMove pow serde tests to unitsby Mitchell Bagot · 9b3c4e0f · Apr 12, 2026 · 3 filesMessage 90 · StrongInformational 15Details
Commit message · Mitchell Bagot

Move pow serde tests to units

The serde regression tests for the proof-of-work types can and should
be moved to units, rather than remaining in bitcoin to ensure that any
regressions are caught by the crate that implements the serde traits,
rather than one that uses them.

Move regression_target and regression_work serde tests to units, moving
the u256 bincode as necessary.

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
fuzzing or regression evidence
AI analysis · Informational 15/100

This commit simply moves existing proof-of-work serialization tests from one crate to another within the same project. It is a test-only refactoring change with no effect on production code or security.

AI review queuedReplace upper-lower serde parsing with from_unprefixed_hexby Mitchell Bagot · c88813fc · Apr 12, 2026 · 1 fileMessage 73 · AdequateInformational 18Details
Commit message · Mitchell Bagot

Replace upper-lower serde parsing with from_unprefixed_hex

The U256 deserialize impl manually splits a known-length string into
upper and lower halves. While this functions correctly, it's a
needless reimplementation of the same functionality present in U256's
from_unprefixed_hex function.

Replace the manual split of strings in U256 deserialize with a single
call to from_unprefixed_hex.

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

This commit is a small internal cleanup in the Rust Bitcoin library. It replaces a hand-written string-splitting routine used when reading a 256-bit number from hexadecimal text with a call to an existing library function that does the same thing. There is no indication this fixes a bug or security issue; it is a code-quality refactor.

AI review queuedRemove visit_bytes from U256 serdeby Mitchell Bagot · bc4d525f · Apr 12, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Remove visit_bytes from U256 serde

In the U256 serde deserialize impl, a visitor struct is defined inside
of an is_human_readable() condition block. The visit_bytes function in
this block can't be used as the data is human readable and thus only
visit_str is required. As such, it can be removed.

Remove the visit_bytes from the HexVisitor struct in U256 Deserialize.

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

This commit removes an unused code path from the deserialization logic for a 256-bit unsigned integer type (U256). The removed code handled raw byte input, but the surrounding code only ever asks serde for a string, so the byte path was unreachable. There is no security fix here—just cleanup of dead code.

Lower-priorityMove U256 serde impl to units powby Mitchell Bagot · 25105d5c · Apr 12, 2026 · 3 filesMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Move U256 serde impl to units pow

The U256 serde traits are only used to derive serde traits for Target
and Work. Since these types are defined in units, the serde trait impls
should thus also live only in units.

Move U256 serde traits from u256.rs to pow.rs in units.

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

This commit is a simple code relocation: it moves the implementation of serde serialization/deserialization for the U256 type from one internal file to another, without changing the actual logic. There is no security-relevant change visible in the diff.

AI review queuedUpdate API filesby Mitchell Bagot · 257a81d9 · Apr 12, 2026 · 3 filesMessage 28 · OpaqueInformational 15Details
Commit message · Mitchell Bagot

Update 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 generated API snapshot text files. It removes lines that listed certain error-conversion functions from the public API surface, but it does not change any actual Rust source code, logic, or behavior. There is no security issue visible in this commit.

Lower-priorityunits: Remove From impls for Un/PrefixedHexError typesby Mitchell Bagot · e591d1f5 · Apr 12, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

units: Remove From impls for Un/PrefixedHexError types

The remaining From conversion trait impls for the hex errors can be
removed now that the relevant hex functionality is all co-located in
units and doesn't require any public API surface.

Remove all From impls for UnprefixedHexError and PrefixedHexError.

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

This commit is a routine internal cleanup in the Rust Bitcoin library. It removes some automatic error-conversion shortcuts (called `From` implementations) for two hex-parsing error types and replaces their uses with explicit constructor calls. There is no indication this fixes a security bug; it is a normal API-refactoring change.

Lower-priorityunits: Add U256 hex parsing as pub(crate) in parse_intby Mitchell Bagot · 0dc97215 · Apr 12, 2026 · 3 filesMessage 73 · AdequateInformational 12Details
Commit message · Mitchell Bagot

units: Add U256 hex parsing as pub(crate) in parse_int

By moving u256 hex parsing into parse_int, the error types can be kept
simple and self-contained, at the expense of requiring parts of U256
to be made pub(crate). Since both cases produce no changes to the
public API, the introduction of u256 parsing in parse_int is better,
since it keeps functionality together.

Make U256 constructors and struct pub(crate).
Introduce pub(crate) u256 parsing in parse_int.

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

This commit is a code cleanup inside the rust-bitcoin library. It moves the logic for parsing 256-bit unsigned integers from hexadecimal strings into a shared internal module, without changing what the library exposes to users. The change is purely internal refactoring.

AI review queuedUpdate API filesby Mitchell Bagot · 98ecaf82 · Apr 12, 2026 · 3 filesMessage 28 · OpaqueInformational 15Details
Commit message · Mitchell Bagot

Update 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 automatically-generated API snapshot files. It records that two error types in the proof-of-work module moved from one public path to another (e.g., bitcoin_units::pow::ParseTargetError to bitcoin_units::pow::error::ParseTargetError). There is no actual code change, no bug fix, and no security-relevant behavior change in this commit itself.

Lower-prioritypow: Move all error types into error submodule and re-exportby Mitchell Bagot · 73430de2 · Apr 12, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

pow: Move all error types into error submodule and re-export

As part of the move for pow types into units, the error type
definitions were left in the top level of the pow module. Following
existing patterns, they should be defined in the error submodule and
re-exported with #[doc(no_inline)] from the module.

Move ParseTargetError and ParseWorkError definitions from macro into
error submodule.
Adjust Display impl strings to use write_err.

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

This commit is a routine internal code cleanup in the rust-bitcoin library. It moves two error types (ParseTargetError and ParseWorkError) from a macro-generated location into a dedicated error submodule and re-exports them. The change does not alter parsing behavior, security checks, or public API functionality; it only reorganizes where the error types are defined and slightly adjusts their error messages to use a consistent internal helper.

Lower-prioritypow: Move error module re-export to the correct placeby Mitchell Bagot · 9385f8b7 · Apr 12, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Mitchell Bagot

pow: Move error module re-export to the correct place

In the process of moving the pow types to units from bitcoin, the error
module re-export accidentally wound up positioned after the definitions
of the Target and Work types, rather than at the top just below the
type imports.

Move the error submodule re-exports to the top of the file under the
private type use statements.

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

This commit is a purely cosmetic code cleanup. It moves a public re-export statement to the top of a source file so the file is better organized. There is no functional change, no bug fix, and no security relevance.