RB
← All projectsRust Bitcoin

rust-bitcoin

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

BitcoinCryptographic librariesNormal
Repository coverage

2086 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
389commits · 60 days
1289commits · 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
320Strong · 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 Poelstra602155078
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 43 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.

AI review queuedchore: fix typosby ANtutov · 1ccf331b · Aug 20, 2025 · 2 filesMessage 40 · ThinInformational 15Details
Commit message · ANtutov

chore: fix typos

40/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit fixes two spelling mistakes in code comments and documentation links. One corrects a grammatical phrase about Bitcoin block version bits, and the other fixes a typo in a hyperlink label pointing to Bitcoin Core's source code. There are no code behavior changes.

Security candidatefix: add all optional feature dependenciesby Nick Johnson · d39a8ac4 · Aug 19, 2025 · 1 fileMessage 90 · StrongInformational 15Details
Commit message · Nick Johnson

fix: add all optional feature dependencies

Ensure all optional feature dependencies are tested when creating the
lock files. rust-bitcoin's dependency tree is so lean right now though,
this is currently a no-op.

90/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
boot or update path
AI analysis · Informational 15/100

This is a small change to an internal maintenance script used to generate dependency lock files for testing. It adds the '--all-features' flag so that optional features are included when checking the project against minimal and recent dependency versions. The commit message explicitly says this is currently a no-op because rust-bitcoin has very few optional dependencies. There is no indication this fixes a security vulnerability.

Security candidatefix: build Cargo-minimal.lock with min depsby Nick Johnson · 328937a5 · Aug 19, 2025 · 2 filesMessage 100 · StrongInformational 15Details
Commit message · Nick Johnson

fix: build Cargo-minimal.lock with min deps

Instead of letting minimal versions potentially drift upwards, reset
the Cargo-minimal.lock file each time dependencies are updated. The
new script uses a combination of the `direct-minimal-version` and
`minimal-versions` flags to ensure the minimal version dependency
tree is tested.

100/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
Why it was queued
boot or update path
AI analysis · Informational 15/100

This commit is a routine build-maintenance change. It tightens how the project tests against the oldest allowed versions of its dependencies, and refreshes a checked-in lock file so those oldest versions are actually used in CI. There is no product bug fix, no change to the library code users run, and no security issue being patched.

Lower-priorityfix: upgrade deps for min version checksby Nick Johnson · b75fa0be · Aug 19, 2025 · 7 filesMessage 100 · StrongInformational 15Details
Commit message · Nick Johnson

fix: upgrade deps for min version checks

* Standardizing the minimal versions of `serde_json` and `serde_test` in
the workspace. This allows the unstable "direct-minimal-versions" flag
of cargo to be used since it requires one minimal version in a
workspace. Since these are all dev dependencies (including the fuzz
crate) there shouldn't be any change for external consumers.
* arrayvec needs a small bump due to its version in hex.
* Arbitrary 1.4.0 does not work with the current 1.63.0 MSRV due to some
core::error usage.
* Update serde's contraint to the minimum version which the
workspace has been using due to implicit upgrades. Attempts were first
made to see if 1.0.103 could be used, but at the very least, would
require a fair amount of lint settings to be overridden due to the old
code. 1.0.195 is used because it also irons out an issue where old
versions of serde pulled in old proc_macro2 versions which use a nightly
feature, `proc_macro_span_shrink`, that has been dropped.

100/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 15/100

This commit only updates the minimum required versions of development and optional third-party dependencies (like serde, arbitrary, arrayvec, and serde_json) across workspace Cargo.toml files and the minimal lockfile. It does not change any application code, cryptographic logic, or network handling. The stated purpose is to make the project's automated minimum-version checks work reliably with newer Cargo tooling, not to fix a security vulnerability.

Lower-prioritydocs: Add rustdocs link policy sectionby Tobin C. Harding · 168cf283 · Aug 19, 2025 · 1 fileMessage 88 · StrongInformational 15Details
Commit message · Tobin C. Harding

docs: Add rustdocs link policy section

With comment

https://github.com/rust-bitcoin/rust-bitcoin/issues/4859#issuecomment-3190333850

we decided to favour links at the bottom of a docs section instead of
inline.

Document this policy.

88/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only adds a documentation policy guideline about how to format hyperlink references in Rust source-code comments. It does not change any executable code, cryptographic logic, network handling, or APIs.

Security candidatedocs: Add a bunch of project management docsby Tobin C. Harding · 9d118129 · Aug 19, 2025 · 6 filesMessage 100 · StrongInformational 15Details
Commit message · Tobin C. Harding

