RB
← All projectsRust Bitcoin

rust-bitcoin

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

BitcoinCryptographic librariesNormal
Repository coverage

2070 commits in the local evidence base

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

459security candidates448second-pass queue2070AI analyses
181commits · 30 days
395commits · 60 days
1282commits · 180 days
2052commits · 365 days
Backfill bands
Aug 5 → Feb 6787 seen32 candidatesComplete
Feb 6 → Jun 6878 seen53 candidatesComplete
Jun 6 → Jul 6211 seen15 candidatesComplete
Jul 6 → Aug 5184 seen2 candidatesComplete
Commit communication

Does the history explain itself?

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

64/100 average clarity
311Strong · 80–100
1054Adequate · 60–79
555Thin · 40–59
150Opaque · 0–39
20security candidates with opaque commit messaging
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
Mitchell Bagot640192640068
Tobin C. Harding40965409063
jrakibi944994068
Nick Johnson18920189059
Andrew Poelstra512051075
Jamil Lambert, PhD11418114061
Fmt Bot311431045
Trevor Arjeski111111069
Shing Him Ng31731056
Ismail Daif22622050
satsfy (Renato Britto)17617066
Martin Habovstiak27527069
Analysis record

Published AI watches

Last scanned 43 minutes ago

Informational 15 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6671: units: reword docs from height to count in relative locktime

This commit only changes documentation comments and variable names in the rust-bitcoin library. It rewords references to 'block height' as 'block count' for relative locktime, because a relative locktime measures elapsed blocks since a coi…

6ea4ff92by Andrew Poelstra+20−172 files
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6670: ci: fix labeller permissions

This is a routine GitHub Actions workflow fix. The change gives the automated PR labeler workflow permission to download artifacts and explicitly tells the GitHub CLI which repository to act on. It does not change the Bitcoin library code,…

CI workflow permission fixNo source code changesNo cryptographic or consensus changes
276e0a61by Andrew Poelstra+5−01 file
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6657: Release tracking PR: `base58ck 0.5.0`

This commit is a routine version bump for the base58ck crate from 0.4.0 to 0.5.0. It only updates version numbers in package manifests, lock files, and adds a changelog entry. There are no code changes and no security relevance.

9deb5de1by Andrew Poelstra+22−168 files
No security note in commit
Informational 21 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6440: base58: Introduce `decode_check_to_array` for alloc-less decoding

This commit is a routine feature addition to the rust-bitcoin base58 crate. It adds a new no-allocator function to decode short base58-check strings into fixed-size byte arrays, and renames the existing error type while keeping a deprecate…

No security-relevant bug fix is described in the commit message or diff.New decoding path uses a fixed 128-byte scratch buffer (`ArrayVec`) and rejects oversized inputs.Checksum verification and invalid-character handling are preserved from the existing alloc implementation.
c5fd1832by Andrew Poelstra+330−775 files
No security note in commit
Informational 15 AI analysisMessage 93 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6646: hashes: add several methods to `Midstate` and release 1.2.0

This commit is a routine feature release for the bitcoin_hashes crate. It adds new public methods and a constant to the SHA-256 Midstate type so developers can compute SHA-256 midstates in const contexts. There is no indication of a securi…

0f68fcb3by Andrew Poelstra+157−4210 files
No security note in commit
Informational 15 AI analysisMessage 73 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

units: reword docs to block count in relative locktime

This commit only rewords documentation comments and variable names in the codebase. It changes references from 'block height' to 'block count' for relative lock times, which is a terminology correction with no functional code changes. Ther…

8867fe2eby satsfy (Renato Britto)+20−172 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6667: Make the `WitnessesEncoder` private

This commit makes a small internal helper type called WitnessesEncoder private. It was previously exported as public API but is only used inside the transaction encoder. The change removes it from the public API surface and moves the code …

07d91f7fby Andrew Poelstra+54−1324 files
No security note in commit
Informational 12 AI analysisMessage 75 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

