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 queue1033AI analyses
184commits · 30 days
403commits · 60 days
1283commits · 180 days
2062commits · 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 Bagot640192359068
Tobin C. Harding40965156063
jrakibi944963068
Nick Johnson18920112059
Andrew Poelstra512035075
Jamil Lambert, PhD1141849061
Fmt Bot311422045
Trevor Arjeski111111069
Shing Him Ng31724056
Ismail Daif22612050
satsfy (Renato Britto)17612066
Martin Habovstiak27512069
Analysis record

Published AI watches

Last scanned 8 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-prioritytaproot-primitives: Fix hex width specifierby Tobin C. Harding · c15954eb · May 23, 2026 · 1 fileMessage 80 · StrongTriage 0Details
Commit message · Tobin C. Harding

taproot-primitives: Fix hex width specifier

This is printing a `u8`, because we are using alternate (includes
`0x`) the width specifier should be 4 not 2. Found by clippy but
debugged by Poelstra.

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Lower-priorityAdd test for LeafVersion formatting changeby Tobin C. Harding · c41d6e41 · May 23, 2026 · 1 fileMessage 78 · AdequateTriage 0Details
Commit message · Tobin C. Harding

Add test for LeafVersion formatting change

We just patch the `Display` impl for `LeafVersion::Future`. Add a test
that if move before the last patch fails, proving the last patch was
indeed a bug fix.

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Lower-priorityFix ArrayVec::try_push Doc commentby Abeeujah · 49cd6bde · May 22, 2026 · 1 fileMessage 60 · AdequateTriage 0Details
Commit message · Abeeujah

Fix ArrayVec::try_push Doc comment

Correct the Doc comment to rightly reference `CapacityExceeded` instead
of `CAPSizeExceeded`.

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Lower-priorityconsensus_encoding: Release stable 1.0.0by Tobin C. Harding · 74c4f526 · May 22, 2026 · 10 filesMessage 68 · AdequateTriage 0Details
Commit message · Tobin C. Harding

consensus_encoding: Release stable 1.0.0

In preparation for releasing `v1.0.0` add a changelog, bump the
version, update all downstream crates, and update the lockfiles.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Lower-priorityconsensus_enocding: Update the readmeby Tobin C. Harding · 4e4dc435 · May 22, 2026 · 1 fileMessage 58 · ThinTriage 0Details
Commit message · Tobin C. Harding

consensus_enocding: Update the readme

In preparation for the 1.0 release remove mention of the RC release
process and make an effort to document how we got from the old
consensus encoding code to this crate.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
Why it was queued
documentation-only discount
Lower-priorityp2p: change CommandString to wrap 12 byte array instead of Cowby yancy · 9160667c · May 21, 2026 · 1 fileMessage 73 · AdequateTriage 0Details
Commit message · yancy

p2p: change CommandString to wrap 12 byte array instead of Cow

Bitcoin P2P Command strings are always 12-byte ASCII, therefore the
underlying data type can be represented as a fixed 12 byte array. This
removes the need for any type of dynamic allocations using COW.

The `CommandString` invariant that the input must not be longer than 12
bytes and must be ASCII is enforced in all possible construction
avenues. As such, both `try_from_stringly()` and the `end()` method in
the decoder enforce the invariant.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Lower-priorityp2p: mark CommandString as Copyby yancy · 7f4ed346 · May 21, 2026 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · yancy

p2p: mark CommandString as Copy

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityp2p: group struct and impl for V1NetworkMessageby yancy · 59e0ef40 · May 21, 2026 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · yancy

p2p: group struct and impl for V1NetworkMessage

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI review queuedconsensus_encoding: Adjust docs spelling and grammarby Mitchell Bagot · 48fca0f9 · May 21, 2026 · 4 filesMessage 50 · ThinInformational 15Details
Commit message · Mitchell Bagot

consensus_encoding: Adjust docs spelling and grammar

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

This commit only fixes spelling and grammar mistakes in code comments and documentation. No program logic, code behavior, or security properties were changed.

AI review queuedconsensus_encoding: run formatterby Nick Johnson · 183bc246 · May 21, 2026 · 1 fileMessage 35 · OpaqueInformational 15Details
Commit message · Nick Johnson