docs: Add a bunch of project management docs

Recently I hypothesised that `rust-bitcoin` needed a project manager -
well here I am stepping up to the plate.

Created by going through #550, wip because I'm up to

https://github.com/rust-bitcoin/rust-bitcoin/issues/550#issuecomment-1252572619

Please review at any level, I'm open to doing anything differently as
suggested.

100/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Links an issue, advisory, or supporting reference
Why it was queued
cryptography-sensitive pathdocumentation-only discount
AI analysis · Informational 15/100

This commit only adds and edits project planning documentation (markdown files about crate organization, roadmap, coding policy, keys/crypto, and Taproot). No source code, build scripts, tests, or dependencies were changed. There is no security-relevant behavior introduced or fixed.

Lower-prioritydocs: Rename address fileby Tobin C. Harding · e053538a · Aug 19, 2025 · 2 filesMessage 70 · AdequateInformational 15Details
Commit message · Tobin C. Harding

docs: Rename address file

We have a pluralisation anomoly

- modules use singular (`address`, `key`)
- crates use plural (`addresses`, and maybe `keys`)

Lets use the crate name for the docs file since the document describes
the crate.

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only renames a documentation file from docs/address.md to docs/addresses.md and updates the matching link in the roadmap. No code, logic, or security behavior changed.

AI review queuedbase58: Remove hex dependencyby Tobin C. Harding · 026d245f · Aug 19, 2025 · 4 filesMessage 68 · AdequateInformational 15Details
Commit message · Tobin C. Harding

base58: Remove hex dependency

The `hex` dev dependency is only used in tests, we have a crate for
that lets use it.

Remove `hex-conservative` dev dependency and use `hex_lit`.

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

This commit simply swaps one internal test-only helper crate for another. It changes how test code converts hex strings into byte arrays, with no effect on the actual library users run in production. There is no security issue here.

Lower-priorityRemove coverallsby Tobin C. Harding · ba16f07d · Aug 18, 2025 · 2 filesMessage 51 · ThinInformational 15Details
Commit message · Tobin C. Harding

Remove coveralls

We added this a while back hoping it would improve the repo but
neither myself nor Andrew ever look at the output before ack'ing or
merging and the notification is annoying. I personally have not seen
any dev or reviewer comment on it leading to better test coverage. On
top of that we have a mutation testing bot that runs on the stable
crates.

So, while a good intention, it seems coveralls is not adding any value
to the restore at this stage - remove it.

51/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context✓ Mentions testing or verification! Too few words to establish purpose
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit removes the Coveralls code-coverage reporting workflow from the repository. It deletes the GitHub Actions file that ran test coverage on every push and pull request, and updates the workflow list in the README. There is no change to the actual Bitcoin library code, its tests, or its security behavior.

Lower-priorityio: Enable features in internals crateby Tobin C. Harding · a3c1d86c · Aug 18, 2025 · 1 fileMessage 68 · AdequateLow 27Details
Commit message · Tobin C. Harding

io: Enable features in internals crate

In `io` we use a macro from `internals` that has feature gated code
dependent on "alloc" but currently we are not enabling "alloc". This
is a bug -

Repeat after me: calling macros across crate boundries is an incessant
source of bugs.

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

This commit fixes a build-configuration bug in the `io` crate of rust-bitcoin. The `io` crate uses a macro from another internal crate (`internals`) whose generated code depends on an `alloc` feature, but `io` was not forwarding the `alloc` feature to `internals`. This could cause compilation failures or missing functionality when `io` is used without the default `std` feature but with `alloc`. It is a correctness and reliability fix rather than a direct exploitable vulnerability.

Security candidateStandardize BIP notation to BIP-XXXXby jrakibi · 7664ff36 · Aug 17, 2025 · 44 filesMessage 68 · AdequateInformational 15Details
Commit message · jrakibi

Standardize BIP notation to BIP-XXXX

use consistent BIP-XXXX notation with 4-digit numbers and hyphen
separators in all documentation, comments, and error messages

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

This commit is a pure documentation and comment cleanup. It changes how Bitcoin Improvement Proposal (BIP) numbers are written throughout the codebase from inconsistent styles like 'BIP32', 'BIP 341', and 'BIP-32' to a uniform 'BIP-XXXX' format with four digits and a hyphen. No program logic, code behavior, or security properties were changed.

Security candidatebitcoin: remove `From<Message>` for `TapSighash`by jrakibi · 933f4c8e · Aug 17, 2025 · 4 filesMessage 68 · AdequateLow 34Details
Commit message · jrakibi