ci: fix labeller permissions

This commit adjusts a GitHub Actions workflow that automatically labels pull requests based on CI results. It adds a built-in repository token, tells the artifact download step to merge single-file artifacts, and explicitly sets the reposi…

Workflow-only change with no library code modificationsUses the default GITHUB_TOKEN rather than a custom secretRuns in workflow_run context, which already has elevated repository access by design
715f2bd6by Nick Johnson+5−01 file
No security note in commit
Informational 19 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6661: units: Remove `From<u16>` from `NumberOfBlocks`

This is a routine API cleanup, not a security fix. The developers removed a shortcut that let programmers create a 'number of blocks' value directly from any u16 number, replacing it with an existing named constructor called from_height. T…

ed5f1b68by Andrew Poelstra+18−358 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6668: Scrub the transaction encoding logic

This commit is a pure code reorganization (refactor) in the rust-bitcoin library. It moves transaction encoding and decoding definitions around within a single file so the code follows the project's preferred layout. No logic, behavior, or…

c5588ba5by Andrew Poelstra+57−571 file
No security note in commit
Informational 18 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6662: build(deps): bump actions/checkout from 6.0.2 to 6.0.3

This commit is a routine update to the GitHub Actions checkout tool used by the project's automated workflows. It changes the pinned version from 6.0.2 to 6.0.3 across many workflow files. The newer version fixes minor bugs related to SHA-…

Dependency bump of actions/checkout to a patch releaseUpstream fixes for SHA-256 repository checkout and merge commit regexAll affected workflows already disable persisted credentials
f7d364cdby Andrew Poelstra+29−2915 files
No security note in commit
Informational 15 AI analysisMessage 96 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6663: build(deps): bump taiki-e/install-action from 2.79.12 to 2.81.4

This is a routine Dependabot update that bumps the version of a third-party GitHub Action used only in automated testing workflows. The change does not touch the project's actual Bitcoin library code, and there is no indication it fixes or…

5c92c0b0by Andrew Poelstra+2−22 files
No security note in commit
Informational 15 AI analysisMessage 45 · Thin
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Update the API text files

This commit only updates two generated API text files that list the public surface of the Rust crate. It removes entries for a `WitnessesEncoder` type, reflecting that the type is no longer part of the public API. There is no source code c…

9c391145by Tobin C. Harding+0−782 files
No security note in commit
Informational 15 AI analysisMessage 68 · Adequate
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Scrub the transaction encoding logic

This commit is a pure code reorganization: it moves existing transaction encoding and decoding definitions around within a single file so they follow the project's preferred layout. No logic was changed, no security bug was fixed, and no n…

80dcef15by Tobin C. Harding+57−571 file
No security note in commit
Informational 15 AI analysisMessage 58 · Thin
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Make the WitnessesEncoder private

This commit simply hides an internal helper type called WitnessesEncoder from public view. It is a routine API-cleanup change with no security relevance visible in the code or commit message.

b37accc3by Tobin C. Harding+54−542 files
No security note in commit
Informational 15 AI analysisMessage 45 · Thin
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

release bitcoin_hashes 1.2.0

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

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

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

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

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

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

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

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

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

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

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

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

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

Routine patch-level dependency bumpNo changes to permissions, inputs, or workflow logicpersist-credentials: false already present on all checkout steps
ae935fadby dependabot[bot]+29−2915 files
No security note in commit
Repository ledger

Explore captured commits

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

Lower-priorityconsensus_encoding: add conventinal manifest metadataby Nick Johnson · e001c89d · Feb 21, 2026 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · Nick Johnson

consensus_encoding: add conventinal manifest metadata

Follows C-METADATA from the API guidelines.

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

This commit only adds two standard metadata fields (homepage and documentation URLs) to a Rust package manifest file. It does not change any executable code, cryptographic logic, or network behavior. There is no security relevance.

Lower-priorityconsensus_encoding: fix up doc linkby Nick Johnson · 2098b582 · Feb 21, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

