RB
← All projectsRust Bitcoin

rust-bitcoin

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

BitcoinCryptographic librariesNormal
Repository coverage

2084 commits in the local evidence base

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

461security candidates452second-pass queue2076AI analyses
183commits · 30 days
396commits · 60 days
1294commits · 180 days
2064commits · 365 days
Backfill bands
Aug 5 → Feb 6787 seen32 candidatesComplete
Feb 6 → Jun 6878 seen53 candidatesComplete
Jun 6 → Jul 6211 seen15 candidatesComplete
Jul 6 → Aug 5184 seen2 candidatesComplete
Commit communication

Does the history explain itself?

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

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

Who is changing the project?

Public Git author strings; identities are not independently verified.

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

Published AI watches

Last scanned 30 minutes ago

Informational 20 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6678: units: Add `Weight::to_vb_*` functions, deprecating `to_vbytes_*`

This is a routine API cleanup in a Rust Bitcoin library. It adds new method names (to_vb_floor, to_vb_ceil) for converting transaction 'weight' to 'virtual bytes' and marks the old names (to_vbytes_floor, to_vbytes_ceil) as deprecated. The…

7f614781by Andrew Poelstra+30−86 files
No security note in commit
Informational 15 AI analysisMessage 91 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6597: Release tracking PR: `crypto 0.3.0`

This commit is a routine release-tracking merge. It only bumps version numbers (bitcoin-crypto to 0.3.0 and bitcoin-primitives to 0.103.1), updates dependency version requirements, refreshes lock files, and adds changelog entries. There ar…

1fddd49eby Andrew Poelstra+34−1210 files
No security note in commit
Informational 15 AI analysisMessage 28 · Opaque
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Update API files

This commit only updates generated API listing files (text snapshots of public functions) to reflect a newly added public function, `to_consensus_u32`, on the `CompactTarget` type. It does not change any source code, logic, or behavior, an…

a75c53c8by Mitchell Bagot+6−03 files
No security note in commit
Informational 15 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

Merge rust-bitcoin/rust-bitcoin#6675: units: Remove unnecessary track_caller

This commit removes two Rust compiler hints (`#[track_caller]`) from helper functions that simply return a value or call a user-provided fallback. These functions never panic, so the hints were unnecessary and had no security effect. The c…

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

Merge rust-bitcoin/rust-bitcoin#6644: primitives: Add serde impls for `WitnessVersion`

This commit adds standard serialization/deserialization support for the WitnessVersion type when the optional serde feature is enabled. It is a routine feature addition with no security relevance visible in the code or commit message.

4981faa7by Andrew Poelstra+50−02 files
No security note in commit
Informational 15 AI analysisMessage 100 · Strong
RB Rust Bitcoinrust-bitcoin BitcoinCryptographic libraries

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

units: reword docs to block count in relative locktime

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

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

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

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

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

ci: fix labeller permissions

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Update the API text files

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

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

Scrub the transaction encoding logic

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

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

Make the WitnessesEncoder private

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

b37accc3by Tobin C. Harding+54−542 files
No security note in commit
Repository ledger

Explore captured commits

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

Lower-priorityunits: Extend test coverage for Sequenceby Mitchell Bagot · acdc3b6f · Dec 16, 2025 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · Mitchell Bagot

units: Extend test coverage for Sequence

The encoding/decoding functionality and from_height constructors of the
Sequence type are not covered in existing tests.

Add tests to cover SequenceEncoder, SequenceDecoder and the from_height
constructor of Sequence.

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

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

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

bip158: Remove doc_auto_cfg

Quick before someone has a seizure.

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

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

Lower-priorityAdd re-export for primitives to Bitcoin crateby yancy · c97d458d · Dec 15, 2025 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · yancy

Add re-export for primitives to Bitcoin crate

Enable use of `primitives` from bitcoin, eg: `use bitcoin::primitives`

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

This commit simply makes an existing internal crate called `primitives` usable directly through the main `bitcoin` crate (e.g., `use bitcoin::primitives`). It is a routine API/export convenience change with no security relevance visible in the code.

Lower-priorityExclude deprecated functions from mutation testingby Jamil Lambert, PhD · b98ffefa · Dec 15, 2025 · 1 fileMessage 75 · AdequateInformational 15Details
Commit message · Jamil Lambert, PhD

Exclude deprecated functions from mutation testing

There are new mutants from two deprecated functions in the units module.

Exclude them from mutation testing.

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