bitcoin: remove `From<Message>` for `TapSighash`

The `Message` type is specific to ECDSA and should not be used for Taproot
sighashes. Taproot sighashes are just 32 bytes long.
With these changes, `From<Message>` is only used for legacy and SegWit v0
sighashes, not for Taproot

68/100 · AdequateMessage clarity
✓ 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 34/100

This commit removes a convenience conversion that let developers accidentally treat a Taproot transaction digest as an ECDSA-style message. It is a defensive API cleanup: it makes the library's types more precise so users cannot pass a Taproot sighash through an ECDSA-specific wrapper. There is no direct bug being fixed, but the change prevents a class of future misuse where a Taproot signature could be computed over a wrongly-wrapped hash.

Security candidate2025-08-17 automated rustfmt nightlyby Fmt Bot · dd416ce0 · Aug 17, 2025 · 19 filesMessage 45 · ThinInformational 15Details
Commit message · Fmt Bot

2025-08-17 automated rustfmt nightly

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

This commit is a routine automated code-formatting run using the nightly version of rustfmt. It only changes whitespace, line breaks, import order, and comment alignment across 19 files. No program logic, security behavior, or API semantics were altered.

AI review queueddocs: fix typo "Depenency" -> "Dependency" in READMEby bigbear · 74cbd3a7 · Aug 16, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · bigbear

docs: fix typo "Depenency" -> "Dependency" in README

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 discountsecond-pass: broader security terminology
AI analysis · Informational 15/100

This commit corrects a spelling mistake in a documentation file, changing the heading 'Depenency tree' to 'Dependency tree' in the README. It does not change any code or affect how the software runs.

Lower-priorityfix: update opcode descriptions for OP_2OVER and OP_3DUPby Tomass · 6306696d · Aug 16, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Tomass

fix: update opcode descriptions for OP_2OVER and OP_3DUP

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

This commit fixes two typos/grammar mistakes in plain-text documentation comments describing Bitcoin script opcodes. The actual code behavior is unchanged; only the human-readable descriptions were corrected.

Lower-priorityAutomated update to Github CI to rustc nightly-2025-08-15by Update Nightly Rustc Bot · d3113394 · Aug 16, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Update Nightly Rustc Bot

Automated update to Github CI to rustc nightly-2025-08-15

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

This commit simply updates the version of the Rust nightly compiler used by the project's automated GitHub CI testing from one weekly nightly build to the next. It changes only one line in a configuration file and has no effect on the actual Bitcoin library code that users run.

Lower-priorityAutomated update to Github CI to cargo-semver-checks version-0.43.0by Update cargo-semver-checks Bot · d9f7af9a · Aug 16, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Update cargo-semver-checks Bot

Automated update to Github CI to cargo-semver-checks version-0.43.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 commit simply bumps the version number of a CI linting tool (cargo-semver-checks) used in GitHub Actions from 0.42.0 to 0.43.0. It does not change any library code, runtime behavior, or user-facing functionality. There is no security relevance visible in the commit.

Security candidateFix overflow bug in `Weight` constructorsby Shing Him Ng · aa086a0a · Aug 16, 2025 · 1 fileMessage 73 · AdequateLow 38Details
Commit message · Shing Him Ng

Fix overflow bug in `Weight` constructors

This fixes an overflow bug which occured when these constructors were
used to construct a `Weight` of max value. Since `Weight` is a `u64`
under the hood, the constructors panicked during the attempt to add
to the max weight value

73/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Names security-relevant behavior explicitly
Why it was queued
memory safety
AI analysis · Low 38/100

This commit fixes a panic (crash) in two Bitcoin weight-unit conversion helpers when they are called on the maximum possible weight value. The functions now use saturating addition, so instead of overflowing and crashing they return a large but valid rounded-up result. This is a reliability fix that prevents denial-of-service-style crashes from untrusted inputs.

Lower-priorityUpdate `from_vb_unchecked` to use constantby Shing Him Ng · e11d3ed4 · Aug 16, 2025 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Shing Him Ng

Update `from_vb_unchecked` to use constant

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

This is a tiny code-quality change that replaces a hardcoded number (4) with a named constant already defined in the same file. It does not change behavior, fix a bug, or address any security issue.

Lower-priorityAdd fuzz target for `Weight`by Shing Him Ng · b762f746 · Aug 16, 2025 · 3 filesMessage 55 · ThinInformational 15Details
Commit message · Shing Him Ng