consensus_encoding: fix up doc link

Confrom to C-LINK API conventions.

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

This is a one-word documentation fix. A comment that referenced another type was changed from plain text to a proper Rust doc link so that documentation tools can create a clickable link. There is no code behavior change and no security relevance.

AI review queuedUpdate apiby Nick Johnson · c9cdde0b · Feb 21, 2026 · 1 fileMessage 0 · OpaqueInformational 15Details
Commit message · Nick Johnson

Update api

0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
documentation-only discountsecond-pass: opaque commit message
AI analysis · Informational 15/100

This commit updates a generated API snapshot file to reflect a minor relaxation of trait bounds on an error type. It is a documentation/tracking file change, not a code change, and has no security relevance.

AI review queuedconsensus_encoding: remove redundant trait boundsby Nick Johnson · 3ea57579 · Feb 20, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

consensus_encoding: remove redundant trait bounds

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 is a minor code cleanup in a Rust Bitcoin library. It removes unnecessary requirements (trait bounds) from an error-handling implementation. The change does not affect security, runtime behavior, or how the library processes Bitcoin data.

Lower-priorityFix Unknown NetworkMessage encodingby Mitchell Bagot · cf1a9fee · Feb 20, 2026 · 1 fileMessage 58 · ThinLow 37Details
Commit message · Mitchell Bagot

Fix Unknown NetworkMessage encoding

The NetworkMessageEncoder and old Encodable implementation for
NetworkMessage make use of the Vec<u8> encoding for the raw payload
data. This introduces a vector length prefix to the start of the
encoding which causes the roundtrip for Unknown NetworkMessage types to
fail. In practice, NetworkMessage will only ever be decoded through the
RawNetworkMessage, where a payload length will be known. As such, this
length prefix is not necessary in the encoding and can be trivially
removed.

Change Unknown NetworkMessage encoding to directly write the vector of
bytes instead of using Vec<u8> encoding logic.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
AI analysis · Low 37/100

This commit fixes a bug in how custom or unknown Bitcoin peer-to-peer messages were being serialized. Previously, the code accidentally added an extra length number to the front of the raw payload bytes, so if you decoded an unknown message and re-encoded it, the bytes would not match. The fix writes the raw payload bytes directly without the extra length prefix, and adds a test to confirm round-trip encoding works.

Security candidateEncapsulate PublicKey and PrivateKeyby Mitchell Bagot · 118b8987 · Feb 20, 2026 · 1 fileMessage 58 · ThinInformational 17Details
Commit message · Mitchell Bagot

Encapsulate PublicKey and PrivateKey

With the previous changes to the PublicKey and PrivateKey types, both
can now be encapsulated to control the scope of access to the inner
fields.

Move PublicKey and PrivateKey to encapsulated module and add them to
the pub use.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive path
AI analysis · Informational 17/100

This commit is a routine internal code reorganization. It moves the PublicKey and PrivateKey type definitions into a private 'encapsulate' module and re-exports them publicly. The fields were already private before this change, so no new access restrictions are introduced. There is no indication this fixes a security bug or changes behavior visible to users of the library.

Security candidateRemove public access to all fields of PublicKey and PrivateKeyby Mitchell Bagot · 41d946ff · Feb 20, 2026 · 3 filesMessage 73 · AdequateInformational 18Details
Commit message · Mitchell Bagot

Remove public access to all fields of PublicKey and PrivateKey

As with the inner fields, the compressed and network fields of PublicKey
and PrivateKey allows public access for read/write. To allow for
encapsulation, read access should be limited to an accessor function.

Introduce compressed() accessor function for PublicKey and compressed()
and network() accessor functions for PrivateKey. Replace all direct
access to inner fields with use of the accessor functions.

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

This commit hides the internal fields of two key types (PublicKey and PrivateKey) so outside code can no longer read or change them directly. Instead, it provides read-only accessor methods. This is a normal defensive-coding change that improves encapsulation and makes future misuse less likely, but the commit itself does not fix a known active bug or vulnerability.