consensus_encoding: run formatter

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only reformats the import statements in a single test file. No code behavior, logic, or security properties changed.

AI review queuedconsensus_encoding: switch tests to should_panic macroby Nick Johnson · 462c000b · May 21, 2026 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · Nick Johnson

consensus_encoding: switch tests to should_panic macro

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only changes how two existing tests are written. It replaces manual panic-catching code with Rust's standard #[should_panic] test attribute. The tests still verify the same behavior: that decoding functions panic when decoded values do not match expected values. There is no change to production code, no security fix, and no vulnerability introduced.

AI review queuedhashes: Update API filesby jrakibi · bf60526c · May 21, 2026 · 3 filesMessage 28 · OpaqueInformational 15Details
Commit message · jrakibi

hashes: 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 in the rust-bitcoin hashes crate. It removes lines that list certain public type aliases and constants from the recorded public API surface. These files are used to detect unintended API changes during CI, not to implement behavior. There is no code change, no bug fix, and no security-relevant content in the diff.

Security candidatehashes: Remove Bytes and LEN from HashEngineby jrakibi · caf9e7b4 · May 21, 2026 · 14 filesMessage 68 · AdequateInformational 18Details
Commit message · jrakibi

hashes: Remove Bytes and LEN from HashEngine

Simplify the API by removing Bytes and LEN from HashEngine.
Callers can still use HashEngine::Hash::LEN and
HashEngine::Hash::Bytes to access those values.

This will automatically solve the inconsistency between
HashEngine and Hash we found for sha512_256 output size.

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

This is a routine code cleanup in the rust-bitcoin hashes library. It removes duplicate ways of asking a hash engine for the output length and byte type, directing callers to use the associated Hash type instead. The commit message explicitly says this fixes an inconsistency discovered for the sha512_256 output size, but the change itself is an API simplification, not a security patch. There is no evidence of a vulnerability being fixed.

Lower-priorityUse check_encode in consensus comparison fuzz targetby Mitchell Bagot · 591a0246 · May 21, 2026 · 1 fileMessage 83 · StrongInformational 15Details
Commit message · Mitchell Bagot

Use check_encode in consensus comparison fuzz target

The consensus encoding comparison fuzz target compares old encodings
to new encodings between master and bitcoin 0.32. Presently, it does
so by allocating a vector for the old serialisation and the new.
The new check_encode function in consensus_encoding can instead be used
to eliminate a Vec allocation and directly compare the encoder output
of the new encoding against the encoded vector of the old encoding.

Replace encode_to_vec and assert_eq! call with use of the check_encode
function.

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

This commit is a small internal cleanup of a fuzz test (a software self-test) in the rust-bitcoin project. It swaps one helper function for another so the test avoids an unnecessary memory allocation. There is no change to the library code that real users rely on, and nothing in the commit suggests a security vulnerability was fixed.

AI review queuedUpdate APIby Nick Johnson · 0dfa7725 · May 20, 2026 · 3 filesMessage 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 only updates three public API snapshot files to list two newly added test helper functions, `check_decode` and `check_decoder`. These are additions to the library's test/verification tooling, not changes to production code, and there is no indication of any security issue.

AI review queuedconsensus_encoding: add checked decode functions for testsby Nick Johnson · 5fc8699e · May 20, 2026 · 6 filesMessage 60 · AdequateInformational 15Details
Commit message · Nick Johnson

consensus_encoding: add checked decode functions for tests

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only adds new helper functions for unit tests and refactors existing tests to use them. It does not change any production behavior, fix a bug, or alter security-relevant code paths. There is no indication this is a security patch.

Lower-priorityAdjust docs for Header importby Mitchell Bagot · 23b3e2d2 · May 20, 2026 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · Mitchell Bagot

Adjust docs for Header import

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityp2p: Change CommandStringDecoder to newtype decoderby Mitchell Bagot · 2e1318ea · May 20, 2026 · 1 fileMessage 73 · AdequateTriage 0Details
Commit message · Mitchell Bagot

p2p: Change CommandStringDecoder to newtype decoder

The CommandStringDecoder is a newtype-style decoder, implemented as a
complete struct with an inner field. This can trivially be converted to
a newtype struct and thus defined using the decoder_newtype macro.