This commit only updates the project's mutation-testing configuration file. It tells the mutation-testing tool to skip two more deprecated functions when looking for untested code changes. There is no change to the actual Bitcoin library code, no bug fix, and no security relevance.

Lower-priorityAdd a test of Witness::size()by Jamil Lambert, PhD · 3ee87b04 · Dec 15, 2025 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Jamil Lambert, PhD

Add a test of Witness::size()

There are new mutants in the Witness::size() function.

Add a test to improve coverage and kill the mutants.

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

This commit only adds a new unit test that checks whether a Witness object's reported byte size matches the length of its actual encoded bytes. It does not change any production code, fix any bug, or alter behavior. There is no security relevance.

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

Depend on primitives instead of bitcoin where type definitions overlap

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

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

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

Lower-priorityhashes: Remove uneeded wildcard importby Tobin C. Harding · 0a975763 · Dec 15, 2025 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · Tobin C. Harding

hashes: Remove uneeded wildcard import

We don't need this atm, found by clippy while updating nightly
toolchain.

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

This commit removes an unused wildcard import in a test module. It is a minor code cleanup with no security relevance.

Security candidateRun the formatterby Tobin C. Harding · 1d8649da · Dec 15, 2025 · 58 filesMessage 63 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Run the formatter

Run `just fmt` then add `;` to two lines where the formatter changes
`{ some statement }` into

```
{
some statement
}
```

The the linter warns because no trailing `;`.

63/100 · AdequateMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
cryptography-sensitive pathsigning or wallet pathparser or protocol path
AI analysis · Informational 15/100

This commit is purely a code-formatting cleanup. It runs the project's formatter across 58 files, adjusting whitespace, line breaks, import order, and adding two missing semicolons the formatter exposed. There are no functional code changes, no bug fixes, and no security-relevant alterations.

Lower-priorityAdd comment explaining push_slice()'s minimalityby Nadav Ivgi · 24d7a39d · Dec 14, 2025 · 2 filesMessage 68 · AdequateInformational 15Details
Commit message · Nadav Ivgi

Add comment explaining push_slice()'s minimality

Following https://github.com/rust-bitcoin/rust-bitcoin/pull/5385#issuecomment-3637548949

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 15/100

This commit only adds documentation comments to two Rust source files. It does not change any executable code, so it cannot introduce or fix a security vulnerability by itself. The comments explain an existing behavior of the push_slice() function regarding how small numeric values are encoded in Bitcoin scripts.

Lower-priorityFix ScriptBuf::push_slice() handling of 0x00by Nadav Ivgi · cf8c1a54 · Dec 14, 2025 · 2 filesMessage 60 · AdequateModerate 63Details
Commit message · Nadav Ivgi

Fix ScriptBuf::push_slice() handling of 0x00

And delegate Builder::push_slice() to use it.

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

This commit fixes how the library builds Bitcoin script instructions when pushing a single zero byte (0x00) onto the stack. Previously, ScriptBuf::push_slice() emitted an invalid opcode (OP_PUSHBYTES_0, which does not exist in Bitcoin) for a 0x00 byte, while Builder::push_slice() had its own separate logic that already handled 0x00 correctly by falling through to a normal non-minimal push. The patch makes Builder delegate to ScriptBuf so both behave the same way, and changes ScriptBuf to emit a valid push instruction for 0x00. A script containing the invalid opcode would be rejected by the Bitcoin network, so this is a correctness/security bug in transaction/script construction.

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

Fix push_slice() tests

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

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

Lower-priorityFix script::Builder::push_slice() handling of 0x00by Nadav Ivgi · 6a662540 · Dec 14, 2025 · 2 filesMessage 73 · AdequateModerate 61Details
Commit message · Nadav Ivgi

Fix script::Builder::push_slice() handling of 0x00

A push for `0x00` (a single byte) is different from a push for the
number 0 (an empty byte array), not a different non-minimally-encoded
representation of the same thing.

Prior to this fix, pushing `0x00` would actually push an empty byte
array instead.

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

This commit fixes a bug in the rust-bitcoin library where calling a function to push the single byte 0x00 onto a Bitcoin script would incorrectly push an empty byte array instead. In Bitcoin scripts, pushing the number 0 (empty) and pushing the byte 0x00 are semantically different operations. The fix ensures 0x00 is pushed as a one-byte data push, while the number 0 remains represented as an empty push (OP_0).

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

Extend test coverage for BlockTime

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

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

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

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

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

p2p: Remove `OutPointExt` from `p2p` tests

Remove a pointless dependency on `bitcoin` WRT #5331

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

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