Security candidateRemove public access to inner field of PublicKey and PrivateKeyby Mitchell Bagot · 9506d45c · Feb 20, 2026 · 6 filesMessage 73 · AdequateLow 35Details
Commit message · Mitchell Bagot

Remove public access to inner field of PublicKey and PrivateKey

While all construction of PublicKey and PublicKey now goes through the
from_secp constructors, read access to the inner fields are still
commonplace.

Introduce as_inner function for PrivateKey and to_inner for PublicKey
and remove all direct access to the inner fields.

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

This commit hides the internal cryptographic key fields of PublicKey and PrivateKey from outside code, replacing direct access with controlled getter methods. It is a defensive hardening change that reduces the chance future callers will accidentally misuse or mutate the raw secp256k1 keys, but the commit itself does not fix a known exploitable bug.

Security candidateIntroduce from_secp constructors for PublicKey and PrivateKeyby Mitchell Bagot · 2f74f3ea · Feb 20, 2026 · 8 filesMessage 73 · AdequateInformational 18Details
Commit message · Mitchell Bagot

Introduce from_secp constructors for PublicKey and PrivateKey

The PublicKey and PrivateKey types rely on public access to the inner
fields in many places to instantiate the type. Since we wish to
encapsulate the type, this inner access should instead be wrapped
through constructors.

Introduce from_secp, and from_secp_uncompressed constructors and
replace all instantiation of PublicKey and PrivateKey with calls to
these.

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

This is a routine code cleanup in the rust-bitcoin library. It introduces new constructor functions named from_secp for creating public and private keys, and replaces direct internal field access with calls to those constructors. The old constructors are kept but marked as deprecated. There is no security bug being fixed here; the change is about improving code structure and preparing to hide internal details in the future.

Lower-priorityprimitives: Unit test hex featureby Tobin C. Harding · 771ebf2d · Feb 20, 2026 · 1 fileMessage 86 · StrongInformational 15Details
Commit message · Tobin C. Harding

primitives: Unit test hex feature