Add fuzz target for `Weight`

55/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 15/100

This commit adds a new automated fuzz test for the Weight type in the rust-bitcoin library. Fuzz tests feed random or semi-random inputs to code to find crashes or unexpected behavior. There is no change to production code, no bug fix, and no security patch in this commit.

Lower-priorityBump version of workflow in fuzz file generationby Shing Him Ng · e815d32e · Aug 16, 2025 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · Shing Him Ng

Bump version of workflow in fuzz file generation

This version was bumped by the bot in commit
489118f0b79fd9e2e78c79db1e5876814e132431 so this keeps it consistent
when the script is run

78/100 · AdequateMessage clarity
✓ 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 updates a single version number in a shell script that auto-generates GitHub Actions workflow files. It changes the referenced GitHub action 'actions/download-artifact' from version 4 to version 5 so newly generated workflows match a version already bumped elsewhere by an automated tool. There is no code change affecting the Rust Bitcoin library, its users, or any runtime behavior.

Lower-prioritywitness: rewrite the docs on several Taproot-related methods and deprecate a methodby Andrew Poelstra · 67e54393 · Aug 15, 2025 · 1 fileMessage 85 · StrongInformational 19Details
Commit message · Andrew Poelstra

witness: rewrite the docs on several Taproot-related methods and deprecate a method

There are multiple issues with these docs:

* They repeatedly say "according to the rules of BIP XYZ". Duh. We should
document when we're *not* following the rules.
* In the Taproot methods, we then say that we're just looking for 0x50 bytes
but not really parsing stuff, which is untrue because we do parse the
witness stack...
* ...and we say that the user should call Script::is_p2tr to see if the
output is really Taproot, which is untrue according to BIP 341...
* ...and we don't even explain that the user needs to call is_p2tr on the
*output script* and not the script being returned from this function.

Furthermore, we have two methods `tapscript` and `taproot_leaf_script`.
We say that the latter "does not return the control block". Neither one
returns the control block. We don't say how these methods are different
(or, for that matter, where to get the control block).

Also inside `tapscript` we have some assinine comment from Kix about how
the method is "broken" because of some distinction between "leaf script"
and "tapscript" that exists only in his head. We don't expose this to
our users. We shouldn't expose it to ourselves either. Clarify what (I
think) he means and promote it to a deprecation notice.

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

This commit is a documentation and deprecation cleanup for Taproot-related witness methods in the rust-bitcoin library. It rewrites misleading doc comments, deprecates the `tapscript` method in favor of `taproot_leaf_script`, and clarifies that callers should validate the output script (not the returned script) when checking for Taproot or Segwit v0 spends. There is no code behavior change beyond adding a Rust deprecation attribute.

Lower-prioritychore: fix typo 'effected' to 'affected'by phrwlk · e5e3b311 · Aug 15, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · phrwlk

chore: fix typo 'effected' to 'affected'

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
AI analysis · Informational 15/100

This commit fixes a spelling mistake in a comment inside a shell script used for checking API changes. It changes 'effected' to 'affected' with no functional code changes.

Lower-priorityMove policy stuff to docs directoryby Tobin C. Harding · d263d94d · Aug 15, 2025 · 2 filesMessage 68 · AdequateInformational 15Details
Commit message · Tobin C. Harding

Move policy stuff to docs directory

We now have a `docs` directory. Move the policy documentation out of
`CONTRIBUTING.md` and into `docs/policy.md`.

This has the benefit of keep the contributing doc shorter so as not to
overwhelm new contributors while allowing the policy doc to evolve and
grow.

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

This commit simply moves the project's coding-style policy guide from CONTRIBUTING.md into a new docs/policy.md file. It is a documentation reorganization with no code changes, no functional changes, and no security relevance.

AI review queuedIntroduce Ntxidby Tobin C. Harding · 3e251c73 · Aug 15, 2025 · 3 filesMessage 41 · ThinInformational 18Details
Commit message · Tobin C. Harding

Introduce Ntxid

Currently the API to calculate a normalized TXID returns a hash type.
We would like to remove `hashes` from the public API of `primitives`
so we can stabilise `primitives` without stabilising `hashes`.

Introduce a `Ntxid` type and return it when computing the normalised
TXID.

41/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context! Too few words to establish purpose
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 18/100

This is a routine API refactor in a Rust Bitcoin library. It introduces a new public type called Ntxid to represent a 'normalized transaction ID' instead of returning a generic hash. The goal is to let the library stabilize one of its core modules without forcing users to depend on another internal module. There is no security bug being fixed here.