Security candidatechore: Implement Copyby Christian Lewe · 397aeb0c · Dec 13, 2025 · 1 fileMessage 55 · ThinInformational 18Details
Commit message · Christian Lewe

chore: Implement Copy

These structs only contain integers or references, so they should be
Copy.

55/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 18/100

This is a routine Rust API improvement that lets two small data types be copied implicitly instead of requiring an explicit clone. It does not fix a security bug and does not introduce an obvious vulnerability.

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

primitives: Remove arrayvec dependency

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

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

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

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

Lower-priorityinternals: Add api filesby Mitchell Bagot · 4b109201 · Dec 13, 2025 · 4 filesMessage 51 · ThinInformational 15Details
Commit message · Mitchell Bagot

internals: Add api files

Currently only stable crates have API files. Since some features may
cross crate boundaries, and involve breaking changes to the API of
crates under the stable crates, it's important to track the API of
these crates also.

Add internals to check-api just function and add initial API files.

51/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit adds text files that record the current public API of the 'bitcoin_internals' crate and updates a shell script to include that crate in routine API-change checks. It does not change any source code, behavior, or fix any bug. It is purely a documentation/CI tracking change.

Lower-priorityprimitives: Remove Witness::from_parts__unstableby Tobin C. Harding · 9b261deb · Dec 13, 2025 · 1 fileMessage 70 · AdequateInformational 15Details
Commit message · Tobin C. Harding

primitives: Remove Witness::from_parts__unstable

This was added solely so that we could call it in the consensus
decoding function in `bitcoin`. That call site no longer exists,
remove the function.

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

This commit simply removes an internal helper function that was no longer used. It is a routine cleanup change with no security relevance.

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

Implement Decodable for Witness using encoding crate

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

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

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

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
second-pass: broader security terminology
AI analysis · Low 34/100

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

Lower-prioritydocs: extend consensus encoding adr with tech designby Nick Johnson · cc459d98 · Dec 12, 2025 · 2 filesMessage 62 · AdequateInformational 15Details
Commit message · Nick Johnson

docs: extend consensus encoding adr with tech design

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only updates documentation. It expands a design document explaining how a new consensus encoding crate works and removes a TODO comment from another document. No code, tests, build files, or configuration were changed, so it cannot introduce or fix a security vulnerability.

Lower-prioritydocs: fix mismatch between docs and codeby Bashmunta · d9af54dc · Dec 12, 2025 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · Bashmunta

docs: fix mismatch between docs and code

fix output heap size and remove stale line about secp buf size

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This is a tiny documentation-only change in an embedded-systems example README. It corrects the documented heap size from 524288 to 262144 and removes an outdated line about a secp256k1 buffer size. No code was changed, so there is no security impact.

Lower-priorityAutomated update to Github CI to rustc stable-1.92.0by Update Stable Rustc Bot · e1ffc921 · Dec 12, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Update Stable Rustc Bot

Automated update to Github CI to rustc stable-1.92.0

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

This is a routine automated update to the project's GitHub CI configuration, changing the pinned stable Rust compiler version from 1.91.1 to 1.92.0. It does not modify any source code, dependencies, or runtime behavior of the rust-bitcoin library. There is no security relevance in this change.

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

Put feature gate below test attribute

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

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

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

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

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

Security candidateUse roundtrip instead of rinsedby Tobin C. Harding · 50c5867c · Dec 11, 2025 · 7 filesMessage 76 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Use roundtrip instead of rinsed

I never was really happy with the local variable `rinsed` but never
came up with a better one. In #5386 Mitch introduced `roundtrip`
instead, I like that better - use it everywhere.

76/100 · AdequateMessage clarity
✓ 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 15/100

This commit simply renames a local variable from 'rinsed' to 'roundtrip' in several test files. It is a cosmetic code cleanup with no functional changes and no security relevance.

Security candidateRemove redundant readme sections in packagesby Nick Johnson · 8a91a71f · Dec 11, 2025 · 10 filesMessage 68 · AdequateInformational 15Details
Commit message · Nick Johnson

Remove redundant readme sections in packages

The MSRV and License sections in each package are redundant with the
required information in the package manifest. The manifest info is already
shown on sites like crates.io, so not much benefit of listing it twice.
This removes another spot to update on changes.

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

This commit only removes documentation sections from README files in multiple Rust packages. It deletes redundant mentions of the minimum Rust version and licensing information, which are already present in the package manifest files. There are no code changes, no functional changes, and no security implications.