When I added the `hex` feature (#5213) I forgot to tie it into CI.

Test the feature both with and without `std` enabled.

86/100 · StrongMessage clarity
✓ 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 · Informational 15/100

This commit only changes a CI configuration file to add the 'hex' feature to automated testing matrices. It does not modify any runtime code, APIs, or dependencies that could affect security. There is no security issue or fix here.

AI review queuedUpdate the API filesby jrakibi · d65b496d · Feb 19, 2026 · 3 filesMessage 28 · OpaqueInformational 15Details
Commit message · jrakibi

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 generated API snapshot files (text listings of public types and methods). It does not change any source code, behavior, or fix any bug. The added lines reflect new methods like non_secure_erase and Drop that already exist in the actual library code; the removed line shows Hkdf no longer implements Copy. These are documentation/snapshot updates, not a security patch.

Security candidatehashes: add non_secure_erase for HMAC and HKDFby jrakibi · 5c844315 · Feb 19, 2026 · 3 filesMessage 68 · AdequateLow 30Details
Commit message · jrakibi

hashes: add non_secure_erase for HMAC and HKDF

HKDF and HMAC hold secret data (prk, ipad/opad). we add
non_secure_erase to overwrite it with zeros (same approach as
rust-secp256k1 non_secure_erase)

this is a best-effort to erase secret data, it does not guarantee
that material never remains in memory. the compiler may still copy or move
data to other places in memory.

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

This commit adds a best-effort memory-clearing helper for two cryptographic building blocks (HMAC and HKDF) in the rust-bitcoin library. It tries to overwrite secret intermediate data with zeros when those objects are destroyed, but the authors explicitly note this is not a guaranteed security fix because the Rust compiler may still leave copies of the data elsewhere in memory. The change is defensive hardening rather than a fix for a known exploitable bug.

Lower-priorityunits: add optional encoding dep to feature flag test matrixby Nick Johnson · 574f7e9e · Feb 19, 2026 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · Nick Johnson

units: add optional encoding dep to feature flag test matrix

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only changes a test configuration file (rbmt.toml) to include an optional 'encoding' feature in the automated feature-flag testing matrix. It does not modify any production code, APIs, or security-sensitive logic. There is no security issue here.

Lower-priorityUse encoding:: prefix on encoding traitby Tobin C. Harding · bca1bb95 · Feb 19, 2026 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Use encoding:: prefix on encoding trait

It makes it a bit more clear that the trait is from another crate if
we use one level of path. Also this seems to be consistent with other
usage repo wide lately for `encoding` stuff at least.

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

This is a trivial code-style change. A developer replaced a direct import of the Encodable trait with its full path (encoding::Encodable) to make it clearer that the trait comes from another crate. No behavior, logic, or security properties changed.

Lower-priorityBlockHash: Move Encodable and Decodable trait implsby Tobin C. Harding · 84c51a8e · Feb 19, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Tobin C. Harding

BlockHash: Move Encodable and Decodable trait impls

Put the trait impls first. They are the most important thing in
regards to the encoding/decoding logic, put them first and the
supporting types and code next.

Code move only, no logic change.

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

This commit simply reorders two blocks of code in a single Rust source file. It moves the implementation of the Decodable trait for BlockHash to appear earlier in the file, before the definition of its helper decoder type. The commit message and the diff confirm there is no change to program logic, behavior, or security.

Lower-priorityBlockHash: Inline all the consensus encoding stuffby Tobin C. Harding · 5bc28cee · Feb 19, 2026 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · Tobin C. Harding

BlockHash: Inline all the consensus encoding stuff

Add `inline` attribute to everything except the error code.

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

This commit only adds Rust compiler hints (`#[inline]`) to a few small functions related to encoding and decoding block hashes. It does not change what the code does, only gives the compiler permission to embed these tiny functions directly at call sites for potential performance gains. There is no security relevance.

Lower-priorityFix display msg for BlockHashDecoderErrorby Tobin C. Harding · 63fce3f2 · Feb 19, 2026 · 1 fileMessage 60 · AdequateInformational 16Details
Commit message · Tobin C. Harding

Fix display msg for BlockHashDecoderError

Looks like a cut'n'paste error; this is the block hash decoder not the
sequence decoder.

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

This commit fixes a typo in an error message. The error message for a block hash decoding failure incorrectly said 'sequence decoder error' instead of 'block hash decoder error'. It is a cosmetic string correction with no security or functional impact.

Lower-prioritytest: add roundtrip test for encoding and decoding an Addressby yancy · 4218190d · Feb 19, 2026 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · yancy

test: add roundtrip test for encoding and decoding an Address

72/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 15/100

This commit adds a new automated test that checks an Address object can be encoded to bytes and then decoded back to the same object. It does not change any production code, fix any bug, or alter behavior visible to users.

Lower-priorityp2p: implement encoding for Addressby yancy · 3c19767f · Feb 19, 2026 · 1 fileMessage 86 · StrongInformational 17Details
Commit message · yancy

p2p: implement encoding for Address

port and address are both converted to an array type &[u8] as required
by `ArrayEncoder` using `to_be_bytes()`. Big endian byte order is
specified in:
https://developer.bitcoin.org/reference/p2p_networking.html

Since address is an array of `[u16; 8]`, each respective `u16` is split
into a `[u8; 2]`, then both elements are filled into the resulting
`[u8; 32]`.

In the test cases, replace `serialize` with `encoding::encode_to_vec`
since the push encoding here is planned to replace the existing
`serialize` implementation.

86/100 · StrongMessage clarity
✓ 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 · Informational 17/100

This commit adds a new way to serialize (convert to bytes) Bitcoin peer-to-peer network addresses in the rust-bitcoin library. It does not appear to fix or introduce a security vulnerability; it is a routine implementation change replacing one serialization method with another while preserving the documented big-endian byte order.

AI review queuedp2p: implement decoding for Addressby yancy · ca7e9259 · Feb 19, 2026 · 1 fileMessage 86 · StrongInformational 16Details
Commit message · yancy

p2p: implement decoding for Address

Endian byte order for both IPv4 and the port number are interpreted as
big endian per the Addr section of:
https://developer.bitcoin.org/reference/p2p_networking.html

In the test cases, `deserialize` is swapped for
`encoding::decode_from_slice` since the addition of push bytes decoding
is planned to replace the existing `deserialize` implementation.

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

This commit adds a new way to decode Bitcoin peer-to-peer network addresses in the rust-bitcoin library. It is a feature/refactoring change that introduces a new decoder implementation and swaps test cases to use it. There is no indication in the commit that it fixes a security bug, vulnerability, or crash.

Lower-priorityRemove pre-cargo-rbmt settings fileby Nick Johnson · c2045c1b · Feb 18, 2026 · 2 filesMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

Remove pre-cargo-rbmt settings file

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

This commit simply deletes two helper shell scripts from the contrib/ directory that were used by the project's internal CI tooling. There is no change to the actual Rust Bitcoin library code, no change to how the software behaves at runtime, and no security relevance.

Lower-priorityClean up accidentally checked in filesby Nick Johnson · 0604e091 · Feb 18, 2026 · 2 filesMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

Clean up accidentally checked in files

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

This commit simply deletes two compiled Rust library files (.rlib) that were accidentally added to the repository. There is no code change, no bug fix, and no security issue in the commit itself.

Security candidateSwitch update lock files to cargo-rbmtby Nick Johnson · 77a8454f · Feb 18, 2026 · 2 filesMessage 45 · ThinInformational 15Details
Commit message · Nick Johnson

Switch update lock files to cargo-rbmt

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
boot or update path
AI analysis · Informational 15/100

This commit is a routine build-maintenance change. It removes an old shell script used to update dependency lock files and switches the project to using a new internal tool called cargo-rbmt for the same task. There is no change to the actual Bitcoin library code, no security fix, and no vulnerability introduced.

AI review queuedconsensus: Add CompactSize range check to deserializationby Erick Cestari · 1dd1d92f · Feb 18, 2026 · 4 filesMessage 83 · StrongModerate 61Details
Commit message · Erick Cestari

consensus: Add CompactSize range check to deserialization

Reject CompactSize values exceeding MAX_COMPACT_SIZE (0x02000000) during
deserialization, matching Bitcoin Core's serialize.h limit.

Extract read_compact_size logic into a private free function to allow
tests to bypass the range check when validating non-minimal encoding.

Add test for the new SizeTooLargeCompactSize error at the boundary.

Update bip152 error test to expect failure at deserialization rather
than at index computation, since the range check now triggers earlier.

83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
second-pass: broader security terminology
AI analysis · Moderate 61/100

This commit adds a safety limit when reading Bitcoin's 'CompactSize' numbers from serialized data. Before, a malformed message could claim an absurdly large number of elements (for example, billions of items in a list), which could be used to exhaust memory or trigger a panic. Now values above Bitcoin Core's documented maximum are rejected early during parsing. The change also refactors the parsing code so tests can still check non-minimal encoding without the new limit.

AI review queuedhashes: simplify trait boundsby Conger Rassen · c423b93a · Feb 18, 2026 · 1 fileMessage 35 · OpaqueInformational 15Details
Commit message · Conger Rassen

hashes: simplify trait bounds

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 is a tiny code cleanup change in the rust-bitcoin hashes library. It replaces a more verbose way of referring to a constant (<H as HashEngine>::BLOCK_SIZE) with a simpler equivalent (H::BLOCK_SIZE). The behavior of the program is unchanged, and there is no security relevance.