Change CommandStringDecoder to follow the tuple struct format.
Change CommandStringDecoder definition to use decoder_newtype macro.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Lower-priorityp2p: Replace decoder definitions with decoder_newtype macroby Mitchell Bagot · bdbfce28 · May 20, 2026 · 12 filesMessage 73 · AdequateTriage 0Details
Commit message · Mitchell Bagot

p2p: Replace decoder definitions with decoder_newtype macro

Introduce decoder_newtype macro in p2p through the include! macro.
Replace most newtype Decoder structs in p2p with decoder_newtype calls.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Security candidateMove PushBytes and co to primitivesby Mitchell Bagot · 58965872 · May 20, 2026 · 12 filesMessage 68 · AdequateInformational 18Details
Commit message · Mitchell Bagot

Move PushBytes and co to primitives

To simplify the move of Address to the new addresses crate, the
PushBytes type can be moved to primitives first.

Move PushBytes, PushBytesBuf, PushBytesError and PushBytesErrorReport
to primitives.

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

This commit is a routine internal code reorganization. It moves the PushBytes family of types and helpers from the main bitcoin crate into a lower-level primitives crate, and adjusts re-exports and trait implementations accordingly. There is no indication of a security bug being fixed or introduced.

Lower-priorityprimitives: Add tests to cover PushBytes typesby Mitchell Bagot · b08a9a7a · May 20, 2026 · 1 fileMessage 78 · AdequateTriage 0Details
Commit message · Mitchell Bagot

primitives: Add tests to cover PushBytes types

Having moved to primitives, the tests must cover all mutants in
PushBytes. Since existing tests from bitcoin all rely on extension
traits or other functionality, new tests should be created just for
PushBytes and PushBytesBuf.

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
AI review queuedUpdate API filesby Mitchell Bagot · f30cff4b · May 20, 2026 · 2 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 two generated API snapshot files that list what functions and types are publicly exposed by the library. No actual source code was changed, and the commit message simply says 'Update API files.' There is no indication of a security fix or vulnerability.

Security candidateFix lint errors in primitives and cryptoby Mitchell Bagot · dd3aaf61 · May 20, 2026 · 2 filesMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Fix lint errors in primitives and crypto

With the move of PushBytes and co to primitives, a handful of lint
errors are now thrown in both crypto and primitives.

Fix lint errors in primitives and crypto.

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 is a routine code cleanup commit that fixes compiler lint warnings. It removes an unused import, simplifies how a borrowed reference is obtained, makes a large numeric constant more readable with underscores, and adds a missing semicolon. None of these changes affect program behavior or fix any security issue.

Security candidateMove impl_asref_push_bytes macro to includeby Mitchell Bagot · ff4362be · May 20, 2026 · 6 filesMessage 68 · AdequateInformational 15Details
Commit message · Mitchell Bagot

Move impl_asref_push_bytes macro to include

This macro will be used in multiple crates following the move of
PushBytes to primitives. To reduce code duplication, it should be
moved to the include directory and introduced into each crate as
needed using include!.

Move impl_asref_push_bytes to include/asref_pushbytes.rs and include!
in bitcoin.

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 routine code reorganization. It moves a small helper macro that implements conversions for script push bytes from one internal file to a shared include directory, then pulls it back in with no functional changes. There is no security issue here.

Security candidatecrypto: Add primitives depby Mitchell Bagot · 327313f2 · May 20, 2026 · 3 filesMessage 70 · AdequateInformational 15Details
Commit message · Mitchell Bagot

crypto: Add primitives dep

In order to stabilise PushBytes now, the impls of AsRef/Borrow, etc
between PushBytes and the crypto types need to be in a non-stable
crate until the crypto types are stable. While we want primitives to
depend on crypto eventually, for now, the only way to stabilise
PushBytes in primitives is to invert the dependency and put the impls
in crypto.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
cryptography-sensitive path
AI analysis · Informational 15/100

This commit only changes build configuration files (Cargo.toml and lock files) to add a new internal crate dependency. It does not modify any executable code, fix a bug, or address a security issue. It is a routine refactoring step to resolve a dependency direction problem during library development.